@wrongstack/tools 0.318.0 → 0.319.1

package/dist/{chunk-PQTONQ4P.js → chunk-QPNYDH5B.js}

@@ -56,17 +56,19 @@ var searchTool = {
     return final;
   },
   async *executeStream(input, ctx, opts) {
-    if (!input?.query || input.query.trim() === "") {
+    if (!input?.query || typeof input.query !== "string" || input.query.trim() === "") {
       throw new ToolValidationError({
         message: "search: query is required and must be a non-empty string",
         field: "query"
       });
     }
     const signal = opts?.signal ?? ctx?.signal ?? new AbortController().signal;
-    const num = Math.max(1, Math.min(input.num_results ?? DEFAULT_NUM, MAX_RESULTS));
+    const rawNum = typeof input.num_results === "number" && !Number.isNaN(input.num_results) ? input.num_results : DEFAULT_NUM;
+    const num = Math.max(1, Math.min(Math.floor(rawNum), MAX_RESULTS));
     const source = input.source ?? "duckduckgo";
     const skipCache = input.skip_cache ?? false;
-    const cacheKey = `${source}:${input.query}`;
+    const normalizedQuery = input.query.trim().toLowerCase().replace(/\s+/g, " ");
+    const cacheKey = `${source}:${normalizedQuery}`;
     if (!skipCache) {
       const entry = cache.get(cacheKey);
       if (entry && Date.now() - entry.timestamp < CACHE_TTL_MS) {
@@ -426,11 +428,23 @@ function capSnippet(snippet) {
   return snippet.length > MAX_SNIPPET_CHARS ? `${snippet.slice(0, MAX_SNIPPET_CHARS - 1)}\u2026` : snippet;
 }
 function decodeHtmlEntities(text) {
-  return text.replace(/&amp;/g, "&").replace(/&lt;/g, "<").replace(/&gt;/g, ">").replace(/&quot;/g, '"').replace(/&#39;/g, "'");
+  return text.replace(/&amp;/g, "&").replace(/&lt;/g, "<").replace(/&gt;/g, ">").replace(/&quot;/g, '"').replace(/&#39;/g, "'").replace(/&#x27;/gi, "'").replace(/&#039;/g, "'").replace(/&nbsp;/g, " ").replace(/&ndash;/g, "\u2013").replace(/&mdash;/g, "\u2014").replace(/&ldquo;|&rdquo;/g, '"').replace(/&lsquo;|&rsquo;/g, "'").replace(/&#(\d+);/g, (_, code) => {
+    try {
+      return String.fromCodePoint(Number(code));
+    } catch {
+      return _;
+    }
+  }).replace(/&#x([0-9a-f]+);/gi, (_, hex) => {
+    try {
+      return String.fromCodePoint(parseInt(hex, 16));
+    } catch {
+      return _;
+    }
+  });
 }
 
 export {
   searchTool,
   __clearSearchCache
 };
-//# sourceMappingURL=chunk-PQTONQ4P.js.map
+//# sourceMappingURL=chunk-QPNYDH5B.js.map

package/dist/{chunk-UE3UBMCP.js → chunk-RLMX75EM.js}

@@ -1,6 +1,6 @@
 import {
   tryLegacyPackageOperation
-} from "./chunk-4E42D7UF.js";
+} from "./chunk-ATSKPZUK.js";
 import {
   spawnStream
 } from "./chunk-ZHH3BEWU.js";
@@ -8,7 +8,7 @@ import {
   detectPackageManager,
   normalizeCommandOutput,
   safeResolveReal
-} from "./chunk-D33S4RQG.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-UE3UBMCP.js.map
+//# sourceMappingURL=chunk-RLMX75EM.js.map

package/dist/{chunk-3U4H6ZRF.js → chunk-RNWUCZ2U.js}

@@ -1,15 +1,15 @@
 import {
   getIndexState,
   searchCodebaseIndex
-} from "./chunk-VZLX2JZG.js";
+} from "./chunk-DP6CXMJZ.js";
 import {
   codebaseIndexDirOverride
-} from "./chunk-BLCSYKNN.js";
+} from "./chunk-KY4PXYSO.js";
 import {
   isBinaryBuffer,
   safeResolveReal,
   sha256hex
-} from "./chunk-D33S4RQG.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-3U4H6ZRF.js.map
+//# sourceMappingURL=chunk-RNWUCZ2U.js.map