Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the ToolingManifest.json file to align with Agent 365 CLI documentation by correcting MCP server names. The changes address issues where certain MCP server names were incorrect or duplicated, causing problems with OBO permissions and server mapping.
Changes:
- Removed "mcp_NLWeb" server (duplicate with incorrect OBO permissions)
- Renamed "OneDriveMCPServer" to "mcp_ODSPRemoteServer" (correct server name per a365 CLI)
- Added "mcp_WordServer"
- Improved JSON formatting (tabs to spaces)
Comment on lines
0
to
+22
| { | ||
| "mcpServers": [ | ||
| { | ||
| "mcpServerName": "mcp_MailTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_CalendarTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "OneDriveMCPServer" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_NLWeb" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_KnowledgeTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_MeServer" | ||
| } | ||
| ] | ||
| } No newline at end of file | ||
| "mcpServers": [ | ||
| { | ||
| "mcpServerName": "mcp_MailTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_CalendarTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_ODSPRemoteServer" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_KnowledgeTools" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_MeServer" | ||
| }, | ||
| { | ||
| "mcpServerName": "mcp_WordServer" | ||
| } | ||
| ] | ||
| } |
There was a problem hiding this comment.
The dotnet/agent-framework/sample-agent/ToolingManifest.json file has the same outdated MCP server names that were corrected here. It still contains "OneDriveMCPServer" (should be "mcp_ODSPRemoteServer") and "mcp_NLWeb" (should be removed). Consider applying the same corrections to that file for consistency across all .NET samples.
pontemonti
approved these changes
Jan 22, 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.
Have Updated ToolingManifest.json to confirm to the documentation for a365 CLU Details, below::
In a365 CLI, "mcp_NLWeb" and "mcpServerName": "mcp_KnowledgeTools" are same OBO permissions are created only for KnowledgeTools. However, when running a call to MCP for mcp_NLWeb is made which does not exist.
In a365 CLI, there is no MCP server for mapping of ""OneDriveMCPServer", have update it to correct name
Nore that the mcp_MeServer also does not work but will make a PR in a365 CLI for it.