mirror of
https://github.com/microsoft/mcp.git
synced 2026-11-04 12:01:31 +00:00
Amos HerschandCopilot App
1d7df774b7
* docs: make Fabric tool description evaluation optional Keep Azure and other non-Fabric evaluation gates, including stricter toolset requirements, and all other validation unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * docs: use absolute links for Fabric evaluation policy Fix the three AGENTS.md policy links rejected by repository link-format guidance. Policy scope is unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * docs: minimize Fabric evaluator exception wording Keep only short Fabric qualifiers in existing guidance, preserving the original requirements and examples. Remove the expanded policy section and links as requested. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
3.2 KiB
3.2 KiB
What does this PR do?
[Provide a clear, concise description of the changes]
[Add additional context, screenshots, or information that helps reviewers]
GitHub issue number?
[Link to the GitHub issue this PR addresses]
Pre-merge Checklist
- Required for All PRs
- Read contribution guidelines
- PR title clearly describes the change
- Commit history is clean with descriptive messages (cleanup guide)
- Added comprehensive tests for new/modified functionality
- Created a changelog entry if the change falls among the following: new feature, bug fix, UI/UX update, breaking change, or updated dependencies. Follow the changelog entry guide
- For MCP tool changes:
- One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
- Updated
servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentation - Validate
README.mdchanges running the script./eng/scripts/Process-PackageReadMe.ps1. See Package README - For new or modified tool descriptions (excluding Fabric MCP tools), ran
ToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test prompts - For tools with new names, including new tools or renamed tools, update
consolidated-tools.json - For renamed tools, follow the Tool Rename Checklist and tag the PR with the
breaking-changelabel - For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
- Extra steps for Azure MCP Server tool changes:
- Updated command list in
servers/Azure.Mcp.Server/docs/azmcp-commands.md - Ran
./eng/scripts/Update-AzCommandsMetadata.ps1to update tool metadata inazmcp-commands.md(required for CI) - Updated test prompts in
servers/Azure.Mcp.Server/docs/e2eTestPrompts.md - 👉 For Community (non-Microsoft team member) PRs:
- Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (
crypto mining, spam, data exfiltration, etc.) - Manual tests run: added comment
/azp run mcp - pullrequest - liveto run Live Test Pipeline
- Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (
- Updated command list in