Skip to content

fix: mcp enable command - #3290

Open
JasonYeMSFT (JasonYeMSFT) wants to merge 2 commits into
microsoft:mainfrom
JasonYeMSFT:chuye/fix-mcp-commands
Open

JasonYeMSFT (JasonYeMSFT) wants to merge 2 commits into
microsoft:mainfrom
JasonYeMSFT:chuye/fix-mcp-commands

Conversation

@JasonYeMSFT

Copy link
Copy Markdown
Member

Description

Remove the non-existent /azure:setup command and clarify when to use /mcp command for enabling Azure MCP.

Checklist

  • Tests pass locally (cd tests && npm test)
  • Title has one of the prefixes: fix:, feat:, feature:, chore:, misc:, test:, eval:
  • If modifying skill descriptions: verified routing correctness with integration tests (In tests/, npm run test:vally -- --plugin <plugin-dirname> --skill <skill>)

Related Issues

Resolves #3284

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Both instructions still direct the agent to invoke an interactive slash command rather than asking the user to run it.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Removes the nonexistent /azure:setup guidance identified in #3284 and clarifies Azure MCP enablement.

Changes:

  • Replaces invalid setup commands with /mcp guidance.
  • Identifies supported host environments.
File Description
plugins/​azure-skills/​skills/​azure-storage/​SKILL.md Updates Azure Storage MCP setup guidance.
plugins/​azure-skills/​skills/​azure-ai/​SKILL.md Updates Azure AI MCP setup guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread plugins/azure-skills/skills/azure-ai/SKILL.md Outdated
Comment thread plugins/azure-skills/skills/azure-storage/SKILL.md Outdated
Updated instructions for enabling Azure MCP in Copilot CLI and other hosts. Added commands for speech and storage functionalities.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: azure-ai and azure-storage references incorrect commands

3 participants