@sylphx/flow 3.24.1 → 3.24.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @sylphx/flow
 
+## 3.24.2 (2026-02-17)
+
+### 🐛 Bug Fixes
+
+- **flow:** graceful exit when Claude Code exits with non-zero code ([44892b5](https://github.com/SylphxAI/flow/commit/44892b51beda34b49020ed1d0324cfcc3e4b28f9))
+
 ## 3.24.1 (2026-02-15)
 
 ### 🐛 Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@sylphx/flow",
-	"version": "3.24.1",
+	"version": "3.24.2",
 	"description": "One CLI to rule them all. Unified orchestration layer for AI coding assistants. Auto-detection, auto-installation, auto-upgrade.",
 	"type": "module",
 	"bin": {

package/src/commands/flow/execute-v2.ts

@@ -19,7 +19,7 @@ import { TargetInstaller } from '../../services/target-installer.js';
 import type { RunCommandOptions } from '../../types.js';
 import { extractAgentInstructions, loadAgentContent } from '../../utils/agent-enhancer.js';
 import { showAttachSummary, showHeader } from '../../utils/display/banner.js';
-import { CLIError, UserCancelledError } from '../../utils/errors.js';
+import { CLIError, ChildProcessExitError, UserCancelledError } from '../../utils/errors.js';
 import { log, promptConfirm, promptSelect } from '../../utils/prompts/index.js';
 import { ensureTargetInstalled, promptForTargetSelection } from '../../utils/target-selection.js';
 import { resolvePrompt } from './prompt.js';
@@ -340,6 +340,17 @@ export async function executeFlowV2(
       process.exit(0);
     }
 
+    // Child process already displayed its own error via inherited stdio —
+    // cleanup silently and exit with the same code.
+    if (error instanceof ChildProcessExitError) {
+      try {
+        await executor.cleanup(projectPath);
+      } catch (cleanupError) {
+        debug('cleanup after child exit failed:', cleanupError);
+      }
+      process.exit(error.exitCode);
+    }
+
     console.error(chalk.red('\n  Error:'), error);
 
     try {

package/src/targets/claude-code.ts

@@ -11,7 +11,7 @@ import {
   pathUtils,
   yamlUtils,
 } from '../utils/config/target-utils.js';
-import { CLIError } from '../utils/errors.js';
+import { CLIError, ChildProcessExitError } from '../utils/errors.js';
 import { sanitize } from '../utils/security/security.js';
 import { DEFAULT_CLAUDE_CODE_ENV } from './functional/claude-code-logic.js';
 import {
@@ -32,11 +32,6 @@ interface ClaudeCodeAgentMetadata {
   model?: string;
 }
 
-/** Error with exit code from child process */
-interface ProcessExitError extends Error {
-  code: number | null;
-}
-
 /** Type guard for Node.js errors with errno/code properties */
 function isNodeError(error: unknown): error is NodeJS.ErrnoException {
   return error instanceof Error && 'code' in error;
@@ -299,9 +294,9 @@ Please begin your response with a comprehensive summary of all the instructions
           if (code === 0) {
             resolve();
           } else {
-            const error = new Error(`Claude Code exited with code ${code}`) as ProcessExitError;
-            error.code = code;
-            reject(error);
+            // Child already displayed its error via inherited stdio — use
+            // ChildProcessExitError so upstream handlers exit silently.
+            reject(new ChildProcessExitError(code ?? 1));
           }
         });
 
@@ -313,13 +308,15 @@ Please begin your response with a comprehensive summary of all the instructions
         });
       });
     } catch (error: unknown) {
+      // ChildProcessExitError means the child already printed its own error —
+      // let it propagate unmodified so upstream can exit silently.
+      if (error instanceof ChildProcessExitError) {
+        throw error;
+      }
       if (isNodeError(error)) {
         if (error.code === 'ENOENT') {
           throw new CLIError('Claude Code not found. Please install it first.', 'CLAUDE_NOT_FOUND');
         }
-        if (error.code !== undefined) {
-          throw new CLIError(`Claude Code exited with code ${error.code}`, 'CLAUDE_ERROR');
-        }
         throw new CLIError(`Failed to execute Claude Code: ${error.message}`, 'CLAUDE_ERROR');
       }
       if (error instanceof Error) {

package/src/utils/errors.ts

@@ -20,3 +20,15 @@ export class CLIError extends Error {
     this.name = 'CLIError';
   }
 }
+
+/**
+ * Child process exited with non-zero code.
+ * Since stdio is inherited, the child already displayed its own error —
+ * upstream handlers should cleanup and exit silently without printing anything.
+ */
+export class ChildProcessExitError extends Error {
+  constructor(public exitCode: number) {
+    super(`Child process exited with code ${exitCode}`);
+    this.name = 'ChildProcessExitError';
+  }
+}