scripts-orchestrator 2.3.0 → 2.4.1

package/lib/orchestrator.js

@@ -225,6 +225,7 @@ export class Orchestrator {
   async run() {
     try {
       let hasFailures = false;
+      let phaseFailed = false;
 
       // Handle both old array format and new phases format
       if (Array.isArray(this.config)) {
@@ -237,6 +238,15 @@ export class Orchestrator {
       } else if (this.config.phases) {
         // New: Run phases sequentially, commands within phases in parallel
         for (const phase of this.config.phases) {
+          if (phaseFailed) {
+            // Mark all commands in remaining phases as skipped
+            phase.parallel.forEach(({ command }) => {
+              this.skippedCommands.push(command);
+              this.commandTimings.set(command, 0);
+            });
+            continue;
+          }
+
           this.logger.info(`\n🔄 Starting phase: ${phase.name}`);
           
           const tasks = phase.parallel.map((commandConfig) =>
@@ -248,8 +258,8 @@ export class Orchestrator {
           
           if (phaseHasFailures) {
             hasFailures = true;
+            phaseFailed = true;
             this.logger.error(`❌ Phase "${phase.name}" completed with failures`);
-            break; // Stop executing remaining phases on failure
           } else {
             this.logger.success(`✅ Phase "${phase.name}" completed successfully`);
           }

package/lib/process-manager.js

@@ -76,6 +76,16 @@ export class ProcessManager {
           const processGroupId = processInstance.pid;
           this.logger.verbose(`Background process spawned with PID: ${processGroupId}`);
 
+          // Track process exit for background processes
+          let processExited = false;
+          let processExitCode = null;
+          
+          processInstance.on('exit', (code, signal) => {
+            processExited = true;
+            processExitCode = code;
+            this.logger.verbose(`Background process ${cmd} (PID: ${processGroupId}) exited with code: ${code}, signal: ${signal}`);
+          });
+
           processInstance.stdout.on('data', (data) => {
             try {
               fs.appendFileSync(LOG_FILE, data.toString());
@@ -98,10 +108,39 @@ export class ProcessManager {
 
             for (let attempt = 1; attempt <= maxAttempts; attempt++) {
               try {
+                // First check if the process has already exited with an error
+                if (processExited && processExitCode !== 0) {
+                  this.logger.error(`Background process ${cmd} exited with code ${processExitCode}`);
+                  let output = '';
+                  try {
+                    output = fs.readFileSync(LOG_FILE, 'utf8');
+                    this.logger.verbose(`Process output: ${output}`);
+                  } catch (error) {
+                    this.logger.error(`Failed to read log file: ${error.message}`);
+                  }
+                  return { success: false, output };
+                }
+                
                 this.logger.verbose(`Verifying process ${processGroupId} (attempt ${attempt}/${maxAttempts})`);
                 process.kill(processGroupId, 0);
                 this.logger.verbose(`Process ${processGroupId} is running`);
                 
+                // Wait a bit more to ensure the process doesn't exit immediately after verification
+                await new Promise((resolve) => setTimeout(resolve, 500));
+                
+                // Check again if the process exited during our wait
+                if (processExited && processExitCode !== 0) {
+                  this.logger.error(`Background process ${cmd} exited with code ${processExitCode} shortly after starting`);
+                  let output = '';
+                  try {
+                    output = fs.readFileSync(LOG_FILE, 'utf8');
+                    this.logger.verbose(`Process output: ${output}`);
+                  } catch (error) {
+                    this.logger.error(`Failed to read log file: ${error.message}`);
+                  }
+                  return { success: false, output };
+                }
+                
                 this.backgroundProcesses.push(processGroupId);
                 this.backgroundProcessesDetails.push({
                   command: cmd,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scripts-orchestrator",
-  "version": "2.3.0",
+  "version": "2.4.1",
   "description": "A powerful script orchestrator for running parallel commands with dependency management, background processes, and health checks",
   "main": "lib/index.js",
   "type": "module",