Skip to content

docs(mcp-analytics): specify the tools list event option - #142

Merged
gesh merged 1 commit into
mainfrom
georgis/mcp-tools-list-capture-option
Oct 28, 2026
Merged

gesh merged 1 commit into
mainfrom
georgis/mcp-tools-list-capture-option

Conversation

@gesh

@gesh gesh commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Problem

Follows #141, which added the same option for $mcp_initialize.

Changes

  • The Configuration table gets a tools list event row, on by default.
  • The name says "event", like the initialize event option in docs(mcp-analytics): specify the initialize event option #141. SDKs already have a manual captureToolsList method.
  • With the option off, the Optional events requirement says the SDK SHALL NOT send $mcp_tools_list, nor the $exception of a failed listing. It SHALL change nothing else: the listing keeps its injected arguments and virtual tools.
  • A new scenario under the existing @mcp_tools_list_capable tag checks that the listing keeps context.
  • The change follows the propose, apply and archive workflow. It lives under openspec/changes/archive/2026-10-23-mcp-tools-list-capture-option/.

Testing

  • openspec validate --specs --strict --no-interactive with CI's pinned 1.4.1 passes all 67 specs.
  • Not run: the new scenario. PostHog/posthog-sdk-test-harness has no MCP step bindings yet. tasks.md lists that as a follow-up.

🤖 Agent context

  • Autonomy: human-driven, agent-assisted.
  • Tool and model: Claude Code, claude-opus-5-5.

🤖 Generated with Claude Code

@lucasheriques lucasheriques left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, approving.

Base automatically changed from georgis/mcp-initialize-capture-option to main October 28, 2026 10:35
@gesh
gesh force-pushed the georgis/mcp-tools-list-capture-option branch from 745e8e7 to e840596 Compare October 28, 2026 13:24
@gesh gesh changed the title docs(mcp-analytics): specify the tools list capture option docs(mcp-analytics): specify the tools list event option Oct 28, 2026
Add a tools list event option, on by default, to the Configuration
table. With the option off, the Optional events requirement now says the
SDK sends no $mcp_tools_list and no $exception for a failed listing. It
changes nothing else. A new scenario checks that the listing keeps its
injected arguments.

The name says "event", like the initialize event option, because SDKs
already have a manual captureToolsList method.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@gesh
gesh force-pushed the georgis/mcp-tools-list-capture-option branch from e840596 to 8634551 Compare October 28, 2026 13:55
@gesh
gesh merged commit 68ee98e into main Oct 28, 2026
10 checks passed
@gesh
gesh deleted the georgis/mcp-tools-list-capture-option branch October 28, 2026 14:09
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