You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Part of #6228. With the remote-side telemetry wrapper module in place (companion issue), this issue wires it into src/mcp/server.ts's actual tool-dispatch path.
Requirements
Find (or, if one doesn't cleanly exist, create) a single chokepoint in src/mcp/server.ts through which every registered tool's invocation passes, and call the telemetry wrapper's recordMcpToolCall there — one instrumentation point, not one per tool.
Must not change any tool's actual behavior or response shape — this is pure observability, zero functional change.
Must not throw or block a tool response if the telemetry call fails — telemetry failures are silently swallowed (matching the wrapper module's own no-op guarantee).
Test Coverage Requirements
99%+ Codecov patch coverage; a regression test confirming a tool call still succeeds and returns its normal response even if the telemetry call fails/throws.
Deliverables
Every tool invocation on the remote server triggers exactly one recordMcpToolCall.
Zero behavior change to any tool's actual response.
Expected Outcome
Real usage data starts flowing for every MCP tool call made against the remote server.
Context
Part of #6228. With the remote-side telemetry wrapper module in place (companion issue), this issue wires it into
src/mcp/server.ts's actual tool-dispatch path.Requirements
src/mcp/server.tsthrough which every registered tool's invocation passes, and call the telemetry wrapper'srecordMcpToolCallthere — one instrumentation point, not one per tool."remote"), success/failure, and coarse latency (start-to-finish duration) — nothing else, per Spec: usage-telemetry instrumentation strategy for MCP (PostHog) #6228's field allowlist.Test Coverage Requirements
99%+ Codecov patch coverage; a regression test confirming a tool call still succeeds and returns its normal response even if the telemetry call fails/throws.
Deliverables
recordMcpToolCall.Expected Outcome
Real usage data starts flowing for every MCP tool call made against the remote server.
Links & Resources
recordMcpToolCall)