oxtail 0.8.0 → 0.10.0

package/scripts/uninstall-hook.mjs

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
-// Remove the oxtail PreToolUse hook entry and marker from
-// ~/.claude/settings.json, and delete the installed ~/.oxtail/hooks/pretooluse.sh.
+// Remove the oxtail hook entries (PreToolUse + Stop) and marker from
+// ~/.claude/settings.json, and delete the installed scripts under
+// ~/.oxtail/hooks/.
 //
 // Idempotent: a clean run on an uninstalled system exits 0 with "nothing to do."
 
@@ -12,13 +13,13 @@ import { applyEdits, modify, parse } from "jsonc-parser";
 import {
   SETTINGS_PATH,
   HOOK_MARKER_KEY,
-  HOOK_SCRIPT_PATH,
+  MANAGED_HOOKS,
 } from "./hook-constants.mjs";
 
 const FORMATTING = { tabSize: 2, insertSpaces: true };
 
-function findOxtailHookIndex(parsed) {
-  const arr = parsed?.hooks?.PreToolUse;
+function findOxtailHookIndex(parsed, event, asset) {
+  const arr = parsed?.hooks?.[event];
   if (!Array.isArray(arr)) return -1;
   return arr.findIndex((entry) => {
     if (!entry || typeof entry !== "object") return false;
@@ -28,44 +29,58 @@ function findOxtailHookIndex(parsed) {
         h &&
         typeof h === "object" &&
         typeof h.command === "string" &&
-        h.command.includes("oxtail/hooks/pretooluse.sh"),
+        h.command.includes(`oxtail/hooks/${asset}`),
     );
   });
 }
 
+async function removeScripts() {
+  for (const h of MANAGED_HOOKS) {
+    if (!existsSync(h.scriptPath)) continue;
+    try {
+      await unlink(h.scriptPath);
+      console.log(`Removed ${h.scriptPath}.`);
+    } catch (err) {
+      console.warn(`Could not remove ${h.scriptPath}: ${err?.message ?? err}`);
+    }
+  }
+}
+
 export async function uninstall() {
   if (!existsSync(SETTINGS_PATH)) {
     console.log(`No ${SETTINGS_PATH} — nothing to do.`);
-    // Still try to remove the installed script in case it's a leftover.
-    if (existsSync(HOOK_SCRIPT_PATH)) {
-      try {
-        await unlink(HOOK_SCRIPT_PATH);
-        console.log(`Removed ${HOOK_SCRIPT_PATH}.`);
-      } catch (err) {
-        console.warn(`Could not remove ${HOOK_SCRIPT_PATH}: ${err?.message ?? err}`);
-      }
-    }
+    // Still try to remove installed scripts in case they're leftovers.
+    await removeScripts();
     return;
   }
 
   const source = await readFile(SETTINGS_PATH, "utf8");
   const parsed = parse(source) ?? {};
 
-  const idx = findOxtailHookIndex(parsed);
   const hasMarker =
     parsed[HOOK_MARKER_KEY] && typeof parsed[HOOK_MARKER_KEY] === "object";
+  const anyEntry = MANAGED_HOOKS.some(
+    (h) => findOxtailHookIndex(parsed, h.event, h.asset) >= 0,
+  );
+  const anyScript = MANAGED_HOOKS.some((h) => existsSync(h.scriptPath));
 
-  if (idx < 0 && !hasMarker && !existsSync(HOOK_SCRIPT_PATH)) {
-    console.log("oxtail hook not installed — nothing to do.");
+  if (!anyEntry && !hasMarker && !anyScript) {
+    console.log("oxtail hooks not installed — nothing to do.");
     return;
   }
 
+  // Remove each event's oxtail entry. Re-parse per iteration so indices stay
+  // valid after the prior edit.
   let text = source;
-  if (idx >= 0) {
-    text = applyEdits(
-      text,
-      modify(text, ["hooks", "PreToolUse", idx], undefined, { formattingOptions: FORMATTING }),
-    );
+  for (const h of MANAGED_HOOKS) {
+    const cur = parse(text) ?? {};
+    const idx = findOxtailHookIndex(cur, h.event, h.asset);
+    if (idx >= 0) {
+      text = applyEdits(
+        text,
+        modify(text, ["hooks", h.event, idx], undefined, { formattingOptions: FORMATTING }),
+      );
+    }
   }
   if (hasMarker) {
     text = applyEdits(
@@ -78,16 +93,9 @@ export async function uninstall() {
   await mkdir(path.dirname(SETTINGS_PATH), { recursive: true });
   await writeFile(settingsTmp, text, "utf8");
   await rename(settingsTmp, SETTINGS_PATH);
-  console.log(`Removed oxtail PreToolUse hook from ${SETTINGS_PATH}.`);
+  console.log(`Removed oxtail hooks from ${SETTINGS_PATH}.`);
 
-  if (existsSync(HOOK_SCRIPT_PATH)) {
-    try {
-      await unlink(HOOK_SCRIPT_PATH);
-      console.log(`Removed ${HOOK_SCRIPT_PATH}.`);
-    } catch (err) {
-      console.warn(`Could not remove ${HOOK_SCRIPT_PATH}: ${err?.message ?? err}`);
-    }
-  }
+  await removeScripts();
 }
 
 const invokedDirectly =