bwb-browser 4.0.1 → 4.1.0

package/lib/vigil.mjs

@@ -7,7 +7,11 @@
  * watcher stream carries memory samples. That IS the "instant" channel.
  */
 
-import { execSync } from "child_process";
+import { execFile } from "child_process";
+import { promisify } from "util";
+import { platform } from "os";
+
+const execFileAsync = promisify(execFile);
 
 // 1GB-box budgets (MB) when lean; roomier desktop defaults otherwise.
 // Override any via BWB_WARN_MB / BWB_CRIT_MB. resolveBudgets() is called
@@ -24,33 +28,50 @@ export function resolveBudgets(isLean) {
 
 const MB = 1024 * 1024;
 
+// The process table is sampled at most every SAMPLE_TTL_MS. Reading it with a
+// synchronous `ps` on EVERY tool call blocked the event loop (including
+// browser_watch's event handlers) and listed every process on the box.
+const SAMPLE_TTL_MS = 5000;
+let sampleCache = { at: 0, procs: null };
+
+function readProcessTable() {
+  if (Date.now() - sampleCache.at < SAMPLE_TTL_MS) return sampleCache.procs;
+  sampleCache = { at: Date.now(), procs: null };
+  if (platform() === "win32") return null; // no ps: resource readings degrade to self
+  execFileAsync("ps", ["-eo", "pid=,ppid=,rss="], { timeout: 3000, maxBuffer: 4 * 1024 * 1024 })
+    .then(({ stdout }) => {
+      const procs = stdout.trim().split("\n").map((l) => {
+        const [pid, ppid, rss] = l.trim().split(/\s+/).map(Number);
+        return { pid, ppid, rss };
+      }).filter((p) => Number.isFinite(p.pid));
+      // Only publish if no newer sample landed while we were reading.
+      if (Date.now() - sampleCache.at >= SAMPLE_TTL_MS - 100) sampleCache = { at: Date.now(), procs };
+    })
+    .catch(() => {});
+  return sampleCache.procs;
+}
+
 function selfMb() {
   return Math.round(process.memoryUsage().rss / MB);
 }
 
-// Sum RSS of a pid + its descendants via ps. Null when unmeasurable.
+// Sum RSS of a pid + its descendants via the cached process table.
+// Async warm-up: the first call returns null, later ones are accurate.
 function treeMb(rootPid) {
   if (!rootPid) return null;
-  try {
-    const out = execSync("ps -o pid=,ppid=,rss= -e", { encoding: "utf8", timeout: 3000 });
-    const procs = out.trim().split("\n").map((l) => {
-      const [pid, ppid, rss] = l.trim().split(/\s+/).map(Number);
-      return { pid, ppid, rss };
-    });
-    const kids = new Set([Number(rootPid)]);
-    let grew = true;
-    while (grew) {
-      grew = false;
-      for (const p of procs) {
-        if (kids.has(p.ppid) && !kids.has(p.pid)) { kids.add(p.pid); grew = true; }
-      }
+  const procs = readProcessTable();
+  if (!procs) return null;
+  const kids = new Set([Number(rootPid)]);
+  let grew = true;
+  while (grew) {
+    grew = false;
+    for (const p of procs) {
+      if (kids.has(p.ppid) && !kids.has(p.pid)) { kids.add(p.pid); grew = true; }
     }
-    let kb = 0;
-    for (const p of procs) if (kids.has(p.pid) && Number.isFinite(p.rss)) kb += p.rss;
-    return Math.round(kb / 1024);
-  } catch {
-    return null;
   }
+  let kb = 0;
+  for (const p of procs) if (kids.has(p.pid) && Number.isFinite(p.rss)) kb += p.rss;
+  return Math.round(kb / 1024);
 }
 
 /** Snapshot: {mcpMb, chromiumMb|null, tabCount} */
@@ -61,6 +82,8 @@ export function sampleResources(browserPid, tabCount = 0) {
 /**
  * Policy verdict: 'ok' | 'watch' | 'critical'.
  * critical = shed load NOW (hibernate tabs → teardown). watch = warn only.
+ * Budgets are tuned for a 1GB box (lean). Desktop defaults are deliberately
+ * high, so 'critical' rarely fires there — that is intentional, not a bug.
  */
 export function assess(sample) {
   const total = (sample.chromiumMb || 0) + sample.mcpMb;
@@ -73,5 +96,5 @@ export function assess(sample) {
 export function resourceFooter(sample, note = "") {
   const c = sample.chromiumMb === null ? "browser:off" : `browser:${sample.chromiumMb}MB`;
   const trend = assess(sample);
-  return `[bwb resources] mcp:${sample.mcpMb}MB ${c} tabs:${sample.tabCount} state:${trend}${note ? " " + note : ""}]`;
+  return `[bwb resources] mcp:${sample.mcpMb}MB ${c} tabs:${sample.tabCount} state:${trend}${note ? " " + note : ""}`;
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "bwb-browser",
-  "version": "4.0.1",
-  "description": "Browser Without Bloat — lightweight MCP server that gives AI agents browser superpowers. Static-first fetch ladder (Chromium only when JS demands it), lean profiles for Termux/1GB VPS, resource vigilance, tab journaling. Runs anywhere. Stays lean like air.",
+  "version": "4.1.0",
+  "description": "Browser Without Bloat \u2014 lightweight MCP server that gives AI agents browser superpowers. Static-first fetch ladder (Chromium only when JS demands it), lean profiles for Termux/1GB VPS, resource vigilance, tab journaling. Runs anywhere. Stays lean like air.",
   "bin": {
     "bwb": "bin/bwb"
   },
@@ -15,7 +15,12 @@
   ],
   "scripts": {
     "start": "node server.mjs",
-    "test": "node --check server.mjs && node --check lib/*.mjs"
+    "test": "node --test test/*.test.mjs",
+    "lint": "for f in server.mjs lib/*.mjs; do node --check \"$f\" || exit 1; done",
+    "smoke": "node test/smoke.mjs",
+    "test:browser": "node --test test/integration.mjs",
+    "test:all": "npm run lint && npm test && npm run verify:docs && npm run smoke && npm run test:browser",
+    "verify:docs": "node scripts/verify-docs.mjs"
   },
   "keywords": [
     "mcp",
@@ -42,7 +47,7 @@
     "fingerprint",
     "session",
     "lightweight",
-    "zero-dependencies",
+    "low-dependency",
     "mobile",
     "devtools"
   ],
@@ -61,9 +66,12 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
-    "chrome-remote-interface": "^0.34.0",
-    "zod": "^3.22.0",
     "@mozilla/readability": "^0.6.0",
-    "linkedom": "^0.18.13"
+    "chrome-remote-interface": "^0.34.0",
+    "linkedom": "^0.18.13",
+    "zod": "^3.22.0"
+  },
+  "devDependencies": {
+    "jsdom": "^26.1.0"
   }
 }