Skip to content

Commit 7dbe357

Browse files
committed
docs(mcp): fix stale 'NOT WIRED YET' comment in telemetry.ts (#6617)
The header said recordMcpToolCall is deliberately not called from the tool-dispatch path, but #6237 (merged PR #6358) wired it in via recordMcpToolTelemetry in src/mcp/server.ts — the single chokepoint every tools/call routes through. Comment-only correction; no runtime change. Closes #6617
1 parent 8ac47e3 commit 7dbe357

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/mcp/telemetry.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,10 @@ import { PostHog } from "posthog-node";
1111
// The wrapper also never throws: a PostHog init/capture failure degrades to recording nothing, exactly like
1212
// the unconfigured path, so it can never surface an error into the MCP tool caller.
1313
//
14-
// NOT WIRED YET: per #6235 this module is deliberately NOT called from the tool-dispatch path — that (and the
15-
// client lifecycle/flush strategy a live Worker needs) is the separate instrumentation issue's job.
14+
// WIRED into the remote tool-dispatch path (#6237, merged PR #6358): {@link recordMcpToolCall} is invoked from
15+
// `recordMcpToolTelemetry` in src/mcp/server.ts — the single chokepoint every `tools/call` request that reaches
16+
// handleMcpRequest routes through exactly once, whether it succeeds or throws. (The local/CLI-side counterpart,
17+
// packages/loopover-mcp/lib/telemetry.js, has its own separate wiring status per #6236/#6238.)
1618

1719
/** PostHog US-cloud ingestion host — the default when POSTHOG_HOST isn't set. */
1820
const DEFAULT_POSTHOG_HOST = "https://us.i.posthog.com";

0 commit comments

Comments
 (0)