@leo000001/claude-code-mcp 2.4.6 → 2.4.8

package/CHANGELOG.md

@@ -22,6 +22,8 @@
 
 ### Documentation
 
+- Restructure documentation architecture: `AGENTS.md` is now execution-first, while `docs/DESIGN.md` is the single detailed source for interface/mapping semantics.
+- Add explicit doc-governance rules, upgrade submission template, and document DoD to reduce AGENTS/DESIGN duplication regression.
 - Align README/DESIGN/AGENTS with current defaults and behavior (timeout clamp, advanced parameter count, lifecycle semantics).
 - Clarify package positioning as CLI-first and remove stale guidance that implied a public programmatic API surface.
 - Update CONTRIBUTING with local environment requirements (Node/npm and Windows Git Bash notes).

package/dist/index.js

@@ -329,7 +329,9 @@ var SessionManager = class _SessionManager {
     if (info.status === "waiting_permission") {
       this.finishAllPending(
         sessionId,
-        { behavior: "deny", message: opts?.reason ?? "Session interrupted", interrupt: true },
+        // Similar to cancel(), avoid emitting interrupt=true while the query stream is
+        // being torn down to reduce SDK-level write/teardown races on stdio clients.
+        { behavior: "deny", message: opts?.reason ?? "Session interrupted", interrupt: false },
         "interrupt"
       );
     }
@@ -2494,11 +2496,14 @@ function detectPathCompatibilityWarnings(session) {
   }
   return warnings;
 }
-function isPosixHomePath(value) {
-  return /^\/home\/[^/\s]+(?:\/|$)/.test(value);
+function isSuspiciousPosixAbsolutePathOnWindows(value) {
+  if (!value.startsWith("/") || value.startsWith("//")) return false;
+  return /^\/home\/[^/\s]+(?:\/|$)/.test(value) || /^\/(?:mnt\/)?[a-zA-Z](?:\/|$)/.test(value) || /^\/(tmp|var|etc|opt|usr|bin|sbin)(?:\/|$)/.test(value);
 }
-function extractPosixHomePath(value) {
-  const match = value.match(/\/home\/[^/\s"'`]+(?:\/[^\s"'`]*)?/);
+function extractSuspiciousPosixAbsolutePath(value) {
+  const match = value.match(
+    /\/(?:home\/[^/\s"'`]+(?:\/[^\s"'`]*)?|(?:mnt\/)?[a-zA-Z](?:\/[^\s"'`]*)?|(?:tmp|var|etc|opt|usr|bin|sbin)(?:\/[^\s"'`]*)?)/
+  );
   return match?.[0];
 }
 function detectPendingPermissionPathWarnings(pending, session) {
@@ -2527,13 +2532,13 @@ function detectPendingPermissionPathWarnings(pending, session) {
     }
     const command = req.input.command;
     if (typeof command === "string") {
-      const embeddedPath = extractPosixHomePath(command);
+      const embeddedPath = extractSuspiciousPosixAbsolutePath(command);
       if (embeddedPath) candidates.push(embeddedPath);
     }
-    const badPath = candidates.find((p) => isPosixHomePath(p));
+    const badPath = candidates.find((p) => isSuspiciousPosixAbsolutePathOnWindows(p));
     if (!badPath) continue;
     warnings.push(
-      `Permission request '${req.requestId}' uses POSIX home path '${badPath}' on Windows. Prefer an absolute Windows path${cwdHint} to avoid out-of-bounds permission prompts.`
+      `Permission request '${req.requestId}' uses POSIX absolute path '${badPath}' on Windows. Prefer an absolute Windows path${cwdHint} to avoid out-of-bounds permission prompts.`
     );
   }
   return warnings;
@@ -3590,7 +3595,7 @@ function registerResources(server, deps) {
 }
 
 // src/server.ts
-var SERVER_VERSION = true ? "2.4.6" : "0.0.0-dev";
+var SERVER_VERSION = true ? "2.4.8" : "0.0.0-dev";
 function createServerContext(serverCwd) {
   const sessionManager = new SessionManager();
   const server = new McpServer(
@@ -4029,7 +4034,8 @@ var BENIGN_MESSAGE_PATTERNS = [
   "write after end",
   "stream was destroyed",
   "cannot call write after a stream was destroyed",
-  "the pipe is being closed"
+  "the pipe is being closed",
+  "query closed before response received"
 ];
 function isBenignRuntimeError(error) {
   if (!(error instanceof Error)) return false;
@@ -4041,6 +4047,15 @@ function isBenignRuntimeError(error) {
   return BENIGN_MESSAGE_PATTERNS.some((pattern) => message.includes(pattern));
 }
 
+// src/utils/stdin-shutdown.ts
+function decideStdinShutdown(params) {
+  if (!params.stdinUnavailable) return "clear";
+  if (params.isConnected) return "reschedule";
+  if (!params.hasActiveSessions) return "shutdown_now";
+  if (params.elapsedMs >= params.maxWaitMs) return "shutdown_timeout";
+  return "reschedule";
+}
+
 // src/index.ts
 var STDIN_SHUTDOWN_CHECK_MS = 750;
 var STDIN_SHUTDOWN_MAX_WAIT_MS = process.platform === "win32" ? 15e3 : 1e4;
@@ -4117,19 +4132,26 @@ async function main() {
   const evaluateStdinTermination = () => {
     if (closing || stdinClosedAt === void 0) return;
     const stdinUnavailable = process.stdin.destroyed || process.stdin.readableEnded || !process.stdin.readable;
-    if (!stdinUnavailable) {
+    const elapsedMs = Date.now() - stdinClosedAt;
+    const active = hasActiveSessions();
+    const connected = server.isConnected();
+    const decision = decideStdinShutdown({
+      stdinUnavailable,
+      elapsedMs,
+      maxWaitMs: STDIN_SHUTDOWN_MAX_WAIT_MS,
+      hasActiveSessions: active,
+      isConnected: connected
+    });
+    if (decision === "clear") {
       stdinClosedAt = void 0;
       stdinClosedReason = void 0;
       return;
     }
-    const elapsedMs = Date.now() - stdinClosedAt;
-    const active = hasActiveSessions();
-    const connected = server.isConnected();
-    if (!active && !connected) {
+    if (decision === "shutdown_now") {
       void shutdown(`stdin_${stdinClosedReason ?? "closed"}`);
       return;
     }
-    if (elapsedMs >= STDIN_SHUTDOWN_MAX_WAIT_MS) {
+    if (decision === "shutdown_timeout") {
       void shutdown(`stdin_${stdinClosedReason ?? "closed"}_timeout`);
       return;
     }
@@ -4166,7 +4188,9 @@ async function main() {
     void shutdown("SIGHUP");
   });
   process.on("beforeExit", () => {
-    void shutdown("beforeExit");
+    if (!server.isConnected()) {
+      void shutdown("beforeExit");
+    }
   });
   process.on("uncaughtException", handleUnexpectedError);
   process.on("unhandledRejection", handleUnexpectedError);