From 4c7b67195034a502da7a28bb4bc079a2fadf240a Mon Sep 17 00:00:00 2001 From: claude code agent 227 Date: Mon, 29 Jun 2026 01:59:10 +0300 Subject: [PATCH] =?UTF-8?q?docs(#193):=20correct=20contract-guard=20commen?= =?UTF-8?q?t=20=E2=80=94=20interface=20is=20a=20subset,=20not=20superset?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The DocmostClientLike mirror covers only methods the in-app adapter consumes; the standalone MCP transport calls additional client methods not tracked here (covered by its own typecheck). Fixes the misleading 'superset' wording (F2). Co-Authored-By: Claude Opus 4.8 (1M context) --- packages/mcp/test/unit/client-host-contract.test.mjs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/mcp/test/unit/client-host-contract.test.mjs b/packages/mcp/test/unit/client-host-contract.test.mjs index f807b082..ad0abab7 100644 --- a/packages/mcp/test/unit/client-host-contract.test.mjs +++ b/packages/mcp/test/unit/client-host-contract.test.mjs @@ -26,7 +26,10 @@ import { DocmostClient } from "../../build/index.js"; // // Keep the HOST_CONTRACT_METHODS NAME list aligned with the method NAMES declared // in the server's DocmostClientLike interface (the in-app per-user tool adapter -// only — it is the superset of what either transport calls). Full type-derivation +// only — it is a SUBSET of the DocmostClient surface — covers only what the in-app adapter +// consumes; the standalone MCP transport (packages/mcp/src/index.ts) calls additional +// client methods (insertImage/replaceImage/deleteComment/updateComment/insertFootnote/ +// uploadImage) that this guard does NOT track — the MCP transport's own typecheck covers those). Full type-derivation // of DocmostClientLike from this class is deferred (see the staged plan in // docmost-client.loader.ts): the package emits no declarations and the real // (inferred, concrete) return types conflict with the host's loose