Skip to content

Enhance tool description for Monitor tool#630

Merged
anannya03 merged 11 commits intomicrosoft:mainfrom
anannya03:enhance-tool-desc-monitor
Oct 7, 2025
Merged

Enhance tool description for Monitor tool#630
anannya03 merged 11 commits intomicrosoft:mainfrom
anannya03:enhance-tool-desc-monitor

Conversation

@anannya03
Copy link
Copy Markdown
Contributor

@anannya03 anannya03 commented Sep 30, 2025

What does this PR do?

Enhance tool description for Monitor tool to reduce ambiguity.

After change-
Screenshot 2025-10-06 141201

Before change-
Screenshot 2025-09-30 090050


After change-

Screenshot 2025-10-02 214949

Before change-

image

After Change-

Screenshot 2025-10-02 214956

GitHub issue number?

#624, #612

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 enhances tool descriptions for Azure Monitor tools to reduce ambiguity and provide clearer guidance. The changes focus on making the descriptions more concise while providing better context about when and how to use each tool.

Key changes:

  • Improved clarity and specificity in tool descriptions
  • Consolidated verbose descriptions into more focused explanations
  • Enhanced context about tool usage scenarios

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ResourceLogQueryCommand.cs Streamlined description with clearer usage guidance for querying Azure resource logs
EntityGetHealthCommand.cs Simplified and clarified description for retrieving entity health information

Comment thread tools/Azure.Mcp.Tools.Monitor/src/Commands/Log/ResourceLogQueryCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Monitor/src/Commands/Log/ResourceLogQueryCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Monitor/src/Commands/Log/ResourceLogQueryCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.Monitor/src/Commands/Log/ResourceLogQueryCommand.cs Outdated
@vcolin7
Copy link
Copy Markdown
Contributor

vcolin7 commented Oct 1, 2025

Don't forget to update the CHANGELOG :)

@anannya03 anannya03 force-pushed the enhance-tool-desc-monitor branch from 24e95c5 to ac0eea1 Compare October 1, 2025 09:50
Copy link
Copy Markdown
Member

@srnagar srnagar left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Oct 1, 2025
@anannya03
Copy link
Copy Markdown
Contributor Author

This PR is still a WIP. I tested manually and the LLM is still not picking up the right tools. So, I'm still working on this. I'll let you know once it works correctly and post the screenshots of the results.

@jongio
Copy link
Copy Markdown
Contributor

jongio commented Oct 1, 2025

If WIP, then please make it a DRAFT PR. Thanks!

@anannya03 anannya03 marked this pull request as draft October 1, 2025 20:53
@anannya03 anannya03 marked this pull request as ready for review October 6, 2025 10:52
@anannya03
Copy link
Copy Markdown
Contributor Author

anannya03 commented Oct 6, 2025

azmcp_monitor_resource_log_query

Screenshot 2025-10-02 213312

azmcp_monitor_workspace_log_query

image

@anannya03
Copy link
Copy Markdown
Contributor Author

azmcp_monitor_healthmodels_entity_gethealth

Screenshot 2025-10-06 132823

Comment thread servers/Azure.Mcp.Server/CHANGELOG.md Outdated
Comment thread servers/Azure.Mcp.Server/CHANGELOG.md Outdated
@anannya03 anannya03 force-pushed the enhance-tool-desc-monitor branch from ff18e87 to 401201c Compare October 7, 2025 00:02
Copy link
Copy Markdown
Member

@srnagar srnagar left a comment

Choose a reason for hiding this comment

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

LGTM

@anannya03 anannya03 merged commit b96cd74 into microsoft:main Oct 7, 2025
26 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Oct 7, 2025
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Dec 8, 2025
* Enhance tool description for Monitor tool

* Update tools/Azure.Mcp.Tools.Monitor/src/Commands/Log/ResourceLogQueryCommand.cs

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

* refined the description

* indenttation fix

* updated changelog

* Changed description

* resolved merge conflict

* fixed prompt for moinotr gethealth and ddescription

* resolved merge conflict

* rebased with main

* addressed review comments

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants