iterate-harness 2.2.3 → 2.2.5

package/lib/bootstrap.js

@@ -587,6 +587,13 @@ function reportBootstrapFailure(error) {
   process.stderr.write(`[iterate-harness] ${message}\n`);
 }
 
+// The install wizard only makes sense when a human can actually answer: both
+// stdin and stderr must be TTYs. Piped/CI/`< /dev/null` runs bootstrap
+// non-interactively so they never block on a prompt that nobody can see.
+function interactiveSession() {
+  return Boolean(process.stdin.isTTY && process.stderr.isTTY);
+}
+
 async function runHarness(args, env) {
   // Print the banner on every run (claude-code style). Skipped automatically
   // when stderr is not a TTY (e.g. piped output), so `ih --version | jq` stays clean.
@@ -594,7 +601,7 @@ async function runHarness(args, env) {
 
   let target;
   try {
-    target = await ensureRuntime(env, { interactive: true });
+    target = await ensureRuntime(env, { interactive: interactiveSession() });
   } catch (error) {
     if (error instanceof CancelledError) {
       ui.info(error.message);
@@ -658,6 +665,7 @@ module.exports = {
   installCandidates,
   installHarness,
   installRemoteArtifact,
+  interactiveSession,
   isRemoteHttpUrl,
   isSupportedPython,
   needsBootstrap,

package/lib/ui.js

@@ -80,24 +80,50 @@ function frameSection(title, lines) {
   console.error(bottom);
 }
 
-function askYesNo(question, defaultNo = false) {
+// Safety net for unattended runs: if stdin is closed (EOF) or the user never
+// answers, fall back to the default instead of hanging forever. Override per
+// call with `options.timeoutMs` (0 disables the timeout entirely).
+const DEFAULT_PROMPT_TIMEOUT_MS = 120000;
+
+function askYesNo(question, defaultNo = false, options = {}) {
+  const input = options.input || process.stdin;
+  const output = options.output || process.stdout;
+  const timeoutMs =
+    options.timeoutMs === undefined ? DEFAULT_PROMPT_TIMEOUT_MS : options.timeoutMs;
   return new Promise((resolve) => {
-    const rl = readline.createInterface({
-      input: process.stdin,
-      output: process.stdout,
-    });
+    let settled = false;
+    let timer = null;
+    const rl = readline.createInterface({ input, output });
+    const finish = (value) => {
+      if (settled) return;
+      settled = true;
+      if (timer) clearTimeout(timer);
+      rl.close();
+      resolve(value);
+    };
+    // readline emits "close" on EOF (piped/`< /dev/null` stdin) — never hang.
+    rl.on("close", () => finish(defaultNo));
+    if (timeoutMs > 0) {
+      timer = setTimeout(() => {
+        process.stderr.write(
+          `\x1b[2m   no response after ${Math.round(timeoutMs / 1000)}s; using default\x1b[0m\n`
+        );
+        finish(defaultNo);
+      }, timeoutMs);
+      if (typeof timer.unref === "function") timer.unref();
+    }
     const hint = defaultNo ? "[y/N]" : "[Y/n]";
     rl.question(`\x1b[36m◆\x1b[0m  ${question} ${hint} `, (answer) => {
-      rl.close();
-      const a = answer.trim().toLowerCase();
-      if (a === "y" || a === "yes") resolve(true);
-      else if (a === "n" || a === "no") resolve(false);
-      else resolve(defaultNo);
+      const a = String(answer).trim().toLowerCase();
+      if (a === "y" || a === "yes") finish(true);
+      else if (a === "n" || a === "no") finish(false);
+      else finish(defaultNo);
     });
   });
 }
 
 module.exports = {
+  DEFAULT_PROMPT_TIMEOUT_MS,
   ITERATE_BANNER,
   printBanner,
   info,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iterate-harness",
-  "version": "2.2.3",
+  "version": "2.2.5",
   "description": "npm distribution wrapper for iterate-harness — installs the Python harness (ih CLI) into a managed venv on install and exposes it as a global command",
   "license": "MIT",
   "type": "commonjs",
@@ -24,6 +24,7 @@
     "lib",
     "scripts",
     "README.md",
+    "README.zh-CN.md",
     "LICENSE"
   ],
   "keywords": [
@@ -34,7 +35,7 @@
     "installer"
   ],
   "scripts": {
-    "test": "node --test test/bootstrap.test.js test/postinstall.test.js",
+    "test": "node --test test/bootstrap.test.js test/postinstall.test.js test/ui.test.js",
     "postinstall": "node scripts/postinstall.js"
   }
 }

package/scripts/postinstall.js

@@ -17,9 +17,18 @@
 const { ensureRuntime, reportBootstrapFailure, SKIP_INSTALL_ENV_VAR } = require("../lib/bootstrap");
 const ui = require("../lib/ui");
 
+// Mirror the truthy-value handling in bootstrap.ensureRuntime so postinstall
+// and the CLI entry point agree on what counts as "skip install" (any of
+// 1/true/yes/on, case-insensitive).
+function skipInstallRequested() {
+  return ["1", "true", "yes", "on"].includes(
+    String(process.env[SKIP_INSTALL_ENV_VAR] || "").toLowerCase()
+  );
+}
+
 async function main() {
-  if (process.env[SKIP_INSTALL_ENV_VAR] === "1") {
-    ui.warning(`Skipped install during npm install (${SKIP_INSTALL_ENV_VAR}=1); harness installs on the first \`ih\` run.`);
+  if (skipInstallRequested()) {
+    ui.warning(`Skipped install during npm install (${SKIP_INSTALL_ENV_VAR}=${process.env[SKIP_INSTALL_ENV_VAR]}); harness installs on the first \`ih\` run.`);
     return;
   }
   try {