@cjhyy/code-shell-capability-coding 0.9.8 → 0.9.9

package/dist/external-runtimes/codex/app-server-client.js

@@ -24,6 +24,10 @@ import { spawn } from "node:child_process";
 import { createInterface } from "node:readline";
 /** Cap per NDJSON line. Generous for reasoning text; guards parse latency. */
 const MAX_LINE_BYTES = 16 * 1024 * 1024;
+/** Keep launch diagnostics useful without copying unbounded process metadata. */
+function diagnosticText(value) {
+    return value.replace(/[\r\n\t]/g, " ").slice(0, 300);
+}
 export class CodexAppServerClient {
     options;
     child;
@@ -79,9 +83,24 @@ export class CodexAppServerClient {
             if (text)
                 this.log("appserver.stderr", { bytes: text.length, preview: text.slice(0, 300) });
         });
-        const onGone = (why) => () => this.failAll(why);
-        this.child.on("exit", onGone("app-server exited"));
-        this.child.on("error", onGone("app-server failed to start"));
+        // A GUI host can have a different PATH from the user's terminal. Preserve
+        // the OS cause so a missing executable/cwd is distinguishable from a server
+        // that launched and exited. Never copy spawnargs or the environment: both
+        // can contain credentials for the runtime's MCP bridge.
+        const launch = {
+            command: diagnosticText(command),
+            cwd: diagnosticText(this.options.cwd ?? process.cwd()),
+        };
+        this.child.on("exit", (code, signal) => {
+            const status = signal ? `signal ${signal}` : `code ${code ?? "unknown"}`;
+            this.failAll(`app-server exited (${status})`, { ...launch, code, signal });
+        });
+        this.child.on("error", (error) => {
+            const code = error.code ? diagnosticText(error.code) : undefined;
+            const message = diagnosticText(error.message);
+            this.failAll(`app-server failed to start${code ? ` (${code})` : ""}: ${message}; ` +
+                `command=${JSON.stringify(launch.command)}, cwd=${JSON.stringify(launch.cwd)}`, { ...launch, code, error: message });
+        });
         // Handlers are registered by contract before start(); release the buffer.
         this.handlersReady = true;
         for (const line of this.preHandlerLines.splice(0))
@@ -203,12 +222,12 @@ export class CodexAppServerClient {
             return;
         child.stdin.write(`${JSON.stringify(payload)}\n`);
     }
-    failAll(reason) {
+    failAll(reason, details = {}) {
         if (this.closed)
             return;
         this.closed = true;
         this.closeReason = reason;
-        this.log("appserver.closed", { reason });
+        this.log("appserver.closed", { ...details, reason });
         for (const [, entry] of this.pending) {
             if (entry.timer)
                 clearTimeout(entry.timer);

package/dist/external-runtimes/codex/runtime.d.ts

@@ -113,6 +113,7 @@ export declare class CodexRuntime {
     interrupt(): Promise<void>;
     close(): Promise<void>;
     private onNotification;
+    private emitEvent;
     /**
      * Codex's native-tool approvals. CodeShell Host Tools never arrive here —
      * they come in over the MCP bridge and are authorized by `ToolExecutor`, which

package/dist/external-runtimes/codex/runtime.js

@@ -105,6 +105,14 @@ export class CodexRuntime {
             threadIdPrefix: threadId.slice(0, 8),
             resumed: this.resumed,
         });
+        // Announce the host identity after a successful start/resume. Codex may
+        // deliver thread/started before this RPC settles (and before the translator
+        // exists), so that notification cannot reliably establish the UI binding.
+        this.emitEvent({
+            type: "session_started",
+            sessionId: this.options.businessSessionId,
+            promptTokens: 0,
+        });
     }
     /**
      * Run one turn. Resolves when the turn is terminal.
@@ -184,20 +192,23 @@ export class CodexRuntime {
     onNotification(method, params) {
         const events = this.translator?.translate({ method, params }) ?? [];
         for (const event of events) {
-            try {
-                this.hooks.onEvent?.(event);
-            }
-            catch (error) {
-                this.log("runtime.event_handler_failed", {
-                    error: error instanceof Error ? error.name : "unknown",
-                });
-            }
+            this.emitEvent(event);
             if (event.type === "turn_complete") {
                 this.activeTurn?.resolve();
                 this.activeTurn = undefined;
             }
         }
     }
+    emitEvent(event) {
+        try {
+            this.hooks.onEvent?.(event);
+        }
+        catch (error) {
+            this.log("runtime.event_handler_failed", {
+                error: error instanceof Error ? error.name : "unknown",
+            });
+        }
+    }
     /**
      * Codex's native-tool approvals. CodeShell Host Tools never arrive here —
      * they come in over the MCP bridge and are authorized by `ToolExecutor`, which

package/dist/external-runtimes/session-factory.js

@@ -78,8 +78,10 @@ export async function startExternalRuntimeSession(options) {
                 ...(options.codexClient ? { client: options.codexClient } : {}),
                 log,
             }, options.hooks ?? {});
-            await codex.start();
+            // Own the process before its handshake can fail, so the catch path also
+            // closes an app-server that launched but could not initialize a thread.
             runtime = codex;
+            await codex.start();
         }
         else {
             runtime = new ClaudeCodeRuntime({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cjhyy/code-shell-capability-coding",
-  "version": "0.9.8",
+  "version": "0.9.9",
   "description": "Coding capability pack for the generic code-shell agent core.",
   "type": "module",
   "main": "./dist/index.js",
@@ -39,7 +39,7 @@
     "clean": "node -e \"require('node:fs').rmSync('dist',{recursive:true,force:true})\""
   },
   "dependencies": {
-    "@cjhyy/code-shell-core": "0.9.8"
+    "@cjhyy/code-shell-core": "0.9.9"
   },
   "engines": {
     "node": ">=20.10"