@zibby/core 0.1.15 → 0.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zibby/core",
-  "version": "0.1.15",
+  "version": "0.1.17",
   "description": "Core test automation engine with multi-agent and multi-MCP support",
   "type": "module",
   "main": "src/index.js",
@@ -26,14 +26,13 @@
     "./package.json": "./package.json"
   },
   "scripts": {
-    "test": "vitest run",
+    "test": "vitest run --exclude '**/memory/**'",
     "test:watch": "vitest",
     "test:state-schema": "vitest run src/framework/__tests__/state-schema.test.js",
     "test:state-schema:e2e": "node src/framework/__tests__/state-schema.e2e.test.js",
     "export:workflows": "node scripts/export-default-workflows.js",
     "lint": "eslint .",
-    "lint:fix": "eslint --fix .",
-    "prepublishOnly": "npm run lint && npm test"
+    "lint:fix": "eslint --fix ."
   },
   "keywords": [
     "testing",

package/src/framework/agents/cursor-strategy.js

@@ -169,9 +169,13 @@ export class CursorAgentStrategy extends AgentStrategy {
     logger.debug(`[Agent] Executing: ${fullCmd.slice(0, 200)}`);
     logger.debug(`[Agent] Workspace: ${workspace}`);
     if (process.env.LOG_LEVEL === 'debug' || process.env.ZIBBY_LOG_CURSOR_CLI === '1') {
-      console.log(`\n🔧 Cursor CLI --model ${model || 'auto'} (from .zibby.config.js agent.cursor.model)\n`);
-      if (process.env.ZIBBY_LOG_CURSOR_CLI === '1') {
-        console.log(`   Full command: ${fullCmd}\n`);
+      try {
+        console.log(`\n🔧 Cursor CLI --model ${model || 'auto'} (from .zibby.config.js agent.cursor.model)\n`);
+        if (process.env.ZIBBY_LOG_CURSOR_CLI === '1') {
+          console.log(`   Full command: ${fullCmd}\n`);
+        }
+      } catch (err) {
+        // Ignore EPIPE errors from console.log
       }
     }
     
@@ -327,6 +331,7 @@ export class CursorAgentStrategy extends AgentStrategy {
       let lineCount = 0;
       let killed = false;
       let processStarted = false;
+      let processClosed = false;
 
       const proc = spawn(bin, args, {
         cwd,
@@ -336,58 +341,21 @@ export class CursorAgentStrategy extends AgentStrategy {
 
       logger.debug(`[Agent] PID: ${proc.pid}`);
 
-      const startupTimer = setTimeout(() => {
-        if (!processStarted && lineCount === 0) {
-          killed = true;
-          const binaryPath = bin.replace(/^"(.*)"$/, '$1');
-          const binaryExists = existsSync(binaryPath);
-          
-          logger.error(`❌ [Agent] Process failed to start within 5 seconds.`);
-          logger.error(`   Binary: ${bin}`);
-          logger.error(`   File exists: ${binaryExists ? 'Yes (but not working - may be corrupted)' : 'No'}`);
-          logger.error(`   PATH includes ~/.local/bin: ${process.env.PATH.includes('.local/bin') ? 'Yes' : 'No'}`);
-          
-          if (binaryExists) {
-            logger.error(`\n   ⚠️  Binary exists but won't start. Try reinstalling:`);
-            logger.error(`   rm -f "${binaryPath}"`);
-            logger.error(`   curl https://cursor.com/install -fsS | bash`);
-            logger.error(`   echo 'export PATH="$HOME/.local/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc`);
-          } else {
-            logger.error(`\n   Install cursor-agent:`);
-            logger.error(`   curl https://cursor.com/install -fsS | bash`);
-            logger.error(`   echo 'export PATH="$HOME/.local/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc`);
-          }
-          
-          proc.kill('SIGTERM');
-          setTimeout(() => { if (!proc.killed) proc.kill('SIGKILL'); }, 2000);
-          reject(new Error(
-            `Cursor Agent failed to start. Binary '${binaryPath}' ${binaryExists ? 'exists but is not working (corrupted?)' : 'not found'}.\n\n` +
-            `${binaryExists 
-              ? '⚠️  The binary file exists but failed to start. It may be corrupted or incomplete.\n\n' +
-                'Try reinstalling cursor-agent:\n' +
-                `  rm -f "${binaryPath}"\n` +
-                '  curl https://cursor.com/install -fsS | bash\n\n' +
-                'Ensure ~/.local/bin is in your PATH:\n' +
-                '  echo \'export PATH="$HOME/.local/bin:$PATH"\' >> ~/.zshrc\n' +
-                '  source ~/.zshrc\n\n' +
-                'Test with: agent --version'
-              : 'Install cursor-agent:\n' +
-                '  curl https://cursor.com/install -fsS | bash\n\n' +
-                'Add ~/.local/bin to your PATH:\n' +
-                '  echo \'export PATH="$HOME/.local/bin:$PATH"\' >> ~/.zshrc\n' +
-                '  source ~/.zshrc\n\n' +
-                'Test with: agent --version'
-            }`
-          ));
-        }
-      }, 5000);
-
+      // Handle stdin write errors gracefully
       if (stdinPrompt) {
-        proc.stdin.write(stdinPrompt);
-        proc.stdin.end();
-        logger.debug(`[Agent] Prompt also piped to stdin (${stdinPrompt.length} chars)`);
+        try {
+          proc.stdin.write(stdinPrompt);
+          proc.stdin.end();
+          logger.debug(`[Agent] Prompt also piped to stdin (${stdinPrompt.length} chars)`);
+        } catch (err) {
+          logger.warn(`[Agent] Failed to write to stdin: ${err.message}`);
+        }
       } else {
-        proc.stdin.end();
+        try {
+          proc.stdin.end();
+        } catch (err) {
+          // Process may have already closed
+        }
       }
 
       const modifiedFiles = new Set();
@@ -473,10 +441,16 @@ export class CursorAgentStrategy extends AgentStrategy {
         } else {
           timeline.stepTool(`Tool: ${displayName}`);
         }
-        if (displayInput != null && typeof displayInput === 'object' && Object.keys(displayInput).length > 0) {
+        if (displayInput != null && typeof displayInput === 'object' && Object.keys(displayInput).length > 0 && !processClosed) {
           const raw = JSON.stringify(displayInput);
           const preview = raw.length > 100 ? `${raw.substring(0, 100)}...` : raw;
-          console.log(`  Input: ${preview}`);
+          try {
+            console.log(`  Input: ${preview}`);
+          } catch (err) {
+            if (err.code !== 'EPIPE') {
+              logger.warn(`[Agent] console.log error: ${err.message}`);
+            }
+          }
         }
       };
       
@@ -487,12 +461,18 @@ export class CursorAgentStrategy extends AgentStrategy {
         
         if (!processStarted) {
           processStarted = true;
-          clearTimeout(startupTimer);
         }
         
         const displayText = streamParser.processChunk(chunk);
-        if (displayText) {
-          process.stdout.write(displayText);
+        if (displayText && !processClosed) {
+          try {
+            process.stdout.write(displayText);
+          } catch (err) {
+            // Handle EPIPE gracefully - parent process stdout may be closed
+            if (err.code !== 'EPIPE') {
+              logger.warn(`[Agent] stdout write error: ${err.message}`);
+            }
+          }
         }
         
         const lines = chunk.split('\n').filter(l => l.trim());
@@ -506,7 +486,6 @@ export class CursorAgentStrategy extends AgentStrategy {
         
         if (!processStarted) {
           processStarted = true;
-          clearTimeout(startupTimer);
         }
         
         const lines = chunk.split('\n').filter(l => l.trim());
@@ -516,8 +495,8 @@ export class CursorAgentStrategy extends AgentStrategy {
       });
 
       proc.on('close', (code, signal) => {
+        processClosed = true;
         clearTimeout(timer);
-        clearTimeout(startupTimer);
         clearInterval(heartbeat);
         streamParser.flush();
         const elapsed = Math.round((Date.now() - startTime) / 1000);
@@ -551,7 +530,6 @@ export class CursorAgentStrategy extends AgentStrategy {
 
       proc.on('error', (err) => {
         clearTimeout(timer);
-        clearTimeout(startupTimer);
         clearInterval(heartbeat);
         reject(new Error(
           `Cursor Agent spawn error: ${err.message}\n` +