@maintainer-pro/ai-bridge 0.1.28 → 0.1.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maintainer-pro/ai-bridge",
-  "version": "0.1.28",
+  "version": "0.1.30",
   "description": "Local bridge daemon that pairs a machine to Maintainer Pro and configures multiple client sandboxes.",
   "keywords": [
     "maintainer-pro",
@@ -28,7 +28,7 @@
     "node": ">=22"
   },
   "dependencies": {
-    "@maintainer-pro/ai-cli": "^0.1.15",
+    "@maintainer-pro/ai-cli": "^0.1.16",
     "@maintainer-pro/ai-server": "^0.1.8"
   }
 }

package/src/daemon.mjs

@@ -451,13 +451,30 @@ async function loadAiCli() {
 /** @type {{ at: number, ids: string[] } | null} */
 let cliProviderCache = null;
 
-async function detectCliProviders() {
-  if (cliProviderCache && Date.now() - cliProviderCache.at < 60_000) {
+function missingCliSetupText(folder) {
+  const folderLine = folder
+    ? `No coding agent CLI is available in this folder:\n  ${folder}`
+    : "No coding agent CLI found on this computer.";
+  return [
+    `${folderLine} Chat will not work until one is installed and reachable from this folder.`,
+    "",
+    "  • Cursor Agent CLI  —  https://cursor.com  (command: agent)",
+    "  • Claude Code       —  https://docs.anthropic.com/en/docs/claude-code  (command: claude)",
+    "  • Antigravity       —  https://antigravity.google/  (command: agy)",
+    "",
+    "Then either sign in and re-run the Maintainer Pro command from a terminal where that CLI works, or set the full path in this folder's .env:",
+    "  AI_CLI_PROVIDER=cursor",
+    "  AI_CLI_COMMAND=C:\\\\Users\\\\you\\\\AppData\\\\Local\\\\cursor-agent\\\\agent.cmd",
+  ].join("\n");
+}
+
+async function detectCliProviders(force = false) {
+  if (!force && cliProviderCache && Date.now() - cliProviderCache.at < 60_000) {
     return cliProviderCache.ids;
   }
   try {
-    const { resolveProvider } = await loadAiCli();
-    const provider = await resolveProvider({ preference: "auto" });
+    const cli = await loadAiCli();
+    const provider = await cli.resolveProvider({ preference: "auto" });
     cliProviderCache = { at: Date.now(), ids: [provider.id] };
     return cliProviderCache.ids;
   } catch {
@@ -466,6 +483,65 @@ async function detectCliProviders() {
   }
 }
 
+async function withCliProbe(result, folders) {
+  const roots = [...new Set((folders || []).filter(Boolean).map((row) => path.resolve(row)))];
+  let probe = { available: false };
+  for (const folder of roots) {
+    probe = await probeFolderCli(folder);
+    if (probe.available) break;
+  }
+  const folder = probe.folder || roots[0] || result.folderPath;
+  if (probe.available) {
+    log(
+      `coding agent CLI ready in ${folder} (${probe.providerId || "auto"}${
+        probe.source ? ` via ${probe.source}` : ""
+      })`
+    );
+  } else if (folder) {
+    activity(
+      result.sandboxId,
+      "warn",
+      `No coding agent CLI in ${folder}. Chat needs agent, claude, or agy — or AI_CLI_COMMAND in this folder's .env.`
+    );
+  }
+  return {
+    ...result,
+    missingCli: !probe.available,
+    cliProvider: probe.providerId || null,
+    cliCommand: probe.command || null,
+    warning: !probe.available
+      ? [result.warning, missingCliSetupText(folder)].filter(Boolean).join("\n\n")
+      : result.warning,
+  };
+}
+
+async function probeFolderCli(folder) {
+  const root = folder ? path.resolve(folder) : "";
+  if (!root) return { available: false };
+  try {
+    const cli = await loadAiCli();
+    if (typeof cli.probeCliInFolder === "function") {
+      return await cli.probeCliInFolder(root);
+    }
+    const provider = await cli.resolveProvider({
+      preference: "auto",
+      workspaceDir: root,
+    });
+    return { available: true, providerId: provider.id, folder: root };
+  } catch {
+    return { available: false, folder: root };
+  }
+}
+
+async function warnIfMissingCli() {
+  const ids = await detectCliProviders(true);
+  if (ids.length) {
+    log(`coding agent CLI ready (${ids.join(", ")})`);
+    return;
+  }
+  warn(missingCliSetupText());
+}
+
 async function resolveWorkspaceHostApps(ws, opts = {}) {
   const folders = workspaceFolders(ws);
   const folder = folders[0] || path.resolve(ws.folderPath || "");
@@ -3714,11 +3790,18 @@ async function startAiServerForWorkspace(ws, opts = {}) {
 
   const storeEnv = await loadSandboxStoreEnv(ws, cfg);
   const folderEnv = readProjectEnvValues(folder);
+  const cliProbe = await probeFolderCli(folder);
   const dataDir = dataDirFor(folder, ws.sandboxId);
   const instanceEnv = {
     ...folderEnv,
     ...overrideEnv,
     ...storeEnv,
+    ...(cliProbe.command && !folderEnv.AI_CLI_COMMAND
+      ? { AI_CLI_COMMAND: cliProbe.command }
+      : {}),
+    ...(cliProbe.providerId && !folderEnv.AI_CLI_PROVIDER
+      ? { AI_CLI_PROVIDER: cliProbe.providerId }
+      : {}),
     AI_SERVER_URL: localAi,
     CORS_ORIGIN: overrideEnv.CORS_ORIGIN || folderEnv.CORS_ORIGIN || "",
     CORS_ORIGINS: overrideEnv.CORS_ORIGINS || folderEnv.CORS_ORIGINS || "",
@@ -4448,7 +4531,7 @@ async function setupWorkspace(cfg, action) {
     const host = workspaceHostReport(existingWs);
     const aiServerUp = await isChatServerOnPort(existingWs.port);
     log(`setup added folder ${resolved} sandbox=${shortId(sandboxId)}`);
-    return {
+    return await withCliProbe({
       sandboxId,
       folderPath: existingWs.folderPath,
       extraFolders: extra,
@@ -4476,7 +4559,9 @@ async function setupWorkspace(cfg, action) {
       reasons: ports.reasons || [],
       projectInfo: existingWs.projectInfo || null,
       ignorePaths: access.ignorePaths,
-    };
+    },
+      workspaceFolders(existingWs)
+    );
   }
 
   const client = configureClient({
@@ -4566,32 +4651,35 @@ async function setupWorkspace(cfg, action) {
   );
 
   const host = workspaceHostReport(entry);
-  return {
-    sandboxId,
-    folderPath: entry.folderPath,
-    extraFolders: entry.extraFolders || [],
-    addFolder: false,
-    port: entry.port,
-    appUrl: host.appUrl || entry.appUrl || appUrl,
-    origins: host.origins.length
-      ? host.origins
-      : [...new Set([corsOrigin, aiOrigin, entry.appUrl].filter(Boolean))],
-    wroteEnv: false,
-    clientKind: client.kind,
-    clientFiles: client.filesWritten,
-    clientNotes: client.notes,
-    aiServerUp,
-    startedHosts: [],
-    openUrl,
-    processIssues,
-    warning,
-    waitingForStart,
-    needsReview,
-    hostApps: ports.apps || entry.hostApps || [],
-    reasons: ports.reasons || [],
-    projectInfo,
-    ignorePaths: access.ignorePaths,
-  };
+  return await withCliProbe(
+    {
+      sandboxId,
+      folderPath: entry.folderPath,
+      extraFolders: entry.extraFolders || [],
+      addFolder: false,
+      port: entry.port,
+      appUrl: host.appUrl || entry.appUrl || appUrl,
+      origins: host.origins.length
+        ? host.origins
+        : [...new Set([corsOrigin, aiOrigin, entry.appUrl].filter(Boolean))],
+      wroteEnv: false,
+      clientKind: client.kind,
+      clientFiles: client.filesWritten,
+      clientNotes: client.notes,
+      aiServerUp,
+      startedHosts: [],
+      openUrl,
+      processIssues,
+      warning,
+      waitingForStart,
+      needsReview,
+      hostApps: ports.apps || entry.hostApps || [],
+      reasons: ports.reasons || [],
+      projectInfo,
+      ignorePaths: access.ignorePaths,
+    },
+    workspaceFolders(entry)
+  );
 }
 
 async function proposeSetupForWorkspace(ws, cfg, opts = {}) {
@@ -4738,16 +4826,28 @@ async function runActions(cfg, actions) {
           };
         }
       } else if (action.code === "recheck") {
+        cliProviderCache = null;
         const sandboxId = action.sandboxId || action.payload?.sandboxId;
         const ws = (cfg.workspaces || []).find((w) => w.sandboxId === sandboxId);
         if (!ws) {
           log(`${label} recheck: no local workspace`);
         }
+        const cli = await detectCliProviders(true);
+        const folderProbe = ws
+          ? await withCliProbe(
+              { sandboxId, folderPath: ws.folderPath },
+              workspaceFolders(ws)
+            )
+          : { missingCli: cli.length === 0 };
         const hostApps = ws
           ? await resolveWorkspaceHostApps(ws, { cfg, force: true, allowAi: false })
           : null;
         result = {
           recheckedAt: new Date().toISOString(),
+          cliProviders: cli,
+          missingCli: folderProbe.missingCli === true,
+          cliProvider: folderProbe.cliProvider || null,
+          cliCommand: folderProbe.cliCommand || null,
           hostApps: hostApps?.apps || [],
           projectInfo: ws?.projectInfo || null,
         };
@@ -5216,18 +5316,41 @@ function adminWsUrl(adminUrl, token) {
 async function buildIssues(cfg, workspaceStates) {
   /** @type {Array<Record<string, unknown>>} */
   const issues = [];
-  const cli = await detectCliProviders();
-  if (!cli.length) {
-    issues.push({
-      code: "missing_cli",
-      severity: "error",
-      title: "No coding agent CLI found",
-      message:
-        "PATH has no agent, claude, or agy. Chat turns will fail until one is installed.",
-      resolution: "Install and authenticate a CLI, then Recheck.",
-      actionCode: "recheck",
-      sandboxId: null,
-    });
+  if (workspaceStates.length) {
+    for (const st of workspaceStates) {
+      const ws = (cfg.workspaces || []).find((row) => row.sandboxId === st.sandboxId);
+      const folders = ws ? workspaceFolders(ws) : [st.folderPath].filter(Boolean);
+      let probe = { available: false, folder: folders[0] || st.folderPath };
+      for (const folder of folders) {
+        probe = await probeFolderCli(folder);
+        if (probe.available) break;
+      }
+      if (probe.available) continue;
+      issues.push({
+        code: "missing_cli",
+        severity: "error",
+        title: "No coding agent CLI in this folder",
+        message: missingCliSetupText(probe.folder || folders[0] || st.folderPath),
+        resolution:
+          "Install a CLI and re-run the Maintainer Pro command from a terminal where it works, or set AI_CLI_COMMAND in this folder's .env, then Check again.",
+        actionCode: "recheck",
+        sandboxId: st.sandboxId,
+      });
+    }
+  } else {
+    const cli = await detectCliProviders();
+    if (!cli.length) {
+      issues.push({
+        code: "missing_cli",
+        severity: "error",
+        title: "No coding agent CLI found",
+        message: missingCliSetupText(),
+        resolution:
+          "Install Cursor (agent), Claude Code (claude), or Antigravity (agy), sign in, then stop and re-run the Maintainer Pro command. Use Check again after it is on PATH.",
+        actionCode: "recheck",
+        sandboxId: null,
+      });
+    }
   }
   for (const st of workspaceStates) {
     if (st.aiServerUp || st.appsRunning || st.startingAi) continue;
@@ -5377,6 +5500,7 @@ async function main() {
   log(
     `online via websocket (presence every ${HEARTBEAT_MS / 1000}s, ping every ${WS_PING_MS / 1000}s; reconnects until stopped)`
   );
+  await warnIfMissingCli();
   await restoreHostsAfterReconnect(cfg);
 
   /** @type {unknown[]} */
@@ -5391,6 +5515,7 @@ async function main() {
         const batch = claimedActions.splice(0, claimedActions.length);
         await runActions(cfg, batch);
       }
+      await sendPresenceOverWs();
     } catch (err) {
       warn(err instanceof Error ? err.message : String(err));
     } finally {

package/src/share-rewrite.mjs

@@ -150,15 +150,22 @@ export function shouldProcessShareResponse(headers) {
   return true;
 }
 
-function isImmutableAssetPath(path) {
-  const p = String(path || "").split("?")[0] || "";
+function hasContentHash(path) {
+  const base = String(path || "").split("?")[0].split("/").pop() || "";
   return (
-    /\/_next\/static\//.test(p) ||
-    /\/__nextjs_font\//.test(p) ||
-    /\.[0-9a-f]{8,}\.(?:js|css|woff2?)$/i.test(p)
+    /\.[0-9a-f]{8,}\.(?:js|css|woff2?)$/i.test(base) ||
+    /^[0-9a-f]{8,}\.(?:js|css|woff2?)$/i.test(base) ||
+    /-[0-9a-f]{8,}[^/]*\.(?:js|css)$/i.test(base)
   );
 }
 
+/** Production hashed assets only. Next/Vite dev chunks reuse the same URL. */
+function isImmutableAssetPath(path) {
+  const p = String(path || "").split("?")[0] || "";
+  if (/\/__nextjs_font\//.test(p)) return true;
+  return hasContentHash(p);
+}
+
 function isViteDevSourcePath(path) {
   const raw = String(path || "");
   const p = raw.split("?")[0] || "";
@@ -184,12 +191,12 @@ function isDocumentPath(path) {
 
 function isNoStorePath(path) {
   const p = String(path || "").split("?")[0] || "";
+  if (isImmutableAssetPath(path) || isCacheableMediaPath(path)) return false;
   return (
     /\/embed-config\.js$/i.test(p) ||
     p === "/api" ||
     p.startsWith("/api/") ||
-    /\/_next\/webpack-hmr/i.test(p) ||
-    /\/_next\/webpack\/hmr/i.test(p) ||
+    /\/_next\//i.test(p) ||
     /\/__nextjs_original-stack-frames/i.test(p) ||
     /\/__mp\//.test(p) ||
     isViteDevSourcePath(path) ||
@@ -241,9 +248,9 @@ function dropHopCacheNoise(headers) {
 }
 
 /**
- * Cache-Control for the share URL. Hashed Next/Vite assets can sit in the
- * browser for a year; HTML/RSC revalidate with ETag; secrets and HMR do not
- * cache.
+ * Cache-Control for the share URL. Only content-hashed assets are immutable.
+ * Next/Vite dev chunks reuse URLs, so they must not be cached or HMR/hydration
+ * breaks. HTML, RSC, and HMR stay no-store.
  *
  * @param {Record<string, string>} headers
  * @param {string} path
@@ -485,8 +492,11 @@ export function prefixShareOriginUrl(value, publicBase) {
     const basePort = base.port || (base.protocol === "https:" ? "443" : "80");
     if (reqPort !== basePort) return value;
     const path = u.pathname || "/";
-    if (path === "/" || path === "") return value;
     if (path === prefix || path.indexOf(prefix + "/") === 0) return u.toString();
+    if (path === "/" || path === "") {
+      u.pathname = prefix || "/";
+      return u.toString();
+    }
     if (path.indexOf("/p/") === 0) return u.toString();
     u.pathname = prefix + path;
     return u.toString();
@@ -722,11 +732,11 @@ function rewriteShareAsLocalEnv(body) {
   let out = String(body);
   out = out.replace(
     /(\/\.\*localhost\.\*\/\.test\()([^)]+)(\))/g,
-    "($1$2$3||location.pathname.indexOf(\"/p/\")===0)"
+    "($1$2$3||window.__MP_SHARE_PREFIX__)"
   );
   out = out.replace(
     /((?:window\.)?location\.hostname)\s*===\s*(['"])localhost\2/g,
-    "($1===$2localhost$2||location.pathname.indexOf(\"/p/\")===0)"
+    "($1===$2localhost$2||window.__MP_SHARE_PREFIX__)"
   );
   return out;
 }
@@ -1026,6 +1036,7 @@ function injectMaintainerProEmbed(html, aiPublicBase) {
 function shareProxyShim(p, portMap, rewriteMappedLocalUrls, bypassCors) {
   if (!p || window.__MP_SHARE_SHIM__) return;
   window.__MP_SHARE_SHIM__ = 1;
+  window.__MP_SHARE_PREFIX__ = p;
   window.__MP_PORT_MAP__ = portMap || {};
   window.__MP_BYPASS_CORS__ = Boolean(bypassCors);
   function rewriteText(text) {
@@ -1121,10 +1132,8 @@ function shareProxyShim(p, portMap, rewriteMappedLocalUrls, bypassCors) {
     if (v.charAt(0) === "/" && v.charAt(1) !== "/") return prefixPath(v);
     try {
       var nav = new URL(v, location.href);
-      if (!loopback(nav.hostname)) return v;
-      if (nav.origin !== location.origin) return v;
-      if (nav.pathname === "/" || nav.pathname === "") return v;
-      nav.pathname = prefixPath(nav.pathname);
+      if (!loopback(nav.hostname) && nav.origin !== location.origin) return v;
+      nav.pathname = prefixPath(nav.pathname || "/");
       return nav.toString();
     } catch (e) {}
     return v;
@@ -1363,6 +1372,24 @@ function shareProxyShim(p, portMap, rewriteMappedLocalUrls, bypassCors) {
       });
     }
   } catch (e) {}
+  document.addEventListener(
+    "click",
+    function (e) {
+      if (e.defaultPrevented || e.button !== 0) return;
+      if (e.metaKey || e.ctrlKey || e.shiftKey || e.altKey) return;
+      var a = e.target && e.target.closest ? e.target.closest("a[href]") : null;
+      if (!a || (a.target && a.target !== "" && a.target !== "_self")) return;
+      if (a.hasAttribute("download")) return;
+      var href = a.getAttribute("href");
+      if (!href || href.charAt(0) === "#") return;
+      if (/^(mailto|tel|javascript):/i.test(href)) return;
+      var next = addNav(href);
+      if (!next || next === href) return;
+      e.preventDefault();
+      location.assign(next);
+    },
+    true
+  );
   try {
     var tokenRoot = p.replace(/\/[^/]+$/, "/");
     if (navigator.serviceWorker && tokenRoot.indexOf("/p/") === 0) {
@@ -1486,9 +1513,8 @@ function shareServiceWorkerMain(portMap, tokenRoot, rewriteMappedLocalUrls, bypa
         return backend + path + (u.search || "") + (u.hash || "");
       }
       if (u.origin !== self.location.origin) return url;
-      if (u.pathname === "/" || u.pathname === "") return url;
       if (u.pathname === prefix || u.pathname.indexOf(prefix + "/") === 0) return url;
-      u.pathname = u.pathname === "/" ? prefix : prefix + u.pathname;
+      u.pathname = u.pathname === "/" || u.pathname === "" ? prefix : prefix + u.pathname;
       return u.toString();
     } catch (e) {}
     return url;