specweave 1.0.30 → 1.0.32

package/plugins/specweave/hooks/shared/bulk-operation-detector.sh

@@ -1,167 +0,0 @@
-#!/bin/bash
-# Bulk Operation Detector (v0.26.0)
-#
-# Detects bulk edit operations (5+ edits in 10 seconds) and triggers batch mode.
-# Part of ADR-0130: Hook Bulk Operation Detection and Batching
-#
-# Usage:
-#   source bulk-operation-detector.sh
-#   BULK_STATUS=$(detect_bulk_operation "$INCREMENT_ID")
-#
-#   if [ "$BULK_STATUS" = "BULK_MODE_DETECTED" ]; then
-#     # Skip individual hook, batch job will run later
-#     exit 0
-#   fi
-
-# Configuration
-BULK_THRESHOLD=${SPECWEAVE_BULK_THRESHOLD:-5}      # 5+ operations = bulk mode
-BULK_WINDOW=${SPECWEAVE_BULK_WINDOW:-10}            # seconds
-IDLE_DELAY=${SPECWEAVE_IDLE_DELAY:-5}               # seconds before batch execution
-
-# State files
-STATE_DIR="$PROJECT_ROOT/.specweave/state"
-OPERATION_COUNTER="$STATE_DIR/.hook-operation-counter"
-OPERATION_TIMESTAMP="$STATE_DIR/.hook-operation-timestamp"
-
-# Ensure state directory exists
-mkdir -p "$STATE_DIR"
-
-##
-# Detect bulk operation
-#
-# Uses sliding window counter to detect operation bursts:
-# - Counts operations within BULK_WINDOW (10 seconds)
-# - If count >= BULK_THRESHOLD (5 operations) → bulk mode
-# - Resets counter after idle period
-#
-# Args:
-#   $1 - INCREMENT_ID (optional, for logging)
-#
-# Returns:
-#   BULK_MODE_DETECTED - Bulk operation detected, skip individual hook
-#   SINGLE_OPERATION   - Single operation, run hook normally
-##
-detect_bulk_operation() {
-  local INCREMENT_ID="${1:-unknown}"
-  local CURRENT_TIME=$(date +%s)
-
-  # Read existing counter and timestamp
-  local RECENT_OPS=0
-  local LAST_OP_TIME=0
-
-  if [ -f "$OPERATION_COUNTER" ]; then
-    RECENT_OPS=$(cat "$OPERATION_COUNTER" 2>/dev/null || echo 0)
-  fi
-
-  if [ -f "$OPERATION_TIMESTAMP" ]; then
-    LAST_OP_TIME=$(cat "$OPERATION_TIMESTAMP" 2>/dev/null || echo 0)
-  fi
-
-  # Reset counter if outside window (idle period detected)
-  local TIME_SINCE_LAST_OP=$((CURRENT_TIME - LAST_OP_TIME))
-  if (( TIME_SINCE_LAST_OP > BULK_WINDOW )); then
-    RECENT_OPS=0
-  fi
-
-  # Increment operation counter
-  RECENT_OPS=$((RECENT_OPS + 1))
-
-  # Write atomically (use temp file + rename for atomicity)
-  echo "$RECENT_OPS" > "${OPERATION_COUNTER}.tmp"
-  mv "${OPERATION_COUNTER}.tmp" "$OPERATION_COUNTER"
-
-  echo "$CURRENT_TIME" > "${OPERATION_TIMESTAMP}.tmp"
-  mv "${OPERATION_TIMESTAMP}.tmp" "$OPERATION_TIMESTAMP"
-
-  # Bulk operation detected?
-  if (( RECENT_OPS >= BULK_THRESHOLD )); then
-    # Log bulk mode activation
-    local DEBUG_LOG="$PROJECT_ROOT/.specweave/logs/hook-debug.log"
-    echo "[$(date)] [BULK] Detected bulk operation for $INCREMENT_ID (op #$RECENT_OPS)" >> "$DEBUG_LOG" 2>/dev/null || true
-
-    # Schedule batch job (if not already scheduled)
-    schedule_batch_job "$INCREMENT_ID"
-
-    echo "BULK_MODE_DETECTED"
-    return 0
-  fi
-
-  # Single operation
-  echo "SINGLE_OPERATION"
-  return 0
-}
-
-##
-# Schedule batch job
-#
-# Schedules a batch job to run after idle period (IDLE_DELAY seconds).
-# If a batch job is already scheduled, it gets canceled and rescheduled.
-#
-# Args:
-#   $1 - INCREMENT_ID
-##
-schedule_batch_job() {
-  local INCREMENT_ID="$1"
-  local BATCH_LOCK="$STATE_DIR/.batch-job-${INCREMENT_ID}.lock"
-  local DEBUG_LOG="$PROJECT_ROOT/.specweave/logs/hook-debug.log"
-
-  # Cancel existing batch job (if any)
-  if [ -f "$BATCH_LOCK" ]; then
-    local OLD_PID=$(cat "$BATCH_LOCK" 2>/dev/null || echo "")
-    if [ -n "$OLD_PID" ] && kill -0 "$OLD_PID" 2>/dev/null; then
-      kill "$OLD_PID" 2>/dev/null || true
-      echo "[$(date)] [BATCH] Canceled previous batch job (PID: $OLD_PID)" >> "$DEBUG_LOG" 2>/dev/null || true
-    fi
-    rm -f "$BATCH_LOCK"
-  fi
-
-  # Schedule new batch job (background process)
-  (
-    # Wait for idle period
-    sleep "$IDLE_DELAY"
-
-    # If operation counter still exists → run batched hook
-    if [ -f "$OPERATION_COUNTER" ]; then
-      local OPERATION_COUNT=$(cat "$OPERATION_COUNTER" 2>/dev/null || echo 0)
-
-      echo "[$(date)] [BATCH] Idle period detected, running batch job for $INCREMENT_ID ($OPERATION_COUNT ops)" >> "$DEBUG_LOG" 2>/dev/null || true
-
-      # Clear operation counter (batch job starting)
-      rm -f "$OPERATION_COUNTER" "$OPERATION_TIMESTAMP"
-
-      # Run consolidated hook in batch mode
-      bash "$PROJECT_ROOT/plugins/specweave/hooks/post-task-completion.sh" \
-        --batch-mode \
-        --increment "$INCREMENT_ID" \
-        >> "$PROJECT_ROOT/.specweave/logs/batch-jobs.log" 2>&1
-
-      echo "[$(date)] [BATCH] ✅ Batch job completed for $INCREMENT_ID" >> "$DEBUG_LOG" 2>/dev/null || true
-    else
-      echo "[$(date)] [BATCH] Operation counter cleared before batch job ran (duplicate batch?)" >> "$DEBUG_LOG" 2>/dev/null || true
-    fi
-
-    # Cleanup batch lock
-    rm -f "$BATCH_LOCK"
-  ) &
-
-  # Save PID for cancellation
-  local BATCH_PID=$!
-  echo "$BATCH_PID" > "$BATCH_LOCK"
-
-  echo "[$(date)] [BATCH] Scheduled batch job for $INCREMENT_ID (PID: $BATCH_PID, delay: ${IDLE_DELAY}s)" >> "$DEBUG_LOG" 2>/dev/null || true
-}
-
-##
-# Reset bulk operation counter
-#
-# Manually resets the operation counter (useful for testing or troubleshooting).
-##
-reset_bulk_counter() {
-  rm -f "$OPERATION_COUNTER" "$OPERATION_TIMESTAMP"
-  echo "[$(date)] [BULK] Operation counter reset" >> "$PROJECT_ROOT/.specweave/logs/hook-debug.log" 2>/dev/null || true
-}
-
-# Export functions for use in hooks
-export -f detect_bulk_operation
-export -f schedule_batch_job
-export -f reset_bulk_counter

package/plugins/specweave/hooks/test-pretooluse-env.sh

@@ -1,72 +0,0 @@
-#!/bin/bash
-#
-# PreToolUse Environment Variable Validation Test
-#
-# Purpose: Validate that PreToolUse hooks receive tool arguments via env vars
-# This tests the core assumption of Tier 2 implementation
-#
-# Usage:
-#   1. Register this hook in plugin.json under PreToolUse:Edit
-#   2. Make 10 test edits to any file
-#   3. Review /tmp/pretooluse-test.log
-#   4. Check if TOOL_USE_ARGS is populated
-#
-# Success Criteria:
-#   - TOOL_USE_ARGS contains file_path
-#   - At least 80% of invocations have non-empty TOOL_USE_ARGS
-#
-# If this test fails, Tier 2 PreToolUse coordination cannot work
-# and we must rely on Tier 1 (mtime fallback) or proceed to Tier 3
-#
-# Version: v0.24.2 (Tier 2 Validation)
-# Date: 2025-11-22
-
-set +e  # Don't fail on errors (this is a diagnostic tool)
-
-TEST_LOG="/tmp/pretooluse-test.log"
-TIMESTAMP=$(date -u +%Y-%m-%dT%H:%M:%SZ)
-
-# Initialize log on first run
-if [[ ! -f "$TEST_LOG" ]]; then
-  echo "==================================" > "$TEST_LOG"
-  echo "PreToolUse Environment Variable Test" >> "$TEST_LOG"
-  echo "Started: $TIMESTAMP" >> "$TEST_LOG"
-  echo "==================================" >> "$TEST_LOG"
-  echo "" >> "$TEST_LOG"
-fi
-
-# Record this invocation
-echo "[$TIMESTAMP] PreToolUse:Edit invocation #$(wc -l < "$TEST_LOG")" >> "$TEST_LOG"
-
-# Check ALL tool-related environment variables
-echo "  TOOL_USE_ARGS:    ${TOOL_USE_ARGS:-<EMPTY>}" >> "$TEST_LOG"
-echo "  TOOL_USE_CONTENT: ${TOOL_USE_CONTENT:-<EMPTY>}" >> "$TEST_LOG"
-echo "  TOOL_RESULT:      ${TOOL_RESULT:-<EMPTY>}" >> "$TEST_LOG"
-echo "  PWD:              $PWD" >> "$TEST_LOG"
-
-# Try to extract file_path if TOOL_USE_ARGS exists
-if [[ -n "${TOOL_USE_ARGS:-}" ]]; then
-  if command -v jq &> /dev/null; then
-    FILE_PATH=$(echo "$TOOL_USE_ARGS" | jq -r '.file_path // "<NOT_FOUND>"' 2>/dev/null)
-    echo "  Extracted file_path: $FILE_PATH" >> "$TEST_LOG"
-  else
-    FILE_PATH=$(echo "$TOOL_USE_ARGS" | grep -o '"file_path"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"\([^"]*\)".*/\1/' || echo "<PARSE_FAILED>")
-    echo "  Extracted file_path (no jq): $FILE_PATH" >> "$TEST_LOG"
-  fi
-fi
-
-# Dump all TOOL* env vars
-echo "  All TOOL* environment variables:" >> "$TEST_LOG"
-env | grep -E "^TOOL" | while read line; do
-  echo "    $line" >> "$TEST_LOG"
-done
-
-echo "" >> "$TEST_LOG"
-
-# Log rotation: Keep last 100 invocations
-if [[ $(wc -l < "$TEST_LOG") -gt 500 ]]; then
-  tail -400 "$TEST_LOG" > "$TEST_LOG.tmp"
-  mv "$TEST_LOG.tmp" "$TEST_LOG"
-fi
-
-exit 0  # Always succeed (diagnostic tool)

package/plugins/specweave/hooks/validate-increment-completion.sh

@@ -1,113 +0,0 @@
-#!/bin/bash
-#
-# Pre-commit hook: Validate Increment Completion
-#
-# Prevents commits where increment status="completed" but:
-# - Acceptance criteria are still open (- [ ] **AC-...)
-# - Tasks are still pending (**Status**: [ ] pending)
-#
-# This prevents false completion and data integrity violations.
-#
-# Exit code 0: All completed increments are valid (allow commit)
-# Exit code 1: Invalid completion detected (block commit)
-
-# ANSI colors
-RED='\033[0;31m'
-YELLOW='\033[1;33m'
-GREEN='\033[0;32m'
-NC='\033[0m' # No Color
-
-# Check if in SpecWeave project
-if [ ! -d ".specweave/increments" ]; then
-  # Not a SpecWeave project, skip validation
-  exit 0
-fi
-
-# Track if any validation failures found
-has_failures=false
-
-# Check all increments
-for increment_dir in .specweave/increments/*/; do
-  # Skip if not a directory
-  [ ! -d "$increment_dir" ] && continue
-
-  increment_id=$(basename "$increment_dir")
-  spec_file="$increment_dir/spec.md"
-  tasks_file="$increment_dir/tasks.md"
-  metadata_file="$increment_dir/metadata.json"
-
-  # Skip if required files don't exist
-  [ ! -f "$spec_file" ] && continue
-  [ ! -f "$tasks_file" ] && continue
-
-  # Get status from spec.md frontmatter
-  spec_status=$(grep -m1 "^status:" "$spec_file" 2>/dev/null | cut -d: -f2 | tr -d ' ')
-
-  # Get status from metadata.json (if exists)
-  metadata_status=""
-  if [ -f "$metadata_file" ]; then
-    metadata_status=$(grep -m1 '"status"' "$metadata_file" 2>/dev/null | cut -d'"' -f4)
-  fi
-
-  # Only validate if status is "completed"
-  if [ "$spec_status" = "completed" ] || [ "$metadata_status" = "completed" ]; then
-    # Count open acceptance criteria (- [ ] **AC-)
-    open_acs=$(grep -c "^- \[ \] \*\*AC-" "$spec_file" 2>/dev/null || echo 0)
-
-    # Count pending tasks (**Status**: [ ] pending)
-    pending_tasks=$(grep -ic "\*\*Status\*\*:\s*\[\s*\]\s*pending" "$tasks_file" 2>/dev/null || echo 0)
-
-    # Validate
-    if [ "$open_acs" -gt 0 ] || [ "$pending_tasks" -gt 0 ]; then
-      has_failures=true
-
-      echo -e "${RED}❌ COMMIT BLOCKED: Invalid completion detected${NC}"
-      echo -e "${YELLOW}Increment: $increment_id${NC}"
-      echo -e "Status: $spec_status (spec.md) / $metadata_status (metadata.json)"
-      echo ""
-
-      if [ "$open_acs" -gt 0 ]; then
-        echo -e "${RED}  • $open_acs acceptance criteria still open${NC}"
-      fi
-
-      if [ "$pending_tasks" -gt 0 ]; then
-        echo -e "${RED}  • $pending_tasks tasks still pending${NC}"
-      fi
-
-      echo ""
-      echo -e "${YELLOW}Fix options:${NC}"
-      echo "  1. Complete the open work before committing"
-      echo "  2. Change status to 'active' or 'paused' in both spec.md and metadata.json"
-      echo ""
-      echo -e "${YELLOW}To see details:${NC}"
-      echo "  cat $spec_file | grep '- \[ \] \*\*AC-'"
-      echo "  cat $tasks_file | grep -i 'Status.*pending'"
-      echo ""
-    fi
-  fi
-done
-
-# Exit with failure if any validation failures
-if [ "$has_failures" = true ]; then
-  echo -e "${RED}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${NC}"
-  echo -e "${RED}COMMIT BLOCKED: Cannot commit increments with invalid completion${NC}"
-  echo -e "${RED}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${NC}"
-  echo ""
-  echo -e "${YELLOW}Why this matters:${NC}"
-  echo "  • Prevents false completion (status='completed' with open work)"
-  echo "  • Ensures data integrity (metadata matches reality)"
-  echo "  • Stops misleading status line and GitHub sync"
-  echo ""
-  echo -e "${YELLOW}To fix:${NC}"
-  echo "  1. Complete all open ACs and pending tasks, OR"
-  echo "  2. Change status to 'active'/'paused' in spec.md and metadata.json"
-  echo ""
-  echo -e "${YELLOW}To bypass (NOT RECOMMENDED):${NC}"
-  echo "  git commit --no-verify"
-  echo ""
-  exit 1
-fi
-
-# All validations passed
-echo -e "${GREEN}✅ Completion validation passed${NC}"
-exit 0

package/plugins/specweave/lib/hooks/consolidated-sync.js

@@ -1,288 +0,0 @@
-#!/usr/bin/env node
-/**
- * Consolidated Sync Hook
- *
- * PERFORMANCE OPTIMIZATION (v0.24.4):
- * =====================================
- * This script consolidates ALL post-task-completion operations into a SINGLE
- * Node.js process instead of spawning 5-6 separate processes.
- *
- * BEFORE (v0.24.3):
- * - Spawn 1: update-tasks-md.js       (10KB)
- * - Spawn 2: sync-living-docs.js      (18KB)
- * - Spawn 3: update-ac-status.js      (2KB)
- * - Spawn 4: translate-living-docs.js (4.4KB)
- * - Spawn 5: run-self-reflection.js   (4.1KB - conditional)
- * Total: 5-6 Node.js processes per TodoWrite
- *
- * AFTER (v0.24.4):
- * - Spawn 1: consolidated-sync.js (runs all operations sequentially)
- * Total: 1 Node.js process per TodoWrite
- *
- * IMPACT:
- * - 83% reduction in process spawning overhead
- * - Faster execution (shared Node.js context)
- * - Better error handling (single error boundary)
- * - Reduced system resource usage
- *
- * ROOT CAUSE ADDRESSED:
- * Multiple rapid TodoWrite calls were spawning 12+ concurrent Node.js processes,
- * causing process exhaustion and Claude Code crashes. This consolidation prevents
- * that by limiting to 1 process per hook fire.
- *
- * See: .specweave/increments/0051-automatic-github-sync/reports/HOOK-CRASH-ANALYSIS.md
- * ADR: (pending - will be created after testing)
- */
-
-// Import REAL implementations from existing modules
-import { updateTasksMd } from './update-tasks-md.js';
-import { syncLivingDocs } from './sync-living-docs.js';
-import { translateLivingDocs } from './translate-living-docs.js';
-
-// Import for AC status (uses ACStatusManager directly)
-import { ACStatusManager } from '../vendor/core/increment/ac-status-manager.js';
-
-// Import for GitHub sync (uses SyncCoordinator directly)
-// NOTE: Import from project root dist (not relative path)
-import { SyncCoordinator } from '../../../../dist/src/sync/sync-coordinator.js';
-import { consoleLogger } from '../vendor/utils/logger.js';
-
-// Import US completion orchestrator (NEW in v0.25.0+)
-import { syncCompletedUserStories } from './us-completion-orchestrator.js';
-
-// Import auto-transition (NEW in v0.35.0+ - CRITICAL for preventing auto-completion bug)
-import { checkAndTransitionToReadyForReview } from '../vendor/core/increment/status-auto-transition.js';
-
-// ============================================================================
-// WRAPPER: UPDATE AC STATUS
-// ============================================================================
-// Note: update-ac-status.js doesn't export a function, it only has CLI logic
-// So we replicate the core logic here using ACStatusManager
-async function updateACStatus(incrementId) {
-  try {
-    console.log(`\n🔄 [3/6] Syncing AC status for increment ${incrementId}...`);
-
-    const projectRoot = process.cwd();
-
-    if (process.env.SKIP_AC_SYNC === 'true') {
-      console.log('ℹ️  AC sync skipped (SKIP_AC_SYNC=true)');
-      return { success: true, message: 'Sync skipped' };
-    }
-
-    const manager = new ACStatusManager(projectRoot);
-    const result = await manager.syncACStatus(incrementId);
-
-    if (result.warnings && result.warnings.length > 0) {
-      console.log('\n⚠️  Warnings:');
-      result.warnings.forEach((warning) => console.log(`   ${warning}`));
-    }
-
-    if (result.conflicts && result.conflicts.length > 0) {
-      console.log('\n⚠️  Conflicts detected:');
-      result.conflicts.forEach((conflict) => console.log(`   ${conflict}`));
-    }
-
-    if (result.updated && result.updated.length > 0) {
-      console.log('\n✅ Updated AC checkboxes:');
-      result.updated.forEach((acId) => console.log(`   ${acId} → [x]`));
-    } else if (result.synced) {
-      console.log('✅ All ACs already in sync (no changes needed)');
-    } else {
-      console.log('ℹ️  No AC updates needed');
-    }
-
-    return { success: true, result };
-  } catch (error) {
-    console.error('❌ Error updating AC status:', error.message);
-    return { success: false, error: error.message };
-  }
-}
-
-// ============================================================================
-// WRAPPER: GITHUB SYNC
-// ============================================================================
-async function syncGitHub(incrementId) {
-  try {
-    console.log(`\n🔗 [5/6] Syncing to GitHub...`);
-
-    const projectRoot = process.cwd();
-
-    const coordinator = new SyncCoordinator({
-      projectRoot,
-      incrementId,
-      logger: consoleLogger
-    });
-
-    const result = await coordinator.syncIncrementCompletion();
-
-    if (result.success) {
-      console.log(`✅ GitHub sync completed (${result.userStoriesSynced} user stories synced)`);
-    } else {
-      console.warn(`⚠️  GitHub sync had errors (see logs)`);
-    }
-
-    return { success: result.success, result };
-  } catch (error) {
-    console.error('❌ Error syncing to GitHub:', error.message);
-    return { success: false, error: error.message };
-  }
-}
-
-// ============================================================================
-// MAIN EXECUTION
-// ============================================================================
-async function runConsolidatedSync(incrementId) {
-  console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-  console.log(`🚀 CONSOLIDATED SYNC: ${incrementId}`);
-  console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-
-  const startTime = Date.now();
-  const results = {};
-
-  try {
-    // OPERATION 0 (v0.35.0+ - CRITICAL): Check for ACTIVE → READY_FOR_REVIEW transition
-    // This MUST run FIRST to prevent the auto-completion bug!
-    // When all tasks are done, increment transitions to READY_FOR_REVIEW (not COMPLETED).
-    // Only /sw:done can then transition to COMPLETED with user approval.
-    console.log('\n📋 [0/6] Checking for status auto-transition...');
-    try {
-      const transitionResult = checkAndTransitionToReadyForReview(incrementId);
-      if (transitionResult.transitioned) {
-        console.log(`\n🎉 ${transitionResult.message}`);
-        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-        console.log('📋 INCREMENT READY FOR REVIEW');
-        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-        console.log(`   Status: ACTIVE → READY_FOR_REVIEW`);
-        console.log(`   Tasks: ${transitionResult.taskStatus?.completedTasks}/${transitionResult.taskStatus?.totalTasks} completed`);
-        console.log('');
-        console.log('   To close this increment, run:');
-        console.log(`   /sw:done ${incrementId}`);
-        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n');
-      } else {
-        console.log(`   ${transitionResult.message}`);
-      }
-      results.autoTransition = { success: true, ...transitionResult };
-    } catch (error) {
-      console.error('❌ Error checking auto-transition:', error.message);
-      results.autoTransition = { success: false, error: error.message };
-    }
-
-    // OPERATION 1: Update tasks.md (uses imported function)
-    console.log('\n🔄 [1/6] Updating tasks.md...');
-    try {
-      await updateTasksMd(incrementId);
-      results.updateTasks = { success: true };
-    } catch (error) {
-      console.error('❌ Error updating tasks.md:', error.message);
-      results.updateTasks = { success: false, error: error.message };
-    }
-
-    // OPERATION 2: Sync living docs (uses imported function)
-    console.log('\n📚 [2/6] Syncing living docs...');
-    try {
-      await syncLivingDocs(incrementId);
-      results.syncDocs = { success: true };
-    } catch (error) {
-      console.error('❌ Error syncing living docs:', error.message);
-      results.syncDocs = { success: false, error: error.message };
-    }
-
-    // OPERATION 3: Update AC status (uses local wrapper)
-    try {
-      results.updateAC = await updateACStatus(incrementId);
-    } catch (error) {
-      console.error('❌ Error updating AC status:', error.message);
-      results.updateAC = { success: false, error: error.message };
-    }
-
-    // OPERATION 4: Translate living docs (uses imported function)
-    console.log('\n🌐 [4/6] Checking translation needs...');
-    try {
-      await translateLivingDocs(incrementId);
-      results.translate = { success: true };
-    } catch (error) {
-      console.error('❌ Error translating docs:', error.message);
-      results.translate = { success: false, error: error.message };
-    }
-
-    // OPERATION 5: Detect and sync newly completed user stories (NEW in v0.25.0+)
-    // CRITICAL: This operation runs AFTER AC sync to ensure ACs are up-to-date
-    // When a user story becomes fully complete (all ACs satisfied), this:
-    // - Detects the completion
-    // - Updates living docs
-    // - Triggers external tool sync (GitHub/JIRA/ADO)
-    try {
-      results.usCompletion = await syncCompletedUserStories(incrementId);
-    } catch (error) {
-      console.error('❌ Error detecting completed user stories:', error.message);
-      results.usCompletion = { success: false, error: error.message };
-    }
-
-    // OPERATION 6: Sync to GitHub (NEW in v0.24.0+)
-    // FIX (v0.26.0): Skip GitHub sync in post-task-completion hook!
-    // WHY: GitHub sync should ONLY run on increment COMPLETION, not task completion
-    // This prevents 27 duplicate comments on every TodoWrite (see root cause analysis)
-    //
-    // GitHub sync is now ONLY triggered by:
-    // - post-increment-completion.sh (when status → "completed")
-    // - Manual sync via /specweave-github:sync command
-    // - US completion orchestrator (OPERATION 5) when user stories complete
-    //
-    // See: .specweave/increments/0051-*/reports/GITHUB-COMMENT-RECURSION-ROOT-CAUSE-2025-11-24.md
-    if (process.env.SKIP_GITHUB_SYNC === 'true') {
-      console.log('\n⏭️  [6/6] GitHub sync SKIPPED (called from post-task-completion hook)');
-      console.log('    GitHub sync will run automatically on increment completion.');
-      results.syncGitHub = { success: true, skipped: true };
-    } else {
-      try {
-        results.syncGitHub = await syncGitHub(incrementId);
-      } catch (error) {
-        console.error('❌ Error syncing to GitHub:', error.message);
-        results.syncGitHub = { success: false, error: error.message };
-      }
-    }
-
-    const duration = Date.now() - startTime;
-
-    console.log('\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
-    console.log(`✅ CONSOLIDATED SYNC COMPLETED in ${duration}ms`);
-    console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n');
-
-    // Summary of results
-    const successCount = Object.values(results).filter(r => r.success).length;
-    const totalCount = Object.keys(results).length;
-    console.log(`📊 Results: ${successCount}/${totalCount} operations successful`);
-
-    // Check for any failures (non-blocking - we tolerate partial failures)
-    const failures = Object.entries(results).filter(([_, result]) => !result.success);
-    if (failures.length > 0) {
-      console.warn('\n⚠️  Some operations had issues (non-blocking):');
-      failures.forEach(([op, result]) => {
-        console.warn(`   ${op}: ${result.error || 'Unknown error'}`);
-      });
-    }
-
-    // Always exit 0 to prevent hook failures from blocking Claude Code
-    process.exit(0);
-  } catch (error) {
-    console.error('\n❌ FATAL ERROR in consolidated sync:', error);
-    // Even on fatal error, exit 0 to prevent blocking Claude Code
-    process.exit(0);
-  }
-}
-
-// CLI Interface
-const isMainModule = import.meta.url === `file://${process.argv[1]}`;
-if (isMainModule) {
-  const incrementId = process.argv[2];
-
-  if (!incrementId) {
-    console.error('Usage: node consolidated-sync.js <increment-id>');
-    console.error('Example: node consolidated-sync.js 0051-automatic-github-sync');
-    process.exit(1);
-  }
-
-  runConsolidatedSync(incrementId);
-}
-
-export { runConsolidatedSync };