projax 3.3.58 → 3.3.59

package/dist/electron/script-runner.js

@@ -51,6 +51,38 @@ const os = __importStar(require("os"));
 const child_process_1 = require("child_process");
 const core_bridge_1 = require("./core-bridge");
 const port_utils_1 = require("./port-utils");
+/**
+ * Safe console logging that handles EPIPE errors gracefully
+ * This prevents uncaught exceptions when stdout/stderr are closed
+ */
+function safeLog(...args) {
+    try {
+        console.log(...args);
+    }
+    catch (error) {
+        // Silently ignore EPIPE errors (pipe closed)
+        if (error?.code !== 'EPIPE') {
+            // For other errors, try to write to stderr as fallback
+            try {
+                process.stderr.write(`[log] ${args.join(' ')}\n`);
+            }
+            catch {
+                // Give up - output streams are not available
+            }
+        }
+    }
+}
+function safeError(...args) {
+    try {
+        console.error(...args);
+    }
+    catch (error) {
+        // Silently ignore EPIPE errors
+        if (error?.code !== 'EPIPE') {
+            // No fallback for stderr errors
+        }
+    }
+}
 /**
  * Detect the project type based on files in the directory
  */
@@ -278,19 +310,19 @@ function getProjectScripts(projectPath) {
 async function handlePortConflict(port, projectName, force) {
     const processInfo = await (0, port_utils_1.getProcessOnPort)(port);
     if (!processInfo) {
-        console.error(`Port ${port} appears to be in use, but couldn't identify the process.`);
+        safeError(`Port ${port} appears to be in use, but couldn't identify the process.`);
         return false;
     }
-    console.error(`\n⚠️  Port ${port} is already in use by process ${processInfo.pid} (${processInfo.command})`);
+    safeError(`\n⚠️  Port ${port} is already in use by process ${processInfo.pid} (${processInfo.command})`);
     if (force) {
-        console.log(`Killing process ${processInfo.pid} on port ${port}...`);
+        safeLog(`Killing process ${processInfo.pid} on port ${port}...`);
         const killed = await (0, port_utils_1.killProcessOnPort)(port);
         if (killed) {
-            console.log(`✓ Process killed. Retrying...\n`);
+            safeLog(`✓ Process killed. Retrying...\n`);
             return true;
         }
         else {
-            console.error(`Failed to kill process on port ${port}`);
+            safeError(`Failed to kill process on port ${port}`);
             return false;
         }
     }
@@ -307,16 +339,16 @@ async function handlePortConflict(port, projectName, force) {
         if (answer.kill) {
             const killed = await (0, port_utils_1.killProcessOnPort)(port);
             if (killed) {
-                console.log(`✓ Process killed. Retrying...\n`);
+                safeLog(`✓ Process killed. Retrying...\n`);
                 return true;
             }
             else {
-                console.error(`Failed to kill process on port ${port}`);
+                safeError(`Failed to kill process on port ${port}`);
                 return false;
             }
         }
         else {
-            console.log('Cancelled.');
+            safeLog('Cancelled.');
             return false;
         }
     }
@@ -416,8 +448,8 @@ function runScript(projectPath, scriptName, args = [], force = false) {
                 commandArgs = [...parts.slice(1), ...args];
                 break;
         }
-        console.log(`Running: ${command} ${commandArgs.join(' ')}`);
-        console.log(`In directory: ${projectPath}\n`);
+        safeLog(`Running: ${command} ${commandArgs.join(' ')}`);
+        safeLog(`In directory: ${projectPath}\n`);
         // Capture stderr for reactive port conflict detection
         let stderrOutput = '';
         let stdoutOutput = '';
@@ -681,7 +713,7 @@ async function stopScript(pid) {
             }
             catch (error) {
                 // Process may have already exited
-                console.error(`Error killing process ${pid}:`, error);
+                safeError(`Error killing process ${pid}:`, error);
             }
         }
         // Remove from tracking regardless of whether kill succeeded
@@ -689,7 +721,7 @@ async function stopScript(pid) {
         return true;
     }
     catch (error) {
-        console.error(`Error stopping script ${pid}:`, error);
+        safeError(`Error stopping script ${pid}:`, error);
         return false;
     }
 }
@@ -869,9 +901,9 @@ function runScriptInBackground(projectPath, projectName, scriptName, args = [],
         // Unref so parent can exit and process runs independently
         child.unref();
         // Show minimal output
-        console.log(`✓ Started "${projectName}" (${scriptName}) in background [PID: ${child.pid}]`);
-        console.log(`  Logs: ${logFile}`);
-        console.log(`  Command: ${command} ${commandArgs.join(' ')}\n`);
+        safeLog(`✓ Started "${projectName}" (${scriptName}) in background [PID: ${child.pid}]`);
+        safeLog(`  Logs: ${logFile}`);
+        safeLog(`  Command: ${command} ${commandArgs.join(' ')}\n`);
         // For background processes, we can't easily do reactive detection
         // But we can check the log file after a short delay for port conflicts and URLs
         setTimeout(async () => {
@@ -883,9 +915,9 @@ function runScriptInBackground(projectPath, projectName, scriptName, args = [],
                     // Check for port conflicts
                     const port = (0, port_utils_1.extractPortFromError)(logContent);
                     if (port) {
-                        console.error(`\n⚠️  Port conflict detected in background process: port ${port} is in use`);
-                        console.error(`   Check log file: ${logFile}`);
-                        console.error(`   Use: prx <project> <script> --force to auto-resolve port conflicts\n`);
+                        safeError(`\n⚠️  Port conflict detected in background process: port ${port} is in use`);
+                        safeError(`   Check log file: ${logFile}`);
+                        safeError(`   Use: prx <project> <script> --force to auto-resolve port conflicts\n`);
                     }
                     // Extract URLs from output
                     const urls = extractUrlsFromOutput(logContent);
@@ -986,10 +1018,10 @@ async function checkAndParseTestResults(logFile, projectPath, scriptName) {
         // Save test results to database
         db.addTestResult(project.id, scriptName, parsed.passed, parsed.failed, parsed.skipped, parsed.total, parsed.duration, parsed.coverage, parsed.framework, logContent.slice(-5000) // Store last 5000 chars of output
         );
-        console.log(`\n📊 Test results saved: ${parsed.passed} passed, ${parsed.failed} failed, ${parsed.skipped} skipped`);
+        safeLog(`\n📊 Test results saved: ${parsed.passed} passed, ${parsed.failed} failed, ${parsed.skipped} skipped`);
     }
     catch (error) {
         // Silently fail - test parsing is optional
-        console.error('Error parsing test results:', error);
+        safeError('Error parsing test results:', error);
     }
 }

package/dist/index.js

@@ -461,7 +461,7 @@ program
             path.join(__dirname, '..', '..', '..', 'node_modules', '.bin', 'tsx'), // When running from workspace root
             'tsx', // Fallback to PATH
         ];
-        let tsxBin = tsxPaths.find(p => p === 'tsx' || fs.existsSync(p));
+        const tsxBin = tsxPaths.find(p => p === 'tsx' || fs.existsSync(p));
         if (!tsxBin) {
             console.error('Error: tsx not found. Please install dependencies: npm install');
             process.exit(1);