wogiflow 2.4.2 → 2.4.4

package/scripts/hooks/entry/claude-code/stop.js

@@ -14,85 +14,53 @@
 
 const { checkLoopExit } = require('../../core/loop-check');
 const { isRoutingPending, incrementStopAttempts } = require('../../core/routing-gate');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
-async function main() {
+runHook('Stop', async ({ parsedInput }) => {
+  // v6.2: Routing enforcement check — catches text-only response bypass
+  // If routing-pending flag is still set when the AI tries to stop, it means
+  // the AI responded to the user's message without ever invoking a /wogi-* command.
+  // This is the exact bypass we need to prevent (especially after context compaction).
   try {
-    const { input: rawInput } = await readHookInput();
-    const input = rawInput || {};
-    const parsedInput = claudeCodeAdapter.parseInput(input);
+    if (isRoutingPending()) {
+      // Use counter-based approach instead of clearing immediately.
+      // This gives the AI multiple chances to comply before giving up.
+      // Gap 4 fix: clearing immediately made this single-shot protection.
+      const { cleared, attempts } = incrementStopAttempts(10);
 
-    // v6.2: Routing enforcement check — catches text-only response bypass
-    // If routing-pending flag is still set when the AI tries to stop, it means
-    // the AI responded to the user's message without ever invoking a /wogi-* command.
-    // This is the exact bypass we need to prevent (especially after context compaction).
-    try {
-      if (isRoutingPending()) {
-        // Use counter-based approach instead of clearing immediately.
-        // This gives the AI multiple chances to comply before giving up.
-        // Gap 4 fix: clearing immediately made this single-shot protection.
-        const { cleared, attempts } = incrementStopAttempts(10);
-
-        if (cleared) {
-          // Max attempts reached — allow stop to prevent infinite loop
-          if (process.env.DEBUG) {
-            console.error(`[Stop] Max routing enforcement attempts reached (${attempts}), allowing stop`);
-          }
-          // Fall through to normal stop logic
-        } else {
-          // Block the stop — force the AI to route through /wogi-start
-          const routingMessage = [
-            `ROUTING VIOLATION (attempt ${attempts}/10): You MUST call Skill(skill="wogi-start") before responding.`,
-            '',
-            'Call Skill(skill="wogi-start", args="<user\'s message>") NOW. No text. No explanation. Just the Skill tool call.'
-          ].join('\n');
-
-          console.log(JSON.stringify({
-            continue: true, // Force continue — don't let the AI stop
-            stopReason: routingMessage
-          }));
-          process.exit(0);
-          return;
+      if (cleared) {
+        // Max attempts reached — allow stop to prevent infinite loop
+        if (process.env.DEBUG) {
+          console.error(`[Stop] Max routing enforcement attempts reached (${attempts}), allowing stop`);
         }
+        // Fall through to normal stop logic
+      } else {
+        // Block the stop — force the AI to route through /wogi-start
+        const routingMessage = [
+          `ROUTING VIOLATION (attempt ${attempts}/10): You MUST call Skill(skill="wogi-start") before responding.`,
+          '',
+          'Call Skill(skill="wogi-start", args="<user\'s message>") NOW. No text. No explanation. Just the Skill tool call.'
+        ].join('\n');
+
+        // Return raw output — skip adapter transform for routing enforcement
+        // (this needs { continue: true, stopReason } format directly)
+        return { __raw: true, continue: true, stopReason: routingMessage };
       }
-    } catch (err) {
-      // Fail-CLOSED for routing check — force continuation on errors.
-      // Gap 5 fix: failing open here disabled the last line of defense.
-      // Worst case: AI retries and hits the 3-attempt limit, which clears naturally.
-      if (process.env.DEBUG) {
-        console.error(`[Stop] Routing check error (fail-closed, forcing continue): ${err.message}`);
-      }
-      console.log(JSON.stringify({
-        continue: true,
-        stopReason: 'Routing enforcement check encountered an error. Please invoke /wogi-start with your request.'
-      }));
-      process.exit(0);
-      return;
     }
-
-    // Check if loop can exit
-    const coreResult = await checkLoopExit();
-
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('Stop', coreResult);
-
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
   } catch (err) {
-    // On error, allow stop (don't block user) — fail-open
-    try {
-      const { logHookError } = require('../../../flow-hook-errors');
-      logHookError('Stop', err, { failMode: 'open', operation: 'stop-check' });
-    } catch (logErr) {
-      console.error(`[WogiFlow] Stop hook error: ${err.message}`);
+    // Fail-CLOSED for routing check — force continuation on errors.
+    // Gap 5 fix: failing open here disabled the last line of defense.
+    // Worst case: AI retries and hits the 3-attempt limit, which clears naturally.
+    if (process.env.DEBUG) {
+      console.error(`[Stop] Routing check error (fail-closed, forcing continue): ${err.message}`);
     }
-    console.log(JSON.stringify({ continue: false })); // Allow stop
-    process.exit(0);
+    return {
+      __raw: true,
+      continue: true,
+      stopReason: 'Routing enforcement check encountered an error. Please invoke /wogi-start with your request.'
+    };
   }
-}
 
-// Handle stdin properly
-process.stdin.setEncoding('utf8');
-main();
+  // Check if loop can exit
+  return await checkLoopExit();
+}, { failMode: 'warn', failOutput: { continue: false } });

package/scripts/hooks/entry/claude-code/task-completed.js

@@ -8,32 +8,8 @@
  */
 
 const { handleTaskCompleted } = require('../../core/task-completed');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
-async function main() {
-  try {
-    const { input: parsedStdin } = await readHookInput();
-    const input = parsedStdin || {};
-    const parsedInput = claudeCodeAdapter.parseInput(input);
-
-    // Handle task completion
-    const coreResult = await handleTaskCompleted(parsedInput);
-
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('TaskCompleted', coreResult);
-
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Non-blocking error - don't prevent task completion
-    console.error(`[Wogi Flow Hook Error] ${err.message}`);
-    console.log(JSON.stringify({ continue: true }));
-    process.exit(0);
-  }
-}
-
-// Handle stdin properly
-process.stdin.setEncoding('utf8');
-main();
+runHook('TaskCompleted', async ({ parsedInput }) => {
+  return await handleTaskCompleted(parsedInput);
+}, { failMode: 'silent' });

package/scripts/hooks/entry/claude-code/task-created.js

@@ -0,0 +1,15 @@
+#!/usr/bin/env node
+
+/**
+ * Wogi Flow - Claude Code TaskCreated Hook
+ *
+ * Called when a native task is created via TaskCreate (Claude Code 2.1.84+).
+ * Links native tasks to the active WogiFlow task for tracking.
+ */
+
+const { handleTaskCreated } = require('../../core/task-created');
+const { runHook } = require('../shared/hook-runner');
+
+runHook('TaskCreated', async ({ parsedInput }) => {
+  return await handleTaskCreated(parsedInput);
+}, { failMode: 'silent' });

package/scripts/hooks/entry/claude-code/user-prompt-submit.js

@@ -12,241 +12,159 @@ const { checkImplementationGate } = require('../../core/implementation-gate');
 const { checkResearchRequirement } = require('../../core/research-gate');
 const { setRoutingPending, clearRoutingPending, ROUTING_CLEARED_PATH } = require('../../core/routing-gate');
 const { getPhaseContextPrompt } = require('../../core/phase-gate');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
 const { markSkillPending, loadDurableSession } = require('../../../flow-durable-session');
 const { captureCurrentPrompt } = require('../../../flow-prompt-capture');
 const { spawnBackgroundDetection } = require('../../../flow-correction-detector');
 const { getConfig } = require('../../../flow-utils');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
-async function main() {
-  try {
-    const { input: parsedStdin } = await readHookInput();
-
-    // Handle empty input gracefully
-    if (!parsedStdin) {
-      console.log(JSON.stringify({ continue: true, hookSpecificOutput: { hookEventName: 'UserPromptSubmit' } }));
-      process.exit(0);
-      return;
-    }
-
-    // Parse JSON safely with prototype pollution protection
-    let input;
-    try {
-      input = parsedStdin;
-      if (!input) {
-        // Invalid JSON - allow through (graceful degradation)
-        console.log(JSON.stringify({ continue: true, hookSpecificOutput: { hookEventName: 'UserPromptSubmit' } }));
-        process.exit(0);
-        return;
-      }
-    } catch (err) {
-      // Parse error - allow through (graceful degradation)
-      console.log(JSON.stringify({ continue: true, hookSpecificOutput: { hookEventName: 'UserPromptSubmit' } }));
-      process.exit(0);
-      return;
-    }
-
-    const parsedInput = claudeCodeAdapter.parseInput(input);
+runHook('UserPromptSubmit', async ({ input, parsedInput }) => {
+  // Handle empty input gracefully
+  if (!input || Object.keys(input).length === 0) {
+    return { __raw: true, continue: true, hookSpecificOutput: { hookEventName: 'UserPromptSubmit' } };
+  }
 
-    const prompt = parsedInput.prompt;
-    const source = parsedInput.source;
+  const prompt = parsedInput.prompt;
+  const source = parsedInput.source;
 
-    // v4.1: Detect skill commands that need execution tracking
-    // This prevents premature exit when /wogi-bulk or /wogi-start is entered
-    if (typeof prompt === 'string') {
-      const skillMatch = prompt.match(/^\/(wogi-bulk|wogi-start)\b/i);
-      if (skillMatch) {
-        const skillName = skillMatch[1].toLowerCase();
-        markSkillPending(skillName, { prompt });
-        if (process.env.DEBUG) {
-          console.error(`[Hook] Marked /${skillName} as pending execution`);
-        }
+  // v4.1: Detect skill commands that need execution tracking
+  if (typeof prompt === 'string') {
+    const skillMatch = prompt.match(/^\/(wogi-bulk|wogi-start)\b/i);
+    if (skillMatch) {
+      const skillName = skillMatch[1].toLowerCase();
+      markSkillPending(skillName, { prompt });
+      if (process.env.DEBUG) {
+        console.error(`[Hook] Marked /${skillName} as pending execution`);
       }
     }
+  }
 
-    // Load config once for feature flag checks
-    let hookConfig;
-    try {
-      hookConfig = getConfig();
-    } catch (err) {
-      hookConfig = {};
-    }
+  // Load config once for feature flag checks
+  let hookConfig;
+  try {
+    hookConfig = getConfig();
+  } catch (err) {
+    hookConfig = {};
+  }
 
-    // v5.0: Capture prompt for learning system (non-blocking)
-    // Controlled by hooks.rules.intelligence.promptCapture.enabled
-    if (hookConfig.hooks?.rules?.intelligence?.promptCapture?.enabled !== false) {
-      if (typeof prompt === 'string' && prompt.trim().length > 0) {
+  // v5.0: Capture prompt for learning system (non-blocking)
+  if (hookConfig.hooks?.rules?.intelligence?.promptCapture?.enabled !== false) {
+    if (typeof prompt === 'string' && prompt.trim().length > 0) {
+      setImmediate(() => {
         try {
           captureCurrentPrompt(prompt);
         } catch (err) {
-          // Non-blocking - don't fail the hook if capture fails
           if (process.env.DEBUG) {
             console.error(`[Hook] Prompt capture failed: ${err.message}`);
           }
         }
-      }
-    }
-
-    // v5.1→v7.0: Detect corrections for learning system (AI-only, non-blocking)
-    // Controlled by hooks.rules.intelligence.correctionDetection.enabled
-    if (hookConfig.hooks?.rules?.intelligence?.correctionDetection?.enabled !== false) {
-      if (typeof prompt === 'string' && prompt.trim().length > 0) {
-        try {
-          const session = loadDurableSession();
-          spawnBackgroundDetection(prompt, session?.taskId || '');
-        } catch (err) {
-          // Non-blocking - don't fail the hook if detection spawn fails
-          if (process.env.DEBUG) {
-            console.error(`[Hook] Correction detection spawn failed: ${err.message}`);
-          }
-        }
-      }
+      });
     }
+  }
 
-    // v6.0: Set routing-pending flag for routing gate enforcement
-    // This blocks ALL gated tool calls until a /wogi-* skill is invoked
-    // v8.0: Always set, even with active tasks — every turn must route through /wogi-start.
-    // Exception: skipped when the prompt IS a /wogi-* command (see isWogiCommand below).
-    // v6.1: Also skip when the prompt IS a /wogi-* command — the user is already routing.
-    // When users type "/wogi-start ..." directly, Claude Code expands the skill inline
-    // (not through the Skill tool), so clearRoutingPending() in PreToolUse never fires.
-    // Setting the flag here would create an uncleable block.
-    // Tightened regex: only match /wogi-[lowercase-alphanumeric-hyphens] to prevent
-    // injection via crafted prompts like "/wogi-<script>" or "/wogi-../../path"
-    const isWogiCommand = typeof prompt === 'string' && /^\/wogi-[a-z0-9-]+\b/i.test(prompt.trim());
-    if (!isWogiCommand) {
-      // v8.1: Delete any stale cleared marker from previous turns.
-      // The cleared marker prevents flag re-setting during skill chains (same AI response).
-      // But across user turns, it must not persist — otherwise tools are unblocked without
-      // routing for the duration of the marker's TTL. A new user prompt (non-wogi-command)
-      // is unambiguously a new turn, so the old marker is invalidated.
-      try {
-        fs.unlinkSync(ROUTING_CLEARED_PATH);
-      } catch (err) {
-        // ENOENT is fine — no marker to delete
-        if (err.code !== 'ENOENT' && process.env.DEBUG) {
-          console.error(`[Hook] Failed to delete cleared marker: ${err.message}`);
-        }
-      }
-
-      try {
-        setRoutingPending();
-      } catch (err) {
-        // Non-blocking - don't fail the hook if routing gate fails (fail-open)
-        if (process.env.DEBUG) {
-          console.error(`[Hook] Routing gate set failed: ${err.message}`);
-        }
-      }
-    } else {
-      // v6.2: Actively CLEAR any existing routing flag when user explicitly types a /wogi-* command.
-      // Previously we only skipped setting it, but a flag from a prior prompt would persist and
-      // block tool calls inside the /wogi-* command when Claude Code expands it inline (not via Skill tool).
+  // v5.1->v7.0: Detect corrections for learning system (AI-only, non-blocking)
+  if (hookConfig.hooks?.rules?.intelligence?.correctionDetection?.enabled !== false) {
+    if (typeof prompt === 'string' && prompt.trim().length > 0) {
       try {
-        clearRoutingPending();
-        if (process.env.DEBUG) {
-          console.error(`[Hook] Cleared routing flag — prompt is a /wogi-* command`);
-        }
+        const session = loadDurableSession();
+        spawnBackgroundDetection(prompt, session?.taskId || '');
       } catch (err) {
         if (process.env.DEBUG) {
-          console.error(`[Hook] Routing gate clear failed: ${err.message}`);
+          console.error(`[Hook] Correction detection spawn failed: ${err.message}`);
         }
       }
     }
+  }
 
-    // Phase context injection (just-in-time phase-specific instructions)
-    let phasePrompt = null;
+  // v6.0: Set routing-pending flag for routing gate enforcement
+  const isWogiCommand = typeof prompt === 'string' && /^\/wogi-[a-z0-9-]+\b/i.test(prompt.trim());
+  if (!isWogiCommand) {
+    // v8.1: Delete any stale cleared marker from previous turns.
     try {
-      const phaseContext = getPhaseContextPrompt();
-      if (phaseContext.inject && phaseContext.prompt) {
-        phasePrompt = phaseContext.prompt;
+      fs.unlinkSync(ROUTING_CLEARED_PATH);
+    } catch (err) {
+      if (err.code !== 'ENOENT' && process.env.DEBUG) {
+        console.error(`[Hook] Failed to delete cleared marker: ${err.message}`);
       }
+    }
+
+    try {
+      setRoutingPending();
     } catch (err) {
-      // Non-blocking - phase context is best-effort
       if (process.env.DEBUG) {
-        console.error(`[Hook] Phase context injection failed: ${err.message}`);
+        console.error(`[Hook] Routing gate set failed: ${err.message}`);
       }
     }
-
-    // Check research gate first (before implementation gate)
-    // Auto-triggers research protocol for capability/existence/feasibility questions
-    const researchResult = checkResearchRequirement({
-      prompt,
-      source
-    });
-
-    // Check implementation gate
-    let coreResult = checkImplementationGate({
-      prompt,
-      source
-    });
-
-    // If research protocol should be injected, add it to system reminder
-    if (researchResult.injectProtocol && researchResult.protocolSteps) {
-      coreResult = {
-        ...coreResult,
-        systemReminder: researchResult.protocolSteps,
-        researchTriggered: true,
-        questionType: researchResult.questionType,
-        suggestedDepth: researchResult.suggestedDepth
-      };
-    } else if (researchResult.warning && coreResult.allowed) {
-      // Soft warning mode (not strict)
-      coreResult = {
-        ...coreResult,
-        warning: true,
-        researchWarning: researchResult.message,
-        suggestedCommand: researchResult.suggestedCommand
-      };
+  } else {
+    // v6.2: Actively CLEAR any existing routing flag when user explicitly types a /wogi-* command.
+    try {
+      clearRoutingPending();
+      if (process.env.DEBUG) {
+        console.error(`[Hook] Cleared routing flag — prompt is a /wogi-* command`);
+      }
+    } catch (err) {
+      if (process.env.DEBUG) {
+        console.error(`[Hook] Routing gate clear failed: ${err.message}`);
+      }
     }
+  }
 
-    // Inject phase-specific context prompt (additionalSystemPrompt)
-    if (phasePrompt) {
-      coreResult = {
-        ...coreResult,
-        phasePrompt
-      };
+  // Phase context injection
+  let phasePrompt = null;
+  try {
+    const phaseContext = getPhaseContextPrompt();
+    if (phaseContext.inject && phaseContext.prompt) {
+      phasePrompt = phaseContext.prompt;
     }
-
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('UserPromptSubmit', coreResult);
-
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
   } catch (err) {
-    // Fail-closed: block the prompt on hook errors to prevent untracked implementation
-    // Users installed WogiFlow to enforce task tracking - failing open would bypass that
     if (process.env.DEBUG) {
-      console.error(`[Wogi Flow Hook Error] ${err.message}`);
-    } else {
-      console.error('[Wogi Flow Hook] Validation error occurred');
+      console.error(`[Hook] Phase context injection failed: ${err.message}`);
     }
-    console.log(JSON.stringify({
-      decision: 'block',
-      reason: 'WogiFlow validation error. Please check your WogiFlow setup or use /wogi-start to route your request.'
-    }));
-    process.exit(0);
   }
-}
 
-// Handle stdin properly
-process.stdin.setEncoding('utf8');
+  // Check research gate first (before implementation gate)
+  const researchResult = checkResearchRequirement({
+    prompt,
+    source
+  });
+
+  // Check implementation gate
+  let coreResult = checkImplementationGate({
+    prompt,
+    source
+  });
+
+  // If research protocol should be injected, add it to system reminder
+  if (researchResult.injectProtocol && researchResult.protocolSteps) {
+    coreResult = {
+      ...coreResult,
+      systemReminder: researchResult.protocolSteps,
+      researchTriggered: true,
+      questionType: researchResult.questionType,
+      suggestedDepth: researchResult.suggestedDepth
+    };
+  } else if (researchResult.warning && coreResult.allowed) {
+    coreResult = {
+      ...coreResult,
+      warning: true,
+      researchWarning: researchResult.message,
+      suggestedCommand: researchResult.suggestedCommand
+    };
+  }
 
-// Must await async main() to prevent race conditions
-// Without await, Node.js may exit before stdin finishes reading
-(async () => {
-  try {
-    await main();
-  } catch (err) {
-    // Fail-closed: block on unexpected errors to prevent untracked implementation
-    if (process.env.DEBUG) {
-      console.error(`[Wogi Flow Hook] Unexpected error: ${err.message}`);
-    }
-    console.log(JSON.stringify({
-      decision: 'block',
-      reason: 'WogiFlow hook error. Use /wogi-start to route your request.'
-    }));
-    process.exit(0);
+  // Inject phase-specific context prompt
+  if (phasePrompt) {
+    coreResult = {
+      ...coreResult,
+      phasePrompt
+    };
+  }
+
+  return coreResult;
+}, {
+  failMode: 'block',
+  failOutput: {
+    decision: 'block',
+    reason: 'WogiFlow validation error. Please check your WogiFlow setup or use /wogi-start to route your request.'
   }
-})();
+});

package/scripts/hooks/entry/claude-code/worktree-create.js

@@ -11,29 +11,10 @@
  */
 
 const { handleWorktreeCreate } = require('../../core/worktree-lifecycle');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
-process.stdin.setEncoding('utf8');
-
-async function main() {
-  try {
-    const { input: parsedStdin } = await readHookInput();
-    const input = parsedStdin || {};
-
-    const worktreePath = input.worktree_path || input.worktreePath || '';
-    const projectRoot = input.cwd || process.cwd();
-
-    const result = handleWorktreeCreate({ worktreePath, projectRoot });
-    const output = claudeCodeAdapter.transformResult('WorktreeCreate', result);
-
-    process.stdout.write(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Never block on worktree lifecycle errors
-    process.stdout.write(JSON.stringify({ continue: true }));
-    process.exit(0);
-  }
-}
-
-main();
+runHook('WorktreeCreate', async ({ input }) => {
+  const worktreePath = input.worktree_path || input.worktreePath || '';
+  const projectRoot = input.cwd || process.cwd();
+  return handleWorktreeCreate({ worktreePath, projectRoot });
+}, { failMode: 'silent', useStdoutWrite: true });

package/scripts/hooks/entry/claude-code/worktree-remove.js

@@ -11,29 +11,10 @@
  */
 
 const { handleWorktreeRemove } = require('../../core/worktree-lifecycle');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
-process.stdin.setEncoding('utf8');
-
-async function main() {
-  try {
-    const { input: parsedStdin } = await readHookInput();
-    const input = parsedStdin || {};
-
-    const worktreePath = input.worktree_path || input.worktreePath || '';
-    const projectRoot = input.cwd || process.cwd();
-
-    const result = handleWorktreeRemove({ worktreePath, projectRoot });
-    const output = claudeCodeAdapter.transformResult('WorktreeRemove', result);
-
-    process.stdout.write(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Never block on worktree lifecycle errors
-    process.stdout.write(JSON.stringify({ continue: true }));
-    process.exit(0);
-  }
-}
-
-main();
+runHook('WorktreeRemove', async ({ input }) => {
+  const worktreePath = input.worktree_path || input.worktreePath || '';
+  const projectRoot = input.cwd || process.cwd();
+  return handleWorktreeRemove({ worktreePath, projectRoot });
+}, { failMode: 'silent', useStdoutWrite: true });