opc-agent 4.0.3 → 4.0.5

package/dist/providers/index.js

@@ -330,12 +330,9 @@ class ClaudeCLIProvider {
         this.model = model || 'sonnet';
     }
     async chat(messages, systemPrompt, options) {
-        const { execFile } = await Promise.resolve().then(() => __importStar(require('child_process')));
-        const { promisify } = await Promise.resolve().then(() => __importStar(require('util')));
         const { writeFileSync, unlinkSync, mkdtempSync } = await Promise.resolve().then(() => __importStar(require('fs')));
         const { join } = await Promise.resolve().then(() => __importStar(require('path')));
         const { tmpdir } = await Promise.resolve().then(() => __importStar(require('os')));
-        const execFileAsync = promisify(execFile);
         // Build the prompt from messages
         const lastMessage = messages[messages.length - 1];
         if (!lastMessage)
@@ -345,7 +342,7 @@ class ClaudeCLIProvider {
         if (options?.tools && options.tools.length > 0) {
             prompt += buildToolPrompt(options.tools);
         }
-        const args = ['-p', '--bare'];
+        const args = ['-p'];
         // Write system prompt to temp file to avoid shell escaping issues
         let tmpFile;
         if (systemPrompt) {
@@ -359,22 +356,44 @@ class ClaudeCLIProvider {
         }
         args.push(prompt);
         try {
-            const { stdout } = await execFileAsync('claude', args, {
-                timeout: 120_000,
-                maxBuffer: 10 * 1024 * 1024,
-                env: { ...process.env },
+            const { spawn } = await Promise.resolve().then(() => __importStar(require('child_process')));
+            const result = await new Promise((resolve, reject) => {
+                const proc = spawn('claude', args, {
+                    env: { ...process.env },
+                    stdio: ['pipe', 'pipe', 'pipe'],
+                });
+                // Close stdin immediately to avoid "no stdin data" warning
+                proc.stdin.end();
+                let stdout = '';
+                let stderr = '';
+                proc.stdout.on('data', (d) => { stdout += d.toString(); });
+                proc.stderr.on('data', (d) => { stderr += d.toString(); });
+                proc.on('close', (code) => {
+                    if (code === 0 || stdout.trim()) {
+                        resolve(stdout.trim());
+                    }
+                    else {
+                        reject(new Error(`Claude CLI exited ${code}: ${stderr}`));
+                    }
+                });
+                proc.on('error', (err) => {
+                    if (err.code === 'ENOENT') {
+                        reject(new Error('Claude CLI not found. Install it: npm install -g @anthropic-ai/claude-code\n' +
+                            'Then authenticate: claude login'));
+                    }
+                    else {
+                        reject(err);
+                    }
+                });
+                // Timeout
+                setTimeout(() => {
+                    proc.kill();
+                    reject(new Error('Claude CLI timed out after 120s'));
+                }, 120_000);
             });
-            return stdout.trim();
+            return result;
         }
         catch (err) {
-            if (err.code === 'ENOENT') {
-                throw new Error('Claude CLI not found. Install it: npm install -g @anthropic-ai/claude-code\n' +
-                    'Then authenticate: claude login');
-            }
-            // If claude returns non-zero but has stdout, use it
-            if (err.stdout && err.stdout.trim()) {
-                return err.stdout.trim();
-            }
             throw new Error(`Claude CLI error: ${err.message}`);
         }
         finally {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opc-agent",
-  "version": "4.0.3",
+  "version": "4.0.5",
   "description": "Open Agent Framework — Build, test, and run AI Agents for business workstations",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/providers/index.ts

@@ -334,12 +334,9 @@ class ClaudeCLIProvider implements LLMProvider {
   }
 
   async chat(messages: Message[], systemPrompt?: string, options?: ChatOptions): Promise<string> {
-    const { execFile } = await import('child_process');
-    const { promisify } = await import('util');
     const { writeFileSync, unlinkSync, mkdtempSync } = await import('fs');
     const { join } = await import('path');
     const { tmpdir } = await import('os');
-    const execFileAsync = promisify(execFile);
 
     // Build the prompt from messages
     const lastMessage = messages[messages.length - 1];
@@ -352,7 +349,7 @@ class ClaudeCLIProvider implements LLMProvider {
       prompt += buildToolPrompt(options.tools);
     }
 
-    const args = ['-p', '--bare'];
+    const args = ['-p'];
     // Write system prompt to temp file to avoid shell escaping issues
     let tmpFile: string | undefined;
     if (systemPrompt) {
@@ -367,23 +364,43 @@ class ClaudeCLIProvider implements LLMProvider {
     args.push(prompt);
 
     try {
-      const { stdout } = await execFileAsync('claude', args, {
-        timeout: 120_000,
-        maxBuffer: 10 * 1024 * 1024,
-        env: { ...process.env },
+      const { spawn } = await import('child_process');
+      const result = await new Promise<string>((resolve, reject) => {
+        const proc = spawn('claude', args, {
+          env: { ...process.env },
+          stdio: ['pipe', 'pipe', 'pipe'],
+        });
+        // Close stdin immediately to avoid "no stdin data" warning
+        proc.stdin.end();
+        let stdout = '';
+        let stderr = '';
+        proc.stdout.on('data', (d: Buffer) => { stdout += d.toString(); });
+        proc.stderr.on('data', (d: Buffer) => { stderr += d.toString(); });
+        proc.on('close', (code) => {
+          if (code === 0 || stdout.trim()) {
+            resolve(stdout.trim());
+          } else {
+            reject(new Error(`Claude CLI exited ${code}: ${stderr}`));
+          }
+        });
+        proc.on('error', (err: any) => {
+          if (err.code === 'ENOENT') {
+            reject(new Error(
+              'Claude CLI not found. Install it: npm install -g @anthropic-ai/claude-code\n' +
+              'Then authenticate: claude login'
+            ));
+          } else {
+            reject(err);
+          }
+        });
+        // Timeout
+        setTimeout(() => {
+          proc.kill();
+          reject(new Error('Claude CLI timed out after 120s'));
+        }, 120_000);
       });
-      return stdout.trim();
+      return result;
     } catch (err: any) {
-      if (err.code === 'ENOENT') {
-        throw new Error(
-          'Claude CLI not found. Install it: npm install -g @anthropic-ai/claude-code\n' +
-          'Then authenticate: claude login'
-        );
-      }
-      // If claude returns non-zero but has stdout, use it
-      if (err.stdout && err.stdout.trim()) {
-        return err.stdout.trim();
-      }
       throw new Error(`Claude CLI error: ${err.message}`);
     } finally {
       if (tmpFile) {