cobro-mcp 0.9.3 → 0.9.4

package/CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project are documented here. The format follows [Kee
 
 ## [Unreleased]
 
+## [0.9.4] — 2026-09-22
+### Fixed
+- Source locations are no longer dropped when the project directory itself sits under a node_modules path (library detection is now relative to the project root).
+
 ## [0.9.3] — 2026-09-22
 ### Changed
 - MCP instructions reorganized into 8 short lines (payload reading and ref/region conventions on separate lines); same rules, about 17% shorter.

package/dist/overlay.js

@@ -976,37 +976,48 @@ ${Math.round(r.width)}\xD7${Math.round(r.height)}`;
     const frame = frames.find((f) => !REACT_INTERNAL_FN.test(f.fn) && !NODE_MODULES_URL.test(f.url));
     return frame ? { url: frame.url, line: frame.line, col: frame.col } : null;
   }
+  function normalizeDrivePath(path) {
+    return path.replace(/\\/g, "/").replace(/^\/?([a-zA-Z]):\//, (_, drive) => `${drive.toLowerCase()}:/`);
+  }
+  function isLibraryPath(path, root) {
+    const norm = normalizeDrivePath(path);
+    let rel = norm;
+    if (root) {
+      const r = normalizeDrivePath(root).replace(/\/+$/, "");
+      if (norm.startsWith(`${r}/`)) rel = norm.slice(r.length + 1);
+    }
+    return /(^|\/)node_modules\//.test(rel);
+  }
 
   // src/overlay/frameworks/react.ts
-  function userLoc(f) {
+  function userLoc(f, root) {
     const src = f._debugSource;
-    const fileName = src?.fileName?.replace(/\\/g, "/");
-    if (fileName && !fileName.includes("node_modules/")) return `${src.fileName}${src.lineNumber ? ":" + src.lineNumber : ""}`;
+    const fileName = src?.fileName;
+    if (fileName && !isLibraryPath(fileName, root)) return `${fileName}${src.lineNumber ? ":" + src.lineNumber : ""}`;
     const stack = typeof f._debugStack === "string" ? f._debugStack : f._debugStack?.stack;
     return stack ? pickUserFrame(stack) ?? void 0 : void 0;
   }
-  function reactInfo(el) {
+  function reactInfo(el, root) {
     const key = Object.keys(el).find((k) => k.startsWith("__reactFiber$"));
     if (!key) return void 0;
     let f = el[key] ?? null;
     let fallback;
     let loc;
     let component;
-    const callers = [];
-    const callerFrames = [];
+    const callerLocs = [];
     let pending;
     const commit = (p) => {
-      if (callers.length + callerFrames.length >= 2) return;
+      if (callerLocs.length >= 2) return;
       if (typeof p === "string") {
-        if (p !== loc && !callers.includes(p)) callers.push(p);
-      } else callerFrames.push(p);
+        if (p !== loc && !callerLocs.includes(p)) callerLocs.push(p);
+      } else callerLocs.push(p);
     };
     let libBelow = false;
     for (let i = 0; f && i < 30; i++, f = f.return ?? null) {
       const t = f.type;
       const name = t && typeof t !== "string" ? t.displayName || t.name : void 0;
       if (loc === void 0) {
-        const l2 = userLoc(f);
+        const l2 = userLoc(f, root);
         if (l2 !== void 0) {
           loc = l2;
           component = name;
@@ -1016,13 +1027,13 @@ ${Math.round(r.width)}\xD7${Math.round(r.height)}`;
         continue;
       }
       if (!name) {
-        if ((f._debugSource !== void 0 || f._debugStack !== void 0) && userLoc(f) === void 0) libBelow = true;
+        if ((f._debugSource !== void 0 || f._debugStack !== void 0) && userLoc(f, root) === void 0) libBelow = true;
         continue;
       }
       if (component === void 0) {
-        if (libBelow && userLoc(f) === void 0) continue;
+        if (libBelow && userLoc(f, root) === void 0) continue;
         component = name;
-        const l2 = userLoc(f);
+        const l2 = userLoc(f, root);
         if (l2 !== void 0) pending = l2;
         continue;
       }
@@ -1030,36 +1041,35 @@ ${Math.round(r.width)}\xD7${Math.round(r.height)}`;
         commit(pending);
         pending = void 0;
       }
-      const l = userLoc(f);
+      const l = userLoc(f, root);
       if (l !== void 0) pending = l;
     }
     if (loc === void 0) return fallback;
     if (!component) return fallback;
     const hit = typeof loc === "string" ? { component, source: loc } : { component, frame: loc };
-    if (callers.length) hit.callers = callers;
-    if (callerFrames.length) hit.callerFrames = callerFrames;
+    if (callerLocs.length) hit.callerLocs = callerLocs;
     return hit;
   }
   var react = { key: "react", detect: reactInfo };
 
   // src/overlay/frameworks/vue.ts
   var ABSOLUTE_RE = /^([a-zA-Z]:\/|\/)/;
-  function normalizeVueSource(file) {
+  function normalizeVueSource(file, root) {
     const norm = file.replace(/\\/g, "/");
-    if (norm.includes("node_modules")) return void 0;
+    if (isLibraryPath(norm, root)) return void 0;
     if (!ABSOLUTE_RE.test(norm)) return norm;
     const idx = norm.lastIndexOf("/src/");
     if (idx !== -1) return norm.slice(idx + 1);
     const segs = norm.split("/").filter(Boolean);
     return segs.slice(-2).join("/");
   }
-  function vueInfo(el) {
+  function vueInfo(el, root) {
     let c = el["__vueParentComponent"] ?? null;
     for (let i = 0; c && i < 10; i++, c = c.parent ?? null) {
       const name = c.type?.name || c.type?.__name;
       if (name) {
         const file = c.type?.__file;
-        const source = file ? normalizeVueSource(file) : void 0;
+        const source = file ? normalizeVueSource(file, root) : void 0;
         return source ? { component: name, source } : { component: name };
       }
     }
@@ -1069,9 +1079,13 @@ ${Math.round(r.width)}\xD7${Math.round(r.height)}`;
 
   // src/overlay/frameworks/index.ts
   var adapters = [react, vue];
+  var projectRoot;
+  function setProjectRoot(root) {
+    projectRoot = root;
+  }
   function detectComponent(el) {
     for (const adapter of adapters) {
-      const info = adapter.detect(el);
+      const info = adapter.detect(el, projectRoot);
       if (info) return { key: adapter.key, info };
     }
     return void 0;
@@ -1173,6 +1187,7 @@ ${Math.round(r.width)}\xD7${Math.round(r.height)}`;
     if (window.top !== window) return;
     const PORT = __COBRO_PORT__;
     const TOKEN = __COBRO_TOKEN__;
+    setProjectRoot(__COBRO_ROOT__);
     const boot = () => {
       if (document.documentElement.hasAttribute("data-cobro")) return;
       document.documentElement.setAttribute("data-cobro", "1");

package/dist/server.js

@@ -308,6 +308,22 @@ var ChannelServer = class {
 
 // src/core/sourcemap.ts
 import { FlattenMap, originalPositionFor } from "@jridgewell/trace-mapping";
+
+// src/core/frame.ts
+function normalizeDrivePath(path) {
+  return path.replace(/\\/g, "/").replace(/^\/?([a-zA-Z]):\//, (_, drive) => `${drive.toLowerCase()}:/`);
+}
+function isLibraryPath(path, root) {
+  const norm = normalizeDrivePath(path);
+  let rel = norm;
+  if (root) {
+    const r = normalizeDrivePath(root).replace(/\/+$/, "");
+    if (norm.startsWith(`${r}/`)) rel = norm.slice(r.length + 1);
+  }
+  return /(^|\/)node_modules\//.test(rel);
+}
+
+// src/core/sourcemap.ts
 var FRAMEWORK_KEYS = ["react", "vue"];
 function findSourceMapUrl(moduleText, moduleUrl) {
   const re = /\/\/[#@]\s*sourceMappingURL=(\S+)/g;
@@ -322,11 +338,7 @@ function findSourceMapUrl(moduleText, moduleUrl) {
     return void 0;
   }
 }
-function normalizeDrivePath(path) {
-  return path.replace(/\\/g, "/").replace(/^\/?([a-zA-Z]):\//, (_, drive) => `${drive.toLowerCase()}:/`);
-}
-function normalizeFileSource(pathname, root) {
-  const path = normalizeDrivePath(decodeURIComponent(pathname));
+function normalizeFileSource(path, root) {
   if (root) {
     const r = normalizeDrivePath(root).replace(/\/+$/, "");
     if (path === r) return "";
@@ -351,18 +363,30 @@ function resolveOriginal(mapJson, opts) {
   } catch {
     return void 0;
   }
-  const path = (url.protocol === "file:" ? normalizeFileSource(url.pathname, opts.root) : decodeURIComponent(url.pathname).replace(/^\/+/, "")).split("?")[0];
-  if (!path || path.includes("node_modules/") || path.split("/").includes("..") || /[\x00-\x1f]/.test(path)) return void 0;
+  const raw = decodeURIComponent(url.pathname);
+  const abs = url.protocol === "file:" ? normalizeDrivePath(raw) : raw.replace(/^\/+/, "");
+  if (isLibraryPath(abs, opts.root)) return void 0;
+  const path = (url.protocol === "file:" ? normalizeFileSource(abs, opts.root) : abs).split("?")[0];
+  if (!path || path.split("/").includes("..") || /[\x00-\x1f]/.test(path)) return void 0;
   return `${path}:${pos.line}`;
 }
 function stripFrame(e) {
   let out = e;
   for (const key of FRAMEWORK_KEYS) {
     const info = out[key];
-    if (!info?.frame && !info?.callerFrames) continue;
-    const rest = { component: info.component };
-    if (info.source) rest.source = info.source;
-    if (info.callers?.length) rest.callers = info.callers;
+    if (!info) continue;
+    if (typeof info.component !== "string") {
+      const rest2 = { ...out };
+      delete rest2[key];
+      out = rest2;
+      continue;
+    }
+    const rest = { component: info.component.slice(0, 200) };
+    if (typeof info.source === "string") rest.source = info.source.slice(0, 200);
+    if (Array.isArray(info.callers)) {
+      const callers = info.callers.filter((c) => typeof c === "string").map((c) => c.slice(0, 200)).slice(0, 2);
+      if (callers.length) rest.callers = callers;
+    }
     out = { ...out, [key]: rest };
   }
   return out;
@@ -409,17 +433,15 @@ async function resolveElementSources(batch, page, fetchText, opts) {
       const source = await resolveFrame(react.frame, page, fetchText, root);
       if (source) react.source = source;
     }
-    if (react.callerFrames?.length) {
-      const callers = react.callers ? [...react.callers] : [];
-      for (const frame of react.callerFrames) {
-        if (callers.length >= 2) break;
-        const source = await resolveFrame(frame, page, fetchText, root);
-        if (!source || source === react.source || callers.includes(source)) continue;
-        callers.push(source);
-      }
-      if (callers.length) react.callers = callers;
-      else delete react.callers;
+    const callers = [];
+    for (const loc of react.callerLocs ?? []) {
+      if (callers.length >= 2) break;
+      const s = typeof loc === "string" ? loc : await resolveFrame(loc, page, fetchText, root);
+      if (!s || s === react.source || callers.includes(s)) continue;
+      callers.push(s);
     }
+    if (callers.length) react.callers = callers;
+    else delete react.callers;
   }
 }
 
@@ -623,7 +645,7 @@ var BrowserLauncher = class {
     return this.launchedOnce;
   }
   injected() {
-    return this.opts.overlaySource.replace(/__COBRO_PORT__/g, String(this.opts.port)).replace(/__COBRO_TOKEN__/g, JSON.stringify(this.opts.token));
+    return this.opts.overlaySource.replace(/__COBRO_PORT__/g, () => String(this.opts.port)).replace(/__COBRO_TOKEN__/g, () => JSON.stringify(this.opts.token)).replace(/__COBRO_ROOT__/g, () => JSON.stringify(this.opts.root));
   }
   async launch() {
     const engine2 = this.opts.engine ?? "chromium";
@@ -913,7 +935,7 @@ var bridge = await createBridge({
 });
 var fixed = configStrategy();
 if (fixed) bridge.core.setStrategy(fixed);
-launcher = new BrowserLauncher({ overlaySource, port: bridge.port, token, profileDir, headless: process.env.COBRO_HEADLESS === "1", engine });
+launcher = new BrowserLauncher({ overlaySource, port: bridge.port, token, root: process.cwd(), profileDir, headless: process.env.COBRO_HEADLESS === "1", engine });
 var mcp = createMcpServer({
   core: bridge.core,
   browser: launcher,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cobro-mcp",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "description": "Co-browse for coding agents: pick elements in the browser, write a note, Send — the context lands in the agent's chat. MCP server with 6 tools.",
   "keywords": [
     "mcp",