@lucasschirm/devinp 0.1.11 → 0.1.13

package/README.md

@@ -34,11 +34,21 @@ invalid JSON (… 0 bytes)"*. It now works harder to get the whole document:
 - Prices are captured through a temp file in `os.tmpdir()` — a pipe stream from
   `devin` can be truncated mid-write, so `devinp` points `devin`'s stdout at a
   real file first. The temp file is always removed after the run.
-- **Retries, each one louder about not being interactive.** If an attempt
-  yields no usable JSON, `devinp` runs `devin` again with the "not interactive"
-  conventions set (`PAGER=cat`, `GIT_PAGER=cat`, `TERM=dumb`, `NO_COLOR=1`,
-  `CI=1`) and, on the last try, captures over a pipe instead of a file. The
-  first successful attempt wins, so the healthy path costs one `devin` call.
+- **Retries that take the terminal away from `devin`.** If an attempt yields no
+  usable JSON, `devinp` runs `devin` again, each time stripping more of the
+  interactive terminal off it:
+
+  | # | stdout goes to | environment | controlling terminal |
+  | - | ------------ | -------------------------------------------------------- | --------------------- |
+  | 1 | temp file | yours, untouched | yes — the healthy path, one `devin` call |
+  | 2 | temp file | `PAGER=cat`, `GIT_PAGER=cat`, `TERM=dumb`, `NO_COLOR=1`, `CI=1` | **none** |
+  | 3 | pipe | same | **none** |
+
+  Spawned in its own session, `devin` cannot open `/dev/tty`: a pager or a TUI
+  that paints a "click to show more" view onto the terminal has nowhere to
+  draw, so the document has to come out of stdout. Nothing lands on the screen
+  you are typing in, and Ctrl-C there no longer kills the lookup. The first
+  attempt that parses wins, so the healthy path is unchanged.
 - Noise around the document is tolerated: ANSI colours, an update banner in
   front, or the `pass a slug to --model` hint behind are stripped before the
   JSON is parsed.
@@ -46,9 +56,11 @@ invalid JSON (… 0 bytes)"*. It now works harder to get the whole document:
   prints a warning, and renders the partial table.
 - A real CLI failure (not signed in, network down) is reported as-is instead of
   being retried.
-- If nothing worked, the error lists every attempt — bytes, exit code, stderr,
-  timeout — and how to feed the data in yourself. That always works, because a
-  redirect or a pipe makes `devin` write the whole document in one go:
+- If nothing worked, the error lists all three attempts — bytes, exit code,
+  stderr, timeout — and how to feed the data in yourself. That always works,
+  because a redirect or a pipe makes `devin` write the whole document in one go:
+  (and if it does not, `command -v devin` tells you whether devinp is even
+  talking to the CLI and not to a shell function or agent wrapper around it).
 
   ```sh
   devin models list --format json > models.json && devinp --source models.json
@@ -125,11 +137,25 @@ so `npm publish`, git-URL installs, and `npm link` always ship a fresh bundle.
 
 ## Publishing
 
-`.github/workflows/publish.yml` bumps the patch version and publishes with
-[npm trusted publishing](https://docs.npmjs.com/trusted-publishers) (OIDC —
+`.github/workflows/publish.yml` publishes a patch release on every push to main
+with [npm trusted publishing](https://docs.npmjs.org/trusted-publishers) (OIDC —
 no `NPM_TOKEN`). Requires the trusted publisher to be configured on npmjs.com:
 repository `lucasschirm/devinp`, workflow `publish.yml`.
 
+It **never commits to main** — the branch ruleset requires every change to
+arrive through a pull request, and a rejected release commit used to strand the
+publish step that ran after it (GH013, then `npm publish` skipped). The released
+version therefore lives in the npm registry and in the git tag, not in main's
+`package.json`:
+
+```sh
+git ls-remote --tags origin                 # released versions
+npm view @lucasschirm/devinp version       # latest published
+```
+
+The workflow takes the first patch number that is free in both places, so a tag
+left behind by an older run cannot wedge it.
+
 ## License
 
 MIT

package/bin/command.js

@@ -55095,10 +55095,11 @@ var NON_INTERACTIVE_ENV = {
   NO_COLOR: "1",
   CI: "1"
 };
+var DETACHED_SPAWN = { detached: true, windowsHide: true };
 var STRATEGIES = [
   { label: "temp file", file: true, env: null },
-  { label: "temp file + non-interactive env", file: true, env: NON_INTERACTIVE_ENV },
-  { label: "pipe + non-interactive env", file: false, env: NON_INTERACTIVE_ENV }
+  { label: "temp file, paging disabled, no controlling terminal", file: true, env: NON_INTERACTIVE_ENV, detached: true },
+  { label: "pipe, paging disabled, no controlling terminal", file: false, env: NON_INTERACTIVE_ENV, detached: true }
 ];
 var ANSI = /\u001b\[[0-9;?]*[ -/]*[@-~]|\u001b\][\s\S]*?(?:\u0007|\u001b\\)|\u001b[@-Z\\-_]/g;
 function stripAnsi(text) {
@@ -55120,11 +55121,12 @@ function parseJsonLoosely(text) {
   }
   return null;
 }
-function spawnDevin({ command, env, stdout, timeoutMs }) {
+function spawnDevin({ command, env, stdout, timeoutMs, detached = false }) {
   return new Promise((resolve, reject) => {
     const child = spawn(command, ["models", "list", "--format", "json"], {
       stdio: ["ignore", stdout, "pipe"],
-      ...env ? { env: { ...process.env, ...env } } : {}
+      ...env ? { env: { ...process.env, ...env } } : {},
+      ...detached ? DETACHED_SPAWN : {}
     });
     let stdoutText = "";
     let stderr = "";
@@ -55149,11 +55151,11 @@ function spawnDevin({ command, env, stdout, timeoutMs }) {
     });
   });
 }
-async function captureToFile({ command, env, timeoutMs }) {
+async function captureToFile({ command, env, timeoutMs, detached }) {
   const file = join2(tmpdir(), `devinp-models-${process.pid}-${Date.now()}.json`);
   const handle = await open(file, "w");
   try {
-    const result = await spawnDevin({ command, env, stdout: handle.fd, timeoutMs });
+    const result = await spawnDevin({ command, env, stdout: handle.fd, timeoutMs, detached });
     result.stdout = await readFile2(file, "utf8");
     return result;
   } finally {
@@ -55167,7 +55169,7 @@ async function runDevinJson({ command = "devin", timeoutMs = DEVIN_TIMEOUT_MS }
   for (const [index, strategy] of STRATEGIES.entries()) {
     let result;
     try {
-      result = strategy.file ? await captureToFile({ command, env: strategy.env, timeoutMs }) : await spawnDevin({ command, env: strategy.env, stdout: "pipe", timeoutMs });
+      result = strategy.file ? await captureToFile({ command, env: strategy.env, timeoutMs, detached: strategy.detached }) : await spawnDevin({ command, env: strategy.env, stdout: "pipe", timeoutMs, detached: strategy.detached });
     } catch (error) {
       if (error.code === "ENOENT") throw error;
       attempts.push({ label: strategy.label, bytes: 0, code: null, stderr: "", timedOut: false, note: error.message });
@@ -55222,7 +55224,7 @@ function describeFailure(attempts, timeoutMs) {
   const lines = [];
   if (bytes === 0) {
     lines.push(
-      'Unable to get model prices: `devin models list --format json` exited without writing the JSON to stdout (0 bytes). Something between devinp and the CLI \u2014 a pager, an interactive "click to show more" view, or a shell/agent wrapper around `devin` \u2014 kept the output for the screen.'
+      'Unable to get model prices: `devin models list --format json` exited without writing the JSON to stdout (0 bytes). devinp re-ran it with paging disabled and in a session with no controlling terminal, so the CLI had no terminal to paint a "click to show more" view on \u2014 whatever swallowed the output sits between devinp and `devin` (a shell function or agent wrapper around it, or a broken install).'
     );
   } else {
     let parseError = "";
@@ -55239,7 +55241,7 @@ function describeFailure(attempts, timeoutMs) {
   lines.push("Capture the JSON yourself and hand it to devinp \u2014 piping or redirecting makes `devin` write the whole document in one go:");
   lines.push("  devin models list --format json > models.json && devinp --source models.json");
   lines.push("  devin models list --format json | devinp --source=-");
-  lines.push("If a plain `devin models list --format json > models.json` still prints a paged view, update the CLI (`devin update`).");
+  lines.push("Check `command -v devin` \u2014 if that is not the real CLI, devinp is asking the wrong program. If a plain redirect still prints a paged view, update the CLI (`devin update`).");
   return lines.join("\n");
 }
 async function readStdin() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lucasschirm/devinp",
-  "version": "0.1.11",
+  "version": "0.1.13",
   "description": "Pretty pricing table for Devin models, powered by `devin models list`",
   "type": "module",
   "bin": {