Skip to content

Fixing MySQL and Postgres command names. Update azmcp_mysql_server_param_set description.#614

Merged
vcolin7 merged 7 commits intomicrosoft:mainfrom
vcolin7:description-tuning/mysql-postgres
Sep 30, 2025
Merged

Fixing MySQL and Postgres command names. Update azmcp_mysql_server_param_set description.#614
vcolin7 merged 7 commits intomicrosoft:mainfrom
vcolin7:description-tuning/mysql-postgres

Conversation

@vcolin7
Copy link
Copy Markdown
Contributor

@vcolin7 vcolin7 commented Sep 29, 2025

What does this PR do?

Updates the names of the MySQL and Postgres "get" commands: config_get, schema_get, and param_get. Before these changes, commands would be surfaced to the LLM as azmcp_postgres_server_config_config or azmcp_postgres_table_schema_schema , for example.

Also updates the tool description for azmcp_mysql_server_param_set, helping disambiguate from similar tools:

Before

image

After

image

GitHub issue number?

N/A

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
    • Updated command list in /docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • 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 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 /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 fixes command naming issues in MySQL and PostgreSQL tools to prevent duplicated naming patterns and improves tool descriptions for better LLM disambiguation. The changes ensure that commands are exposed to LLMs with clearer, non-redundant names.

Key changes:

  • Renamed MySQL and PostgreSQL "get" commands from specific names (config, schema, param) to a unified "get" pattern
  • Updated MySQL server parameter set tool description to be more concise and clear

Reviewed Changes

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

Show a summary per file
File Description
tools/Azure.Mcp.Tools.Postgres/src/Commands/Table/TableSchemaGetCommand.cs Changed command name from "schema" to "get"
tools/Azure.Mcp.Tools.Postgres/src/Commands/Server/ServerParamGetCommand.cs Changed command name from "param" to "get"
tools/Azure.Mcp.Tools.Postgres/src/Commands/Server/ServerConfigGetCommand.cs Changed command name from "config" to "get"
tools/Azure.Mcp.Tools.MySql/src/Commands/Table/TableSchemaGetCommand.cs Changed command name from "schema" to "get"
tools/Azure.Mcp.Tools.MySql/src/Commands/Server/ServerParamSetCommand.cs Simplified tool description for better clarity
tools/Azure.Mcp.Tools.MySql/src/Commands/Server/ServerParamGetCommand.cs Changed command name from "param" to "get"
tools/Azure.Mcp.Tools.MySql/src/Commands/Server/ServerConfigGetCommand.cs Changed command name from "config" to "get"

@vcolin7 vcolin7 added tools-MySQL tools-Postgres PostgreSQL tools-ToolSelection Issues related to the Tools Description Evaluator and incorrect tool selection by LLMs labels Sep 29, 2025
@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Sep 29, 2025
@vcolin7 vcolin7 enabled auto-merge (squash) September 30, 2025 18:06
@vcolin7 vcolin7 added this to the 2025-09 milestone Sep 30, 2025
@vcolin7 vcolin7 merged commit ccc1568 into microsoft:main Sep 30, 2025
23 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Sep 30, 2025
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Dec 8, 2025
…aram_set` description. (microsoft#614)

* Fixed MySQL and Postgres tool names

* Updated the description of ServerParamSetCommand.cs

* Fixed unit test

* Updated other unit tests

* Updated CHANGELOG
@vcolin7 vcolin7 deleted the description-tuning/mysql-postgres branch January 9, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools-MySQL tools-Postgres PostgreSQL tools-ToolSelection Issues related to the Tools Description Evaluator and incorrect tool selection by LLMs

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants