ada-agent 0.6.0 → 0.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ada-agent",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "A from-zero terminal coding agent with a Cursor-style routing backend, ~285 skills, MCP connectors, and ask/plan/auto modes",
   "type": "module",
   "license": "MIT",

package/src/client/autostart.ts

@@ -30,13 +30,22 @@ export function healthUrl(backendUrl: string): string {
   }
 }
 
-async function probe(url: string, timeoutMs = 800): Promise<boolean> {
-  try {
-    const res = await fetch(url, { signal: AbortSignal.timeout(timeoutMs) });
-    return res.ok;
-  } catch {
-    return false;
-  }
+// Plain node:http with agent:false, NOT fetch: undici's keep-alive socket from a probe lingers into
+// process teardown and deterministically prints "Assertion failed: !(handle->flags &
+// UV_HANDLE_CLOSING)" on Windows at exit. agent:false closes the socket with the response.
+function probe(url: string, timeoutMs = 800): Promise<boolean> {
+  return new Promise((resolve) => {
+    import("node:http")
+      .then((http) => {
+        const req = http.get(url, { agent: false, timeout: timeoutMs }, (res) => {
+          res.resume(); // drain so the socket can close
+          resolve((res.statusCode ?? 500) < 400);
+        });
+        req.on("timeout", () => req.destroy());
+        req.on("error", () => resolve(false));
+      })
+      .catch(() => resolve(false));
+  });
 }
 
 /** Resolved path to bin/ada-server.mjs (sibling of bin/ada.mjs, packaged in the npm tarball). */

package/src/client/tools.ts

@@ -99,14 +99,21 @@ export function formatFile(abs: string): boolean {
 }
 
 // node-pty gives the bash tool a real terminal. It's a required dependency; if the native build is
-// ever broken on a platform, fall back to spawnSync so bash still works.
-const pty: typeof PtyType | null = (() => {
-  try {
-    return createRequire(import.meta.url)("node-pty") as typeof PtyType;
-  } catch {
-    return null;
+// ever broken on a platform, fall back to spawnSync so bash still works. Loaded LAZILY on the first
+// bash call: merely loading the native module on Windows sets up async handles whose teardown races
+// process.exit and prints "Assertion failed: !(handle->flags & UV_HANDLE_CLOSING)" — commands that
+// never spawn a PTY (--version, catalog, --list-models, …) shouldn't pay that.
+let ptyMod: typeof PtyType | null | undefined;
+function getPty(): typeof PtyType | null {
+  if (ptyMod === undefined) {
+    try {
+      ptyMod = createRequire(import.meta.url)("node-pty") as typeof PtyType;
+    } catch {
+      ptyMod = null;
+    }
   }
-})();
+  return ptyMod;
+}
 
 // Built via new RegExp (string escapes) so no literal ESC/BEL bytes live in the source.
 const ANSI = new RegExp("[\\u001B\\u009B][\\[\\]()#;?]*(?:(?:[a-zA-Z\\d]*(?:;[a-zA-Z\\d]*)*)?\\u0007|(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~])", "g");
@@ -120,7 +127,7 @@ function runPty(command: string, timeoutMs = 120_000): Promise<{ output: string;
     const win = process.platform === "win32";
     const shell = win ? process.env.COMSPEC ?? "cmd.exe" : process.env.SHELL ?? "/bin/bash";
     const shellArgs = win ? ["/c", command] : ["-lc", command];
-    const p = pty!.spawn(shell, shellArgs, { name: "xterm-256color", cols: 120, rows: 30, cwd: process.cwd(), env: process.env as Record<string, string> });
+    const p = getPty()!.spawn(shell, shellArgs, { name: "xterm-256color", cols: 120, rows: 30, cwd: process.cwd(), env: process.env as Record<string, string> });
     let out = "";
     const cap = 10 * 1024 * 1024;
     p.onData((d) => {
@@ -360,7 +367,7 @@ export const tools: Tool[] = [
     needsApproval: true,
     async run(args) {
       const command = String(args.command);
-      if (pty) {
+      if (getPty()) {
         const { output, code } = await runPty(command);
         return { output: `exit ${code ?? "null"}\n${spillIfHuge(stripAnsi(output).trim() || "(no output)")}`, isError: code !== 0 };
       }