Skip to content

Commit bc5ad15

Browse files
committed
test(dogfood): the MCP context-thread pin admits the bridge's per-request seams (#30892)
`handleMcp` now builds the bridge as `buildMcpBridge(deps, context, { recordFiles })`. The pin still requires the request context as the second argument and only admits a trailing seams object literal, so a bridge built from anything else still drops the key and goes STALE. Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016WV4cyqd3A7fbCWrqM3KCz
1 parent e3f0aa4 commit bc5ad15

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

‎packages/qa/dogfood/test/authz-conformance.test.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -519,10 +519,13 @@ const PROBES: readonly Probe[] = [
519519
re: /async\s+handleMcp\s*\(/g,
520520
key: () => 'mcp:http-dispatcher.ts:handleMcp',
521521
},
522+
// [#30892] The request context stays the bridge's SECOND argument; a trailing
523+
// seams object literal (the per-request file seam) is admitted beside it, so
524+
// a bridge built from anything but the request context still vanishes here.
522525
{
523526
kind: 'GATE_PIN',
524527
file: 'packages/runtime/src/domains/mcp.ts',
525-
re: /buildMcpBridge\(deps, context\)/g,
528+
re: /buildMcpBridge\(deps, context(?:, \{[^}]*\})?\)/g,
526529
key: () => 'mcp:domains/mcp.ts:buildMcpBridge(context-threaded)',
527530
},
528531
// (3) The stdio transport's PRINCIPAL binding (ADR-0101): the long-lived

0 commit comments

Comments
 (0)