@jhizzard/termdeck 1.8.0 → 1.8.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jhizzard/termdeck",
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "Browser-based terminal multiplexer with metadata overlays, panel flashback memory recall, and AI-aware session management",
   "bin": {
     "termdeck": "./packages/cli/src/index.js"
@@ -25,13 +25,14 @@
     "packages/server",
     "packages/client",
     "packages/cli",
-    "packages/stack-installer"
+    "packages/stack-installer",
+    "packages/mcp-bridge"
   ],
   "scripts": {
     "dev": "node packages/server/src/index.js",
     "server": "node packages/server/src/index.js",
     "start": "NODE_ENV=production node packages/cli/src/index.js",
-    "test": "node --test packages/server/tests/**/*.test.js packages/cli/tests/**/*.test.js packages/stack-installer/tests/**/*.test.js",
+    "test": "WEB_CHAT_DRIVER_NO_BROWSER=1 node --test packages/server/tests/**/*.test.js packages/cli/tests/**/*.test.js packages/stack-installer/tests/**/*.test.js packages/mcp-bridge/test/*.test.js packages/web-chat-driver/tests/*.test.js",
     "install:app": "bash install.sh",
     "sync-rumen-functions": "bash scripts/sync-rumen-functions.sh",
     "sync:agents": "node scripts/sync-agent-instructions.js"

package/packages/server/src/agent-adapters/agy.js

@@ -309,32 +309,25 @@ function bootPromptTemplate(lane = {}, sprint = {}) {
 }
 
 // ──────────────────────────────────────────────────────────────────────────
-// mcpConfig — UNVERIFIED at lane time. The brief specifies the path
-// `~/.gemini/antigravity-cli/mcp_config.json`, which does NOT exist on disk yet,
-// and agy's `settings.json` carries no `mcpServers` key — so agy's actual
-// MCP-registry read path could not be confirmed against the binary. Modeled on
-// the Gemini-family record shape (`mcpServers.NAME = {command,args,env}`, the
-// same schema gemini.js uses) since Antigravity is a Gemini-family CLI. The
-// shared mcp-autowire helper would CREATE this file on panel spawn. This is a
-// non-load-bearing nicety (auto-wiring Mnestra into agy panels); if a future
-// probe shows agy reads MCP from settings.json or another path/shape, correct
-// here. Env-key omission discipline matches gemini.js (concrete-or-omit; agy,
-// like Claude/Gemini, does not shell-expand `${VAR}` in MCP env).
+// mcpConfig — null (Mnestra MCP auto-wire intentionally OFF for agy).
+// VERIFIED 2026-06-08 (live 4-CLI 360): Antigravity's MCP is NOT file-config-
+// driven — agy's MCP servers are managed by its embedded "exa" language-server
+// (RPCs `RefreshMcpServers` / `GetMcpServerStates`; type
+// `gemini.GeminiMCPServerConfig`), not a readable `mcp_config.json`. Ruled out
+// empirically against a LIVE agy panel: a de-secreted mnestra block written to
+// BOTH `~/.gemini/config/mcp_config.json` AND the appDataDir
+// `~/.gemini/antigravity-cli/mcp_config.json` left agy reporting
+// `NO-MNESTRA-TOOL`; `~/.gemini/settings.json` already carries mnestra (gemini
+// reads it) yet agy ignores it; `agy --help` exposes no `mcp` subcommand and
+// `agy plugin list` is empty. A file-based mcpConfig here only targets a dead
+// path, so it is `null` → the shared mcp-autowire helper cleanly skips (exactly
+// the Claude case). Wiring Mnestra into agy is a deferred follow-up via the
+// Antigravity language-server registration mechanism (likely IDE- /
+// `RefreshMcpServers`-driven). This was always a non-load-bearing nicety: agy's
+// PTY panel + the memory CAPTURE path (source_agent=antigravity, Sprint 70) both
+// work; only the agy-side memory READ is deferred.
 // ──────────────────────────────────────────────────────────────────────────
 
-const MNESTRA_ENV_KEYS = ['SUPABASE_URL', 'SUPABASE_SERVICE_ROLE_KEY', 'OPENAI_API_KEY'];
-
-function buildMnestraBlock({ secrets } = {}) {
-  const env = {};
-  for (const key of MNESTRA_ENV_KEYS) {
-    const value = secrets && secrets[key];
-    if (typeof value === 'string' && value.length > 0 && !/^\$\{[^}]*\}$/.test(value)) {
-      env[key] = value;
-    }
-  }
-  return { mnestra: { command: 'mnestra', args: [], env } };
-}
-
 const antigravityAdapter = {
   name: 'antigravity',
   sessionType: 'antigravity',
@@ -385,12 +378,10 @@ const antigravityAdapter = {
   // true (bracketed-paste fast path), flip to false if a lane-time test shows
   // the TUI input box eats the paste markers.
   acceptsPaste: true,
-  mcpConfig: {
-    path: '~/.gemini/antigravity-cli/mcp_config.json',
-    format: 'json',
-    mcpServersKey: 'mcpServers',
-    mnestraBlock: buildMnestraBlock,
-  },
+  // See the mcpConfig note above — Antigravity MCP is language-server-mediated,
+  // not file-config; null so mcp-autowire skips (Claude-style) instead of writing
+  // a dead-path file. agy memory READ is a deferred follow-up; CAPTURE works.
+  mcpConfig: null,
 };
 
 module.exports = antigravityAdapter;

package/packages/server/src/index.js

@@ -1590,6 +1590,57 @@ function createServer(config) {
   // sync), wire screencast→WS, completion→capture, disconnect→close. Fail-soft
   // at every step — a missing/partial/throwing driver degrades the panel to
   // 'errored', never crashes the server.
+  // Render-watchdog: self-heal a wedged web-chat cold-start. On a brand-new
+  // browser profile the first load very occasionally paints nothing (empty
+  // body.innerText) even though attach + screencast are healthy; a full
+  // re-navigation clears it (a reload does NOT). Polls briefly for paint, then
+  // re-navigates up to `attempts` times. Returns true if the page painted (or
+  // we cannot measure — never block readiness on the watchdog itself), false if
+  // it stayed blank. Provider-neutral: "painted" == the body has any visible
+  // text, which empirically separates the white cold-start wedge (innerText
+  // length 0) from a rendered SPA (>0). (Sprint-72 hardening — 2026-06-09.)
+  async function ensureWebChatRendered(session, handle, startUrl, opts = {}) {
+    const settleMs = opts.settleMs || Number(process.env.TERMDECK_WEBCHAT_RENDER_SETTLE_MS) || 8000;
+    const attempts = opts.attempts != null ? opts.attempts
+      : (Number(process.env.TERMDECK_WEBCHAT_RENDER_ATTEMPTS) || 2);
+    const stepMs = opts.stepMs || Number(process.env.TERMDECK_WEBCHAT_RENDER_STEP_MS) || 500;
+    if (!handle || !handle.page || typeof handle.page.evaluate !== 'function') return true;
+    const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
+    const painted = async () => {
+      try {
+        return await handle.page.evaluate(
+          () => !!(document && document.body && (document.body.innerText || '').trim().length > 0),
+        );
+      } catch (_e) {
+        return false;
+      }
+    };
+    const settle = async () => {
+      for (let waited = 0; waited < settleMs; waited += stepMs) {
+        if (session._webChatClosed) return true;
+        if (await painted()) return true;
+        await sleep(stepMs);
+      }
+      return painted();
+    };
+    if (await settle()) return true;
+    for (let tries = 1; tries <= attempts; tries++) {
+      if (session._webChatClosed) return true;
+      applyWebChatStatus(session, { status: 'starting', statusDetail: `Recovering blank page (try ${tries}/${attempts})…` });
+      try {
+        if (typeof handle.navigate === 'function') {
+          await handle.navigate(startUrl, { waitUntil: 'domcontentloaded' });
+        } else {
+          await handle.page.goto(startUrl, { waitUntil: 'domcontentloaded', timeout: 30000 });
+        }
+      } catch (_e) {
+        /* navigation hiccup — re-check paint anyway */
+      }
+      if (await settle()) return true;
+    }
+    return false;
+  }
+
   function setupWebChatSession(session) {
     session.pty = null;
     session.pid = null;
@@ -1647,8 +1698,18 @@ function createServer(config) {
             catch (_e) { /* never disrupt */ }
           }
         };
-        if (handle && typeof handle.screencast === 'function') handle.screencast(onFrame);
-        else if (driver.cdp && typeof driver.cdp.screencast === 'function') driver.cdp.screencast(handle, onFrame);
+        // Render quality (Sprint-72 hardening, 2026-06-09): pass crisp, Retina-friendly
+        // screencast opts. The driver's bare default was a blurry 1280x800 @ jpeg-q60 — fine
+        // on a 1x display, soft on a 2x Mac (the HiDPI canvas then upscales it). Env-tunable
+        // down for slow links: TERMDECK_WEBCHAT_QUALITY / _MAXW / _MAXH / _FORMAT.
+        const scOpts = {
+          format: process.env.TERMDECK_WEBCHAT_FORMAT || 'jpeg',
+          quality: Number(process.env.TERMDECK_WEBCHAT_QUALITY) || 85,
+          maxWidth: Number(process.env.TERMDECK_WEBCHAT_MAXW) || 2560,
+          maxHeight: Number(process.env.TERMDECK_WEBCHAT_MAXH) || 1600,
+        };
+        if (handle && typeof handle.screencast === 'function') handle.screencast(onFrame, scOpts);
+        else if (driver.cdp && typeof driver.cdp.screencast === 'function') driver.cdp.screencast(handle, onFrame, scOpts);
       } catch (err) {
         console.error('[web-chat] screencast wiring failed:', err && err.message ? err.message : err);
       }
@@ -1673,7 +1734,17 @@ function createServer(config) {
         }
       } catch (_e) { /* optional hook — absence is fine */ }
 
-      applyWebChatStatus(session, { status: 'idle', statusDetail: 'Ready' });
+      // Self-heal a flaky blank cold-start before declaring the panel Ready.
+      let rendered = true;
+      try {
+        rendered = await ensureWebChatRendered(session, handle, startUrl);
+      } catch (err) {
+        console.error('[web-chat] render-watchdog error:', err && err.message ? err.message : err);
+      }
+      if (session._webChatClosed) return;
+      applyWebChatStatus(session, rendered
+        ? { status: 'idle', statusDetail: 'Ready' }
+        : { status: 'errored', statusDetail: 'page did not render (blank after retries)' });
     })();
   }
 
@@ -3589,6 +3660,24 @@ if (require.main === module) {
   process.on('SIGINT', () => handleShutdown('SIGINT'));
   process.on('SIGTERM', () => handleShutdown('SIGTERM'));
 
+  // Fail-soft crash guards (Sprint-72 hardening, 2026-06-09). One bad async
+  // rejection or uncaught error anywhere — a panel handler, a request, a hook —
+  // must NOT crash the whole server and take every live terminal panel (and the
+  // user's work) down with it. We LOG prominently (per-event ISO timestamp, like
+  // the boot banner, so crash boundaries stay greppable) and keep running. This
+  // trades the small risk of continuing in a degraded state for the much larger
+  // cost of losing every panel; a process supervisor is the backstop if the
+  // process ever truly wedges. Shutdown is exempt — let handleShutdown finish.
+  process.on('unhandledRejection', (reason) => {
+    if (shutdownInProgress) return;
+    const msg = (reason && reason.stack) || (reason && reason.message) || String(reason);
+    console.error(`[server] unhandledRejection (kept alive · ${new Date().toISOString()}):\n${msg}`);
+  });
+  process.on('uncaughtException', (err) => {
+    if (shutdownInProgress) return;
+    console.error(`[server] uncaughtException (kept alive · ${new Date().toISOString()}):\n${(err && err.stack) || err}`);
+  });
+
   server.listen(port, host, () => {
     // Sprint 60 v1.0.14 (Item 5) — per-boot banner with ISO timestamp + PID.
     // Brad's 2026-05-07 forensic: a single 260KB termdeck.log spanned Apr 25

package/packages/server/src/sprints/status-parser.js

@@ -21,9 +21,19 @@ function parseStatusMd(filePath) {
   const content = fs.readFileSync(filePath, 'utf8');
   const lines = content.split('\n');
 
-  // Regex per BRIEF (loosened for multiple suffixes):
-  // ^### \[(T\d+(?:-[A-Z0-9-]+)?|ORCH)\] (FINDING|PROPOSE|LANDED|DONE|AUDIT-RED|AUDIT-CONCERN|CHECKPOINT|FINAL-VERDICT|STATUS|RULING) (\d{4}-\d{2}-\d{2}) (\d{2}:\d{2}) ET — (.*?)$
-  const POST_RE = /^### \[(T\d+(?:-[A-Z0-9-]+)?|ORCH)\] (FINDING|PROPOSE|LANDED|DONE|AUDIT-RED|AUDIT-CONCERN|CHECKPOINT|FINAL-VERDICT|STATUS|RULING) (\d{4}-\d{2}-\d{2}) (\d{2}:\d{2}) ET — (.*?)$/;
+  // Canonical post header (global CLAUDE.md § lane post-shape uniformity):
+  //   ### [T<n>] VERB[ (qualifier)] YYYY-MM-DD HH:MM ET — <gist>
+  // The verb is HARD-ANCHORED to the header position (immediately after the
+  // lane tag), so a verb word appearing in a gist/prose — e.g. "DONE" inside a
+  // CHECKPOINT line's gist — is never mis-counted as that verb; it survives
+  // only as gist (capture group 5). Verb vocabulary tracks the REAL lane
+  // vocabulary, incl. FIX-PROPOSED / FIX-LANDED / AUDIT-PASS / AUDIT-FAIL, plus
+  // an optional parenthetical qualifier after the verb (e.g. the real shape
+  // `AUDIT-PASS (cdp/render)`). Order longer compounds before their shorter
+  // prefixes is unnecessary here (all alternatives are anchored + whitespace-
+  // delimited), but FIX-*/AUDIT-* are listed before bare PROPOSE/LANDED for
+  // readability.
+  const POST_RE = /^### \[(T\d+(?:-[A-Z0-9-]+)?|ORCH)\] (FINDING|FIX-PROPOSED|PROPOSE|FIX-LANDED|LANDED|DONE|AUDIT-RED|AUDIT-CONCERN|AUDIT-PASS|AUDIT-FAIL|CHECKPOINT|FINAL-VERDICT|STATUS|RULING)(?:\s+\([^)]*\))? (\d{4}-\d{2}-\d{2}) (\d{2}:\d{2}) ET — (.*?)$/;
 
   const laneLandeds = {}; // laneTag -> latest LANDED timestamp
   const openReds = []; // List of {tag, timestamp, gist}
@@ -57,7 +67,7 @@ function parseStatusMd(filePath) {
       gist
     };
 
-    if (verb === 'LANDED') {
+    if (verb === 'LANDED' || verb === 'FIX-LANDED') {
       laneLandeds[tag] = timestamp;
     }