noninteractive 0.3.27 → 0.3.32

package/bin/noninteractive.js

@@ -172,11 +172,14 @@ function runDaemon(sessionName, executable, args) {
   let outputBuffer = "";
   let processExited = false;
   let exitCode = null;
+  let lastReadLength = 0;
   const detectedUrls = new Set;
   const reportedUrls = new Set;
   const waiters = [];
   let notifyDebounce = null;
   const NOTIFY_SETTLE_MS = 50;
+  let lastStdinWrite = 0;
+  const INPUT_SETTLE_MS = 150;
   function notifyWaiters() {
     if (waiters.length === 0)
       return;
@@ -282,14 +285,17 @@ function runDaemon(sessionName, executable, args) {
       } catch {}
     });
   });
-  function respondWithOutput(socket) {
+  function respondWithOutput(socket, updateSnapshot = true) {
     readInterceptedUrls();
+    if (updateSnapshot)
+      lastReadLength = outputBuffer.length;
     const newUrls = Array.from(detectedUrls).filter((u) => !reportedUrls.has(u));
     for (const u of newUrls)
       reportedUrls.add(u);
     socket.end(JSON.stringify({
       ok: true,
       output: outputBuffer,
+      outputLength: outputBuffer.length,
       exited: processExited,
       exitCode,
       ...newUrls.length > 0 ? { urls: newUrls } : {}
@@ -311,12 +317,26 @@ function runDaemon(sessionName, executable, args) {
     };
     waiters.push(waiter);
   }
+  function writeToStdin(data) {
+    lastStdinWrite = Date.now();
+    stdin?.write(data);
+  }
+  function withSettleDelay(fn) {
+    const now = Date.now();
+    const elapsed = now - lastStdinWrite;
+    const delay = Math.max(0, INPUT_SETTLE_MS - elapsed);
+    if (delay > 0) {
+      setTimeout(fn, delay);
+    } else {
+      fn();
+    }
+  }
   function handle(msg, socket) {
     switch (msg.action) {
       case "read":
         if (msg.wait) {
           const timeout = msg.timeout ?? 30000;
-          waitForNewOutput(socket, outputBuffer.length, timeout);
+          waitForNewOutput(socket, lastReadLength, timeout);
         } else {
           respondWithOutput(socket);
         }
@@ -326,8 +346,11 @@ function runDaemon(sessionName, executable, args) {
           socket.end(JSON.stringify({ ok: false, error: "process exited" }));
           break;
         }
-        stdin?.write(msg.data);
-        socket.end(JSON.stringify({ ok: true }));
+        withSettleDelay(() => {
+          writeToStdin(msg.data);
+          lastReadLength = outputBuffer.length;
+          socket.end(JSON.stringify({ ok: true }));
+        });
         break;
       case "sendread": {
         if (processExited) {
@@ -336,8 +359,10 @@ function runDaemon(sessionName, executable, args) {
         }
         const beforeLength = outputBuffer.length;
         const timeout = msg.timeout ?? 30000;
-        stdin?.write(msg.data);
-        waitForNewOutput(socket, beforeLength, timeout);
+        withSettleDelay(() => {
+          writeToStdin(msg.data);
+          waitForNewOutput(socket, beforeLength, timeout);
+        });
         break;
       }
       case "stop":
@@ -379,7 +404,7 @@ var init_daemon = __esm(() => {
 var require_package = __commonJS((exports, module) => {
   module.exports = {
     name: "noninteractive",
-    version: "0.3.27",
+    version: "0.3.32",
     type: "module",
     bin: {
       noninteractive: "./bin/noninteractive.js"
@@ -656,25 +681,28 @@ async function start(cmdArgs, noOpen = false, sessionName, cwd) {
   const executable = cmdArgs[0];
   const args = cmdArgs.slice(1);
   const baseName = sessionName || deriveSessionName(executable, args);
-  let name = baseName;
-  let suffix = 1;
-  while (true) {
+  const name = baseName;
+  {
     const sock2 = socketPath(name);
     try {
-      const res = await sendMessage(sock2, { action: "read" });
+      const res = await sendMessage(sock2, { action: "read" }, 2000);
       if (res.ok) {
         if (res.exited) {
           try {
-            await sendMessage(sock2, { action: "stop" });
+            await sendMessage(sock2, { action: "stop" }, 2000);
           } catch {}
-          break;
+        } else {
+          console.error(`session '${name}' is already running. use a different --name or stop it first:`);
+          console.error(`  npx noninteractive stop ${name}`);
+          process.exit(1);
         }
-        suffix++;
-        name = `${baseName}-${suffix}`;
-        continue;
       }
-    } catch {}
-    break;
+    } catch {
+      try {
+        const { unlinkSync: unlinkSync2 } = await import("fs");
+        unlinkSync2(sock2);
+      } catch {}
+    }
   }
   const sock = socketPath(name);
   ensureSessionsDir();
@@ -714,15 +742,17 @@ make sure the command exists. examples:`);
         process.stdout.write(stripAnsi(res.output));
         if (res.exited) {
           console.log(`
-[session '${name}' exited ${res.exitCode} \u2014 the command failed]`);
-          console.log(`hint: the first argument to "start" is the command to run, NOT a session name.`);
-          console.log(`  npx noninteractive start npx vercel        # run an npx package`);
-          console.log(`  npx noninteractive start vercel login       # run a command directly`);
+[session '${name}' exited ${res.exitCode}]`);
+          if (clean.length < 100 && i < 10) {
+            console.log(`hint: the first argument to "start" is the command to run, NOT a session name.`);
+            console.log(`  npx noninteractive start npx vercel        # run an npx package`);
+            console.log(`  npx noninteractive start vercel login       # run a command directly`);
+          }
         } else {
           console.log(`
-[session '${name}' started \u2014 read the output above, then use:]`);
-          console.log(`  npx noninteractive send ${name} "<text>"          # send and get response`);
-          console.log(`  npx noninteractive read ${name} --wait        # wait for new output`);
+[session '${name}' started \u2014 the first prompt is shown above. use:]`);
+          console.log(`  npx noninteractive send ${name} "<text>"      # send and get response (waits by default)`);
+          console.log(`  npx noninteractive read ${name} --wait        # only needed for long waits (e.g. OAuth, npm install)`);
           console.log(`  npx noninteractive stop ${name}               # stop the session`);
         }
         return;
@@ -739,8 +769,8 @@ make sure the command exists. examples:`);
     } catch {}
   }
   console.log(`[session '${name}' started but no output yet \u2014 use:]`);
-  console.log(`  npx noninteractive send ${name} "<text>"          # send and get response`);
-  console.log(`  npx noninteractive read ${name} --wait        # wait for new output`);
+  console.log(`  npx noninteractive send ${name} "<text>"      # send and get response (waits by default)`);
+  console.log(`  npx noninteractive read ${name} --wait        # only needed for long waits (e.g. OAuth, npm install)`);
   console.log(`  npx noninteractive stop ${name}               # stop the session`);
 }
 async function read(name, wait, timeout, noOpen = false) {
@@ -756,9 +786,11 @@ async function read(name, wait, timeout, noOpen = false) {
     if (res.output !== undefined)
       process.stdout.write(stripAnsi(res.output));
     handleUrls(res, noOpen);
-    if (res.exited)
+    if (res.exited) {
       console.log(`
 [exited ${res.exitCode}]`);
+      console.log(`[session complete \u2014 output above is final, no need to read again]`);
+    }
   } catch {
     const outputFile = sessionOutputFile(name);
     if (existsSync2(outputFile)) {
@@ -784,9 +816,11 @@ async function send(name, text, wait, timeout, noOpen = false) {
       if (res.output !== undefined)
         process.stdout.write(stripAnsi(res.output));
       handleUrls(res, noOpen);
-      if (res.exited)
+      if (res.exited) {
         console.log(`
 [exited ${res.exitCode}]`);
+        console.log(`[session complete \u2014 output above is final, no need to read again]`);
+      }
     } else {
       await sendMessage(sock, { action: "send", data: text });
       console.log(`[sent to '${name}' \u2014 run "npx noninteractive read ${name}" to see the result]`);
@@ -852,7 +886,23 @@ async function main() {
         cwd = startArgs[cwdIdx + 1];
         startArgs.splice(cwdIdx, 2);
       }
-      const filtered = startArgs.filter((a) => a !== "--no-open");
+      if (startArgs.includes("--")) {
+        console.error(`hint: the -- separator is not needed. just put the command after the flags.`);
+      }
+      const filtered = startArgs.filter((a) => a !== "--no-open" && a !== "--");
+      if (filtered.includes("--help") || filtered.includes("-h")) {
+        console.log(`usage: noninteractive start [--name <session>] [--cwd <dir>] <cmd> [args...]
+
+examples:
+  npx noninteractive start npx eslint --init
+  npx noninteractive start --name myeslint --cwd /tmp/project npx eslint --init
+  npx noninteractive start node server.js
+
+flags:
+  --name <session>   set session name (default: auto-derived from command)
+  --cwd <dir>        set working directory for the command`);
+        process.exit(0);
+      }
       if (filtered.length < 1) {
         console.error(`usage: noninteractive start [--name <session>] [--cwd <dir>] <cmd> [args...]
 
@@ -947,8 +997,11 @@ example: npx noninteractive stop vercel`);
 hint: use --name for session name, --cwd for working directory.`);
         process.exit(1);
       }
+      if (mutableArgs.includes("--")) {
+        console.error(`hint: the -- separator is not needed. just put the command after the flags.`);
+      }
       const noOpen = mutableArgs.includes("--no-open");
-      const filteredArgs = mutableArgs.filter((a) => a !== "--no-open");
+      const filteredArgs = mutableArgs.filter((a) => a !== "--no-open" && a !== "--");
       console.log(`[installing and running: npx ${filteredArgs.join(" ")}]`);
       return start(["npx", "--yes", ...filteredArgs], noOpen, sessionName, cwd);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "noninteractive",
-  "version": "0.3.27",
+  "version": "0.3.32",
   "type": "module",
   "bin": {
     "noninteractive": "./bin/noninteractive.js"