@wrongstack/plugins 1.0.7 → 1.0.10

package/dist/accessibility-auditor/index.d.ts

@@ -24,7 +24,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 export type A11yRule = 'missing-alt' | 'missing-input-label' | 'low-contrast-placeholder' | 'missing-button-text' | 'duplicate-id';
 export interface A11yFinding {
     file: string;

package/dist/accessibility-auditor.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/accessibility-auditor/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 = {};
@@ -53,6 +54,13 @@ function readConfig(raw) {
     onWriteEdit: (r["onWriteEdit"] ?? r["on_write_edit"] ?? r["onSave"]) !== false
   };
 }
+async function assertPathExists(rawPath) {
+  try {
+    await stat(resolve(process.cwd(), rawPath));
+  } catch (err) {
+    throw new Error(`path not found: ${rawPath}`, { cause: err });
+  }
+}
 function normalizeExtensions(exts) {
   return exts.map((e) => e.startsWith(".") ? e.toLowerCase() : `.${e.toLowerCase()}`);
 }
@@ -381,12 +389,16 @@ var plugin = {
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "accessibility-auditor is disabled" };
+        if (!cfg.enabled) throw new Error("accessibility-auditor 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["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["targetFile"] === "string" ? raw["targetFile"] : void 0) ?? (typeof raw["file"] === "string" ? raw["file"] : void 0) ?? ".";
         if (!(0, runtime_exports.withinProject)(rawPath)) {
-          return { ok: false, error: "path must be inside the project" };
+          throw new ToolValidationError({
+            message: "path must be inside the project",
+            field: "path"
+          });
         }
+        await assertPathExists(rawPath);
         state.auditCount += 1;
         const result = await auditPath(rawPath, cfg);
         state.fileCount += result.fileCount;

package/dist/agent-handoff.js

@@ -238,8 +238,8 @@ var plugin = {
       category: "Coordination",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "agent-handoff is disabled" };
-        if (!mailbox) return { ok: false, error: "mailbox not available" };
+        if (!cfg.enabled) throw new Error("agent-handoff is disabled");
+        if (!mailbox) throw new Error("mailbox not available");
         const raw = input;
         const summary = (typeof raw["summary"] === "string" ? raw["summary"] : void 0) ?? (typeof raw["note"] === "string" ? raw["note"] : void 0) ?? (typeof raw["message"] === "string" ? raw["message"] : void 0) ?? (typeof raw["content"] === "string" ? raw["content"] : void 0) ?? (typeof raw["text"] === "string" ? raw["text"] : void 0) ?? (typeof raw["body"] === "string" ? raw["body"] : void 0);
         const task = (typeof raw["task"] === "string" ? raw["task"] : void 0) ?? (typeof raw["title"] === "string" ? raw["title"] : void 0) ?? (typeof raw["subject"] === "string" ? raw["subject"] : void 0);
@@ -266,10 +266,10 @@ var plugin = {
           return { ok: true, messageId: result.id, to: recipient };
         } catch (err) {
           state.errorCount += 1;
-          return {
-            ok: false,
-            error: err instanceof Error ? err.message : String(err)
-          };
+          throw new Error(
+            `handoff_note: mailbox send to "${recipient}" failed: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
       }
     });

package/dist/auto-doc/index.d.ts

@@ -14,7 +14,7 @@
  * the session default. Best-effort: any LLM failure (or a bad response)
  * falls back to the template for that entity — a doc comment always lands.
  */
-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/auto-doc.js

@@ -1,5 +1,6 @@
 // src/auto-doc/index.ts
 import { isAbsolute, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 var AUTO_DOC_API_VERSION = "^0.1.10";
 function resolveProjectPath(rawPath, cwd = process.cwd()) {
   if (typeof rawPath !== "string" || rawPath.length === 0) return null;
@@ -163,41 +164,37 @@ function injectDocComment(content, entity, doc) {
 async function runAutoDoc(input, api) {
   const rawInput = input;
   if (rawInput["files"] !== void 0 && !Array.isArray(rawInput["files"])) {
-    return {
-      ok: false,
-      error: "input.files must be an array of file paths",
-      filesProcessed: 0,
-      changes: []
-    };
+    throw new ToolValidationError({
+      message: "input.files must be an array of file paths",
+      field: "files"
+    });
   }
   const rawFiles = rawInput["files"] ?? rawInput["file"] ?? rawInput["path"] ?? rawInput["filePath"] ?? rawInput["TargetFile"] ?? rawInput["targetFile"];
   const files = Array.isArray(rawFiles) ? rawFiles.filter((f) => typeof f === "string" && f.trim().length > 0).map((f) => f.trim()) : typeof rawFiles === "string" && rawFiles.trim().length > 0 ? [rawFiles.trim()] : void 0;
   if (!files || !Array.isArray(files)) {
-    return {
-      ok: false,
-      error: "input.files must be an array of file paths",
-      filesProcessed: 0,
-      changes: []
-    };
+    throw new ToolValidationError({
+      message: "input.files must be an array of file paths",
+      field: "files"
+    });
   }
   if (files.length === 0) {
-    return {
-      ok: false,
-      error: "input.files is empty \u2014 provide at least one file path",
-      filesProcessed: 0,
-      changes: []
-    };
+    throw new ToolValidationError({
+      message: "input.files is empty \u2014 provide at least one file path",
+      field: "files"
+    });
   }
   const extConfig = api.config.extensions?.["auto-doc"] ?? {};
   const includeTypes = extConfig["includeTypes"] ?? false;
   const useLlm = (input.use_llm ?? rawInput["useLlm"] ?? extConfig["useLlm"] ?? false) === true && Boolean(api.llm);
   const maxLlmEntities = typeof extConfig["maxLlmEntities"] === "number" && extConfig["maxLlmEntities"] >= 0 ? extConfig["maxLlmEntities"] : 25;
   const results = [];
+  const skipped = [];
   let llmBudget = maxLlmEntities;
   for (const rawFile of files) {
     const safeFile = resolveProjectPath(rawFile);
     if (!safeFile) {
       api.log.warn(`auto-doc: skipped file outside project directory: ${rawFile}`);
+      skipped.push({ file: rawFile, reason: "outside project directory" });
       continue;
     }
     try {
@@ -205,8 +202,12 @@ async function runAutoDoc(input, api) {
       let content;
       try {
         content = readFileSync(safeFile, "utf-8");
-      } catch {
+      } catch (err) {
         api.log.warn(`auto-doc: could not read file ${safeFile}`);
+        skipped.push({
+          file: safeFile,
+          reason: `could not read: ${err instanceof Error ? err.message : String(err)}`
+        });
         continue;
       }
       const entities = parseSource(content);
@@ -246,11 +247,21 @@ async function runAutoDoc(input, api) {
       }
     } catch (err) {
       api.log.error(`auto-doc: error processing ${safeFile}: ${err}`);
+      skipped.push({
+        file: safeFile,
+        reason: err instanceof Error ? err.message : String(err)
+      });
     }
   }
+  if (skipped.length === files.length) {
+    throw new Error(
+      `auto_doc processed no files: ${skipped.map((s) => `${s.file} (${s.reason})`).join("; ")}`
+    );
+  }
   return {
     ok: true,
-    filesProcessed: files.length,
+    filesProcessed: files.length - skipped.length,
+    ...skipped.length > 0 ? { skipped } : {},
     changes: results,
     llm: useLlm ? { docs: state.llmDocs, fallbacks: state.llmFallbacks } : void 0
   };

package/dist/auto-i18n-extractor/index.d.ts

@@ -28,7 +28,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/auto-i18n-extractor.js

@@ -14,6 +14,7 @@ var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "defau
 
 // src/auto-i18n-extractor/index.ts
 import { readFileSync } from "node:fs";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -244,28 +245,31 @@ ${lines.join("\n")}${more}`;
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "auto-i18n-extractor is disabled" };
+        if (!cfg.enabled) throw new Error("auto-i18n-extractor is disabled");
         const raw = input;
         const rawPath = (typeof input.path === "string" && input.path.trim().length > 0 ? input.path.trim() : void 0) ?? (typeof raw["TargetFile"] === "string" && raw["TargetFile"].trim().length > 0 ? raw["TargetFile"].trim() : void 0) ?? (typeof raw["targetFile"] === "string" && raw["targetFile"].trim().length > 0 ? raw["targetFile"].trim() : void 0) ?? (typeof raw["filePath"] === "string" && raw["filePath"].trim().length > 0 ? raw["filePath"].trim() : void 0) ?? (typeof raw["file_path"] === "string" && raw["file_path"].trim().length > 0 ? raw["file_path"].trim() : void 0) ?? (typeof raw["file"] === "string" && raw["file"].trim().length > 0 ? raw["file"].trim() : void 0);
         const filePath = typeof rawPath === "string" ? rawPath.trim() : "";
         if (!filePath) {
-          return { ok: false, error: "path is required" };
+          throw new ToolValidationError({ message: "path is required", field: "path" });
         }
         if (!(0, runtime_exports.withinProject)(filePath)) {
-          return { ok: false, error: "path must be inside the project" };
+          throw new ToolValidationError({
+            message: "path must be inside the project",
+            field: "path"
+          });
         }
         const ext = fileExtension(filePath);
         if (!cfg.fileExtensions.includes(ext)) {
-          return {
-            ok: false,
-            error: `unsupported extension "${ext}"; allowed: ${cfg.fileExtensions.join(", ")}`
-          };
+          throw new ToolValidationError({
+            message: `unsupported extension "${ext}"; allowed: ${cfg.fileExtensions.join(", ")}`,
+            field: "path"
+          });
         }
         state.filesScanned += 1;
         const content = readSourceFile(filePath);
         if (content === null) {
           state.readErrors += 1;
-          return { ok: false, error: `could not read ${filePath}` };
+          throw new Error(`could not read ${filePath}`);
         }
         const extracted = extractStrings(content, cfg);
         state.stringsFound += extracted.length;

package/dist/branch-guard.js

@@ -114,13 +114,13 @@ async function detectUncommittedChanges(cwd, signal) {
 }
 function detectGitCommand(command) {
   const cmd = command.trim();
-  if (/\bgit\s+commit\b/.test(cmd)) {
+  if (/\bgit\s+commit(?![a-zA-Z0-9_-])/.test(cmd)) {
     return { type: "commit", snippet: cmd.slice(0, 120) };
   }
-  if (/\bgit\s+push\b/.test(cmd)) {
+  if (/\bgit\s+push(?![a-zA-Z0-9_-])/.test(cmd)) {
     return { type: "push", snippet: cmd.slice(0, 120) };
   }
-  if (/\bgit\s+merge\s/.test(cmd)) {
+  if (/\bgit\s+merge(?![a-zA-Z0-9_-])/.test(cmd)) {
     return { type: "merge", snippet: cmd.slice(0, 120) };
   }
   return null;

package/dist/changelog-writer/index.d.ts

@@ -43,7 +43,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 type Section = 'Added' | 'Changed' | 'Fixed' | 'Removed' | 'Security' | 'Documentation' | 'Maintenance';
 export interface ChangelogEntry {
     section: Section;

package/dist/changelog-writer.js

@@ -15,6 +15,7 @@ var __reExport = (target, mod, secondTarget) => (__copyProps(target, mod, "defau
 // src/changelog-writer/index.ts
 import { readFileSync, writeFileSync } from "node:fs";
 import { isAbsolute, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 
 // src/runtime/index.ts
 var runtime_exports = {};
@@ -318,11 +319,13 @@ var plugin = {
       category: "Docs",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "changelog-writer is disabled" };
+        if (!cfg.enabled) throw new Error("changelog-writer is disabled");
         const raw = input ?? {};
         const rawText = input.text ?? raw["message"] ?? raw["entry"] ?? raw["content"] ?? raw["description"] ?? raw["desc"] ?? raw["summary"] ?? raw["body"] ?? raw["note"] ?? raw["item"];
         const text = String(rawText ?? "").trim();
-        if (!text) return { ok: false, error: "entry text must not be empty" };
+        if (!text) {
+          throw new ToolValidationError({ message: "entry text must not be empty", field: "text" });
+        }
         const section = SECTION_ORDER.includes(input.section) ? input.section : "Changed";
         addEntry({ section, text, origin: "manual", when: (/* @__PURE__ */ new Date()).toISOString() });
         return { ok: true, pendingEntries: state.entries.length };
@@ -377,12 +380,12 @@ var plugin = {
       category: "Docs",
       mutating: true,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "changelog-writer is disabled" };
+        if (!cfg.enabled) throw new Error("changelog-writer is disabled");
         if (!cfg.filePath) {
-          return { ok: false, error: "filePath must stay within the current project directory" };
+          throw new Error("filePath must stay within the current project directory");
         }
         if (state.entries.length === 0) {
-          return { ok: false, error: "no pending entries \u2014 add some with changelog_add first" };
+          throw new Error("no pending entries \u2014 add some with changelog_add first");
         }
         const rawInput = input ?? {};
         const shouldPolish = input.polish === true || rawInput["use_llm"] === true || rawInput["useLlm"] === true || rawInput["ai"] === true || rawInput["use_ai"] === true;
@@ -390,16 +393,22 @@ var plugin = {
         let existing = null;
         try {
           existing = readFileSync(cfg.filePath, "utf-8");
-        } catch {
+        } catch (err) {
+          if (err.code !== "ENOENT") {
+            throw new Error(
+              `failed to read ${cfg.filePath}: ${err instanceof Error ? err.message : String(err)}`,
+              { cause: err }
+            );
+          }
           existing = null;
         }
         try {
           writeFileSync(cfg.filePath, mergeIntoChangelog(existing, block));
         } catch (err) {
-          return {
-            ok: false,
-            error: `failed to write ${cfg.filePath}: ${err instanceof Error ? err.message : String(err)}`
-          };
+          throw new Error(
+            `failed to write ${cfg.filePath}: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
         const written = state.entries.length;
         state.entries = [];

package/dist/checkpoint/index.d.ts

@@ -35,7 +35,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 export interface Snapshot {
     id: string;
     createdAt: string;

package/dist/checkpoint.js

@@ -1,6 +1,7 @@
 // src/checkpoint/index.ts
 import { mkdir, readFile, realpath, stat, writeFile } from "node:fs/promises";
 import { dirname, isAbsolute, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 var state = {
   snapshots: [],
   nextId: 1,
@@ -83,7 +84,13 @@ async function captureFile(path, maxBytes) {
     if (st.size > maxBytes) return "too-large";
     const content = await readFile(path, "utf-8");
     return { path, content, bytes: st.size };
-  } catch {
+  } catch (err) {
+    if (err.code !== "ENOENT") {
+      throw new Error(
+        `could not snapshot ${path}: ${err instanceof Error ? err.message : String(err)}`,
+        { cause: err }
+      );
+    }
     return { path, content: null, bytes: 0 };
   }
 }
@@ -225,7 +232,7 @@ var plugin = {
       category: "Safety",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "checkpoint is disabled" };
+        if (!cfg.enabled) throw new Error("checkpoint is disabled");
         let paths = [];
         const rawInput = input;
         const raw = rawInput["paths"] ?? rawInput["path"] ?? rawInput["files"] ?? rawInput["file"] ?? rawInput["filePath"] ?? rawInput["file_path"] ?? rawInput["TargetFile"] ?? rawInput["targetFile"];
@@ -234,16 +241,25 @@ var plugin = {
         } else if (Array.isArray(raw)) {
           paths = raw.filter((p) => typeof p === "string" && p.trim().length > 0);
         }
-        if (paths.length === 0) return { ok: false, error: "paths must not be empty" };
-        const files = [];
+        if (paths.length === 0) {
+          throw new ToolValidationError({ message: "paths must not be empty", field: "paths" });
+        }
         const rejectedOutsideProject = [];
-        let skipped = 0;
+        const safePaths = [];
         for (const p of paths) {
           const safePath = await resolveProjectPath(p);
-          if (!safePath) {
-            rejectedOutsideProject.push(p);
-            continue;
-          }
+          if (safePath) safePaths.push(safePath);
+          else rejectedOutsideProject.push(p);
+        }
+        if (rejectedOutsideProject.length > 0) {
+          throw new ToolValidationError({
+            message: `paths must stay within the current project directory: ${rejectedOutsideProject.join(", ")}`,
+            field: "paths"
+          });
+        }
+        const files = [];
+        let skipped = 0;
+        for (const safePath of safePaths) {
           const captured = await captureFile(safePath, cfg.maxFileBytes);
           if (captured === "too-large") {
             skipped += 1;
@@ -252,15 +268,10 @@ var plugin = {
           }
           files.push(captured);
         }
-        if (rejectedOutsideProject.length > 0) {
-          return {
-            ok: false,
-            error: "paths must stay within the current project directory",
-            rejectedOutsideProject
-          };
-        }
         if (files.length === 0) {
-          return { ok: false, error: "all files were skipped (too large)" };
+          throw new Error(
+            `all files were skipped (larger than maxFileBytes=${cfg.maxFileBytes}); nothing was snapshotted`
+          );
         }
         const snapshot = {
           id: `cp-${state.nextId++}`,
@@ -335,21 +346,18 @@ var plugin = {
       category: "Safety",
       mutating: true,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "checkpoint is disabled" };
+        if (!cfg.enabled) throw new Error("checkpoint is disabled");
         const raw = input ?? {};
         const rawId = (typeof input.id === "string" && input.id.trim().length > 0 ? input.id.trim() : void 0) ?? (typeof raw["snapshotId"] === "string" ? raw["snapshotId"] : void 0) ?? (typeof raw["snapshot_id"] === "string" ? raw["snapshot_id"] : void 0);
         const rawPath = (typeof input.path === "string" && input.path.trim().length > 0 ? input.path.trim() : 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);
         const snapshot = rawId ? state.snapshots.find((s) => s.id === rawId) : state.snapshots[state.snapshots.length - 1];
         if (!snapshot) {
-          return {
-            ok: false,
-            error: rawId ? `no snapshot with id "${rawId}"` : "no snapshots captured yet"
-          };
+          throw new Error(rawId ? `no snapshot with id "${rawId}"` : "no snapshots captured yet");
         }
         const targetPath = rawPath ? await resolveProjectPath(rawPath) ?? rawPath : null;
         const targets = targetPath ? snapshot.files.filter((f) => f.path === targetPath || f.path === rawPath) : snapshot.files;
         if (targets.length === 0) {
-          return { ok: false, error: `snapshot ${snapshot.id} has no entry for "${rawPath}"` };
+          throw new Error(`snapshot ${snapshot.id} has no entry for "${rawPath}"`);
         }
         const restored = [];
         const createdByTool = [];
@@ -371,8 +379,14 @@ var plugin = {
           state.restores += 1;
           api.metrics.counter("restores");
         }
+        if (errors.length > 0) {
+          const failed = errors.map((e) => `${e.path} (${e.error})`).join("; ");
+          throw new Error(
+            `checkpoint_restore ${snapshot.id}: ${errors.length} file(s) failed to restore: ${failed}` + (restored.length > 0 ? `. Restored: ${restored.join(", ")}` : "")
+          );
+        }
         return {
-          ok: errors.length === 0,
+          ok: true,
           snapshotId: snapshot.id,
           restored,
           notRestoredFileDidNotExist: createdByTool,

package/dist/code-metrics/index.d.ts

@@ -23,7 +23,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 /**
  * Heuristic cyclomatic-complexity formula (per file, not per function):
  *   control keywords (if | else if | for | while | switch | catch)

package/dist/code-metrics.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/code-metrics/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 = {};
@@ -257,19 +258,26 @@ var plugin = {
       category: "Diagnostics",
       mutating: false,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "code-metrics is disabled" };
+        if (!cfg.enabled) throw new Error("code-metrics 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) ?? ".";
         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.measureCount += 1;
         let result;
         try {
+          await stat(resolve(process.cwd(), rawPath));
           result = await measurePath(rawPath, cfg);
         } catch (err) {
           state.errorCount += 1;
-          return { ok: false, error: String(err) };
+          throw new Error(
+            `measure_code_metrics failed for ${rawPath}: ${err instanceof Error ? err.message : String(err)}`,
+            { cause: err }
+          );
         }
         state.fileCount += result.files.length;
         return {

package/dist/commit-validator.js

@@ -144,11 +144,11 @@ function parseCommitMessage(message, cfg) {
 }
 var GIT_MESSAGE_FLAG_RE = new RegExp(
   [
-    // -m "…" | -m '…' | -m=… ; --message "…" | --message='…' | --message=…
-    String.raw`(?:^|\s)(?:-m|--message)(?:\s+|=)"([^"]*)"`,
-    String.raw`(?:^|\s)(?:-m|--message)(?:\s+|=)'([^']*)'`,
+    // -m "…" | -am "…" | -m '…' | -m=… ; --message "…" | --message='…' | --message=…
+    String.raw`(?:^|\s)(?:-[a-zA-Z]*m|--message)(?:\s+|=)"([^"]*)"`,
+    String.raw`(?:^|\s)(?:-[a-zA-Z]*m|--message)(?:\s+|=)'([^']*)'`,
     // Bare value: stops at whitespace or a shell separator.
-    String.raw`(?:^|\s)(?:-m|--message)(?:\s+|=)([^\s;&|"']+)`
+    String.raw`(?:^|\s)(?:-[a-zA-Z]*m|--message)(?:\s+|=)([^\s;&|"']+)`
   ].join("|"),
   "g"
 );
@@ -258,7 +258,7 @@ var plugin = {
       } else if (toolName === "bash") {
         const command = inp["command"] ?? inp["CommandLine"] ?? inp["cmd"] ?? inp["script"] ?? inp["input"];
         if (typeof command !== "string") return;
-        if (!/\bgit\s+commit\b/.test(command)) return;
+        if (!/\bgit\s+commit(?![a-zA-Z0-9_-])/.test(command)) return;
         message = extractMessageFromBash(command);
         if (!message) return;
       } else {

package/dist/context-pins/index.d.ts

@@ -31,7 +31,7 @@
  *
  * @public
  */
-import type { Plugin } from '@wrongstack/core/types';
+import { type Plugin } from '@wrongstack/core/types';
 export interface Pin {
     id: string;
     label: string | null;

package/dist/context-pins.js

@@ -1,6 +1,7 @@
 // src/context-pins/index.ts
 import * as fs from "node:fs";
 import { dirname, isAbsolute, relative, resolve } from "node:path";
+import { ToolValidationError } from "@wrongstack/core/types";
 import { atomicWrite, ensureDir } from "@wrongstack/core/utils";
 var state = {
   pins: [],
@@ -141,16 +142,17 @@ var plugin = {
       category: "Memory",
       mutating: true,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "context-pins is disabled" };
+        if (!cfg.enabled) throw new Error("context-pins is disabled");
         const raw = input ?? {};
         const rawText = input.text ?? raw["pin"] ?? raw["content"] ?? raw["message"] ?? raw["note"] ?? raw["fact"] ?? raw["data"];
         const text = String(rawText ?? "").trim();
-        if (!text) return { ok: false, error: "pin text must not be empty" };
+        if (!text) {
+          throw new ToolValidationError({ message: "pin text must not be empty", field: "text" });
+        }
         if (state.pins.length >= cfg.maxPins) {
-          return {
-            ok: false,
-            error: `pin limit reached (${cfg.maxPins}). Remove a pin first with pin_remove.`
-          };
+          throw new Error(
+            `pin limit reached (${cfg.maxPins}). Remove a pin first with pin_remove.`
+          );
         }
         const pin = {
           id: `pin-${state.nextId++}`,
@@ -179,16 +181,17 @@ var plugin = {
       category: "Memory",
       mutating: true,
       async execute(input) {
-        if (!cfg.enabled) return { ok: false, error: "context-pins is disabled" };
+        if (!cfg.enabled) throw new Error("context-pins is disabled");
         const raw = input ?? {};
         const key = String(
           input.id ?? input.label ?? raw["pinId"] ?? raw["pin_id"] ?? raw["name"] ?? raw["key"] ?? ""
         ).trim();
-        if (!key) return { ok: false, error: "id or label is required" };
+        if (!key)
+          throw new ToolValidationError({ message: "id or label is required", field: "id" });
         const before = state.pins.length;
         state.pins = state.pins.filter((p) => p.id !== key && p.label !== key);
         const removed = before - state.pins.length;
-        if (removed === 0) return { ok: false, error: `no pin matches "${key}"` };
+        if (removed === 0) throw new Error(`no pin matches "${key}"`);
         state.removals += removed;
         api.metrics.counter("removals", removed);
         const persisted = await persistPins(cfg.filePath);