@wrongstack/tools 0.73.1 → 0.82.6

package/dist/pack.js

@@ -19,6 +19,12 @@ var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require
   if (typeof require !== "undefined") return require.apply(this, arguments);
   throw Error('Dynamic require of "' + x + '" is not supported');
 });
+function expectDefined(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 async function* spawnStream(opts) {
   const max = opts.maxBytes ?? 2e5;
   const flushAt = opts.flushBytes ?? 4 * 1024;
@@ -70,7 +76,7 @@ async function* spawnStream(opts) {
         waiter = resolve7;
       });
     }
-    const chunk = queue.shift();
+    const chunk = expectDefined(queue.shift());
     if (chunk.kind === "close") {
       if (!spawnFailed) exitCode = chunk.code ?? 0;
       break;
@@ -97,6 +103,12 @@ async function* spawnStream(opts) {
     error
   };
 }
+function expectDefined2(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 async function detectPackageManager(cwd) {
   const { stat: stat10 } = await import('node:fs/promises');
   try {
@@ -186,9 +198,9 @@ function collapseConsecutiveDuplicates(text, minRun = REPEAT_RUN_THRESHOLD) {
     while (j < lines.length && lines[j] === lines[i]) j++;
     const run = j - i;
     if (run >= minRun) {
-      out.push(lines[i], `\u2026 \u27E8repeated ${run}\xD7\u27E9`);
+      out.push(expectDefined2(lines[i]), `\u2026 \u27E8repeated ${run}\xD7\u27E9`);
     } else {
-      for (let k = i; k < j; k++) out.push(lines[k]);
+      for (let k = i; k < j; k++) out.push(expectDefined2(lines[k]));
     }
     i = j;
   }
@@ -265,7 +277,9 @@ var auditTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of auditTool.executeStream(input, ctx, opts)) {
+    const executeStream = auditTool.executeStream;
+    if (!executeStream) throw new Error("auditTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("audit: stream ended without final event");
@@ -482,6 +496,12 @@ var CircuitBreaker = class {
 };
 
 // src/process-registry.ts
+function expectDefined3(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var SENSITIVE_FLAG_PATTERNS = [
   // --flag=value  or  --flag "value"  (value captured up to next space or comma)
   /--(?:token|password|passwd|pwd|secret|api[-_]?key|api[-_]?secret|auth|credential|private[-_]?key|access[-_]?key|github[-_]?token|gh[-_]?token|bearer|jwt|oauth|pin|pincode|passphrase|access[-_]?token)(?:[=\s,][^\s]*)?/gi,
@@ -502,7 +522,7 @@ function redactCommand(cmd) {
       const sp = match.search(/\s/);
       const delim = eq !== -1 ? "=" : sp !== -1 ? match[sp] : null;
       if (delim !== null) {
-        const flag = match.slice(0, match.indexOf(delim) + 1);
+        const flag = match.slice(0, match.indexOf(expectDefined3(delim)) + 1);
         return `${flag}[REDACTED]`;
       }
       const flagEnd = match.match(/^--?[a-zA-Z][a-zA-Z0-9_-]*/)?.[0] ?? match;
@@ -721,7 +741,9 @@ var bashTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of bashTool.executeStream(input, ctx, opts)) {
+    const executeStream = bashTool.executeStream;
+    if (!executeStream) throw new Error("bashTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("bash: stream ended without final event");
@@ -1087,6 +1109,12 @@ function lspKindToInternalKind(k) {
 }
 
 // src/codebase-index/writer.ts
+function expectDefined4(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var DB_FILE = "index.db";
 function resolveIndexDir(projectRoot, override) {
   return override ?? resolveWstackPaths({ projectRoot }).projectCodebaseIndex;
@@ -1231,7 +1259,7 @@ var IndexStore = class {
       "SELECT file, lang, mtime_ms, symbol_count, last_indexed FROM files WHERE file = ?"
     ).all(file);
     if (!rows.length) return null;
-    const r = rows[0];
+    const r = expectDefined4(rows[0]);
     return { file: r.file, lang: r.lang, mtimeMs: r.mtime_ms, symbolCount: r.symbol_count, lastIndexed: r.last_indexed };
   }
   getAllFileMetas() {
@@ -1313,7 +1341,7 @@ var IndexStore = class {
     const lastRows = this.db.prepare(
       "SELECT value FROM metadata WHERE key = 'last_indexed'"
     ).all();
-    const lastIndexed = lastRows.length ? Number(lastRows[0].value) : null;
+    const lastIndexed = lastRows.length ? Number(lastRows[0]?.value) : null;
     const totalRows = this.db.prepare("SELECT COUNT(*) FROM symbols").all();
     const totalSymbols = totalRows[0] ? Number(totalRows[0]["COUNT(*)"]) : 0;
     const fileRows = this.db.prepare("SELECT COUNT(*) FROM files").all();
@@ -1388,8 +1416,9 @@ var IndexStore = class {
     let resolved = 0;
     for (const row of unresolved) {
       const target = this.db.prepare("SELECT id FROM symbols WHERE name = ? LIMIT 1").all(row.to_name);
-      if (target.length) {
-        this.db.prepare("UPDATE refs SET to_id = ? WHERE id = ?").run(target[0].id, row.id);
+      const first = target[0];
+      if (first) {
+        this.db.prepare("UPDATE refs SET to_id = ? WHERE id = ?").run(first.id, row.id);
         resolved++;
       }
     }
@@ -2143,6 +2172,12 @@ function syncPyParse(filePath, lang) {
     return { file: filePath, lang, symbols: [], mtimeMs: Date.now() };
   }
 }
+function expectDefined5(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 function parseSymbols4(opts) {
   const { file, content, lang } = opts;
   const nativeAvailable = checkNativeParser();
@@ -2224,14 +2259,14 @@ function regexParse(opts) {
   const lines = content.split("\n");
   const lineOffsets = [0];
   for (let i = 0; i < lines.length; i++) {
-    lineOffsets.push(lineOffsets[i] + lines[i].length + 1);
+    lineOffsets.push((lineOffsets[i] ?? 0) + (lines[i]?.length ?? 0) + 1);
   }
   function lineFromOffset(offset) {
     let lo = 0;
     let hi = lineOffsets.length - 1;
     while (lo < hi) {
       const mid = lo + hi + 1 >>> 1;
-      if (lineOffsets[mid] <= offset) lo = mid;
+      if (expectDefined5(lineOffsets[mid]) <= offset) lo = mid;
       else hi = mid - 1;
     }
     return lo + 1;
@@ -2243,8 +2278,8 @@ function regexParse(opts) {
   for (const pattern of RS_PATTERNS) {
     pattern.regex.lastIndex = 0;
     for (let match = pattern.regex.exec(content); match !== null; match = pattern.regex.exec(content)) {
-      const name = match[1];
-      const offset = match.index;
+      const name = expectDefined5(match[1]);
+      const offset = match.index ?? 0;
       const line = lineFromOffset(offset);
       const col = offset - (lineOffsets[line - 1] ?? 0);
       const lineIdx = line - 1;
@@ -2273,6 +2308,12 @@ function regexParse(opts) {
   });
   return { file, lang, symbols: deduped, mtimeMs: Date.now() };
 }
+function expectDefined6(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 function parseSymbols5(opts) {
   const { file, content, lang } = opts;
   try {
@@ -2292,21 +2333,21 @@ function regexParse2(opts) {
   const lines = content.split("\n");
   const lineOffsets = [0];
   for (let i = 0; i < lines.length; i++) {
-    lineOffsets.push(lineOffsets[i] + lines[i].length + 1);
+    lineOffsets.push((lineOffsets[i] ?? 0) + (lines[i]?.length ?? 0) + 1);
   }
   function lineFromOffset(offset) {
     let lo = 0;
     let hi = lineOffsets.length - 1;
     while (lo < hi) {
       const mid = lo + hi + 1 >>> 1;
-      if (lineOffsets[mid] <= offset) lo = mid;
+      if (expectDefined6(lineOffsets[mid]) <= offset) lo = mid;
       else hi = mid - 1;
     }
     return lo + 1;
   }
   const rootMatch = content.match(/^\s*\{/m);
   if (rootMatch) {
-    const offset = rootMatch.index;
+    const offset = expectDefined6(rootMatch.index);
     const line = lineFromOffset(offset);
     symbols.push(
       makeSymbol({
@@ -2322,8 +2363,8 @@ function regexParse2(opts) {
   }
   const topLevelKeyRegex = /^\s*"([^"]+)"\s*:/gm;
   for (let match = topLevelKeyRegex.exec(content); match !== null; match = topLevelKeyRegex.exec(content)) {
-    const key = match[1];
-    const offset = match.index;
+    const key = expectDefined6(match[1]);
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
     let kind = "property";
@@ -2369,7 +2410,7 @@ function regexParse2(opts) {
   const defsRegex = /"\$defs"\s*:|"\$defs"\s*:/g;
   const defsMatch = defsRegex.exec(content);
   if (defsMatch !== null) {
-    const offset = defsMatch.index;
+    const offset = expectDefined6(defsMatch.index);
     const line = lineFromOffset(offset);
     symbols.push(
       makeSymbol({
@@ -2392,9 +2433,9 @@ function regexParse2(opts) {
   for (const pat of defsPatterns) {
     pat.lastIndex = 0;
     for (let match = pat.exec(content); match !== null; match = pat.exec(content)) {
-      const offset = match.index;
+      const offset = match.index ?? 0;
       const line = lineFromOffset(offset);
-      const key = match[0].match(/"([^"]+)"/)?.[1] ?? match[0];
+      const key = match[0]?.match(/"([^"]+)"/)?.[1] ?? expectDefined6(match[0]);
       symbols.push(
         makeSymbol({
           name: key,
@@ -2413,12 +2454,12 @@ function regexParse2(opts) {
 function extractPackageScripts(content, symbols, file, lang, lineOffsets, lineFromOffset) {
   const scriptsBlockRegex = /"scripts"\s*:\s*\{([^}]+)\}/g;
   for (let match = scriptsBlockRegex.exec(content); match !== null; match = scriptsBlockRegex.exec(content)) {
-    const blockContent = match[0];
-    const blockOffset = match.index;
+    const blockContent = expectDefined6(match[0]);
+    const blockOffset = match.index ?? 0;
     const scriptKeyRegex = /"(\w[\w-]*)"\s*:/g;
     for (let scriptMatch = scriptKeyRegex.exec(blockContent); scriptMatch !== null; scriptMatch = scriptKeyRegex.exec(blockContent)) {
-      const key = scriptMatch[1];
-      const keyOffset = blockOffset + scriptMatch.index;
+      const key = expectDefined6(scriptMatch[1]);
+      const keyOffset = blockOffset + expectDefined6(scriptMatch.index);
       const line = lineFromOffset(keyOffset);
       symbols.push(
         makeSymbol({
@@ -2437,12 +2478,12 @@ function extractPackageScripts(content, symbols, file, lang, lineOffsets, lineFr
 function extractCompilerOptions(content, symbols, file, lang, lineOffsets, parentLine, lineFromOffset) {
   const optsBlockRegex = /"compilerOptions"\s*:\s*\{([^}]+)\}/g;
   for (let match = optsBlockRegex.exec(content); match !== null; match = optsBlockRegex.exec(content)) {
-    const blockContent = match[0];
-    const blockOffset = match.index;
+    const blockContent = expectDefined6(match[0]);
+    const blockOffset = match.index ?? 0;
     const optKeyRegex = /"(\w[\w]*)"\s*:/g;
     for (let optMatch = optKeyRegex.exec(blockContent); optMatch !== null; optMatch = optKeyRegex.exec(blockContent)) {
-      const key = optMatch[1];
-      const keyOffset = blockOffset + optMatch.index;
+      const key = expectDefined6(optMatch[1]);
+      const keyOffset = blockOffset + expectDefined6(optMatch.index);
       const line = lineFromOffset(keyOffset);
       if (line <= parentLine) continue;
       symbols.push(
@@ -2476,6 +2517,12 @@ function makeSymbol(opts) {
 }
 
 // src/codebase-index/yaml-parser.ts
+function expectDefined7(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 function parseSymbols6(opts) {
   const { file, content, lang } = opts;
   try {
@@ -2490,22 +2537,22 @@ function regexParse3(opts) {
   const lines = content.split("\n");
   const lineOffsets = [0];
   for (let i = 0; i < lines.length; i++) {
-    lineOffsets.push(lineOffsets[i] + lines[i].length + 1);
+    lineOffsets.push((lineOffsets[i] ?? 0) + (lines[i]?.length ?? 0) + 1);
   }
   function lineFromOffset(offset) {
     let lo = 0;
     let hi = lineOffsets.length - 1;
     while (lo < hi) {
       const mid = lo + hi + 1 >>> 1;
-      if (lineOffsets[mid] <= offset) lo = mid;
+      if (expectDefined7(lineOffsets[mid]) <= offset) lo = mid;
       else hi = mid - 1;
     }
     return lo + 1;
   }
   const anchorRegex = /&(\w[\w-]*)/g;
   for (let match = anchorRegex.exec(content); match !== null; match = anchorRegex.exec(content)) {
-    const name = match[1];
-    const offset = match.index;
+    const name = expectDefined7(match[1]);
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
     symbols.push(
@@ -2522,8 +2569,8 @@ function regexParse3(opts) {
   }
   const aliasRegex = /\*(\w[\w-]*)/g;
   for (let match = aliasRegex.exec(content); match !== null; match = aliasRegex.exec(content)) {
-    const name = match[1];
-    const offset = match.index;
+    const name = expectDefined7(match[1]);
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
     symbols.push(
@@ -2540,27 +2587,28 @@ function regexParse3(opts) {
   }
   const kvRegex = /^(\s*)([^:#\s][^:#\s]*)\s*:/gm;
   for (let match = kvRegex.exec(content); match !== null; match = kvRegex.exec(content)) {
-    const indent = match[1].length;
+    const indent = match[1]?.length ?? 0;
     const key = match[2];
-    const offset = match.index;
+    if (!key) continue;
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
     const lineContent = lines[line - 1] ?? "";
     if (/^[|&>]/.test(lineContent.trim())) continue;
     if (key === "---" || key === "...") continue;
     if (indent > 12) continue;
-    const value = extractValue(content, match.index);
+    const value = extractValue(content, match.index ?? 0);
     const kind = isScalar(value) ? "literal" : "property";
     const signature = `${key}: ${truncate(value, 60)}`;
     symbols.push(makeSymbol2({ name: key, kind, line, col, signature, file, lang }));
   }
   const listItemRegex = /^-(\s+)([^:#\s][^:#\s]*)\s*:/gm;
   for (let match = listItemRegex.exec(content); match !== null; match = listItemRegex.exec(content)) {
-    const key = match[2];
-    const offset = match.index;
+    const key = expectDefined7(match[2]);
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
-    const value = extractValue(content, offset + match[0].length);
+    const value = extractValue(content, offset + match[0]?.length);
     const kind = isScalar(value) ? "literal" : "property";
     symbols.push(
       makeSymbol2({
@@ -2576,8 +2624,8 @@ function regexParse3(opts) {
   }
   const blockScalarRegex = /^(\s*)([^:#\s][^:#\s]*)\s*:\s*[|>](\s|$)/gm;
   for (let match = blockScalarRegex.exec(content); match !== null; match = blockScalarRegex.exec(content)) {
-    const key = match[2];
-    const offset = match.index;
+    const key = expectDefined7(match[2]);
+    const offset = match.index ?? 0;
     const line = lineFromOffset(offset);
     const col = offset - (lineOffsets[line - 1] ?? 0);
     symbols.push(
@@ -2676,7 +2724,47 @@ async function loadGitignoreMatcher(projectRoot) {
   return compileGitignore(lines);
 }
 
+// src/codebase-index/background-indexer.ts
+var _ready = false;
+var _indexing = false;
+var _currentFile = 0;
+var _totalFiles = 0;
+var _lastError = null;
+function setIndexReady() {
+  _ready = true;
+}
+function getIndexState() {
+  return {
+    ready: _ready,
+    indexing: _indexing,
+    currentFile: _currentFile,
+    totalFiles: _totalFiles,
+    lastError: _lastError
+  };
+}
+var _listeners = [];
+function emitState() {
+  const state = getIndexState();
+  for (const l of _listeners) l(state);
+}
+function _setIndexProgress(current, total) {
+  _currentFile = current;
+  _totalFiles = total;
+  emitState();
+}
+Promise.resolve();
+
 // src/codebase-index/indexer.ts
+function expectDefined8(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
+var YIELD_EVERY_N = 50;
+function yieldEventLoop() {
+  return new Promise((resolve7) => setImmediate(resolve7));
+}
 var DEFAULT_IGNORE = [
   "node_modules",
   ".git",
@@ -2780,7 +2868,12 @@ async function runIndexer(_ctx, opts) {
   if (!force) {
     for (const meta of store.getAllFileMetas()) existingMeta.set(meta.file, meta);
   }
-  for (const file of files) {
+  for (let fi = 0; fi < files.length; fi++) {
+    const file = expectDefined8(files[fi]);
+    _setIndexProgress(fi + 1, files.length);
+    if (fi > 0 && fi % YIELD_EVERY_N === 0) {
+      await yieldEventLoop();
+    }
     let stat10;
     try {
       stat10 = await fs12.stat(file);
@@ -2833,7 +2926,7 @@ async function runIndexer(_ctx, opts) {
     langStats[lang] = (langStats[lang] ?? 0) + count;
     if (parsed.refs && parsed.refs.length > 0) {
       for (let i = 0; i < symbolsWithIds.length; i++) {
-        const sym = symbolsWithIds[i];
+        const sym = expectDefined8(symbolsWithIds[i]);
         const symRefs = parsed.refs.filter((r) => r.line === sym.line);
         if (symRefs.length > 0) {
           const refsWithFromId = symRefs.map((r) => ({ ...r, fromId: sym.id }));
@@ -2900,6 +2993,7 @@ var codebaseIndexTool = {
       langs: input.langs,
       indexDir: codebaseIndexDirOverride(ctx)
     });
+    setIndexReady();
     return result;
   }
 };
@@ -2993,6 +3087,12 @@ var Bm25Index = class {
 };
 
 // src/codebase-index/codebase-search-tool.ts
+function expectDefined9(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var codebaseSearchTool = {
   name: "codebase-search",
   category: "Project",
@@ -3035,6 +3135,31 @@ var codebaseSearchTool = {
     required: ["query"]
   },
   async execute(input, ctx) {
+    const state = getIndexState();
+    if (!state.ready) {
+      return {
+        results: [],
+        total: 0,
+        query: input.query,
+        indexStatus: state.indexing ? `Indexing in progress (${state.currentFile}/${state.totalFiles} files) \u2014 retry in a moment.` : "Index not yet built. The codebase is being indexed at startup \u2014 search will be available shortly."
+      };
+    }
+    if (state.indexing) {
+      return {
+        results: [],
+        total: 0,
+        query: input.query,
+        indexStatus: `Index refresh in progress (${state.currentFile}/${state.totalFiles} files). Results may be incomplete.`
+      };
+    }
+    if (state.lastError) {
+      return {
+        results: [],
+        total: 0,
+        query: input.query,
+        indexStatus: `Index build failed: ${state.lastError}. Try /codebase-reindex.`
+      };
+    }
     const store = new IndexStore(ctx.projectRoot, { indexDir: codebaseIndexDirOverride(ctx) });
     try {
       const limit = Math.min(input.limit ?? 20, 100);
@@ -3057,7 +3182,7 @@ var codebaseSearchTool = {
       const top = scored.slice(0, limit);
       const qTokens = tokenise(input.query);
       const results = top.map(({ id, score }) => {
-        const c = candidates.find((c2) => c2.id === id);
+        const c = expectDefined9(candidates.find((c2) => c2.id === id));
         const snippet = bm25.extractSnippet(id, qTokens);
         return {
           ...c,
@@ -3092,6 +3217,32 @@ var codebaseStatsTool = {
     additionalProperties: false
   },
   async execute(_input, ctx) {
+    const idxState = getIndexState();
+    if (!idxState.ready) {
+      return {
+        totalSymbols: 0,
+        totalFiles: 0,
+        byLang: {},
+        byKind: {},
+        lastIndexed: null,
+        sizeBytes: 0,
+        indexPath: "",
+        version: SCHEMA_VERSION,
+        indexStatus: idxState.indexing ? `Indexing in progress (${idxState.currentFile}/${idxState.totalFiles} files).` : "Index not yet built."
+      };
+    }
+    if (idxState.indexing) {
+      const store2 = new IndexStore(ctx.projectRoot, { indexDir: codebaseIndexDirOverride(ctx) });
+      try {
+        const stats = store2.getStats();
+        return {
+          ...stats,
+          indexStatus: `Index refresh in progress (${idxState.currentFile}/${idxState.totalFiles} files). Stats may be incomplete.`
+        };
+      } finally {
+        store2.close();
+      }
+    }
     const store = new IndexStore(ctx.projectRoot, { indexDir: codebaseIndexDirOverride(ctx) });
     try {
       const stats = store.getStats();
@@ -3895,7 +4046,9 @@ var fetchTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of fetchTool.executeStream(input, ctx, opts)) {
+    const executeStream = fetchTool.executeStream;
+    if (!executeStream) throw new Error("fetchTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("fetch: stream ended without final event");
@@ -4154,7 +4307,9 @@ var formatTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of formatTool.executeStream(input, ctx, opts)) {
+    const executeStream = formatTool.executeStream;
+    if (!executeStream) throw new Error("formatTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("format: stream ended without final event");
@@ -4598,6 +4753,12 @@ function capSubject(line) {
 }
 
 // src/grep.ts
+function expectDefined10(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var DEFAULT_IGNORE3 = ["node_modules", ".git", "dist", "build", ".next", "coverage"];
 var grepTool = {
   name: "grep",
@@ -4646,7 +4807,9 @@ var grepTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of grepTool.executeStream(input, ctx, opts)) {
+    const executeStream = grepTool.executeStream;
+    if (!executeStream) throw new Error("grepTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("grep: stream ended without final event");
@@ -4737,7 +4900,7 @@ async function* runRgStream(input, base, mode, limit, signal) {
         waiter = r;
       });
     }
-    const c = queue.shift();
+    const c = expectDefined10(queue.shift());
     if (c.kind === "error") {
       errored = true;
       continue;
@@ -4923,7 +5086,9 @@ var installTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of installTool.executeStream(input, ctx, opts)) {
+    const executeStream = installTool.executeStream;
+    if (!executeStream) throw new Error("installTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("install: stream ended without final event");
@@ -5138,7 +5303,9 @@ var lintTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of lintTool.executeStream(input, ctx, opts)) {
+    const executeStream = lintTool.executeStream;
+    if (!executeStream) throw new Error("lintTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("lint: stream ended without final event");
@@ -5765,14 +5932,15 @@ var planTool = {
 };
 function mkResult(plan, ok, message, todos) {
   const open = plan.items.filter((i) => i.status !== "done").length;
-  return {
+  const result = {
     ok,
     message,
     plan: formatPlan(plan),
     count: plan.items.length,
-    open,
-    todos
+    open
   };
+  if (todos !== void 0) result.todos = todos;
+  return result;
 }
 var MAX_BYTES2 = 5 * 1024 * 1024;
 var readTool = {
@@ -5846,6 +6014,12 @@ var readTool = {
     };
   }
 };
+function expectDefined11(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var DEFAULT_IGNORE4 = ["node_modules", ".git", "dist", "build", ".next", "coverage"];
 var replaceTool = {
   name: "replace",
@@ -5925,8 +6099,8 @@ var replaceTool = {
       const count = matches.length;
       let newContentLf = contentLf;
       for (let i = matches.length - 1; i >= 0; i--) {
-        const m = matches[i];
-        newContentLf = newContentLf.slice(0, m.index) + input.replacement + newContentLf.slice(m.index + m[0].length);
+        const m = expectDefined11(matches[i]);
+        newContentLf = newContentLf.slice(0, m.index) + input.replacement + newContentLf.slice(expectDefined11(m.index) + m[0].length);
       }
       re.lastIndex = 0;
       totalReplacements += count;
@@ -6227,6 +6401,12 @@ function substituteVars(content, name, vars) {
 }
 
 // src/search.ts
+function expectDefined12(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var DEFAULT_NUM = 10;
 var MAX_RESULTS = 50;
 var TIMEOUT_MS4 = 15e3;
@@ -6259,7 +6439,9 @@ var searchTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of searchTool.executeStream(input, ctx, opts)) {
+    const executeStream = searchTool.executeStream;
+    if (!executeStream) throw new Error("searchTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("search: stream ended without final event");
@@ -6320,11 +6502,11 @@ function parseDuckDuckGo(html, num) {
   const snippetRegex = /<a class="result-link"[^>]+href="([^"]+)"[^>]*>([^<]+)<\/a>/gi;
   const snippet2Regex = /<a class="result-snippet"[^>]*>([^<]+)<\/a>/gi;
   const linkMatches = takeFrom(
-    [...html.matchAll(snippetRegex)].filter((m) => m[1] && m[2]).map((m) => ({ url: m[1], title: stripTags2(m[2]) })),
+    [...html.matchAll(snippetRegex)].filter((m) => m[1] && m[2]).map((m) => ({ url: expectDefined12(m[1]), title: stripTags2(expectDefined12(m[2])) })),
     num
   );
   const snippetMatches = takeFrom(
-    [...html.matchAll(snippet2Regex)].filter((m) => m[1]).map((m) => stripTags2(m[1])),
+    [...html.matchAll(snippet2Regex)].filter((m) => m[1]).map((m) => stripTags2(expectDefined12(m[1]))),
     num
   );
   for (let i = 0; i < linkMatches.length && i < num; i++) {
@@ -6355,15 +6537,15 @@ function parseGoogleResults(html, num) {
   const urlRegex = /<cite[^>]*>([^<]+)<\/cite>/gi;
   const snippetRegex = /<span[^>]*class="[^"]*aXCZ0b[^>]*>([^<]+)<\/span>/gi;
   const titles = takeFrom(
-    [...html.matchAll(titleRegex)].filter((m) => m[1]).map((m) => stripTags2(m[1])),
+    [...html.matchAll(titleRegex)].filter((m) => m[1]).map((m) => stripTags2(expectDefined12(m[1]))),
     num
   );
   const urls = takeFrom(
-    [...html.matchAll(urlRegex)].filter((m) => m[1]).map((m) => stripTags2(m[1]).replace(/^\*(https?:\/\/[^\s]+).*$/, "$1")).filter((u) => u.startsWith("http")),
+    [...html.matchAll(urlRegex)].filter((m) => m[1]).map((m) => stripTags2(expectDefined12(m[1])).replace(/^\*(https?:\/\/[^\s]+).*$/, "$1")).filter((u) => u.startsWith("http")),
     num
   );
   const snippets = takeFrom(
-    [...html.matchAll(snippetRegex)].filter((m) => m[1]).map((m) => stripTags2(m[1])),
+    [...html.matchAll(snippetRegex)].filter((m) => m[1]).map((m) => stripTags2(expectDefined12(m[1]))),
     num
   );
   for (let i = 0; i < Math.min(titles.length, num); i++) {
@@ -6392,11 +6574,11 @@ function parseBingResults(html, num) {
   const titleRegex = /<h2[^>]*>\s*<a[^>]+href="([^"]+)"[^>]*>([^<]+)<\/a>\s*<\/h2>/gi;
   const snippetRegex = /<p[^>]*class="[^"]*b_paractl[^"]*"[^>]*>([^<]+)<\/p>/gi;
   const entries = takeFrom(
-    [...html.matchAll(titleRegex)].filter((m) => m[1] && m[2]).map((m) => ({ url: m[1], title: stripTags2(m[2]) })),
+    [...html.matchAll(titleRegex)].filter((m) => m[1] && m[2]).map((m) => ({ url: expectDefined12(m[1]), title: stripTags2(expectDefined12(m[2])) })),
     num
   );
   const snippets = takeFrom(
-    [...html.matchAll(snippetRegex)].filter((m) => m[1]).map((m) => stripTags2(m[1])),
+    [...html.matchAll(snippetRegex)].filter((m) => m[1]).map((m) => stripTags2(expectDefined12(m[1]))),
     num
   );
   for (let i = 0; i < entries.length; i++) {
@@ -6466,7 +6648,9 @@ var testTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of testTool.executeStream(input, ctx, opts)) {
+    const executeStream = testTool.executeStream;
+    if (!executeStream) throw new Error("testTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("test: stream ended without final event");
@@ -6589,9 +6773,10 @@ var todoTool = {
   name: "todo",
   category: "Session",
   description: "Manage the session-level todo list. This is the primary mechanism for tracking multi-step work. The list is fully replaced on every call (not appended).",
-  usageHint: "BEST PRACTICE for complex tasks:\n- At the beginning of a non-trivial task, create a clear todo list with specific, actionable items.\n- Only **one** item should be `in_progress` at any time.\n- Update the list frequently as work progresses (mark items done, add new ones, change status).\n- The system and user can see this list, so keep it honest and up-to-date.\nThis tool is extremely valuable for maintaining focus and giving the user visibility into your plan.",
+  usageHint: "BEST PRACTICE for complex tasks:\n- At the beginning of a non-trivial task, create a clear todo list with specific, actionable items.\n- Only **one** item should be `in_progress` at any time.\n- Update the list frequently as work progresses (mark items done, add new ones, change status).\n- **Re-order items** to reflect current priorities \u2014 the full list is replaced each call, so item order is entirely under your control.\n- When all items are completed the board auto-clears \u2014 you do NOT need to send an empty list.\n- The system and user can see this list, so keep it honest and up-to-date.\nThis tool is extremely valuable for maintaining focus and giving the user visibility into your plan.",
   permission: "auto",
   mutating: false,
+  // mutates only conversation state (ctx.todos), not external state — no confirmation needed
   timeoutMs: 1e3,
   inputSchema: {
     type: "object",
@@ -6917,6 +7102,12 @@ var toolUseTool = {
     }
   }
 };
+function expectDefined13(value) {
+  if (value === null || value === void 0) {
+    throw new Error("Expected value to be defined");
+  }
+  return value;
+}
 var DEFAULT_IGNORE5 = [
   "node_modules",
   ".git",
@@ -6977,7 +7168,9 @@ var treeTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of treeTool.executeStream(input, ctx, opts)) {
+    const executeStream = treeTool.executeStream;
+    if (!executeStream) throw new Error("treeTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("tree: stream ended without final event");
@@ -7027,7 +7220,7 @@ var treeTool = {
     });
     while (!walkDone || queue.length > 0) {
       if (queue.length > 0) {
-        yield queue.shift();
+        yield expectDefined13(queue.shift());
       } else {
         let pollTimer;
         const poll = new Promise((r) => {
@@ -7117,7 +7310,9 @@ var typecheckTool = {
   },
   async execute(input, ctx, opts) {
     let final;
-    for await (const ev of typecheckTool.executeStream(input, ctx, opts)) {
+    const executeStream = typecheckTool.executeStream;
+    if (!executeStream) throw new Error("typecheckTool: stream execution unavailable");
+    for await (const ev of executeStream(input, ctx, opts)) {
       if (ev.type === "final") final = ev.output;
     }
     if (!final) throw new Error("typecheck: stream ended without final event");