@fitchmultz/pi-mcp-adapter 8.0.6 → 8.0.8

package/dist/unix-socket-transport.d.ts

@@ -1,5 +1,4 @@
-import type { Transport } from "@modelcontextprotocol/client";
-import type { JSONRPCMessage } from "@modelcontextprotocol/client";
+import { type Transport, type JSONRPCMessage } from "@modelcontextprotocol/client";
 /** MCP JSONL transport for an explicitly configured Unix-domain socket. */
 export declare class UnixSocketClientTransport implements Transport {
     private socket;

package/dist/unix-socket-transport.js

@@ -1,5 +1,5 @@
 import { createConnection } from "node:net";
-import { ReadBuffer, serializeMessage } from "@modelcontextprotocol/client";
+import { ReadBuffer, serializeMessage, } from "@modelcontextprotocol/client";
 /** MCP JSONL transport for an explicitly configured Unix-domain socket. */
 export class UnixSocketClientTransport {
     socket;
@@ -23,13 +23,14 @@ export class UnixSocketClientTransport {
                 connected = true;
                 resolve();
             });
-            socket.on("data", chunk => {
+            socket.on("data", (chunk) => {
                 try {
                     this.readBuffer.append(chunk);
                     while (true) {
                         const message = this.readBuffer.readMessage();
-                        if (message === null)
+                        if (message === null) {
                             break;
+                        }
                         this.onmessage?.(message);
                     }
                 }
@@ -39,14 +40,16 @@ export class UnixSocketClientTransport {
                     void this.close();
                 }
             });
-            socket.on("error", error => {
-                if (!connected)
+            socket.on("error", (error) => {
+                if (!connected) {
                     reject(error);
+                }
                 this.onerror?.(error);
             });
             socket.on("close", () => {
-                if (this.socket === socket)
+                if (this.socket === socket) {
                     this.socket = undefined;
+                }
                 this.readBuffer.clear();
                 this.onclose?.();
             });
@@ -56,9 +59,10 @@ export class UnixSocketClientTransport {
         const socket = this.socket;
         this.socket = undefined;
         this.readBuffer.clear();
-        if (!socket || socket.destroyed)
+        if (!socket || socket.destroyed) {
             return;
-        await new Promise(resolve => {
+        }
+        await new Promise((resolve) => {
             const timeout = setTimeout(() => socket.destroy(), 2_000);
             timeout.unref();
             socket.once("close", () => {
@@ -70,14 +74,17 @@ export class UnixSocketClientTransport {
     }
     async send(message) {
         const socket = this.socket;
-        if (!socket || socket.destroyed)
+        if (!socket || socket.destroyed) {
             throw new Error("Unix socket is not connected");
+        }
         await new Promise((resolve, reject) => {
-            socket.write(serializeMessage(message), error => {
-                if (error)
+            socket.write(serializeMessage(message), (error) => {
+                if (error) {
                     reject(error);
-                else
+                }
+                else {
                     resolve();
+                }
             });
         });
     }

package/dist/utils.d.ts

@@ -2,6 +2,8 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import type { McpConfig, ServerEntry } from "./types.ts";
 export declare const ADAPTER_VERSION: string;
 export declare const DEFAULT_MCP_SCRIPT_TIMEOUT_MS = 30000;
+/** Most servers one multi-server pass connects at once. */
+export declare const MAX_CONCURRENT_SERVER_CONNECTS = 10;
 export declare function normalizeRequestTimeoutMs(timeoutMs: number | undefined): number | undefined;
 export declare function openUrl(pi: ExtensionAPI, url: string, browser?: string, signal?: AbortSignal): Promise<void>;
 export declare function openPath(pi: ExtensionAPI, targetPath: string): Promise<void>;

package/dist/utils.js

@@ -7,8 +7,9 @@ function readAdapterVersion() {
     for (const path of ["./package.json", "../package.json"]) {
         try {
             const manifest = JSON.parse(readFileSync(new URL(path, import.meta.url), "utf8"));
-            if (manifest.name === "@fitchmultz/pi-mcp-adapter" && typeof manifest.version === "string")
+            if (manifest.name === "@fitchmultz/pi-mcp-adapter" && typeof manifest.version === "string") {
                 return manifest.version;
+            }
         }
         catch { }
     }
@@ -16,6 +17,8 @@ function readAdapterVersion() {
 }
 export const ADAPTER_VERSION = readAdapterVersion();
 export const DEFAULT_MCP_SCRIPT_TIMEOUT_MS = 30_000;
+/** Most servers one multi-server pass connects at once. */
+export const MAX_CONCURRENT_SERVER_CONNECTS = 10;
 export function normalizeRequestTimeoutMs(timeoutMs) {
     return typeof timeoutMs === "number" && Number.isFinite(timeoutMs) && timeoutMs > 0
         ? timeoutMs
@@ -55,13 +58,16 @@ export async function parallelLimit(items, limit, fn) {
     async function worker() {
         while (true) {
             const next = iterator.next();
-            if (next.done)
+            if (next.done) {
                 return;
+            }
             const [index, item] = next.value;
             results[index] = await fn(item);
         }
     }
-    const workers = Array(Math.min(limit, items.length)).fill(null).map(() => worker());
+    const workers = Array(Math.min(limit, items.length))
+        .fill(null)
+        .map(() => worker());
     await Promise.all(workers);
     return results;
 }
@@ -82,37 +88,41 @@ function getMissingEnvVars(value) {
     return [...missing];
 }
 export function toStringRecord(value) {
-    if (!value || typeof value !== "object" || Array.isArray(value))
+    if (!value || typeof value !== "object" || Array.isArray(value)) {
         return undefined;
+    }
     const result = {};
     for (const [key, entry] of Object.entries(value)) {
-        if (typeof entry === "string")
+        if (typeof entry === "string") {
             result[key] = entry;
+        }
     }
     return Object.keys(result).length > 0 ? result : undefined;
 }
 function interpolateSecretExpression(value) {
-    if (value.startsWith("!!"))
+    if (value.startsWith("!!")) {
         return interpolateEnvVars(value.slice(1));
+    }
     return value.startsWith("!") ? value : interpolateEnvVars(value);
 }
 export function interpolateEnvRecord(values) {
-    if (!values)
+    if (!values) {
         return undefined;
-    return Object.fromEntries(Object.entries(values).map(([key, value]) => [
-        key,
-        interpolateSecretExpression(value),
-    ]));
+    }
+    return Object.fromEntries(Object.entries(values).map(([key, value]) => [key, interpolateSecretExpression(value)]));
 }
 const COMMAND_SECRET_TIMEOUT_MS = 10_000;
 const COMMAND_SECRET_MAX_OUTPUT_BYTES = 1024 * 1024;
 export function resolveCommandSecret(value, context) {
-    if (value === undefined)
+    if (value === undefined) {
         return undefined;
-    if (value.startsWith("!!"))
+    }
+    if (value.startsWith("!!")) {
         return interpolateEnvVars(value.slice(1));
-    if (!value.startsWith("!"))
+    }
+    if (!value.startsWith("!")) {
         return interpolateEnvVars(value);
+    }
     let result;
     try {
         result = spawnSync(value.slice(1), {
@@ -140,22 +150,22 @@ export function resolveCommandSecret(value, context) {
         throw new Error(`Failed to resolve ${context}: command exited with code ${result.status ?? "unknown"}`);
     }
     const resolved = result.stdout.trim();
-    if (!resolved)
+    if (!resolved) {
         throw new Error(`Failed to resolve ${context}: command returned empty output`);
+    }
     return resolved;
 }
 /** Resolve command markers in a configured record without mutating the input. */
 export function resolveCommandSecretsRecord(values, context) {
-    if (!values)
+    if (!values) {
         return undefined;
-    return Object.fromEntries(Object.entries(values).map(([key, value]) => [
-        key,
-        resolveCommandSecret(value, context(key)),
-    ]));
+    }
+    return Object.fromEntries(Object.entries(values).map(([key, value]) => [key, resolveCommandSecret(value, context(key))]));
 }
 export function resolveServerUrl(definition) {
-    if (definition.url == null)
+    if (definition.url == null) {
         return undefined;
+    }
     if (typeof definition.url !== "string") {
         throw new Error("MCP server URL must be a string");
     }
@@ -168,16 +178,20 @@ export function resolveServerUrl(definition) {
         new URL(resolved);
     }
     catch (error) {
-        throw new Error(`Invalid MCP server URL after environment interpolation: ${resolved}`, { cause: error });
+        throw new Error(`Invalid MCP server URL after environment interpolation: ${resolved}`, {
+            cause: error,
+        });
     }
     return resolved;
 }
 export function resolveConfigPath(value) {
-    if (value === undefined)
+    if (value === undefined) {
         return undefined;
+    }
     const resolved = interpolateEnvVars(value);
-    if (resolved === "~")
+    if (resolved === "~") {
         return homedir();
+    }
     if (resolved.startsWith("~/") || resolved.startsWith("~\\")) {
         return join(homedir(), resolved.slice(2));
     }
@@ -203,8 +217,9 @@ export function stripOscSequences(text) {
         index += isEscOsc ? 2 : 1;
         while (index < text.length) {
             const code = text.charCodeAt(index++);
-            if (code === 0x07 || code === 0x9c)
+            if (code === 0x07 || code === 0x9c) {
                 break;
+            }
             if (code === 0x1b && text[index] === "\\") {
                 index++;
                 break;
@@ -224,25 +239,32 @@ export function formatTerminalError(error) {
     const messages = [];
     const seen = new Set();
     const collect = (value) => {
-        if (seen.has(value))
+        if (seen.has(value)) {
             return;
-        if ((typeof value === "object" && value !== null) || typeof value === "function")
+        }
+        if ((typeof value === "object" && value !== null) || typeof value === "function") {
             seen.add(value);
+        }
         if (value instanceof AggregateError) {
             const countBefore = messages.length;
-            for (const nested of value.errors)
+            for (const nested of value.errors) {
                 collect(nested);
-            if (value.cause !== undefined)
+            }
+            if (value.cause !== undefined) {
                 collect(value.cause);
-            if (messages.length === countBefore && value.message)
+            }
+            if (messages.length === countBefore && value.message) {
                 messages.push(value.message);
+            }
             return;
         }
         if (value instanceof Error) {
-            if (value.message)
+            if (value.message) {
                 messages.push(value.message);
-            if (value.cause !== undefined)
+            }
+            if (value.cause !== undefined) {
                 collect(value.cause);
+            }
             return;
         }
         messages.push(String(value));
@@ -251,8 +273,9 @@ export function formatTerminalError(error) {
     return sanitizeTerminalText([...new Set(messages)].join(": "));
 }
 export function truncateAtWord(text, target) {
-    if (!text || text.length <= target)
+    if (!text || text.length <= target) {
         return text;
+    }
     const truncated = text.slice(0, target);
     const lastSpace = truncated.lastIndexOf(" ");
     if (lastSpace > target * 0.6) {
@@ -270,8 +293,9 @@ export function formatAuthRequiredMessage(config, serverName, defaultMessage) {
     return template ? template.replaceAll("${server}", serverName) : defaultMessage;
 }
 export function formatMcpStatus(config, message) {
-    if (config.settings?.mcpFooterStatus === "off")
+    if (config.settings?.mcpFooterStatus === "off") {
         return undefined;
+    }
     return `${config.settings?.showStatusIcon === false ? "MCP: " : "🔌 MCP: "}${message}`;
 }
 /**
@@ -279,8 +303,9 @@ export function formatMcpStatus(config, message) {
  */
 export function extractToolUiStreamMode(toolMeta) {
     const uiMeta = toolMeta?.ui;
-    if (!uiMeta || typeof uiMeta !== "object")
+    if (!uiMeta || typeof uiMeta !== "object") {
         return undefined;
+    }
     const streamMode = uiMeta["pi-mcp-adapter.streamMode"];
     if (streamMode === "eager" || streamMode === "stream-first") {
         return streamMode;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fitchmultz/pi-mcp-adapter",
-  "version": "8.0.6",
+  "version": "8.0.8",
   "description": "Token-efficient MCP adapter for the Pi coding agent",
   "type": "module",
   "types": "./dist/index.d.ts",
@@ -33,10 +33,32 @@
     "prepare": "node ./scripts/prepare.mjs",
     "check:compat": "node scripts/compat-host.mjs && npm run build && npm run typecheck && vitest run --exclude __tests__/interactive-visualizer-server.test.ts && npm run test:oauth && node scripts/compat-package-smoke.mjs",
     "typecheck": "node ./scripts/tsc.mjs --noEmit",
+    "typecheck:all": "node ./scripts/typecheck-all.mjs",
     "test": "vitest run && npm run test:oauth",
     "test:watch": "vitest",
     "test:oauth": "PI_MCP_ADAPTER_TEST_AUTH_STORE=memory node --test --test-concurrency=1 mcp-auth.test.ts mcp-auth-flow.test.ts mcp-callback-server.test.ts mcp-oauth-provider.test.ts",
-    "test:conformance": "bash conformance/run.sh"
+    "test:conformance": "bash conformance/run.sh",
+    "check:directives": "node scripts/check-directives.mjs",
+    "check:policy": "node scripts/check-policy.mjs",
+    "lint": "oxlint --deny-warnings .",
+    "lint:conformance": "oxlint --deny-warnings conformance",
+    "lint:prompts": "oxlint --deny-warnings __tests__/prompts.test.ts __tests__/prompts-sdk-integration.test.ts __tests__/test-state.ts __tests__/test-extension-context.ts",
+    "lint:runtime-tests": "oxlint --deny-warnings __tests__/runtime",
+    "lint:tracing": "oxlint --deny-warnings mcp-trace.ts",
+    "lint:sdk-http": "oxlint --deny-warnings __tests__/sdk-v2-http.test.ts __tests__/fixtures/native-capture-extension.ts",
+    "lint:lifecycle-tests": "oxlint --deny-warnings __tests__/index-lifecycle.test.ts __tests__/fixtures/extension-lifecycle.ts __tests__/test-extension-context.ts",
+    "lint:auth-tests": "oxlint --deny-warnings __tests__/auth",
+    "lint:discovery-tests": "oxlint --deny-warnings __tests__/discovery __tests__/fixtures/pinned-tools.ts",
+    "lint:state-tests": "oxlint --deny-warnings __tests__/state",
+    "lint:output-tests": "oxlint --deny-warnings __tests__/output",
+    "lint:packaging-tests": "oxlint --deny-warnings __tests__/packaging",
+    "lint:quality": "oxlint --deny-warnings scripts/check-directives.mjs scripts/check-policy.mjs scripts/quality-exceptions.mjs __tests__/quality",
+    "lint:ui-host-tests": "oxlint --deny-warnings __tests__/ui-host",
+    "lint:panel-tests": "oxlint --deny-warnings __tests__/panel",
+    "lint:agent": "oxlint --deny-warnings --format=agent .",
+    "lint:fix": "oxlint --fix .",
+    "format": "oxfmt --write .",
+    "format:check": "oxfmt --check ."
   },
   "repository": {
     "type": "git",
@@ -112,6 +134,10 @@
     "@types/cross-spawn": "6.0.6",
     "@types/node": "24.19.1",
     "esbuild": "0.28.2",
+    "oxc-parser": "0.156.0",
+    "oxfmt": "0.75.0",
+    "oxlint": "1.90.0",
+    "oxlint-tsgolint": "7.0.2005",
     "typebox": "1.3.27",
     "typescript": "7.0.2",
     "vitest": "5.0.3"

package/skills/mcp-scripting/SKILL.md

@@ -66,12 +66,12 @@ return page;
 Gateway equivalents require explicit actions:
 
 ```js
-mcp({ action: "search", query: "search issues", server: "github" })
-mcp({ action: "describe", tool: "github_search_issues" })
-mcp({ action: "call", tool: "github_search_issues", args: { query: "is:open" } })
-mcp({ action: "resources", server: "docs" })
-mcp({ action: "read-resource", server: "docs", uri: "docs://guide" })
-mcp({ action: "read-result", ref: "/returned/result/path", path: "/structuredContent" })
+mcp({ action: "search", query: "search issues", server: "github" });
+mcp({ action: "describe", tool: "github_search_issues" });
+mcp({ action: "call", tool: "github_search_issues", args: { query: "is:open" } });
+mcp({ action: "resources", server: "docs" });
+mcp({ action: "read-resource", server: "docs", uri: "docs://guide" });
+mcp({ action: "read-result", ref: "/returned/result/path", path: "/structuredContent" });
 ```
 
 Gateway search returns schemas without activating functions; use it when only `mcp` is allowed. New gateway calls use object `args`. Optional v5 mode fields and JSON-string arguments are only a stored-call ingress compatibility path.