@wrongstack/tools 1.0.8 → 1.0.9

package/dist/{chunk-B62LKMWQ.js → chunk-OKZJJDOG.js}

@@ -1,6 +1,6 @@
 import {
   todoTool
-} from "./chunk-S6CGSFLX.js";
+} from "./chunk-DBG7UQGK.js";
 import {
   mirrorSessionTasksToKanban
 } from "./chunk-YZECCW3F.js";
@@ -138,13 +138,10 @@ var taskTool = {
       "planify"
     ]);
     if (!input?.action || !VALID_ACTIONS.has(input.action)) {
-      return {
-        ok: false,
-        message: `Unknown action "${input?.action}". Use replace | add | status | show | promote | planify.`,
-        count: 0,
-        completed: 0,
-        inProgress: 0
-      };
+      throw new ToolValidationError({
+        message: `task: Unknown action "${input?.action}". Use replace | add | status | show | promote | planify.`,
+        field: "action"
+      });
     }
     if (input.scope !== void 0 && input.scope !== "session" && input.scope !== "project") {
       throw new ToolValidationError({
@@ -167,13 +164,7 @@ var taskTool = {
       taskPath = sessionTaskPath;
     }
     if (typeof taskPath !== "string" || !taskPath) {
-      return {
-        ok: false,
-        message: "Task storage path not configured.",
-        count: 0,
-        completed: 0,
-        inProgress: 0
-      };
+      throw new Error("task: Task storage path not configured.");
     }
     const sessionId = ctx.session?.id ?? "unknown";
     if (input.action === "show") {
@@ -189,7 +180,10 @@ var taskTool = {
         inProgress: p2.inProgress
       };
     }
-    let early = null;
+    let failure;
+    const refuse = (message, field) => {
+      failure = field ? new ToolValidationError({ message: `task: ${message}`, field }) : new Error(`task: ${message}`);
+    };
     const promoteMeta = { count: 0, title: "" };
     const planifyMeta = { title: "", details: "" };
     let didPlanify = false;
@@ -202,13 +196,7 @@ var taskTool = {
             break;
           case "replace": {
             if (!Array.isArray(input.tasks)) {
-              early = {
-                ok: false,
-                message: "action=replace requires `tasks` array.",
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse("action=replace requires `tasks` array.", "tasks");
               return f;
             }
             const newIds = new Set(input.tasks.map((t) => t.id));
@@ -223,25 +211,20 @@ var taskTool = {
                   })
                 )
               ];
-              early = {
-                ok: false,
-                message: `action=replace has duplicate task IDs: ${dupes.join(", ")}. Each task id must be unique.`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(
+                `action=replace has duplicate task IDs: ${dupes.join(", ")}. Each task id must be unique.`,
+                "tasks"
+              );
               return f;
             }
             const omittedUnfinished = f.tasks.filter(
               (task) => task.status !== "completed" && !newIds.has(task.id)
             );
             if (omittedUnfinished.length > 0) {
-              early = {
-                ok: false,
-                message: `action=replace cannot omit unfinished tasks: ${omittedUnfinished.map((task) => task.id).join(", ")}. Complete them first.`,
-                count: f.tasks.length,
-                ...computeTaskItemProgress(f.tasks)
-              };
+              refuse(
+                `action=replace cannot omit unfinished tasks: ${omittedUnfinished.map((task) => task.id).join(", ")}. Complete them first.`,
+                "tasks"
+              );
               return f;
             }
             const taskStatusById = new Map(input.tasks.map((t) => [t.id, t.status]));
@@ -249,13 +232,10 @@ var taskTool = {
               if (t.dependsOn && t.dependsOn.length > 0) {
                 const missing = t.dependsOn.filter((d) => !newIds.has(d));
                 if (missing.length > 0) {
-                  early = {
-                    ok: false,
-                    message: `dependsOn validation failed: task "${t.id}" references unknown IDs: ${missing.join(", ")}`,
-                    count: 0,
-                    completed: 0,
-                    inProgress: 0
-                  };
+                  refuse(
+                    `dependsOn validation failed: task "${t.id}" references unknown IDs: ${missing.join(", ")}`,
+                    "tasks"
+                  );
                   return f;
                 }
               }
@@ -264,12 +244,9 @@ var taskTool = {
                   (dependencyId) => taskStatusById.get(dependencyId) !== "completed"
                 );
                 if (unmet.length > 0) {
-                  early = {
-                    ok: false,
-                    message: `dependency status validation failed: task "${t.id}" cannot be ${t.status} before completion of ${unmet.join(", ")}.`,
-                    count: f.tasks.length,
-                    ...computeTaskItemProgress(f.tasks)
-                  };
+                  refuse(
+                    `dependency status validation failed: task "${t.id}" cannot be ${t.status} before completion of ${unmet.join(", ")}.`
+                  );
                   return f;
                 }
               }
@@ -285,26 +262,17 @@ var taskTool = {
           case "add": {
             const t = input.task;
             if (!t?.title) {
-              early = {
-                ok: false,
-                message: "action=add requires `task` with at least `title`.",
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse("action=add requires `task` with at least `title`.", "task");
               return f;
             }
             if (t.dependsOn && t.dependsOn.length > 0) {
               const existingIds = new Set(f.tasks.map((e) => e.id));
               const missing = t.dependsOn.filter((d) => !existingIds.has(d));
               if (missing.length > 0) {
-                early = {
-                  ok: false,
-                  message: `dependsOn validation failed: unknown task IDs: ${missing.join(", ")}`,
-                  count: 0,
-                  completed: 0,
-                  inProgress: 0
-                };
+                refuse(
+                  `dependsOn validation failed: unknown task IDs: ${missing.join(", ")}`,
+                  "task"
+                );
                 return f;
               }
             }
@@ -336,33 +304,19 @@ var taskTool = {
               "completed"
             ]);
             if (!input.id || !input.status) {
-              early = {
-                ok: false,
-                message: "action=status requires `id` and `status`.",
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse("action=status requires `id` and `status`.", input.id ? "status" : "id");
               return f;
             }
             if (!VALID_TASK_STATUSES.has(input.status)) {
-              early = {
-                ok: false,
-                message: `action=status requires valid status ('pending' | 'in_progress' | 'blocked' | 'failed' | 'review' | 'completed'), got "${input.status}".`,
-                count: f.tasks.length,
-                ...computeTaskItemProgress(f.tasks)
-              };
+              refuse(
+                `action=status requires valid status ('pending' | 'in_progress' | 'blocked' | 'failed' | 'review' | 'completed'), got "${input.status}".`,
+                "status"
+              );
               return f;
             }
             const task = f.tasks.find((t) => t.id === input.id);
             if (!task) {
-              early = {
-                ok: false,
-                message: `Task "${input.id}" not found.`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(`Task "${input.id}" not found.`, "id");
               return f;
             }
             if (input.status === "in_progress" || input.status === "completed") {
@@ -370,12 +324,9 @@ var taskTool = {
                 (dependencyId) => f.tasks.find((candidate) => candidate.id === dependencyId)?.status !== "completed"
               );
               if (unmet.length > 0) {
-                early = {
-                  ok: false,
-                  message: `Task "${task.id}" cannot be ${input.status} before dependencies complete: ${unmet.join(", ")}.`,
-                  count: f.tasks.length,
-                  ...computeTaskItemProgress(f.tasks)
-                };
+                refuse(
+                  `Task "${task.id}" cannot be ${input.status} before dependencies complete: ${unmet.join(", ")}.`
+                );
                 return f;
               }
             }
@@ -386,35 +337,16 @@ var taskTool = {
           case "promote": {
             const target = (input.target ?? input.id)?.trim();
             if (!target) {
-              early = {
-                ok: false,
-                message: "action=promote requires `target` (task id, index, or title substring).",
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(
+                "action=promote requires `target` (task id, index, or title substring).",
+                "target"
+              );
               return f;
             }
             const idx = findTaskIndex(f.tasks, target);
-            if (idx === -1) {
-              early = {
-                ok: false,
-                message: `No task matched "${target}".`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
-              return f;
-            }
-            const match = f.tasks[idx];
+            const match = idx === -1 ? void 0 : f.tasks[idx];
             if (!match) {
-              early = {
-                ok: false,
-                message: `No task matched "${target}".`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(`No task matched "${target}".`, "target");
               return f;
             }
             if (match.status !== "completed" && match.status !== "failed") {
@@ -457,35 +389,16 @@ var taskTool = {
           case "planify": {
             const target = (input.target ?? input.id)?.trim();
             if (!target) {
-              early = {
-                ok: false,
-                message: "action=planify requires `target` (task id, index, or title substring).",
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(
+                "action=planify requires `target` (task id, index, or title substring).",
+                "target"
+              );
               return f;
             }
             const idx = findTaskIndex(f.tasks, target);
-            if (idx === -1) {
-              early = {
-                ok: false,
-                message: `No task matched "${target}".`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
-              return f;
-            }
-            const match = f.tasks[idx];
+            const match = idx === -1 ? void 0 : f.tasks[idx];
             if (!match) {
-              early = {
-                ok: false,
-                message: `No task matched "${target}".`,
-                count: 0,
-                completed: 0,
-                inProgress: 0
-              };
+              refuse(`No task matched "${target}".`, "target");
               return f;
             }
             planifyMeta.title = match.title;
@@ -494,26 +407,21 @@ var taskTool = {
             break;
           }
           default:
-            early = {
-              ok: false,
-              message: `Unknown action "${input.action}". Use replace | add | status | show | promote | planify.`,
-              count: 0,
-              completed: 0,
-              inProgress: 0
-            };
+            refuse(
+              `Unknown action "${input.action}". Use replace | add | status | show | promote | planify.`,
+              "action"
+            );
             return f;
         }
         return f;
       });
     } catch (err) {
-      return {
-        ok: false,
-        message: `Task change not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
-        count: 0,
-        completed: 0,
-        inProgress: 0
-      };
+      throw new Error(
+        `task: change not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
+        { cause: err }
+      );
     }
+    if (failure) throw failure;
     ctx.meta["task.path.resolved"] = taskPath;
     if (todosToReplace) {
       await todoTool.execute({ todos: todosToReplace }, ctx, {
@@ -521,46 +429,36 @@ var taskTool = {
       });
     }
     mirrorSessionTasksToKanban(ctx.projectRoot, file.tasks, sessionId);
-    if (early) return early;
     if (didPlanify) {
       const { title, details } = planifyMeta;
       const planPathRaw = ctx.meta["plan.path"];
       const prog = computeTaskItemProgress(file.tasks);
-      if (typeof planPathRaw === "string" && planPathRaw) {
-        let planPath = planPathRaw;
-        if (input.scope === "project") {
-          const lastSep = Math.max(planPath.lastIndexOf("/"), planPath.lastIndexOf("\\"));
-          planPath = lastSep >= 0 ? planPath.slice(0, lastSep + 1) + "backlog.plan.json" : "backlog.plan.json";
-        }
-        let formatted = "";
-        try {
-          await mutatePlan(planPath, sessionId, (pf) => {
-            const { plan: updated } = addPlanItem(pf, title, details || void 0);
-            formatted = formatPlan(updated);
-            return updated;
-          });
-          ctx.meta["plan.path.resolved"] = planPath;
-        } catch (err) {
-          return {
-            ok: false,
-            message: `planify: plan not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
-            count: file.tasks.length,
-            completed: prog.completed,
-            inProgress: prog.inProgress
-          };
-        }
-        return {
-          ok: true,
-          message: `planify ok \u2014 added "${title}" to plan.
-${formatted}`,
-          count: file.tasks.length,
-          completed: prog.completed,
-          inProgress: prog.inProgress
-        };
+      if (typeof planPathRaw !== "string" || !planPathRaw) {
+        throw new Error("task: Plan storage path not configured \u2014 cannot planify.");
       }
+      let planPath = planPathRaw;
+      if (input.scope === "project") {
+        const lastSep = Math.max(planPath.lastIndexOf("/"), planPath.lastIndexOf("\\"));
+        planPath = lastSep >= 0 ? planPath.slice(0, lastSep + 1) + "backlog.plan.json" : "backlog.plan.json";
+      }
+      let formatted = "";
+      try {
+        await mutatePlan(planPath, sessionId, (pf) => {
+          const { plan: updated } = addPlanItem(pf, title, details || void 0);
+          formatted = formatPlan(updated);
+          return updated;
+        });
+      } catch (err) {
+        throw new Error(
+          `task: planify: plan not saved \u2014 ${err instanceof Error ? err.message : String(err)}`,
+          { cause: err }
+        );
+      }
+      ctx.meta["plan.path.resolved"] = planPath;
       return {
-        ok: false,
-        message: "Plan storage path not configured \u2014 cannot planify.",
+        ok: true,
+        message: `planify ok \u2014 added "${title}" to plan.
+${formatted}`,
         count: file.tasks.length,
         completed: prog.completed,
         inProgress: prog.inProgress
@@ -582,4 +480,4 @@ ${formatTaskList(file.tasks)}` : file.tasks.length > 0 ? formatTaskList(file.tas
 export {
   taskTool
 };
-//# sourceMappingURL=chunk-B62LKMWQ.js.map
+//# sourceMappingURL=chunk-OKZJJDOG.js.map

package/dist/{chunk-HOFVYIPI.js → chunk-ON6ET5ER.js}

@@ -64,6 +64,21 @@ var globTool = {
       return { files: [], truncated: true };
     }
     const base = input.path ? await safeResolveReal(input.path, ctx) : ctx.cwd;
+    if (input.path) {
+      const baseStat = await fs.stat(base).catch(() => null);
+      if (!baseStat) {
+        throw new ToolValidationError({
+          message: `glob: path "${input.path}" does not exist`,
+          field: "path"
+        });
+      }
+      if (!baseStat.isDirectory()) {
+        throw new ToolValidationError({
+          message: `glob: path "${input.path}" is not a directory`,
+          field: "path"
+        });
+      }
+    }
     const rawLimit = typeof input.limit === "number" && !Number.isNaN(input.limit) ? input.limit : 1e3;
     const limit = Math.max(1, Math.min(Math.floor(rawLimit), 5e3));
     const isGitIgnored = await loadGitignoreMatcher(base);
@@ -188,4 +203,4 @@ var globTool = {
 export {
   globTool
 };
-//# sourceMappingURL=chunk-HOFVYIPI.js.map
+//# sourceMappingURL=chunk-ON6ET5ER.js.map

package/dist/{chunk-KF7GMVNU.js → chunk-OUERRFEM.js}

@@ -1,9 +1,9 @@
 import {
   tryLegacyCodeOperation
-} from "./chunk-3LQJXJDM.js";
+} from "./chunk-E7A4SLKR.js";
 import {
   spawnStream
-} from "./chunk-EMA2XENR.js";
+} from "./chunk-UVYRQKPZ.js";
 import {
   normalizeCommandOutput,
   safeResolveReal
@@ -16,7 +16,7 @@ var testTool = {
   name: "test",
   category: "Code Quality",
   description: "Execute the project's test suite. This is one of the most critical tools for validating that your changes are correct.",
-  usageHint: "ESSENTIAL BEFORE CONSIDERING WORK DONE:\n\n- Use `files` or `grep` to run only relevant tests during development.\n- `coverage: true` is useful when working on critical paths.\nRun tests frequently. A clean test run is usually required before the task can be considered complete.",
+  usageHint: "ESSENTIAL BEFORE CONSIDERING WORK DONE:\n\n- Use `files` or `grep` to run only relevant tests during development.\n- `coverage: true` is useful when working on critical paths.\n- Check `status`: `passed`, `failed`, or `no_tests` (no runner configured / no test files matched). `no_tests` is NOT a pass.\nRun tests frequently. A clean test run is usually required before the task can be considered complete.",
   permission: "confirm",
   mutating: false,
   icon: "test",
@@ -75,16 +75,26 @@ var testTool = {
       });
       if (bridge?.run) {
         const run = bridge.run;
+        if (run.status === "unavailable") {
+          throw new Error(
+            `test: ${bridge.language} test runner unavailable: ${run.error || run.output || "no detail"}`
+          );
+        }
+        const passed = run.summary.passed ?? 0;
+        const failed = run.summary.failed ?? 0;
+        const exitCode = run.exitCode ?? 0;
+        const rawOutput = run.output || run.error || "";
         yield {
           type: "final",
           output: {
             runner: bridge.language,
-            exit_code: run.exitCode ?? 0,
-            tests_run: (run.summary.passed ?? 0) + (run.summary.failed ?? 0),
-            passed: run.summary.passed ?? 0,
-            failed: run.summary.failed ?? 0,
+            status: resolveStatus(exitCode, passed + failed, rawOutput),
+            exit_code: exitCode,
+            tests_run: passed + failed,
+            passed,
+            failed,
             duration_ms: run.durationMs,
-            output: normalizeCommandOutput(run.output || run.error || ""),
+            output: normalizeCommandOutput(rawOutput),
             truncated: run.truncated
           }
         };
@@ -97,12 +107,14 @@ var testTool = {
         type: "final",
         output: {
           runner: "none",
-          exit_code: 0,
+          // Nothing ran: not a pass, and no exit code to report.
+          status: "no_tests",
+          exit_code: null,
           tests_run: 0,
           passed: 0,
           failed: 0,
           duration_ms: 0,
-          output: "No test runner found (vitest.config.ts, jest.config.js, .mocharc.json)",
+          output: "No tests: no test runner configured (looked for vitest.config.*, jest.config.*, .mocharc.*).",
           truncated: false
         }
       };
@@ -119,6 +131,9 @@ var testTool = {
       maxBytes: 2e5
     });
     const duration = Date.now() - start;
+    if (result.error) {
+      throw new Error(`test: failed to start ${detected}: ${result.error}`);
+    }
     yield { type: "final", output: parseResult(detected, result, duration) };
   }
 };
@@ -183,6 +198,11 @@ function buildArgs(runner, input) {
   }
   return args;
 }
+var NO_TESTS_PATTERN = /No test files found|No tests found|no test files|running 0 tests|^\s*0 passing/im;
+function resolveStatus(exitCode, testsRun, output) {
+  if (testsRun === 0 && NO_TESTS_PATTERN.test(output)) return "no_tests";
+  return exitCode === 0 ? "passed" : "failed";
+}
 function parseResult(runner, result, duration) {
   const out = result.stdout + result.stderr;
   let tests_run = 0;
@@ -212,6 +232,7 @@ function parseResult(runner, result, duration) {
 ${result.stderr}` : result.stdout || result.stderr || result.error || "";
   return {
     runner,
+    status: resolveStatus(result.exitCode, tests_run, out),
     exit_code: result.exitCode,
     tests_run,
     passed,
@@ -231,4 +252,4 @@ ${result.stderr}` : result.stdout || result.stderr || result.error || "";
 export {
   testTool
 };
-//# sourceMappingURL=chunk-KF7GMVNU.js.map
+//# sourceMappingURL=chunk-OUERRFEM.js.map