Skip to content

Update bestpractices tool description to ask LLM to use azure skills#1622

Merged
fanyang-mono merged 3 commits intomicrosoft:mainfrom
fanyang-mono:mention_skills
Feb 3, 2026
Merged

Update bestpractices tool description to ask LLM to use azure skills#1622
fanyang-mono merged 3 commits intomicrosoft:mainfrom
fanyang-mono:mention_skills

Conversation

@fanyang-mono
Copy link
Copy Markdown
Contributor

@fanyang-mono fanyang-mono commented Feb 2, 2026

What does this PR do?

Ask LLM to use azure skills

GitHub issue number?

Fixes #1623

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
    • Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • 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.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Validate README.md changes using script at eng/scripts/Process-PackageReadMe.ps1. See Package README
    • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • Run .\eng\scripts\Update-AzCommandsMetadata.ps1 to update tool metadata in azmcp-commands.md (required for CI)
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or 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 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 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 - live to run Live Test Pipeline

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a single line of instruction to the Azure Best Practices tool description, asking the LLM to use its "Azure skills" when available.

Changes:

  • Added LLM instruction in the Azure Best Practices tool description to use Azure skills

Comment thread tools/Azure.Mcp.Tools.AzureBestPractices/src/AzureBestPracticesSetup.cs Outdated
Copy link
Copy Markdown
Member

@joshfree joshfree left a comment

Choose a reason for hiding this comment

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

please add a CHANGELOG entry too

Comment thread tools/Azure.Mcp.Tools.AzureBestPractices/src/AzureBestPracticesSetup.cs Outdated
@joshfree joshfree added this to the 2026-02 milestone Feb 2, 2026
@joshfree joshfree moved this from Untriaged to In Progress in Azure MCP Server Feb 2, 2026
@fanyang-mono fanyang-mono requested a review from a team as a code owner February 2, 2026 20:11
@fanyang-mono fanyang-mono changed the title Ask LLM to use azure skills Update bestpractices tool description to ask LLM to use azure skills Feb 2, 2026
@fanyang-mono fanyang-mono enabled auto-merge (squash) February 2, 2026 20:22
auto-merge was automatically disabled February 2, 2026 22:56

Pull request was closed

@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Feb 2, 2026
@fanyang-mono fanyang-mono reopened this Feb 2, 2026
@github-project-automation github-project-automation Bot moved this from Done to Untriaged in Azure MCP Server Feb 2, 2026
@fanyang-mono fanyang-mono enabled auto-merge (squash) February 2, 2026 23:51
@fanyang-mono fanyang-mono merged commit 4250714 into microsoft:main Feb 3, 2026
15 of 17 checks passed
@github-project-automation github-project-automation Bot moved this from Untriaged to Done in Azure MCP Server Feb 3, 2026
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Apr 20, 2026
…icrosoft#1622)

* Mention skills

* Make the message more specific

* Added change log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Update Azure best practice tool to take into account any available skills

5 participants