@wrongstack/tools 0.306.0 → 0.306.3

package/dist/index.js

@@ -11503,6 +11503,85 @@ function runSqliteWithRetry(fn) {
   throw lastError;
 }
 
+// src/codebase-index/vector-search.ts
+var RRF_K = 60;
+var VECTOR_DIMENSIONS = 384;
+var NGRAM_SIZE = 3;
+function embedText(text) {
+  const vec = new Float32Array(VECTOR_DIMENSIONS);
+  const normalized = text.toLowerCase().trim();
+  if (normalized.length < NGRAM_SIZE) {
+    const padded = ` ${normalized} `.slice(0, Math.max(NGRAM_SIZE, normalized.length + 2));
+    for (let i = 0; i <= padded.length - NGRAM_SIZE; i++) {
+      const ngram = padded.slice(i, i + NGRAM_SIZE);
+      const bucket = hashNgram(ngram) % VECTOR_DIMENSIONS;
+      vec[bucket] += 1;
+    }
+  } else {
+    for (let i = 0; i <= normalized.length - NGRAM_SIZE; i++) {
+      const ngram = normalized.slice(i, i + NGRAM_SIZE);
+      const bucket = hashNgram(ngram) % VECTOR_DIMENSIONS;
+      vec[bucket] += 1;
+    }
+  }
+  let norm = 0;
+  for (let i = 0; i < VECTOR_DIMENSIONS; i++) {
+    norm += vec[i] * vec[i];
+  }
+  norm = Math.sqrt(norm);
+  if (norm > 0) {
+    for (let i = 0; i < VECTOR_DIMENSIONS; i++) {
+      vec[i] /= norm;
+    }
+  }
+  return vec;
+}
+function hashNgram(str) {
+  let hash = 2166136261;
+  for (let i = 0; i < str.length; i++) {
+    hash ^= str.charCodeAt(i);
+    hash = Math.imul(hash, 16777619);
+  }
+  return hash >>> 0;
+}
+function cosineSimilarity(a, b) {
+  let dot = 0;
+  const len = Math.min(a.length, b.length);
+  for (let i = 0; i < len; i++) {
+    dot += a[i] * b[i];
+  }
+  return dot;
+}
+function encodeVector(vec) {
+  return Buffer.from(vec.buffer, vec.byteOffset, vec.byteLength);
+}
+function decodeVector(buf) {
+  const view = new DataView(
+    buf.buffer,
+    buf.byteOffset,
+    buf.byteLength
+  );
+  const copy = new Float32Array(buf.byteLength / 4);
+  for (let i = 0; i < copy.length; i++) {
+    copy[i] = view.getFloat32(i * 4, true);
+  }
+  return copy;
+}
+function reciprocalRankFusion(bm25Ranks, vectorRanks, k = RRF_K) {
+  const allIds = /* @__PURE__ */ new Set([...bm25Ranks.keys(), ...vectorRanks.keys()]);
+  const scored = [];
+  for (const id of allIds) {
+    const bm25Rank = bm25Ranks.get(id);
+    const vecRank = vectorRanks.get(id);
+    let score = 0;
+    if (bm25Rank !== void 0) score += 1 / (k + bm25Rank);
+    if (vecRank !== void 0) score += 1 / (k + vecRank);
+    scored.push([id, score]);
+  }
+  scored.sort((a, b) => b[1] - a[1]);
+  return scored;
+}
+
 // src/codebase-index/writer-admin.ts
 import * as fs9 from "node:fs";
 import * as path14 from "node:path";
@@ -12864,90 +12943,19 @@ var StorePool = class {
   }
 };
 
-// src/codebase-index/vector-search.ts
-var RRF_K = 60;
-var VECTOR_DIMENSIONS = 384;
-var NGRAM_SIZE = 3;
-function embedText(text) {
-  const vec = new Float32Array(VECTOR_DIMENSIONS);
-  const normalized = text.toLowerCase().trim();
-  if (normalized.length < NGRAM_SIZE) {
-    const padded = ` ${normalized} `.slice(0, Math.max(NGRAM_SIZE, normalized.length + 2));
-    for (let i = 0; i <= padded.length - NGRAM_SIZE; i++) {
-      const ngram = padded.slice(i, i + NGRAM_SIZE);
-      const bucket = hashNgram(ngram) % VECTOR_DIMENSIONS;
-      vec[bucket] += 1;
-    }
-  } else {
-    for (let i = 0; i <= normalized.length - NGRAM_SIZE; i++) {
-      const ngram = normalized.slice(i, i + NGRAM_SIZE);
-      const bucket = hashNgram(ngram) % VECTOR_DIMENSIONS;
-      vec[bucket] += 1;
-    }
-  }
-  let norm = 0;
-  for (let i = 0; i < VECTOR_DIMENSIONS; i++) {
-    norm += vec[i] * vec[i];
-  }
-  norm = Math.sqrt(norm);
-  if (norm > 0) {
-    for (let i = 0; i < VECTOR_DIMENSIONS; i++) {
-      vec[i] /= norm;
-    }
-  }
-  return vec;
-}
-function hashNgram(str) {
-  let hash = 2166136261;
-  for (let i = 0; i < str.length; i++) {
-    hash ^= str.charCodeAt(i);
-    hash = Math.imul(hash, 16777619);
-  }
-  return hash >>> 0;
-}
-function cosineSimilarity(a, b) {
-  let dot = 0;
-  const len = Math.min(a.length, b.length);
-  for (let i = 0; i < len; i++) {
-    dot += a[i] * b[i];
-  }
-  return dot;
-}
-function encodeVector(vec) {
-  return Buffer.from(vec.buffer, vec.byteOffset, vec.byteLength);
-}
-function decodeVector(buf) {
-  const view = new DataView(
-    buf.buffer,
-    buf.byteOffset,
-    buf.byteLength
-  );
-  const copy = new Float32Array(buf.byteLength / 4);
-  for (let i = 0; i < copy.length; i++) {
-    copy[i] = view.getFloat32(i * 4, true);
-  }
-  return copy;
-}
-function reciprocalRankFusion(bm25Ranks, vectorRanks, k = RRF_K) {
-  const allIds = /* @__PURE__ */ new Set([...bm25Ranks.keys(), ...vectorRanks.keys()]);
-  const scored = [];
-  for (const id of allIds) {
-    const bm25Rank = bm25Ranks.get(id);
-    const vecRank = vectorRanks.get(id);
-    let score = 0;
-    if (bm25Rank !== void 0) score += 1 / (k + bm25Rank);
-    if (vecRank !== void 0) score += 1 / (k + vecRank);
-    scored.push([id, score]);
-  }
-  scored.sort((a, b) => b[1] - a[1]);
-  return scored;
-}
-
 // src/codebase-index/writer.ts
 var DB_FILE2 = "index.db";
 var MAX_STATEMENT_CACHE = 128;
 var IndexStore = class _IndexStore {
   db;
+  /**
+   * True while an index run owns one outer SQLite transaction. Individual
+   * writer methods normally protect themselves with BEGIN/COMMIT, but during
+   * a refresh they join this transaction so readers observe either the last
+   * completed index or the next completed index, never an in-between batch.
+   */
+  atomicIndexUpdateActive = false;
+  writeSavepointSequence = 0;
   /** Absolute path to this project's index directory. */
   indexDir;
   /**
@@ -13028,6 +13036,51 @@ var IndexStore = class _IndexStore {
   runWithRetry(fn) {
     return runSqliteWithRetry(fn);
   }
+  /** Run a complete index mutation as one WAL-visible publication. */
+  async runAtomicIndexUpdate(job) {
+    if (this.atomicIndexUpdateActive) return job();
+    this.runWithRetry(() => this.db.exec("BEGIN IMMEDIATE"));
+    this.atomicIndexUpdateActive = true;
+    try {
+      const result = await job();
+      this.db.exec("COMMIT");
+      return result;
+    } catch (error) {
+      try {
+        this.db.exec("ROLLBACK");
+      } catch {
+      }
+      throw error;
+    } finally {
+      this.atomicIndexUpdateActive = false;
+    }
+  }
+  /**
+   * Begin a method-local transaction. Inside an atomic index publication a
+   * SAVEPOINT preserves the old per-batch rollback boundary, which is needed
+   * when commitBatch falls back to per-file writes after one batch fails.
+   */
+  beginWriteTransaction() {
+    if (this.atomicIndexUpdateActive) {
+      const savepoint = `index_write_${++this.writeSavepointSequence}`;
+      this.db.exec(`SAVEPOINT ${savepoint}`);
+      return savepoint;
+    }
+    this.db.exec("BEGIN IMMEDIATE");
+    return null;
+  }
+  commitWriteTransaction(savepoint) {
+    if (savepoint) this.db.exec(`RELEASE SAVEPOINT ${savepoint}`);
+    else this.db.exec("COMMIT");
+  }
+  rollbackWriteTransaction(savepoint) {
+    if (savepoint) {
+      this.db.exec(`ROLLBACK TO SAVEPOINT ${savepoint}`);
+      this.db.exec(`RELEASE SAVEPOINT ${savepoint}`);
+    } else {
+      this.db.exec("ROLLBACK");
+    }
+  }
   /**
    * Mirror the in-process language→family map into SQLite.
    *
@@ -13262,7 +13315,7 @@ var IndexStore = class _IndexStore {
   insertSymbols(symbols) {
     this.invalidateBm25();
     return this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         let nextId = this.allocateSymbolIds(symbols.length);
         const result = [];
@@ -13289,7 +13342,9 @@ var IndexStore = class _IndexStore {
           }
           vectorRows.push({
             id,
-            vector: encodeVector(embedText(s.text || buildIndexableText(s.name, s.signature, s.docComment)))
+            vector: encodeVector(
+              embedText(s.text || buildIndexableText(s.name, s.signature, s.docComment))
+            )
           });
           result.push({ ...s, id });
         }
@@ -13307,10 +13362,10 @@ var IndexStore = class _IndexStore {
             vectorRows
           );
         }
-        this.db.exec("COMMIT");
+        this.commitWriteTransaction(ownsTransaction);
         return result;
       } catch (err) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw err;
       }
     });
@@ -13318,7 +13373,7 @@ var IndexStore = class _IndexStore {
   deleteSymbolsForFile(file) {
     this.invalidateBm25();
     this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         const affectedNames = this.invalidateIncomingRefsForFiles([file]);
         if (this.ftsAvailable) {
@@ -13333,9 +13388,9 @@ var IndexStore = class _IndexStore {
         }
         this.stmt("DELETE FROM symbols WHERE file_fk = ?").run(file);
         this.resolveRefsForNamesUnsafe(affectedNames);
-        this.db.exec("COMMIT");
+        this.commitWriteTransaction(ownsTransaction);
       } catch (error) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw error;
       }
     });
@@ -13348,7 +13403,7 @@ var IndexStore = class _IndexStore {
   deleteFile(file) {
     this.invalidateBm25();
     this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         const affectedNames = this.invalidateIncomingRefsForFiles([file]);
         if (this.ftsAvailable) {
@@ -13367,9 +13422,9 @@ var IndexStore = class _IndexStore {
         this.stmt("DELETE FROM symbols WHERE file_fk = ?").run(file);
         this.stmt("DELETE FROM files WHERE file = ?").run(file);
         this.resolveRefsForNamesUnsafe(affectedNames);
-        this.db.exec("COMMIT");
+        this.commitWriteTransaction(ownsTransaction);
       } catch (err) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw err;
       }
     });
@@ -13756,7 +13811,7 @@ var IndexStore = class _IndexStore {
   clearAll() {
     this.invalidateBm25();
     this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         this.db.exec("DROP TABLE IF EXISTS refs");
         this.db.exec("DROP TABLE IF EXISTS symbols");
@@ -13764,15 +13819,15 @@ var IndexStore = class _IndexStore {
         this.db.exec("DROP TABLE IF EXISTS metadata");
         if (this.ftsAvailable) this.db.exec("DROP TABLE IF EXISTS symbols_fts");
         this.db.exec("DROP TABLE IF EXISTS symbol_vectors");
-        this.db.exec("COMMIT");
         this.stmtCache.clear();
         this.initSchema();
         this.stmt("INSERT OR REPLACE INTO metadata(key, value) VALUES (?, ?)").run(
           _IndexStore.NEXT_SYMBOL_ID_KEY,
           "1"
         );
+        this.commitWriteTransaction(ownsTransaction);
       } catch (err) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw err;
       }
     });
@@ -13834,7 +13889,7 @@ var IndexStore = class _IndexStore {
     }
     this.invalidateBm25();
     return this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         const affectedNames = /* @__PURE__ */ new Set();
         for (const entry of entries) {
@@ -13895,7 +13950,9 @@ var IndexStore = class _IndexStore {
             }
             vectorRows.push({
               id,
-              vector: encodeVector(embedText(s.text || buildIndexableText(s.name, s.signature, s.docComment)))
+              vector: encodeVector(
+                embedText(s.text || buildIndexableText(s.name, s.signature, s.docComment))
+              )
             });
             const inserted = { ...s, id };
             allInserted.push(inserted);
@@ -13940,10 +13997,10 @@ var IndexStore = class _IndexStore {
           );
         }
         this.resolveRefsForNamesUnsafe(affectedNames);
-        this.db.exec("COMMIT");
+        this.commitWriteTransaction(ownsTransaction);
         return allInserted;
       } catch (err) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw err;
       }
     });
@@ -14020,7 +14077,7 @@ var IndexStore = class _IndexStore {
   replaceEmptyFile(meta) {
     this.invalidateBm25();
     this.runWithRetry(() => {
-      this.db.exec("BEGIN IMMEDIATE");
+      const ownsTransaction = this.beginWriteTransaction();
       try {
         const affectedNames = this.invalidateIncomingRefsForFiles([meta.file]);
         if (this.ftsAvailable) {
@@ -14055,9 +14112,9 @@ var IndexStore = class _IndexStore {
           meta.lastIndexed
         );
         this.resolveRefsForNamesUnsafe(affectedNames);
-        this.db.exec("COMMIT");
+        this.commitWriteTransaction(ownsTransaction);
       } catch (err) {
-        this.db.exec("ROLLBACK");
+        this.rollbackWriteTransaction(ownsTransaction);
         throw err;
       }
     });
@@ -15015,6 +15072,7 @@ import { Worker as Worker2 } from "node:worker_threads";
 // src/codebase-index/indexer.ts
 import { expectDefined as expectDefined6 } from "@wrongstack/core/utils";
 import { execFile } from "node:child_process";
+import { createHash as createHash5 } from "node:crypto";
 import * as fs18 from "node:fs/promises";
 import { availableParallelism } from "node:os";
 import * as path24 from "node:path";
@@ -15855,6 +15913,10 @@ var DEFAULT_IGNORE_FILES = /* @__PURE__ */ new Set(["package-lock.json", "pnpm-l
 var INDEXABLE_EXTENSION_SET = new Set(INDEXABLE_EXTENSIONS);
 var MAX_INDEX_FILE_BYTES = 5 * 1024 * 1024;
 var MAX_GIT_FILE_LIST_BYTES = 64 * 1024 * 1024;
+var GIT_SNAPSHOT_METADATA_KEY = "git_discovery_snapshot";
+var IndexSourceChangedError = class extends Error {
+  name = "IndexSourceChangedError";
+};
 function isWithinProject(projectRoot, file) {
   const rel = path24.relative(projectRoot, file);
   return rel !== "" && !rel.startsWith(`..${path24.sep}`) && rel !== ".." && !path24.isAbsolute(rel);
@@ -15891,7 +15953,7 @@ async function findGitSourceFiles(projectRoot, ignore, signal) {
     if (normalizeComparablePath(topLevel) !== normalizeComparablePath(projectRoot)) return null;
     throwIfAborted(signal);
     const ignoreSet = /* @__PURE__ */ new Set([...DEFAULT_IGNORE, ...ignore]);
-    const [output, statusOutput] = await Promise.all([
+    const [output, statusOutput, stagedOutput] = await Promise.all([
       gitOutput(projectRoot, ["ls-files", "--cached", "--others", "--exclude-standard", "-z"]),
       gitOutput(projectRoot, [
         "status",
@@ -15899,7 +15961,8 @@ async function findGitSourceFiles(projectRoot, ignore, signal) {
         "-z",
         "--untracked-files=all",
         "--ignored=no"
-      ])
+      ]),
+      gitOutput(projectRoot, ["ls-files", "--stage", "-z"])
     ]);
     throwIfAborted(signal);
     const dirty = /* @__PURE__ */ new Set();
@@ -15929,9 +15992,17 @@ async function findGitSourceFiles(projectRoot, ignore, signal) {
       const ext = path24.extname(relative13).toLowerCase();
       if (INDEXABLE_EXTENSION_SET.has(ext) || detectLang(full) !== null) files.push(full);
     }
+    const snapshot = createHash5("sha256").update(stagedOutput).update("\0").update(statusOutput);
+    const indexedFiles = new Set(files);
+    for (const dirtyFile of [...dirty].sort()) {
+      if (!indexedFiles.has(dirtyFile) || deleted.has(dirtyFile)) continue;
+      snapshot.update("\0").update(dirtyFile).update("\0");
+      snapshot.update(xxhash64String(await fs18.readFile(dirtyFile, "utf8")));
+    }
     return {
       files,
-      trustedUnchanged: new Set(files.filter((file) => !dirty.has(file)))
+      trustedUnchanged: new Set(files.filter((file) => !dirty.has(file))),
+      snapshotKey: snapshot.digest("hex")
     };
   } catch {
     return null;
@@ -15944,7 +16015,8 @@ async function findSourceFiles(projectRoot, ignore, isGitIgnored, signal) {
       files: gitFiles.files,
       complete: true,
       errors: [],
-      trustedUnchanged: gitFiles.trustedUnchanged
+      trustedUnchanged: gitFiles.trustedUnchanged,
+      snapshotKey: gitFiles.snapshotKey
     };
   }
   const results = [];
@@ -16031,6 +16103,17 @@ async function resolveProjectRelations(store, projectRoot, opts) {
   }
 }
 async function runIndexerWithStore(store, opts) {
+  let result;
+  try {
+    result = await store.runAtomicIndexUpdate(() => runIndexerAtomic(store, opts));
+  } catch (error) {
+    if (!(error instanceof IndexSourceChangedError)) throw error;
+    result = await store.runAtomicIndexUpdate(() => runIndexerAtomic(store, opts));
+  }
+  if (!opts.files) store.compactIfNeeded();
+  return result;
+}
+async function runIndexerAtomic(store, opts) {
   const { projectRoot, langs, ignore = [], signal } = opts;
   const relationGraphVersion = "2";
   const refResolutionVersion = "2";
@@ -16050,6 +16133,7 @@ async function runIndexerWithStore(store, opts) {
   let discoveredFiles = null;
   let discoveryComplete = true;
   let trustedUnchanged;
+  let discoverySnapshotKey;
   if (opts.files && opts.files.length > 0) {
     files = opts.files.map((f) => path24.resolve(projectRoot, f)).filter((f) => {
       if (!isWithinProject(projectRoot, f)) return false;
@@ -16063,6 +16147,7 @@ async function runIndexerWithStore(store, opts) {
     discoveryComplete = discovery.complete;
     discoveredFiles = new Set(files);
     trustedUnchanged = discovery.trustedUnchanged;
+    discoverySnapshotKey = discovery.snapshotKey;
   }
   if (langs && langs.length > 0) {
     const langSet = new Set(langs);
@@ -16076,6 +16161,8 @@ async function runIndexerWithStore(store, opts) {
   if (!force) {
     for (const meta of store.getAllFileMetas()) existingMeta.set(meta.file, meta);
   }
+  const snapshotTrusted = !force && discoverySnapshotKey !== void 0 && store.getMetadata(GIT_SNAPSHOT_METADATA_KEY) === discoverySnapshotKey;
+  if (!snapshotTrusted) trustedUnchanged = void 0;
   const totalFilesForProgress = files.length;
   let filesPreSkipped = 0;
   if (!force && trustedUnchanged) {
@@ -16138,9 +16225,6 @@ async function runIndexerWithStore(store, opts) {
             };
           }
           const meta = existingMeta.get(file);
-          if (!force && meta && meta.mtimeMs === Math.floor(stat20.mtimeMs)) {
-            return { file, stat: stat20, lang, parsed: null, skippedMeta: meta };
-          }
           let content;
           try {
             content = await fs18.readFile(file, { encoding: "utf8", signal });
@@ -16369,9 +16453,18 @@ async function runIndexerWithStore(store, opts) {
   });
   store.setMetadata("ref_resolution_version", refResolutionVersion);
   store.setMetadata("relation_graph_version", relationGraphVersion);
+  const completeProjectScope = !opts.files && (!langs || langs.length === 0) && (!opts.ignore || opts.ignore.length === 0);
+  if (completeProjectScope && discoverySnapshotKey !== void 0) {
+    const finalSnapshot = await findGitSourceFiles(projectRoot, ignore, signal);
+    if (!finalSnapshot || finalSnapshot.snapshotKey !== discoverySnapshotKey) {
+      throw new IndexSourceChangedError(
+        "Project files changed during indexing; retrying before publishing the generation."
+      );
+    }
+    store.setMetadata(GIT_SNAPSHOT_METADATA_KEY, errors.length === 0 ? discoverySnapshotKey : "");
+  }
   if (!opts.files || filesIndexed >= 50) store.optimize();
   store.setLastIndexed(Date.now());
-  if (!opts.files) store.compactIfNeeded();
   const durationMs = Date.now() - startMs;
   return {
     filesIndexed,
@@ -16682,6 +16775,13 @@ function callIndexOp(op, args, opts) {
   });
 }
 async function callInline(op, args, opts) {
+  if (op !== "index" && _indexing) {
+    const error = new Error(
+      "Codebase index refresh in progress; retry after the completed generation is published."
+    );
+    error.name = "IndexRefreshInProgressError";
+    throw error;
+  }
   const ac = new AbortController();
   const onOuterAbort = () => ac.abort(opts.signal?.reason ?? new Error("Indexing cancelled"));
   if (opts.signal?.aborted) onOuterAbort();
@@ -17026,12 +17126,12 @@ var codebaseSearchTool = {
   },
   async execute(input, ctx, execOpts) {
     const state = getIndexState();
-    if (state.indexing && !state.ready) {
+    if (state.indexing) {
       return {
         results: [],
         total: 0,
         query: input.query,
-        indexStatus: `Indexing in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry in a moment.`
+        indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry after the completed generation is published.`
       };
     }
     if (state.lastError) {
@@ -17212,12 +17312,12 @@ var codebaseIncomingCallsTool = {
   },
   async execute(input, ctx) {
     const state = getIndexState();
-    if (state.indexing && !state.ready) {
+    if (state.indexing) {
       return {
         symbol: input.symbol,
         calls: [],
         total: 0,
-        indexStatus: `Indexing in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry in a moment.`
+        indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry after the completed generation is published.`
       };
     }
     if (state.lastError) {
@@ -17234,16 +17334,14 @@ var codebaseIncomingCallsTool = {
     const transitive = input.transitive === true;
     let serviced;
     try {
-      serviced = await incomingCallsService2(
-        {
-          projectRoot: ctx.projectRoot,
-          indexDir: codebaseIndexDirOverride(ctx),
-          symbol: input.symbol,
-          file: input.file,
-          limit,
-          transitive
-        }
-      );
+      serviced = await incomingCallsService2({
+        projectRoot: ctx.projectRoot,
+        indexDir: codebaseIndexDirOverride(ctx),
+        symbol: input.symbol,
+        file: input.file,
+        limit,
+        transitive
+      });
     } catch (err) {
       return {
         symbol: input.symbol,
@@ -17282,10 +17380,14 @@ var codebaseIncomingCallsTool = {
     }
     const notes = [];
     if (totalMatches > limit) {
-      notes.push(`Results capped at ${limit} of ${totalMatches} call sites. Increase \`limit\` or use \`file\` to narrow.`);
+      notes.push(
+        `Results capped at ${limit} of ${totalMatches} call sites. Increase \`limit\` or use \`file\` to narrow.`
+      );
     }
     if (ambiguous) {
-      notes.push(`Symbol "${input.symbol}" exists in multiple files. Results include callers of all same-named symbols. Use codebase-search to find the exact file and pass it as \`file\`.`);
+      notes.push(
+        `Symbol "${input.symbol}" exists in multiple files. Results include callers of all same-named symbols. Use codebase-search to find the exact file and pass it as \`file\`.`
+      );
     }
     return {
       symbol: input.symbol,
@@ -17335,12 +17437,12 @@ var codebaseOutgoingCallsTool = {
   },
   async execute(input, ctx) {
     const state = getIndexState();
-    if (state.indexing && !state.ready) {
+    if (state.indexing) {
       return {
         symbol: input.symbol,
         calls: [],
         total: 0,
-        indexStatus: `Indexing in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry in a moment.`
+        indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry after the completed generation is published.`
       };
     }
     if (state.lastError) {
@@ -17357,16 +17459,14 @@ var codebaseOutgoingCallsTool = {
     const transitive = input.transitive === true;
     let serviced;
     try {
-      serviced = await outgoingCallsService2(
-        {
-          projectRoot: ctx.projectRoot,
-          indexDir: codebaseIndexDirOverride(ctx),
-          symbol: input.symbol,
-          file: input.file,
-          limit,
-          transitive
-        }
-      );
+      serviced = await outgoingCallsService2({
+        projectRoot: ctx.projectRoot,
+        indexDir: codebaseIndexDirOverride(ctx),
+        symbol: input.symbol,
+        file: input.file,
+        limit,
+        transitive
+      });
     } catch (err) {
       return {
         symbol: input.symbol,
@@ -17405,10 +17505,14 @@ var codebaseOutgoingCallsTool = {
     }
     const notes = [];
     if (totalMatches > limit) {
-      notes.push(`Results capped at ${limit} of ${totalMatches} call sites. Increase \`limit\` or use \`file\` to narrow.`);
+      notes.push(
+        `Results capped at ${limit} of ${totalMatches} call sites. Increase \`limit\` or use \`file\` to narrow.`
+      );
     }
     if (unresolvedCount > 0) {
-      notes.push(`${unresolvedCount} unresolved reference(s) not shown \u2014 their targets could not be resolved during indexing.`);
+      notes.push(
+        `${unresolvedCount} unresolved reference(s) not shown \u2014 their targets could not be resolved during indexing.`
+      );
     }
     return {
       symbol: input.symbol,
@@ -25035,6 +25139,7 @@ function sourceStatus(task) {
 function todoStatus(task) {
   const status = sourceStatus(task);
   if (status === "completed") return "completed";
+  if (status === "review" && task.assignment?.status === "completed") return "completed";
   if (status === "in_progress" || status === "review") return "in_progress";
   return "pending";
 }
@@ -25156,11 +25261,12 @@ async function applySessionKanbanTaskToSource(context, task, options = {}) {
   const graphId = task.origin?.graphId ?? "";
   if (!originId) return { source: null };
   if (task.origin?.system === "session-todo" || graphId.startsWith("todo:")) {
+    const mappedStatus = todoStatus(task);
     const next = options.remove ? context.todos.filter((todo) => todo.id !== originId) : context.todos.map(
       (todo) => todo.id === originId ? {
         ...todo,
         content: task.title,
-        status: sourceStatus(task) === "completed" ? "completed" : sourceStatus(task) === "in_progress" || sourceStatus(task) === "review" ? "in_progress" : "pending"
+        status: mappedStatus
       } : todo
     );
     suppressedTodoMirrors.add(context);
@@ -27077,6 +27183,20 @@ async function synchronizeManagedKanban(items, board, ctx, signal) {
     }
     const task = board.tasks.find((candidate) => candidate.id === item.kanbanTaskId);
     if (!task || task.status === "completed") continue;
+    const stage = task.lifecycle?.currentStage;
+    if (stage === "backlog" || stage === "todo") {
+      const started = await execute({
+        action: "start_task",
+        boardId: board.id,
+        taskId: task.id,
+        author: actor,
+        agentId: actor,
+        transitionComment: `Auto-started for completion: ${item.content}`
+      });
+      if (!started.ok) {
+        continue;
+      }
+    }
     await execute({
       action: "mark_assignment",
       boardId: board.id,