specweave 1.0.550 → 1.0.552

package/plugins/specweave/hooks/universal/dispatcher.mjs

@@ -1,336 +0,0 @@
-#!/usr/bin/env node
-/**
- * Universal Hook Dispatcher (Cross-Platform)
- *
- * Routes hook calls to the appropriate TypeScript implementation.
- * Works on Windows, macOS, and Linux.
- *
- * Usage:
- *   node dispatcher.mjs <hook-type>
- *
- * Where hook-type is one of:
- *   - session-start
- *   - post-tool-use
- *   - completion-guard
- *   - increment-duplicate-guard
- *
- * @module hooks/universal/dispatcher
- */
-
-import { spawn } from 'child_process';
-import { fileURLToPath } from 'url';
-import { dirname, join, resolve } from 'path';
-import { existsSync } from 'fs';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
-// Hook type from arguments
-const hookType = process.argv[2] || 'unknown';
-
-// Global timeout for hook execution (30 seconds max)
-const HOOK_TIMEOUT_MS = 30000;
-
-/**
- * Detect if we're in a development environment with duplicate hooks
- *
- * Problem: When working IN the specweave project AND having the marketplace
- * plugin installed, Claude Code loads hooks from BOTH sources, causing:
- * - Duplicate hook execution
- * - Potential race conditions
- * - Confusing output
- *
- * Solution: If we detect we're running from the marketplace copy while
- * the CWD is the specweave project itself, skip execution (let local hooks handle it).
- *
- * @returns {boolean} true if we should skip this hook execution
- */
-function shouldSkipDueToDevEnvironment() {
-  const pluginRoot = process.env.CLAUDE_PLUGIN_ROOT || '';
-  const cwd = process.cwd();
-
-  // Check if running from marketplace (installed plugin)
-  const isFromMarketplace = pluginRoot.includes('.claude/plugins/marketplaces/');
-
-  // Check if CWD is the specweave project itself
-  const isInSpecweaveProject = existsSync(join(cwd, 'plugins', 'specweave', 'hooks', 'hooks.json'));
-
-  // Skip marketplace hooks when developing specweave locally
-  if (isFromMarketplace && isInSpecweaveProject) {
-    // Output diagnostic only for session-start (once per session)
-    if (hookType === 'session-start') {
-      console.log(JSON.stringify({
-        continue: true,
-        systemMessage: 'SpecWeave dev environment: Using local hooks (marketplace hooks skipped)'
-      }));
-    } else {
-      console.log(JSON.stringify({ continue: true }));
-    }
-    return true;
-  }
-
-  return false;
-}
-
-/**
- * Wrap a promise with a timeout
- * @param promise - Promise to wrap
- * @param timeoutMs - Timeout in milliseconds
- * @param cleanup - Optional cleanup function (e.g., to kill child process)
- */
-function withTimeout(promise, timeoutMs, cleanup) {
-  let timeoutId;
-  const timeoutPromise = new Promise((_, reject) => {
-    timeoutId = setTimeout(() => {
-      if (cleanup) cleanup();
-      reject(new Error(`Hook timeout after ${timeoutMs}ms`));
-    }, timeoutMs);
-  });
-
-  return Promise.race([promise, timeoutPromise]).finally(() => {
-    clearTimeout(timeoutId);
-  });
-}
-
-/**
- * Find the dist/src/hooks directory (TypeScript compiled hooks)
- */
-function findHooksDir() {
-  // Try multiple locations
-  const candidates = [
-    // Development: project root dist/src/hooks
-    join(__dirname, '..', '..', '..', '..', 'dist', 'src', 'hooks'),
-    // Fallback: older path without src/ (backward compatibility)
-    join(__dirname, '..', '..', '..', '..', 'dist', 'hooks'),
-  ];
-
-  for (const candidate of candidates) {
-    const resolved = resolve(candidate);
-    if (existsSync(resolved)) {
-      return resolved;
-    }
-  }
-
-  return null;
-}
-
-/**
- * Run a hook script with timeout protection
- */
-async function runHook(scriptName) {
-  const hooksDir = findHooksDir();
-  if (!hooksDir) {
-    // No hooks directory - just continue
-    console.log(JSON.stringify({ continue: true }));
-    return;
-  }
-
-  const scriptPath = join(hooksDir, scriptName);
-  if (!existsSync(scriptPath) && !existsSync(scriptPath + '.js')) {
-    console.log(JSON.stringify({ continue: true }));
-    return;
-  }
-
-  // Spawn node to run the script
-  const child = spawn(process.execPath, [scriptPath], {
-    stdio: ['inherit', 'inherit', 'inherit'],
-    windowsHide: true,
-  });
-
-  const execPromise = new Promise((resolve) => {
-    child.on('exit', (code) => resolve(code || 0));
-    child.on('error', () => {
-      console.log(JSON.stringify({ continue: true }));
-      resolve(1);
-    });
-  });
-
-  try {
-    return await withTimeout(execPromise, HOOK_TIMEOUT_MS, () => {
-      // Kill the child process on timeout
-      try {
-        child.kill('SIGTERM');
-        setTimeout(() => child.kill('SIGKILL'), 1000);
-      } catch { /* ignore */ }
-    });
-  } catch (err) {
-    console.log(JSON.stringify({ continue: true, error: err.message }));
-    return 1;
-  }
-}
-
-/**
- * Cached Git Bash path (computed once per session)
- * undefined = not yet checked, null = checked and not found, string = path found
- */
-let gitBashCache = undefined;
-
-/**
- * Find Git Bash on Windows (with caching)
- *
- * Checks common Git for Windows installation paths.
- * Caches result to avoid repeated filesystem checks.
- */
-function findGitBash() {
-  // Return cached result if available
-  if (gitBashCache !== undefined) {
-    return gitBashCache;
-  }
-
-  const gitBashPaths = [
-    // Environment variable paths with proper fallbacks
-    join(process.env.PROGRAMFILES || 'C:\\Program Files', 'Git', 'bin', 'bash.exe'),
-    join(process.env['PROGRAMFILES(X86)'] || 'C:\\Program Files (x86)', 'Git', 'bin', 'bash.exe'),
-    join(process.env.LOCALAPPDATA || '', 'Programs', 'Git', 'bin', 'bash.exe'),
-    // Hardcoded fallbacks for edge cases where env vars are missing
-    'C:\\Program Files\\Git\\bin\\bash.exe',
-    'C:\\Program Files (x86)\\Git\\bin\\bash.exe',
-    'C:\\Git\\bin\\bash.exe',
-  ];
-
-  for (const p of gitBashPaths) {
-    if (p && existsSync(p)) {
-      gitBashCache = p;
-      return p;
-    }
-  }
-
-  gitBashCache = null;
-  return null;
-}
-
-/**
- * Helper to spawn a bash script with timeout protection
- */
-async function spawnBashWithTimeout(bashExe, args, options = {}) {
-  const child = spawn(bashExe, args, {
-    stdio: ['inherit', 'inherit', 'inherit'],
-    windowsHide: true,
-    ...options,
-  });
-
-  const execPromise = new Promise((resolve) => {
-    child.on('exit', (code) => resolve(code || 0));
-    child.on('error', () => {
-      console.log(JSON.stringify({ continue: true }));
-      resolve(1);
-    });
-  });
-
-  try {
-    return await withTimeout(execPromise, HOOK_TIMEOUT_MS, () => {
-      // Kill the child process on timeout
-      try {
-        child.kill('SIGTERM');
-        setTimeout(() => child.kill('SIGKILL'), 1000);
-      } catch { /* ignore */ }
-    });
-  } catch (err) {
-    console.log(JSON.stringify({ continue: true, error: err.message }));
-    return 1;
-  }
-}
-
-/**
- * Fallback to bash script if TypeScript not built (with timeout protection)
- *
- * @param bashScript - Name of the bash script (e.g., 'post-tool-use.sh')
- * @param subdir - Subdirectory under v2 (e.g., 'dispatchers', 'guards')
- */
-async function fallbackToBash(bashScript, subdir = 'dispatchers') {
-  const isWindows = process.platform === 'win32';
-  const bashDir = join(__dirname, '..', 'v2', subdir);
-  const scriptPath = join(bashDir, bashScript);
-
-  if (!existsSync(scriptPath)) {
-    console.log(JSON.stringify({ continue: true }));
-    return;
-  }
-
-  if (isWindows) {
-    // Strategy 1: Git Bash (preferred - most common)
-    const bashExe = findGitBash();
-    if (bashExe) {
-      return spawnBashWithTimeout(bashExe, [scriptPath]);
-    }
-
-    // Strategy 2: WSL (if Git Bash not available) - FALLBACK only
-    const wslPath = join(process.env.SYSTEMROOT || 'C:\\Windows', 'System32', 'wsl.exe');
-    if (existsSync(wslPath)) {
-      // Convert Windows path to WSL path:
-      // 1. Backslashes to forward slashes: C:\path\file -> C:/path/file
-      // 2. Drive letter to /mnt/: C:/path/file -> /mnt/c/path/file
-      // NOTE: Handle BOTH uppercase and lowercase drive letters (C: and c:)
-      const wslScriptPath = scriptPath
-        .replace(/\\/g, '/')
-        .replace(/^([A-Za-z]):/, (_, d) => `/mnt/${d.toLowerCase()}`);
-      return spawnBashWithTimeout('wsl', ['bash', wslScriptPath]);
-    }
-
-    // Strategy 3: No bash available - output warning and continue
-    // Hooks should not block Claude Code operation
-    console.log(JSON.stringify({
-      continue: true,
-      systemMessage: 'SpecWeave hooks require Git Bash on Windows. Install from https://git-scm.com'
-    }));
-    return;
-  }
-
-  // POSIX (macOS, Linux) - run directly with timeout
-  return spawnBashWithTimeout('bash', [scriptPath]);
-}
-
-// Main routing
-async function main() {
-  // CRITICAL: Skip if we're in dev environment with duplicate hooks
-  // This prevents marketplace hooks from running when local hooks are available
-  if (shouldSkipDueToDevEnvironment()) {
-    return;
-  }
-
-  const isWindows = process.platform === 'win32';
-
-  try {
-    switch (hookType) {
-      case 'session-start':
-        // CRITICAL: On Windows, ALWAYS use TypeScript implementation
-        // Bash nohup/disown is unreliable in Git Bash (process may terminate early)
-        // TypeScript uses proper Windows detached spawn with windowsHide
-        if (isWindows) {
-          await runHook('session-start.js');
-        } else {
-          // POSIX: try TypeScript first (preferred), fallback to bash for compatibility
-          try {
-            await runHook('session-start.js');
-          } catch {
-            await fallbackToBash('session-start.sh', 'dispatchers');
-          }
-        }
-        break;
-
-      case 'post-tool-use':
-        // Currently only bash implementation
-        // On Windows without Git Bash, will gracefully skip
-        await fallbackToBash('post-tool-use.sh', 'dispatchers');
-        break;
-
-      case 'completion-guard':
-        // Guards are in a different subdirectory
-        // On Windows without Git Bash, will gracefully skip
-        await fallbackToBash('completion-guard.sh', 'guards');
-        break;
-
-      case 'increment-duplicate-guard':
-        // Prevents duplicate increment IDs
-        await fallbackToBash('increment-duplicate-guard.sh', 'guards');
-        break;
-
-      default:
-        console.log(JSON.stringify({ continue: true, error: `Unknown hook type: ${hookType}` }));
-    }
-  } catch (err) {
-    console.log(JSON.stringify({ continue: true, error: String(err) }));
-  }
-}
-
-main();

package/plugins/specweave/hooks/universal/fail-fast-wrapper.sh

@@ -1,325 +0,0 @@
-#!/bin/bash
-# fail-fast-wrapper.sh - Non-blocking timeout wrapper for hooks
-#
-# v1.0.153: Hook-specific timeouts (user-prompt-submit: 30s for LLM detection)
-# v1.0.102+: Cross-platform support (Linux, macOS, BSD, Windows Git Bash)
-#
-# CRITICAL DESIGN PRINCIPLE:
-#   - NEVER block tool operations
-#   - All errors become warnings shown to user
-#   - Safe JSON output even on catastrophic failures
-#   - Universal timeout support (Perl fallback for all platforms)
-#
-# Usage: bash fail-fast-wrapper.sh <hook-script> [args...]
-#
-# Environment:
-#   HOOK_TIMEOUT - max seconds (default: 5)
-#   SPECWEAVE_HOOK_VERBOSE - show all warnings (default: 1)
-#
-# Returns safe JSON on timeout or error. Never hangs.
-
-set +e  # CRITICAL: Never exit on error
-
-HOOK_TIMEOUT="${HOOK_TIMEOUT:-5}"
-WRAPPER_VERSION="1.0.153"
-
-# ============================================================================
-# HOOK-SPECIFIC TIMEOUT OVERRIDES (v1.0.153)
-# ============================================================================
-# Some hooks require longer timeouts due to LLM calls or network operations.
-# user-prompt-submit.sh: LLM-based detect-intent (~15s) + plugin installs (~4s)
-# Override defaults here while keeping fast timeouts for simple hooks.
-#
-# Environment overrides (if you need custom values):
-#   HOOK_TIMEOUT_USER_PROMPT - timeout for user-prompt-submit (default: 30)
-#   HOOK_TIMEOUT_SESSION_START - timeout for session-start (default: 20)
-
-script_name=$(basename "${1:-}" 2>/dev/null)
-case "$script_name" in
-  user-prompt-submit.sh)
-    # LLM detection can timeout (30s) + keyword fallback + plugin installs (~10s)
-    # Total worst case: 30s LLM timeout + 10s installs + 5s buffer = 45s
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_USER_PROMPT:-45}"
-    ;;
-  session-start.sh)
-    # Project detection may involve file scanning and optional LLM
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_SESSION_START:-20}"
-    ;;
-  stop-auto.sh)
-    # May run tests + LLM evaluation + validation
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_STOP_AUTO:-120}"
-    ;;
-  post-tool-use.sh)
-    # Dispatcher runs task-ac-sync (awk/sed/jq) + spawns background detectors
-    # 5s is too tight for synchronous file processing + background spawning
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_POST_TOOL_USE:-15}"
-    ;;
-  pre-compact.sh)
-    # Pressure signal is ultra-fast (just writes a JSON file)
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_PRE_COMPACT:-5}"
-    ;;
-  stop-*.sh)
-    # Other stop hooks (reflect, sync, grill) are usually fast
-    HOOK_TIMEOUT="${HOOK_TIMEOUT_STOP:-15}"
-    ;;
-esac
-
-# ============================================================================
-# PROJECT ROOT DETECTION (for logging) - CRITICAL: must NOT fallback to pwd!
-# ============================================================================
-
-find_project_root() {
-  local dir="$PWD"
-  while [[ "$dir" != "/" ]]; do
-    if [[ -f "$dir/.specweave/config.json" ]]; then
-      echo "$dir"
-      return 0
-    fi
-    dir=$(dirname "$dir")
-  done
-  # Return empty - NOT pwd (prevents .specweave pollution)
-  return 1
-}
-
-PROJECT_ROOT=$(find_project_root)
-
-# Exit early if not a SpecWeave project (prevents .specweave pollution)
-if [[ -z "$PROJECT_ROOT" ]] || [[ ! -f "$PROJECT_ROOT/.specweave/config.json" ]]; then
-  # Not a SpecWeave project - output success JSON and exit
-  # UserPromptSubmit and Stop hooks need {"decision":"approve"}, others need {"continue":true}
-  if [[ "$script_name" == stop-* ]] || [[ "$script_name" == user-prompt-submit* ]]; then
-    echo '{"decision":"approve"}'
-  else
-    echo '{"continue": true}'
-  fi
-  exit 0
-fi
-
-LOGS_DIR="$PROJECT_ROOT/.specweave/logs"
-WARNING_LOG="$LOGS_DIR/hook-warnings.log"
-mkdir -p "$LOGS_DIR" 2>/dev/null || true
-
-# ============================================================================
-# JSON UTILITIES (v1.0.102+)
-# ============================================================================
-
-# Escape string for safe JSON embedding
-escape_json() {
-  local str="$1"
-  # Escape backslash, double quote, newline, carriage return, tab
-  str="${str//\\/\\\\}"  # Backslash
-  str="${str//\"/\\\"}"  # Double quote
-  str="${str//$'\n'/\\n}"  # Newline
-  str="${str//$'\r'/\\r}"  # Carriage return
-  str="${str//$'\t'/\\t}"  # Tab
-  echo "$str"
-}
-
-# ============================================================================
-# LOGGING
-# ============================================================================
-
-log_warning() {
-  local script="$1"
-  local message="$2"
-  local details="${3:-}"
-
-  local script_name
-  script_name=$(basename "$script" 2>/dev/null || echo "unknown")
-
-  # Log to file only (v1.0.102+: warnings now in JSON response)
-  {
-    echo "[$(date '+%Y-%m-%d %H:%M:%S')] WARNING [fail-fast-wrapper]: $message"
-    echo "  Script: $script_name"
-    [[ -n "$details" ]] && echo "  Details: $details"
-  } >> "$WARNING_LOG" 2>/dev/null || true
-
-  # NOTE: No console output - warnings are in JSON response
-}
-
-# ============================================================================
-# SAFE OUTPUT GENERATION (v1.0.102+)
-# ============================================================================
-
-# Safe output with warnings (JSON-escaped)
-get_safe_output_with_warnings() {
-  local script="$1"
-  local severity="$2"
-  local message="$3"
-  local recommendation="$4"
-
-  local script_name
-  script_name=$(basename "$script" 2>/dev/null || echo "unknown")
-
-  # Escape all strings for JSON safety
-  local escaped_name
-  local escaped_message
-  local escaped_recommendation
-  escaped_name=$(escape_json "$script_name")
-  escaped_message=$(escape_json "$message")
-  escaped_recommendation=$(escape_json "$recommendation")
-
-  # Build JSON with warnings array
-  # Detect correct response format based on hook lifecycle phase:
-  # - PreToolUse dispatchers/guards: {"decision":"allow"} or {"decision":"block"}
-  # - PostToolUse dispatchers: {"continue":true}
-  # - Stop/UserPromptSubmit: {"decision":"approve"}
-  local base_script
-  base_script=$(basename "$script" 2>/dev/null)
-  if [[ "$script" == *"guard"* ]] || [[ "$script" == *"validator"* ]] || [[ "$base_script" == pre-tool-use* ]]; then
-    cat <<EOF
-{"decision":"allow","warnings":[{"severity":"${severity}","message":"${escaped_name}: ${escaped_message}","recommendation":"${escaped_recommendation}"}]}
-EOF
-  elif [[ "$base_script" == stop-* ]] || [[ "$base_script" == user-prompt-submit* ]]; then
-    cat <<EOF
-{"decision":"approve","warnings":[{"severity":"${severity}","message":"${escaped_name}: ${escaped_message}","recommendation":"${escaped_recommendation}"}]}
-EOF
-  else
-    cat <<EOF
-{"continue":true,"warnings":[{"severity":"${severity}","message":"${escaped_name}: ${escaped_message}","recommendation":"${escaped_recommendation}"}]}
-EOF
-  fi
-}
-
-# Safe output without warnings (success case)
-get_safe_output() {
-  local script="$1"
-  local base_script
-  base_script=$(basename "$script" 2>/dev/null)
-  if [[ "$script" == *"guard"* ]] || [[ "$script" == *"validator"* ]] || [[ "$base_script" == pre-tool-use* ]]; then
-    echo '{"decision":"allow","warnings":[]}'
-  elif [[ "$base_script" == stop-* ]] || [[ "$base_script" == user-prompt-submit* ]]; then
-    echo '{"decision":"approve","warnings":[]}'
-  else
-    echo '{"continue":true,"warnings":[]}'
-  fi
-}
-
-# ============================================================================
-# MAIN EXECUTION
-# ============================================================================
-
-script="$1"
-shift
-
-# No script = safe default
-if [[ -z "$script" ]]; then
-  echo '{"continue":true}'
-  exit 0
-fi
-
-# Script doesn't exist = safe default with warning
-if [[ ! -f "$script" ]]; then
-  log_warning "$script" "Script not found (may be refreshing)" "Path: $script"
-  get_safe_output_with_warnings "$script" "WARNING" "Script not found (may be refreshing)" "Run 'specweave refresh-marketplace' to update plugins"
-  exit 0
-fi
-
-# Read stdin with non-blocking timeout (v1.0.102+)
-# Uses Perl for universal cross-platform support
-stdin_content=""
-if command -v perl >/dev/null 2>&1; then
-  # Perl-based timeout (works on all Unix systems)
-  stdin_content=$(perl -e '
-    use strict;
-    use warnings;
-
-    eval {
-      local $SIG{ALRM} = sub { die "timeout\n" };
-      alarm(1);
-      local $/;
-      my $input = <STDIN>;
-      alarm(0);
-      print $input;
-    };
-    if ($@) {
-      print "{}";
-    }
-  ' 2>/dev/null || echo '{}')
-elif command -v gtimeout >/dev/null 2>&1; then
-  stdin_content=$(gtimeout 1 cat 2>/dev/null || echo '{}')
-elif command -v timeout >/dev/null 2>&1; then
-  stdin_content=$(timeout 1 cat 2>/dev/null || echo '{}')
-else
-  # Last resort: try reading without blocking (may hang if stdin has data)
-  stdin_content=$(cat 2>/dev/null || echo '{}')
-fi
-
-# Run with timeout (v1.0.102+: Universal cross-platform support)
-tmp_out=$(mktemp -t hook-out.XXXXXX 2>/dev/null || echo "/tmp/hook-out-$$")
-tmp_err=$(mktemp -t hook-err.XXXXXX 2>/dev/null || echo "/tmp/hook-err-$$")
-
-# Try GNU timeout (Linux), then BSD timeout (macOS with coreutils), then Perl fallback
-if command -v gtimeout >/dev/null 2>&1; then
-  # GNU timeout (Linux, coreutils on macOS)
-  echo "$stdin_content" | gtimeout --kill-after=2 "$HOOK_TIMEOUT" bash "$script" "$@" > "$tmp_out" 2> "$tmp_err"
-  exit_code=$?
-elif command -v timeout >/dev/null 2>&1; then
-  # BSD timeout (some systems)
-  echo "$stdin_content" | timeout --kill-after=2 "$HOOK_TIMEOUT" bash "$script" "$@" > "$tmp_out" 2> "$tmp_err"
-  exit_code=$?
-else
-  # Bash-only fallback (for systems without timeout/gtimeout - e.g., macOS)
-  # Run hook in background
-  echo "$stdin_content" | bash "$script" "$@" > "$tmp_out" 2> "$tmp_err" &
-  script_pid=$!
-
-  # Wait with timeout using simple loop
-  elapsed=0
-  while kill -0 $script_pid 2>/dev/null && [ $elapsed -lt $HOOK_TIMEOUT ]; do
-    sleep 1
-    elapsed=$((elapsed + 1))
-  done
-
-  # Check if still running (timeout occurred)
-  if kill -0 $script_pid 2>/dev/null; then
-    # Timeout - kill the process
-    kill -TERM $script_pid 2>/dev/null
-    sleep 2
-    kill -KILL $script_pid 2>/dev/null
-    wait $script_pid 2>/dev/null
-    exit_code=124  # Timeout exit code
-  else
-    # Process finished - get exit code
-    wait $script_pid
-    exit_code=$?
-  fi
-fi
-
-output=$(cat "$tmp_out" 2>/dev/null)
-errors=$(cat "$tmp_err" 2>/dev/null)
-rm -f "$tmp_out" "$tmp_err" 2>/dev/null
-
-# Handle different exit scenarios
-case $exit_code in
-  0)
-    # Success - return output or safe default with empty warnings
-    if [[ -n "$output" ]]; then
-      echo "$output"
-    else
-      get_safe_output "$script"
-    fi
-    ;;
-
-  124|137)
-    # Timeout (124 = timeout, 137 = SIGKILL)
-    log_warning "$script" "Hook timed out after ${HOOK_TIMEOUT}s" "Consider optimizing or increasing HOOK_TIMEOUT"
-    get_safe_output_with_warnings "$script" "WARNING" "Hook timed out after ${HOOK_TIMEOUT}s" "Consider optimizing or increasing HOOK_TIMEOUT"
-    ;;
-
-  *)
-    # Other error
-    error_msg="Hook failed (exit $exit_code)"
-    recommendation="Run 'specweave check-hooks' to diagnose issues"
-
-    if [[ -n "$errors" ]]; then
-      log_warning "$script" "$error_msg" "$errors"
-      get_safe_output_with_warnings "$script" "ERROR" "$error_msg: $errors" "$recommendation"
-    else
-      log_warning "$script" "$error_msg"
-      get_safe_output_with_warnings "$script" "ERROR" "$error_msg" "$recommendation"
-    fi
-    ;;
-esac
-
-exit 0

package/plugins/specweave/hooks/universal/hook-wrapper.cmd

@@ -1,26 +0,0 @@
-@echo off
-REM hook-wrapper.cmd - Windows resilient hook launcher
-REM Prevents crashes when dispatcher.mjs is temporarily unavailable
-
-setlocal enabledelayedexpansion
-
-set "HOOK_TYPE=%~1"
-if "%HOOK_TYPE%"=="" set "HOOK_TYPE=unknown"
-
-set "SCRIPT_DIR=%~dp0"
-set "DISPATCHER=%SCRIPT_DIR%dispatcher.mjs"
-
-REM Check if dispatcher exists
-if not exist "%DISPATCHER%" (
-    echo {"continue":true,"systemMessage":"Hook skipped: dispatcher.mjs not found"}
-    exit /b 0
-)
-
-REM Run dispatcher with error suppression
-node "%DISPATCHER%" "%HOOK_TYPE%" 2>nul
-if errorlevel 1 (
-    echo {"continue":true,"systemMessage":"Hook error, continuing"}
-    exit /b 0
-)
-
-exit /b 0