@wrongstack/tools 1.0.8 → 1.0.9

package/dist/{chunk-JB5JLY7V.js → chunk-ENE22I5W.js}

@@ -3,7 +3,7 @@ import {
 } from "./chunk-XRNN44QY.js";
 import {
   spawnStream
-} from "./chunk-EMA2XENR.js";
+} from "./chunk-UVYRQKPZ.js";
 import {
   codebaseContext,
   codebaseIndexStats,
@@ -15,7 +15,7 @@ import {
   outgoingCallsService,
   runStartupIndex,
   searchCodebaseIndex
-} from "./chunk-TU6UVATU.js";
+} from "./chunk-HYHE4VNA.js";
 import {
   IndexTimeoutError,
   SCHEMA_VERSION,
@@ -26,7 +26,7 @@ import {
   posixIndexPath,
   resolveIndexDir,
   xxhash64String
-} from "./chunk-C4DSQBTT.js";
+} from "./chunk-NEYUPZL6.js";
 import {
   EXT_TO_LANG,
   detectLang,
@@ -477,6 +477,16 @@ async function loadTs2() {
   }
   return tsModule2;
 }
+function indentAt(content, pos) {
+  const lineStart = content.lastIndexOf("\n", pos - 1) + 1;
+  return /^[ \t]*/.exec(content.slice(lineStart, pos))?.[0] ?? "";
+}
+function reindent(text, indent) {
+  const lines = text.replace(/^\s*\n/, "").trimEnd().split("\n");
+  const widths = lines.filter((l) => l.trim()).map((l) => /^[ \t]*/.exec(l)?.[0].length ?? 0);
+  const common = widths.length > 0 ? Math.min(...widths) : 0;
+  return lines.map((l) => l.trim() ? `${indent}${l.slice(common)}` : "").join("\n");
+}
 async function mutateTsSymbol(content, lang, symbolName, newBody, target = "body") {
   const ts = await loadTs2();
   const scriptKind = lang === "tsx" ? ts.ScriptKind.TSX : lang === "jsx" ? ts.ScriptKind.JSX : lang === "js" ? ts.ScriptKind.JS : ts.ScriptKind.TS;
@@ -487,49 +497,88 @@ async function mutateTsSymbol(content, lang, symbolName, newBody, target = "body
     true,
     scriptKind
   );
-  let targetNode = null;
-  let bodyNode = null;
+  const nameOf = (node) => {
+    const n = node.name;
+    if (!n) return void 0;
+    return ts.isIdentifier(n) || ts.isPrivateIdentifier(n) || ts.isStringLiteral(n) ? n.text : void 0;
+  };
+  const matches = [];
+  const scope = [];
   function visit(node) {
-    if (targetNode) return;
-    if (ts.isFunctionDeclaration(node) || ts.isMethodDeclaration(node) || ts.isConstructorDeclaration(node) || ts.isGetAccessor(node) || ts.isSetAccessor(node) || ts.isClassDeclaration(node) || ts.isInterfaceDeclaration(node)) {
-      let name;
-      if ("name" in node && node.name && ts.isIdentifier(node.name)) {
-        name = node.name.text;
-      }
-      if (name === symbolName) {
-        targetNode = node;
-        if ("body" in node && node.body) {
-          bodyNode = node.body;
-        }
-        return;
-      }
-    } else if (ts.isVariableStatement(node)) {
-      for (const decl of node.declarationList.declarations) {
-        if (ts.isIdentifier(decl.name) && decl.name.text === symbolName) {
-          targetNode = decl;
-          if (decl.initializer && (ts.isArrowFunction(decl.initializer) || ts.isFunctionExpression(decl.initializer))) {
-            bodyNode = decl.initializer.body;
-          }
-          return;
-        }
-      }
+    let declared;
+    let body = null;
+    if (ts.isFunctionDeclaration(node) || ts.isMethodDeclaration(node) || ts.isGetAccessor(node) || ts.isSetAccessor(node)) {
+      declared = nameOf(node);
+      body = node.body ?? null;
+    } else if (ts.isConstructorDeclaration(node)) {
+      declared = "constructor";
+      body = node.body ?? null;
+    } else if (ts.isClassDeclaration(node) || ts.isInterfaceDeclaration(node) || ts.isEnumDeclaration(node) || ts.isModuleDeclaration(node)) {
+      declared = nameOf(node);
+    } else if (ts.isTypeAliasDeclaration(node)) {
+      declared = nameOf(node);
+      body = node.type;
+    } else if (ts.isVariableDeclaration(node) || ts.isPropertyDeclaration(node)) {
+      declared = nameOf(node);
+      const init = node.initializer;
+      if (init && (ts.isArrowFunction(init) || ts.isFunctionExpression(init))) {
+        body = init.body;
+      }
+    }
+    if (declared === void 0) {
+      ts.forEachChild(node, visit);
+      return;
     }
+    matches.push({
+      node,
+      body,
+      qualified: [...scope, declared].join("."),
+      line: sourceFile.getLineAndCharacterOfPosition(node.getStart(sourceFile)).line + 1
+    });
+    scope.push(declared);
     ts.forEachChild(node, visit);
+    scope.pop();
   }
   ts.forEachChild(sourceFile, visit);
-  if (!targetNode) return null;
-  const replaceTarget = target === "full" || !bodyNode ? targetNode : bodyNode;
+  let candidates = matches.filter(
+    (m) => m.qualified === symbolName || m.qualified.endsWith(`.${symbolName}`)
+  );
+  if (candidates.length === 0) return null;
+  if (candidates.length > 1) {
+    const first = candidates[0]?.qualified;
+    const overloadable = candidates.every(
+      (m) => m.qualified === first && (ts.isFunctionDeclaration(m.node) || ts.isMethodDeclaration(m.node) || ts.isConstructorDeclaration(m.node))
+    );
+    const implemented = candidates.filter((m) => m.body !== null);
+    if (overloadable && implemented.length === 1) candidates = implemented;
+  }
+  if (candidates.length > 1) {
+    const listed = candidates.map((m) => `${m.qualified} (L${m.line})`).join(", ");
+    const example = candidates.find((m) => m.qualified.includes("."))?.qualified;
+    throw new Error(
+      `Symbol '${symbolName}' is ambiguous: ${candidates.length} declarations match \u2014 ${listed}. ` + (example ? `Pass a qualified name, e.g. symbol: "${example}".` : "Rename-free disambiguation is impossible here; use the edit tool.")
+    );
+  }
+  const match = candidates[0];
+  let fullNode = match.node;
+  if (ts.isVariableDeclaration(match.node)) {
+    const list = match.node.parent;
+    const stmt = list?.parent;
+    if (list && ts.isVariableDeclarationList(list) && list.declarations.length === 1 && stmt && ts.isVariableStatement(stmt)) {
+      fullNode = stmt;
+    }
+  }
+  const replaceTarget = target === "full" || !match.body ? fullNode : match.body;
   const startPos = replaceTarget.getStart(sourceFile);
   const endPos = replaceTarget.getEnd();
   const startLine = sourceFile.getLineAndCharacterOfPosition(startPos).line + 1;
   const endLine = sourceFile.getLineAndCharacterOfPosition(endPos).line + 1;
   let formattedReplacement = newBody.trim();
-  if (target === "body" && bodyNode && ts.isBlock(bodyNode)) {
-    if (!formattedReplacement.startsWith("{")) {
-      formattedReplacement = `{
-  ${formattedReplacement.split("\n").join("\n  ")}
-}`;
-    }
+  if (target === "body" && match.body && ts.isBlock(match.body) && !formattedReplacement.startsWith("{")) {
+    const baseIndent = indentAt(content, fullNode.getStart(sourceFile));
+    formattedReplacement = `{
+${reindent(newBody, `${baseIndent}  `)}
+${baseIndent}}`;
   }
   const updatedContent = content.slice(0, startPos) + formattedReplacement + content.slice(endPos);
   const newEndLine = startLine + formattedReplacement.split("\n").length - 1;
@@ -599,6 +648,21 @@ async function mutateTreeSitterSymbol(content, lang, symbolName, newBody, target
     parser.delete();
   }
 }
+var TEST_CALLEES = String.raw`(?:it|test|describe|suite|bench|context|specify)`;
+function notFoundMessage(symbol, relPath, content) {
+  const base = `Symbol '${symbol}' could not be located in AST of '${relPath}'`;
+  const escaped = symbol.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  const testCall = new RegExp(
+    String.raw`\b${TEST_CALLEES}(?:\.\w+)*\s*\(\s*(['"\x60])${escaped}\1`
+  );
+  if (testCall.test(content)) {
+    return `${base}: that is a test-case title, not a declaration. codebase-ast-replace only resolves named declarations (function, method, class, interface, type, enum, variable); use the edit tool to change a test case.`;
+  }
+  if (!/^[\p{L}_$#][\p{L}\p{N}_$]*(?:\.[\p{L}_$#][\p{L}\p{N}_$]*)*$/u.test(symbol)) {
+    return `${base}: '${symbol}' is not a declaration name. Pass an identifier such as "calculateTotal", or a qualified "ClassName.method" for a nested member.`;
+  }
+  return `${base}. Check the name with codebase-search, or qualify a nested member as "Outer.inner".`;
+}
 async function replaceSymbolInFile(opts, projectRoot) {
   const resolved = path.isAbsolute(opts.file) ? opts.file : path.resolve(projectRoot, opts.file);
   const content = await fs.readFile(resolved, "utf8");
@@ -610,9 +674,7 @@ async function replaceSymbolInFile(opts, projectRoot) {
     result = await mutateTreeSitterSymbol(content, lang, opts.symbol, opts.newBody, opts.target);
   }
   if (!result) {
-    throw new Error(
-      `Symbol '${opts.symbol}' could not be located in AST of '${path.relative(projectRoot, resolved)}'`
-    );
+    throw new Error(notFoundMessage(opts.symbol, path.relative(projectRoot, resolved), content));
   }
   result.file = resolved;
   if (opts.checkInvariants !== false) {
@@ -642,7 +704,6 @@ Pass allowBreakingChanges: true if this breaking change is intended.`
 
 // src/codebase-index/codebase-ast-replace-tool.ts
 import * as path2 from "node:path";
-import { toErrorMessage } from "@wrongstack/core/utils";
 var codebaseAstReplaceTool = {
   name: "codebase-ast-replace",
   category: "Edit",
@@ -651,7 +712,7 @@ var codebaseAstReplaceTool = {
   subjectKey: "file",
   mutating: true,
   capabilities: ["fs.write", "fs.read"],
-  description: "Surgically replace the implementation body or full definition of a function, method, class, or type alias using AST parsing. Guarantees zero string-matching errors, regex escape bugs, or whitespace indentation mismatches. Automatically verifies AST backward-compatibility invariants (no mandatory param addition, no export deletion, no interface breaking).",
+  description: "Surgically replace the implementation body or full definition of a named declaration (function, method, class, interface, type alias, enum, variable) using AST parsing. Test cases (`it(...)`/`test(...)` titles) are call arguments, not declarations \u2014 use the edit tool for those. Guarantees zero string-matching errors, regex escape bugs, or whitespace indentation mismatches. Automatically verifies AST backward-compatibility invariants (no mandatory param addition, no export deletion, no interface breaking).",
   usageHint: 'SURGICAL AST-BASED CODE EDITING:\n\n- Use when updating an existing function, method, or class without worrying about context lines.\n- Pass `symbol: "calculateTotal"` and `newBody: "return items.reduce(...);"`.\n- Default `target: "body"` replaces only the inner `{ ... }` block while preserving the signature and JSDoc.\n- Use `target: "full"` if you also need to modify the function parameters or return type.\n- Setting `allowBreakingChanges: true` bypasses the invariant engine if breaking changes are deliberate.',
   inputSchema: {
     type: "object",
@@ -662,7 +723,7 @@ var codebaseAstReplaceTool = {
       },
       symbol: {
         type: "string",
-        description: "Name of the function, method, class, interface, or variable to replace."
+        description: 'Declaration name (function, method, class, interface, type, enum, variable). Qualify nested members as "ClassName.method" when the bare name is ambiguous. Not a test-case title.'
       },
       newBody: {
         type: "string",
@@ -685,33 +746,27 @@ var codebaseAstReplaceTool = {
     required: ["file", "symbol", "newBody"],
     additionalProperties: false
   },
+  // Failures THROW. A returned `{ status: 'error' }` is a successful tool call
+  // to the executor (is_error:false) — the UI showed "ok", the audit log and
+  // spans recorded success, and only a model reading the payload noticed.
   async execute(input, ctx) {
-    try {
-      const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
-      const file = await safeResolveProjectPath(input.file, ctx);
-      const result = await replaceSymbolInFile({ ...input, file }, projectRoot);
-      return {
-        status: "ok",
-        file: path2.relative(projectRoot, result.file).replace(/\\/g, "/"),
-        symbol: result.symbol,
-        originalRange: result.originalRange,
-        newRange: result.newRange,
-        violations: result.violations?.map((v) => `[${v.ruleId}] ${v.message}`)
-      };
-    } catch (err) {
-      return {
-        status: "error",
-        file: input.file,
-        symbol: input.symbol,
-        error: toErrorMessage(err)
-      };
-    }
+    const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
+    const file = await safeResolveProjectPath(input.file, ctx);
+    const result = await replaceSymbolInFile({ ...input, file }, projectRoot);
+    return {
+      status: "ok",
+      file: path2.relative(projectRoot, result.file).replace(/\\/g, "/"),
+      symbol: result.symbol,
+      originalRange: result.originalRange,
+      newRange: result.newRange,
+      violations: result.violations?.map((v) => `[${v.ruleId}] ${v.message}`)
+    };
   }
 };
 
 // src/codebase-index/codebase-impact-analysis-tool.ts
 import * as path3 from "node:path";
-import { toErrorMessage as toErrorMessage2 } from "@wrongstack/core/utils";
+import { toErrorMessage } from "@wrongstack/core/utils";
 var TEST_FILE_REGEX = /(?:test|spec|mock|fixture|bench)[s]?[/\\._]/i;
 var codebaseImpactAnalysisTool = {
   name: "codebase-impact-analysis",
@@ -742,141 +797,114 @@ var codebaseImpactAnalysisTool = {
     additionalProperties: false
   },
   async execute(input, ctx) {
+    const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
+    const indexDir = codebaseIndexDirOverride(ctx);
+    const limit = 200;
+    const transitive = input.transitive ?? true;
+    let rawSites = [];
+    let symbolFound = true;
+    let indexUnavailable = false;
+    let indexError;
     try {
-      const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
-      const indexDir = codebaseIndexDirOverride(ctx);
-      const limit = 200;
-      const transitive = input.transitive ?? true;
-      let rawSites = [];
-      let symbolFound = true;
-      let indexUnavailable = false;
-      let indexError;
-      try {
-        const serviced = await incomingCallsService({
-          projectRoot,
-          indexDir,
-          symbol: input.symbol,
-          file: input.file,
-          limit,
-          transitive
-        });
-        rawSites = serviced.calls;
-        symbolFound = serviced.symbolFound;
-      } catch (err) {
-        symbolFound = false;
-        indexUnavailable = true;
-        indexError = toErrorMessage2(err);
-      }
-      if (indexUnavailable) {
-        const detail = indexError ? ` (${indexError})` : "";
-        const reason = `Index query failed for '${input.symbol}'. Run codebase-index, then retry.${detail}`;
-        return {
-          status: "error",
-          symbol: input.symbol,
-          riskLevel: "low",
-          summary: reason,
-          totalCallSites: 0,
-          affectedProductionFiles: [],
-          affectedTestFiles: [],
-          callSites: [],
-          recommendedActionPlan: [reason],
-          symbolFound: false,
-          error: reason
-        };
-      }
-      if (!symbolFound) {
-        const reason = `Symbol '${input.symbol}' was not found in the index` + (input.file ? ` for file filter '${input.file}'` : "") + ". Use codebase-search to verify the name.";
-        return {
-          status: "ok",
-          symbol: input.symbol,
-          riskLevel: "low",
-          summary: reason,
-          totalCallSites: 0,
-          affectedProductionFiles: [],
-          affectedTestFiles: [],
-          callSites: [],
-          recommendedActionPlan: [reason],
-          symbolFound: false
-        };
-      }
-      const prodFilesSet = /* @__PURE__ */ new Set();
-      const testFilesSet = /* @__PURE__ */ new Set();
-      const callSites = [];
-      for (const site of rawSites) {
-        const relPath = path3.relative(projectRoot, site.symbol.file).replace(/\\/g, "/");
-        const isTest = TEST_FILE_REGEX.test(relPath);
-        if (isTest) {
-          testFilesSet.add(relPath);
-        } else {
-          prodFilesSet.add(relPath);
-        }
-        callSites.push({
-          file: relPath,
-          line: site.line,
-          callerName: site.symbol.name,
-          callerKind: site.symbol.kind,
-          callType: site.callType,
-          isTest
-        });
-      }
-      const prodFiles = [...prodFilesSet];
-      const testFiles = [...testFilesSet];
-      const totalCallSites = callSites.length;
-      let riskLevel = "low";
-      if (prodFiles.length > 5 || totalCallSites > 10) {
-        riskLevel = "high";
-      } else if (prodFiles.length > 1 || totalCallSites > 3) {
-        riskLevel = "medium";
-      }
-      const recommendedActionPlan = [];
-      if (prodFiles.length > 0) {
-        recommendedActionPlan.push(
-          `Update ${totalCallSites} call site(s) across ${prodFiles.length} production file(s): ${prodFiles.slice(0, 3).join(", ")}${prodFiles.length > 3 ? ` +${prodFiles.length - 3} more` : ""}`
-        );
-      }
-      if (testFiles.length > 0) {
-        recommendedActionPlan.push(
-          `Run targeted tests in ${testFiles.length} test suite(s): ${testFiles.join(", ")}`
-        );
-      } else {
-        recommendedActionPlan.push(
-          `No existing test files directly cover '${input.symbol}'. Consider adding a test suite.`
-        );
-      }
-      const summary = `Blast Radius for '${input.symbol}': ${riskLevel.toUpperCase()} RISK (${totalCallSites} call sites in ${prodFiles.length} prod files, ${testFiles.length} test suites).`;
-      return {
-        status: "ok",
+      const serviced = await incomingCallsService({
+        projectRoot,
+        indexDir,
         symbol: input.symbol,
-        riskLevel,
-        summary,
-        totalCallSites,
-        affectedProductionFiles: prodFiles,
-        affectedTestFiles: testFiles,
-        callSites,
-        recommendedActionPlan,
-        symbolFound: true
-      };
+        file: input.file,
+        limit,
+        transitive
+      });
+      rawSites = serviced.calls;
+      symbolFound = serviced.symbolFound;
     } catch (err) {
+      symbolFound = false;
+      indexUnavailable = true;
+      indexError = toErrorMessage(err);
+    }
+    if (indexUnavailable) {
+      const detail = indexError ? ` (${indexError})` : "";
+      throw new Error(
+        `Index query failed for '${input.symbol}'. Run codebase-index, then retry.${detail}`
+      );
+    }
+    if (!symbolFound) {
+      const reason = `Symbol '${input.symbol}' was not found in the index` + (input.file ? ` for file filter '${input.file}'` : "") + ". Use codebase-search to verify the name.";
       return {
-        status: "error",
+        status: "ok",
         symbol: input.symbol,
         riskLevel: "low",
-        summary: "",
+        summary: reason,
         totalCallSites: 0,
         affectedProductionFiles: [],
         affectedTestFiles: [],
         callSites: [],
-        recommendedActionPlan: [],
-        symbolFound: false,
-        error: toErrorMessage2(err)
+        recommendedActionPlan: [reason],
+        symbolFound: false
       };
     }
+    const prodFilesSet = /* @__PURE__ */ new Set();
+    const testFilesSet = /* @__PURE__ */ new Set();
+    const callSites = [];
+    for (const site of rawSites) {
+      const relPath = path3.relative(projectRoot, site.symbol.file).replace(/\\/g, "/");
+      const isTest = TEST_FILE_REGEX.test(relPath);
+      if (isTest) {
+        testFilesSet.add(relPath);
+      } else {
+        prodFilesSet.add(relPath);
+      }
+      callSites.push({
+        file: relPath,
+        line: site.line,
+        callerName: site.symbol.name,
+        callerKind: site.symbol.kind,
+        callType: site.callType,
+        isTest
+      });
+    }
+    const prodFiles = [...prodFilesSet];
+    const testFiles = [...testFilesSet];
+    const totalCallSites = callSites.length;
+    let riskLevel = "low";
+    if (prodFiles.length > 5 || totalCallSites > 10) {
+      riskLevel = "high";
+    } else if (prodFiles.length > 1 || totalCallSites > 3) {
+      riskLevel = "medium";
+    }
+    const recommendedActionPlan = [];
+    if (prodFiles.length > 0) {
+      recommendedActionPlan.push(
+        `Update ${totalCallSites} call site(s) across ${prodFiles.length} production file(s): ${prodFiles.slice(0, 3).join(", ")}${prodFiles.length > 3 ? ` +${prodFiles.length - 3} more` : ""}`
+      );
+    }
+    if (testFiles.length > 0) {
+      recommendedActionPlan.push(
+        `Run targeted tests in ${testFiles.length} test suite(s): ${testFiles.join(", ")}`
+      );
+    } else {
+      recommendedActionPlan.push(
+        `No existing test files directly cover '${input.symbol}'. Consider adding a test suite.`
+      );
+    }
+    const summary = `Blast Radius for '${input.symbol}': ${riskLevel.toUpperCase()} RISK (${totalCallSites} call sites in ${prodFiles.length} prod files, ${testFiles.length} test suites).`;
+    return {
+      status: "ok",
+      symbol: input.symbol,
+      riskLevel,
+      summary,
+      totalCallSites,
+      affectedProductionFiles: prodFiles,
+      affectedTestFiles: testFiles,
+      callSites,
+      recommendedActionPlan,
+      symbolFound: true
+    };
   }
 };
 
 // src/codebase-index/codebase-incoming-calls-tool.ts
 import { ToolValidationError } from "@wrongstack/core/types";
-import { toErrorMessage as toErrorMessage3 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage2 } from "@wrongstack/core/utils";
 var codebaseIncomingCallsTool = {
   name: "codebase-incoming-calls",
   category: "Project",
@@ -923,12 +951,7 @@ var codebaseIncomingCallsTool = {
     if (state.lastError) {
       const circuit = state.circuit;
       const retryHint = circuit.state === "open" ? `Indexing is paused (circuit open, retry in ${Math.ceil(circuit.cooldownRemainingMs / 1e3)}s).` : "Try /codebase-reindex.";
-      return {
-        symbol: input.symbol,
-        calls: [],
-        total: 0,
-        indexStatus: `Index build failed: ${state.lastError}. ${retryHint}`
-      };
+      throw new Error(`Index build failed: ${state.lastError}. ${retryHint}`);
     }
     const limit = Math.max(1, Math.min(Math.trunc(input.limit ?? 50), 200));
     const transitive = input.transitive === true;
@@ -945,19 +968,15 @@ var codebaseIncomingCallsTool = {
       });
     } catch (err) {
       if (err.name === "IndexRefreshInProgressError") {
-        return {
-          symbol: input.symbol,
-          calls: [],
-          total: 0,
-          indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this symbol has no cached answer yet \u2014 retry after the completed generation is published.`
-        };
+        throw new Error(
+          `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this symbol has no cached answer yet \u2014 retry after the completed generation is published.`,
+          { cause: err }
+        );
       }
-      return {
-        symbol: input.symbol,
-        calls: [],
-        total: 0,
-        indexStatus: `Index query failed: ${toErrorMessage3(err)}. Fall back to grep for this lookup.`
-      };
+      throw new Error(
+        `Index query failed: ${toErrorMessage2(err)}. Fall back to grep for this lookup.`,
+        { cause: err }
+      );
     }
     const { calls, symbolFound, ambiguous, totalMatches, stale } = serviced;
     if (!symbolFound) {
@@ -969,16 +988,17 @@ var codebaseIncomingCallsTool = {
             indexDir: codebaseIndexDirOverride(ctx)
           });
           hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
-        } catch {
+        } catch (err) {
+          throw new Error(
+            `Symbol "${input.symbol}" was not found and the persisted index could not be verified: ${toErrorMessage2(err)}. Try /codebase-reindex or fall back to grep.`,
+            { cause: err }
+          );
         }
       }
       if (!hasPersistedIndex) {
-        return {
-          symbol: input.symbol,
-          calls: [],
-          total: 0,
-          indexStatus: "No persisted index data found. Run codebase-index to build it."
-        };
+        throw new Error(
+          "No persisted index data found. Run codebase-index to build it, then retry."
+        );
       }
       return {
         symbol: input.symbol,
@@ -1018,7 +1038,7 @@ import { ToolValidationError as ToolValidationError3 } from "@wrongstack/core/ty
 
 // src/codebase-index/codebase-search-tool.ts
 import { ToolValidationError as ToolValidationError2 } from "@wrongstack/core/types";
-import { toErrorMessage as toErrorMessage4 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage3 } from "@wrongstack/core/utils";
 var INDEXABLE_LANG_IDS = [
   ...new Set(Object.values(EXT_TO_LANG))
 ].sort();
@@ -1112,12 +1132,7 @@ var codebaseSearchTool = {
     if (state.lastError) {
       const circuit = state.circuit;
       const retryHint = circuit.state === "open" ? `Indexing is paused (circuit open, retry in ${Math.ceil(circuit.cooldownRemainingMs / 1e3)}s); the user can run /codebase-reindex to retry now.` : "Try /codebase-reindex.";
-      return {
-        results: [],
-        total: 0,
-        query: input.query,
-        indexStatus: `Index build failed: ${state.lastError}. ${retryHint}`
-      };
+      throw new Error(`Index build failed: ${state.lastError}. ${retryHint}`);
     }
     const limit = Math.max(1, Math.min(Math.trunc(input.limit ?? 20), 100));
     let searched;
@@ -1140,19 +1155,15 @@ var codebaseSearchTool = {
         signal.throwIfAborted();
       }
       if (err.name === "IndexRefreshInProgressError") {
-        return {
-          results: [],
-          total: 0,
-          query: input.query,
-          indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this query has no cached answer yet \u2014 retry after the completed generation is published.`
-        };
+        throw new Error(
+          `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this query has no cached answer yet \u2014 retry after the completed generation is published.`,
+          { cause: err }
+        );
       }
-      return {
-        results: [],
-        total: 0,
-        query: input.query,
-        indexStatus: `Index query failed: ${toErrorMessage4(err)}. Fall back to grep/glob for this lookup.`
-      };
+      throw new Error(
+        `Index query failed: ${toErrorMessage3(err)}. Fall back to grep/glob for this lookup.`,
+        { cause: err }
+      );
     }
     const { results, total, stale, indexSummary } = searched;
     let hasPersistedIndex = state.ready || total > 0;
@@ -1166,9 +1177,19 @@ var codebaseSearchTool = {
           { signal }
         );
         hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
-      } catch {
+      } catch (err) {
+        if (signal?.aborted) signal.throwIfAborted();
+        throw new Error(
+          `codebase-search found no results and could not verify the persisted index: ${toErrorMessage3(err)}. Try /codebase-reindex or fall back to grep/glob.`,
+          { cause: err }
+        );
       }
     }
+    if (!hasPersistedIndex) {
+      throw new Error(
+        "No persisted index data found. Run codebase-index to build it, then retry codebase-search."
+      );
+    }
     return {
       results,
       total,
@@ -1176,8 +1197,7 @@ var codebaseSearchTool = {
       ...stale ? {
         stale: true,
         indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); results served from the previous generation \u2014 symbols from files being indexed may lag.`
-      } : {},
-      ...hasPersistedIndex ? {} : { indexStatus: "No persisted index data found. Run codebase-index to build it." }
+      } : {}
     };
   }
 };
@@ -1235,28 +1255,18 @@ var codebaseIndexTool = {
       }
     }
     if (isIndexing()) {
-      return {
-        filesIndexed: 0,
-        symbolsIndexed: 0,
-        langStats: {},
-        durationMs: 0,
-        errors: [],
-        note: "A full index is already in progress. Retry codebase-index after it completes (check codebase-stats)."
-      };
+      throw new Error(
+        "A full index is already in progress. Retry codebase-index after it completes (check codebase-stats)."
+      );
     }
     if (input.force) {
       indexCircuitBreaker.reset();
     } else {
       const circuit = indexCircuitBreaker.snapshot();
       if (circuit.state === "open" && circuit.cooldownRemainingMs > 0) {
-        return {
-          filesIndexed: 0,
-          symbolsIndexed: 0,
-          langStats: {},
-          durationMs: 0,
-          errors: [],
-          note: `Codebase indexing is paused after repeated failures (last: ${circuit.lastFailure ?? "unknown"}). Auto-retry possible in ${Math.max(1, Math.ceil(circuit.cooldownRemainingMs / 1e3))}s; use force: true or run /codebase-reindex to retry immediately.`
-        };
+        throw new Error(
+          `Codebase indexing is paused after repeated failures (last: ${circuit.lastFailure ?? "unknown"}). Auto-retry possible in ${Math.max(1, Math.ceil(circuit.cooldownRemainingMs / 1e3))}s; use force: true or run /codebase-reindex to retry immediately.`
+        );
       }
     }
     const result = await runStartupIndex({
@@ -1279,7 +1289,7 @@ var codebaseIndexTool = {
 
 // src/codebase-index/codebase-invariant-check-tool.ts
 import * as fs2 from "node:fs/promises";
-import { toErrorMessage as toErrorMessage5 } from "@wrongstack/core/utils";
+import { ToolValidationError as ToolValidationError4 } from "@wrongstack/core/types";
 var codebaseInvariantCheckTool = {
   name: "codebase-invariant-check",
   category: "Inspect",
@@ -1313,52 +1323,44 @@ var codebaseInvariantCheckTool = {
     required: ["modifiedCode"],
     additionalProperties: false
   },
+  // Failures THROW. The old catch returned `valid: false` — a successful call
+  // (is_error:false) whose payload was indistinguishable from "this change
+  // breaks compatibility", so an unreadable file read as a real violation.
   async execute(input, ctx) {
-    try {
-      let originalCode = input.originalCode;
-      let lang = input.lang;
-      if (!originalCode && input.file) {
-        const resolved = await safeResolveProjectPath(input.file, ctx);
-        originalCode = await fs2.readFile(resolved, "utf8");
-        if (!lang) {
-          lang = detectLang(resolved) ?? "ts";
-        }
-      }
-      if (originalCode === void 0) {
-        return {
-          valid: false,
-          violations: [],
-          summary: "Neither originalCode nor file was provided for invariant comparison.",
-          error: "Missing originalCode or file path."
-        };
+    let originalCode = input.originalCode;
+    let lang = input.lang;
+    if (!originalCode && input.file) {
+      const resolved = await safeResolveProjectPath(input.file, ctx);
+      originalCode = await fs2.readFile(resolved, "utf8");
+      if (!lang) {
+        lang = detectLang(resolved) ?? "ts";
       }
-      const res = await polyglotInvariantEngine.evaluate({
-        originalCode,
-        modifiedCode: input.modifiedCode,
-        lang,
-        filePath: input.file
+    }
+    if (originalCode === void 0) {
+      throw new ToolValidationError4({
+        message: "Neither originalCode nor file was provided for invariant comparison.",
+        field: "originalCode"
       });
-      const summary = res.valid ? "AST Invariants PASSED. Mutation is 100% backward compatible." : `AST Invariants FAILED with ${res.violations.length} violation(s):
-` + res.violations.map((v) => ` - [${v.ruleId}] ${v.symbolName}: ${v.message}`).join("\n");
-      return {
-        valid: res.valid,
-        violations: res.violations,
-        summary
-      };
-    } catch (err) {
-      return {
-        valid: false,
-        violations: [],
-        summary: `Error during invariant evaluation: ${toErrorMessage5(err)}`,
-        error: toErrorMessage5(err)
-      };
     }
+    const res = await polyglotInvariantEngine.evaluate({
+      originalCode,
+      modifiedCode: input.modifiedCode,
+      lang,
+      filePath: input.file
+    });
+    const summary = res.valid ? "AST Invariants PASSED. Mutation is 100% backward compatible." : `AST Invariants FAILED with ${res.violations.length} violation(s):
+` + res.violations.map((v) => ` - [${v.ruleId}] ${v.symbolName}: ${v.message}`).join("\n");
+    return {
+      valid: res.valid,
+      violations: res.violations,
+      summary
+    };
   }
 };
 
 // src/codebase-index/codebase-outgoing-calls-tool.ts
-import { ToolValidationError as ToolValidationError4 } from "@wrongstack/core/types";
-import { toErrorMessage as toErrorMessage6 } from "@wrongstack/core/utils";
+import { ToolValidationError as ToolValidationError5 } from "@wrongstack/core/types";
+import { toErrorMessage as toErrorMessage4 } from "@wrongstack/core/utils";
 var codebaseOutgoingCallsTool = {
   name: "codebase-outgoing-calls",
   category: "Project",
@@ -1396,7 +1398,7 @@ var codebaseOutgoingCallsTool = {
   },
   async execute(input, ctx) {
     if (!input?.symbol || typeof input.symbol !== "string" || !input.symbol.trim()) {
-      throw new ToolValidationError4({
+      throw new ToolValidationError5({
         message: "codebase-outgoing-calls: symbol is required and cannot be empty",
         field: "symbol"
       });
@@ -1405,12 +1407,7 @@ var codebaseOutgoingCallsTool = {
     if (state.lastError) {
       const circuit = state.circuit;
       const retryHint = circuit.state === "open" ? `Indexing is paused (circuit open, retry in ${Math.ceil(circuit.cooldownRemainingMs / 1e3)}s).` : "Try /codebase-reindex.";
-      return {
-        symbol: input.symbol,
-        calls: [],
-        total: 0,
-        indexStatus: `Index build failed: ${state.lastError}. ${retryHint}`
-      };
+      throw new Error(`Index build failed: ${state.lastError}. ${retryHint}`);
     }
     const limit = Math.max(1, Math.min(Math.trunc(input.limit ?? 50), 200));
     const transitive = input.transitive === true;
@@ -1427,19 +1424,15 @@ var codebaseOutgoingCallsTool = {
       });
     } catch (err) {
       if (err.name === "IndexRefreshInProgressError") {
-        return {
-          symbol: input.symbol,
-          calls: [],
-          total: 0,
-          indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this symbol has no cached answer yet \u2014 retry after the completed generation is published.`
-        };
+        throw new Error(
+          `Index refresh in progress (${state.currentFile}/${state.totalFiles} files); this symbol has no cached answer yet \u2014 retry after the completed generation is published.`,
+          { cause: err }
+        );
       }
-      return {
-        symbol: input.symbol,
-        calls: [],
-        total: 0,
-        indexStatus: `Index query failed: ${toErrorMessage6(err)}. Fall back to grep for this lookup.`
-      };
+      throw new Error(
+        `Index query failed: ${toErrorMessage4(err)}. Fall back to grep for this lookup.`,
+        { cause: err }
+      );
     }
     const { calls, symbolFound, unresolvedCount, totalMatches, stale } = serviced;
     if (!symbolFound) {
@@ -1451,16 +1444,17 @@ var codebaseOutgoingCallsTool = {
             indexDir: codebaseIndexDirOverride(ctx)
           });
           hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
-        } catch {
+        } catch (err) {
+          throw new Error(
+            `Symbol "${input.symbol}" was not found and the persisted index could not be verified: ${toErrorMessage4(err)}. Try /codebase-reindex or fall back to grep.`,
+            { cause: err }
+          );
         }
       }
       if (!hasPersistedIndex) {
-        return {
-          symbol: input.symbol,
-          calls: [],
-          total: 0,
-          indexStatus: "No persisted index data found. Run codebase-index to build it."
-        };
+        throw new Error(
+          "No persisted index data found. Run codebase-index to build it, then retry."
+        );
       }
       return {
         symbol: input.symbol,
@@ -1495,9 +1489,6 @@ var codebaseOutgoingCallsTool = {
   }
 };
 
-// src/codebase-index/codebase-repo-map-tool.ts
-import { toErrorMessage as toErrorMessage7 } from "@wrongstack/core/utils";
-
 // src/codebase-index/repo-map.ts
 import { existsSync } from "node:fs";
 import * as path6 from "node:path";
@@ -2171,38 +2162,28 @@ var codebaseRepoMapTool = {
     },
     additionalProperties: false
   },
+  // Failures THROW so the executor marks the call is_error (a returned
+  // `status: 'error'` payload was recorded as a successful call).
   async execute(input, ctx) {
-    try {
-      const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
-      const result = await generateRepoMap({
-        projectRoot,
-        maxTokens: input.maxTokens,
-        focusFiles: input.focusFiles,
-        // Honour a caller-supplied index location so the map reads the same
-        // index the other codebase-* tools do.
-        indexDir: codebaseIndexDirOverride(ctx)
-      });
-      return {
-        status: "ok",
-        ...result
-      };
-    } catch (err) {
-      return {
-        status: "error",
-        map: "",
-        filesCount: 0,
-        totalFilesScanned: 0,
-        estimatedTokens: 0,
-        rankedFiles: [],
-        error: toErrorMessage7(err)
-      };
-    }
+    const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
+    const result = await generateRepoMap({
+      projectRoot,
+      maxTokens: input.maxTokens,
+      focusFiles: input.focusFiles,
+      // Honour a caller-supplied index location so the map reads the same
+      // index the other codebase-* tools do.
+      indexDir: codebaseIndexDirOverride(ctx)
+    });
+    return {
+      status: "ok",
+      ...result
+    };
   }
 };
 
 // src/codebase-index/codebase-skeleton-tool.ts
 import * as fs6 from "node:fs/promises";
-import { toErrorMessage as toErrorMessage8 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage5 } from "@wrongstack/core/utils";
 var codebaseSkeletonTool = {
   name: "codebase-skeleton",
   category: "Project",
@@ -2249,7 +2230,7 @@ var codebaseSkeletonTool = {
       stat3 = await fs6.stat(targetPath);
     } catch (err) {
       throw new Error(
-        `codebase-skeleton: file or directory not found at "${input.path}": ${toErrorMessage8(err)}`
+        `codebase-skeleton: file or directory not found at "${input.path}": ${toErrorMessage5(err)}`
       );
     }
     const options = {
@@ -2298,7 +2279,7 @@ var codebaseSkeletonTool = {
 };
 
 // src/codebase-index/codebase-stats-tool.ts
-import { toErrorMessage as toErrorMessage9 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage6 } from "@wrongstack/core/utils";
 var codebaseStatsTool = {
   name: "codebase-stats",
   category: "Project",
@@ -2350,18 +2331,10 @@ var codebaseStatsTool = {
       if (signal?.aborted) {
         signal.throwIfAborted();
       }
-      return {
-        totalSymbols: 0,
-        totalFiles: 0,
-        byLang: {},
-        byKind: {},
-        lastIndexed: null,
-        sizeBytes: 0,
-        indexPath,
-        version: SCHEMA_VERSION,
-        statsAvailable: false,
-        indexStatus: err instanceof IndexTimeoutError || err?.name === "IndexTimeoutError" ? "Index statistics timed out. Do not repeatedly retry stats; try codebase-search directly or use the appropriate grep/glob/tree fallback." : `Index statistics query failed: ${toErrorMessage9(err)}. The index may be corrupted or inaccessible; try /codebase-reindex.`
-      };
+      throw new Error(
+        err instanceof IndexTimeoutError || err?.name === "IndexTimeoutError" ? "Index statistics timed out. Do not repeatedly retry stats; try codebase-search directly or use the appropriate grep/glob/tree fallback." : `Index statistics query failed: ${toErrorMessage6(err)}. The index may be corrupted or inaccessible; try /codebase-reindex.`,
+        { cause: err }
+      );
     }
     const circuit = idxState.circuit;
     const hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
@@ -2390,7 +2363,7 @@ var codebaseStatsTool = {
 // src/codebase-index/codebase-targeted-test-tool.ts
 import * as fs7 from "node:fs/promises";
 import * as path7 from "node:path";
-import { toErrorMessage as toErrorMessage10 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage7 } from "@wrongstack/core/utils";
 var TEST_FILE_REGEX2 = /(?:test|spec|mock|fixture|bench)[s]?[/\\._]/i;
 async function findConventionTestFiles(projectRoot, sourceRelPath) {
   const parsed = path7.parse(sourceRelPath);
@@ -2535,16 +2508,10 @@ ${streamRes.stderr}`.trim(),
         output: combinedOutput
       };
     } catch (err) {
-      return {
-        status: "error",
-        discoveredSuites: [...suitesSet],
-        testsRun: 0,
-        passed: 0,
-        failed: 0,
-        durationMs: 0,
-        output: "",
-        error: toErrorMessage10(err)
-      };
+      const suites = suitesSet.size > 0 ? ` (discovered suites: ${[...suitesSet].join(", ")})` : "";
+      throw new Error(`Targeted test run failed to execute: ${toErrorMessage7(err)}${suites}`, {
+        cause: err
+      });
     }
   }
 };
@@ -2552,6 +2519,7 @@ ${streamRes.stderr}`.trim(),
 // src/codebase-index/dead-code-scan.ts
 import * as fs8 from "node:fs";
 import * as path8 from "node:path";
+import { ToolValidationError as ToolValidationError6 } from "@wrongstack/core/types";
 var deadCodeScanTool = {
   name: "dead-code-scan",
   category: "Project",
@@ -2582,12 +2550,36 @@ var deadCodeScanTool = {
   },
   async execute(input, ctx, _execOpts) {
     const startMs = Date.now();
-    const projectRoot = input.projectRoot ?? ctx.projectRoot ?? ctx.cwd ?? process.cwd();
+    const projectRoot = input.projectRoot !== void 0 ? await safeResolveProjectPath(input.projectRoot, ctx) : ctx.projectRoot ?? ctx.cwd ?? process.cwd();
     const indexDir = input.indexDir ?? codebaseIndexDirOverride(ctx) ?? void 0;
-    const result = runDeadCodeScan(projectRoot, {
-      indexDir,
-      userEntryPoints: input.entryPoints
-    });
+    let userEntryPoints;
+    if (input.entryPoints) {
+      userEntryPoints = [];
+      for (const ep of input.entryPoints) {
+        const resolved = await safeResolveProjectPath(
+          path8.isAbsolute(ep) ? ep : path8.resolve(projectRoot, ep),
+          ctx
+        );
+        if (!fs8.existsSync(resolved)) {
+          throw new ToolValidationError6({
+            message: `dead-code-scan: entry point not found: "${ep}"`,
+            field: "entryPoints"
+          });
+        }
+        userEntryPoints.push(resolved);
+      }
+    }
+    const result = runDeadCodeScan(projectRoot, { indexDir, userEntryPoints });
+    if (result.stats.totalSymbols === 0) {
+      throw new Error(
+        `dead-code-scan: the codebase index for "${projectRoot}" has no symbols. Run codebase-index, then retry.`
+      );
+    }
+    if (result.stats.alive === 0) {
+      throw new Error(
+        `dead-code-scan: no indexed entry point was found under "${projectRoot}" (package.json main/bin/exports/types or src/index.ts), so reachability cannot be computed. Pass \`entryPoints\`.`
+      );
+    }
     return {
       ...result,
       stats: { ...result.stats, durationMs: Date.now() - startMs }
@@ -3568,8 +3560,8 @@ async function buildProjectAtlasBrief(projectRoot, opts = {}) {
 }
 
 // src/codebase-index/codebase-context-tool.ts
-import { ToolValidationError as ToolValidationError5 } from "@wrongstack/core/types";
-import { toErrorMessage as toErrorMessage11 } from "@wrongstack/core/utils";
+import { ToolValidationError as ToolValidationError7 } from "@wrongstack/core/types";
+import { toErrorMessage as toErrorMessage8 } from "@wrongstack/core/utils";
 var TOOL_TIMEOUT_MS = 35e3;
 var VECTOR_HIT_LIMIT = 12;
 var VECTOR_MIN_SCORE = 0.25;
@@ -3604,7 +3596,7 @@ var codebaseContextTool = {
   capabilities: ["fs.read"],
   timeoutMs: TOOL_TIMEOUT_MS,
   description: "Find the files and declarations a task touches, from a plain-language description. Combines ranked symbol search with a personalised walk over the reference graph, so results include not just what matched by name but what those matches are structurally attached to \u2014 the interface behind a handler, the store behind a route, the module every match imports. Returns ranked files with the relevant declarations, their signatures and line numbers.",
-  usageHint: 'START HERE FOR ANY TASK THAT SPANS MORE THAN ONE FILE:\n\n- Describe the task, not a symbol name: `codebase-context({ query: "retry a provider request after a 429" })`.\n- Prefer this over `codebase-search` when you do not already know the symbol you want; prefer `codebase-search` when you do.\n- Use the returned `file` + `line` to `read` only the parts you need, instead of reading whole files.\n- `matched: true` means the file matched the query directly; the rest were reached through the graph.\n- `relevance` is relative to this answer only (top result is always 1.0) \u2014 never compare it across queries.\n- Narrow with `pathPrefix: "packages/core"` when you already know the area.\n- `indexStatus: "no-index"` means run `/codebase-reindex` first.',
+  usageHint: 'START HERE FOR ANY TASK THAT SPANS MORE THAN ONE FILE:\n\n- Describe the task, not a symbol name: `codebase-context({ query: "retry a provider request after a 429" })`.\n- Prefer this over `codebase-search` when you do not already know the symbol you want; prefer `codebase-search` when you do.\n- Use the returned `file` + `line` to `read` only the parts you need, instead of reading whole files.\n- `matched: true` means the file matched the query directly; the rest were reached through the graph.\n- `relevance` is relative to this answer only (top result is always 1.0) \u2014 never compare it across queries.\n- Narrow with `pathPrefix: "packages/core"` when you already know the area.\n- If the call fails because no index exists, run `/codebase-reindex` (or `codebase-index`) first.',
   inputSchema: {
     type: "object",
     properties: {
@@ -3631,19 +3623,20 @@ var codebaseContextTool = {
   async execute(input, ctx, execOpts) {
     const query = typeof input.query === "string" ? input.query.trim() : "";
     if (query.length === 0) {
-      throw new ToolValidationError5({
+      throw new ToolValidationError7({
         message: "codebase-context requires a non-empty `query`.",
         field: "query"
       });
     }
     const signal = execOpts?.signal ?? ctx.signal;
     const projectRoot = ctx.projectRoot ?? ctx.cwd ?? process.cwd();
+    signal?.throwIfAborted();
+    const indexDir = codebaseIndexDirOverride(ctx);
+    let result;
     try {
-      signal?.throwIfAborted();
-      const indexDir = codebaseIndexDirOverride(ctx);
       const vectorFiles = await semanticSeeds(projectRoot, indexDir, query);
       signal?.throwIfAborted();
-      const result = await codebaseContext({
+      result = await codebaseContext({
         projectRoot,
         indexDir,
         query,
@@ -3652,28 +3645,28 @@ var codebaseContextTool = {
         symbolsPerFile: input.symbolsPerFile,
         pathPrefix: input.pathPrefix
       });
-      return {
-        query: result.query,
-        entries: result.entries,
-        seedCount: result.seedCount,
-        semanticSeedCount: result.semanticSeedCount,
-        totalCandidates: result.totalCandidates,
-        indexStatus: result.indexStatus,
-        ..."stale" in result && result.stale === true ? { stale: true } : {}
-      };
     } catch (err) {
       if (signal?.aborted) throw err;
       const refreshing = err instanceof Error && err.name === "IndexRefreshInProgressError";
-      return {
-        query,
-        entries: [],
-        seedCount: 0,
-        semanticSeedCount: 0,
-        totalCandidates: 0,
-        indexStatus: "error",
-        error: refreshing ? "Codebase index is refreshing; retry once the current generation is published." : toErrorMessage11(err)
-      };
+      throw new Error(
+        refreshing ? "Codebase index is refreshing; retry once the current generation is published." : `codebase-context lookup failed: ${toErrorMessage8(err)}`,
+        { cause: err }
+      );
+    }
+    if (result.indexStatus === "no-index") {
+      throw new Error(
+        "No codebase index data found. Run /codebase-reindex (or codebase-index), then retry codebase-context."
+      );
     }
+    return {
+      query: result.query,
+      entries: result.entries,
+      seedCount: result.seedCount,
+      semanticSeedCount: result.semanticSeedCount,
+      totalCandidates: result.totalCandidates,
+      indexStatus: result.indexStatus,
+      ..."stale" in result && result.stale === true ? { stale: true } : {}
+    };
   }
 };
 
@@ -4047,4 +4040,4 @@ export {
   embedFiles,
   embedProjectFiles
 };
-//# sourceMappingURL=chunk-JB5JLY7V.js.map
+//# sourceMappingURL=chunk-ENE22I5W.js.map