@hasna/hooks 0.9.3 → 0.9.4

package/bin/hooks-mcp.js

@@ -2708,7 +2708,7 @@ function mementosCommandSuffix(value) {
   return value.enabled ? ` --enable-mementos${value.secretRef ? ` --openrouter-secret-ref ${value.secretRef}` : ""}` : " --disable-mementos";
 }
 function retainedMementosRegistration(commands) {
-  const values = commands.filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
+  const values = commands.map((command) => command.trim()).filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
     const args = command.trim().split(/\s+/).slice(3);
     const flags = {};
     const seen = new Set;
@@ -3436,8 +3436,36 @@ function registerHook(name, scope = "global", target = "claude", profile, mement
   if (!settings.hooks)
     settings.hooks = {};
   const retained = name === "mementos-context" && mementos === undefined ? retainedMementosRegistration(Object.values(settings.hooks).flatMap((groups) => Array.isArray(groups) ? groups.flatMap((group) => Array.isArray(group.hooks) ? group.hooks.map((hook) => hook.command).filter((command) => typeof command === "string") : []) : [])) : undefined;
-  removeHookFromAllEvents(settings, name);
   const hookCommand = (profile ? `hooks run ${name} --profile ${profile}` : `hooks run ${name}`) + mementosCommandSuffix(mementos ?? retained);
+  if (target === "codex") {
+    const positions = [];
+    for (const [event, groups] of Object.entries(settings.hooks)) {
+      for (const entry of groups) {
+        for (let index = 0;index < entry.hooks.length; index++) {
+          const command = entry.hooks[index]?.command;
+          const match = typeof command === "string" && command.trim().match(/^(?:hooks run ([\w-]+)(?:\s|$)|hook-([\w-]+)$)/);
+          if (match && (match[1] ?? match[2]) === name) {
+            if (registrationName(command) !== name)
+              throw new Error("Unrecognized Codex registration; reconcile it before updating this hook.");
+            positions.push({ event, entry, index });
+          }
+        }
+      }
+    }
+    if (positions.length) {
+      if (positions.length !== uniqueEventKeys.length || uniqueEventKeys.some((event) => positions.filter((position) => position.event === event).length !== 1) || positions.some((position) => (position.entry.matcher ?? "") !== (meta.matcher ?? ""))) {
+        throw new Error("Codex hook events, matchers or duplicate registrations require explicit reconciliation; no settings were changed.");
+      }
+      for (const { entry, index } of positions) {
+        entry.hooks[index] = { ...entry.hooks[index], type: "command", command: hookCommand };
+        if (typeof meta.timeoutSeconds === "number" && meta.timeoutSeconds > 0)
+          entry.hooks[index].timeout = meta.timeoutSeconds;
+      }
+      writeSettings(settings, scope, target);
+      return;
+    }
+  }
+  removeHookFromAllEvents(settings, name);
   for (const eventKey of uniqueEventKeys) {
     if (!settings.hooks[eventKey])
       settings.hooks[eventKey] = [];

package/bin/index.js

@@ -4528,7 +4528,7 @@ function mementosCommandSuffix(value) {
   return value.enabled ? ` --enable-mementos${value.secretRef ? ` --openrouter-secret-ref ${value.secretRef}` : ""}` : " --disable-mementos";
 }
 function retainedMementosRegistration(commands) {
-  const values = commands.filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
+  const values = commands.map((command) => command.trim()).filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
     const args = command.trim().split(/\s+/).slice(3);
     const flags = {};
     const seen = new Set;
@@ -5790,8 +5790,36 @@ function registerHook(name, scope = "global", target = "claude", profile, mement
   if (!settings.hooks)
     settings.hooks = {};
   const retained = name === "mementos-context" && mementos === undefined ? retainedMementosRegistration(Object.values(settings.hooks).flatMap((groups) => Array.isArray(groups) ? groups.flatMap((group) => Array.isArray(group.hooks) ? group.hooks.map((hook) => hook.command).filter((command) => typeof command === "string") : []) : [])) : undefined;
-  removeHookFromAllEvents(settings, name);
   const hookCommand = (profile ? `hooks run ${name} --profile ${profile}` : `hooks run ${name}`) + mementosCommandSuffix(mementos ?? retained);
+  if (target === "codex") {
+    const positions = [];
+    for (const [event, groups] of Object.entries(settings.hooks)) {
+      for (const entry of groups) {
+        for (let index = 0;index < entry.hooks.length; index++) {
+          const command = entry.hooks[index]?.command;
+          const match = typeof command === "string" && command.trim().match(/^(?:hooks run ([\w-]+)(?:\s|$)|hook-([\w-]+)$)/);
+          if (match && (match[1] ?? match[2]) === name) {
+            if (registrationName(command) !== name)
+              throw new Error("Unrecognized Codex registration; reconcile it before updating this hook.");
+            positions.push({ event, entry, index });
+          }
+        }
+      }
+    }
+    if (positions.length) {
+      if (positions.length !== uniqueEventKeys.length || uniqueEventKeys.some((event) => positions.filter((position) => position.event === event).length !== 1) || positions.some((position) => (position.entry.matcher ?? "") !== (meta.matcher ?? ""))) {
+        throw new Error("Codex hook events, matchers or duplicate registrations require explicit reconciliation; no settings were changed.");
+      }
+      for (const { entry, index } of positions) {
+        entry.hooks[index] = { ...entry.hooks[index], type: "command", command: hookCommand };
+        if (typeof meta.timeoutSeconds === "number" && meta.timeoutSeconds > 0)
+          entry.hooks[index].timeout = meta.timeoutSeconds;
+      }
+      writeSettings(settings, scope, target);
+      return;
+    }
+  }
+  removeHookFromAllEvents(settings, name);
   for (const eventKey of uniqueEventKeys) {
     if (!settings.hooks[eventKey])
       settings.hooks[eventKey] = [];

package/dist/index.js

@@ -7473,7 +7473,7 @@ function mementosCommandSuffix(value) {
   return value.enabled ? ` --enable-mementos${value.secretRef ? ` --openrouter-secret-ref ${value.secretRef}` : ""}` : " --disable-mementos";
 }
 function retainedMementosRegistration(commands) {
-  const values = commands.filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
+  const values = commands.map((command) => command.trim()).filter((command) => /^hooks run mementos-context(?:\s|$)/.test(command)).map((command) => {
     const args = command.trim().split(/\s+/).slice(3);
     const flags = {};
     const seen = new Set;
@@ -8015,8 +8015,36 @@ function registerHook(name, scope = "global", target = "claude", profile, mement
   if (!settings.hooks)
     settings.hooks = {};
   const retained = name === "mementos-context" && mementos === undefined ? retainedMementosRegistration(Object.values(settings.hooks).flatMap((groups) => Array.isArray(groups) ? groups.flatMap((group) => Array.isArray(group.hooks) ? group.hooks.map((hook) => hook.command).filter((command) => typeof command === "string") : []) : [])) : undefined;
-  removeHookFromAllEvents(settings, name);
   const hookCommand = (profile ? `hooks run ${name} --profile ${profile}` : `hooks run ${name}`) + mementosCommandSuffix(mementos ?? retained);
+  if (target === "codex") {
+    const positions = [];
+    for (const [event, groups] of Object.entries(settings.hooks)) {
+      for (const entry of groups) {
+        for (let index = 0;index < entry.hooks.length; index++) {
+          const command = entry.hooks[index]?.command;
+          const match = typeof command === "string" && command.trim().match(/^(?:hooks run ([\w-]+)(?:\s|$)|hook-([\w-]+)$)/);
+          if (match && (match[1] ?? match[2]) === name) {
+            if (registrationName(command) !== name)
+              throw new Error("Unrecognized Codex registration; reconcile it before updating this hook.");
+            positions.push({ event, entry, index });
+          }
+        }
+      }
+    }
+    if (positions.length) {
+      if (positions.length !== uniqueEventKeys.length || uniqueEventKeys.some((event) => positions.filter((position) => position.event === event).length !== 1) || positions.some((position) => (position.entry.matcher ?? "") !== (meta.matcher ?? ""))) {
+        throw new Error("Codex hook events, matchers or duplicate registrations require explicit reconciliation; no settings were changed.");
+      }
+      for (const { entry, index } of positions) {
+        entry.hooks[index] = { ...entry.hooks[index], type: "command", command: hookCommand };
+        if (typeof meta.timeoutSeconds === "number" && meta.timeoutSeconds > 0)
+          entry.hooks[index].timeout = meta.timeoutSeconds;
+      }
+      writeSettings(settings, scope, target);
+      return;
+    }
+  }
+  removeHookFromAllEvents(settings, name);
   for (const eventKey of uniqueEventKeys) {
     if (!settings.hooks[eventKey])
       settings.hooks[eventKey] = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/hooks",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "description": "Open source hooks library for AI coding agents - Install safety, quality, and automation hooks with a single command",
   "type": "module",
   "bin": {