@hasna/hooks 0.8.0 → 0.9.0

package/bin/index.js

@@ -2083,6 +2083,18 @@ var init_registry = __esm(() => {
       matcher: "^(Bash|Write|Edit|MultiEdit|NotebookEdit|apply_patch|ApplyPatch|functions\\.apply_patch)$",
       tags: ["workspace", "repos", "structure", "guard", "safety", "orgs", "multi-agent"]
     },
+    {
+      name: "trash-guard",
+      displayName: "Trash Guard",
+      description: "Rewrites rm issued through Bash into `trash guard`, so the delete lands in a recoverable trash store; refuses the delete when there is nothing to redirect to (trash absent, a delete verb it cannot rewrite, or the protected class: /, ~, ~/.hasna, ~/.ssh, ~/.aws)",
+      version: "0.1.0",
+      category: "Git Safety",
+      event: "PreToolUse",
+      matcher: "Bash",
+      tags: ["rm", "delete", "trash", "recoverable", "guard", "safety", "bash"],
+      rewritesInput: true,
+      timeoutSeconds: 5
+    },
     {
       name: "checktests",
       displayName: "Check Tests",
@@ -4477,6 +4489,111 @@ var init_resolve = __esm(() => {
   init_manifest();
 });
 
+// src/lib/registration.ts
+function registrationName(command) {
+  if (typeof command !== "string")
+    return;
+  const match = command.trim().match(REGISTRATION_COMMAND_RE);
+  return match ? match[1] ?? match[2] : undefined;
+}
+function findStaleRegistrations(settings, file, resolves = (name) => resolveHook(name) !== undefined) {
+  const hooks = settings.hooks;
+  if (!hooks || typeof hooks !== "object")
+    return [];
+  const stale = [];
+  for (const eventKey of Object.keys(hooks)) {
+    const entries = hooks[eventKey];
+    if (!Array.isArray(entries))
+      continue;
+    for (const entry of entries) {
+      if (!Array.isArray(entry?.hooks))
+        continue;
+      for (const hook of entry.hooks) {
+        const name = registrationName(hook?.command);
+        if (name === undefined || resolves(name))
+          continue;
+        stale.push({ file, event: eventKey, hook: name, command: hook.command });
+      }
+    }
+  }
+  return stale;
+}
+function matchersOverlap(a, b) {
+  return a === b || a.includes(b) || b.includes(a);
+}
+function hookEventsOf(info) {
+  return info.events?.length ? info.events : info.event ? [info.event] : [];
+}
+function findRewriteOverlaps(names, lookup) {
+  const rewriting = names.map((name) => ({ name, info: lookup(name) })).filter((row) => Boolean(row.info?.rewritesInput && row.info.matcher));
+  const overlaps = [];
+  for (let i = 0;i < rewriting.length; i++) {
+    for (let j = i + 1;j < rewriting.length; j++) {
+      const a = rewriting[i];
+      const b = rewriting[j];
+      const aEvents = hookEventsOf(a.info);
+      const bEvents = hookEventsOf(b.info);
+      const event = aEvents.find((name) => name === "PreToolUse" && bEvents.includes(name));
+      if (!event)
+        continue;
+      const aMatcher = a.info.matcher;
+      const bMatcher = b.info.matcher;
+      if (!matchersOverlap(aMatcher.toLowerCase(), bMatcher.toLowerCase()))
+        continue;
+      overlaps.push({ hooks: [a.name, b.name], event, matchers: [aMatcher, bMatcher] });
+    }
+  }
+  return overlaps;
+}
+function safeMatcherTest(pattern, value) {
+  try {
+    return new RegExp(pattern).test(value);
+  } catch {
+    return false;
+  }
+}
+function countSettingsWiring(settings) {
+  const hooks = settings.hooks;
+  if (!hooks || typeof hooks !== "object")
+    return 0;
+  let count = 0;
+  for (const eventKey of Object.keys(hooks)) {
+    const entries = hooks[eventKey];
+    if (!Array.isArray(entries))
+      continue;
+    for (const entry of entries) {
+      if (Array.isArray(entry?.hooks))
+        count += entry.hooks.length;
+    }
+  }
+  return count;
+}
+function hookRegisteredInSettings(settings, name, eventSpec, hookMatcher) {
+  const separator = eventSpec.indexOf(":");
+  const eventName = separator === -1 ? eventSpec : eventSpec.slice(0, separator);
+  const specMatcher = separator === -1 ? "" : eventSpec.slice(separator + 1);
+  const matcher = specMatcher || hookMatcher || "";
+  const hooks = settings.hooks?.[eventName];
+  if (!Array.isArray(hooks))
+    return false;
+  return hooks.some((entry) => entry?.hooks?.some((h) => {
+    const match = h?.command?.match(/^hooks run ([\w-]+)/);
+    if (!match || match[1] !== name)
+      return false;
+    if (!matcher)
+      return true;
+    const entryMatcher = typeof entry.matcher === "string" ? entry.matcher : "";
+    if (!entryMatcher)
+      return true;
+    return entryMatcher === matcher || safeMatcherTest(entryMatcher, matcher) || safeMatcherTest(matcher, entryMatcher);
+  }));
+}
+var REGISTRATION_COMMAND_RE;
+var init_registration = __esm(() => {
+  init_resolve();
+  REGISTRATION_COMMAND_RE = /^(?:hooks run ([\w-]+)(?:\s+--profile\s+[\w-]+)?|hook-([\w-]+))$/;
+});
+
 // src/db/schema.ts
 var CREATE_HOOK_EVENTS_TABLE = `
   CREATE TABLE IF NOT EXISTS hook_events (
@@ -5090,8 +5207,10 @@ var init_store = __esm(() => {
 var exports_installer = {};
 __export(exports_installer, {
   uninstallHook: () => uninstallHook,
+  scanStaleRegistrations: () => scanStaleRegistrations,
   removeHook: () => removeHook,
   removeCodewithHookEntry: () => removeCodewithHookEntry,
+  pruneStaleRegistrations: () => pruneStaleRegistrations,
   isEventSupported: () => isEventSupported,
   installHooks: () => installHooks,
   installHook: () => installHook,
@@ -5101,6 +5220,7 @@ __export(exports_installer, {
   getRegisteredHooks: () => getRegisteredHooks,
   getInstalledHooks: () => getInstalledHooks,
   getHookPath: () => getHookPath,
+  detectRewriteConflict: () => detectRewriteConflict,
   buildCodewithTomlFragment: () => buildCodewithTomlFragment
 });
 import { existsSync as existsSync10, mkdirSync as mkdirSync4, readFileSync as readFileSync4, readdirSync as readdirSync3, rmdirSync, unlinkSync, writeFileSync as writeFileSync3 } from "fs";
@@ -5207,6 +5327,7 @@ function codewithTimeout(name) {
     case "prompt-guard":
       return 3;
     case "worktree-guard":
+    case "trash-guard":
     case "stop-sync":
       return 5;
     default:
@@ -5225,6 +5346,8 @@ function codewithStatusMessage(name) {
       return "Checking prompt safety";
     case "worktree-guard":
       return "Checking worktree safety";
+    case "trash-guard":
+      return "Redirecting rm into trash";
     case "stop-sync":
       return "Syncing turn-end heartbeat";
     default:
@@ -5292,14 +5415,32 @@ function detectConflict(name, scope, target) {
       continue;
     if (!getHookEvents(existing).some((event) => events.has(event)))
       continue;
-    const a = meta.matcher.toLowerCase();
-    const b = existing.matcher.toLowerCase();
-    if (a === b || a.includes(b) || b.includes(a)) {
+    if (matchersOverlap(meta.matcher.toLowerCase(), existing.matcher.toLowerCase())) {
       return `conflicts with '${existingName}' (same event ${meta.event}, overlapping matcher '${existing.matcher}')`;
     }
   }
   return;
 }
+function detectRewriteConflict(name, scope, target, lookup = getHook) {
+  const meta = resolveHookMeta(name);
+  if (!meta?.matcher || !meta.rewritesInput)
+    return;
+  if (!getHookEvents(meta).includes("PreToolUse"))
+    return;
+  for (const existingName of getRegisteredHooksForTarget(scope, target)) {
+    if (existingName === name)
+      continue;
+    const existing = lookup(existingName);
+    if (!existing?.matcher || !existing.rewritesInput)
+      continue;
+    if (!getHookEvents(existing).includes("PreToolUse"))
+      continue;
+    if (!matchersOverlap(meta.matcher.toLowerCase(), existing.matcher.toLowerCase()))
+      continue;
+    return `both '${name}' and '${existingName}' rewrite the tool input on overlapping PreToolUse matchers ('${meta.matcher}' / '${existing.matcher}'). Only one rewrite per tool call is applied, so one guard would silently disappear \u2014 install one of them, or narrow a matcher so they no longer overlap.`;
+  }
+  return;
+}
 function installForTarget(name, scope, overwrite, target, profile, codewithMode = "fragment", codewithConfigPath) {
   const shortName = shortHookName(name);
   if (!hookExists(shortName)) {
@@ -5359,6 +5500,10 @@ function installForTarget(name, scope, overwrite, target, profile, codewithMode
     return { hook: shortName, success: false, error: "Already installed. Use --overwrite to replace.", scope, target };
   }
   const conflict = detectConflict(shortName, scope, target);
+  const rewriteConflict = detectRewriteConflict(shortName, scope, target);
+  if (rewriteConflict) {
+    return { hook: shortName, success: false, error: `Refused: ${rewriteConflict}`, scope, target };
+  }
   try {
     registerHook(shortName, scope, target, profile);
     return { hook: shortName, success: true, scope, target, ...conflict ? { conflict } : {} };
@@ -5430,8 +5575,12 @@ function registerHook(name, scope = "global", target = "claude", profile) {
   for (const eventKey of uniqueEventKeys) {
     if (!settings.hooks[eventKey])
       settings.hooks[eventKey] = [];
+    const hookEntry = { type: "command", command: hookCommand };
+    if (typeof meta.timeoutSeconds === "number" && meta.timeoutSeconds > 0) {
+      hookEntry.timeout = meta.timeoutSeconds;
+    }
     const entry = {
-      hooks: [{ type: "command", command: hookCommand }]
+      hooks: [hookEntry]
     };
     if (meta.matcher) {
       entry.matcher = meta.matcher;
@@ -5515,6 +5664,74 @@ function removeHookForTarget(name, scope, target) {
   unregisterHook(name, scope, target);
   return true;
 }
+function readSettingsAt(path) {
+  try {
+    if (!existsSync10(path))
+      return;
+    const raw = readFileSync4(path, "utf-8");
+    return { raw, settings: JSON.parse(raw) };
+  } catch {
+    return;
+  }
+}
+function scanStaleRegistrations(scope = "global") {
+  const stale = [];
+  for (const target of JSON_SETTINGS_TARGETS) {
+    const path = getSettingsPath(scope, target);
+    const parsed = readSettingsAt(path);
+    if (!parsed)
+      continue;
+    stale.push(...findStaleRegistrations(parsed.settings, path));
+  }
+  return stale;
+}
+function pruneStaleRegistrations(scope = "global") {
+  const results = [];
+  for (const target of JSON_SETTINGS_TARGETS) {
+    const path = getSettingsPath(scope, target);
+    const parsed = readSettingsAt(path);
+    if (!parsed)
+      continue;
+    const stale = findStaleRegistrations(parsed.settings, path);
+    if (stale.length === 0)
+      continue;
+    const staleCommandsByEvent = new Map;
+    for (const finding of stale) {
+      const commands = staleCommandsByEvent.get(finding.event) ?? new Set;
+      commands.add(finding.command);
+      staleCommandsByEvent.set(finding.event, commands);
+    }
+    const settings = parsed.settings;
+    for (const [eventKey, commands] of staleCommandsByEvent) {
+      const entries = settings.hooks?.[eventKey];
+      if (!Array.isArray(entries))
+        continue;
+      const keptEntries = [];
+      for (const entry of entries) {
+        if (!Array.isArray(entry?.hooks)) {
+          keptEntries.push(entry);
+          continue;
+        }
+        const keptHooks = entry.hooks.filter((hook) => !(typeof hook?.command === "string" && commands.has(hook.command)));
+        if (keptHooks.length === entry.hooks.length)
+          keptEntries.push(entry);
+        else if (keptHooks.length > 0)
+          keptEntries.push({ ...entry, hooks: keptHooks });
+      }
+      if (keptEntries.length > 0)
+        settings.hooks[eventKey] = keptEntries;
+      else
+        delete settings.hooks[eventKey];
+    }
+    if (settings.hooks && Object.keys(settings.hooks).length === 0)
+      delete settings.hooks;
+    const backupPath = `${path}.bak`;
+    writeFileSync3(backupPath, parsed.raw);
+    writeSettings(settings, scope, target);
+    results.push({ file: path, removed: stale, backupPath });
+  }
+  return results;
+}
 function removeCodewithHookEntry(configText, name) {
   const sections = configText.split(/\n\s*\n/);
   const nameRe = new RegExp(`^command\\s*=\\s*"hooks run ${escapeRegExp(name)}(?:\\s+--profile\\s+[\\w-]+)?"$`);
@@ -5659,11 +5876,12 @@ function uninstallHook(name, scope = "global", target = "claude") {
     registrationsRemaining
   };
 }
-var __dirname2, HOOKS_DIR, EVENT_MAP, getInstalledHooks;
+var __dirname2, HOOKS_DIR, EVENT_MAP, getInstalledHooks, JSON_SETTINGS_TARGETS;
 var init_installer = __esm(() => {
   init_registry();
   init_resolve();
   init_manifest();
+  init_registration();
   init_store();
   __dirname2 = dirname3(fileURLToPath2(import.meta.url));
   HOOKS_DIR = existsSync10(join10(__dirname2, "..", "..", "hooks", "hook-gitguard")) ? join10(__dirname2, "..", "..", "hooks") : join10(__dirname2, "..", "hooks");
@@ -5700,53 +5918,9 @@ var init_installer = __esm(() => {
     }
   };
   getInstalledHooks = getRegisteredHooks;
+  JSON_SETTINGS_TARGETS = ["claude", "gemini"];
 });
 
-// src/lib/registration.ts
-function safeMatcherTest(pattern, value) {
-  try {
-    return new RegExp(pattern).test(value);
-  } catch {
-    return false;
-  }
-}
-function countSettingsWiring(settings) {
-  const hooks = settings.hooks;
-  if (!hooks || typeof hooks !== "object")
-    return 0;
-  let count = 0;
-  for (const eventKey of Object.keys(hooks)) {
-    const entries = hooks[eventKey];
-    if (!Array.isArray(entries))
-      continue;
-    for (const entry of entries) {
-      if (Array.isArray(entry?.hooks))
-        count += entry.hooks.length;
-    }
-  }
-  return count;
-}
-function hookRegisteredInSettings(settings, name, eventSpec, hookMatcher) {
-  const separator = eventSpec.indexOf(":");
-  const eventName = separator === -1 ? eventSpec : eventSpec.slice(0, separator);
-  const specMatcher = separator === -1 ? "" : eventSpec.slice(separator + 1);
-  const matcher = specMatcher || hookMatcher || "";
-  const hooks = settings.hooks?.[eventName];
-  if (!Array.isArray(hooks))
-    return false;
-  return hooks.some((entry) => entry?.hooks?.some((h) => {
-    const match = h?.command?.match(/^hooks run ([\w-]+)/);
-    if (!match || match[1] !== name)
-      return false;
-    if (!matcher)
-      return true;
-    const entryMatcher = typeof entry.matcher === "string" ? entry.matcher : "";
-    if (!entryMatcher)
-      return true;
-    return entryMatcher === matcher || safeMatcherTest(entryMatcher, matcher) || safeMatcherTest(matcher, entryMatcher);
-  }));
-}
-
 // src/lib/redact.ts
 function redactText(value) {
   let out = value;
@@ -5922,7 +6096,7 @@ import { isIP as isIP2 } from "net";
 import { spawnSync } from "child_process";
 import { closeSync, fstatSync, openSync, readFileSync as readFileSync6 } from "fs";
 import { O_NOFOLLOW, O_NONBLOCK, O_RDONLY } from "constants";
-import { createRequire } from "module";
+import { createRequire as createRequire2 } from "module";
 import { hostname as osHostname } from "os";
 import { isAbsolute as isAbsolute2, join as join12 } from "path";
 function envToken(name) {
@@ -6043,7 +6217,7 @@ function readAppConfigFile(path) {
     const uid = process.getuid?.() ?? process.geteuid?.();
     if (uid !== undefined && before.uid !== uid)
       unsafe("the file is not owned by the current user");
-    if (before.size > MAX_CREDENTIAL_FILE_BYTES)
+    if (before.size > MAX_CREDENTIAL_FILE_BYTES2)
       unsafe("the file exceeds the size limit");
     const bytes = readFileSync6(fd);
     const after = fstatSync(fd);
@@ -6127,13 +6301,13 @@ function sealCredential(fields) {
       configurable: false
     });
   }
-  Object.defineProperty(sealed, INSPECT_CUSTOM, {
+  Object.defineProperty(sealed, INSPECT_CUSTOM2, {
     value: () => ({ ...visible, apiKey: "[redacted]" }),
     enumerable: false,
     writable: false,
     configurable: false
   });
-  Object.defineProperty(sealed, CREDENTIAL_SEAL, {
+  Object.defineProperty(sealed, CREDENTIAL_SEAL2, {
     value: true,
     enumerable: false,
     writable: false,
@@ -6152,7 +6326,7 @@ function firstEnvValue(env, keys) {
   return null;
 }
 function isAmbientEnvironment(env) {
-  return env === process.env || env[AMBIENT_ENVIRONMENT] === true;
+  return env === process.env || env[AMBIENT_ENVIRONMENT2] === true;
 }
 function defaultKeychainRunner(argv) {
   const result = spawnSync(KEYCHAIN_SECURITY_BIN, [...argv], {
@@ -6246,7 +6420,7 @@ function snapshotClientEnvironment(name, env) {
     snapshot[key] = descriptor.value;
   }
   if (ambient) {
-    Object.defineProperty(snapshot, AMBIENT_ENVIRONMENT, {
+    Object.defineProperty(snapshot, AMBIENT_ENVIRONMENT2, {
       value: true,
       enumerable: false,
       writable: false,
@@ -6597,7 +6771,7 @@ function credentialDiskSourcesForMessage(name, env) {
   const paths = credentialDiskSources(name, env);
   return paths.length > 0 ? paths.join(" or ") : "<no HOME or HASNA_HOME set in this environment, so no credential file was consulted>";
 }
-var CREDENTIAL_PROFILE_ENV_KEY = "HASNA_PROFILE", CredentialResolutionError, CredentialFileUnsafeError, HASNA_HOME_ENV_KEY = "HASNA_HOME", HASNA_CONFIG_HOME_ENV_KEY = "HASNA_CONFIG_HOME", KEYCHAIN_STATION_ENV_KEY = "HASNA_STATION", HASNA_HOME_DIR = ".hasna", CONFIG_SUBDIR = "config", CREDENTIALS_FILE = "credentials", KEYCHAIN_SECURITY_BIN = "/usr/bin/security", KEYCHAIN_SERVICE_PREFIX = "hasna.credentials", KEYCHAIN_ITEM_NOT_FOUND_STATUS = 44, KEYCHAIN_SPAWN_TIMEOUT_MS = 1e4, MAX_CREDENTIAL_FILE_BYTES, SAFE_APP_SLUG, SAFE_PROFILE, ILLEGAL_IN_HEADER_VALUE, VAULT_POINTER_SHAPE, CREDENTIAL_SHAPED_KEY, INSPECT_CUSTOM, CREDENTIAL_SEAL, AMBIENT_ENVIRONMENT, SECRETS_PACKAGE_SPECIFIER, requireSecretsSdk, DEFAULT_FLEET_GATEWAY_ORIGIN = "https://api.hasna.com", DEFAULT_AUTHORITY_SOURCE = "default", ASCII_CONTROL_PATTERN, DNS_LABEL_PATTERN, ClientTransportConfigurationError, IDEMPOTENT_METHODS, AUTHORITY_OVERRIDE_HEADERS;
+var CREDENTIAL_PROFILE_ENV_KEY = "HASNA_PROFILE", CredentialResolutionError, CredentialFileUnsafeError, HASNA_HOME_ENV_KEY = "HASNA_HOME", HASNA_CONFIG_HOME_ENV_KEY = "HASNA_CONFIG_HOME", KEYCHAIN_STATION_ENV_KEY = "HASNA_STATION", HASNA_HOME_DIR = ".hasna", CONFIG_SUBDIR = "config", CREDENTIALS_FILE = "credentials", KEYCHAIN_SECURITY_BIN = "/usr/bin/security", KEYCHAIN_SERVICE_PREFIX = "hasna.credentials", KEYCHAIN_ITEM_NOT_FOUND_STATUS = 44, KEYCHAIN_SPAWN_TIMEOUT_MS = 1e4, MAX_CREDENTIAL_FILE_BYTES2, SAFE_APP_SLUG, SAFE_PROFILE, ILLEGAL_IN_HEADER_VALUE, VAULT_POINTER_SHAPE, CREDENTIAL_SHAPED_KEY, INSPECT_CUSTOM2, CREDENTIAL_SEAL2, AMBIENT_ENVIRONMENT2, SECRETS_PACKAGE_SPECIFIER2, requireSecretsSdk2, DEFAULT_FLEET_GATEWAY_ORIGIN = "https://api.hasna.com", DEFAULT_AUTHORITY_SOURCE = "default", ASCII_CONTROL_PATTERN, DNS_LABEL_PATTERN, ClientTransportConfigurationError, IDEMPOTENT_METHODS2, AUTHORITY_OVERRIDE_HEADERS2;
 var init_transport = __esm(() => {
   CredentialResolutionError = class CredentialResolutionError extends Error {
     appName;
@@ -6617,17 +6791,17 @@ var init_transport = __esm(() => {
       this.path = path;
     }
   };
-  MAX_CREDENTIAL_FILE_BYTES = 64 * 1024;
+  MAX_CREDENTIAL_FILE_BYTES2 = 64 * 1024;
   SAFE_APP_SLUG = /^[a-z0-9](?:[a-z0-9-]*[a-z0-9])?$/;
   SAFE_PROFILE = /^[A-Za-z0-9](?:[A-Za-z0-9._-]*[A-Za-z0-9])?$/;
   ILLEGAL_IN_HEADER_VALUE = /[^\t\x20-\x7e]/;
   VAULT_POINTER_SHAPE = /^[a-z0-9][a-z0-9-]*(?:\/[a-z0-9][a-z0-9-_.]*){2,}$/;
   CREDENTIAL_SHAPED_KEY = /(?:^|_)(?:API_KEY|KEY|TOKEN|SECRET|PASSWORD|PASSWD|CREDENTIAL|AUTH)(?:_|$)/;
-  INSPECT_CUSTOM = Symbol.for("nodejs.util.inspect.custom");
-  CREDENTIAL_SEAL = Symbol.for("hasna:contracts:sealedCredential");
-  AMBIENT_ENVIRONMENT = Symbol.for("hasna:contracts:ambientClientEnvironment");
-  SECRETS_PACKAGE_SPECIFIER = "@hasna/" + "secrets";
-  requireSecretsSdk = createRequire(import.meta.url);
+  INSPECT_CUSTOM2 = Symbol.for("nodejs.util.inspect.custom");
+  CREDENTIAL_SEAL2 = Symbol.for("hasna:contracts:sealedCredential");
+  AMBIENT_ENVIRONMENT2 = Symbol.for("hasna:contracts:ambientClientEnvironment");
+  SECRETS_PACKAGE_SPECIFIER2 = "@hasna/" + "secrets";
+  requireSecretsSdk2 = createRequire2(import.meta.url);
   ASCII_CONTROL_PATTERN = /[\u0000-\u001f\u007f]/;
   DNS_LABEL_PATTERN = /^[a-z0-9](?:[a-z0-9-]*[a-z0-9])?$/;
   ClientTransportConfigurationError = class ClientTransportConfigurationError extends Error {
@@ -6640,8 +6814,8 @@ var init_transport = __esm(() => {
       this.sources = Object.freeze([...sources]);
     }
   };
-  IDEMPOTENT_METHODS = new Set(["GET", "HEAD", "PUT", "DELETE", "OPTIONS"]);
-  AUTHORITY_OVERRIDE_HEADERS = new Set([
+  IDEMPOTENT_METHODS2 = new Set(["GET", "HEAD", "PUT", "DELETE", "OPTIONS"]);
+  AUTHORITY_OVERRIDE_HEADERS2 = new Set([
     "host",
     ":authority",
     "forwarded",
@@ -8785,6 +8959,13 @@ function createHooksServer() {
       if (hookHealthy)
         healthy.push(name);
     }
+    for (const overlap of findRewriteOverlaps(registered, (name) => getHook(name))) {
+      issues.push({
+        hook: overlap.hooks.join(" + "),
+        issue: `Both rewrite the tool input on overlapping ${overlap.event} matchers ('${overlap.matchers[0]}' / '${overlap.matchers[1]}'); only one rewrite is applied per tool call, so one of these guards is silently disarmed. Remove one, or narrow a matcher so they no longer overlap.`,
+        severity: "error"
+      });
+    }
     return { content: [{ type: "text", text: JSON.stringify({ healthy: issues.length === 0, healthy_hooks: healthy, issues, registered, scope }) }] };
   });
   defineTool("hooks_categories", "List all hook categories with counts", {}, async () => {
@@ -9517,6 +9698,7 @@ var init_server = __esm(() => {
   init_profiles();
   init_manifest();
   init_resolve();
+  init_registration();
   init_store();
   init_redact();
   init_storage();
@@ -9612,6 +9794,7 @@ import { spawn } from "child_process";
 import { request as nodeHttpRequest } from "http";
 import { request as nodeHttpsRequest } from "https";
 import { randomUUID as randomUUID2 } from "crypto";
+import { createRequire } from "module";
 function getPathValue(input, path) {
   return path.split(".").reduce((value, part) => {
     if (value && typeof value === "object" && part in value) {
@@ -10709,6 +10892,22 @@ function setPath(input, path, replacement) {
   if (last && last in cursor)
     cursor[last] = replacement;
 }
+var MAX_CREDENTIAL_FILE_BYTES = 64 * 1024;
+var INSPECT_CUSTOM = Symbol.for("nodejs.util.inspect.custom");
+var CREDENTIAL_SEAL = Symbol.for("hasna:contracts:sealedCredential");
+var AMBIENT_ENVIRONMENT = Symbol.for("hasna:contracts:ambientClientEnvironment");
+var SECRETS_PACKAGE_SPECIFIER = "@hasna/" + "secrets";
+var requireSecretsSdk = createRequire(import.meta.url);
+var IDEMPOTENT_METHODS = new Set(["GET", "HEAD", "PUT", "DELETE", "OPTIONS"]);
+var AUTHORITY_OVERRIDE_HEADERS = new Set([
+  "host",
+  ":authority",
+  "forwarded",
+  "x-forwarded-host",
+  "x-original-host"
+]);
+var MAX_ENVELOPE_BYTES = 256 * 1024;
+var MAX_REQUEST_BYTES = MAX_ENVELOPE_BYTES * 2 + 8192;
 function createEvent(input) {
   return {
     id: input.id ?? randomUUID2(),
@@ -12427,6 +12626,7 @@ function App({ initialHooks, overwrite = false }) {
 // src/cli/index.tsx
 init_registry();
 init_installer();
+init_registration();
 init_redact();
 init_profiles();
 init_manifest();
@@ -13192,11 +13392,21 @@ ${meta.displayName}
     console.log(chalk2.dim("  \u25CB Not installed in project"));
   }
 });
-program2.command("doctor").option("-g, --global", "Check global settings", false).option("-p, --project", "Check project settings", false).option("-j, --json", "Output as JSON", false).description("Check health of installed hooks").action((options) => {
+program2.command("doctor").option("-g, --global", "Check global settings", false).option("-p, --project", "Check project settings", false).option("-f, --fix", "Remove stale registrations (a .bak is written beside each settings file)", false).option("-j, --json", "Output as JSON", false).description("Check health of installed hooks; --fix removes registrations whose hook cannot resolve").action((options) => {
   const scope = resolveScope(options);
   const settingsPath = getSettingsPath(scope);
   const issues = [];
   const healthy = [];
+  const fixed = options.fix ? pruneStaleRegistrations(scope) : [];
+  const stale = scanStaleRegistrations(scope);
+  for (const finding of stale) {
+    issues.push({
+      hook: finding.hook,
+      issue: `Stale registration in ${finding.file} (${finding.event}): \`${finding.command}\` does not resolve to a hook`,
+      severity: "error"
+    });
+  }
+  const staleNames = new Set(stale.map((finding) => finding.hook));
   const settingsExist = existsSync15(settingsPath);
   if (!settingsExist) {
     issues.push({ hook: "(settings)", issue: `${settingsPath} not found`, severity: "warning" });
@@ -13210,6 +13420,8 @@ program2.command("doctor").option("-g, --global", "Check global settings", false
     } catch {}
   }
   for (const name of registered) {
+    if (staleNames.has(name))
+      continue;
     const custom = readCustomManifest(name);
     const meta = custom ? resolveHookMeta(name) : getHook(name);
     let hookHealthy = true;
@@ -13245,17 +13457,53 @@ program2.command("doctor").option("-g, --global", "Check global settings", false
       healthy.push(name);
     }
   }
+  for (const overlap of findRewriteOverlaps(registered, (name) => getHook(name))) {
+    issues.push({
+      hook: overlap.hooks.join(" + "),
+      issue: `Both rewrite the tool input on overlapping ${overlap.event} matchers ('${overlap.matchers[0]}' / '${overlap.matchers[1]}'); only one rewrite is applied per tool call, so one of these guards is silently disarmed. Remove one, or narrow a matcher so they no longer overlap.`,
+      severity: "error"
+    });
+  }
   if (options.json) {
-    console.log(JSON.stringify({ healthy: issues.length === 0, healthy_hooks: healthy, issues, registered, wiring_count: wiringCount, scope }));
+    console.log(JSON.stringify({
+      healthy: issues.length === 0,
+      healthy_hooks: healthy,
+      issues,
+      stale,
+      fixed: fixed.flatMap((result) => result.removed),
+      backups: fixed.map((result) => result.backupPath),
+      registered,
+      wiring_count: wiringCount,
+      scope
+    }));
+    if (issues.some((issue) => issue.severity === "error"))
+      process.exitCode = 1;
     return;
   }
   console.log(chalk2.bold(`
 Hook Health Check (${scope})
 `));
+  if (fixed.length > 0) {
+    for (const result of fixed) {
+      console.log(chalk2.green(`  \u2713 removed ${result.removed.length} stale registration(s) from ${result.file}`));
+      for (const finding of result.removed) {
+        console.log(chalk2.dim(`      ${finding.hook} (${finding.event}: ${finding.command})`));
+      }
+      console.log(chalk2.dim(`      backup: ${result.backupPath}`));
+    }
+    console.log();
+  }
   if (registered.length === 0) {
     console.log(chalk2.dim("  No hooks registered."));
     console.log(chalk2.dim("  Run: hooks install gitguard"));
     console.log(chalk2.dim(`  (checked 0 registered \`hooks run\` entries of ${wiringCount} settings wiring entries; direct-path wiring outside the registered surface is not covered by this check)`));
+    if (issues.length > 0) {
+      console.log();
+      for (const issue of issues) {
+        const icon = issue.severity === "error" ? chalk2.red("\u2717") : chalk2.yellow("!");
+        console.log(`  ${icon} ${chalk2.cyan(issue.hook)}: ${issue.issue}`);
+      }
+    }
     if (issues.length > 0)
       process.exitCode = 1;
     return;

package/bin/serve.js

@@ -182,6 +182,18 @@ var HOOKS = [
     matcher: "^(Bash|Write|Edit|MultiEdit|NotebookEdit|apply_patch|ApplyPatch|functions\\.apply_patch)$",
     tags: ["workspace", "repos", "structure", "guard", "safety", "orgs", "multi-agent"]
   },
+  {
+    name: "trash-guard",
+    displayName: "Trash Guard",
+    description: "Rewrites rm issued through Bash into `trash guard`, so the delete lands in a recoverable trash store; refuses the delete when there is nothing to redirect to (trash absent, a delete verb it cannot rewrite, or the protected class: /, ~, ~/.hasna, ~/.ssh, ~/.aws)",
+    version: "0.1.0",
+    category: "Git Safety",
+    event: "PreToolUse",
+    matcher: "Bash",
+    tags: ["rm", "delete", "trash", "recoverable", "guard", "safety", "bash"],
+    rewritesInput: true,
+    timeoutSeconds: 5
+  },
   {
     name: "checktests",
     displayName: "Check Tests",