jaitus 0.3.12 → 0.3.14

package/bin/jaitus.js

@@ -1,73 +1,86 @@
-#!/usr/bin/env node
-// The app is imported only after an unfinished update has completed: importing it
-// statically would load a half-installed copy and crash before any check could run.
-// Node 20 is the floor — checked FIRST: loading the app on an older Node crashed in its
-// dependencies before this friendly message could print.
-if (Number(process.versions.node.split('.')[0]) < 20) {
-  process.stderr.write(`jaitus needs Node 20 or newer (you have ${process.versions.node}).
-`);
-  process.exit(1);
-}
-
-import { waitForPendingUpdate } from '../src/pendingUpdate.js';
-
-await waitForPendingUpdate().catch(() => false);
-const { default: main } = await import('../src/index.js');
-
-// Node 20 is the floor: the CLI uses global fetch, and on 18 it exists but streaming
-// bodies behave differently enough to matter. Better a clear message than a confusing
-// failure halfway through a task.
-const major = Number(process.versions.node.split('.')[0]);
-if (major < 20) {
-  process.stderr.write(`jaitus needs Node 20 or newer (you have ${process.versions.node}).\n`);
-  process.exit(1);
-}
-
-// Ctrl-C should leave the terminal usable rather than mid-escape-sequence. Mouse
-// reporting has to be turned off here too: the terminal keeps sending reports after the
-// process is gone, and every click in the shell afterwards prints escape codes.
-const restoreTerminal = () => {
-  // Only to a real terminal. Written unconditionally, these codes end up in the output
-  // of every piped invocation — `jaitus models --json | jq` would be handed a JSON
-  // document with escape sequences glued to the end of it.
-  if (!process.stdout.isTTY) return;
-  try { process.stdout.write('\x1b[?1006l\x1b[?1002l\x1b[?1000l\x1b[?25h'); } catch { /* going away anyway */ }
-};
-process.on('SIGINT', () => {
-  restoreTerminal();
-  process.stdout.write('\n');
-  process.exit(130);
-});
-process.on('exit', restoreTerminal);
-
-/**
- * Finish WITHOUT process.exit().
- *
- * Calling process.exit() after two fetches have shared a keep-alive connection crashes
- * Node on Windows with `Assertion failed: !(handle->flags & UV_HANDLE_CLOSING)` — a
- * libuv teardown race, not something this code does wrong, but it would print a crash
- * after every successful run. Closing undici's dispatcher first does not help; only not
- * calling exit() does.
- *
- * Setting exitCode and letting the loop drain is also measurably faster (295ms vs 406ms
- * on the same work), because exit() was racing teardown rather than waiting for it. The
- * dispatcher is still closed first so idle sockets are released immediately instead of
- * holding the process open for their keep-alive timeout.
- */
-async function finish(code) {
-  process.exitCode = code ?? 0;
-  try {
-    const dispatcher = globalThis[Symbol.for('undici.globalDispatcher.1')];
-    if (dispatcher?.close) await dispatcher.close();
-  } catch {
-    // Internal and version-dependent; if it is gone, natural drain still gets there.
-  }
-}
-
-main(process.argv.slice(2))
-  .then(finish)
-  .catch(async (err) => {
-    process.stderr.write(`\x1b[31m✗\x1b[39m ${err?.message || err}\n`);
-    if (process.env.JAITUS_DEBUG) process.stderr.write(`${err?.stack}\n`);
-    await finish(1);
-  });
+#!/usr/bin/env node
+// The app is imported only after an unfinished update has completed: importing it
+// statically would load a half-installed copy and crash before any check could run.
+// Node 20 is the floor — checked FIRST: loading the app on an older Node crashed in its
+// dependencies before this friendly message could print.
+if (Number(process.versions.node.split('.')[0]) < 20) {
+  process.stderr.write(`jaitus needs Node 20 or newer (you have ${process.versions.node}).
+`);
+  process.exit(1);
+}
+
+import { waitForPendingUpdate } from '../src/pendingUpdate.js';
+
+await waitForPendingUpdate().catch(() => false);
+// React picks its build from NODE_ENV the moment it is first loaded, and the app is
+// loaded right here. Unset (or a developer's own NODE_ENV=development), it picked the
+// DEVELOPMENT build, which records a performance.measure() for every component of
+// every frame — and Node keeps every one of them. At the spinner's frame rate that is
+// thousands a minute: a long session grew past 4 GB and died in "JavaScript heap out
+// of memory", getting slower the whole way (each step cost ~5× the CPU, then GC
+// thrash). So the app always loads production React, and the user's own NODE_ENV is
+// put back straight after: every command the agent runs inherits this environment,
+// and NODE_ENV=production would make `npm install` skip devDependencies.
+const userNodeEnv = process.env.NODE_ENV;
+process.env.NODE_ENV = 'production';
+const { default: main } = await import('../src/index.js');
+if (userNodeEnv === undefined) delete process.env.NODE_ENV;
+else process.env.NODE_ENV = userNodeEnv;
+
+// Node 20 is the floor: the CLI uses global fetch, and on 18 it exists but streaming
+// bodies behave differently enough to matter. Better a clear message than a confusing
+// failure halfway through a task.
+const major = Number(process.versions.node.split('.')[0]);
+if (major < 20) {
+  process.stderr.write(`jaitus needs Node 20 or newer (you have ${process.versions.node}).\n`);
+  process.exit(1);
+}
+
+// Ctrl-C should leave the terminal usable rather than mid-escape-sequence. Mouse
+// reporting has to be turned off here too: the terminal keeps sending reports after the
+// process is gone, and every click in the shell afterwards prints escape codes.
+const restoreTerminal = () => {
+  // Only to a real terminal. Written unconditionally, these codes end up in the output
+  // of every piped invocation — `jaitus models --json | jq` would be handed a JSON
+  // document with escape sequences glued to the end of it.
+  if (!process.stdout.isTTY) return;
+  try { process.stdout.write('\x1b[?1006l\x1b[?1002l\x1b[?1000l\x1b[?25h'); } catch { /* going away anyway */ }
+};
+process.on('SIGINT', () => {
+  restoreTerminal();
+  process.stdout.write('\n');
+  process.exit(130);
+});
+process.on('exit', restoreTerminal);
+
+/**
+ * Finish WITHOUT process.exit().
+ *
+ * Calling process.exit() after two fetches have shared a keep-alive connection crashes
+ * Node on Windows with `Assertion failed: !(handle->flags & UV_HANDLE_CLOSING)` — a
+ * libuv teardown race, not something this code does wrong, but it would print a crash
+ * after every successful run. Closing undici's dispatcher first does not help; only not
+ * calling exit() does.
+ *
+ * Setting exitCode and letting the loop drain is also measurably faster (295ms vs 406ms
+ * on the same work), because exit() was racing teardown rather than waiting for it. The
+ * dispatcher is still closed first so idle sockets are released immediately instead of
+ * holding the process open for their keep-alive timeout.
+ */
+async function finish(code) {
+  process.exitCode = code ?? 0;
+  try {
+    const dispatcher = globalThis[Symbol.for('undici.globalDispatcher.1')];
+    if (dispatcher?.close) await dispatcher.close();
+  } catch {
+    // Internal and version-dependent; if it is gone, natural drain still gets there.
+  }
+}
+
+main(process.argv.slice(2))
+  .then(finish)
+  .catch(async (err) => {
+    process.stderr.write(`\x1b[31m✗\x1b[39m ${err?.message || err}\n`);
+    if (process.env.JAITUS_DEBUG) process.stderr.write(`${err?.stack}\n`);
+    await finish(1);
+  });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jaitus",
-  "version": "0.3.12",
+  "version": "0.3.14",
   "description": "Jaitus — a coding agent in your terminal.",
   "type": "module",
   "bin": {
@@ -16,7 +16,7 @@
     "node": ">=20"
   },
   "scripts": {
-    "test": "node tests/source-sanity.js && node tests/rules.js && node tests/run.js && node tests/commands.js && node tests/login.js && node tests/session.js && node tests/openUrl.js && node tests/ink-tui.js && node tests/tui-render.js && node tests/mouse-e2e.js && node tests/environment.js && node tests/update.js && node tests/continuation.js && node tests/steps.js && node tests/files.js && node tests/processes.js && node tests/security.js && node tests/context-budget.js && node tests/context-trim.js",
+    "test": "node tests/source-sanity.js && node tests/react-build.js && node tests/stalls.js && node tests/clipboard-write.js && node tests/rewind-images.js && node tests/rules.js && node tests/run.js && node tests/commands.js && node tests/login.js && node tests/session.js && node tests/openUrl.js && node tests/ink-tui.js && node tests/tui-render.js && node tests/mouse-e2e.js && node tests/environment.js && node tests/update.js && node tests/service-notices.js && node tests/continuation.js && node tests/steps.js && node tests/files.js && node tests/processes.js && node tests/security.js && node tests/context-budget.js && node tests/context-trim.js",
     "preview": "node scripts/tui-preview.js",
     "prepublishOnly": "npm test"
   },