specweave 1.0.550 → 1.0.552

package/plugins/specweave/hooks/v2/guards/tdd-enforcement-guard.sh

@@ -1,268 +0,0 @@
-#!/bin/bash
-# tdd-enforcement-guard.sh - Warn on TDD discipline violations (v1.0.105)
-#
-# Called from: post-tool-use.sh when tasks.md is edited
-# Action: WARNING only (not blocking) - educates users
-#
-# TDD Violations detected:
-# 1. GREEN task completed before RED task
-# 2. REFACTOR task completed before GREEN task
-#
-# IMPORTANT: This is a WARNING-ONLY hook. It never blocks operations.
-# The goal is to educate users about TDD discipline, not punish them.
-#
-# COMPATIBILITY: Works on macOS (bash 3.2) and Linux (bash 4+)
-
-set +e  # CRITICAL: Never exit on error - this is a warning-only hook
-
-# ============================================================================
-# CONFIGURATION
-# ============================================================================
-
-PROJECT_ROOT="${1:-$(pwd)}"
-FILE_PATH="${2:-}"
-TOOL_NAME="${3:-Edit}"
-
-# Debug logging
-LOGS_DIR="$PROJECT_ROOT/.specweave/logs"
-DEBUG_LOG="$LOGS_DIR/tdd-enforcement.log"
-mkdir -p "$LOGS_DIR" 2>/dev/null || true
-
-log_debug() {
-  echo "[$(date '+%Y-%m-%d %H:%M:%S')] $*" >> "$DEBUG_LOG" 2>/dev/null || true
-}
-
-log_debug "tdd-enforcement-guard invoked: FILE_PATH=$FILE_PATH TOOL_NAME=$TOOL_NAME"
-
-# ============================================================================
-# VALIDATION
-# ============================================================================
-
-# Only process tasks.md files
-case "$FILE_PATH" in
-  *tasks.md) ;;
-  *)
-    log_debug "Not a tasks.md file, skipping"
-    exit 0
-    ;;
-esac
-
-# Extract increment directory
-INC_DIR=$(dirname "$FILE_PATH")
-META_FILE="$INC_DIR/metadata.json"
-TASKS_FILE="$FILE_PATH"
-
-# Check if files exist
-if [ ! -f "$META_FILE" ]; then
-  log_debug "No metadata.json found at $META_FILE, skipping"
-  exit 0
-fi
-
-if [ ! -f "$TASKS_FILE" ]; then
-  log_debug "No tasks.md found at $TASKS_FILE, skipping"
-  exit 0
-fi
-
-# ============================================================================
-# READ TEST MODE FROM METADATA
-# ============================================================================
-
-TEST_MODE=""
-
-if command -v jq >/dev/null 2>&1; then
-  TEST_MODE=$(jq -r '.testMode // "test-after"' "$META_FILE" 2>/dev/null || echo "test-after")
-else
-  # Fallback: portable grep parsing (no -P flag for macOS compatibility)
-  TEST_MODE=$(grep '"testMode"' "$META_FILE" 2>/dev/null | sed 's/.*"testMode"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/' || echo "test-after")
-fi
-
-# Default to test-after if empty
-TEST_MODE="${TEST_MODE:-test-after}"
-
-log_debug "testMode detected: $TEST_MODE"
-
-# Skip if not TDD mode
-if [ "$TEST_MODE" != "TDD" ]; then
-  log_debug "testMode is not TDD ($TEST_MODE), skipping enforcement"
-  exit 0
-fi
-
-# ============================================================================
-# READ ENFORCEMENT LEVEL FROM CONFIG (v1.0.111+)
-# ============================================================================
-
-TDD_ENFORCEMENT="warn"  # Default: warn but allow
-
-CONFIG_FILE="$PROJECT_ROOT/.specweave/config.json"
-if [ -f "$CONFIG_FILE" ]; then
-  if command -v jq >/dev/null 2>&1; then
-    TDD_ENFORCEMENT=$(jq -r '.testing.tddEnforcement // "warn"' "$CONFIG_FILE" 2>/dev/null || echo "warn")
-  else
-    # Fallback: grep parsing
-    TDD_ENFORCEMENT=$(grep '"tddEnforcement"' "$CONFIG_FILE" 2>/dev/null | sed 's/.*"tddEnforcement"[[:space:]]*:[[:space:]]*"\([^"]*\)".*/\1/' || echo "warn")
-  fi
-fi
-
-# Validate enforcement level
-case "$TDD_ENFORCEMENT" in
-  strict|warn|off) ;;
-  *) TDD_ENFORCEMENT="warn" ;;
-esac
-
-log_debug "tddEnforcement level: $TDD_ENFORCEMENT"
-
-# Skip if enforcement is off
-if [ "$TDD_ENFORCEMENT" = "off" ]; then
-  log_debug "TDD enforcement is off, skipping"
-  exit 0
-fi
-
-# ============================================================================
-# PARSE TASKS FOR TDD VIOLATIONS (macOS-compatible, no associative arrays)
-# ============================================================================
-
-VIOLATION_LOG="$LOGS_DIR/tdd-violations.log"
-
-# Use temp files for cross-platform compatibility (no associative arrays)
-COMPLETED_TASKS_FILE=$(mktemp)
-cleanup() {
-  rm -f "$COMPLETED_TASKS_FILE" 2>/dev/null || true
-}
-trap cleanup EXIT
-
-# Parse task sections and find completed TDD tasks
-# Format: TASK_ID:PHASE
-current_task=""
-current_phase=""
-
-while IFS= read -r line; do
-  # Task header: ### T-001: [RED] ... or ### T-001: [GREEN] ... or ### T-001: [REFACTOR] ...
-  case "$line" in
-    "### T-"*": ["*"]"*)
-      # Extract task ID (T-XXX)
-      current_task=$(echo "$line" | sed -n 's/^### \(T-[0-9]*\):.*/\1/p')
-      # Extract phase
-      case "$line" in
-        *"[RED]"*) current_phase="RED" ;;
-        *"[GREEN]"*) current_phase="GREEN" ;;
-        *"[REFACTOR]"*) current_phase="REFACTOR" ;;
-        *) current_phase="" ;;
-      esac
-      log_debug "Found task $current_task with phase $current_phase"
-      ;;
-  esac
-
-  # Status line: **Status**: [x] completed
-  case "$line" in
-    "**Status**: [x]"*)
-      if [ -n "$current_task" ] && [ -n "$current_phase" ]; then
-        echo "$current_task:$current_phase" >> "$COMPLETED_TASKS_FILE"
-        log_debug "Task $current_task ($current_phase) is completed"
-      fi
-      current_task=""
-      current_phase=""
-      ;;
-  esac
-done < "$TASKS_FILE"
-
-# ============================================================================
-# CHECK FOR VIOLATIONS
-# ============================================================================
-
-VIOLATIONS=""
-VIOLATION_COUNT=0
-
-# Helper to check if a task is completed with a specific phase
-is_completed_with_phase() {
-  local task_id="$1"
-  local expected_phase="$2"
-  grep -q "^${task_id}:${expected_phase}$" "$COMPLETED_TASKS_FILE" 2>/dev/null
-}
-
-# Check each completed task
-while IFS=: read -r task_id phase; do
-  [ -z "$task_id" ] && continue
-
-  if [ "$phase" = "GREEN" ]; then
-    # GREEN task completed - check if corresponding RED is done
-    # Extract number, subtract 1, format back
-    task_num=$(echo "$task_id" | sed 's/T-0*//')
-    red_task_num=$((task_num - 1))
-    red_task_id=$(printf "T-%03d" "$red_task_num")
-
-    if ! is_completed_with_phase "$red_task_id" "RED"; then
-      VIOLATIONS="$VIOLATIONS
-   • $task_id (GREEN) completed but $red_task_id (RED) not found or not completed"
-      VIOLATION_COUNT=$((VIOLATION_COUNT + 1))
-      log_debug "VIOLATION: $task_id GREEN completed before $red_task_id RED"
-    fi
-  fi
-
-  if [ "$phase" = "REFACTOR" ]; then
-    # REFACTOR task completed - check if corresponding GREEN is done
-    task_num=$(echo "$task_id" | sed 's/T-0*//')
-    green_task_num=$((task_num - 1))
-    green_task_id=$(printf "T-%03d" "$green_task_num")
-
-    if ! is_completed_with_phase "$green_task_id" "GREEN"; then
-      VIOLATIONS="$VIOLATIONS
-   • $task_id (REFACTOR) completed but $green_task_id (GREEN) not found or not completed"
-      VIOLATION_COUNT=$((VIOLATION_COUNT + 1))
-      log_debug "VIOLATION: $task_id REFACTOR completed before $green_task_id GREEN"
-    fi
-  fi
-done < "$COMPLETED_TASKS_FILE"
-
-# ============================================================================
-# OUTPUT WARNINGS
-# ============================================================================
-
-if [ "$VIOLATION_COUNT" -gt 0 ]; then
-  # Log violations
-  {
-    echo "[$(date '+%Y-%m-%d %H:%M:%S')] TDD violations in $(basename "$INC_DIR") (enforcement: $TDD_ENFORCEMENT):"
-    echo "$VIOLATIONS" | while IFS= read -r v; do
-      [ -n "$v" ] && echo "  -$v"
-    done
-    echo ""
-  } >> "$VIOLATION_LOG" 2>/dev/null || true
-
-  # Handle based on enforcement level
-  if [ "$TDD_ENFORCEMENT" = "strict" ]; then
-    echo ""
-    echo "❌ TDD ENFORCEMENT BLOCKED"
-    echo "   ────────────────────────"
-    echo "   Your increment is configured for TDD mode with STRICT enforcement."
-    echo ""
-    echo "   Violations detected:$VIOLATIONS"
-    echo ""
-    echo "   💡 TDD Discipline: RED → GREEN → REFACTOR"
-    echo "      1. 🔴 Complete RED (test) task FIRST"
-    echo "      2. 🟢 Then complete GREEN (implementation) task"
-    echo "      3. 🔵 Finally complete REFACTOR task"
-    echo ""
-    echo "   To bypass: Set testing.tddEnforcement: \"warn\" in .specweave/config.json"
-    echo ""
-    log_debug "BLOCKING due to TDD violations (strict mode)"
-    exit 1  # BLOCK in strict mode
-  else
-    # warn mode
-    echo ""
-    echo "⚠️  TDD DISCIPLINE WARNING"
-    echo "   ────────────────────────"
-    echo "   Your increment is configured for TDD mode (testMode: TDD)"
-    echo ""
-    echo "   Potential violations detected:$VIOLATIONS"
-    echo ""
-    echo "   💡 TDD Best Practice: RED → GREEN → REFACTOR"
-    echo "      1. 🔴 Write failing test FIRST"
-    echo "      2. 🟢 Make test pass with minimal code"
-    echo "      3. 🔵 Refactor while keeping tests green"
-    echo ""
-    echo "   💡 To enforce strictly: Set testing.tddEnforcement: \"strict\" in config.json"
-    echo ""
-  fi
-fi
-
-# Exit 0 for warn mode (allow), exit 1 for strict mode (handled above)
-exit 0

package/plugins/specweave/hooks/v2/handlers/ac-sync-dispatcher.sh

@@ -1,332 +0,0 @@
-#!/bin/bash
-
-# SpecWeave Provider-Agnostic AC Sync Dispatcher (v1.0.255+)
-# Background handler: syncs AC progress to ALL enabled providers (GitHub, JIRA, ADO).
-#
-# Replaces the GitHub-only github-ac-sync-handler.sh with a unified dispatcher
-# that calls syncACProgressToProviders() for all configured providers.
-#
-# Triggered by post-tool-use.sh after task-ac-sync-guard updates spec.md ACs.
-# Runs in BACKGROUND via safe_run_background — never blocks the main chain.
-#
-# Features:
-# - 5s debounce (batches rapid AC changes)
-# - Per-provider circuit breakers (managed in TypeScript)
-# - File locking (prevents concurrent handler executions)
-# - Non-blocking (all errors → exit 0 with logging)
-#
-# Usage: ac-sync-dispatcher.sh <INCREMENT_ID>
-# Called by: safe_run_background in post-tool-use.sh
-
-set +e  # Never crash Claude Code
-
-# EMERGENCY KILL SWITCH
-if [[ "${SPECWEAVE_DISABLE_HOOKS:-0}" == "1" ]]; then
-  exit 0
-fi
-
-# ============================================================================
-# PROJECT ROOT DETECTION
-# ============================================================================
-
-find_project_root() {
-  local dir="$1"
-  while [ "$dir" != "/" ]; do
-    if [ -f "$dir/.specweave/config.json" ]; then
-      echo "$dir"
-      return 0
-    fi
-    dir="$(dirname "$dir")"
-  done
-  return 1
-}
-
-PROJECT_ROOT="$(find_project_root "$(pwd)")"
-if [[ -z "$PROJECT_ROOT" ]]; then
-  exit 0
-fi
-cd "$PROJECT_ROOT" 2>/dev/null || exit 0
-
-# ============================================================================
-# ARGUMENTS
-# ============================================================================
-
-INC_ID="$1"
-if [[ -z "$INC_ID" ]]; then
-  exit 0
-fi
-
-# Validate INC_ID format (defense-in-depth against injection)
-if [[ ! "$INC_ID" =~ ^[0-9]{4}[A-Za-z0-9_-]*$ ]]; then
-  exit 0
-fi
-
-SPEC_PATH="$PROJECT_ROOT/.specweave/increments/$INC_ID/spec.md"
-METADATA_PATH="$PROJECT_ROOT/.specweave/increments/$INC_ID/metadata.json"
-CONFIG_PATH="$PROJECT_ROOT/.specweave/config.json"
-
-if [[ ! -f "$SPEC_PATH" ]]; then
-  exit 0
-fi
-
-# ============================================================================
-# LOGGING
-# ============================================================================
-
-LOGS_DIR="$PROJECT_ROOT/.specweave/logs"
-LOG_FILE="$LOGS_DIR/ac-sync-dispatcher.log"
-mkdir -p "$LOGS_DIR" 2>/dev/null || true
-
-log() {
-  echo "[$(date '+%Y-%m-%d %H:%M:%S')] [ac-sync] $1" >> "$LOG_FILE" 2>/dev/null || true
-}
-
-# ============================================================================
-# CHECK ANY PROVIDER ENABLED
-# ============================================================================
-
-if [[ ! -f "$CONFIG_PATH" ]] || ! command -v jq >/dev/null 2>&1; then
-  log "Config not found or jq unavailable. Skipping."
-  exit 0
-fi
-
-# Use shared provider detection (supports PROFILES, LEGACY DIRECT, LEGACY PROVIDER formats)
-HANDLER_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-SHARED_LIB="$HANDLER_DIR/../lib/check-provider-enabled.sh"
-if [[ -f "$SHARED_LIB" ]]; then
-  source "$SHARED_LIB"
-fi
-
-# Source resolve-package.sh for dynamic specweave path resolution
-RESOLVE_LIB="$HANDLER_DIR/../../lib/resolve-package.sh"
-if [[ -f "$RESOLVE_LIB" ]]; then
-  source "$RESOLVE_LIB"
-fi
-
-GH_ENABLED="false"
-JIRA_ENABLED="false"
-ADO_ENABLED="false"
-
-if type check_provider_enabled &>/dev/null; then
-  check_provider_enabled "$CONFIG_PATH" "github" && GH_ENABLED="true"
-  check_provider_enabled "$CONFIG_PATH" "jira" && JIRA_ENABLED="true"
-  check_provider_enabled "$CONFIG_PATH" "ado" && ADO_ENABLED="true"
-else
-  # Fallback to legacy jq check if shared lib not available
-  GH_ENABLED=$(jq -r '.sync.github.enabled // false' "$CONFIG_PATH" 2>/dev/null)
-  JIRA_ENABLED=$(jq -r '.sync.jira.enabled // false' "$CONFIG_PATH" 2>/dev/null)
-  ADO_ENABLED=$(jq -r '.sync.ado.enabled // false' "$CONFIG_PATH" 2>/dev/null)
-fi
-
-if [[ "$GH_ENABLED" != "true" && "$JIRA_ENABLED" != "true" && "$ADO_ENABLED" != "true" ]]; then
-  log "No providers enabled. Skipping."
-  exit 0
-fi
-
-# ============================================================================
-# FILE LOCKING
-# ============================================================================
-
-STATE_DIR="$PROJECT_ROOT/.specweave/state"
-mkdir -p "$STATE_DIR" 2>/dev/null || true
-
-LOCK_FILE="$STATE_DIR/.hook-ac-sync-dispatcher.lock"
-LOCK_TIMEOUT=15
-
-LOCK_ACQUIRED=false
-for i in {1..10}; do
-  if mkdir "$LOCK_FILE" 2>/dev/null; then
-    LOCK_ACQUIRED=true
-    trap 'rmdir "$LOCK_FILE" 2>/dev/null || true' EXIT
-    break
-  fi
-
-  # Check for stale lock
-  if [[ -d "$LOCK_FILE" ]]; then
-    LOCK_AGE=$(($(date +%s) - $(stat -f%m "$LOCK_FILE" 2>/dev/null || stat -c%Y "$LOCK_FILE" 2>/dev/null || echo 0)))
-    if (( LOCK_AGE > LOCK_TIMEOUT )); then
-      rmdir "$LOCK_FILE" 2>/dev/null || true
-      continue
-    fi
-  fi
-
-  sleep 0.3
-done
-
-if [[ "$LOCK_ACQUIRED" == "false" ]]; then
-  log "Lock not acquired. Another instance running. Skipping."
-  exit 0
-fi
-
-# ============================================================================
-# DEBOUNCE (5s window)
-# ============================================================================
-
-SIGNAL_FILE="$STATE_DIR/.ac-sync-pending-$INC_ID"
-
-if [[ -f "$SIGNAL_FILE" ]]; then
-  SIGNAL_AGE=$(($(date +%s) - $(stat -f%m "$SIGNAL_FILE" 2>/dev/null || stat -c%Y "$SIGNAL_FILE" 2>/dev/null || echo 0)))
-  if (( SIGNAL_AGE < 5 )); then
-    log "Debounce: signal file age ${SIGNAL_AGE}s < 5s. Deferring."
-    exit 0
-  fi
-fi
-
-# Create/update signal file and wait for debounce window
-echo "$(date +%s)" > "$SIGNAL_FILE" 2>/dev/null || true
-sleep 5
-
-# After sleeping, check if a NEWER invocation took over
-if [[ -f "$SIGNAL_FILE" ]]; then
-  CURRENT_SIGNAL=$(cat "$SIGNAL_FILE" 2>/dev/null || echo 0)
-  NOW=$(date +%s)
-  SIGNAL_AGE=$((NOW - CURRENT_SIGNAL))
-  if (( SIGNAL_AGE > 6 )); then
-    log "Debounce: newer invocation detected. Exiting."
-    exit 0
-  fi
-fi
-
-rm -f "$SIGNAL_FILE" 2>/dev/null || true
-
-# ============================================================================
-# PRECONDITIONS
-# ============================================================================
-
-if ! command -v node &>/dev/null; then
-  log "Node.js not found. Skipping."
-  exit 0
-fi
-
-# ============================================================================
-# EXTRACT AFFECTED USER STORY IDS
-# ============================================================================
-
-# Get all US IDs that have external links from metadata.json
-# Reads BOTH new format (externalLinks.github.issues) and old format (github.issues[].userStory)
-AFFECTED_US_IDS="[]"
-if [[ -f "$METADATA_PATH" ]]; then
-  AFFECTED_US_IDS=$(jq -r '
-    [
-      (.externalLinks.github.issues // {} | keys[]),
-      (.externalLinks.jira.userStories // {} | keys[]),
-      (.externalLinks.ado.userStories // {} | keys[]),
-      (.github.issues // [] | .[].userStory // empty)
-    ] | flatten | unique
-  ' "$METADATA_PATH" 2>/dev/null) || AFFECTED_US_IDS="[]"
-fi
-
-if [[ "$AFFECTED_US_IDS" == "[]" || -z "$AFFECTED_US_IDS" ]]; then
-  log "No affected US IDs found. Skipping."
-  exit 0
-fi
-
-# Validate AFFECTED_US_IDS is valid JSON (defense-in-depth against injection)
-if ! echo "$AFFECTED_US_IDS" | jq empty 2>/dev/null; then
-  log "Invalid US IDs JSON. Skipping."
-  exit 0
-fi
-
-# ============================================================================
-# BUILD CONFIG FROM SPECWEAVE CONFIG + METADATA
-# ============================================================================
-
-# Find the sync module (compiled output) — check multiple locations
-# Same fallback pattern as universal-auto-create-dispatcher.sh
-SYNC_MODULE=""
-# Derive package root from script location (handlers/ → v2/ → hooks/ → specweave/ → plugins/ → package root)
-PKG_ROOT="$(cd "$HANDLER_DIR/../../../../.." 2>/dev/null && pwd)"
-# 0. SPECWEAVE_PKG (set by Claude Code hook infrastructure, most reliable)
-if [[ -n "${SPECWEAVE_PKG:-}" ]]; then
-  CANDIDATE="${SPECWEAVE_PKG}/dist/src/core/ac-progress-sync.js"
-  [[ -f "$CANDIDATE" ]] && SYNC_MODULE="$CANDIDATE"
-fi
-# 1. Package root (derived from script location — works for both npm and dev)
-if [[ -z "$SYNC_MODULE" ]]; then
-  CANDIDATE="${PKG_ROOT:-$PROJECT_ROOT}/dist/src/core/ac-progress-sync.js"
-  [[ -f "$CANDIDATE" ]] && SYNC_MODULE="$CANDIDATE"
-fi
-# 2. resolve-package.sh (dynamic resolution)
-if [[ -z "$SYNC_MODULE" ]]; then
-  CANDIDATE=$(find_specweave_script "dist/src/core/ac-progress-sync.js" 2>/dev/null)
-  [[ -n "$CANDIDATE" ]] && SYNC_MODULE="$CANDIDATE"
-fi
-# 3. Legacy: PROJECT_ROOT/dist/ (dev-only, running from source repo)
-if [[ -z "$SYNC_MODULE" ]]; then
-  CANDIDATE="$PROJECT_ROOT/dist/src/core/ac-progress-sync.js"
-  [[ -f "$CANDIDATE" ]] && SYNC_MODULE="$CANDIDATE"
-fi
-
-if [[ -z "$SYNC_MODULE" ]]; then
-  log "Sync module not found at any known path. Checked: SPECWEAVE_PKG=${SPECWEAVE_PKG:-unset}, PKG_ROOT=${PKG_ROOT:-unset}, node_modules, PROJECT_ROOT/dist. Skipping."
-  exit 0
-fi
-log "Using sync module: $SYNC_MODULE"
-
-log "Syncing AC progress for increment $INC_ID to enabled providers..."
-log "Providers: github=$GH_ENABLED jira=$JIRA_ENABLED ado=$ADO_ENABLED"
-log "Affected US IDs: $AFFECTED_US_IDS"
-
-# ============================================================================
-# RUN PROVIDER-AGNOSTIC SYNC
-# ============================================================================
-
-# Build inline Node.js script that constructs config and calls sync
-# Pass all values via environment variables to prevent command injection
-OUTPUT=$(SW_SYNC_MODULE="$SYNC_MODULE" \
-  SW_CONFIG_PATH="$CONFIG_PATH" \
-  SW_METADATA_PATH="$METADATA_PATH" \
-  SW_SPEC_PATH="$SPEC_PATH" \
-  SW_INC_ID="$INC_ID" \
-  SW_AFFECTED_US_IDS="$AFFECTED_US_IDS" \
-  SW_CLOSE_ALL="${SPECWEAVE_CLOSE_ALL:-0}" \
-  node --input-type=module -e "
-import { readFile } from 'fs/promises';
-
-const { SW_SYNC_MODULE, SW_CONFIG_PATH, SW_METADATA_PATH,
-        SW_SPEC_PATH, SW_INC_ID, SW_AFFECTED_US_IDS, SW_CLOSE_ALL } = process.env;
-
-const mod = await import(SW_SYNC_MODULE);
-const affectedUSIds = JSON.parse(SW_AFFECTED_US_IDS);
-
-try {
-  const rawConfig = JSON.parse(await readFile(SW_CONFIG_PATH, 'utf-8'));
-  const metadata = JSON.parse(await readFile(SW_METADATA_PATH, 'utf-8'));
-
-  const config = {
-    sync: {
-      github: { enabled: rawConfig?.sync?.github?.enabled === true },
-      jira: { enabled: rawConfig?.sync?.jira?.enabled === true },
-      ado: { enabled: rawConfig?.sync?.ado?.enabled === true },
-    },
-    github: rawConfig?.sync?.github || {},
-    jira: rawConfig?.sync?.jira || {},
-    ado: rawConfig?.sync?.ado || {},
-    externalLinks: metadata?.externalLinks || {},
-  };
-
-  let result;
-  if (SW_CLOSE_ALL === '1' && mod.closeIncrementIssues) {
-    // Force-close all external issues for completed increment
-    result = await mod.closeIncrementIssues(SW_INC_ID, SW_SPEC_PATH, config);
-  } else {
-    result = await mod.syncACProgressToProviders(
-      SW_INC_ID, affectedUSIds, SW_SPEC_PATH, config,
-    );
-  }
-
-  console.log(JSON.stringify(result, null, 2));
-} catch (err) {
-  console.error('AC sync dispatcher error:', err.message || err);
-  process.exit(1);
-}
-" 2>&1)
-EXIT_CODE=$?
-
-if [[ $EXIT_CODE -ne 0 ]]; then
-  log "Sync failed (exit $EXIT_CODE): $OUTPUT"
-  exit 0
-fi
-
-log "Sync completed: $OUTPUT"
-exit 0

package/plugins/specweave/hooks/v2/handlers/ac-validation-handler.sh

@@ -1,50 +0,0 @@
-#!/bin/bash
-# ac-validation-handler.sh - Validate AC completion status
-# Checks that completed tasks have their ACs checked in spec.md
-# Non-blocking, logs warnings only
-#
-# IMPORTANT: Never crash Claude, always exit 0
-set +e
-
-[[ "${SPECWEAVE_DISABLE_HOOKS:-0}" == "1" ]] && exit 0
-
-INC_ID="${1:-}"
-[[ -z "$INC_ID" ]] && exit 0
-
-# Find project root
-PROJECT_ROOT="$PWD"
-while [[ "$PROJECT_ROOT" != "/" ]] && [[ ! -f "$PROJECT_ROOT/.specweave/config.json" ]]; do
-  PROJECT_ROOT=$(dirname "$PROJECT_ROOT")
-done
-[[ ! -f "$PROJECT_ROOT/.specweave/config.json" ]] && exit 0
-
-INC_DIR="$PROJECT_ROOT/.specweave/increments/$INC_ID"
-TASKS_FILE="$INC_DIR/tasks.md"
-SPEC_FILE="$INC_DIR/spec.md"
-LOG_FILE="$PROJECT_ROOT/.specweave/logs/ac-validation.log"
-
-[[ ! -f "$TASKS_FILE" ]] || [[ ! -f "$SPEC_FILE" ]] && exit 0
-
-mkdir -p "$(dirname "$LOG_FILE")" 2>/dev/null
-
-# Find completed tasks and their ACs
-WARNINGS=""
-while IFS= read -r line; do
-  if [[ "$line" =~ ^\*\*Satisfies\ ACs\*\*:\ (.+)$ ]]; then
-    ACS="${BASH_REMATCH[1]}"
-    # Check each AC
-    for ac in $(echo "$ACS" | tr ',' ' '); do
-      ac=$(echo "$ac" | tr -d ' ')
-      [[ -z "$ac" ]] && continue
-      # Check if AC is checked in spec.md
-      if ! grep -qE "^\s*-\s*\[x\]\s*\*\*${ac}\*\*" "$SPEC_FILE" 2>/dev/null; then
-        WARNINGS="$WARNINGS\n  - $ac not checked in spec.md"
-      fi
-    done
-  fi
-done < <(grep -A5 "\[x\] completed" "$TASKS_FILE" 2>/dev/null | grep "Satisfies ACs")
-
-if [[ -n "$WARNINGS" ]]; then
-  echo "[$(date)] $INC_ID: AC validation warnings:$WARNINGS" >> "$LOG_FILE"
-fi
-exit 0