@wrongstack/plugins 1.0.8 → 1.0.10

package/dist/cost-tracker.js

@@ -64,6 +64,9 @@ function estimateCost(model, freshTokens, completionTokens, cachedTokens = 0) {
   const outputCost = completionTokens / 1e6 * pricing.output;
   return inputCost + outputCost;
 }
+function toFiniteNumber(value) {
+  return Number.isFinite(value) ? value : 0;
+}
 var plugin = {
   name: "cost-tracker",
   version: "0.1.0",
@@ -137,11 +140,12 @@ var plugin = {
         const input = v["input"];
         const output = v["output"];
         if (typeof input !== "number" || typeof output !== "number") continue;
+        if (!Number.isFinite(input) || !Number.isFinite(output)) continue;
         const cacheRead = v["cacheRead"];
         pricingOverrides[model.toLowerCase()] = {
           input,
           output,
-          ...typeof cacheRead === "number" ? { cacheRead } : {}
+          ...typeof cacheRead === "number" && Number.isFinite(cacheRead) ? { cacheRead } : {}
         };
       }
     }
@@ -154,11 +158,11 @@ var plugin = {
           if (!providerModels) continue;
           for (const [modelId, model] of Object.entries(providerModels)) {
             const cost = model?.cost;
-            if (cost && typeof cost.input === "number" && typeof cost.output === "number") {
+            if (cost && typeof cost.input === "number" && typeof cost.output === "number" && Number.isFinite(cost.input) && Number.isFinite(cost.output)) {
               bundledFromRegistry[modelId.toLowerCase()] = {
                 input: cost.input,
                 output: cost.output,
-                ...typeof cost.cache_read === "number" ? { cacheRead: cost.cache_read } : {}
+                ...typeof cost.cache_read === "number" && Number.isFinite(cost.cache_read) ? { cacheRead: cost.cache_read } : {}
               };
               hydrated += 1;
             }
@@ -186,14 +190,22 @@ var plugin = {
       const usage = payload.usage;
       const model = payload.ctx?.model ?? "unknown";
       const u = usage ?? {};
-      const cachedTokens = Number(u["cacheRead"] ?? u["cache_read_input_tokens"] ?? u["cached_prompt_tokens"] ?? 0) || 0;
-      const rawInput = Number(u["input"] ?? u["prompt_tokens"] ?? u["inputTokens"] ?? u["promptTokens"] ?? 0) || 0;
-      const rawCacheWrite = Number(u["cacheWrite"] ?? u["cache_creation_input_tokens"] ?? 0) || 0;
+      const cachedTokens = toFiniteNumber(
+        Number(u["cacheRead"] ?? u["cache_read_input_tokens"] ?? u["cached_prompt_tokens"] ?? 0)
+      );
+      const rawInput = toFiniteNumber(
+        Number(u["input"] ?? u["prompt_tokens"] ?? u["inputTokens"] ?? u["promptTokens"] ?? 0)
+      );
+      const rawCacheWrite = toFiniteNumber(
+        Number(u["cacheWrite"] ?? u["cache_creation_input_tokens"] ?? 0)
+      );
       const freshTokens = rawInput + rawCacheWrite;
       const promptTokens = freshTokens + cachedTokens;
-      const completionTokens = Number(
-        u["output"] ?? u["completion_tokens"] ?? u["outputTokens"] ?? u["completionTokens"] ?? 0
-      ) || 0;
+      const completionTokens = toFiniteNumber(
+        Number(
+          u["output"] ?? u["completion_tokens"] ?? u["outputTokens"] ?? u["completionTokens"] ?? 0
+        )
+      );
       const totalTokens = promptTokens + completionTokens;
       const costUsd = estimateCost(model, freshTokens, completionTokens, cachedTokens);
       const record = {

package/dist/cron/index.d.ts

@@ -6,7 +6,7 @@
  * - cron_list: List all scheduled jobs
  * - cron_cancel: Cancel a scheduled job
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 declare const plugin: Plugin;
 export default plugin;
 //# sourceMappingURL=index.d.ts.map

package/dist/cron.js

@@ -1,4 +1,5 @@
 // src/cron/index.ts
+import { ToolValidationError } from "@wrongstack/core/types";
 var COORDINATION_CRON_CAPABILITY = "coordination.cron";
 var API_VERSION = "^0.1.10";
 var state = {
@@ -167,16 +168,28 @@ var plugin = {
         const action = input["action"] ?? input["task"] ?? input["command"] ?? input["run"];
         const enabled = input["enabled"] ?? true;
         if (!name || typeof name !== "string" || name.trim() === "") {
-          return { ok: false, error: "name is required and must be a non-empty string" };
+          throw new ToolValidationError({
+            message: "name is required and must be a non-empty string",
+            field: "name"
+          });
         }
         if (Number.isNaN(intervalMs) || rawInterval === void 0 || rawInterval === null) {
-          return { ok: false, error: "intervalMs must be a number >= 1000" };
+          throw new ToolValidationError({
+            message: "intervalMs must be a number >= 1000",
+            field: "intervalMs"
+          });
+        }
+        if (typeof action !== "string" || action.trim() === "") {
+          throw new ToolValidationError({
+            message: "action is required and must be a non-empty string",
+            field: "action"
+          });
         }
         if (state.jobs.has(name)) {
-          return { ok: false, error: `Cron job '${name}' already exists. Use cron_cancel first.` };
+          throw new Error(`Cron job '${name}' already exists. Use cron_cancel first.`);
         }
         if (state.jobs.size >= maxConcurrent) {
-          return { ok: false, error: `Maximum concurrent jobs (${maxConcurrent}) reached.` };
+          throw new Error(`Maximum concurrent jobs (${maxConcurrent}) reached.`);
         }
         const job = {
           name,
@@ -242,7 +255,7 @@ var plugin = {
       async execute(input) {
         const name = input["name"] ?? input["jobName"] ?? input["job_name"] ?? input["job"] ?? input["id"];
         if (!name || typeof name !== "string" || !state.jobs.has(name)) {
-          return { ok: false, error: `No cron job named '${name}'` };
+          throw new Error(`No cron job named '${name}'`);
         }
         cancelJob(name);
         api.metrics.gauge("cron_active_jobs", state.jobs.size);

package/dist/dead-code-detector/index.d.ts

@@ -26,7 +26,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 declare const plugin: Plugin;
 export default plugin;
 //# sourceMappingURL=index.d.ts.map

package/dist/dead-code-detector.js

@@ -15,6 +15,7 @@ var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "defau
 // src/dead-code-detector/index.ts
 import { readdir, readFile, stat } from "node:fs/promises";
 import { extname, join, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -168,14 +169,8 @@ async function scan(root, depth, cfg) {
 }
 async function resolveScanRoot(rawPath) {
   const resolved = resolve(process.cwd(), rawPath);
-  try {
-    const stats = await stat(resolved);
-    if (!stats.isDirectory()) {
-      return resolve(resolved, "..");
-    }
-  } catch {
-  }
-  return resolved;
+  const stats = await stat(resolved);
+  return stats.isDirectory() ? resolved : resolve(resolved, "..");
 }
 function toPosix(p) {
   return p.replace(/\\/g, "/");
@@ -294,22 +289,29 @@ Consider removing the export if it is not part of the public API.`;
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "dead-code-detector is disabled" };
+        if (!cfg.enabled) throw new Error("dead-code-detector is disabled");
         const raw = input ?? {};
         const rawPath = (typeof input.path === "string" && input.path.trim().length > 0 ? input.path.trim() : void 0) ?? (typeof raw["directory"] === "string" ? raw["directory"] : void 0) ?? (typeof raw["dir"] === "string" ? raw["dir"] : void 0) ?? (typeof raw["SearchDirectory"] === "string" ? raw["SearchDirectory"] : void 0) ?? (typeof raw["filePath"] === "string" ? raw["filePath"] : void 0) ?? (typeof raw["TargetFile"] === "string" ? raw["TargetFile"] : void 0) ?? (typeof raw["targetFile"] === "string" ? raw["targetFile"] : void 0) ?? (typeof raw["file_path"] === "string" ? raw["file_path"] : void 0) ?? (typeof raw["file"] === "string" ? raw["file"] : void 0) ?? ".";
         const rawDepth = typeof input.depth === "number" ? input.depth : cfg.defaultDepth;
         const depth = Math.max(0, Math.min(Math.floor(rawDepth), cfg.maxDepth));
         if (!(0, runtime_exports.withinProject)(rawPath)) {
-          return { ok: false, error: "scan path is outside the project root" };
+          throw new ToolValidationError({
+            message: "scan path is outside the project root",
+            field: "path"
+          });
         }
         state.scanCount += 1;
-        const scanRoot = await resolveScanRoot(rawPath);
         let result;
+        let scanRoot;
         try {
+          scanRoot = await resolveScanRoot(rawPath);
           result = await scan(scanRoot, depth, cfg);
         } catch (err) {
           state.errorCount += 1;
-          return { ok: false, error: String(err) };
+          throw new Error(
+            `dead_code_scan failed for ${rawPath}: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
         return {
           ok: true,

package/dist/duplicate-code-detector/index.d.ts

@@ -25,7 +25,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 /**
  * Tunable budgets that bound the hook fingerprint index. Keeping the defaults
  * together makes the file, fingerprint, and byte limits explicit.

package/dist/duplicate-code-detector.js

@@ -15,6 +15,7 @@ var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "defau
 // src/duplicate-code-detector/index.ts
 import { readFile, realpath, stat } from "node:fs/promises";
 import { extname, isAbsolute, relative, resolve, sep } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -442,19 +443,26 @@ var plugin = {
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "duplicate-code-detector is disabled" };
+        if (!cfg.enabled) throw new Error("duplicate-code-detector is disabled");
         const raw = input;
         const rawPath = (typeof raw["path"] === "string" ? raw["path"] : void 0) ?? (typeof raw["directory"] === "string" ? raw["directory"] : void 0) ?? (typeof raw["dir"] === "string" ? raw["dir"] : void 0) ?? (typeof raw["SearchDirectory"] === "string" ? raw["SearchDirectory"] : void 0) ?? (typeof raw["filePath"] === "string" ? raw["filePath"] : void 0) ?? (typeof raw["file_path"] === "string" ? raw["file_path"] : void 0) ?? (typeof raw["TargetFile"] === "string" ? raw["TargetFile"] : void 0) ?? (typeof raw["targetFile"] === "string" ? raw["targetFile"] : void 0) ?? (typeof raw["file"] === "string" ? raw["file"] : void 0) ?? ".";
         if (!(0, runtime_exports.withinProject)(rawPath)) {
-          return { ok: false, error: "scan path is outside the project root" };
+          throw new ToolValidationError({
+            message: "scan path is outside the project root",
+            field: "path"
+          });
         }
         state.scanCount += 1;
         let result;
         try {
+          await stat(resolve(process.cwd(), rawPath));
           result = await scanPath(rawPath, cfg);
         } catch (err) {
           state.errorCount += 1;
-          return { ok: false, error: String(err) };
+          throw new Error(
+            `detect_duplicate_code failed for ${rawPath}: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
         state.findingCount += result.findings.length;
         return {

package/dist/feature-flag-tracker/index.d.ts

@@ -23,7 +23,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 export interface FeatureFlagUsage {
     flag: string;
     file: string;

package/dist/feature-flag-tracker.js

@@ -13,8 +13,9 @@ var __copyProps = (to, from, except, desc) => {
 var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "default"), secondTarget && __copyProps(secondTarget, mod, "default"));
 
 // src/feature-flag-tracker/index.ts
-import { readFile } from "node:fs/promises";
+import { readFile, stat } from "node:fs/promises";
 import { isAbsolute, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -246,19 +247,26 @@ Make sure flag behavior is intentional and consider updating flag inventory/docs
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "feature-flag-tracker is disabled" };
+        if (!cfg.enabled) throw new Error("feature-flag-tracker is disabled");
         const raw = input ?? {};
         const rawPath = (typeof input.path === "string" && input.path.trim().length > 0 ? input.path.trim() : void 0) ?? (typeof raw["directory"] === "string" ? raw["directory"] : void 0) ?? (typeof raw["SearchDirectory"] === "string" ? raw["SearchDirectory"] : void 0) ?? (typeof raw["dir"] === "string" ? raw["dir"] : void 0) ?? (typeof raw["TargetFile"] === "string" ? raw["TargetFile"] : void 0) ?? (typeof raw["targetFile"] === "string" ? raw["targetFile"] : void 0) ?? (typeof raw["filePath"] === "string" ? raw["filePath"] : void 0) ?? (typeof raw["file_path"] === "string" ? raw["file_path"] : void 0) ?? (typeof raw["file"] === "string" ? raw["file"] : void 0) ?? ".";
         if (!(0, runtime_exports.withinProject)(rawPath)) {
-          return { ok: false, error: "path is outside the project root" };
+          throw new ToolValidationError({
+            message: "path is outside the project root",
+            field: "path"
+          });
         }
         state.scanCount += 1;
         let result;
         try {
+          await stat(resolve(process.cwd(), rawPath));
           result = await scanPath(rawPath, cfg);
         } catch (err) {
           state.errorCount += 1;
-          return { ok: false, error: String(err) };
+          throw new Error(
+            `scan_feature_flags failed for ${rawPath}: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
         state.flagCount += result.usages.length;
         return {

package/dist/file-watcher/index.d.ts

@@ -6,7 +6,7 @@
  * - watch_stop: Stop a watch by ID
  * - watch_list: List all active watches
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 declare const plugin: Plugin;
 export default plugin;
 //# sourceMappingURL=index.d.ts.map

package/dist/file-watcher.js

@@ -15,6 +15,7 @@ var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "defau
 // src/file-watcher/index.ts
 import { watch as fsWatch } from "node:fs";
 import { join } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -209,11 +210,10 @@ var plugin = {
         let rawPaths;
         if (explicitPaths !== void 0) {
           if (!Array.isArray(explicitPaths)) {
-            return {
-              ok: false,
-              error: "paths must be an array of file/directory paths",
-              watch_id: null
-            };
+            throw new ToolValidationError({
+              message: "paths must be an array of file/directory paths",
+              field: "paths"
+            });
           }
           rawPaths = explicitPaths;
         } else {
@@ -221,55 +221,46 @@ var plugin = {
           rawPaths = Array.isArray(fallback) ? fallback : typeof fallback === "string" && fallback.trim().length > 0 ? [fallback.trim()] : void 0;
         }
         if (!rawPaths || !Array.isArray(rawPaths)) {
-          return {
-            ok: false,
-            error: "paths must be an array of file/directory paths",
-            watch_id: null
-          };
+          throw new ToolValidationError({
+            message: "paths must be an array of file/directory paths",
+            field: "paths"
+          });
         }
         const paths = [...new Set(rawPaths)];
         if (paths.length === 0) {
-          return {
-            ok: false,
-            error: "paths array is empty \u2014 provide at least one path",
-            watch_id: null
-          };
+          throw new ToolValidationError({
+            message: "paths array is empty \u2014 provide at least one path",
+            field: "paths"
+          });
         }
         if (paths.length > MAX_PATHS_PER_WATCH) {
-          return {
-            ok: false,
-            error: `a watch may contain at most ${MAX_PATHS_PER_WATCH} unique paths`,
-            watch_id: null
-          };
+          throw new ToolValidationError({
+            message: `a watch may contain at most ${MAX_PATHS_PER_WATCH} unique paths`,
+            field: "paths"
+          });
         }
         if (watches.size >= MAX_WATCH_GROUPS) {
-          return {
-            ok: false,
-            error: `active watch group limit reached (${MAX_WATCH_GROUPS})`,
-            watch_id: null
-          };
+          throw new Error(
+            `active watch group limit reached (${MAX_WATCH_GROUPS}); stop a watch with watch_stop first`
+          );
         }
         const activeFilesystemWatchers = [...watches.values()].reduce(
           (total, handle2) => total + handle2.watchers.length,
           0
         );
         if (activeFilesystemWatchers + paths.length > MAX_FILESYSTEM_WATCHERS) {
-          return {
-            ok: false,
-            error: `filesystem watcher limit reached (${MAX_FILESYSTEM_WATCHERS})`,
-            watch_id: null
-          };
+          throw new Error(
+            `filesystem watcher limit reached (${MAX_FILESYSTEM_WATCHERS}); stop a watch with watch_stop first`
+          );
         }
         const events = input["events"] ?? ["change", "add", "delete"];
         const recursive = input["recursive"] ?? true;
         const bad = paths.find((p) => !(0, runtime_exports.withinProject)(p));
         if (bad !== void 0) {
-          return {
-            ok: false,
-            error: `path is outside the project root: ${bad}`,
-            watch_id: null,
-            rejectedOutsideProject: true
-          };
+          throw new ToolValidationError({
+            message: `path is outside the project root: ${bad}`,
+            field: "paths"
+          });
         }
         const id = nextId();
         const handle = {
@@ -281,8 +272,15 @@ var plugin = {
           createdAt: (/* @__PURE__ */ new Date()).toISOString()
         };
         const watchedPaths = [];
+        const failedPaths = [];
         for (const p of paths) {
           if (safeWatchDir(p, recursive, handle)) watchedPaths.push(p);
+          else failedPaths.push(p);
+        }
+        if (watchedPaths.length === 0) {
+          throw new Error(
+            `could not watch any of the requested paths: ${failedPaths.join(", ")} (missing path or OS watcher limit)`
+          );
         }
         handle.paths = watchedPaths;
         watches.set(id, handle);
@@ -291,9 +289,10 @@ var plugin = {
           ok: true,
           watch_id: id,
           paths: watchedPaths,
+          ...failedPaths.length > 0 ? { failedPaths } : {},
           events,
           recursive,
-          message: `Started watching ${watchedPaths.length} path(s). Use watch_stop to cancel.`
+          message: `Started watching ${watchedPaths.length} path(s). Use watch_stop to cancel.` + (failedPaths.length > 0 ? ` Could not watch: ${failedPaths.join(", ")}.` : "")
         };
       }
     });
@@ -315,7 +314,7 @@ var plugin = {
         const watch_id = typeof rawId === "string" ? rawId.trim() : "";
         const handle = watches.get(watch_id);
         if (!handle) {
-          return { ok: false, error: `No active watch with ID: ${watch_id}` };
+          throw new Error(`No active watch with ID: ${watch_id}`);
         }
         for (const w of handle.watchers) {
           try {

package/dist/git-autocommit/index.d.ts

@@ -24,7 +24,7 @@
  * - For staging: use `git_autocommit` with `files` or `paths` (it stages automatically), or `bash` with `git add`.
  * - For status: use the built-in `git` tool with `command: "status"` or `command: "diff"`.
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 /**
  * Parse one `git status --porcelain` line into the path to stage.
  *

package/dist/git-autocommit.js

@@ -2,6 +2,7 @@
 import { execFile } from "node:child_process";
 import { existsSync } from "node:fs";
 import { resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 var API_VERSION = "^0.1.10";
 var commitCount = { value: 0 };
 var lastCommit = { hash: null, at: null };
@@ -382,7 +383,10 @@ var plugin = {
           const rawFiles = input["files"] ?? input["fileList"] ?? input["file_list"];
           if (rawFiles !== void 0) {
             if (!Array.isArray(rawFiles)) {
-              return { ok: false, error: "files must be an array of file paths" };
+              throw new ToolValidationError({
+                message: "files must be an array of file paths",
+                field: "files"
+              });
             }
             files = rawFiles;
           } else if (typeof (input["file"] ?? input["file_path"]) === "string" && String(input["file"] ?? input["file_path"]).trim().length > 0) {
@@ -396,20 +400,26 @@ var plugin = {
           const rawPaths = input["paths"] ?? input["pathList"] ?? input["path_list"];
           if (rawPaths !== void 0) {
             if (!Array.isArray(rawPaths)) {
-              return { ok: false, error: "paths must be an array of pathspec patterns" };
+              throw new ToolValidationError({
+                message: "paths must be an array of pathspec patterns",
+                field: "paths"
+              });
             }
             pathspecs = rawPaths.filter((p) => typeof p === "string" && p.length > 0);
             if (pathspecs.length === 0) {
-              return { ok: false, error: "paths must contain at least one non-empty pattern" };
+              throw new ToolValidationError({
+                message: "paths must contain at least one non-empty pattern",
+                field: "paths"
+              });
             }
           } else if (typeof input["path"] === "string" && input["path"].trim().length > 0) {
             pathspecs = [input["path"].trim()];
           }
           if (rawPaths !== void 0 && files && files.length > 0) {
-            return {
-              ok: false,
-              error: "Pass either files (exact paths) or paths (pathspec globs), not both \u2014 the other would be silently ignored."
-            };
+            throw new ToolValidationError({
+              message: "Pass either files (exact paths) or paths (pathspec globs), not both \u2014 the other would be silently ignored.",
+              field: "paths"
+            });
           }
           let commitScope;
           let staged = [];
@@ -417,10 +427,10 @@ var plugin = {
             try {
               await stageFiles(pathspecs);
             } catch (err) {
-              return {
-                ok: false,
-                error: `Failed to stage files matching paths: ${err instanceof Error ? err.message : String(err)}`
-              };
+              throw new Error(
+                `Failed to stage files matching paths: ${err instanceof Error ? err.message : String(err)}`,
+                { cause: err }
+              );
             }
             try {
               staged = await getScopedStagedFiles(pathspecs);
@@ -428,10 +438,9 @@ var plugin = {
               staged = [];
             }
             if (staged.length === 0) {
-              return {
-                ok: false,
-                error: "No changed files match the given paths \u2014 refusing to commit anything else."
-              };
+              throw new Error(
+                "No changed files match the given paths \u2014 refusing to commit anything else."
+              );
             }
             commitScope = staged;
             try {
@@ -443,10 +452,10 @@ var plugin = {
             try {
               commitScope = await stageFiles(files);
             } catch (err) {
-              return {
-                ok: false,
-                error: `Failed to stage files: ${err instanceof Error ? err.message : String(err)}`
-              };
+              throw new Error(
+                `Failed to stage files: ${err instanceof Error ? err.message : String(err)}`,
+                { cause: err }
+              );
             }
             try {
               staged = await getStagedFiles();
@@ -511,17 +520,16 @@ var plugin = {
                 message: `Would create: ${summary || "update code"}`
               };
             }
-            return {
-              ok: false,
-              error: "type is required and must be a valid conventional commit type"
-            };
+            throw new ToolValidationError({
+              message: "type is required and must be a valid conventional commit type",
+              field: "type"
+            });
           }
           const msg = generateCommitMessage(type, scope, summary || "update code", body);
           if (staged.length === 0) {
-            return {
-              ok: false,
-              error: 'Nothing staged. Pass files (exact paths) or paths (pathspec globs) to scope this commit, stage with git add beforehand, or set extensions["git-autocommit"].autoStage=true to allow staging every changed file (legacy whole-tree behavior).'
-            };
+            throw new Error(
+              'Nothing staged. Pass files (exact paths) or paths (pathspec globs) to scope this commit, stage with git add beforehand, or set extensions["git-autocommit"].autoStage=true to allow staging every changed file (legacy whole-tree behavior).'
+            );
           }
           let scopeWarning = null;
           if (commitScope) {
@@ -563,20 +571,19 @@ ${stagedDiff}
             if (drifted.length > 0) {
               const preview = drifted.slice(0, 10).join(", ");
               const suffix = drifted.length > 10 ? ` and ${drifted.length - 10} more` : "";
-              return {
-                ok: false,
-                error: `Working tree changed after staging for: ${preview}${suffix}. A scoped commit takes working-tree content, so committing now could include changes that were never staged or previewed. Re-run the tool to re-stage the current content.`
-              };
+              throw new Error(
+                `Working tree changed after staging for: ${preview}${suffix}. A scoped commit takes working-tree content, so committing now could include changes that were never staged or previewed. Re-run the tool to re-stage the current content.`
+              );
             }
           }
           let hash = "";
           try {
             hash = await commitWithMessage(msg, void 0, commitScope);
           } catch (err) {
-            return {
-              ok: false,
-              error: `Failed to commit: ${err instanceof Error ? err.message : String(err)}`
-            };
+            throw new Error(
+              `Failed to commit: ${err instanceof Error ? err.message : String(err)}`,
+              { cause: err }
+            );
           }
           api.log.info("git-autocommit: created commit", { hash, type, scope });
           commitCount.value += 1;
@@ -615,10 +622,8 @@ ${stagedDiff}
 \`\`\``
           };
         } catch (err) {
-          return {
-            ok: false,
-            error: `Uncaught error in git_autocommit: ${err instanceof Error ? err.message : String(err)}`
-          };
+          if (err instanceof Error) throw err;
+          throw new Error(`Uncaught error in git_autocommit: ${String(err)}`, { cause: err });
         }
       }
     });

package/dist/gitignore-guard/index.d.ts

@@ -52,7 +52,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 /**
  * Curated default list of build-artifact-looking names. High precision on
  * purpose: this hook fires on every write/edit, so the defaults avoid