Skip to content

feat: add Toolnames checkup workflow to prevent duplicate tool-name issues#381

Merged
rajbos merged 5 commits intomainfrom
copilot/add-labels-and-workflow-checks
Mar 11, 2026
Merged

feat: add Toolnames checkup workflow to prevent duplicate tool-name issues#381
rajbos merged 5 commits intomainfrom
copilot/add-labels-and-workflow-checks

Conversation

Copy link
Contributor

Copilot AI commented Mar 11, 2026

Summary

Closes the gap identified in #378 — where a tool-name issue was submitted but the tool was already mapped in src/toolNames.json on main.

Changes

New workflow: .github/workflows/check-toolnames.yml

check-toolnames (triggered when the "Toolnames checkup" label is added to any issue)

  • Checks out main and reads src/toolNames.json.
  • Extracts backtick-wrapped identifiers from the issue body (filtering out file paths and strings with spaces to reduce false positives).
  • Posts a comment on the issue with a table showing:
    • Already mapped — tools that are already in toolNames.json (issue is a duplicate)
    • Not yet added — tools that genuinely need to be mapped

Extension update: src/webview/usage/main.ts

createMcpToolIssueUrl() now applies both the "MCP Toolnames" and "Toolnames checkup" labels when a user opens an issue from the extension's "Report unknown tools" button. This means the checkup workflow fires automatically on every new issue — no maintainer action required.

How to use

  1. Merge this PR.
  2. Issues opened from the extension automatically carry both labels, triggering the checkup workflow immediately.
  3. The workflow posts a comment showing which tool names are already mapped and which still need to be added.

Workflow run link

${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI and others added 2 commits March 11, 2026 21:47
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
@rajbos
Copy link
Owner

rajbos commented Mar 11, 2026

@copilot , also update the extension code to include this label when the user creates the issue with the missing toolnames from the extension.

…el to issue URL

Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 11, 2026

@copilot , also update the extension code to include this label when the user creates the issue with the missing toolnames from the extension.

Done in commit aae7f62. createMcpToolIssueUrl in src/webview/usage/main.ts now includes both labels:

const labels = encodeURIComponent('MCP Toolnames,Toolnames checkup');

Issues created from the extension will automatically carry both labels, so the checkup workflow fires immediately without a maintainer having to add the label manually. Also removed the now-unnecessary setup-label job and workflow_dispatch trigger from the workflow.

@rajbos rajbos marked this pull request as ready for review March 11, 2026 22:16
@rajbos rajbos enabled auto-merge March 11, 2026 22:16
@rajbos rajbos disabled auto-merge March 11, 2026 22:17
@rajbos rajbos enabled auto-merge March 11, 2026 22:18
@rajbos
Copy link
Owner

rajbos commented Mar 11, 2026

Reverted the adding of both labels automatically to potentially prevent issues auto running workflows with issues:write access.

@rajbos rajbos merged commit a5e03ef into main Mar 11, 2026
13 checks passed
@rajbos rajbos deleted the copilot/add-labels-and-workflow-checks branch March 11, 2026 22:20
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