-
Notifications
You must be signed in to change notification settings - Fork 338
[Tool Description] Improve Grafana tool description for better LLM selection #831
Conversation
Co-authored-by: charris-msft <[email protected]>
Co-authored-by: charris-msft <[email protected]>
dotnet-install.sh
Outdated
@@ -0,0 +1,1888 @@ | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete the file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done! I've removed the dotnet-install.sh
file as requested. The file was inadvertently added during my exploration phase and should not have been included in the repository. Fixed in commit 57684b9.
Co-authored-by: xiangyan99 <[email protected]>
There was a problem hiding this 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 improves the Grafana tool description to enhance LLM tool selection accuracy by providing more comprehensive and specific guidance about when and how to use the Grafana workspace operations tool.
- Updated tool description from generic "resources" terminology to more precise "instances"
- Added detailed context about data source integrations and specific use cases
- Included explicit boundaries and exclusions to prevent misuse of the tool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from a wording standpoint. Though I'm not clear on which features are supported.
Sorry for the inconvenience, but we have moved the Azure MCP Server source code to https://github.com/microsoft/mcp. This change allows us to build any Microsoft MCP server with the same engineering system and allows us to brand any MCP server with either Azure or Microsoft branding. This repo change means that you need to move this PR to that new repo. Please do so and then add a link here to that new PR. We'll close this PR now and we look forward to seeing this over on the new repo. |
The Grafana tool description was lacking the detail and clarity needed for optimal LLM tool selection, resulting in an estimated confidence score of ~0.3 (below the target of ≥0.5). This change significantly improves the description to provide comprehensive guidance on when and how to use the Grafana workspace operations tool.
Changes Made
Updated the tool description in
GrafanaSetup.cs
to address the following issues:Before
After
The improved description provides the detail and clarity needed for optimal LLM tool selection while maintaining consistency with other tool description patterns in the codebase.
Fixes #827.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.