wogiflow 2.4.2 → 2.4.4

package/scripts/hooks/entry/claude-code/session-end.js

@@ -8,32 +8,8 @@
  */
 
 const { handleSessionEnd } = require('../../core/session-end');
-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 session end
-    const coreResult = handleSessionEnd(parsedInput);
-
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('SessionEnd', coreResult);
-
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Non-blocking error
-    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('SessionEnd', async ({ parsedInput }) => {
+  return handleSessionEnd(parsedInput);
+}, { failMode: 'silent' });

package/scripts/hooks/entry/claude-code/session-start.js

@@ -8,12 +8,11 @@
  */
 
 const { gatherSessionContext } = require('../../core/session-context');
-const { claudeCodeAdapter } = require('../../adapters/claude-code');
 const { setCliSessionId, clearStaleCurrentTaskAsync } = require('../../../flow-session-state');
 const { checkAndResetStalePhase } = require('../../core/phase-gate');
 const { setRoutingPending } = require('../../core/routing-gate');
 const { getConfig } = require('../../../flow-utils');
-const { readHookInput } = require('../shared/read-stdin');
+const { runHook } = require('../shared/hook-runner');
 
 // Lazy-load bridge state to avoid circular dependencies
 let autoSyncBridge = null;
@@ -28,294 +27,257 @@ function getAutoSyncBridge() {
   return autoSyncBridge;
 }
 
-async function main() {
-  try {
-    // Start bridge auto-sync in parallel with stdin reading (both are independent I/O)
-    const bridgeSyncPromise = (async () => {
-      try {
-        const syncFn = getAutoSyncBridge();
-        await syncFn('claude-code', { silent: true });
-      } catch (err) {
-        if (process.env.DEBUG) {
-          console.error(`[session-start] Bridge auto-sync failed: ${err.message}`);
-        }
-      }
-    })();
-
-    // Read input from stdin (runs concurrently with bridge sync)
-    const { input: parsedStdin } = await readHookInput();
-    const input = parsedStdin || {};
-    const parsedInput = claudeCodeAdapter.parseInput(input);
-
-    // Wait for bridge sync to complete before proceeding
-    await bridgeSyncPromise;
-
-    // CLAUDE.md drift detection — check if manually edited since last sync
-    let driftDetected = false;
-    let driftMarkerMissing = false;
-    try {
-      const { checkClaudeMdDrift } = require('../../../flow-bridge-state');
-      const drift = checkClaudeMdDrift();
-      if (drift.drifted && drift.reason === 'content-changed') {
-        if (process.env.DEBUG) {
-          console.error('[session-start] CLAUDE.md drift detected — content changed since last sync');
-        }
-        driftDetected = true;
-      } else if (drift.drifted && drift.reason === 'marker-missing') {
-        if (process.env.DEBUG) {
-          console.error('[session-start] CLAUDE.md appears manually maintained (no generation marker)');
-        }
-        driftDetected = true;
-        driftMarkerMissing = true;
-      }
-    } catch (err) {
-      if (process.env.DEBUG) {
-        console.error(`[session-start] Drift detection failed: ${err.message}`);
-      }
-    }
-
-    // --- Version compatibility checks (parallelized) ---
-    // 1. Claude Code: warns if below hard minimum (2.1.23) where hooks don't work
-    // 2. WogiFlow npm: warns if a newer version is available (checks once per 24h)
-    let versionWarning = null;
-    let updateWarning = null;
+runHook('SessionStart', async ({ parsedInput }) => {
+  // Start bridge auto-sync in parallel with other init work
+  const bridgeSyncPromise = (async () => {
     try {
-      const { checkClaudeCodeVersionOnce, checkWogiFlowUpdateOnce } = require('../../../flow-version-check');
-      const [vw, uw] = await Promise.all([
-        (async () => { try { return await checkClaudeCodeVersionOnce(); } catch (_err) { return null; } })(),
-        (async () => { try { return await checkWogiFlowUpdateOnce(); } catch (_err) { return null; } })()
-      ]);
-      versionWarning = vw;
-      updateWarning = uw;
+      const syncFn = getAutoSyncBridge();
+      await syncFn('claude-code', { silent: true });
     } catch (err) {
       if (process.env.DEBUG) {
-        console.error(`[session-start] Version check failed: ${err.message}`);
+        console.error(`[session-start] Bridge auto-sync failed: ${err.message}`);
       }
     }
+  })();
 
-    // --- Batch 1: Independent pre-context operations (async + sync) ---
-    // These all operate on separate state files and have no data dependencies.
+  // Wait for bridge sync to complete
+  await bridgeSyncPromise;
 
-    // Sync operations (run immediately, no await needed)
-    let scriptWarnings = [];
-    try {
-      const wasReset = checkAndResetStalePhase();
-      if (wasReset && process.env.DEBUG) {
-        console.error('[session-start] Reset stale workflow phase to idle');
+  // CLAUDE.md drift detection — check if manually edited since last sync
+  let driftDetected = false;
+  let driftMarkerMissing = false;
+  try {
+    const { checkClaudeMdDrift } = require('../../../flow-bridge-state');
+    const drift = checkClaudeMdDrift();
+    if (drift.drifted && drift.reason === 'content-changed') {
+      if (process.env.DEBUG) {
+        console.error('[session-start] CLAUDE.md drift detected — content changed since last sync');
       }
-    } catch (err) {
+      driftDetected = true;
+    } else if (drift.drifted && drift.reason === 'marker-missing') {
       if (process.env.DEBUG) {
-        console.error(`[session-start] Failed to check stale phase: ${err.message}`);
+        console.error('[session-start] CLAUDE.md appears manually maintained (no generation marker)');
       }
+      driftDetected = true;
+      driftMarkerMissing = true;
+    }
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Drift detection failed: ${err.message}`);
     }
+  }
 
-    try {
-      const routingResult = setRoutingPending();
-      if (process.env.DEBUG) {
-        console.error(`[session-start] Set routing-pending: ${routingResult.reason}`);
-      }
-    } catch (err) {
-      if (process.env.DEBUG) {
-        console.error(`[session-start] Failed to set routing-pending: ${err.message}`);
-      }
+  // --- Version compatibility checks (parallelized) ---
+  let versionWarning = null;
+  let updateWarning = null;
+  try {
+    const { checkClaudeCodeVersionOnce, checkWogiFlowUpdateOnce } = require('../../../flow-version-check');
+    const [vw, uw] = await Promise.all([
+      (async () => { try { return await checkClaudeCodeVersionOnce(); } catch (_err) { return null; } })(),
+      (async () => { try { return await checkWogiFlowUpdateOnce(); } catch (_err) { return null; } })()
+    ]);
+    versionWarning = vw;
+    updateWarning = uw;
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Version check failed: ${err.message}`);
     }
+  }
 
-    try {
-      const { validateScripts } = require('../../../flow-script-resolver');
-      scriptWarnings = validateScripts();
-    } catch (err) {
-      if (process.env.DEBUG) {
-        console.error(`[session-start] Script validation failed: ${err.message}`);
-      }
+  // --- Batch 1: Independent pre-context operations (async + sync) ---
+  let scriptWarnings = [];
+  try {
+    const wasReset = checkAndResetStalePhase();
+    if (wasReset && process.env.DEBUG) {
+      console.error('[session-start] Reset stale workflow phase to idle');
+    }
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Failed to check stale phase: ${err.message}`);
     }
+  }
 
-    // BUG-005 fix: Create durable-session.json for active tasks on session start.
-    // Without this, the first prompt of every task is lost because user-prompt-submit
-    // fires BEFORE post-tool-use (which creates the session file). This also fixes
-    // session resume after context compaction — durable-session.json may have been
-    // archived but the task is still inProgress in ready.json.
-    try {
-      const { getReadyData } = require('../../../flow-utils');
-      const readyData = getReadyData();
-      if (Array.isArray(readyData.inProgress) && readyData.inProgress.length > 0) {
-        const task = readyData.inProgress[0];
-        const taskId = task && task.id;
-        if (taskId) {
-          const { loadDurableSession, createDurableSession } = require('../../../flow-durable-session');
-          const existing = loadDurableSession();
-          if (!existing || existing.taskId !== taskId) {
-            const criteria = task.acceptanceCriteria || task.scenarios || [];
-            const steps = Array.isArray(criteria) ? criteria : [];
-            const sessionSteps = steps.length > 0 ? steps : [task.title || taskId];
-            createDurableSession(taskId, 'task', sessionSteps);
-            if (process.env.DEBUG) {
-              console.error(`[session-start] Created durable session for active task ${taskId}`);
-            }
-          }
-        }
-      }
-    } catch (err) {
-      // Non-blocking — prompt capture is best-effort
-      if (process.env.DEBUG) {
-        console.error(`[session-start] Durable session init failed: ${err.message}`);
-      }
+  try {
+    const routingResult = setRoutingPending();
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Set routing-pending: ${routingResult.reason}`);
     }
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Failed to set routing-pending: ${err.message}`);
+    }
+  }
 
-    // Async operations — batch with Promise.all (both use file locking, independent targets)
-    const asyncPreOps = [];
+  try {
+    const { validateScripts } = require('../../../flow-script-resolver');
+    scriptWarnings = validateScripts();
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Script validation failed: ${err.message}`);
+    }
+  }
 
-    if (parsedInput.sessionId) {
-      asyncPreOps.push(
-        setCliSessionId(parsedInput.sessionId).catch(err => {
+  // BUG-005 fix: Create durable-session.json for active tasks on session start.
+  try {
+    const { getReadyData } = require('../../../flow-utils');
+    const readyData = getReadyData();
+    if (Array.isArray(readyData.inProgress) && readyData.inProgress.length > 0) {
+      const task = readyData.inProgress[0];
+      const taskId = task && task.id;
+      if (taskId) {
+        const { loadDurableSession, createDurableSession } = require('../../../flow-durable-session');
+        const existing = loadDurableSession();
+        if (!existing || existing.taskId !== taskId) {
+          const criteria = task.acceptanceCriteria || task.scenarios || [];
+          const steps = Array.isArray(criteria) ? criteria : [];
+          const sessionSteps = steps.length > 0 ? steps : [task.title || taskId];
+          createDurableSession(taskId, 'task', sessionSteps);
           if (process.env.DEBUG) {
-            console.error(`[session-start] Failed to store session ID: ${err.message}`);
+            console.error(`[session-start] Created durable session for active task ${taskId}`);
           }
-        })
-      );
+        }
+      }
     }
+  } catch (err) {
+    if (process.env.DEBUG) {
+      console.error(`[session-start] Durable session init failed: ${err.message}`);
+    }
+  }
+
+  // Async operations — batch with Promise.all
+  const asyncPreOps = [];
 
+  if (parsedInput.sessionId) {
     asyncPreOps.push(
-      clearStaleCurrentTaskAsync().catch(err => {
+      setCliSessionId(parsedInput.sessionId).catch(err => {
         if (process.env.DEBUG) {
-          console.error(`[session-start] Failed to clear stale task: ${err.message}`);
+          console.error(`[session-start] Failed to store session ID: ${err.message}`);
         }
       })
     );
+  }
 
-    // Gather session context concurrently with the async pre-ops
-    const [, coreResult] = await Promise.all([
-      Promise.all(asyncPreOps),
-      gatherSessionContext({
-        includeSuspended: true,
-        includeDecisions: true,
-        includeActivity: true
-      })
-    ]);
-
-    // --- Batch 2: Post-context operations (plugin scan + community pull) ---
-    // Both modify coreResult.context but touch different keys, so they can run concurrently.
-
-    const postContextOps = [];
-
-    // Plugin auto-scan (non-blocking)
-    postContextOps.push((async () => {
-      try {
-        const config = getConfig();
-        if (config.plugins?.enabled && config.plugins?.autoScanOnSessionStart) {
-          const { scanUnregisteredMcpServers, registerPlugin, deactivateStaleMcpPlugins, listPlugins } = require('../../../flow-plugin-registry');
-
-          const unregistered = scanUnregisteredMcpServers();
-          for (const server of unregistered) {
-            registerPlugin({
-              name: server.serverName,
-              description: `Auto-discovered MCP server: ${server.serverName}`,
-              source: 'auto-scan',
-              triggers: [`use ${server.serverName}`, `send to ${server.serverName}`, server.serverName],
-              capabilities: [],
-              metadata: { mcpServer: server.serverName }
-            });
-            if (process.env.DEBUG) {
-              console.error(`[session-start] Auto-registered plugin: ${server.serverName}`);
-            }
+  asyncPreOps.push(
+    clearStaleCurrentTaskAsync().catch(err => {
+      if (process.env.DEBUG) {
+        console.error(`[session-start] Failed to clear stale task: ${err.message}`);
+      }
+    })
+  );
+
+  // Gather session context concurrently with the async pre-ops
+  const [, coreResult] = await Promise.all([
+    Promise.all(asyncPreOps),
+    gatherSessionContext({
+      includeSuspended: true,
+      includeDecisions: true,
+      includeActivity: true
+    })
+  ]);
+
+  // --- Batch 2: Post-context operations (plugin scan + community pull) ---
+  const postContextOps = [];
+
+  // Plugin auto-scan (non-blocking)
+  postContextOps.push((async () => {
+    try {
+      const config = getConfig();
+      if (config.plugins?.enabled && config.plugins?.autoScanOnSessionStart) {
+        const { scanUnregisteredMcpServers, registerPlugin, deactivateStaleMcpPlugins, listPlugins } = require('../../../flow-plugin-registry');
+
+        const unregistered = scanUnregisteredMcpServers();
+        for (const server of unregistered) {
+          registerPlugin({
+            name: server.serverName,
+            description: `Auto-discovered MCP server: ${server.serverName}`,
+            source: 'auto-scan',
+            triggers: [`use ${server.serverName}`, `send to ${server.serverName}`, server.serverName],
+            capabilities: [],
+            metadata: { mcpServer: server.serverName }
+          });
+          if (process.env.DEBUG) {
+            console.error(`[session-start] Auto-registered plugin: ${server.serverName}`);
           }
+        }
 
-          const deactivated = deactivateStaleMcpPlugins();
-          if (deactivated.length > 0 && process.env.DEBUG) {
-            console.error(`[session-start] Deactivated ${deactivated.length} stale plugin(s): ${deactivated.join(', ')}`);
-          }
+        const deactivated = deactivateStaleMcpPlugins();
+        if (deactivated.length > 0 && process.env.DEBUG) {
+          console.error(`[session-start] Deactivated ${deactivated.length} stale plugin(s): ${deactivated.join(', ')}`);
+        }
 
-          if (coreResult && coreResult.context) {
-            const activePlugins = listPlugins({ activeOnly: true });
-            if (unregistered.length > 0 || activePlugins.length > 0) {
-              coreResult.context.pluginScan = {
-                newlyRegistered: unregistered.map(s => s.serverName),
-                activePlugins: activePlugins.map(p => ({ name: p.name, capabilities: (p.capabilities || []).length }))
-              };
-            }
+        if (coreResult && coreResult.context) {
+          const activePlugins = listPlugins({ activeOnly: true });
+          if (unregistered.length > 0 || activePlugins.length > 0) {
+            coreResult.context.pluginScan = {
+              newlyRegistered: unregistered.map(s => s.serverName),
+              activePlugins: activePlugins.map(p => ({ name: p.name, capabilities: (p.capabilities || []).length }))
+            };
           }
         }
-      } catch (err) {
-        if (process.env.DEBUG) {
-          console.error(`[session-start] Plugin auto-scan failed: ${err.message}`);
-        }
       }
-    })());
-
-    // Community knowledge pull + suggestion retry (non-blocking)
-    postContextOps.push((async () => {
-      try {
-        const communityConfig = getConfig();
-        if (communityConfig.community?.enabled) {
-          const community = require('../../../flow-community');
-
-          community.retryPendingSuggestions(communityConfig).catch(() => {});
-
-          if (communityConfig.community?.pullOnSessionStart !== false) {
-            const knowledge = await community.pullFromServer(communityConfig);
-            if (knowledge && coreResult && coreResult.context) {
-              coreResult.context.communityKnowledge = knowledge;
+    } catch (err) {
+      if (process.env.DEBUG) {
+        console.error(`[session-start] Plugin auto-scan failed: ${err.message}`);
+      }
+    }
+  })());
 
-              try {
-                community.mergeCommunityKnowledge(knowledge, communityConfig);
-              } catch (err) {
-                if (process.env.DEBUG) {
-                  console.error(`[session-start] Community merge failed: ${err.message}`);
-                }
+  // Community knowledge pull + suggestion retry (non-blocking)
+  postContextOps.push((async () => {
+    try {
+      const communityConfig = getConfig();
+      if (communityConfig.community?.enabled) {
+        const community = require('../../../flow-community');
+
+        community.retryPendingSuggestions(communityConfig).catch(() => {});
+
+        if (communityConfig.community?.pullOnSessionStart !== false) {
+          const knowledge = await community.pullFromServer(communityConfig);
+          if (knowledge && coreResult && coreResult.context) {
+            coreResult.context.communityKnowledge = knowledge;
+
+            try {
+              community.mergeCommunityKnowledge(knowledge, communityConfig);
+            } catch (err) {
+              if (process.env.DEBUG) {
+                console.error(`[session-start] Community merge failed: ${err.message}`);
               }
             }
           }
         }
-      } catch (err) {
-        if (process.env.DEBUG) {
-          console.error(`[session-start] Community pull failed: ${err.message}`);
-        }
       }
-    })());
-
-    await Promise.all(postContextOps);
-
-    // Inject script warnings into context (if any)
-    if (scriptWarnings.length > 0 && coreResult && coreResult.context) {
-      coreResult.context.scriptWarnings = scriptWarnings.map(w => w.message);
+    } catch (err) {
+      if (process.env.DEBUG) {
+        console.error(`[session-start] Community pull failed: ${err.message}`);
+      }
     }
+  })());
 
-    // Inject version compatibility warning (if any)
-    if (versionWarning && coreResult && coreResult.context) {
-      coreResult.context.versionWarning = versionWarning;
-    }
+  await Promise.all(postContextOps);
 
-    // Inject WogiFlow update warning (if any)
-    if (updateWarning && coreResult && coreResult.context) {
-      coreResult.context.updateWarning = updateWarning;
-    }
+  // Inject script warnings into context (if any)
+  if (scriptWarnings.length > 0 && coreResult && coreResult.context) {
+    coreResult.context.scriptWarnings = scriptWarnings.map(w => w.message);
+  }
 
-    // Inject drift detection results (if any)
-    if (driftDetected && coreResult && coreResult.context) {
-      if (driftMarkerMissing) {
-        coreResult.context.driftWarning = 'CLAUDE.md appears to have been manually edited (generation marker missing). Was this intentional? If yes, WogiFlow will respect your custom CLAUDE.md. If not, run `flow bridge sync` to regenerate from template.';
-      } else {
-        coreResult.context.driftWarning = 'CLAUDE.md content has changed since the last bridge sync. Was this intentional? If yes, WogiFlow will preserve your changes. If not, run `flow bridge sync` to regenerate from template.';
-      }
-    }
+  // Inject version compatibility warning (if any)
+  if (versionWarning && coreResult && coreResult.context) {
+    coreResult.context.versionWarning = versionWarning;
+  }
 
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('SessionStart', coreResult);
+  // Inject WogiFlow update warning (if any)
+  if (updateWarning && coreResult && coreResult.context) {
+    coreResult.context.updateWarning = updateWarning;
+  }
 
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Non-blocking error - log with unified handler, exit 1
-    try {
-      const { logHookError } = require('../../../flow-hook-errors');
-      logHookError('SessionStart', err, { failMode: 'open', operation: 'session-initialization' });
-    } catch (logErr) {
-      console.error(`[WogiFlow] SessionStart hook error: ${err.message}`);
+  // Inject drift detection results (if any)
+  if (driftDetected && coreResult && coreResult.context) {
+    if (driftMarkerMissing) {
+      coreResult.context.driftWarning = 'CLAUDE.md appears to have been manually edited (generation marker missing). Was this intentional? If yes, WogiFlow will respect your custom CLAUDE.md. If not, run `flow bridge sync` to regenerate from template.';
+    } else {
+      coreResult.context.driftWarning = 'CLAUDE.md content has changed since the last bridge sync. Was this intentional? If yes, WogiFlow will preserve your changes. If not, run `flow bridge sync` to regenerate from template.';
     }
-    process.exit(1);
   }
-}
 
-// Handle stdin properly
-process.stdin.setEncoding('utf8');
-main();
+  return coreResult;
+}, { failMode: 'warn' });

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

@@ -10,55 +10,14 @@
  */
 
 const { handleSetup, handleMaintenance } = require('../../core/setup-handler');
-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);
+runHook('Setup', async ({ parsedInput }) => {
+  const trigger = parsedInput.source || 'init';
+  const isMaintenance = trigger === 'maintenance' || trigger === '--maintenance';
 
-    // Determine what type of setup event this is
-    // Claude Code passes the trigger via source or a specific field
-    const trigger = parsedInput.source || 'init';
-    const isMaintenance = trigger === 'maintenance' || trigger === '--maintenance';
-
-    let coreResult;
-
-    if (isMaintenance) {
-      // Run maintenance tasks
-      coreResult = handleMaintenance({
-        cwd: parsedInput.cwd
-      });
-    } else {
-      // Run setup check
-      coreResult = handleSetup({
-        trigger,
-        cwd: parsedInput.cwd
-      });
-    }
-
-    // Transform to Claude Code format
-    const output = claudeCodeAdapter.transformResult('Setup', coreResult);
-
-    // Output JSON
-    console.log(JSON.stringify(output));
-    process.exit(0);
-  } catch (err) {
-    // Non-blocking error - log to stderr, exit with allow
-    console.error(`[Wogi Flow Hook Error] ${err.message}`);
-    // Exit 0 with allow to not block on hook errors (graceful degradation)
-    console.log(JSON.stringify({
-      continue: true,
-      hookSpecificOutput: {
-        hookEventName: 'Setup'
-      }
-    }));
-    process.exit(0);
+  if (isMaintenance) {
+    return handleMaintenance({ cwd: parsedInput.cwd });
   }
-}
-
-// Handle stdin properly
-process.stdin.setEncoding('utf8');
-main();
+  return handleSetup({ trigger, cwd: parsedInput.cwd });
+}, { failMode: 'silent' });