@shepai/cli 1.194.1 → 1.194.2-pr584.4f4957c

package/dist/packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"claude-code-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAajD;;;GAGG;AACH,qBAAa,yBAA0B,YAAW,cAAc;IAMlD,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;IAE3D,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAEM,KAAK,EAAE,aAAa;IAEjD,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAML,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAoHtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAgG3C;;;OAGG;IACH,OAAO,CAAC,cAAc;IA0CtB,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAI/C,OAAO,CAAC,SAAS;IAejB,OAAO,CAAC,eAAe;IAUvB,OAAO,CAAC,iBAAiB;IAyBzB,OAAO,CAAC,eAAe;IA6BvB;;;OAGG;IACH,OAAO,CAAC,YAAY;IAqBpB,OAAO,CAAC,eAAe;CAkExB"}
+{"version":3,"file":"claude-code-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAwBjD;;;GAGG;AACH,qBAAa,yBAA0B,YAAW,cAAc;IAMlD,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;IAE3D,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAEM,KAAK,EAAE,aAAa;IAEjD,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAML,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAsItF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAgG3C;;;OAGG;IACH,OAAO,CAAC,cAAc;IA0CtB,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAI/C,OAAO,CAAC,SAAS;IAejB,OAAO,CAAC,eAAe;IAUvB,OAAO,CAAC,iBAAiB;IAyBzB,OAAO,CAAC,eAAe;IA6BvB;;;OAGG;IACH,OAAO,CAAC,YAAY;IAqBpB,OAAO,CAAC,eAAe;CAkExB"}

package/dist/packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.js

@@ -18,6 +18,16 @@ const SUPPORTED_FEATURES = new Set([
     'structured-output',
     'session-listing',
 ]);
+/**
+ * Maximum time to wait for the `claude` subprocess to exit on its own
+ * after it has emitted its final `result` event. The CLI is supposed to
+ * tear down its MCP servers and exit, but in practice it can hang
+ * indefinitely if MCP children (e.g. Playwright) or background bash tools
+ * (e.g. `pnpm dev:web`) keep stdio open. After this grace period we send
+ * SIGKILL so the close handler fires and execute() resolves with the
+ * captured result instead of leaving the worker stuck for hours.
+ */
+const RESULT_TO_CLOSE_GRACE_MS = 30_000;
 /**
  * Executor service for Claude Code agent.
  * Uses subprocess spawning to interact with the `claude` CLI.
@@ -59,6 +69,7 @@ export class ClaudeCodeExecutorService {
             let stderr = '';
             let timedOut = false;
             let timeoutId;
+            let postResultKillTimer;
             // Collected from the stream — only the final result line matters
             let resultText = '';
             let sessionId;
@@ -82,6 +93,20 @@ export class ClaudeCodeExecutorService {
                         const { type: _t, result: _r, session_id: _s, usage: _u, ...rest } = parsed;
                         if (Object.keys(rest).length > 0)
                             metadata = rest;
+                        // The CLI emitted its final result. Give it a grace period to tear
+                        // down MCP servers and exit on its own; if it still hasn't closed,
+                        // send SIGKILL so the close handler runs and we resolve. Without
+                        // this, leaked MCP/background children (issue: feature 92701aa8
+                        // hung 3+ hours after [result]) trap the worker forever.
+                        postResultKillTimer ??= setTimeout(() => {
+                            this.log(`Subprocess did not exit within ${RESULT_TO_CLOSE_GRACE_MS / 1000}s of [result] — sending SIGKILL`);
+                            try {
+                                proc.kill('SIGKILL');
+                            }
+                            catch {
+                                /* already dead — close handler will resolve */
+                            }
+                        }, RESULT_TO_CLOSE_GRACE_MS);
                     }
                 }
                 catch {
@@ -121,6 +146,8 @@ export class ClaudeCodeExecutorService {
                 this.log(`Process closed with code ${code}, result=${resultText.length} chars`);
                 if (timeoutId)
                     clearTimeout(timeoutId);
+                if (postResultKillTimer)
+                    clearTimeout(postResultKillTimer);
                 if (timedOut) {
                     reject(new Error('Agent execution timed out'));
                     return;