Remove foundry-mcp from plugin/.mcp.json (proxied through azure MCP)#1231
Merged
JasonYeMSFT merged 3 commits intomainfrom Mar 12, 2026
Merged
Remove foundry-mcp from plugin/.mcp.json (proxied through azure MCP)#1231JasonYeMSFT merged 3 commits intomainfrom
JasonYeMSFT merged 3 commits intomainfrom
Conversation
Co-authored-by: JasonYeMSFT <39359541+JasonYeMSFT@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Remove foundry-mcp from plugin/.mcp.json
Remove foundry-mcp from plugin/.mcp.json (proxied through azure MCP)
Mar 11, 2026
Member
|
@copilot Bump the skill version in microsoft-foundry/SKILL.md |
Co-authored-by: JasonYeMSFT <39359541+JasonYeMSFT@users.noreply.github.com>
Contributor
Author
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Microsoft Foundry “observe” skill and its unit test to refer to the azure MCP server name (instead of foundry-mcp), and removes the foundry-mcp server entry from the plugin MCP configuration.
Changes:
- Update
observe.mdQuick Reference to useMCP server | \azure`` and adjust related wording. - Update the observe unit test to assert the new server name.
- Remove the
foundry-mcpMCP server entry fromplugin/.mcp.json.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tests/microsoft-foundry/foundry-agent/observe/unit.test.ts | Updates unit test expectation for the observe skill’s MCP server naming. |
| plugin/skills/microsoft-foundry/foundry-agent/observe/observe.md | Updates documentation to reference the azure MCP server in the Quick Reference and introduction. |
| plugin/.mcp.json | Removes the foundry-mcp MCP server configuration entry, leaving azure as the configured server. |
JasonYeMSFT
temporarily deployed
to
cideploytest
GitHub Actions
Inactive
Member
|
Test run from this branch: https://github.com/microsoft/GitHub-Copilot-for-Azure/actions/runs/22976960268 |
This was referenced Mar 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
foundry-mcpis now redundant — Foundry tools are proxied through the unified@azure/mcpserver already present in.mcp.json.Changes
plugin/.mcp.json— Remove thefoundry-mcpHTTP entry (https://mcp.ai.azure.com)plugin/skills/microsoft-foundry/foundry-agent/observe/observe.md— Update Quick Reference table and description to referenceazureMCP server instead offoundry-mcptests/microsoft-foundry/foundry-agent/observe/unit.test.ts— Update assertion to check forazureinstead offoundry-mcpin observe content📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.