chat-recall 0.5.28 → 0.5.30

package/dist/watch.js

@@ -7265,7 +7265,7 @@ async function archiveRawSession(store, sessionId) {
   return await store.putRawSession(sessionId, exp.tool, Math.floor(exp.mtime), gz, size);
 }
 function detectForkPredecessor(sessionPath) {
-  const { readFileSync: readFileSync47, readdirSync: readdirSync34 } = requireFs();
+  const { readFileSync: readFileSync47, readdirSync: readdirSync35 } = requireFs();
   const { dirname: dirname26, join: join56, basename: basename28 } = requirePath();
   let text;
   try {
@@ -7295,7 +7295,7 @@ function detectForkPredecessor(sessionPath) {
   const self = basename28(sessionPath);
   let entries;
   try {
-    entries = readdirSync34(dir);
+    entries = readdirSync35(dir);
   } catch {
     return null;
   }
@@ -14102,9 +14102,9 @@ function doResolve(realPath) {
   const bot = resolveBotWorktree(realPath);
   if (bot)
     return bot;
-  const git = resolveGit(realPath);
-  if (git)
-    return git;
+  const git2 = resolveGit(realPath);
+  if (git2)
+    return git2;
   const ws = resolveAutoWorkspace(realPath, cfg.autoWorkspaceMinRepos ?? DEFAULT_AUTO_WORKSPACE_MIN_REPOS);
   if (ws)
     return ws;
@@ -14173,9 +14173,9 @@ function resolveGit(realPath) {
   const toplevel = findGitToplevel(realPath);
   if (!toplevel)
     return null;
-  const remote = getGitRemote(toplevel);
-  if (remote) {
-    const parsed = parseGitRemote(remote);
+  const probe = getGitRemote(toplevel);
+  if (probe.kind === "remote") {
+    const parsed = parseGitRemote(probe.url);
     if (parsed) {
       return {
         id: `git:${parsed.host}/${parsed.owner}/${parsed.repo}`,
@@ -14184,6 +14184,9 @@ function resolveGit(realPath) {
       };
     }
   }
+  if (probe.kind === "failed") {
+    console.error(`[project] could not read git remotes in ${toplevel} \u2014 filing it as a local-only repo for now, which is probably wrong: ${probe.reason}`);
+  }
   const hash = createHash7("sha1").update(toplevel).digest("hex").slice(0, 12);
   return {
     id: `git-local:${hash}`,
@@ -14202,23 +14205,39 @@ function findGitToplevel(realPath) {
   }
   return null;
 }
+function git(toplevel, args) {
+  return execSync(`git ${args}`, {
+    cwd: toplevel,
+    stdio: ["ignore", "pipe", "ignore"],
+    encoding: "utf-8",
+    timeout: GIT_TIMEOUT_MS
+  });
+}
+function probeGitRemote(toplevel) {
+  let names;
+  try {
+    names = git(toplevel, "remote").split("\n").map((n) => n.trim()).filter(Boolean);
+  } catch (err) {
+    return { kind: "failed", reason: err instanceof Error ? err.message : String(err) };
+  }
+  if (names.length === 0)
+    return { kind: "none" };
+  const pick = names.includes("origin") ? "origin" : names[0];
+  try {
+    const url = git(toplevel, `remote get-url ${pick}`).trim();
+    return url ? { kind: "remote", url } : { kind: "none" };
+  } catch (err) {
+    return { kind: "failed", reason: err instanceof Error ? err.message : String(err) };
+  }
+}
 function getGitRemote(toplevel) {
   const cached = gitCache.get(toplevel);
   if (cached)
-    return cached.remote;
-  let remote = null;
-  try {
-    remote = execSync("git remote get-url origin", {
-      cwd: toplevel,
-      stdio: ["ignore", "pipe", "ignore"],
-      encoding: "utf-8",
-      timeout: 2e3
-    }).trim() || null;
-  } catch {
-    remote = null;
-  }
-  gitCache.set(toplevel, { remote });
-  return remote;
+    return cached;
+  const probe = probeGitRemote(toplevel);
+  if (probe.kind !== "failed")
+    gitCache.set(toplevel, probe);
+  return probe;
 }
 function parseGitRemote(url) {
   const trimmed = url.trim().replace(/\.git$/, "");
@@ -14348,7 +14367,7 @@ function globToRegex(glob) {
 function escapeRe(s) {
   return s.replace(/[.+^${}()|[\]\\]/g, "\\$&");
 }
-var DEFAULT_AUTO_WORKSPACE_MIN_REPOS, BOT_WORKTREE_MARKERS, resolveCache, gitCache, workspaceCache, cachedConfig, AUTO_WORKSPACE_MAX_DEPTH;
+var DEFAULT_AUTO_WORKSPACE_MIN_REPOS, BOT_WORKTREE_MARKERS, resolveCache, gitCache, workspaceCache, cachedConfig, GIT_TIMEOUT_MS, AUTO_WORKSPACE_MAX_DEPTH;
 var init_project_resolver = __esm({
   "../engine/dist/core/project-resolver.js"() {
     "use strict";
@@ -14359,6 +14378,7 @@ var init_project_resolver = __esm({
     gitCache = /* @__PURE__ */ new Map();
     workspaceCache = /* @__PURE__ */ new Map();
     cachedConfig = null;
+    GIT_TIMEOUT_MS = 1e4;
     AUTO_WORKSPACE_MAX_DEPTH = 2;
   }
 });
@@ -19213,7 +19233,7 @@ var init_http = __esm({
       CLI_VERSION = JSON.parse(readFileSync33(new URL("../package.json", import.meta.url), "utf-8")).version || CLI_VERSION;
     } catch {
       try {
-        if (true) CLI_VERSION = "0.5.28";
+        if (true) CLI_VERSION = "0.5.30";
       } catch {
       }
     }
@@ -19721,7 +19741,7 @@ var init_telemetry = __esm({
     init_sync_client();
     cliVersion = () => {
       try {
-        return true ? "0.5.28" : "dev";
+        return true ? "0.5.30" : "dev";
       } catch {
         return "dev";
       }
@@ -19998,12 +20018,13 @@ __export(auto_update_exports, {
   planAutoUpdate: () => planAutoUpdate,
   runAutoUpdate: () => runAutoUpdate,
   runningPrefix: () => runningPrefix,
+  sweepStaleStaging: () => sweepStaleStaging,
   tarballUrl: () => tarballUrl
 });
 import { execSync as execSync2 } from "node:child_process";
 import { readFileSync as readFileSync36 } from "node:fs";
 import { fileURLToPath as fileURLToPath2 } from "node:url";
-import { writeFileSync as writeFileSync8, mkdtempSync as mkdtempSync3 } from "node:fs";
+import { writeFileSync as writeFileSync8, mkdtempSync as mkdtempSync3, rmSync as rmSync5, readdirSync as readdirSync32, statSync as statSync38 } from "node:fs";
 import { join as join43, dirname as dirname18, sep as sep2 } from "node:path";
 import { tmpdir as tmpdir2 } from "node:os";
 import { createHash as createHash11 } from "node:crypto";
@@ -20042,6 +20063,31 @@ function planAutoUpdate(base2, caps, ownVersion3, flag) {
   if (compareVersions(rel.version, ownVersion3) <= 0) return { update: false, reason: `already current (own ${ownVersion3} >= server ${rel.version})` };
   return { update: true, reason: `server ${rel.version} > own ${ownVersion3}`, url: tarballUrl(base2), sha256: rel.sha256, from: ownVersion3, to: rel.version };
 }
+function sweepStaleStaging(root = tmpdir2(), now = Date.now()) {
+  const MKDTEMP_SUFFIX = /^cr-update-.{6}$/s;
+  const COLD_MS = 60 * 60 * 1e3;
+  let removed = 0;
+  let entries;
+  try {
+    entries = readdirSync32(root);
+  } catch {
+    return 0;
+  }
+  for (const name of entries) {
+    if (!MKDTEMP_SUFFIX.test(name)) continue;
+    const dir = join43(root, name);
+    try {
+      const st = statSync38(dir);
+      if (!st.isDirectory() || now - st.mtimeMs < COLD_MS) continue;
+      const inside = readdirSync32(dir);
+      if (inside.length > 1 || inside.length === 1 && inside[0] !== "chat-recall.tgz") continue;
+      rmSync5(dir, { recursive: true, force: true });
+      removed++;
+    } catch {
+    }
+  }
+  return removed;
+}
 async function executeAutoUpdate(plan, deps) {
   if (!plan.update || !plan.url || !plan.sha256) return { updated: false, reason: plan.reason };
   let buf;
@@ -20053,13 +20099,21 @@ async function executeAutoUpdate(plan, deps) {
   const got = createHash11("sha256").update(buf).digest("hex");
   if (got !== plan.sha256) return { updated: false, reason: `checksum mismatch (expected ${plan.sha256.slice(0, 12)}\u2026, got ${got.slice(0, 12)}\u2026) \u2014 NOT installing` };
   const dir = mkdtempSync3(join43(tmpdir2(), "cr-update-"));
+  const discardStaging = () => {
+    try {
+      rmSync5(dir, { recursive: true, force: true });
+    } catch {
+    }
+  };
   const tgz = join43(dir, "chat-recall.tgz");
   writeFileSync8(tgz, buf);
   try {
     deps.install(tgz);
   } catch (e) {
+    discardStaging();
     return { updated: false, reason: `install failed: ${e instanceof Error ? e.message : e}` };
   }
+  discardStaging();
   const landed = (deps.verify ?? installedVersion)();
   if (landed && plan.to && compareVersions(landed, plan.to) < 0) {
     return {
@@ -20141,7 +20195,7 @@ var init_auto_update = __esm({
 
 // src/sync-client.ts
 import { join as join44, dirname as dirname19, basename as basename22 } from "node:path";
-import { readFileSync as readFileSync37, writeFileSync as writeFileSync9, mkdirSync as mkdirSync9, mkdtempSync as mkdtempSync4, rmSync as rmSync5, chmodSync as chmodSync3, unlinkSync as unlinkSync2, readdirSync as readdirSync32, statSync as statSync38 } from "node:fs";
+import { readFileSync as readFileSync37, writeFileSync as writeFileSync9, mkdirSync as mkdirSync9, mkdtempSync as mkdtempSync4, rmSync as rmSync6, chmodSync as chmodSync3, unlinkSync as unlinkSync2, readdirSync as readdirSync33, statSync as statSync39 } from "node:fs";
 import { tmpdir as tmpdir3, hostname as hostname2 } from "node:os";
 import { createHash as createHash12 } from "node:crypto";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
@@ -20460,7 +20514,7 @@ function reapStaleScanDirs(dir = tmpdir3(), maxAgeMs = 6 * 36e5, now = Date.now(
   let reaped = 0;
   let names;
   try {
-    names = readdirSync32(dir);
+    names = readdirSync33(dir);
   } catch {
     return 0;
   }
@@ -20468,8 +20522,8 @@ function reapStaleScanDirs(dir = tmpdir3(), maxAgeMs = 6 * 36e5, now = Date.now(
     if (!name.startsWith("cr-batchscan-")) continue;
     const p = join44(dir, name);
     try {
-      if (now - statSync38(p).mtimeMs < maxAgeMs) continue;
-      rmSync5(p, { recursive: true, force: true });
+      if (now - statSync39(p).mtimeMs < maxAgeMs) continue;
+      rmSync6(p, { recursive: true, force: true });
       reaped++;
     } catch {
     }
@@ -20501,7 +20555,7 @@ function batchScanExternal(items, opts = {}) {
       slices++;
     }
     try {
-      rmSync5(scanDir, { recursive: true, force: true });
+      rmSync6(scanDir, { recursive: true, force: true });
     } catch {
     }
     safeToId = /* @__PURE__ */ new Map();
@@ -20526,7 +20580,7 @@ function batchScanExternal(items, opts = {}) {
     flushSlice();
   } finally {
     try {
-      rmSync5(scanDir, { recursive: true, force: true });
+      rmSync6(scanDir, { recursive: true, force: true });
     } catch {
     }
   }
@@ -20654,9 +20708,10 @@ async function syncToTargetInScope(cred, opts = {}) {
             setServerIngestConcurrency(base2, Math.floor(adv));
           }
           if (body2.cli && body2.cli.version) {
-            const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
+            const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2, sweepStaleStaging: sweepStaleStaging2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
             const authHeaders = cred.token ? { authorization: `Bearer ${cred.token}` } : {};
             const plan = planAutoUpdate2(base2, { cli: body2.cli }, ownVersion, process.env.CHAT_RECALL_AUTO_UPDATE);
+            sweepStaleStaging2();
             if (plan.update) {
               void runAutoUpdate2(base2, authHeaders, ownVersion).catch(() => {
               });
@@ -20782,9 +20837,10 @@ async function syncToTargetInScope(cred, opts = {}) {
       if (!res.ok) throw new Error(`append sync failed: HTTP ${res.status} ${await res.text().catch(() => "")}`);
       const body = await res.json().catch(() => ({}));
       if (body.cli && body.cli.version) {
-        const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
+        const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2, sweepStaleStaging: sweepStaleStaging2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
         const authHeaders = cred.token ? { authorization: `Bearer ${cred.token}` } : {};
         const plan = planAutoUpdate2(base2, { cli: body.cli }, ownVersion, process.env.CHAT_RECALL_AUTO_UPDATE);
+        sweepStaleStaging2();
         if (plan.update) {
           void runAutoUpdate2(base2, authHeaders, ownVersion).catch(() => {
           });
@@ -21586,7 +21642,7 @@ var init_sync_client = __esm({
       ownVersion = JSON.parse(readFileSync37(join44(dir, "../package.json"), "utf-8")).version || ownVersion;
     } catch {
       try {
-        if (true) ownVersion = "0.5.28";
+        if (true) ownVersion = "0.5.30";
       } catch {
       }
     }
@@ -22185,11 +22241,11 @@ var init_toolkit_sync = __esm({
 });
 
 // ../engine/dist/core/which.js
-import { existsSync as existsSync45, statSync as statSync40, accessSync, constants } from "node:fs";
+import { existsSync as existsSync45, statSync as statSync41, accessSync, constants } from "node:fs";
 import { join as join48, isAbsolute as isAbsolute2, delimiter as delimiter2, sep as sep3 } from "node:path";
 function executable(p) {
   try {
-    if (!statSync40(p).isFile())
+    if (!statSync41(p).isFile())
       return false;
   } catch {
     return false;
@@ -22251,11 +22307,11 @@ __export(toolkit_pull_exports, {
   planPull: () => planPull,
   portableCommand: () => portableCommand
 });
-import { existsSync as existsSync46, mkdirSync as mkdirSync12, statSync as statSync41, writeFileSync as writeFileSync12 } from "node:fs";
+import { existsSync as existsSync46, mkdirSync as mkdirSync12, statSync as statSync42, writeFileSync as writeFileSync12 } from "node:fs";
 import { dirname as dirname22, join as join49, basename as basename25, isAbsolute as isAbsolute3 } from "node:path";
 function isExecutableFile(p) {
   try {
-    return statSync41(p).isFile();
+    return statSync42(p).isFile();
   } catch {
     return false;
   }
@@ -22749,14 +22805,14 @@ __export(shadow_prune_exports, {
   DEFAULT_MAX_TOTAL_BYTES: () => DEFAULT_MAX_TOTAL_BYTES,
   pruneShadow: () => pruneShadow
 });
-import { readdirSync as readdirSync33, statSync as statSync43, unlinkSync as unlinkSync3 } from "fs";
+import { readdirSync as readdirSync34, statSync as statSync44, unlinkSync as unlinkSync3 } from "fs";
 import { join as join52 } from "path";
 function listShadows(isFullyAcked) {
   const out = [];
   const root = shadowRoot();
   let tools;
   try {
-    tools = readdirSync33(root);
+    tools = readdirSync34(root);
   } catch {
     return out;
   }
@@ -22764,7 +22820,7 @@ function listShadows(isFullyAcked) {
     const dir = join52(root, tool);
     let names;
     try {
-      names = readdirSync33(dir);
+      names = readdirSync34(dir);
     } catch {
       continue;
     }
@@ -22774,7 +22830,7 @@ function listShadows(isFullyAcked) {
       const path = join52(dir, name);
       let st;
       try {
-        st = statSync43(path);
+        st = statSync44(path);
       } catch {
         continue;
       }
@@ -22889,7 +22945,7 @@ var init_code_intel = __esm({
 });
 
 // ../engine/dist/core/companions.js
-import { existsSync as existsSync49, mkdirSync as mkdirSync15, statSync as statSync44, chmodSync as chmodSync4, unlinkSync as unlinkSync4, renameSync as renameSync5, createWriteStream, readFileSync as readFileSync44, writeFileSync as writeFileSync15 } from "node:fs";
+import { existsSync as existsSync49, mkdirSync as mkdirSync15, statSync as statSync45, chmodSync as chmodSync4, unlinkSync as unlinkSync4, renameSync as renameSync5, createWriteStream, readFileSync as readFileSync44, writeFileSync as writeFileSync15 } from "node:fs";
 import { homedir as homedir19 } from "node:os";
 import { join as join53 } from "node:path";
 import { execSync as execSync3 } from "node:child_process";
@@ -22940,7 +22996,7 @@ function checkCodeindexStatus() {
     status.installed = true;
     status.path = foundPath;
     try {
-      status.size = statSync44(foundPath).size;
+      status.size = statSync45(foundPath).size;
     } catch {
     }
     try {
@@ -24102,7 +24158,7 @@ import { homedir as homedir18 } from "node:os";
 
 // src/project-tasks.ts
 init_http();
-import { existsSync as existsSync44, readFileSync as readFileSync40, writeFileSync as writeFileSync11, mkdirSync as mkdirSync11, statSync as statSync39 } from "node:fs";
+import { existsSync as existsSync44, readFileSync as readFileSync40, writeFileSync as writeFileSync11, mkdirSync as mkdirSync11, statSync as statSync40 } from "node:fs";
 import { join as join47, dirname as dirname21 } from "node:path";
 import { homedir as homedir17 } from "node:os";
 
@@ -24213,7 +24269,7 @@ async function pushProjectTaskStatuses(base2, authHeaders, opts = {}) {
       if (!existsSync44(file)) continue;
       let mtime;
       try {
-        mtime = statSync39(file).mtimeMs;
+        mtime = statSync40(file).mtimeMs;
       } catch {
         continue;
       }
@@ -24374,7 +24430,7 @@ try {
   ownVersion2 = JSON.parse(readFileSync41(join50(dir, "../package.json"), "utf-8")).version || ownVersion2;
 } catch {
   try {
-    if (true) ownVersion2 = "0.5.28";
+    if (true) ownVersion2 = "0.5.30";
   } catch {
   }
 }
@@ -24390,8 +24446,9 @@ async function drainSyncIntents(opts = {}) {
       const data = await res.json();
       pending = data.intents || [];
       if (data.cli && data.cli.version) {
-        const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
+        const { planAutoUpdate: planAutoUpdate2, runAutoUpdate: runAutoUpdate2, sweepStaleStaging: sweepStaleStaging2 } = await Promise.resolve().then(() => (init_auto_update(), auto_update_exports));
         const plan = planAutoUpdate2(base2, { cli: data.cli }, ownVersion2, process.env.CHAT_RECALL_AUTO_UPDATE);
+        sweepStaleStaging2();
         if (plan.update) {
           void runAutoUpdate2(base2, authHeaders, ownVersion2).catch(() => {
           });
@@ -24658,11 +24715,11 @@ function checkSelfRestart(opts) {
 }
 
 // src/log-rotate.ts
-import { statSync as statSync42, copyFileSync as copyFileSync3, truncateSync } from "fs";
+import { statSync as statSync43, copyFileSync as copyFileSync3, truncateSync } from "fs";
 var LOG_MAX_BYTES = 32 * 1024 * 1024;
 function rotateLogIfLarge(logPath, maxBytes = LOG_MAX_BYTES) {
   try {
-    const size = statSync42(logPath).size;
+    const size = statSync43(logPath).size;
     if (size < maxBytes) return { rotated: false, sizeBefore: 0 };
     copyFileSync3(logPath, `${logPath}.1`);
     truncateSync(logPath, 0);
@@ -24718,7 +24775,7 @@ async function runCollectorMigration(opts) {
 init_sync_ledger();
 init_collector_health();
 import { existsSync as existsSync50, readFileSync as readFileSync46 } from "fs";
-var CLI_VERSION3 = true ? "0.5.28" : "0.0.0";
+var CLI_VERSION3 = true ? "0.5.30" : "0.0.0";
 var DEBOUNCE_MS = 5e3;
 var CLAUDE_DIR = claudeBackend.projectsDir();
 var PLANS_DIR = claudeBackend.plansDir();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chat-recall",
-  "version": "0.5.28",
+  "version": "0.5.30",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/munhq/chat-recall.git"

package/skills/chat-recall-project/SKILL.md

@@ -59,12 +59,25 @@ diary conclusions, and cost. Accepts a path, a name substring, or a project id
 
 ## Tasks and team (shared, not personal notes)
 
+**Working the board out loud.** When you take work from here, say so: that you are
+pulling tasks from the chat-recall board, which ones you picked, and — as each one
+lands — that it is done and what changed. The board is the user's, and a card that
+claims and closes itself with no word to them is how it stopped being trusted the
+first time. `recall_tasks detail:true` gives you the brief, the locations and the
+agent prompt for each card, so "what can I pick up" is one call.
+
+
 - `mcp__chat-recall__recall_tasks` — the shared task board: status, assignee,
   project. This is the answer to "what are my open tasks".
 - `mcp__chat-recall__recall_task_create` — create a task, optionally assigned to
   a teammate and attached to a project. Returns the new task id.
 - `mcp__chat-recall__recall_task_update` — change status, reassign, rename or
   comment. Takes the `t_…` id from `recall_tasks`.
+  - **`done` needs the CHANGE.** Pass `diff` — the unified diff of what you changed
+    for that card — and `commits` if you committed it. Nothing else can supply it:
+    the board cannot see edits made through a shell, and its commit scan only
+    searches repositories the session already touched with file tools. The diff you
+    record is what the person reads on the card.
   - **`done` needs the session that did the work.** Pass your own session id;
     without one the update is refused. A card asserts a problem in the code, so
     closing it asserts the code changed, and the board shows the files and commits
@@ -73,6 +86,10 @@ diary conclusions, and cost. Accepts a path, a name substring, or a project id
   - Cards filed from findings also close THEMSELVES once a re-index stops
     reporting the finding, so fixing the code and re-indexing is usually enough.
     Attach your session anyway: the closure keeps its evidence.
+  - **Stay inside the card's project.** A card names one repository; fix it there
+    and nowhere else. If the real fix needs a change in another repo, say so and
+    leave the card open — the API refuses evidence whose file paths climb out of
+    the project, and a diff spanning repositories cannot be reviewed as one change.
   - Filing is capped — at most 10 new cards per run and 50 open at once. A board
     at the ceiling files nothing until cards close; the backlog stays visible in
     the ranked findings view.