@zibby/core 0.1.11 → 0.1.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zibby/core",
-  "version": "0.1.11",
+  "version": "0.1.15",
   "description": "Core test automation engine with multi-agent and multi-MCP support",
   "type": "module",
   "main": "src/index.js",

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

@@ -23,10 +23,15 @@ export class CursorAgentStrategy extends AgentStrategy {
 
   canHandle(_context) {
     const paths = [
-      'agent',
-      '/usr/local/bin/agent',
+      // Try absolute paths first (most reliable)
+      join(homedir(), '.local', 'bin', 'cursor-agent'),
+      join(homedir(), '.cursor', 'bin', 'cursor-agent'),
       '/usr/local/bin/cursor-agent',
-      '/Applications/Cursor.app/Contents/Resources/app/bin/cursor'
+      '/usr/local/bin/agent',
+      '/Applications/Cursor.app/Contents/Resources/app/bin/cursor',
+      // Try PATH last (may have symlink issues)
+      'agent',
+      'cursor-agent'
     ];
     
     for (const path of paths) {
@@ -81,10 +86,15 @@ export class CursorAgentStrategy extends AgentStrategy {
     this._setupMcpConfig(sessionPath, workspace, config);
 
     const possibleBins = [
-      'agent',
-      '/usr/local/bin/agent',
+      // Try absolute paths first (most reliable)
+      join(homedir(), '.local', 'bin', 'cursor-agent'),
+      join(homedir(), '.cursor', 'bin', 'cursor-agent'),
       '/usr/local/bin/cursor-agent',
-      '/Applications/Cursor.app/Contents/Resources/app/bin/cursor'
+      '/usr/local/bin/agent',
+      '/Applications/Cursor.app/Contents/Resources/app/bin/cursor',
+      // Try PATH last (may have symlink issues)
+      'agent',
+      'cursor-agent'
     ];
     
     let cursorBin = null;
@@ -113,11 +123,14 @@ export class CursorAgentStrategy extends AgentStrategy {
     
     if (!cursorBin) {
       throw new Error(
-        'Cursor Agent CLI not found or not working.\n\n' +
-        'Install it with:\n' +
-        '  npm install -g @cursor/agent\n\n' +
-        'Or set CURSOR_API_KEY and ensure "agent" is in your PATH.\n' +
-        'Test with: agent --version'
+        `Cursor Agent CLI not found or not working.\n\n` +
+        `Checked paths:\n` +
+        `${possibleBins.map(p => `  - ${p}`).join('\n')}\n\n` +
+        `Install cursor-agent:\n` +
+        `  curl https://cursor.com/install -fsS | bash\n\n` +
+        `Then add to PATH:\n` +
+        `  echo 'export PATH="$HOME/.local/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc\n\n` +
+        `Test with: agent --version`
       );
     }
     
@@ -138,28 +151,19 @@ export class CursorAgentStrategy extends AgentStrategy {
     writeFileSync(promptFile, prompt, 'utf-8');
     logger.debug(`📝 [Agent] Prompt written to ${promptFile} (${prompt.length} chars)`);
     
-    let args;
-    if (cursorBin === 'agent') {
-      args = [
-        'chat', 
-        '--print', 
-        '--force', 
-        '--approve-mcps',
-        '--output-format', 'stream-json',
-        '--stream-partial-output',
-        '--model', model || 'auto', 
-      ];
-      if (process.env.CURSOR_API_KEY) {
-        args.push('--api-key', process.env.CURSOR_API_KEY);
-      }
-      args.push(`@${promptFile}`);
-    } else {
-      args = ['agent', '--print', '--force', '--trust', '--approve-mcps'];
-      if (process.env.CURSOR_API_KEY) {
-        args.push('--api-key', process.env.CURSOR_API_KEY);
-      }
-      args.push('--model', model || 'auto', `@${promptFile}`);
+    // All cursor-agent binaries use the same command structure (no subcommand needed)
+    const args = [
+      '--print', 
+      '--force', 
+      '--approve-mcps',
+      '--output-format', 'stream-json',
+      '--stream-partial-output',
+      '--model', model || 'auto', 
+    ];
+    if (process.env.CURSOR_API_KEY) {
+      args.push('--api-key', process.env.CURSOR_API_KEY);
     }
+    args.push(`@${promptFile}`);
     
     const fullCmd = [cursorBin, ...args].join(' ');
     logger.debug(`[Agent] Executing: ${fullCmd.slice(0, 200)}`);
@@ -322,6 +326,7 @@ export class CursorAgentStrategy extends AgentStrategy {
       let lastOutputTime = Date.now();
       let lineCount = 0;
       let killed = false;
+      let processStarted = false;
 
       const proc = spawn(bin, args, {
         cwd,
@@ -331,6 +336,52 @@ 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);
+
       if (stdinPrompt) {
         proc.stdin.write(stdinPrompt);
         proc.stdin.end();
@@ -434,6 +485,11 @@ export class CursorAgentStrategy extends AgentStrategy {
         stdout += chunk;
         lastOutputTime = Date.now();
         
+        if (!processStarted) {
+          processStarted = true;
+          clearTimeout(startupTimer);
+        }
+        
         const displayText = streamParser.processChunk(chunk);
         if (displayText) {
           process.stdout.write(displayText);
@@ -448,6 +504,11 @@ export class CursorAgentStrategy extends AgentStrategy {
         stderr += chunk;
         lastOutputTime = Date.now();
         
+        if (!processStarted) {
+          processStarted = true;
+          clearTimeout(startupTimer);
+        }
+        
         const lines = chunk.split('\n').filter(l => l.trim());
         for (const line of lines) {
           logger.warn(`⚠️  [Agent stderr] ${line}`);
@@ -456,6 +517,7 @@ export class CursorAgentStrategy extends AgentStrategy {
 
       proc.on('close', (code, signal) => {
         clearTimeout(timer);
+        clearTimeout(startupTimer);
         clearInterval(heartbeat);
         streamParser.flush();
         const elapsed = Math.round((Date.now() - startTime) / 1000);
@@ -489,8 +551,14 @@ export class CursorAgentStrategy extends AgentStrategy {
 
       proc.on('error', (err) => {
         clearTimeout(timer);
+        clearTimeout(startupTimer);
         clearInterval(heartbeat);
-        reject(new Error(`Cursor Agent spawn error: ${err.message}`));
+        reject(new Error(
+          `Cursor Agent spawn error: ${err.message}\n` +
+          `Binary: ${bin}\n` +
+          `This usually means the binary is not in PATH. Try:\n` +
+          `  echo 'export PATH="$HOME/.local/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc`
+        ));
       });
     });
   }

package/templates/browser-test-automation/graph.mjs

@@ -47,13 +47,12 @@ export class BrowserTestAutomationAgent extends WorkflowAgent {
     BrowserTestResultHandler.saveTitle(result, cwd);
     await BrowserTestResultHandler.saveExecutionData(result);
 
-    if (process.env.ZIBBY_MEMORY) {
-      try {
-        const { memoryEndRun, memorySyncPush } = await import('@zibby/memory');
-        const sessionId = result.state.sessionPath?.split('/').pop();
-        memoryEndRun(cwd, { sessionId, passed: result.success !== false });
-        memorySyncPush(cwd);
-      } catch { /* @zibby/memory not available */ }
-    }
+    // Memory end-run hook (if @zibby/memory is installed)
+    try {
+      const { memoryEndRun, memorySyncPush } = await import('@zibby/memory');
+      const sessionId = result.state.sessionPath?.split('/').pop();
+      memoryEndRun(cwd, { sessionId, passed: result.success !== false });
+      memorySyncPush(cwd);
+    } catch { /* @zibby/memory not available */ }
   }
 }

package/templates/browser-test-automation/nodes/execute-live.mjs

@@ -14,7 +14,7 @@ import { formatAssertionChecklist } from './utils.mjs';
 
 export const executeLiveNode = {
   name: 'execute_live',
-  skills: [SKILLS.BROWSER, ...(process.env.ZIBBY_MEMORY ? [SKILLS.MEMORY] : [])],
+  skills: [SKILLS.BROWSER, SKILLS.MEMORY],
   timeout: 600000,
 
   prompt: (state) => {