@zibby/core 0.1.18 → 0.1.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zibby/core",
-  "version": "0.1.18",
+  "version": "0.1.21",
   "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

@@ -6,7 +6,7 @@ import { tmpdir, homedir } from 'os';
 import { logger } from '../../utils/logger.js';
 import { DEFAULT_MODELS, TIMEOUTS } from '../../constants.js';
 import { DEFAULT_OUTPUT_BASE, SESSION_INFO_FILE } from '../constants.js';
-import { getAllSkills } from '../skill-registry.js';
+import { getAllSkills, getSkill } from '../skill-registry.js';
 import { StreamingParser } from '../../utils/streaming-parser.js';
 import { CursorOutputFormatter } from './utils/cursor-output-formatter.js';
 import { formatWithOpenAIProxy } from './utils/openai-proxy-formatter.js';
@@ -83,7 +83,7 @@ export class CursorAgentStrategy extends AgentStrategy {
 
     logger.debug(`[Cursor] Invoking (model: ${model}, timeout: ${timeout / 1000}s, skills: ${JSON.stringify(skills)})`);
 
-    this._setupMcpConfig(sessionPath, workspace, config);
+    this._setupMcpConfig(sessionPath, workspace, config, skills);
 
     const possibleBins = [
       // Try absolute paths first (most reliable)
@@ -174,7 +174,7 @@ export class CursorAgentStrategy extends AgentStrategy {
         if (process.env.ZIBBY_LOG_CURSOR_CLI === '1') {
           console.log(`   Full command: ${fullCmd}\n`);
         }
-      } catch (err) {
+      } catch (_err) {
         // Ignore EPIPE errors from console.log
       }
     }
@@ -252,7 +252,7 @@ export class CursorAgentStrategy extends AgentStrategy {
     return rawOutput;
   }
 
-  _setupMcpConfig(sessionPath, workspace, config) {
+  _setupMcpConfig(sessionPath, workspace, config, skills = null) {
     const cursorDir = join(homedir(), '.cursor');
     const mcpConfigPath = join(cursorDir, 'mcp.json');
     
@@ -267,8 +267,12 @@ export class CursorAgentStrategy extends AgentStrategy {
     const outputBase = config?.paths?.output || DEFAULT_OUTPUT_BASE;
     const sessionInfoPath = join(workspace || process.cwd(), outputBase, SESSION_INFO_FILE);
 
+    const skillsToResolve = Array.isArray(skills)
+      ? skills.map(id => getSkill(id)).filter(Boolean)
+      : [...getAllSkills()].map(([, skill]) => skill);
+
     const configured = new Set();
-    for (const [, skill] of getAllSkills()) {
+    for (const skill of skillsToResolve) {
       if (typeof skill.resolve !== 'function') continue;
       if (configured.has(skill.serverName)) continue;
       configured.add(skill.serverName);
@@ -341,21 +345,31 @@ export class CursorAgentStrategy extends AgentStrategy {
 
       logger.debug(`[Agent] PID: ${proc.pid}`);
 
-      // Handle stdin write errors gracefully
+      // Register error handlers on ALL child stdio streams to prevent
+      // unhandled EPIPE crashes (Node v23+ throws these as uncaught exceptions)
+      proc.stdin.on('error', (err) => {
+        if (err.code === 'EPIPE') return;
+        logger.warn(`[Agent] stdin error: ${err.message}`);
+      });
+      proc.stdout.on('error', (err) => {
+        if (err.code === 'EPIPE') return;
+        logger.warn(`[Agent] stdout error: ${err.message}`);
+      });
+      proc.stderr.on('error', (err) => {
+        if (err.code === 'EPIPE') return;
+        logger.warn(`[Agent] stderr error: ${err.message}`);
+      });
+
       if (stdinPrompt) {
-        try {
-          proc.stdin.write(stdinPrompt);
+        proc.stdin.write(stdinPrompt, (err) => {
+          if (err && err.code !== 'EPIPE') {
+            logger.warn(`[Agent] Failed to write to stdin: ${err.message}`);
+          }
           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}`);
-        }
+        });
+        logger.debug(`[Agent] Prompt also piped to stdin (${stdinPrompt.length} chars)`);
       } else {
-        try {
-          proc.stdin.end();
-        } catch (err) {
-          // Process may have already closed
-        }
+        proc.stdin.end();
       }
 
       const modifiedFiles = new Set();
@@ -444,13 +458,7 @@ export class CursorAgentStrategy extends AgentStrategy {
         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;
-          try {
-            console.log(`  Input: ${preview}`);
-          } catch (err) {
-            if (err.code !== 'EPIPE') {
-              logger.warn(`[Agent] console.log error: ${err.message}`);
-            }
-          }
+          console.log(`  Input: ${preview}`);
         }
       };
       
@@ -465,14 +473,7 @@ export class CursorAgentStrategy extends AgentStrategy {
         
         const displayText = streamParser.processChunk(chunk);
         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}`);
-            }
-          }
+          process.stdout.write(displayText);
         }
         
         const lines = chunk.split('\n').filter(l => l.trim());

package/src/framework/agents/index.js

@@ -70,7 +70,7 @@ export async function invokeAgent(prompt, context = {}, options = {}) {
     workspace: context.state?.workspace || options.workspace,
     schema: options.schema || context.schema,
     images: options.images || context.images || [],
-    skills: options.skills || context.skills || null,
+    skills: options.skills || context.skills || [],
     config,
     ...options
   };

package/src/framework/node.js

@@ -167,7 +167,7 @@ export class Node {
           model,
           workspace: cwd,
           schema: this.isZodSchema ? this.outputSchema : null,
-          skills: this.config.skills || null,
+          skills: this.config.skills || [],
           sessionPath,
           config: zibbyConfig
         };