Skip to content

feat(module): honor X-MCP-Tools header to subset the tool catalog - #319

Merged
HugoRCD merged 1 commit into
mainfrom
feat/module-x-mcp-tools
Aug 14, 2026
Merged

feat(module): honor X-MCP-Tools header to subset the tool catalog#319
HugoRCD merged 1 commit into
mainfrom
feat/module-x-mcp-tools

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Aug 14, 2026

Copy link
Copy Markdown
Member

Summary

  • Clients can send X-MCP-Tools (comma-separated names matching tools/list) to limit which tools this request registers.
  • Unknown names return HTTP 400. Absent header leaves the catalog unchanged. Applied after enabled() and mcp:config:resolved, so no custom server/mcp/index.ts is required.
  • Matches the header convention already copied into nuxt/ui#6788 and nuxt.com#2383.

Test plan

  • Unit tests for parse/filter (absent, empty, whitespace, duplicates, kebab-case filename names, unknown names)
  • E2E: X-MCP-Tools: test_tool, string_tool subsets tools/list
  • E2E: unknown name returns HTTP 400

Clients can send a comma-separated allowlist so docs sites do not need a custom handler just to shrink tools/list.
@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nuxt-mcp-toolkit-docs Ready Ready Preview Aug 14, 2026 9:15am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

@pkg-pr-new

pkg-pr-new Bot commented Aug 14, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/nitro-mcp-toolkit@319
npm i https://pkg.pr.new/@nuxtjs/mcp-toolkit@319

commit: d33e0ca

@HugoRCD HugoRCD self-assigned this Aug 14, 2026
@HugoRCD
HugoRCD merged commit b31ce02 into main Aug 14, 2026
16 checks passed
@HugoRCD
HugoRCD deleted the feat/module-x-mcp-tools branch August 14, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant