@rubytech/create-maxy-code 0.1.280 → 0.1.281

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-maxy-code",
-  "version": "0.1.280",
+  "version": "0.1.281",
   "description": "Install Maxy — AI for Productive People",
   "bin": {
     "create-maxy-code": "./dist/index.js"

package/payload/platform/lib/mcp-spawn-tee/dist/index.d.ts

@@ -1,53 +1,41 @@
 #!/usr/bin/env node
 /**
- * MCP spawn-tee — parent-side stderr capture wrapper.
+ * MCP spawn-tee — parent-side stderr capture + lifecycle observability.
  *
- * Claude Code's `--mcp-config` accepts `{command, args, env}` descriptors and
- * spawns each MCP server itself; the platform never holds a ChildProcess
- * handle. The in-process `mcp-stderr-tee` patches `process.stderr.write` from
- * inside the MCP server, but its writes go through `createWriteStream` —
- * async, buffered. A synchronous module-load throw (e.g. schema-loader's
- * line-168 width check on memory) calls `process.exit(1)` before the buffer
- * flushes, so the per-server log file is empty and the platform's
- * `[mcp-init-error] tail="(no stderr file)"` probe is operationally useless.
- * That class shipped as the chronic memory-MCP silent-fail loop fixed by
- * (and hit graph — solved there with per-plugin
- * `appendFileSync` discipline that this wrapper now generalises).
+ * Claude Code spawns each MCP server itself; the platform never holds a
+ * ChildProcess handle. This wrapper sits between Claude Code and the real
+ * MCP server: Claude Code runs `node <this> <real-entry>`, and the wrapper
+ * spawns the real entry with `stdio:['inherit','inherit','pipe']` so stdin
+ * and stdout (the JSON-RPC channel) pass through byte-identical. Only stderr
+ * is intercepted, and only for logging.
  *
- * The wrapper sits between Claude Code and the real MCP server: it spawns
- * the real entry with `stdio: ['inherit', 'inherit', 'pipe']`, then
- * synchronously appends every child stderr chunk to
- * `${LOG_DIR}/mcp-${name}-stderr-<date>.log`. Synchronous writes survive
- * `process.exit` because the kernel queues the syscall before the call
- * returns. The chunk is also written to the wrapper's own stderr so
- * Claude Code's existing stderr consumer is unchanged — the mechanism is
- * additive, not interceptive.
+ * Because the wrapper IS the helper's parent, it observes the helper's true
+ * exit code, signal, and lifetime for every death mode — including a SIGKILL
+ * or a transport-layer crash that an in-process handler would miss.
  *
- *   Claude Code CLI
- *        │ spawns
- *        ▼
- *   wrapper (this file) — argv[2] = real entry, env.MCP_SPAWN_TEE_NAME = name
- *        │ spawns child with stdio:['inherit','inherit','pipe']
- *        ▼
- *   child = node <real-entry>
- *        │
- *        ├──▶ stdin/stdout: inherited from wrapper (Claude Code pipe)
- *        └──▶ stderr → wrapper.on('data', chunk =>)
- *               ├──▶ appendFileSync(${LOG_DIR}/mcp-${name}-stderr-<date>.log)
- *               └──▶ process.stderr.write(chunk)  → Claude Code consumer
+ *   Claude Code CLI → wrapper (this file) → child = node <real-entry>
+ *        child stdin/stdout : inherited (Claude Code pipe, untouched)
+ *        child stderr       : piped → per-date log + per-session log + passthrough
  *
- * The wrapper catches:
- *   - MODULE_NOT_FOUND on the entry script itself (success criterion #3:
- *     `mv plugins/memory/mcp/dist plugins/memory/mcp/dist-broken && reboot`
- *     leaves the cause on disk).
- *   - Synchronous throws during module load before the in-process tee runs.
- *   - Any stderr a normally-running plugin emits (steady-state telemetry
- *     remains visible too — the in-process tee handles per-line stream-log
- *     prefixing; this wrapper handles the raw per-server file).
+ * Destinations (Task 706):
+ *   - `${LOG_DIR}/mcp-<name>-stderr-<date>.log`  — per-date raw (back-compat;
+ *     the in-process mcp-stderr-tee and the [mcp-init-error] probe read it).
+ *   - `${LOG_DIR}/mcp-<name>-<SESSION_ID>.log`   — per-session raw + lifecycle
+ *     lines; wrapper-only, so it never mixes the enumeration copy with the
+ *     live copy. Authoritative sink. Absent when SESSION_ID is unset
+ *     (enumeration spawn) — then the per-date file is the fallback.
+ *   - `server.log` via the loopback log-ingest route — best-effort mirror of
+ *     the [mcp-helper] lifecycle lines. A dropped POST loses nothing because
+ *     the per-session file already holds the line (written synchronously).
  *
- * SIGTERM/SIGINT propagation: forwarded to the child so a Claude Code
- * shutdown does not orphan the MCP server. Child exit code is propagated
- * verbatim so upstream (Claude Code) sees the real outcome.
+ * Lifecycle lines, correlation key `session=<id8> server=<name>`:
+ *   [mcp-helper] op=spawn ... pid= entry=
+ *   [mcp-helper] op=boot  ... head=<first stderr bytes>
+ *   [mcp-helper] op=exit  ... code= signal= lifetimeMs= stderr-tail=   (always)
+ *
+ * The wrapper never writes to fd 1 (stdout) — that is the JSON-RPC channel.
+ * SIGTERM/SIGINT are forwarded to the child so a Claude Code shutdown does
+ * not orphan it; the child exit code/signal is propagated verbatim.
  */
 export {};
 //# sourceMappingURL=index.d.ts.map

package/payload/platform/lib/mcp-spawn-tee/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AACA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiDG"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AACA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG"}

package/payload/platform/lib/mcp-spawn-tee/dist/index.js

@@ -1,54 +1,42 @@
 #!/usr/bin/env node
 "use strict";
 /**
- * MCP spawn-tee — parent-side stderr capture wrapper.
+ * MCP spawn-tee — parent-side stderr capture + lifecycle observability.
  *
- * Claude Code's `--mcp-config` accepts `{command, args, env}` descriptors and
- * spawns each MCP server itself; the platform never holds a ChildProcess
- * handle. The in-process `mcp-stderr-tee` patches `process.stderr.write` from
- * inside the MCP server, but its writes go through `createWriteStream` —
- * async, buffered. A synchronous module-load throw (e.g. schema-loader's
- * line-168 width check on memory) calls `process.exit(1)` before the buffer
- * flushes, so the per-server log file is empty and the platform's
- * `[mcp-init-error] tail="(no stderr file)"` probe is operationally useless.
- * That class shipped as the chronic memory-MCP silent-fail loop fixed by
- * (and hit graph — solved there with per-plugin
- * `appendFileSync` discipline that this wrapper now generalises).
+ * Claude Code spawns each MCP server itself; the platform never holds a
+ * ChildProcess handle. This wrapper sits between Claude Code and the real
+ * MCP server: Claude Code runs `node <this> <real-entry>`, and the wrapper
+ * spawns the real entry with `stdio:['inherit','inherit','pipe']` so stdin
+ * and stdout (the JSON-RPC channel) pass through byte-identical. Only stderr
+ * is intercepted, and only for logging.
  *
- * The wrapper sits between Claude Code and the real MCP server: it spawns
- * the real entry with `stdio: ['inherit', 'inherit', 'pipe']`, then
- * synchronously appends every child stderr chunk to
- * `${LOG_DIR}/mcp-${name}-stderr-<date>.log`. Synchronous writes survive
- * `process.exit` because the kernel queues the syscall before the call
- * returns. The chunk is also written to the wrapper's own stderr so
- * Claude Code's existing stderr consumer is unchanged — the mechanism is
- * additive, not interceptive.
+ * Because the wrapper IS the helper's parent, it observes the helper's true
+ * exit code, signal, and lifetime for every death mode — including a SIGKILL
+ * or a transport-layer crash that an in-process handler would miss.
  *
- *   Claude Code CLI
- *        │ spawns
- *        ▼
- *   wrapper (this file) — argv[2] = real entry, env.MCP_SPAWN_TEE_NAME = name
- *        │ spawns child with stdio:['inherit','inherit','pipe']
- *        ▼
- *   child = node <real-entry>
- *        │
- *        ├──▶ stdin/stdout: inherited from wrapper (Claude Code pipe)
- *        └──▶ stderr → wrapper.on('data', chunk =>)
- *               ├──▶ appendFileSync(${LOG_DIR}/mcp-${name}-stderr-<date>.log)
- *               └──▶ process.stderr.write(chunk)  → Claude Code consumer
+ *   Claude Code CLI → wrapper (this file) → child = node <real-entry>
+ *        child stdin/stdout : inherited (Claude Code pipe, untouched)
+ *        child stderr       : piped → per-date log + per-session log + passthrough
  *
- * The wrapper catches:
- *   - MODULE_NOT_FOUND on the entry script itself (success criterion #3:
- *     `mv plugins/memory/mcp/dist plugins/memory/mcp/dist-broken && reboot`
- *     leaves the cause on disk).
- *   - Synchronous throws during module load before the in-process tee runs.
- *   - Any stderr a normally-running plugin emits (steady-state telemetry
- *     remains visible too — the in-process tee handles per-line stream-log
- *     prefixing; this wrapper handles the raw per-server file).
+ * Destinations (Task 706):
+ *   - `${LOG_DIR}/mcp-<name>-stderr-<date>.log`  — per-date raw (back-compat;
+ *     the in-process mcp-stderr-tee and the [mcp-init-error] probe read it).
+ *   - `${LOG_DIR}/mcp-<name>-<SESSION_ID>.log`   — per-session raw + lifecycle
+ *     lines; wrapper-only, so it never mixes the enumeration copy with the
+ *     live copy. Authoritative sink. Absent when SESSION_ID is unset
+ *     (enumeration spawn) — then the per-date file is the fallback.
+ *   - `server.log` via the loopback log-ingest route — best-effort mirror of
+ *     the [mcp-helper] lifecycle lines. A dropped POST loses nothing because
+ *     the per-session file already holds the line (written synchronously).
  *
- * SIGTERM/SIGINT propagation: forwarded to the child so a Claude Code
- * shutdown does not orphan the MCP server. Child exit code is propagated
- * verbatim so upstream (Claude Code) sees the real outcome.
+ * Lifecycle lines, correlation key `session=<id8> server=<name>`:
+ *   [mcp-helper] op=spawn ... pid= entry=
+ *   [mcp-helper] op=boot  ... head=<first stderr bytes>
+ *   [mcp-helper] op=exit  ... code= signal= lifetimeMs= stderr-tail=   (always)
+ *
+ * The wrapper never writes to fd 1 (stdout) — that is the JSON-RPC channel.
+ * SIGTERM/SIGINT are forwarded to the child so a Claude Code shutdown does
+ * not orphan it; the child exit code/signal is propagated verbatim.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 const node_child_process_1 = require("node:child_process");
@@ -56,77 +44,116 @@ const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
 const SERVER_NAME = process.env.MCP_SPAWN_TEE_NAME ?? "unknown";
 const LOG_DIR = process.env.LOG_DIR;
+const SESSION_ID = process.env.SESSION_ID;
+const PLATFORM_PORT = process.env.PLATFORM_PORT;
 const ENTRY = process.argv[2];
-// Sync-emit: appendFileSync to per-server log + stderr passthrough. Used for
-// wrapper-internal diagnostics (attach line, errors) and for forwarding child
-// stderr chunks. Each destination wrapped independently — an unwritable log
-// must not mask the primary output.
-function syncEmitLine(line) {
-    if (LOG_DIR) {
-        try {
-            (0, node_fs_1.mkdirSync)(LOG_DIR, { recursive: true });
-            const date = new Date().toISOString().slice(0, 10);
-            (0, node_fs_1.appendFileSync)((0, node_path_1.resolve)(LOG_DIR, `mcp-${SERVER_NAME}-stderr-${date}.log`), line);
-        }
-        catch {
-            /* unwritable destination — preserve primary failure */
-        }
+const SESSION_ID8 = SESSION_ID ? SESSION_ID.slice(0, 8) : "—";
+const spawnStamp = Date.now();
+// Per-session raw + lifecycle log (Task 706). Wrapper-only. Empty SESSION_ID
+// (enumeration spawn) → undefined, and the per-date file is the fallback.
+const perSessionPath = LOG_DIR && SESSION_ID
+    ? (0, node_path_1.resolve)(LOG_DIR, `mcp-${SERVER_NAME}-${SESSION_ID}.log`)
+    : undefined;
+const perDatePath = LOG_DIR
+    ? (0, node_path_1.resolve)(LOG_DIR, `mcp-${SERVER_NAME}-stderr-${new Date(spawnStamp).toISOString().slice(0, 10)}.log`)
+    : undefined;
+// Rolling tail of child stderr for op=exit. Capped so a chatty child cannot
+// grow the buffer without bound.
+const TAIL_CAP = 2048;
+let stderrTail = "";
+let bootEmitted = false;
+function appendSafe(path, data) {
+    if (!path || !LOG_DIR)
+        return;
+    try {
+        (0, node_fs_1.mkdirSync)(LOG_DIR, { recursive: true });
+        (0, node_fs_1.appendFileSync)(path, data);
+    }
+    catch {
+        /* unwritable destination — never mask primary output */
     }
+}
+// Best-effort mirror of a lifecycle line to server.log via the loopback
+// log-ingest route. Fire-and-forget: the per-session file is authoritative,
+// so a dropped POST (unreachable port, process exiting) loses nothing.
+function postToServerLog(suffix, level) {
+    if (!PLATFORM_PORT)
+        return;
     try {
-        process.stderr.write(line);
+        const ctrl = new AbortController();
+        const t = setTimeout(() => ctrl.abort(), 500);
+        t.unref?.(); // never let the abort timer keep the wrapper's event loop alive
+        void fetch(`http://127.0.0.1:${PLATFORM_PORT}/api/admin/log-ingest`, {
+            method: "POST",
+            headers: { "content-type": "application/json" },
+            body: JSON.stringify({ tag: "mcp-helper", level, line: suffix }),
+            signal: ctrl.signal,
+        }).then(() => clearTimeout(t)).catch(() => clearTimeout(t));
     }
     catch {
-        /* stderr closed — nothing else to do */
+        /* fetch threw synchronously — best-effort only */
+    }
+}
+// Emit one [mcp-helper] lifecycle line: authoritative per-session file (sync,
+// survives process.exit), the wrapper's own stderr (journald / Claude Code
+// visibility), and a best-effort server.log mirror. `suffix` is the line body
+// after the tag and carries no newline (the log-ingest route rejects newlines).
+function emitLifecycle(suffix, level) {
+    const line = `[mcp-helper] ${suffix}\n`;
+    appendSafe(perSessionPath, line);
+    try {
+        process.stderr.write(line);
     }
+    catch { /* stderr closed */ }
+    postToServerLog(suffix, level);
 }
 if (!ENTRY) {
-    syncEmitLine(`[mcp-spawn-tee-error] server=${SERVER_NAME} reason="no entry given (argv[2] missing)"\n`);
+    emitLifecycle(`op=error session=${SESSION_ID8} server=${SERVER_NAME} reason="no entry given (argv[2] missing)"`, "error");
     process.exit(2);
 }
 const child = (0, node_child_process_1.spawn)(process.execPath, [ENTRY], {
     stdio: ["inherit", "inherit", "pipe"],
     env: process.env,
 });
-syncEmitLine(`[mcp-spawn-tee-attached] server=${SERVER_NAME} pid=${child.pid ?? -1} entry=${ENTRY}\n`);
+emitLifecycle(`op=spawn session=${SESSION_ID8} server=${SERVER_NAME} pid=${child.pid ?? -1} entry=${ENTRY}`, "info");
 child.on("error", (err) => {
     const msg = err instanceof Error ? err.message : String(err);
-    syncEmitLine(`[mcp-spawn-tee-error] server=${SERVER_NAME} reason=${JSON.stringify(`spawn error: ${msg}`)}\n`);
+    emitLifecycle(`op=error session=${SESSION_ID8} server=${SERVER_NAME} reason=${JSON.stringify(`spawn error: ${msg}`)}`, "error");
     process.exit(127);
 });
 if (child.stderr) {
     child.stderr.on("data", (chunk) => {
-        if (LOG_DIR) {
-            try {
-                const date = new Date().toISOString().slice(0, 10);
-                (0, node_fs_1.appendFileSync)((0, node_path_1.resolve)(LOG_DIR, `mcp-${SERVER_NAME}-stderr-${date}.log`), chunk);
-            }
-            catch {
-                /* unwritable destination — preserve passthrough */
-            }
-        }
+        appendSafe(perDatePath, chunk); // per-date raw (back-compat)
+        appendSafe(perSessionPath, chunk); // per-session raw (Task 706)
         try {
             process.stderr.write(chunk);
         }
-        catch {
-            /* stderr closed — nothing else to do */
+        catch { /* stderr closed */ } // passthrough
+        stderrTail = (stderrTail + chunk.toString("utf8")).slice(-TAIL_CAP);
+        if (!bootEmitted) {
+            bootEmitted = true;
+            const head = chunk.toString("utf8").split("\n")[0].slice(0, 200);
+            emitLifecycle(`op=boot session=${SESSION_ID8} server=${SERVER_NAME} head=${JSON.stringify(head)}`, "info");
         }
     });
 }
-// Forward SIGTERM/SIGINT so Claude Code shutdown doesn't orphan the child.
-const forward = (signal) => {
-    if (!child.killed)
-        child.kill(signal);
-};
+const forward = (signal) => { if (!child.killed)
+    child.kill(signal); };
 process.on("SIGTERM", () => forward("SIGTERM"));
 process.on("SIGINT", () => forward("SIGINT"));
+// Signal → number, so a signal-killed child propagates as 128+signum (the
+// shell convention). `code` is null on a signal exit, so `128 + (code ?? 0)`
+// would collapse every signal to 128 and lose the signal identity in the
+// wrapper's own exit status.
+const SIGNUM = { SIGHUP: 1, SIGINT: 2, SIGQUIT: 3, SIGKILL: 9, SIGTERM: 15 };
 child.on("exit", (code, signal) => {
-    if (signal) {
-        syncEmitLine(`[mcp-spawn-tee-exit] server=${SERVER_NAME} pid=${child.pid ?? -1} signal=${signal}\n`);
-        process.exit(128 + (code ?? 0));
-    }
-    if (code !== 0) {
-        syncEmitLine(`[mcp-spawn-tee-exit] server=${SERVER_NAME} pid=${child.pid ?? -1} code=${code}\n`);
-    }
+    const lifetimeMs = Date.now() - spawnStamp;
+    const tail = stderrTail.slice(-200);
+    const level = signal || (code ?? 0) !== 0 ? "error" : "info";
+    emitLifecycle(`op=exit session=${SESSION_ID8} server=${SERVER_NAME} pid=${child.pid ?? -1} ` +
+        `code=${code ?? "—"} signal=${signal ?? "—"} lifetimeMs=${lifetimeMs} stderr-tail=${JSON.stringify(tail)}`, level);
+    if (signal)
+        process.exit(128 + (SIGNUM[signal] ?? 0));
     process.exit(code ?? 0);
 });
 //# sourceMappingURL=index.js.map

package/payload/platform/lib/mcp-spawn-tee/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;AACA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiDG;;AAEH,2DAA2C;AAC3C,qCAAoD;AACpD,yCAAoC;AAEpC,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC,kBAAkB,IAAI,SAAS,CAAC;AAChE,MAAM,OAAO,GAAG,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC;AACpC,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAE9B,6EAA6E;AAC7E,8EAA8E;AAC9E,4EAA4E;AAC5E,oCAAoC;AACpC,SAAS,YAAY,CAAC,IAAY;IAChC,IAAI,OAAO,EAAE,CAAC;QACZ,IAAI,CAAC;YACH,IAAA,mBAAS,EAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YACxC,MAAM,IAAI,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACnD,IAAA,wBAAc,EAAC,IAAA,mBAAO,EAAC,OAAO,EAAE,OAAO,WAAW,WAAW,IAAI,MAAM,CAAC,EAAE,IAAI,CAAC,CAAC;QAClF,CAAC;QAAC,MAAM,CAAC;YACP,uDAAuD;QACzD,CAAC;IACH,CAAC;IACD,IAAI,CAAC;QACH,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC7B,CAAC;IAAC,MAAM,CAAC;QACP,wCAAwC;IAC1C,CAAC;AACH,CAAC;AAED,IAAI,CAAC,KAAK,EAAE,CAAC;IACX,YAAY,CAAC,gCAAgC,WAAW,8CAA8C,CAAC,CAAC;IACxG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAClB,CAAC;AAED,MAAM,KAAK,GAAG,IAAA,0BAAK,EAAC,OAAO,CAAC,QAAQ,EAAE,CAAC,KAAK,CAAC,EAAE;IAC7C,KAAK,EAAE,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC;IACrC,GAAG,EAAE,OAAO,CAAC,GAAG;CACjB,CAAC,CAAC;AAEH,YAAY,CAAC,mCAAmC,WAAW,QAAQ,KAAK,CAAC,GAAG,IAAI,CAAC,CAAC,UAAU,KAAK,IAAI,CAAC,CAAC;AAEvG,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,GAAG,EAAE,EAAE;IACxB,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IAC7D,YAAY,CAAC,gCAAgC,WAAW,WAAW,IAAI,CAAC,SAAS,CAAC,gBAAgB,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;IAC9G,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACpB,CAAC,CAAC,CAAC;AAEH,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;IACjB,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE;QACxC,IAAI,OAAO,EAAE,CAAC;YACZ,IAAI,CAAC;gBACH,MAAM,IAAI,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;gBACnD,IAAA,wBAAc,EAAC,IAAA,mBAAO,EAAC,OAAO,EAAE,OAAO,WAAW,WAAW,IAAI,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC;YACnF,CAAC;YAAC,MAAM,CAAC;gBACP,mDAAmD;YACrD,CAAC;QACH,CAAC;QACD,IAAI,CAAC;YACH,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC9B,CAAC;QAAC,MAAM,CAAC;YACP,wCAAwC;QAC1C,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC;AAED,2EAA2E;AAC3E,MAAM,OAAO,GAAG,CAAC,MAAsB,EAAE,EAAE;IACzC,IAAI,CAAC,KAAK,CAAC,MAAM;QAAE,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AACxC,CAAC,CAAC;AACF,OAAO,CAAC,EAAE,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;AAChD,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;AAE9C,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE;IAChC,IAAI,MAAM,EAAE,CAAC;QACX,YAAY,CAAC,+BAA+B,WAAW,QAAQ,KAAK,CAAC,GAAG,IAAI,CAAC,CAAC,WAAW,MAAM,IAAI,CAAC,CAAC;QACrG,OAAO,CAAC,IAAI,CAAC,GAAG,GAAG,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC;IAClC,CAAC;IACD,IAAI,IAAI,KAAK,CAAC,EAAE,CAAC;QACf,YAAY,CAAC,+BAA+B,WAAW,QAAQ,KAAK,CAAC,GAAG,IAAI,CAAC,CAAC,SAAS,IAAI,IAAI,CAAC,CAAC;IACnG,CAAC;IACD,OAAO,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC;AAC1B,CAAC,CAAC,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;AACA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;;AAEH,2DAA2C;AAC3C,qCAAoD;AACpD,yCAAoC;AAEpC,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC,kBAAkB,IAAI,SAAS,CAAC;AAChE,MAAM,OAAO,GAAG,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC;AACpC,MAAM,UAAU,GAAG,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC;AAC1C,MAAM,aAAa,GAAG,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC;AAChD,MAAM,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAE9B,MAAM,WAAW,GAAG,UAAU,CAAC,CAAC,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;AAC9D,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;AAE9B,6EAA6E;AAC7E,0EAA0E;AAC1E,MAAM,cAAc,GAAG,OAAO,IAAI,UAAU;IAC1C,CAAC,CAAC,IAAA,mBAAO,EAAC,OAAO,EAAE,OAAO,WAAW,IAAI,UAAU,MAAM,CAAC;IAC1D,CAAC,CAAC,SAAS,CAAC;AACd,MAAM,WAAW,GAAG,OAAO;IACzB,CAAC,CAAC,IAAA,mBAAO,EAAC,OAAO,EAAE,OAAO,WAAW,WAAW,IAAI,IAAI,CAAC,UAAU,CAAC,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC;IACtG,CAAC,CAAC,SAAS,CAAC;AAEd,4EAA4E;AAC5E,iCAAiC;AACjC,MAAM,QAAQ,GAAG,IAAI,CAAC;AACtB,IAAI,UAAU,GAAG,EAAE,CAAC;AACpB,IAAI,WAAW,GAAG,KAAK,CAAC;AAExB,SAAS,UAAU,CAAC,IAAwB,EAAE,IAAyB;IACrE,IAAI,CAAC,IAAI,IAAI,CAAC,OAAO;QAAE,OAAO;IAC9B,IAAI,CAAC;QACH,IAAA,mBAAS,EAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACxC,IAAA,wBAAc,EAAC,IAAI,EAAE,IAAI,CAAC,CAAC;IAC7B,CAAC;IAAC,MAAM,CAAC;QACP,wDAAwD;IAC1D,CAAC;AACH,CAAC;AAED,wEAAwE;AACxE,4EAA4E;AAC5E,uEAAuE;AACvE,SAAS,eAAe,CAAC,MAAc,EAAE,KAAuB;IAC9D,IAAI,CAAC,aAAa;QAAE,OAAO;IAC3B,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,IAAI,eAAe,EAAE,CAAC;QACnC,MAAM,CAAC,GAAG,UAAU,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,KAAK,EAAE,EAAE,GAAG,CAAC,CAAC;QAC9C,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,gEAAgE;QAC7E,KAAK,KAAK,CAAC,oBAAoB,aAAa,uBAAuB,EAAE;YACnE,MAAM,EAAE,MAAM;YACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE;YAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,GAAG,EAAE,YAAY,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;YAChE,MAAM,EAAE,IAAI,CAAC,MAAM;SACpB,CAAC,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC;IAC9D,CAAC;IAAC,MAAM,CAAC;QACP,kDAAkD;IACpD,CAAC;AACH,CAAC;AAED,8EAA8E;AAC9E,2EAA2E;AAC3E,8EAA8E;AAC9E,gFAAgF;AAChF,SAAS,aAAa,CAAC,MAAc,EAAE,KAAuB;IAC5D,MAAM,IAAI,GAAG,gBAAgB,MAAM,IAAI,CAAC;IACxC,UAAU,CAAC,cAAc,EAAE,IAAI,CAAC,CAAC;IACjC,IAAI,CAAC;QAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,mBAAmB,CAAC,CAAC;IACjE,eAAe,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;AACjC,CAAC;AAED,IAAI,CAAC,KAAK,EAAE,CAAC;IACX,aAAa,CAAC,oBAAoB,WAAW,WAAW,WAAW,4CAA4C,EAAE,OAAO,CAAC,CAAC;IAC1H,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAClB,CAAC;AAED,MAAM,KAAK,GAAG,IAAA,0BAAK,EAAC,OAAO,CAAC,QAAQ,EAAE,CAAC,KAAK,CAAC,EAAE;IAC7C,KAAK,EAAE,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC;IACrC,GAAG,EAAE,OAAO,CAAC,GAAG;CACjB,CAAC,CAAC;AAEH,aAAa,CAAC,oBAAoB,WAAW,WAAW,WAAW,QAAQ,KAAK,CAAC,GAAG,IAAI,CAAC,CAAC,UAAU,KAAK,EAAE,EAAE,MAAM,CAAC,CAAC;AAErH,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,GAAG,EAAE,EAAE;IACxB,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IAC7D,aAAa,CAAC,oBAAoB,WAAW,WAAW,WAAW,WAAW,IAAI,CAAC,SAAS,CAAC,gBAAgB,GAAG,EAAE,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;IAChI,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACpB,CAAC,CAAC,CAAC;AAEH,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;IACjB,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE;QACxC,UAAU,CAAC,WAAW,EAAE,KAAK,CAAC,CAAC,CAAK,6BAA6B;QACjE,UAAU,CAAC,cAAc,EAAE,KAAK,CAAC,CAAC,CAAE,6BAA6B;QACjE,IAAI,CAAC;YAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAAC,CAAC;QAAC,MAAM,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,cAAc;QACjF,UAAU,GAAG,CAAC,UAAU,GAAG,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC,CAAC;QACpE,IAAI,CAAC,WAAW,EAAE,CAAC;YACjB,WAAW,GAAG,IAAI,CAAC;YACnB,MAAM,IAAI,GAAG,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;YACjE,aAAa,CAAC,mBAAmB,WAAW,WAAW,WAAW,SAAS,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,EAAE,EAAE,MAAM,CAAC,CAAC;QAC7G,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC;AAED,MAAM,OAAO,GAAG,CAAC,MAAsB,EAAE,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM;IAAE,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;AACvF,OAAO,CAAC,EAAE,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC;AAChD,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC;AAE9C,0EAA0E;AAC1E,6EAA6E;AAC7E,yEAAyE;AACzE,6BAA6B;AAC7B,MAAM,MAAM,GAA2B,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC;AAErG,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE;IAChC,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,UAAU,CAAC;IAC3C,MAAM,IAAI,GAAG,UAAU,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC;IACpC,MAAM,KAAK,GAAqB,MAAM,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC;IAC/E,aAAa,CACX,mBAAmB,WAAW,WAAW,WAAW,QAAQ,KAAK,CAAC,GAAG,IAAI,CAAC,CAAC,GAAG;QAC9E,QAAQ,IAAI,IAAI,GAAG,WAAW,MAAM,IAAI,GAAG,eAAe,UAAU,gBAAgB,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,EAAE,EAC1G,KAAK,CACN,CAAC;IACF,IAAI,MAAM;QAAE,OAAO,CAAC,IAAI,CAAC,GAAG,GAAG,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtD,OAAO,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC;AAC1B,CAAC,CAAC,CAAC"}

package/payload/platform/lib/mcp-spawn-tee/src/__tests__/spawn-tee.test.ts

@@ -0,0 +1,111 @@
+import { afterEach, beforeAll, describe, expect, it } from 'vitest'
+import { execFileSync, spawn, spawnSync } from 'node:child_process'
+import {
+  existsSync, mkdtempSync, readFileSync, readdirSync, rmSync, writeFileSync,
+} from 'node:fs'
+import { tmpdir } from 'node:os'
+import { dirname, join, resolve } from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const here = dirname(fileURLToPath(import.meta.url))
+const libRoot = resolve(here, '..', '..')                 // platform/lib/mcp-spawn-tee
+const wrapper = join(libRoot, 'dist', 'index.js')
+const tsc = resolve(libRoot, '..', '..', 'node_modules', '.bin', 'tsc')
+
+// Build the real artifact once; tests then exercise the shipped wrapper, not a
+// re-implementation — that is what catches stdio-wiring regressions.
+beforeAll(() => {
+  execFileSync(tsc, ['-p', 'tsconfig.json'], { cwd: libRoot, stdio: 'inherit' })
+  expect(existsSync(wrapper)).toBe(true)
+})
+
+let tmp = ''
+afterEach(() => { if (tmp) { rmSync(tmp, { recursive: true, force: true }); tmp = '' } })
+
+function makeStub(body: string): { stub: string; logDir: string } {
+  tmp = mkdtempSync(join(tmpdir(), 'spawn-tee-'))
+  const stub = join(tmp, 'stub-child.js')
+  writeFileSync(stub, body, 'utf8')
+  return { stub, logDir: join(tmp, 'logs') }
+}
+
+function runSync(stub: string, env: Record<string, string>) {
+  return spawnSync(process.execPath, [wrapper, stub], {
+    env: { ...process.env, SESSION_ID: '', PLATFORM_PORT: '', ...env },
+    encoding: 'utf8',
+  })
+}
+
+function sessionLog(logDir: string, server: string, sessionId: string): string {
+  return readFileSync(join(logDir, `mcp-${server}-${sessionId}.log`), 'utf8')
+}
+
+describe('mcp-spawn-tee wrapper', () => {
+  it('captures child stderr to a per-session file and emits one op=exit on clean exit', () => {
+    const { stub, logDir } = makeStub('process.stderr.write("hello-from-child\\n"); process.exit(0)')
+    runSync(stub, { LOG_DIR: logDir, SESSION_ID: 'abcd1234-session', MCP_SPAWN_TEE_NAME: 'stub' })
+    const log = sessionLog(logDir, 'stub', 'abcd1234-session')
+    expect(log).toContain('hello-from-child')
+    const exits = log.split('\n').filter((l) => l.includes('op=exit'))
+    expect(exits).toHaveLength(1)
+    expect(exits[0]).toContain('code=0')
+    expect(exits[0]).toMatch(/lifetimeMs=\d+/)
+    expect(exits[0]).toContain('stderr-tail=')
+    expect(exits[0]).toContain('hello-from-child')
+  })
+
+  it('emits op=spawn at start and op=boot on first child stderr', () => {
+    const { stub, logDir } = makeStub('process.stderr.write("[stub] boot line\\n"); process.exit(0)')
+    runSync(stub, { LOG_DIR: logDir, SESSION_ID: 'sess-0001', MCP_SPAWN_TEE_NAME: 'stub' })
+    const log = sessionLog(logDir, 'stub', 'sess-0001')
+    expect(log).toMatch(/op=spawn .*server=stub pid=\d+ entry=/)
+    expect(log).toContain('op=boot')
+    expect(log).toContain('head="[stub] boot line"')
+  })
+
+  it('op=exit carries a non-zero exit code', () => {
+    const { stub, logDir } = makeStub('process.stderr.write("dying\\n"); process.exit(3)')
+    const r = runSync(stub, { LOG_DIR: logDir, SESSION_ID: 's3', MCP_SPAWN_TEE_NAME: 'stub' })
+    expect(r.status).toBe(3)
+    const exit = sessionLog(logDir, 'stub', 's3').split('\n').find((l) => l.includes('op=exit'))!
+    expect(exit).toContain('code=3')
+    expect(exit).toContain('signal=—')
+  })
+
+  it('op=exit carries the signal name when the child is killed', async () => {
+    const { stub, logDir } = makeStub('process.stderr.write("alive\\n"); setInterval(() => {}, 1000)')
+    const cp = spawn(process.execPath, [wrapper, stub], {
+      env: { ...process.env, SESSION_ID: 'sig1', PLATFORM_PORT: '', LOG_DIR: logDir, MCP_SPAWN_TEE_NAME: 'stub' },
+    })
+    await new Promise((res) => setTimeout(res, 500))
+    cp.kill('SIGTERM')
+    const exitCode = await new Promise<number | null>((res) => cp.on('exit', (code) => res(code)))
+    const exit = sessionLog(logDir, 'stub', 'sig1').split('\n').find((l) => l.includes('op=exit'))!
+    expect(exit).toContain('signal=SIGTERM')
+    // The wrapper propagates a signal kill as 128+signum (SIGTERM=15 → 143),
+    // not a flat 128 — otherwise the signal identity is lost in the exit status.
+    expect(exitCode).toBe(143)
+  })
+
+  it('passes child stdout through untouched (MCP transport intact)', () => {
+    const { stub, logDir } = makeStub('process.stdout.write("JSONRPC-PAYLOAD"); process.exit(0)')
+    const r = runSync(stub, { LOG_DIR: logDir, SESSION_ID: 'out1', MCP_SPAWN_TEE_NAME: 'stub' })
+    expect(r.stdout).toBe('JSONRPC-PAYLOAD')
+  })
+
+  it('falls back to the per-date file when SESSION_ID is absent', () => {
+    const { stub, logDir } = makeStub('process.stderr.write("enum-boot\\n"); process.exit(0)')
+    runSync(stub, { LOG_DIR: logDir, MCP_SPAWN_TEE_NAME: 'stub' }) // SESSION_ID forced empty by runSync
+    const files = readdirSync(logDir)
+    expect(files.some((f) => /^mcp-stub-stderr-\d{4}-\d{2}-\d{2}\.log$/.test(f))).toBe(true)
+    const perDate = readFileSync(join(logDir, files.find((f) => f.includes('stderr-'))!), 'utf8')
+    expect(perDate).toContain('enum-boot')
+  })
+
+  it('does not crash when PLATFORM_PORT is unreachable', () => {
+    const { stub, logDir } = makeStub('process.stderr.write("x\\n"); process.exit(0)')
+    const r = runSync(stub, { LOG_DIR: logDir, SESSION_ID: 'p1', MCP_SPAWN_TEE_NAME: 'stub', PLATFORM_PORT: '59999' })
+    expect(r.status).toBe(0)
+    expect(sessionLog(logDir, 'stub', 'p1')).toContain('op=exit')
+  })
+})