From 5d22ab6368f92c892a48d88167bc04e25fdbbc6f Mon Sep 17 00:00:00 2001 From: Alex Alecu Date: Wed, 25 Feb 2026 13:17:07 +0200 Subject: [PATCH] fix: document SDK version for private _process field access Add a comment noting that (transport as any)._process relies on a private implementation detail of StdioClientTransport from @modelcontextprotocol/sdk@1.25.2. Future SDK upgrades should verify this field still exists. --- packages/opencode/src/mcp/index.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/packages/opencode/src/mcp/index.ts b/packages/opencode/src/mcp/index.ts index 8143555e467..d561600e87d 100644 --- a/packages/opencode/src/mcp/index.ts +++ b/packages/opencode/src/mcp/index.ts @@ -450,6 +450,9 @@ export namespace MCP { }) // kilocode_change start — wrap start/close for process tracking and force-kill safety net + // NOTE: _process is a private field of StdioClientTransport from @modelcontextprotocol/sdk. + // Verified against @modelcontextprotocol/sdk@1.25.2. If the SDK is upgraded, verify this + // field still exists (see node_modules/@modelcontextprotocol/sdk/dist/esm/client/stdio.js). const originalStart = transport.start.bind(transport) transport.start = async () => { try {