agileflow 4.0.0-alpha.2 → 4.0.0-alpha.21

package/src/cli/commands/hook.js

@@ -12,12 +12,13 @@
  * stdin, calls the orchestrator's `runEvent`, and exits with the
  * chain's resolved code. Always fails open on internal errors.
  */
-const path = require('path');
-const { runEvent } = require('../../runtime/hooks/orchestrator.js');
+const path = require("path");
+const { runEvent } = require("../../runtime/hooks/orchestrator.js");
 const {
   VALID_EVENTS,
   MATCHER_EVENTS,
-} = require('../../runtime/hooks/manifest-loader.js');
+} = require("../../runtime/hooks/manifest-loader.js");
+const { InvalidArgumentError, fail } = require("../../lib/errors.js");
 
 /**
  * @param {string} event
@@ -27,25 +28,27 @@ async function hook(event, options = {}) {
   // Validate event name BEFORE doing any I/O. A typo like "SesionStart"
   // would otherwise produce a silent empty-chain no-op.
   if (!VALID_EVENTS.has(event)) {
-    // eslint-disable-next-line no-console
-    console.error(
-      `agileflow hook: unknown event "${event}". Valid events: ${[...VALID_EVENTS].sort().join(', ')}`,
+    fail(
+      new InvalidArgumentError(`unknown event "${event}"`, {
+        suggestion: `use one of: ${[...VALID_EVENTS].sort().join(", ")}`,
+      }),
+      { command: "hook" },
     );
-    process.exit(1);
   }
   // Tool-related events MUST come with a matcher (Claude Code passes
   // tool_name in stdin, but we register matcher-keyed entries in
   // settings.json so each tool gets its own dispatcher invocation).
   if (MATCHER_EVENTS.has(event) && !options.matcher) {
-    // eslint-disable-next-line no-console
-    console.error(
-      `agileflow hook: event "${event}" requires --matcher (e.g. --matcher Bash)`,
+    fail(
+      new InvalidArgumentError(`event "${event}" requires --matcher`, {
+        suggestion: `add a tool name, e.g. --matcher Bash`,
+      }),
+      { command: "hook" },
     );
-    process.exit(1);
   }
 
   const projectDir = process.env.CLAUDE_PROJECT_DIR || process.cwd();
-  const agileflowDir = path.join(projectDir, '.agileflow');
+  const agileflowDir = path.join(projectDir, ".agileflow");
 
   /** @type {Buffer[]} */
   const chunks = [];
@@ -70,8 +73,8 @@ async function hook(event, options = {}) {
     process.exit(0);
   }
 
-  // PreCompact and Stop must NOT block, regardless of chain outcome.
-  if (event === 'PreCompact' || event === 'Stop') {
+  // PostCompact and Stop must NOT block, regardless of chain outcome.
+  if (event === "PostCompact" || event === "Stop") {
     process.exit(0);
   }
   process.exit(result.exitCode);