donsetch 3.4.4 → 3.5.1

package/README.md

@@ -6,7 +6,7 @@
 [![CI](https://img.shields.io/github/actions/workflow/status/dondai44423/donsetch/ci.yml?label=CI&style=flat-square)](https://github.com/dondai44423/donsetch/actions/workflows/ci.yml)
 [![License: AGPL-3.0](https://img.shields.io/badge/License-AGPL--3.0-00d4aa?style=flat-square)](https://github.com/dondai44423/donsetch/blob/master/LICENSE)
 
-DonSeTch gives AI agents web research from a single local process — fetch any URL, search across 10+ engines, and crawl multi-page docs. Chrome-true TLS fingerprinting, bot-wall bypass, PDF extraction with OCR, semantic reranking. Zero API keys.
+DonSeTch gives AI agents web research from a single local process : fetch any URL, search across 10+ engines, and crawl multi-page docs. Chrome-true TLS fingerprinting, bot-wall bypass, PDF extraction with OCR, semantic reranking. Zero API keys.
 
 ## Install
 
@@ -60,6 +60,6 @@ donsetch update
 
 ## License
 
-AGPL-3.0 — Copyright (c) 2026 Bishesh Bhandari
+AGPL-3.0 : Copyright (c) 2026 Bishesh Bhandari
 
 Full documentation: [github.com/dondai44423/donsetch](https://github.com/dondai44423/donsetch)

package/install.js

@@ -4,7 +4,7 @@
 // donsetch postinstall: download the prebuilt binary for this platform
 // from GitHub Releases, verify SHA256, and extract to ./binaries/.
 //
-// The binary is NOT bundled in the npm package — it's fetched at
+// The binary is NOT bundled in the npm package : it's fetched at
 // install time from the GitHub release matching this package version.
 // This keeps the npm registry clean (no 35 MB binary tarballs) and
 // uses the same release artifacts that manual users download.
@@ -40,8 +40,8 @@ const plat = PLATFORMS[platKey];
 
 if (!plat) {
   const known = {
-    'darwin-x64': 'prebuilt binaries exist — update donsetch to a version that ships one',
-    'win32-arm64': 'no prebuilt binary yet — build from source (see below)',
+    'darwin-x64': 'prebuilt binaries exist : update donsetch to a version that ships one',
+    'win32-arm64': 'no prebuilt binary yet : build from source (see below)',
   }[platKey];
   console.error(`donsetch: unsupported platform ${platKey}${known ? ` (${known})` : ''}`);
   console.error('');
@@ -58,7 +58,7 @@ if (!plat) {
 
 // ── musl detection (Alpine etc.) ────────────────────────────────
 // The Linux binaries are glibc-linked. On musl systems they install
-// fine but can never exec (missing ld-linux loader) — fail HERE with
+// fine but can never exec (missing ld-linux loader) : fail HERE with
 // the actual cause instead of a cryptic spawn error on first run.
 //
 // Checking for /lib/ld-musl-*.so.1 existence is a false positive on
@@ -143,7 +143,7 @@ if (fs.existsSync(binaryPath)) {
     console.log(`donsetch: binary already present (${plat.binary})`);
     process.exit(0);
   }
-  console.log(`donsetch: leftover ${plat.binary} is ${size} bytes — re-downloading`);
+  console.log(`donsetch: leftover ${plat.binary} is ${size} bytes : re-downloading`);
   try { fs.unlinkSync(binaryPath); } catch (_) {}
 }
 
@@ -236,7 +236,7 @@ async function main() {
 
   // 4. Extract (tar is built into Linux, macOS, and Windows 10+)
   console.log('donsetch: extracting...');
-  // execFileSync: no shell, no string interpolation — the install
+  // execFileSync: no shell, no string interpolation : the install
   // path (which can contain quotes/spaces) is passed as argv.
   execFileSync('tar', ['xzf', tarball, '-C', binDir], { stdio: 'inherit' });
 
@@ -244,7 +244,7 @@ async function main() {
   try { fs.unlinkSync(tarball); } catch (_) {}
   try { fs.unlinkSync(checksumFile); } catch (_) {}
 
-  // 6. Verify the binary exists after extraction (BEFORE chmod —
+  // 6. Verify the binary exists after extraction (BEFORE chmod :
   //    chmod on a missing file throws an opaque error).
   if (!fs.existsSync(binaryPath)) {
     console.error(`donsetch: expected ${plat.binary} not found after extraction`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "donsetch",
-  "version": "3.4.4",
+  "version": "3.5.1",
   "description": "Web fetch, search and crawl for AI agents. Zero API keys. Chrome-true TLS.",
   "license": "AGPL-3.0-only",
   "author": "Bishesh Bhandari",

package/pi-extension.ts

@@ -1,5 +1,5 @@
 /**
- * DonSeTch pi extension — bridges the donsetch MCP binary into pi.
+ * DonSeTch pi extension : bridges the donsetch MCP binary into pi.
  *
  * `pi install npm:donsetch` installs this package. At session_start the
  * extension spawns `donsetch mcp`, performs the MCP handshake, discovers
@@ -8,14 +8,14 @@
  *
  * Zero maintenance: tool definitions are fetched dynamically from the
  * binary. When donsetch adds or changes tools, this extension picks
- * them up automatically — no code changes needed here.
+ * them up automatically : no code changes needed here.
  *
  * Auto-download: if the binary is missing (e.g. postinstall was
  * blocked by npm 10+), the extension runs install.js at session_start
  * to fetch it from GitHub Releases.
  *
  * Custom TUI: each tool has clean renderCall/renderResult showing
- * a compact summary card — not the full raw output. The LLM still
+ * a compact summary card : not the full raw output. The LLM still
  * receives complete content; the user sees a minimal status line +
  * one-line preview. Amber theme matching DonSeTch's identity.
  */
@@ -31,14 +31,20 @@ const INIT_TIMEOUT_MS = 10_000;
 const CALL_TIMEOUT_MS = 120_000;
 const SHUTDOWN_GRACE_MS = 2_000;
 
+// A raw process.stderr.write bypasses pi's TUI paint cycle and
+// corrupts the viewport (worse across parallel agents). Gate it here.
+const DEBUG_MODE =
+  (process.env.DONSETCH_DEBUG ?? "").length > 0 ||
+  (process.env.DEBUG ?? "").length > 0;
+
 // ── TUI rendering note ──
 // Pi's TUI wraps tool calls in its own green (success) / red (failure)
-// highlight. We output PLAIN TEXT only — no ANSI codes at all.
+// highlight. We output PLAIN TEXT only : no ANSI codes at all.
 // pi-tui's truncateToWidth injects \x1b[0m RESET around the ellipsis,
 // which breaks pi's overlay mid-line. We use our own truncate()
 // that adds zero ANSI codes.
 
-/** Plain text truncation — no ANSI codes. */
+/** Plain text truncation : no ANSI codes. */
 function truncate(text: string, maxLen: number): string {
   if (text.length <= maxLen) return text;
   if (maxLen <= 3) return text.slice(0, maxLen);
@@ -120,7 +126,7 @@ function startServer(): Promise<void> {
     try {
       // --supervised: crash-only daemon. If the MCP server is SIGKILLed
       // (OOM, crash), the supervisor respawns it and replays in-flight
-      // requests — pi users never see a dead tool until the process
+      // requests : pi users never see a dead tool until the process
       // itself dies.
       proc = spawn(binaryPath, ["mcp", "--supervised"], {
         stdio: ["pipe", "pipe", "pipe"],
@@ -158,8 +164,29 @@ function startServer(): Promise<void> {
       }
     });
 
+    // The daemon writes diagnostics to stderr (Xvfb warnings,
+    // fallback notices, ghost telemetry). Pi's TUI surfaces stderr
+    // as user-facing output: raw forwarding used to pop non-fatal
+    // warnings up (issue #95). Gate it:
+    // - DONSETCH_DEBUG / DEBUG set: forward everything (dev mode)
+    // - otherwise only crash/fatal lines survive; the rest is
+    //   tool-level information the caller already has
+    const FATAL_RE = /fatal|panic|crash|SIGSEGV|abort|OOM|out of memory/i;
+    let stderrTail = "";
     proc.stderr?.on("data", (chunk: Buffer) => {
-      process.stderr.write(chunk);
+      if (DEBUG_MODE) {
+        process.stderr.write(chunk);
+        return;
+      }
+      stderrTail = (stderrTail + chunk.toString()).slice(-2048);
+      const lines = stderrTail.split("\n");
+      stderrTail = lines.pop() || "";
+      for (const line of lines) {
+        const trimmed = line.trim();
+        if (trimmed && FATAL_RE.test(trimmed)) {
+          process.stderr.write(`[donsetch] ${trimmed}\n`);
+        }
+      }
     });
 
     proc.on("error", (err) => {
@@ -518,7 +545,7 @@ export default function (pi: ExtensionAPI) {
           } else if (args?.query) {
             key = truncate(`"${args.query}"`, 50);
           }
-          // Plain text only — no ANSI codes. Pi wraps tool calls
+          // Plain text only : no ANSI codes. Pi wraps tool calls
           // in its own green (success) / red (failure) highlight.
           // Any ANSI we emit breaks pi's overlay mid-line.
           return new Text(
@@ -567,7 +594,7 @@ export default function (pi: ExtensionAPI) {
             line2 = d.preview;
           }
 
-          // Plain text only — no ANSI. Pi handles all coloring.
+          // Plain text only : no ANSI. Pi handles all coloring.
           const line1 = `${toolName} \u00B7 ${meta}`;
           const output = line2
             ? `${line1}\n  ${line2}`
@@ -578,7 +605,11 @@ export default function (pi: ExtensionAPI) {
       });
     }
 
-    process.stderr.write(`[donsetch] ${mcpTools.length} tools registered: ${toolNames.join(", ")}\n`);
+    // Fires every session_start (unlike the failure paths above), so
+    // this one corrupted the viewport on every normal run.
+    if (DEBUG_MODE) {
+      process.stderr.write(`[donsetch] ${mcpTools.length} tools registered: ${toolNames.join(", ")}\n`);
+    }
   });
 
   pi.on("session_shutdown", () => {