Skip to content

Carry MCP tool _meta through to the persisted catalog - #1813

Merged
RhysSullivan merged 1 commit into
mainfrom
issue-1683
Aug 28, 2026
Merged

Carry MCP tool _meta through to the persisted catalog#1813
RhysSullivan merged 1 commit into
mainfrom
issue-1683

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

Fixes #1683

The MCP spec reserves _meta on Tool, but the plugin decoded each tools/list entry with a closed struct that did not declare the field, so it was dropped before the manifest entry was built. The listed-tool decode now declares _meta, the manifest entry carries it, and toToolDef stamps it into the mcp envelope already persisted in each tool row's annotations — Executor's own Tool has no _meta field, so a host reads it back from annotations.mcp._meta. The stamp schema declares it too, so it is not stripped again when a row is read at invoke time.

The map stays opaque; nothing interprets it and it never reaches the model. Because it is server-controlled, it is decoded permissively: a _meta that is not the spec's map shape is ignored for that tool rather than failing the whole-list decode and dropping every tool.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing 3c974d0 Commit Preview URL

Branch Preview URL
Aug 28 2026, 08:14 AM

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud 3c974d0 Aug 28 2026, 08:15 AM

@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@pkg-pr-new

pkg-pr-new Bot commented Aug 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@1813

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@1813

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@1813

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@1813

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@1813

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@1813

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@1813

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@1813

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@1813

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@1813

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@1813

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@1813

executor

npm i https://pkg.pr.new/executor@1813

commit: 3c974d0

@RhysSullivan
RhysSullivan marked this pull request as ready for review August 28, 2026 18:13
@RhysSullivan
RhysSullivan merged commit 6305b6d into main Aug 28, 2026
44 checks passed
@RhysSullivan RhysSullivan mentioned this pull request Aug 28, 2026
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.

plugin-mcp drops MCP tool _meta when decoding tools/list

1 participant