@wrongstack/tools 1.0.8 → 1.0.10

package/dist/{chunk-O6AGFJBO.js → chunk-7V33PHT6.js}

@@ -1,6 +1,6 @@
 import {
   todoTool
-} from "./chunk-S6CGSFLX.js";
+} from "./chunk-DBG7UQGK.js";
 import {
   projectSessionPlanToKanban
 } from "./chunk-YZECCW3F.js";
@@ -124,13 +124,9 @@ var planTool = {
           sessionPlanPath.lastIndexOf("\\")
         );
         if (lastSep < 0) {
-          return {
-            ok: false,
-            message: `Cannot derive the project-scoped plan path: session plan path "${sessionPlanPath}" has no directory component.`,
-            plan: "",
-            count: 0,
-            open: 0
-          };
+          throw new Error(
+            `plan: cannot derive the project-scoped plan path: session plan path "${sessionPlanPath}" has no directory component.`
+          );
         }
         planPath = sessionPlanPath.slice(0, lastSep + 1) + "backlog.plan.json";
       }
@@ -138,13 +134,7 @@ var planTool = {
       planPath = sessionPlanPath;
     }
     if (typeof planPath !== "string" || !planPath) {
-      return {
-        ok: false,
-        message: "Plan storage path is not configured for this session.",
-        plan: "",
-        count: 0,
-        open: 0
-      };
+      throw new Error("plan: Plan storage path is not configured for this session.");
     }
     const sessionId = ctx.session?.id ?? "unknown";
     if (input.action === "show") {
@@ -155,9 +145,13 @@ var planTool = {
         updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
         items: []
       };
-      return mkResult(plan2, true, "Plan show ok.");
+      return mkResult(plan2, "Plan show ok.");
     }
     let early = null;
+    let failure;
+    const refuse = (message, field) => {
+      failure = field ? new ToolValidationError({ message: `plan: ${message}`, field }) : new Error(`plan: ${message}`);
+    };
     const taskifyMeta = { title: "", details: "" };
     let didTaskify = false;
     let todosToReplace = null;
@@ -170,7 +164,7 @@ var planTool = {
           case "add": {
             const title = input.title?.trim();
             if (!title) {
-              early = mkResult(p, false, "add requires `title`.");
+              refuse("add requires `title`.", "title");
               return p;
             }
             const { plan: updated } = addPlanItem(p, title, input.details?.trim() || void 0);
@@ -179,11 +173,7 @@ var planTool = {
           case "start":
           case "done": {
             if (!input.target) {
-              early = mkResult(
-                p,
-                false,
-                `${input.action} requires \`target\` (id|index|substring).`
-              );
+              refuse(`${input.action} requires \`target\` (id|index|substring).`, "target");
               return p;
             }
             const next = setPlanItemStatus(
@@ -192,52 +182,52 @@ var planTool = {
               input.action === "start" ? "in_progress" : "done"
             );
             if (next === p) {
-              early = mkResult(p, false, `No plan item matched "${input.target}".`);
+              refuse(`No plan item matched "${input.target}".`, "target");
               return p;
             }
             return next;
           }
           case "status": {
-            const VALID_PLAN_STATUSES = /* @__PURE__ */ new Set(["open", "in_progress", "done"]);
+            const VALID_PLAN_STATUSES = /* @__PURE__ */ new Set([
+              "open",
+              "in_progress",
+              "done"
+            ]);
             if (!input.target || !input.status) {
-              early = mkResult(
-                p,
-                false,
-                "status requires `target` (id|index|substring) and `status`."
+              refuse(
+                "status requires `target` (id|index|substring) and `status`.",
+                input.target ? "status" : "target"
               );
               return p;
             }
             if (!VALID_PLAN_STATUSES.has(input.status)) {
-              early = mkResult(
-                p,
-                false,
-                `status requires valid status ('open' | 'in_progress' | 'done'), got "${input.status}".`
+              refuse(
+                `status requires valid status ('open' | 'in_progress' | 'done'), got "${input.status}".`,
+                "status"
               );
               return p;
             }
             const next = setPlanItemStatus(p, input.target, input.status);
             if (next === p) {
-              early = mkResult(p, false, `No plan item matched "${input.target}".`);
+              refuse(`No plan item matched "${input.target}".`, "target");
               return p;
             }
             return next;
           }
           case "remove": {
             if (!input.target) {
-              early = mkResult(p, false, "remove requires `target` (id|index|substring).");
+              refuse("remove requires `target` (id|index|substring).", "target");
               return p;
             }
             const next = removePlanItem(p, input.target);
             if (next === p) {
-              early = mkResult(p, false, `No plan item matched "${input.target}".`);
+              refuse(`No plan item matched "${input.target}".`, "target");
               return p;
             }
             const nextIds = new Set(next.items.map((item) => item.id));
             const removed = p.items.find((item) => !nextIds.has(item.id));
             if (removed?.status !== "done") {
-              early = mkResult(
-                p,
-                false,
+              refuse(
                 `Plan item "${removed?.title ?? input.target}" is not done and cannot be removed. Complete it first.`
               );
               return p;
@@ -246,22 +236,17 @@ var planTool = {
           }
           case "promote": {
             if (!input.target) {
-              early = mkResult(
-                p,
-                false,
-                `${input.action} requires \`target\` (id|index|substring).`
-              );
+              refuse(`${input.action} requires \`target\` (id|index|substring).`, "target");
               return p;
             }
             const derived = deriveTodosFromPlanItem(p, input.target, input.subtasks);
             if (!derived) {
-              early = mkResult(p, false, `No plan item matched "${input.target}".`);
+              refuse(`No plan item matched "${input.target}".`, "target");
               return p;
             }
             todosToReplace = derived.todos;
             early = mkResult(
               derived.plan,
-              true,
               `${input.action} ok \u2014 ${derived.todos.length} todo(s) created.`,
               derived.todos
             );
@@ -270,12 +255,12 @@ var planTool = {
           case "template_use": {
             const templateName = input.template?.trim();
             if (!templateName) {
-              early = mkResult(p, false, "template_use requires `template` name.");
+              refuse("template_use requires `template` name.", "template");
               return p;
             }
             const template = getPlanTemplate(templateName);
             if (!template) {
-              early = mkResult(p, false, `Unknown template "${templateName}".`);
+              refuse(`Unknown template "${templateName}".`, "template");
               return p;
             }
             let updated = p;
@@ -284,28 +269,19 @@ var planTool = {
             }
             early = mkResult(
               updated,
-              true,
               `Applied template "${template.name}" \u2014 ${template.items.length} items added.`
             );
             return updated;
           }
           case "clear":
             if (p.items.some((item) => item.status !== "done")) {
-              early = mkResult(
-                p,
-                false,
-                "Plan contains unfinished items and cannot be cleared. Complete them first."
-              );
+              refuse("Plan contains unfinished items and cannot be cleared. Complete them first.");
               return p;
             }
             return clearPlan(p);
           case "taskify": {
             if (!input.target) {
-              early = mkResult(
-                p,
-                false,
-                "taskify requires `target` (plan item id|index|substring)."
-              );
+              refuse("taskify requires `target` (plan item id|index|substring).", "target");
               return p;
             }
             let itemIdx = p.items.findIndex((it) => it.id === input.target);
@@ -320,7 +296,7 @@ var planTool = {
               itemIdx = p.items.findIndex((it) => it.title.toLowerCase().includes(lower));
             }
             if (itemIdx === -1 || !p.items[itemIdx]) {
-              early = mkResult(p, false, `No plan item matched "${input.target}".`);
+              refuse(`No plan item matched "${input.target}".`, "target");
               return p;
             }
             const item = p.items[itemIdx];
@@ -330,20 +306,18 @@ var planTool = {
             break;
           }
           default:
-            early = mkResult(p, false, `Unknown action "${input.action}".`);
+            refuse(`Unknown action "${input.action}".`, "action");
             return p;
         }
         return p;
       });
     } catch (err) {
-      return {
-        ok: false,
-        message: `Plan change not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
-        plan: "",
-        count: 0,
-        open: 0
-      };
+      throw new Error(
+        `plan: change not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
+        { cause: err }
+      );
     }
+    if (failure) throw failure;
     ctx.meta["plan.path.resolved"] = planPath;
     await projectSessionPlanToKanban(ctx.projectRoot, plan.items, sessionId);
     if (todosToReplace) {
@@ -355,7 +329,7 @@ var planTool = {
     if (didTaskify) {
       const taskPathRaw = ctx.meta["task.path"];
       if (typeof taskPathRaw !== "string" || !taskPathRaw) {
-        return mkResult(plan, false, "Task storage path not configured \u2014 cannot taskify.");
+        throw new Error("plan: Task storage path not configured \u2014 cannot taskify.");
       }
       let taskPath = taskPathRaw;
       if (input.scope === "project") {
@@ -380,28 +354,26 @@ var planTool = {
         ctx.meta["task.path.resolved"] = taskPath;
         return mkResult(
           plan,
-          true,
           `taskify ok \u2014 added "${taskifyMeta.title}" to tasks.
 ${formatTaskList(taskFile.tasks)}`
         );
       } catch (err) {
-        return mkResult(
-          plan,
-          false,
-          `taskify: task not saved \u2014 ${err instanceof Error ? err.message : String(err)}`
+        throw new Error(
+          `plan: taskify: task not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
+          { cause: err }
         );
       }
     }
-    return mkResult(plan, true, `Plan ${input.action} ok.`);
+    return mkResult(plan, `Plan ${input.action} ok.`);
   }
 };
-function mkResult(plan, ok, message, todos) {
+function mkResult(plan, message, todos) {
   let open = 0;
   for (let i = 0; i < plan.items.length; i++) {
     if (plan.items[i].status !== "done") open++;
   }
   const result = {
-    ok,
+    ok: true,
     message,
     plan: formatPlan(plan),
     count: plan.items.length,
@@ -414,4 +386,4 @@ function mkResult(plan, ok, message, todos) {
 export {
   planTool
 };
-//# sourceMappingURL=chunk-O6AGFJBO.js.map
+//# sourceMappingURL=chunk-7V33PHT6.js.map

package/dist/{chunk-CAF4L6KN.js → chunk-C3WP6ZOT.js}

@@ -54,15 +54,7 @@ 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;
-    if (signal.aborted) {
-      return {
-        source: "none",
-        entries: [],
-        total: 0,
-        truncated: false,
-        stream_mode: false
-      };
-    }
+    signal.throwIfAborted();
     const VALID_SINCE = /* @__PURE__ */ new Set(["1h", "6h", "24h", "all"]);
     if (input.since !== void 0 && !VALID_SINCE.has(input.since)) {
       throw new ToolValidationError({
@@ -119,61 +111,42 @@ var logsTool = {
       }
       return await fileLogs(resolved, lines, filterRe);
     }
-    return {
-      source: "none",
-      entries: [],
-      total: 0,
-      truncated: false,
-      stream_mode: false
-    };
+    throw new ToolValidationError({
+      message: "logs: provide `service` (Docker container) or `path` (log file)",
+      field: "path"
+    });
   }
 };
 async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
+  if (!/^[a-zA-Z0-9][a-zA-Z0-9._:-]+$/.test(service)) {
+    throw new ToolValidationError({
+      message: `logs: invalid Docker container name "${service}"`,
+      field: "service"
+    });
+  }
   const args = ["logs"];
   if (lines > 0) args.push("--tail", String(lines));
   if (since && since !== "all") {
     const sinceMap = { "1h": "1h", "6h": "6h", "24h": "24h" };
     args.push("--since", sinceMap[since] ?? "1h");
   }
-  if (!/^[a-zA-Z0-9][a-zA-Z0-9._:-]+$/.test(service)) {
-    return {
-      source: `docker:${service}`,
-      entries: [],
-      total: 0,
-      truncated: false,
-      stream_mode: false
-    };
-  }
   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) => {
+  signal.throwIfAborted();
+  return new Promise((resolve, reject) => {
     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}`,
-      entries: [],
-      total: 0,
-      truncated: false,
-      stream_mode: false
-    });
-    const finish = (result) => {
+    const settle = (fn) => {
       if (settled) return;
       settled = true;
       clearTimeout(timer);
-      resolve(result);
+      fn();
     };
+    const finish = (result) => settle(() => resolve(result));
+    const fail = (err) => settle(() => reject(err));
     const child = spawn("docker", args, {
       cwd,
       signal,
@@ -183,7 +156,11 @@ async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
     });
     const timer = setTimeout(() => {
       child.kill("SIGTERM");
-      finish(empty());
+      fail(
+        new Error(
+          `logs: docker logs did not finish within ${DOCKER_LOGS_TIMEOUT_MS}ms (is the Docker daemon running?)`
+        )
+      );
     }, DOCKER_LOGS_TIMEOUT_MS);
     child.stdout?.on("data", (c) => {
       if (stdout.length < MAX) {
@@ -207,7 +184,15 @@ async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
         JSON.stringify({ level: "debug", event: "pipe_error", stream: "stderr", error: e.message })
       );
     });
-    child.on("close", () => {
+    child.on("close", (code) => {
+      if (code !== 0) {
+        fail(
+          new Error(
+            `logs: docker logs exited with code ${code ?? "null"}: ${stderr.trim() || "no output"}`
+          )
+        );
+        return;
+      }
       const output = stdout + stderr;
       const entries = parseLogLines(output, filterRe);
       finish({
@@ -218,7 +203,10 @@ async function dockerLogs(service, lines, filterRe, cwd, signal, since) {
         stream_mode: false
       });
     });
-    child.on("error", () => finish(empty()));
+    child.on(
+      "error",
+      (err) => fail(new Error(`logs: could not run docker: ${err.message}`, { cause: err }))
+    );
   });
 }
 var DOCKER_LOGS_TIMEOUT_MS = 3e3;
@@ -299,4 +287,4 @@ function parseLine(line) {
 export {
   logsTool
 };
-//# sourceMappingURL=chunk-CAF4L6KN.js.map
+//# sourceMappingURL=chunk-C3WP6ZOT.js.map

package/dist/{chunk-KMXPP32M.js → chunk-C6ZOXNXV.js}

@@ -6,7 +6,7 @@ import {
 } from "./chunk-ZBMANLYH.js";
 import {
   enqueueReindex
-} from "./chunk-TU6UVATU.js";
+} from "./chunk-HYHE4VNA.js";
 import {
   isBinaryBuffer,
   safeResolveReal,
@@ -18,6 +18,7 @@ import {
 import { spawn } from "node:child_process";
 import * as fs from "node:fs/promises";
 import * as path from "node:path";
+import { StringDecoder } from "node:string_decoder";
 import { ToolValidationError } from "@wrongstack/core/types";
 import {
   atomicWrite,
@@ -287,6 +288,13 @@ async function resolveFiles(filesInput, ctx, extraGlob) {
     const stat2 = await fs.stat(absPath).catch(() => null);
     if (stat2?.isFile()) {
       resolved.push(absPath);
+      continue;
+    }
+    if (!stat2 && !await fs.lstat(absPath).catch(() => null)) {
+      throw new ToolValidationError({
+        message: `replace: file not found "${p}"`,
+        field: "files"
+      });
     }
   }
   return [...new Set(resolved)];
@@ -343,9 +351,10 @@ function spawnRgFind(pattern, base) {
   const MAX_BUF_CHARS = 8 * 1024 * 1024;
   let buf = "";
   let truncated = false;
+  const stdoutDecoder = new StringDecoder("utf8");
   child.stdout?.on("data", (chunk) => {
     if (truncated) return;
-    buf += chunk.toString();
+    buf += stdoutDecoder.write(chunk);
     if (buf.length > MAX_BUF_CHARS) {
       truncated = true;
       buf = buf.slice(0, buf.lastIndexOf("\n") + 1);
@@ -360,6 +369,7 @@ function spawnRgFind(pattern, base) {
           reject(new Error(`rg exited with code ${code}`));
           return;
         }
+        if (!truncated) buf += stdoutDecoder.end();
         resolve(
           buf.split(/\r?\n/).map((s) => s.trim()).filter(Boolean)
         );
@@ -413,4 +423,4 @@ export {
   replaceTool,
   __resetRgDetectionForTests
 };
-//# sourceMappingURL=chunk-KMXPP32M.js.map
+//# sourceMappingURL=chunk-C6ZOXNXV.js.map