@wrongstack/tools 0.319.0 → 0.319.1

package/dist/{chunk-4Q7LFU4T.js → chunk-RLMX75EM.js}

@@ -1,6 +1,6 @@
 import {
   tryLegacyPackageOperation
-} from "./chunk-ZDUGI5BC.js";
+} from "./chunk-ATSKPZUK.js";
 import {
   spawnStream
 } from "./chunk-ZHH3BEWU.js";
@@ -8,7 +8,7 @@ import {
   detectPackageManager,
   normalizeCommandOutput,
   safeResolveReal
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/install.ts
 import { join } from "node:path";
@@ -16,6 +16,7 @@ import {
   detectEcosystem as detectPackageEcosystem,
   recordPackageAction
 } from "@wrongstack/core/coordination";
+import { ToolValidationError } from "@wrongstack/core/types";
 var installTool = {
   name: "install",
   category: "Package Management",
@@ -74,8 +75,17 @@ var installTool = {
   async *executeStream(input, ctx, opts) {
     const cwd = input.cwd ? await safeResolveReal(input.cwd, ctx) : ctx.cwd;
     const signal = opts?.signal ?? ctx.signal ?? new AbortController().signal;
+    signal.throwIfAborted();
+    const VALID_SAVES = /* @__PURE__ */ new Set(["dependency", "dev", "optional"]);
+    if (input.save !== void 0 && !VALID_SAVES.has(input.save)) {
+      throw new ToolValidationError({
+        message: `install: invalid save option "${input.save}". Allowed: dependency, dev, optional`,
+        field: "save"
+      });
+    }
     if (!input.global) {
-      const pkgList2 = input.packages ? Array.isArray(input.packages) ? input.packages : input.packages.split(",").map((p) => p.trim()) : [];
+      const rawList2 = input.packages ? (Array.isArray(input.packages) ? input.packages : input.packages.split(",")).map((p) => p.trim()).filter(Boolean) : [];
+      const pkgList2 = Array.from(new Set(rawList2));
       const bridge = await tryLegacyPackageOperation(
         "package-install",
         {
@@ -107,9 +117,8 @@ var installTool = {
     yield { type: "log", text: `Resolving with ${pkgManager}\u2026`, data: { phase: "resolve" } };
     const globalFlag = input.global ? ["-g"] : [];
     const ignoreScripts = input.lifecycleScripts !== true;
-    const pkgList = input.packages ? (Array.isArray(input.packages) ? input.packages : input.packages.split(",")).map(
-      (p) => p.trim()
-    ) : [];
+    const rawList = input.packages ? (Array.isArray(input.packages) ? input.packages : input.packages.split(",")).map((p) => p.trim()).filter(Boolean) : [];
+    const pkgList = Array.from(new Set(rawList));
     const PKG_NAME_RE = /^(?:@[a-z0-9._-]+\/)?[a-z0-9._-]+(?:@[a-z0-9^~><=*.+-]+)?$/i;
     for (const pkg of pkgList) {
       if (!PKG_NAME_RE.test(pkg) || pkg.startsWith("-") || pkg.length > 200) {
@@ -166,10 +175,12 @@ var installTool = {
       signal,
       maxBytes: 1e5
     });
+    const rawOutput = result.stdout && result.stderr ? `${result.stdout}
+${result.stderr}` : result.stdout || result.stderr || result.error || "";
     const output = {
       packages: pkgList,
       exit_code: result.exitCode,
-      output: normalizeCommandOutput(result.stdout || result.stderr || result.error || ""),
+      output: normalizeCommandOutput(rawOutput),
       dry_run: args.includes("--dry-run"),
       truncated: result.truncated
     };
@@ -221,4 +232,4 @@ function resolveManifestPath(cwd, pkgManager) {
 export {
   installTool
 };
-//# sourceMappingURL=chunk-4Q7LFU4T.js.map
+//# sourceMappingURL=chunk-RLMX75EM.js.map

package/dist/{chunk-C3W3V54D.js → chunk-RNWUCZ2U.js}

@@ -1,15 +1,15 @@
 import {
   getIndexState,
   searchCodebaseIndex
-} from "./chunk-MRKWZCHZ.js";
+} from "./chunk-DP6CXMJZ.js";
 import {
   codebaseIndexDirOverride
-} from "./chunk-TFMVHYJI.js";
+} from "./chunk-KY4PXYSO.js";
 import {
   isBinaryBuffer,
   safeResolveReal,
   sha256hex
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/read.ts
 import * as fs from "node:fs/promises";
@@ -113,8 +113,8 @@ var readTool = {
     const limit = Math.max(0, Math.min(input.limit ?? 2e3, 5e3));
     const prior = getReadRangeRecord(ctx, absPath);
     const requestedEnd = prior ? Math.min(offset + limit - 1, prior.totalLines) : offset + limit - 1;
-    if (input.mode !== "summary" && limit > 0 && prior && coversRange(prior, stat2.mtimeMs, offset, requestedEnd)) {
-      ctx.recordRead(absPath, stat2.mtimeMs);
+    if (input.mode !== "summary" && limit > 0 && prior && offset <= requestedEnd && coversRange(prior, stat2.mtimeMs, stat2.size, offset, requestedEnd)) {
+      ctx.recordRead(absPath, stat2.mtimeMs, "user", ctx.lastReadHash?.(absPath));
       const symResult2 = shouldIncludeSymbols ? await fetchSymbolsForFile(absPath, ctx, signal) : void 0;
       return {
         text: `[unchanged since previous read: "${input.path}" mtime=${Math.round(stat2.mtimeMs)}; requested lines ${offset}-${requestedEnd} were already shown. Use offset/limit for a new range if needed.]`,
@@ -142,13 +142,14 @@ var readTool = {
     const total = allLines.length;
     if (input.mode === "summary") {
       ctx.recordRead(absPath, stat2.mtimeMs, "user", contentHash);
-      rememberReadRange(ctx, absPath, stat2.mtimeMs, total, 1, Math.min(total, 200));
+      rememberReadRange(ctx, absPath, stat2.mtimeMs, stat2.size, total, 1, Math.min(total, 200));
       const symResult2 = shouldIncludeSymbols ? await fetchSymbolsForFile(absPath, ctx, signal) : void 0;
+      const summary = summarizeFile(input.path, stat2.size, allLines);
       return {
-        text: summarizeFile(input.path, stat2.size, allLines),
+        text: summary.text,
         total_lines: total,
         encoding: "utf8",
-        truncated: total > 200,
+        truncated: summary.truncated,
         note: mergeSymbolNote(
           "Summary mode returned compact structure instead of full file content.",
           symResult2?.note
@@ -158,7 +159,7 @@ var readTool = {
     }
     if (limit === 0) {
       ctx.recordRead(absPath, stat2.mtimeMs, "user", contentHash);
-      rememberReadRange(ctx, absPath, stat2.mtimeMs, total, 1, 0);
+      rememberReadRange(ctx, absPath, stat2.mtimeMs, stat2.size, total, 1, 0);
       const symResult2 = shouldIncludeSymbols ? await fetchSymbolsForFile(absPath, ctx, signal) : void 0;
       return {
         text: "",
@@ -171,7 +172,6 @@ var readTool = {
     }
     if (offset > total) {
       ctx.recordRead(absPath, stat2.mtimeMs, "user", contentHash);
-      rememberReadRange(ctx, absPath, stat2.mtimeMs, total, total + 1, total + 1);
       const symResult2 = shouldIncludeSymbols ? await fetchSymbolsForFile(absPath, ctx, signal) : void 0;
       return {
         text: `[offset ${offset} is past end of file "${input.path}" \u2014 file has ${total} line(s). Do not retry this offset.]`,
@@ -184,10 +184,26 @@ var readTool = {
     }
     const slice = allLines.slice(offset - 1, offset - 1 + limit);
     const truncated = offset - 1 + slice.length < total;
-    const width = String(offset + slice.length - 1).length;
-    const numbered = slice.map((line, i) => `${String(offset + i).padStart(width, " ")}\u2192${line}`).join("\n");
+    const sliceCount = slice.length;
+    const width = String(offset + sliceCount - 1).length;
+    const parts = new Array(sliceCount);
+    for (let i = 0; i < sliceCount; i++) {
+      const numStr = String(offset + i);
+      const padLen = width - numStr.length;
+      const pad = padLen > 0 ? " ".repeat(padLen) : "";
+      parts[i] = `${pad}${numStr}\u2192${slice[i]}`;
+    }
+    const numbered = parts.join("\n");
     ctx.recordRead(absPath, stat2.mtimeMs, "user", contentHash);
-    rememberReadRange(ctx, absPath, stat2.mtimeMs, total, offset, offset + slice.length - 1);
+    rememberReadRange(
+      ctx,
+      absPath,
+      stat2.mtimeMs,
+      stat2.size,
+      total,
+      offset,
+      offset + slice.length - 1
+    );
     const symResult = shouldIncludeSymbols ? await fetchSymbolsForFile(absPath, ctx, signal) : void 0;
     return {
       text: numbered,
@@ -208,7 +224,7 @@ async function fetchSymbolsForFile(absPath, ctx, signal) {
         projectRoot: ctx.projectRoot,
         indexDir: codebaseIndexDirOverride(ctx),
         query: "",
-        file: absPath,
+        file: absPath.replace(/\\/g, "/"),
         limit: 500
       },
       { signal }
@@ -248,20 +264,23 @@ function getReadRanges(ctx) {
 function getReadRangeRecord(ctx, absPath) {
   return getReadRanges(ctx)[absPath];
 }
-function rememberReadRange(ctx, absPath, mtimeMs, totalLines, start, end) {
+var MTIME_TOLERANCE_MS = process.platform === "win32" ? 2e3 : 1;
+function rememberReadRange(ctx, absPath, mtimeMs, size, totalLines, start, end) {
   if (end < start) return;
   const ranges = getReadRanges(ctx);
   const prior = ranges[absPath];
-  const nextRanges = prior && Math.abs(prior.mtimeMs - mtimeMs) <= 1 ? prior.ranges.slice() : [];
+  const nextRanges = prior && prior.size === size && Math.abs(prior.mtimeMs - mtimeMs) <= MTIME_TOLERANCE_MS ? prior.ranges.slice() : [];
   nextRanges.push({ start, end });
   ranges[absPath] = {
     mtimeMs,
+    size,
     totalLines,
     ranges: mergeRanges(nextRanges)
   };
 }
-function coversRange(record, mtimeMs, start, end) {
-  if (Math.abs(record.mtimeMs - mtimeMs) > 1) return false;
+function coversRange(record, mtimeMs, size, start, end) {
+  if (record.size !== void 0 && record.size !== size) return false;
+  if (Math.abs(record.mtimeMs - mtimeMs) > MTIME_TOLERANCE_MS) return false;
   return record.ranges.some((range) => range.start <= start && range.end >= end);
 }
 function mergeRanges(ranges) {
@@ -278,21 +297,30 @@ function mergeRanges(ranges) {
   return merged;
 }
 function summarizeFile(filePath, bytes, lines) {
-  const interesting = lines.map((line, index) => ({ line: line.trim(), number: index + 1 })).filter(
-    ({ line }) => /^(import\s|export\s|class\s|interface\s|type\s|function\s|const\s+\w+\s*=|let\s+\w+\s*=|var\s+\w+\s*=|def\s+|async\s+function\s)/.test(
-      line
-    )
-  ).slice(0, 80).map(({ line, number }) => `${number}: ${line}`);
-  return [
+  const interesting = [];
+  const symbolRegex = /^(import\s|export\s|class\s|interface\s|type\s|function\s|const\s+\w+\s*=|let\s+\w+\s*=|var\s+\w+\s*=|def\s+|async\s+(?:def|function)\s|func\s+|fn\s+|pub\s+(?:fn|struct|enum|trait|type|const)\s+|struct\s+|enum\s+|impl\s+)/;
+  let truncated = false;
+  for (let i = 0; i < lines.length; i++) {
+    if (interesting.length >= 80) {
+      truncated = true;
+      break;
+    }
+    const trimmed = lines[i].trim();
+    if (symbolRegex.test(trimmed)) {
+      interesting.push(`${i + 1}: ${trimmed}`);
+    }
+  }
+  const text = [
     `summary: ${filePath}`,
     `bytes=${bytes}`,
     `total_lines=${lines.length}`,
     interesting.length > 0 ? `symbols/imports:
 ${interesting.join("\n")}` : "symbols/imports: (none detected)"
   ].join("\n");
+  return { text, truncated };
 }
 
 export {
   readTool
 };
-//# sourceMappingURL=chunk-C3W3V54D.js.map
+//# sourceMappingURL=chunk-RNWUCZ2U.js.map

package/dist/{chunk-RK47IWO3.js → chunk-RTZWBNTY.js}

@@ -1,6 +1,6 @@
 import {
   parseTreeSitterAst
-} from "./chunk-OCUXN6Y3.js";
+} from "./chunk-XRNN44QY.js";
 import {
   spawnStream
 } from "./chunk-ZHH3BEWU.js";
@@ -13,7 +13,7 @@ import {
   outgoingCallsService,
   runStartupIndex,
   searchCodebaseIndex
-} from "./chunk-MRKWZCHZ.js";
+} from "./chunk-DP6CXMJZ.js";
 import {
   IndexTimeoutError,
   SCHEMA_VERSION,
@@ -21,7 +21,7 @@ import {
   indexCircuitBreaker,
   indexStorePool,
   resolveIndexDir
-} from "./chunk-TFMVHYJI.js";
+} from "./chunk-KY4PXYSO.js";
 import {
   EXT_TO_LANG,
   detectLang,
@@ -30,7 +30,7 @@ import {
 import {
   normalizeCommandOutput,
   safeResolveProjectPath
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/codebase-index/codebase-ast-replace-tool.ts
 import * as path2 from "node:path";
@@ -1088,9 +1088,10 @@ var codebaseIncomingCallsTool = {
 };
 
 // src/codebase-index/codebase-index-tool.ts
-import { ToolValidationError } from "@wrongstack/core/types";
+import { ToolValidationError as ToolValidationError2 } from "@wrongstack/core/types";
 
 // src/codebase-index/codebase-search-tool.ts
+import { ToolValidationError } from "@wrongstack/core/types";
 import { toErrorMessage as toErrorMessage5 } from "@wrongstack/core/utils";
 var INDEXABLE_LANG_IDS = [
   ...new Set(Object.values(EXT_TO_LANG))
@@ -1167,6 +1168,18 @@ var codebaseSearchTool = {
     required: ["query"]
   },
   async execute(input, ctx, execOpts) {
+    if (!input?.query || typeof input.query !== "string" || !input.query.trim()) {
+      throw new ToolValidationError({
+        message: "codebase-search: query is required and cannot be empty",
+        field: "query"
+      });
+    }
+    if (input.lspKind !== void 0 && (!Number.isInteger(input.lspKind) || input.lspKind < 1 || input.lspKind > 26)) {
+      throw new ToolValidationError({
+        message: "codebase-search: lspKind must be an integer between 1 and 26",
+        field: "lspKind"
+      });
+    }
     const signal = execOpts?.signal ?? ctx?.signal;
     signal?.throwIfAborted();
     const state = getIndexState();
@@ -1190,7 +1203,7 @@ var codebaseSearchTool = {
           query: input.query,
           kind: input.kind?.toLowerCase(),
           lang: input.lang?.toLowerCase(),
-          file: input.file,
+          file: input.file ? input.file.replace(/\\/g, "/").replace(/^\.\//, "") : void 0,
           lspKind: input.lspKind,
           limit
         },
@@ -1224,7 +1237,7 @@ var codebaseSearchTool = {
       try {
         const stats = await codebaseIndexStats(
           { projectRoot: ctx.projectRoot, indexDir: codebaseIndexDirOverride(ctx) },
-          { signal: execOpts?.signal }
+          { signal }
         );
         hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
       } catch {
@@ -1276,12 +1289,20 @@ var codebaseIndexTool = {
     }
   },
   async execute(input, ctx, execOpts) {
+    const signal = execOpts?.signal ?? ctx?.signal;
+    signal?.throwIfAborted();
     if (input.langs) {
+      if (!Array.isArray(input.langs) || input.langs.length === 0) {
+        throw new ToolValidationError2({
+          message: "codebase-index: langs cannot be an empty array. Pass at least one valid language or omit langs.",
+          field: "langs"
+        });
+      }
       const unknown = input.langs.filter(
         (lang) => !INDEXABLE_LANG_IDS.includes(lang)
       );
       if (unknown.length > 0) {
-        throw new ToolValidationError({
+        throw new ToolValidationError2({
           message: `codebase-index: unknown lang(s) ${unknown.map((l) => `"${l}"`).join(", ")}. Valid ids: ${INDEXABLE_LANG_IDS.join(", ")}.`,
           field: "langs"
         });
@@ -1297,23 +1318,27 @@ var codebaseIndexTool = {
         note: "A full index is already in progress. Retry codebase-index after it completes (check codebase-stats)."
       };
     }
-    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.ceil(circuit.cooldownRemainingMs / 1e3)}s; the user can run /codebase-reindex to retry immediately.`
-      };
+    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.`
+        };
+      }
     }
     const result = await runStartupIndex({
       projectRoot: ctx.projectRoot,
       force: input.force ?? false,
       langs: input.langs,
       indexDir: codebaseIndexDirOverride(ctx),
-      signal: execOpts?.signal
+      signal
     });
     if (result.errors.length > MAX_REPORTED_ERRORS) {
       const hidden = result.errors.length - MAX_REPORTED_ERRORS;
@@ -2047,7 +2072,9 @@ var codebaseSkeletonTool = {
     required: ["path"],
     additionalProperties: false
   },
-  async execute(input, ctx) {
+  async execute(input, ctx, execOpts) {
+    const signal = execOpts?.signal ?? ctx?.signal;
+    signal?.throwIfAborted();
     const targetPath = await safeResolveProjectPath(input.path, ctx);
     let stat3;
     try {
@@ -2103,6 +2130,7 @@ var codebaseSkeletonTool = {
 };
 
 // src/codebase-index/codebase-stats-tool.ts
+import { toErrorMessage as toErrorMessage9 } from "@wrongstack/core/utils";
 var codebaseStatsTool = {
   name: "codebase-stats",
   category: "Project",
@@ -2122,6 +2150,8 @@ var codebaseStatsTool = {
     additionalProperties: false
   },
   async execute(_input, ctx, execOpts) {
+    const signal = execOpts?.signal ?? ctx?.signal;
+    signal?.throwIfAborted();
     const idxState = getIndexState();
     const indexPath = resolveIndexDir(ctx.projectRoot, codebaseIndexDirOverride(ctx));
     if (idxState.indexing) {
@@ -2146,11 +2176,11 @@ var codebaseStatsTool = {
     try {
       stats = await codebaseIndexStats(
         { projectRoot: ctx.projectRoot, indexDir: codebaseIndexDirOverride(ctx) },
-        { signal: execOpts?.signal }
+        { signal }
       );
     } catch (err) {
-      if (!(err instanceof IndexTimeoutError) && err?.name !== "IndexTimeoutError") {
-        throw err;
+      if (signal?.aborted) {
+        signal.throwIfAborted();
       }
       return {
         totalSymbols: 0,
@@ -2162,14 +2192,15 @@ var codebaseStatsTool = {
         indexPath,
         version: SCHEMA_VERSION,
         statsAvailable: false,
-        indexStatus: "Index statistics timed out. Do not repeatedly retry stats; try codebase-search directly or use the appropriate grep/glob/tree fallback."
+        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.`
       };
     }
     const circuit = idxState.circuit;
     const hasPersistedIndex = stats.totalFiles > 0 || stats.lastIndexed !== null;
     let indexStatus;
     if (circuit.state === "open") {
-      indexStatus = `${hasPersistedIndex ? "Indexing" : "No persisted index data found, and indexing"} is paused after repeated failures (last: ${circuit.lastFailure ?? "unknown"}); auto-retry in ${Math.ceil(circuit.cooldownRemainingMs / 1e3)}s, or run /codebase-reindex. ` + (hasPersistedIndex ? "Stats reflect the last successful build." : "Build the index after recovery.");
+      const cooldownSec = Math.max(1, Math.ceil(circuit.cooldownRemainingMs / 1e3));
+      indexStatus = `${hasPersistedIndex ? "Indexing" : "No persisted index data found, and indexing"} is paused after repeated failures (last: ${circuit.lastFailure ?? "unknown"}); auto-retry in ${cooldownSec}s, or run /codebase-reindex. ` + (hasPersistedIndex ? "Stats reflect the last successful build." : "Build the index after recovery.");
     } else if (!hasPersistedIndex) {
       indexStatus = "No persisted index data found. Run codebase-index to build it before broad code exploration.";
     }
@@ -2191,7 +2222,7 @@ var codebaseStatsTool = {
 // src/codebase-index/codebase-targeted-test-tool.ts
 import * as fs6 from "node:fs/promises";
 import * as path6 from "node:path";
-import { toErrorMessage as toErrorMessage9 } from "@wrongstack/core/utils";
+import { toErrorMessage as toErrorMessage10 } from "@wrongstack/core/utils";
 var TEST_FILE_REGEX2 = /(?:test|spec|mock|fixture|bench)[s]?[/\\._]/i;
 async function findConventionTestFiles(projectRoot, sourceRelPath) {
   const parsed = path6.parse(sourceRelPath);
@@ -2344,7 +2375,7 @@ ${streamRes.stderr}`.trim(),
         failed: 0,
         durationMs: 0,
         output: "",
-        error: toErrorMessage9(err)
+        error: toErrorMessage10(err)
       };
     }
   }
@@ -2834,4 +2865,4 @@ export {
   deadCodeScanTool,
   runDeadCodeScan
 };
-//# sourceMappingURL=chunk-RK47IWO3.js.map
+//# sourceMappingURL=chunk-RTZWBNTY.js.map

package/dist/{chunk-UC3RKRIV.js → chunk-TTZOK6IY.js}

@@ -4,7 +4,7 @@ import {
 import {
   normalizeCommandOutput,
   safeResolveReal
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/languages/detect.ts
 import { createHash } from "node:crypto";
@@ -3515,4 +3515,4 @@ export {
   languagePackageTool,
   languageInfoTool
 };
-//# sourceMappingURL=chunk-UC3RKRIV.js.map
+//# sourceMappingURL=chunk-TTZOK6IY.js.map

package/dist/{chunk-W4GAAI3M.js → chunk-UDVKRX5R.js}

@@ -1,6 +1,6 @@
 import {
   resolveIndexDir
-} from "./chunk-TFMVHYJI.js";
+} from "./chunk-KY4PXYSO.js";
 
 // src/codebase-index/binary-frame.ts
 import { decode, encode } from "@msgpack/msgpack";
@@ -120,4 +120,4 @@ export {
   PROJECT_INDEX_SERVER_MAX_FRAME_CHARS,
   encodeProjectServerMessage
 };
-//# sourceMappingURL=chunk-W4GAAI3M.js.map
+//# sourceMappingURL=chunk-UDVKRX5R.js.map

package/dist/{chunk-Y7QXSE7N.js → chunk-UQQS3VNT.js}

@@ -1,16 +1,17 @@
 import {
   tryLegacyCodeOperation
-} from "./chunk-ZDUGI5BC.js";
+} from "./chunk-ATSKPZUK.js";
 import {
   spawnStream
 } from "./chunk-ZHH3BEWU.js";
 import {
   normalizeCommandOutput,
   safeResolveReal
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/test.ts
 import * as path from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 var testTool = {
   name: "test",
   category: "Code Quality",
@@ -57,6 +58,14 @@ var testTool = {
   async *executeStream(input, ctx, opts) {
     const cwd = input.cwd ? await safeResolveReal(input.cwd, ctx) : ctx.cwd;
     const signal = opts?.signal ?? ctx.signal ?? new AbortController().signal;
+    signal.throwIfAborted();
+    const VALID_RUNNERS = /* @__PURE__ */ new Set(["vitest", "jest", "mocha", "auto", "none"]);
+    if (input.runner !== void 0 && !VALID_RUNNERS.has(input.runner)) {
+      throw new ToolValidationError({
+        message: `test: unsupported runner "${input.runner}". Allowed runners: vitest, jest, mocha, auto, none`,
+        field: "runner"
+      });
+    }
     const runner = input.runner ?? "auto";
     if (runner === "auto") {
       const bridge = await tryLegacyCodeOperation("test", {
@@ -83,7 +92,7 @@ var testTool = {
       }
     }
     const detected = runner === "auto" ? await detectRunner(cwd) : runner;
-    if (!detected) {
+    if (!detected || detected === "none") {
       yield {
         type: "final",
         output: {
@@ -115,7 +124,21 @@ var testTool = {
 };
 async function detectRunner(cwd) {
   const { stat } = await import("node:fs/promises");
-  const candidates = ["vitest.config.ts", "jest.config.js", ".mocharc.json"];
+  const candidates = [
+    "vitest.config.ts",
+    "vitest.config.js",
+    "vitest.config.mjs",
+    "vitest.config.mts",
+    "vitest.config.cjs",
+    "vitest.config.cts",
+    "jest.config.ts",
+    "jest.config.js",
+    "jest.config.mjs",
+    "jest.config.cjs",
+    ".mocharc.json",
+    ".mocharc.js",
+    ".mocharc.cjs"
+  ];
   for (const f of candidates) {
     try {
       await stat(path.join(cwd, f));
@@ -129,7 +152,8 @@ async function detectRunner(cwd) {
 }
 function buildArgs(runner, input) {
   const args = [];
-  const timeout = input.timeout ?? 3e4;
+  const rawTimeout = typeof input.timeout === "number" && Number.isFinite(input.timeout) && input.timeout > 0 ? Math.floor(input.timeout) : 3e4;
+  const timeout = Math.max(100, rawTimeout);
   switch (runner) {
     case "vitest":
       args.push(input.watch ? "watch" : "run");
@@ -152,8 +176,10 @@ function buildArgs(runner, input) {
       break;
   }
   if (input.files) {
-    const files = Array.isArray(input.files) ? input.files : input.files.split(",");
-    args.push("--", ...files.map((f) => f.trim()));
+    const files = (Array.isArray(input.files) ? input.files : input.files.split(",")).map((f) => f.trim().replace(/\\/g, "/")).filter(Boolean);
+    if (files.length > 0) {
+      args.push("--", ...files);
+    }
   }
   return args;
 }
@@ -175,7 +201,15 @@ function parseResult(runner, result, duration) {
     tests_run = Number.parseInt(suitesMatch?.[1] ?? "0", 10);
     passed = Number.parseInt(passedMatch?.[1] ?? "0", 10);
     failed = Number.parseInt(failedMatch?.[1] ?? "0", 10);
+  } else if (runner === "mocha") {
+    const passedMatch = out.match(/(\d+)\s+passing/);
+    const failedMatch = out.match(/(\d+)\s+failing/);
+    if (passedMatch?.[1]) passed = Number.parseInt(passedMatch[1], 10);
+    if (failedMatch?.[1]) failed = Number.parseInt(failedMatch[1], 10);
+    tests_run = passed + failed;
   }
+  const rawOutput = result.stdout && result.stderr ? `${result.stdout}
+${result.stderr}` : result.stdout || result.stderr || result.error || "";
   return {
     runner,
     exit_code: result.exitCode,
@@ -187,7 +221,7 @@ function parseResult(runner, result, duration) {
     // already parsed above and the FULL log is on disk (spool marker rides
     // the stdout tail). Failures keep the standard command-output cap so
     // the agent sees the failure details inline.
-    output: normalizeCommandOutput(result.stdout || result.error || "", {
+    output: normalizeCommandOutput(rawOutput, {
       maxBytes: result.exitCode === 0 ? 4096 : void 0
     }),
     truncated: result.truncated
@@ -197,4 +231,4 @@ function parseResult(runner, result, duration) {
 export {
   testTool
 };
-//# sourceMappingURL=chunk-Y7QXSE7N.js.map
+//# sourceMappingURL=chunk-UQQS3VNT.js.map