claude-bridge-cli 2.0.22 → 2.0.24

package/lib/bridge.js

@@ -64,6 +64,14 @@ function dataDir() {
   return d;
 }
 
+// Where always-on personal skills live for THIS machine (a plugin dir, since
+// ~/.claude/skills is not discovered by the CLI). Shared by the turn builder
+// (--plugin-dir) and the /skills endpoint so they can never disagree.
+function globalSkillsDir() {
+  return process.env.CLAUDE_BRIDGE_GLOBAL_SKILLS_DIR ||
+         path.join(dataDir(), "global-skills");
+}
+
 function readJson(filePath, def) {
   try { return JSON.parse(fs.readFileSync(filePath, "utf8")); } catch { return def; }
 }
@@ -427,8 +435,13 @@ function searchSessions(query, limit = 30) {
     } catch { return 0; }
   });
 
+  // Scan far more than the old 200 newest — on a busy machine that cut off
+  // sessions only a few days old, so a term you KNEW was there returned
+  // nothing. The loop already stops once `limit` results are collected, and
+  // the whole-file prefilter below makes a non-matching session cheap.
   const results = [];
-  for (const { id, project, filePath } of files.slice(0, 200)) {
+  for (const { id, project, filePath } of files.slice(0, 2000)) {
+    if (results.length >= limit) break;
     let stat;
     try { stat = fs.statSync(filePath); } catch { continue; }
     let content;
@@ -450,9 +463,30 @@ function searchSessions(query, limit = 30) {
       let rec;
       try { rec = JSON.parse(line); } catch { continue; }
       const t = rec.type;
-      if (t !== "user" && t !== "assistant") continue;
-      const msg = rec.message || {};
       const texts = [];
+      // Text also lives OUTSIDE message.content: a queued message you typed,
+      // the recorded last prompt, compaction summaries, system notes. Parsing
+      // only user/assistant records made those unfindable — the old whole-file
+      // grep did match them, so skipping them was a regression.
+      if (t !== "user" && t !== "assistant") {
+        for (const k of ["summary", "lastPrompt", "content", "text"]) {
+          if (typeof rec[k] === "string" && rec[k]) texts.push(rec[k]);
+        }
+        for (const text of texts) {
+          const tl = text.toLowerCase();
+          let from = tl.indexOf(q);
+          if (from < 0) continue;
+          while (from >= 0) { matchCount++; from = tl.indexOf(q, from + q.length); }
+          if (!snippet) {
+            const i = tl.indexOf(q);
+            const s = Math.max(0, i - 40), e = Math.min(text.length, i + q.length + 80);
+            snippet = (s > 0 ? "…" : "") + text.slice(s, e).replace(/\s+/g, " ").trim()
+                    + (e < text.length ? "…" : "");
+          }
+        }
+        continue;
+      }
+      const msg = rec.message || {};
       const c = msg.content;
       if (typeof c === "string") texts.push(c);
       else if (Array.isArray(c)) {
@@ -864,8 +898,7 @@ function askClaude(config, { prompt, session_id, images, cwd, plan_mode, allow_t
     // cwd. So personal always-on skills live in <data>/global-skills as a plugin
     // and are attached to every turn — global instead of per-project.
     try {
-      const gsDir = process.env.CLAUDE_BRIDGE_GLOBAL_SKILLS_DIR ||
-                    path.join(dataDir(), "global-skills");
+      const gsDir = globalSkillsDir();
       if (fs.existsSync(path.join(gsDir, "skills"))) args.push("--plugin-dir", gsDir);
     } catch {}
     // Optional model override from the extension's model picker (alias or id).
@@ -1174,6 +1207,87 @@ function startBridge(config) {
     }
 
     // ChatGPT bindings
+    // ── /skills — global-skills CRUD, per machine ────────────────────────────
+    // The extension's "/" autocomplete and Skills tab call this on whichever
+    // machine is active. Without it the request 404s, the host list comes back
+    // empty, and the menu silently shows ONLY the CLI built-ins — which reads
+    // as "this machine has no skills". Mirrors the Python bridge's contract.
+    if (url.pathname === "/skills" || url.pathname.startsWith("/skills/")) {
+      const skillsRoot = path.join(globalSkillsDir(), "skills");
+      const nameOf = decodeURIComponent(url.pathname.slice("/skills/".length) || "");
+      const validName = (n) => /^[a-z0-9][a-z0-9-]{0,63}$/.test(n);
+      const fileOf = (n) => path.join(skillsRoot, n, "SKILL.md");
+
+      if (req.method === "GET" && url.pathname === "/skills") {
+        const out = [];
+        let entries = [];
+        try { entries = fs.readdirSync(skillsRoot); } catch {}
+        for (const n of entries.sort()) {
+          const f = fileOf(n);
+          let md; try { md = fs.readFileSync(f, "utf8"); } catch { continue; }
+          const fm = /^---\s*\n([\s\S]*?)\n---\s*\n/.exec(md);
+          const desc = fm ? (/^description:\s*(.+)$/m.exec(fm[1]) || [])[1] : "";
+          let readonly = false;
+          try { readonly = fs.lstatSync(path.join(skillsRoot, n)).isSymbolicLink(); } catch {}
+          out.push({ name: n, description: (desc || "").trim().replace(/^["']|["']$/g, ""),
+                     size: md.length, readonly });
+        }
+        send(200, { skills: out, dir: skillsRoot });
+        return;
+      }
+      if (!validName(nameOf)) { send(404, { error: "no such skill" }); return; }
+      if (req.method === "GET") {
+        try {
+          const md = fs.readFileSync(fileOf(nameOf), "utf8");
+          let readonly = false;
+          try { readonly = fs.lstatSync(path.join(skillsRoot, nameOf)).isSymbolicLink(); } catch {}
+          send(200, { name: nameOf, content: md, readonly });
+        } catch { send(404, { error: "no such skill: " + nameOf }); }
+        return;
+      }
+      if (req.method === "POST") {
+        const body = await readBody(req);
+        const content = body && body.content;
+        if (typeof content !== "string" || !content.trim()) {
+          send(400, { error: "content is required" }); return;
+        }
+        const fm = /^---\s*\n([\s\S]*?)\n---\s*\n/.exec(content);
+        if (!fm || !/^description:\s*\S/m.test(fm[1])) {
+          send(400, { error: "SKILL.md must start with '---' frontmatter containing "
+                             + "name: and description: lines — description is what "
+                             + "makes Claude auto-select the skill" });
+          return;
+        }
+        const dir = path.join(skillsRoot, nameOf);
+        try {
+          if (fs.existsSync(dir) && fs.lstatSync(dir).isSymbolicLink()) {
+            send(400, { error: `'${nameOf}' is repo-owned (symlink) — edit it in its repo` });
+            return;
+          }
+          fs.mkdirSync(dir, { recursive: true });
+          const tmp = path.join(dir, ".SKILL.md.tmp");
+          fs.writeFileSync(tmp, content);
+          fs.renameSync(tmp, fileOf(nameOf));
+          send(200, { ok: true, name: nameOf });
+        } catch (e) { send(500, { error: "write failed: " + e.message }); }
+        return;
+      }
+      if (req.method === "DELETE") {
+        const dir = path.join(skillsRoot, nameOf);
+        try {
+          if (fs.lstatSync(dir).isSymbolicLink()) {
+            fs.unlinkSync(dir);                       // detach only; repo copy survives
+            send(200, { ok: true, name: nameOf, detached_symlink: true }); return;
+          }
+          fs.rmSync(dir, { recursive: true });
+          send(200, { ok: true, name: nameOf });
+        } catch { send(404, { error: "no such skill: " + nameOf }); }
+        return;
+      }
+      send(405, { error: "method not allowed" });
+      return;
+    }
+
     if (url.pathname === "/chatgpt-bindings") {
       const bindingsFile = path.join(dd, "bindings.json");
       if (req.method === "GET") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-bridge-cli",
-  "version": "2.0.22",
+  "version": "2.0.24",
   "description": "Use Claude Code from your browser. Runs a local server that connects your browser tools to the Claude CLI.",
   "main": "lib/bridge.js",
   "bin": {