@wrongstack/tools 0.319.0 → 0.319.1

package/dist/{chunk-ODFVG54U.js → chunk-EADULOZV.js}

@@ -1,15 +1,16 @@
 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/format.ts
+import { ToolValidationError } from "@wrongstack/core/types";
 var formatTool = {
   name: "format",
   category: "Code Quality",
@@ -55,6 +56,14 @@ var formatTool = {
   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_FIXERS = /* @__PURE__ */ new Set(["biome", "prettier", "auto"]);
+    if (input.fixer !== void 0 && !VALID_FIXERS.has(input.fixer)) {
+      throw new ToolValidationError({
+        message: `format: unsupported fixer "${input.fixer}". Allowed fixers: biome, prettier, auto`,
+        field: "fixer"
+      });
+    }
     const fixer = input.fixer ?? "auto";
     if (fixer === "auto" && !input.files) {
       const op = input.check ? "format-check" : "format-write";
@@ -98,7 +107,7 @@ var formatTool = {
       text: `Running ${detected}\u2026`,
       data: { fixer: detected, check: !!input.check }
     };
-    const fileList = input.files ? (Array.isArray(input.files) ? input.files : input.files.split(",")).map((f) => f.trim()) : [];
+    const fileList = input.files ? (Array.isArray(input.files) ? input.files : input.files.split(",")).map((f) => f.trim().replace(/\\/g, "/")).filter(Boolean) : [];
     let args;
     if (detected === "prettier") {
       args = [input.check ? "--check" : "--write"];
@@ -116,33 +125,45 @@ var formatTool = {
     });
     const combinedOut = `${result.stdout}
 ${result.stderr}`;
-    const counts = parseFormatterCounts(detected, combinedOut);
+    const counts = parseFormatterCounts(detected, combinedOut, !!input.check);
+    const rawOutput = result.stdout && result.stderr ? `${result.stdout}
+${result.stderr}` : result.stdout || result.stderr || result.error || "";
     yield {
       type: "final",
       output: {
         fixer: detected,
         files_checked: counts.checked,
         files_changed: counts.changed,
-        output: normalizeCommandOutput(result.stdout || result.stderr || result.error || ""),
+        output: normalizeCommandOutput(rawOutput),
         truncated: result.truncated
       }
     };
   }
 };
-function parseFormatterCounts(fixer, output) {
+function parseFormatterCounts(fixer, output, isCheck = false) {
   if (fixer !== "biome") return { checked: void 0, changed: void 0 };
   const checkedMatch = /\b(?:Checked|Formatted)\s+(\d+)\s+files?\b/i.exec(output);
-  const changedMatch = /\bFixed\s+(\d+)\s+files?\b/i.exec(output);
+  const fixedMatch = /\bFixed\s+(\d+)\s+files?\b/i.exec(output);
+  const formattedCountMatch = /\bFormatted\s+(\d+)\s+files?\b/i.exec(output);
+  let changed;
+  if (isCheck) {
+    changed = 0;
+  } else if (fixedMatch?.[1] !== void 0) {
+    changed = Number(fixedMatch[1]);
+  } else if (formattedCountMatch?.[1] !== void 0) {
+    changed = Number(formattedCountMatch[1]);
+  }
   return {
     checked: checkedMatch?.[1] !== void 0 ? Number(checkedMatch[1]) : void 0,
-    changed: changedMatch?.[1] !== void 0 ? Number(changedMatch[1]) : void 0
+    changed
   };
 }
 async function detectFixer(cwd) {
   const fs = await import("node:fs/promises");
+  const { join } = await import("node:path");
   const exists = async (file) => {
     try {
-      await fs.stat(`${cwd}/${file}`);
+      await fs.stat(join(cwd, file));
       return true;
     } catch {
       return false;
@@ -152,20 +173,23 @@ async function detectFixer(cwd) {
   const PRETTIER_CONFIGS = [
     ".prettierrc",
     ".prettierrc.json",
+    ".prettierrc.json5",
     ".prettierrc.yml",
     ".prettierrc.yaml",
+    ".prettierrc.toml",
     ".prettierrc.js",
     ".prettierrc.cjs",
     ".prettierrc.mjs",
     "prettier.config.js",
     "prettier.config.cjs",
-    "prettier.config.mjs"
+    "prettier.config.mjs",
+    "prettier.config.ts"
   ];
   for (const cfg of PRETTIER_CONFIGS) {
     if (await exists(cfg)) return "prettier";
   }
   try {
-    const raw = await fs.readFile(`${cwd}/package.json`, "utf8");
+    const raw = await fs.readFile(join(cwd, "package.json"), "utf8");
     const pkg = JSON.parse(raw);
     if (pkg["prettier"] !== void 0) return "prettier";
   } catch {
@@ -176,4 +200,4 @@ async function detectFixer(cwd) {
 export {
   formatTool
 };
-//# sourceMappingURL=chunk-ODFVG54U.js.map
+//# sourceMappingURL=chunk-EADULOZV.js.map

package/dist/{chunk-6RNIFLG4.js → chunk-ELRRTWNP.js}

@@ -14,6 +14,8 @@ import {
   setActiveKit,
   setDesignOverrides
 } from "@wrongstack/core/design";
+import { ToolValidationError } from "@wrongstack/core/types";
+import { atomicWrite } from "@wrongstack/core/utils";
 async function resolveReal(p) {
   const resolved = path.resolve(p);
   let probe = resolved;
@@ -102,10 +104,33 @@ var designTool = {
     },
     required: []
   },
-  async execute(input, ctx) {
+  async execute(input, ctx, _opts) {
+    const signal = _opts?.signal ?? ctx?.signal;
+    signal?.throwIfAborted();
+    const VALID_ACTIONS = /* @__PURE__ */ new Set([
+      "list",
+      "use",
+      "foundations",
+      "set",
+      "tune",
+      "materialize",
+      "verify"
+    ]);
+    if (input.action !== void 0 && !VALID_ACTIONS.has(input.action)) {
+      throw new ToolValidationError({
+        message: `design: unknown action "${input.action}". Allowed actions: ${[...VALID_ACTIONS].join(", ")}`,
+        field: "action"
+      });
+    }
+    if (input.stack !== void 0 && !isDesignStack(input.stack)) {
+      throw new ToolValidationError({
+        message: `design: invalid stack "${input.stack}". Allowed stacks: web, react-native, flutter, swiftui, compose`,
+        field: "stack"
+      });
+    }
     const loader = getDesignKitLoader(ctx.projectRoot);
     const action = input.action ?? "list";
-    const stack = input.stack && isDesignStack(input.stack) ? input.stack : void 0;
+    const stack = input.stack;
     if (action === "foundations") {
       const text = await loader.foundationsText(stack);
       return { action, stack, output: text || "No foundations document is installed." };
@@ -231,7 +256,10 @@ Run \`design {action:"materialize"}\` to write the tuned tokens to your theme fi
       const abs = path.join(absParent, path.basename(absResolved));
       const rel = path.relative(root, abs);
       if (rel.startsWith("..") || path.isAbsolute(rel)) {
-        throw new Error(`design: materialize path "${result.path}" would escape the project root`);
+        throw new ToolValidationError({
+          message: `design: materialize path "${result.path}" would escape the project root`,
+          field: "out"
+        });
       }
       let exists = false;
       try {
@@ -253,7 +281,7 @@ ${result.content}
         };
       }
       await fs.mkdir(path.dirname(abs), { recursive: true });
-      await fs.writeFile(abs, result.content);
+      await atomicWrite(abs, result.content);
       return {
         action,
         kit: active.kit,
@@ -263,6 +291,7 @@ ${result.content}
       };
     }
     if (action === "verify") {
+      signal?.throwIfAborted();
       const active = await loadActiveKit(ctx.projectRoot);
       if (!active) {
         return {
@@ -270,12 +299,15 @@ ${result.content}
           output: 'No active kit to verify against. Pick one: `design {action:"use", kit:"<id>"}`.'
         };
       }
+      signal?.throwIfAborted();
       const rawTokens = await loader.readTokens(active.kit);
       if (!rawTokens) {
         return { action, kit: active.kit, output: `Kit "${active.kit}" has no tokens.json.` };
       }
+      signal?.throwIfAborted();
       const tokens = applyTokenOverrides(rawTokens, active.overrides);
-      const report = await runDesignVerify(ctx.projectRoot, tokens, input.files);
+      const normalizedFiles = input.files ? (Array.isArray(input.files) ? input.files : String(input.files).split(",")).map((f) => f.trim().replace(/\\/g, "/")).filter(Boolean) : void 0;
+      const report = await runDesignVerify(ctx.projectRoot, tokens, normalizedFiles);
       const pct = Math.round(report.score * 100);
       const top = report.violations.slice(0, 25).map((v) => `  ${v.file}:${v.line} \u2014 ${v.reason}: ${v.snippet}`).join("\n");
       const summary = `Adherence: ${pct}% on-palette across ${report.filesScanned} file(s). ${report.violations.length} violation(s).` + (report.violations.length ? `
@@ -302,4 +334,4 @@ Replace off-palette colors with kit tokens (or the materialized CSS vars / token
 export {
   designTool
 };
-//# sourceMappingURL=chunk-6RNIFLG4.js.map
+//# sourceMappingURL=chunk-ELRRTWNP.js.map

package/dist/{chunk-YZHK4TLF.js → chunk-FQNXYA6W.js}

@@ -3,11 +3,13 @@ import {
 } from "./chunk-Q5JPPCZC.js";
 import {
   safeResolveReal
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/logs.ts
 import { spawn } from "node:child_process";
+import { StringDecoder } from "node:string_decoder";
 import { buildChildEnv } from "@wrongstack/core/utils";
+import { ToolValidationError } from "@wrongstack/core/types";
 var logsTool = {
   name: "logs",
   category: "Logs",
@@ -51,12 +53,32 @@ var logsTool = {
   async execute(input, ctx, opts) {
     const cwd = input.cwd ? await safeResolveReal(input.cwd, ctx) : ctx.cwd;
     const signal = opts?.signal ?? ctx.signal ?? new AbortController().signal;
-    const lines = input.lines ?? 100;
+    if (signal.aborted) {
+      return {
+        source: "none",
+        entries: [],
+        total: 0,
+        truncated: false,
+        stream_mode: false
+      };
+    }
+    const VALID_SINCE = /* @__PURE__ */ new Set(["1h", "6h", "24h", "all"]);
+    if (input.since !== void 0 && !VALID_SINCE.has(input.since)) {
+      throw new ToolValidationError({
+        message: `logs: invalid since option "${input.since}". Allowed: 1h, 6h, 24h, all`,
+        field: "since"
+      });
+    }
+    const rawLines = typeof input.lines === "number" && !Number.isNaN(input.lines) ? input.lines : 100;
+    const lines = Math.max(0, Math.floor(rawLines));
     let filterRe = null;
     if (input.filter) {
       const compiled = compileUserRegex(input.filter, "i");
       if (!compiled.ok) {
-        throw new Error(`logs: ${compiled.reason}`);
+        throw new ToolValidationError({
+          message: `logs: ${compiled.reason}`,
+          field: "filter"
+        });
       }
       filterRe = compiled.regex;
     }
@@ -92,10 +114,21 @@ async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
     };
   }
   args.push("--timestamps", service);
+  if (signal.aborted) {
+    return Promise.resolve({
+      source: `docker:${service}`,
+      entries: [],
+      total: 0,
+      truncated: false,
+      stream_mode: false
+    });
+  }
   return new Promise((resolve) => {
     let stdout = "";
     let stderr = "";
     const MAX = 2e5;
+    const stdoutDecoder = new StringDecoder("utf8");
+    const stderrDecoder = new StringDecoder("utf8");
     let settled = false;
     const empty = () => ({
       source: `docker:${service}`,
@@ -122,10 +155,16 @@ async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
       finish(empty());
     }, DOCKER_LOGS_TIMEOUT_MS);
     child.stdout?.on("data", (c) => {
-      if (stdout.length < MAX) stdout += c.toString();
+      if (stdout.length < MAX) {
+        const text = stdoutDecoder.write(c);
+        stdout += text.slice(0, MAX - stdout.length);
+      }
     });
     child.stderr?.on("data", (c) => {
-      if (stderr.length < MAX) stderr += c.toString();
+      if (stderr.length < MAX) {
+        const text = stderrDecoder.write(c);
+        stderr += text.slice(0, MAX - stderr.length);
+      }
     });
     child.stdout?.on("error", (e) => {
       console.log(
@@ -229,4 +268,4 @@ function parseLine(line) {
 export {
   logsTool
 };
-//# sourceMappingURL=chunk-YZHK4TLF.js.map
+//# sourceMappingURL=chunk-FQNXYA6W.js.map

package/dist/{chunk-65EMYN5G.js → chunk-GMZYW52L.js}

@@ -1,10 +1,10 @@
 import {
   builtinToolsPack
-} from "./chunk-SYXW3Y5Z.js";
+} from "./chunk-DC4KMPJO.js";
 import {
   TIER1_TOOLS,
   TIER2_TOOLS
-} from "./chunk-VALQFAY2.js";
+} from "./chunk-OTINMOKK.js";
 
 // src/tool-tier.ts
 function toolNameSet(tools) {
@@ -51,4 +51,4 @@ export {
   selectBuiltinToolsForTier,
   registerBuiltinToolTier
 };
-//# sourceMappingURL=chunk-65EMYN5G.js.map
+//# sourceMappingURL=chunk-GMZYW52L.js.map

package/dist/{chunk-HBM4QNXU.js → chunk-IU4DV2PA.js}

@@ -1,7 +1,7 @@
 import {
   ensureInsideRoot,
   safeResolveReal
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/document.ts
 import * as fs from "node:fs/promises";
@@ -40,7 +40,9 @@ var documentTool = {
       cwd: { type: "string", description: "Working directory (default: cwd)" }
     }
   },
-  async execute(input, ctx) {
+  async execute(input, ctx, _opts) {
+    const signal = _opts?.signal ?? ctx?.signal;
+    signal?.throwIfAborted();
     const cwd = input.cwd ? await safeResolveReal(input.cwd, ctx) : ctx.cwd;
     const style = input.style ?? "jsdoc";
     const results = [];
@@ -52,6 +54,7 @@ var documentTool = {
       ctx
     ) : input.path ? [await safeResolveReal(input.path, ctx)] : [];
     for (const absPath of fileList) {
+      if (signal?.aborted) break;
       try {
         const content = await fs.readFile(absPath, "utf8");
         filesProcessed++;
@@ -67,7 +70,7 @@ var documentTool = {
       } catch (e) {
         results.push({
           path: absPath,
-          name: absPath.split("/").pop() ?? absPath,
+          name: path.basename(absPath),
           signature: "",
           docstring: "",
           status: "error",
@@ -83,6 +86,13 @@ var documentTool = {
     };
   }
 };
+function getLineNumber(content, index) {
+  let line = 1;
+  for (let i = 0; i < index; i++) {
+    if (content.charCodeAt(i) === 10) line++;
+  }
+  return line;
+}
 async function resolveFiles(filesInput, cwd, ctx) {
   const files = filesInput.split(",");
   const resolved = [];
@@ -117,7 +127,7 @@ function processFile(content, absPath, _style, _overwrite, target) {
         name: m[1],
         sig: m[2] ?? "",
         type: "function",
-        line: content.slice(0, m.index).split("\n").length
+        line: getLineNumber(content, m.index)
       });
     }
     for (const m of content.matchAll(arrowRegex)) {
@@ -126,7 +136,7 @@ function processFile(content, absPath, _style, _overwrite, target) {
         name: m[1],
         sig: m[2] ?? "",
         type: "arrow",
-        line: content.slice(0, m.index).split("\n").length
+        line: getLineNumber(content, m.index)
       });
     }
   }
@@ -137,7 +147,7 @@ function processFile(content, absPath, _style, _overwrite, target) {
         name: m[1],
         sig: "",
         type: "class",
-        line: content.slice(0, m.index).split("\n").length
+        line: getLineNumber(content, m.index)
       });
     }
   }
@@ -148,7 +158,7 @@ function processFile(content, absPath, _style, _overwrite, target) {
         name: m[1],
         sig: m[0] ?? "",
         type: "type",
-        line: content.slice(0, m.index).split("\n").length
+        line: getLineNumber(content, m.index)
       });
     }
   }
@@ -167,4 +177,4 @@ function processFile(content, absPath, _style, _overwrite, target) {
 export {
   documentTool
 };
-//# sourceMappingURL=chunk-HBM4QNXU.js.map
+//# sourceMappingURL=chunk-IU4DV2PA.js.map

package/dist/{chunk-COUNXG4X.js → chunk-IWS56OY7.js}

@@ -1,17 +1,18 @@
 import {
   enqueueReindex
-} from "./chunk-MRKWZCHZ.js";
+} from "./chunk-DP6CXMJZ.js";
 import {
   resolveRealInsideRoot,
   safeResolveReal,
   sha256hex
-} from "./chunk-QLF74MSV.js";
+} from "./chunk-ZUFHAHTE.js";
 
 // src/patch.ts
 import { spawn } from "node:child_process";
 import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 import { buildChildEnv, toErrorMessage } from "@wrongstack/core/utils";
 var patchTool = {
   name: "patch",
@@ -42,9 +43,15 @@ var patchTool = {
     required: ["patch"]
   },
   async execute(input, ctx, opts) {
-    if (!input?.patch) throw new Error("patch: patch content is required");
+    if (!input?.patch || typeof input.patch !== "string" || !input.patch.trim()) {
+      throw new ToolValidationError({
+        message: "patch: patch content is required and cannot be empty",
+        field: "patch"
+      });
+    }
     const signal = opts?.signal ?? ctx.signal ?? new AbortController().signal;
-    const strip = Math.max(1, input.strip ?? 1);
+    signal.throwIfAborted();
+    const strip = typeof input.strip === "number" && Number.isFinite(input.strip) && input.strip >= 0 ? Math.floor(input.strip) : 1;
     const dryRun = input.dry_run ?? false;
     const refuse = (message) => ({
       applied: 0,
@@ -196,7 +203,11 @@ function extractDiffTargets(patch) {
   const out = [];
   const clean = (raw) => {
     if (!raw) return "";
-    return (raw.length > 4096 ? raw.slice(0, 4096) : raw).trim();
+    let s = (raw.length > 4096 ? raw.slice(0, 4096) : raw).trim();
+    if (s.startsWith('"') && s.endsWith('"') || s.startsWith("'") && s.endsWith("'")) {
+      s = s.slice(1, -1).trim();
+    }
+    return s;
   };
   let lastOld;
   let inHunk = false;
@@ -269,6 +280,14 @@ function runPatch(args, cwd, signal, fallback) {
   });
 }
 function runPatchProcess(command, args, cwd, signal) {
+  if (signal.aborted) {
+    return Promise.resolve({
+      exitCode: 124,
+      stdout: "",
+      stderr: "Aborted",
+      unavailable: false
+    });
+  }
   return new Promise((resolve2) => {
     let stdout = "";
     let stderr = "";
@@ -280,6 +299,7 @@ function runPatchProcess(command, args, cwd, signal) {
       stdio: ["pipe", "pipe", "pipe"],
       windowsHide: true
     });
+    child.stdin?.end();
     child.stdout?.on("data", (c) => {
       stdout += c.toString();
     });
@@ -305,7 +325,10 @@ function extractPatchedFiles(output) {
   const files = [];
   const re = /(?:patching|checking) file (.+)/gi;
   for (const m of output.matchAll(re)) {
-    if (m[1]) files.push(m[1]);
+    if (m[1]) {
+      const sanitized = m[1].replace(/^['"`]|['"`\r\n]+$/g, "").trim();
+      if (sanitized) files.push(sanitized);
+    }
   }
   return files;
 }
@@ -313,4 +336,4 @@ function extractPatchedFiles(output) {
 export {
   patchTool
 };
-//# sourceMappingURL=chunk-COUNXG4X.js.map
+//# sourceMappingURL=chunk-IWS56OY7.js.map