@zibby/core 0.1.9 → 0.1.11

package/package.json

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

package/scripts/setup-official-playwright-mcp.sh

@@ -5,13 +5,30 @@ set -e
 # Get script directory for relative imports
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
-# Step 1: Check cursor-agent
-if ! command -v cursor-agent &> /dev/null; then
+# Step 1: Check cursor-agent (check actual locations, not just PATH)
+CURSOR_AGENT_PATH=""
+if command -v cursor-agent &> /dev/null; then
+    CURSOR_AGENT_PATH=$(command -v cursor-agent)
+elif [ -x "$HOME/.local/bin/cursor-agent" ]; then
+    CURSOR_AGENT_PATH="$HOME/.local/bin/cursor-agent"
+elif [ -x "$HOME/.cursor/bin/cursor-agent" ]; then
+    CURSOR_AGENT_PATH="$HOME/.cursor/bin/cursor-agent"
+fi
+
+if [ -z "$CURSOR_AGENT_PATH" ]; then
     echo "⚠️  cursor-agent not found. Installing..."
     curl https://cursor.com/install -fsS | bash
-    export PATH="$HOME/.cursor/bin:$HOME/.local/bin:$PATH"
+    # Check again after install
+    if [ -x "$HOME/.local/bin/cursor-agent" ]; then
+        CURSOR_AGENT_PATH="$HOME/.local/bin/cursor-agent"
+    elif [ -x "$HOME/.cursor/bin/cursor-agent" ]; then
+        CURSOR_AGENT_PATH="$HOME/.cursor/bin/cursor-agent"
+    fi
 fi
 
+# Always ensure these directories are in PATH for this script
+export PATH="$HOME/.cursor/bin:$HOME/.local/bin:$PATH"
+
 # Step 2: Find Node.js absolute paths (Cursor GUI doesn't have shell PATH)
 NODE_PATH=$(which node 2>/dev/null || echo "")
 NPX_PATH=$(which npx 2>/dev/null || echo "")

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

@@ -32,19 +32,31 @@ export class CursorAgentStrategy extends AgentStrategy {
     for (const path of paths) {
       try {
         if (path.startsWith('/')) {
+          // Check file exists and is executable
           accessSync(path, constants.X_OK);
-          logger.debug(`[Cursor] Found agent at: ${path}`);
-          return true;
+          // Verify it actually runs
+          const out = execSync(`"${path}" --version 2>&1`, { encoding: 'utf-8', timeout: 3000, stdio: 'pipe' });
+          if (out && out.length > 0) {
+            logger.debug(`[Cursor] Found agent at: ${path} (version: ${out.trim().slice(0, 50)})`);
+            return true;
+          }
+        } else {
+          // Check if in PATH and runs
+          const which = execSync(`which ${path}`, { encoding: 'utf-8', timeout: 2000, stdio: 'pipe' }).trim();
+          if (!which) continue;
+          const out = execSync(`${path} --version 2>&1`, { encoding: 'utf-8', timeout: 3000, stdio: 'pipe' });
+          if (out && out.length > 0) {
+            logger.debug(`[Cursor] Found '${path}' in PATH at ${which} (version: ${out.trim().slice(0, 50)})`);
+            return true;
+          }
         }
-        execSync(`${path} --version 2>/dev/null`, { stdio: 'ignore', timeout: 2000 });
-        logger.debug(`[Cursor] Found '${path}' in PATH`);
-        return true;
       } catch (_e) {
+        // Binary doesn't exist or doesn't work
         continue;
       }
     }
     
-    logger.warn('[Cursor] ❌ Cursor Agent CLI not found');
+    logger.warn('[Cursor] ❌ Cursor Agent CLI not found or not working. Run: agent --version');
     return false;
   }
 
@@ -79,20 +91,34 @@ export class CursorAgentStrategy extends AgentStrategy {
     for (const bin of possibleBins) {
       try {
         if (bin.startsWith('/')) {
+          // For absolute paths, check file exists and is executable
           accessSync(bin, constants.X_OK);
+          // Also verify it actually runs
+          execSync(`"${bin}" --version 2>&1`, { encoding: 'utf-8', timeout: 3000, stdio: 'pipe' });
         } else {
-          execSync(`which ${bin} 2>/dev/null`, { stdio: 'ignore', timeout: 2000 });
+          // For commands in PATH, verify they exist and run
+          const which = execSync(`which ${bin}`, { encoding: 'utf-8', timeout: 2000 }).trim();
+          if (!which) throw new Error('not in PATH');
+          // Verify the binary works
+          execSync(`${bin} --version 2>&1`, { encoding: 'utf-8', timeout: 3000, stdio: 'pipe' });
         }
         cursorBin = bin;
         logger.debug(`[Agent] Using binary: ${bin}`);
         break;
-      } catch {
+      } catch (err) {
+        logger.debug(`[Agent] Binary '${bin}' check failed: ${err.message}`);
         continue;
       }
     }
     
     if (!cursorBin) {
-      throw new Error('Cursor Agent CLI not found. Install it or add it to PATH.');
+      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'
+      );
     }
     
     // File-based structured output: agent writes JSON to this file
@@ -349,12 +375,16 @@ export class CursorAgentStrategy extends AgentStrategy {
 
         logger.debug(`💓 [Agent] Running for ${elapsed}s | ${lineCount} lines output${activity}`);
 
-        if (lineCount === 0 && elapsed >= 90 && elapsed < 120 && modifiedFiles.size === 0) {
-          logger.warn(`⚠️  [Agent] No output and no file changes after ${elapsed}s — agent may be stuck.`);
-          try {
-            const procInfo = execSync(`ps -p ${proc.pid} -o pid,stat,rss,time 2>/dev/null || echo "Process info unavailable"`, { encoding: 'utf-8', timeout: 5000 });
-            logger.warn(`⚠️  [Agent] Process: ${procInfo.trim()}`);
-          } catch {}
+        if (lineCount === 0 && elapsed >= 30 && modifiedFiles.size === 0) {
+          if (elapsed < 35) {
+            logger.warn(`⚠️  [Agent] No output after ${elapsed}s — agent may be stuck. Check your CURSOR_API_KEY.`);
+          }
+          if (elapsed >= 60) {
+            killed = true;
+            logger.error(`❌ [Agent] No response after ${elapsed}s — killing. Verify CURSOR_API_KEY is valid and agent CLI works: agent --version`);
+            proc.kill('SIGTERM');
+            setTimeout(() => { if (!proc.killed) proc.kill('SIGKILL'); }, 3000);
+          }
         }
       }, 30000);
 

package/src/index.js

@@ -120,7 +120,7 @@ export async function runTest(specPath, config = {}) {
     throw new Error(
       `No agent found. Please run:\n` +
       `  zibby init\n\n` +
-      `This will create .zibby/graph.js with your workflow definition.`
+      `This will create .zibby/graph.mjs with your workflow definition.`
     );
   }
   
@@ -243,10 +243,10 @@ export async function listWorkflows(cwd = process.cwd()) {
     const { existsSync: fsExistsSync } = await import('fs');
     const { pathToFileURL } = await import('url');
     
-    const localAgentPath = pathJoin(cwd, '.zibby/graph.js');
+    const localAgentPath = pathJoin(cwd, '.zibby/graph.mjs');
     
     if (!fsExistsSync(localAgentPath)) {
-      return { available: [], default: null, error: 'No .zibby/graph.js found' };
+      return { available: [], default: null, error: 'No .zibby/graph.mjs found' };
     }
     
     const agentModule = await import(pathToFileURL(localAgentPath).href);
@@ -273,7 +273,7 @@ async function loadLocalAgent(cwd, config) {
     const { existsSync: fsExistsSync } = await import('fs');
     const { pathToFileURL } = await import('url');
     
-    const localAgentPath = pathJoin(cwd, '.zibby/graph.js');
+    const localAgentPath = pathJoin(cwd, '.zibby/graph.mjs');
     
     if (!fsExistsSync(localAgentPath)) {
       return { agent: null, error: null };
@@ -320,7 +320,7 @@ async function loadLocalAgent(cwd, config) {
       }
       
       if (!AgentClass.name?.includes('auto-detected')) {
-        console.log('✓ Using local agent from .zibby/graph.js');
+        console.log('✓ Using local agent from .zibby/graph.mjs');
       }
     }
     

package/templates/browser-test-automation/{graph.js → graph.mjs}

@@ -10,8 +10,8 @@ import {
   preflightNode, 
   executeLiveNode, 
   generateScriptNode,
-} from './nodes/index.js';
-import { BrowserTestResultHandler } from './result-handler.js';
+} from './nodes/index.mjs';
+import { BrowserTestResultHandler } from './result-handler.mjs';
 
 let memoryMiddleware = null;
 try {

package/templates/browser-test-automation/nodes/{execute-live.js → execute-live.mjs}

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

package/templates/browser-test-automation/nodes/{generate-script.js → generate-script.mjs}

@@ -1,5 +1,5 @@
 import { z } from '@zibby/core';
-import { formatRecordedActions, formatAssertionsWithResults } from './utils.js';
+import { formatRecordedActions, formatAssertionsWithResults } from './utils.mjs';
 
 const GenerateScriptOutputSchema = z.object({
   success: z.boolean(),

package/templates/browser-test-automation/nodes/index.mjs

@@ -0,0 +1,3 @@
+export { preflightNode } from './preflight.mjs';
+export { executeLiveNode } from './execute-live.mjs';
+export { generateScriptNode } from './generate-script.mjs';

package/templates/index.js

@@ -41,7 +41,7 @@ export class TemplateFactory {
   }
 
   static validateTemplate(templatePath) {
-    const requiredFiles = ['graph.js', 'nodes', 'README.md'];
+    const requiredFiles = ['graph.mjs', 'nodes', 'README.md'];
     
     for (const file of requiredFiles) {
       const filePath = join(templatePath, file);
@@ -57,9 +57,9 @@ export class TemplateFactory {
     const template = this.getTemplate(templateName);
     this.validateTemplate(template.path);
     
-    const resultHandlerPath = join(template.path, 'result-handler.js');
+    const resultHandlerPath = join(template.path, 'result-handler.mjs');
     return {
-      graphPath: join(template.path, 'graph.js'),
+      graphPath: join(template.path, 'graph.mjs'),
       nodesPath: join(template.path, 'nodes'),
       readmePath: join(template.path, 'README.md'),
       resultHandlerPath: existsSync(resultHandlerPath) ? resultHandlerPath : null,

package/templates/browser-test-automation/nodes/index.js

@@ -1,3 +0,0 @@
-export { preflightNode } from './preflight.js';
-export { executeLiveNode } from './execute-live.js';
-export { generateScriptNode } from './generate-script.js';