Skip to content

Fix/update android dri mcp tool prefix, Fixes AB#3601970#432

Merged
somalaya merged 1 commit into
masterfrom
fix/update-mcp-tool-prefix
May 12, 2026
Merged

Fix/update android dri mcp tool prefix, Fixes AB#3601970#432
somalaya merged 1 commit into
masterfrom
fix/update-mcp-tool-prefix

Conversation

@somalaya
Copy link
Copy Markdown
Contributor

@somalaya somalaya commented May 12, 2026

In my previous PR, I merged the android dri mcp tool but a lot of references in the code were still pointing to older mcp tool names. In this PR, I have modified all the references.

Fixes AB#3601970

@somalaya somalaya requested a review from a team as a code owner May 12, 2026 15:36
@github-actions
Copy link
Copy Markdown

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@somalaya somalaya changed the title Fix/update android dri mcp tool prefix Fix/update android dri mcp tool prefix, Fixes AB#3601970 May 12, 2026
@github-actions
Copy link
Copy Markdown

✅ Work item link check complete. Description contains link AB#3601970 to an Azure Boards work item.

@github-actions
Copy link
Copy Markdown

❌ Work item link check failed. Description contains AB#3601970 but the Bot could not link it to an Azure Boards work item.

Click here to learn more.

@richardtz12
Copy link
Copy Markdown
Contributor

Hey Sowmya, why did the naming convention go from s2 -> s? Why was the previous naming convention named "s2"?

@somalaya
Copy link
Copy Markdown
Contributor Author

Hey Sowmya, why did the naming convention go from s2 -> s? Why was the previous naming convention named "s2"?

The prefix is auto-generated by VS Code — it truncates the MCP server key name from mcp.json to create tool prefixes. Previously the server was registered under a different key name (e.g. android-dri-search2 or similar) which produced the mcp_android-dri-s2_* prefix.

@somalaya somalaya merged commit 3a0fcc3 into master May 12, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants