agent-yes 1.62.0 → 1.62.2

package/dist/{SUPPORTED_CLIS-4TAXzvkb.js → SUPPORTED_CLIS-CxaWfAAA.js}

@@ -10,7 +10,7 @@ import sflow from "sflow";
 import { TerminalRenderStream } from "terminal-render";
 import { homedir } from "os";
 import winston from "winston";
-import { closeSync, existsSync, fsyncSync, openSync } from "fs";
+import { closeSync, existsSync, fsyncSync, mkdirSync, openSync } from "fs";
 import { lock } from "proper-lockfile";
 import { execSync as execSync$1 } from "node:child_process";
 import { fileURLToPath } from "url";
@@ -798,11 +798,11 @@ async function sendEnter(context, waitms = 1e3) {
 	context.shell.write("\r");
 	logger.debug(`enterSent| idleWaiter.wait(${String(waitms)}) took ${String(et - st)}ms`);
 	await Promise.race([context.nextStdout.wait(), new Promise((resolve) => setTimeout(() => {
-		if (!context.nextStdout.ready) context.shell.write("\r");
+		if (!context.nextStdout.isReady) context.shell.write("\r");
 		resolve();
 	}, 1e3))]);
 	await Promise.race([context.nextStdout.wait(), new Promise((resolve) => setTimeout(() => {
-		if (!context.nextStdout.ready) context.shell.write("\r");
+		if (!context.nextStdout.isReady) context.shell.write("\r");
 		resolve();
 	}, 3e3))]);
 }
@@ -819,7 +819,10 @@ async function sendMessage(context, message, { waitForReady = true } = {}) {
 	context.shell.write(message);
 	context.idleWaiter.ping();
 	logger.debug(`waiting next stdout|${message}`);
-	await context.nextStdout.wait();
+	await Promise.race([context.nextStdout.wait(), new Promise((resolve) => setTimeout(() => {
+		logger.warn(`nextStdout.wait() timed out after 30s for message: ${message}`);
+		resolve();
+	}, 3e4))]);
 	logger.debug(`sending enter`);
 	await sendEnter(context, 1e3);
 	logger.debug(`sent enter`);
@@ -904,7 +907,7 @@ function tryCatch(catchFn, fn) {
 //#endregion
 //#region package.json
 var name = "agent-yes";
-var version = "1.62.0";
+var version = "1.62.2";
 
 //#endregion
 //#region ts/pty-fix.ts
@@ -1069,7 +1072,7 @@ function isCommandNotFoundError(e) {
 * ```
 */
 function spawnAgent(options) {
-	const { cli, cliConf, cliArgs, verbose: _verbose, install, ptyOptions } = options;
+	const { cli, cliConf, cliArgs, install, ptyOptions } = options;
 	const spawn = () => {
 		let [bin, ...args] = [...parseCommandString(cliConf?.binary || cli), ...cliArgs];
 		logger.debug(`Spawning ${bin} with args: ${JSON.stringify(args)}`);
@@ -1077,7 +1080,7 @@ function spawnAgent(options) {
 		logger.info(`[${cli}-yes] Spawned ${bin} with PID ${spawned.pid} (agent-yes v${version})`);
 		return spawned;
 	};
-	return tryCatch((error, _attempts, spawn, ...args) => {
+	return tryCatch((error, attempts, spawn, ...args) => {
 		logger.error(`Fatal: Failed to start ${cli}.`);
 		const isNotFound = isCommandNotFoundError(error);
 		if (cliConf?.install && isNotFound) {
@@ -1777,7 +1780,7 @@ async function agentYes({ cli, cliArgs = [], prompt, robust = true, cwd, env, ex
 		cancel(_reason) {
 			process.stdin.pause();
 		}
-	});
+	}, { highWaterMark: 16 });
 	let aborted = false;
 	await sflow(stdinStream).map((buffer) => {
 		const str = buffer.toString();
@@ -1864,7 +1867,6 @@ async function agentYes({ cli, cliArgs = [], prompt, robust = true, cwd, env, ex
 	}).by(function consoleResponder(e) {
 		let lastRendered = "";
 		return e.forEach((chunk) => {
-			terminalRender.write(chunk);
 			if (chunk.includes("\x1B[c") || chunk.includes("\x1B[0c")) {
 				shell.write("\x1B[?1;2c");
 				if (verbose) logger.debug("device|respond DA: VT100 with Advanced Video Option");
@@ -1951,4 +1953,4 @@ const SUPPORTED_CLIS = Object.keys(CLIS_CONFIG);
 
 //#endregion
 export { AgentContext as a, PidStore as c, config as i, removeControlCharacters as l, CLIS_CONFIG as n, name as o, agentYes as r, version as s, SUPPORTED_CLIS as t };
-//# sourceMappingURL=SUPPORTED_CLIS-4TAXzvkb.js.map
+//# sourceMappingURL=SUPPORTED_CLIS-CxaWfAAA.js.map

package/dist/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env bun
-import { c as PidStore, o as name, s as version, t as SUPPORTED_CLIS } from "./SUPPORTED_CLIS-4TAXzvkb.js";
+import { c as PidStore, o as name, s as version, t as SUPPORTED_CLIS } from "./SUPPORTED_CLIS-CxaWfAAA.js";
 import { t as logger } from "./logger-CX77vJDA.js";
 import { argv } from "process";
 import { spawn } from "child_process";
@@ -271,7 +271,6 @@ function getBinDir() {
 */
 function findRustBinary(verbose = false) {
 	const binaryName = getBinaryName();
-	binaryName.replace(/\.exe$/, "");
 	const searchPaths = [
 		path.join(getBinDir(), binaryName),
 		path.resolve(import.meta.dirname ?? import.meta.dir, "../rs/target/release/agent-yes"),

package/dist/index.js

@@ -1,4 +1,4 @@
-import { a as AgentContext, i as config, l as removeControlCharacters, n as CLIS_CONFIG, r as agentYes } from "./SUPPORTED_CLIS-4TAXzvkb.js";
+import { a as AgentContext, i as config, l as removeControlCharacters, n as CLIS_CONFIG, r as agentYes } from "./SUPPORTED_CLIS-CxaWfAAA.js";
 import "./logger-CX77vJDA.js";
 
 export { AgentContext, CLIS_CONFIG, config, agentYes as default, removeControlCharacters };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-yes",
-  "version": "1.62.0",
+  "version": "1.62.2",
   "description": "A wrapper tool that automates interactions with various AI CLI tools by automatically handling common prompts and responses.",
   "keywords": [
     "ai",

package/ts/index.ts

@@ -14,7 +14,8 @@ import pty, { ptyPackage } from "./pty.ts";
 import { removeControlCharacters } from "./removeControlCharacters.ts";
 import { acquireLock, releaseLock, shouldUseLock } from "./runningLock.ts";
 import { logger } from "./logger.ts";
-// import { createFifoStream } from "./beta/fifo.ts";
+// oxlint-disable-next-line no-unused-vars -- kept for FIFO re-enable (see TODO at index.ts:833)
+import { createFifoStream } from "./beta/fifo.ts";
 import { PidStore } from "./pidStore.ts";
 import { SUPPORTED_CLIS } from "./SUPPORTED_CLIS.ts";
 import { sendEnter, sendMessage } from "./core/messaging.ts";
@@ -26,10 +27,8 @@ import {
 } from "./core/logging.ts";
 import { spawnAgent } from "./core/spawner.ts";
 import { AgentContext } from "./core/context.ts";
-// import { createAutoResponseHandler } from "./core/responders.ts";
 import { createTerminatorStream } from "./core/streamHelpers.ts";
 import { globalAgentRegistry } from "./agentRegistry.ts";
-// import { ReadyManager } from "./ReadyManager.ts";
 import { notifyWebhook } from "./webhookNotifier.ts";
 
 export { removeControlCharacters };
@@ -688,52 +687,55 @@ export default async function agentYes({
   // CRITICAL FIX: fromReadable() from 'from-node-stream' doesn't work properly with stdin
   // because it doesn't handle Node.js stream modes correctly. We create a custom ReadableStream
   // that properly manages stdin's flowing mode and event listeners.
-  const stdinStream = new ReadableStream<Buffer>({
-    start(controller) {
-      // Set up stdin in flowing mode so 'data' events fire
-      process.stdin.resume();
-
-      let closed = false;
-
-      // Handle data events
-      const dataHandler = (chunk: Buffer) => {
-        try {
-          controller.enqueue(chunk);
-        } catch {
-          // Ignore enqueue errors (stream may be closed)
-        }
-      };
+  const stdinStream = new ReadableStream<Buffer>(
+    {
+      start(controller) {
+        // Set up stdin in flowing mode so 'data' events fire
+        process.stdin.resume();
+
+        let closed = false;
+
+        // Handle data events
+        const dataHandler = (chunk: Buffer) => {
+          try {
+            controller.enqueue(chunk);
+          } catch {
+            // Ignore enqueue errors (stream may be closed)
+          }
+        };
 
-      // Handle end/close - both events can fire, so track state
-      const endHandler = () => {
-        if (closed) return;
-        closed = true;
-        try {
-          controller.close();
-        } catch {
-          // Ignore close errors (already closed)
-        }
-      };
+        // Handle end/close - both events can fire, so track state
+        const endHandler = () => {
+          if (closed) return;
+          closed = true;
+          try {
+            controller.close();
+          } catch {
+            // Ignore close errors (already closed)
+          }
+        };
 
-      const errorHandler = (err: Error) => {
-        if (closed) return;
-        closed = true;
-        try {
-          controller.error(err);
-        } catch {
-          // Ignore error after close
-        }
-      };
+        const errorHandler = (err: Error) => {
+          if (closed) return;
+          closed = true;
+          try {
+            controller.error(err);
+          } catch {
+            // Ignore error after close
+          }
+        };
 
-      process.stdin.on("data", dataHandler);
-      process.stdin.on("end", endHandler);
-      process.stdin.on("close", endHandler);
-      process.stdin.on("error", errorHandler);
+        process.stdin.on("data", dataHandler);
+        process.stdin.on("end", endHandler);
+        process.stdin.on("close", endHandler);
+        process.stdin.on("error", errorHandler);
+      },
+      cancel(_reason) {
+        process.stdin.pause();
+      },
     },
-    cancel(_reason) {
-      process.stdin.pause();
-    },
-  });
+    { highWaterMark: 16 },
+  );
 
   let aborted = false;
   await sflow(stdinStream)
@@ -802,7 +804,7 @@ export default async function agentYes({
               // Only check for /auto if line is short enough
               if (line.length <= 20) {
                 const cleanLine = line
-                  // oxlint-disable-next-line no-control-regex
+                  // oxlint-disable-next-line no-control-regex -- intentional: strip ANSI/control chars
                   .replace(/[\x00-\x1f]|\x1b\[[0-9;]*[A-Za-z]|\[[A-Z]/g, "")
                   .trim();
                 if (cleanLine === "/auto") {
@@ -894,8 +896,10 @@ export default async function agentYes({
       let lastRendered = "";
       return e
         .forEach((chunk) => {
-          // Render terminal output for log file
-          terminalRender.write(chunk);
+          // NOTE: terminalRender is already updated by terminalStream.writable.write() in onData()
+          // (terminal-render v1.5+ calls renderer.write() as a side effect of the writable).
+          // Calling terminalRender.write(chunk) here again would double-process every chunk,
+          // corrupting cursor state and breaking all pattern matching.
           // ============ HANDLE special control sequences
           // Handle Device Attributes query (DA) - ESC[c or ESC[0c
           // This must be handled regardless of TTY status

package/ts/parseCliArgs.ts

@@ -2,8 +2,6 @@ import ms from "ms";
 import yargs from "yargs";
 import { hideBin } from "yargs/helpers";
 import { SUPPORTED_CLIS } from "./SUPPORTED_CLIS.ts";
-
-// const pkg = await JSON.parse(await readFile(path.resolve((import.meta.dir) + "/../package.json"), 'utf8'))
 /**
  * Parse CLI arguments the same way cli.ts does
  * This is a test helper that mirrors the parsing logic in cli.ts

package/ts/rustBinary.ts

@@ -60,7 +60,6 @@ export function getBinDir(): string {
  */
 export function findRustBinary(verbose = false): string | undefined {
   const binaryName = getBinaryName();
-  const _baseName = binaryName.replace(/\.exe$/, "");
 
   const searchPaths = [
     // 1. Check in npm package bin directory