specweave 0.33.2 → 0.33.3

package/plugins/specweave/hooks/user-prompt-submit.sh

@@ -204,7 +204,91 @@ fi
 # DISCIPLINE VALIDATION: Warn about WIP limits (configurable, not hard block!)
 # ==============================================================================
 
-if echo "$PROMPT" | grep -q "/specweave:increment"; then
+# ==============================================================================
+# PROJECT CONTEXT + WIP LIMITS FOR /specweave:increment (v0.34.0)
+# ==============================================================================
+# CRITICAL: Inject project/board context BEFORE Claude generates spec.md
+# This ensures Claude knows available projects and uses correct IDs
+# ALSO: Check WIP limits in same block to avoid duplicate command detection
+
+if echo "$PROMPT" | grep -qE "^/specweave:increment"; then
+  # Get project context (uses specweave CLI if available)
+  PROJECT_CONTEXT=""
+
+  if command -v specweave >/dev/null 2>&1; then
+    # Use CLI for accurate project/board detection
+    CONTEXT_JSON=$(specweave context projects 2>/dev/null || echo '{}')
+
+    # Validate JSON before parsing (defensive coding)
+    if [[ -n "$CONTEXT_JSON" ]] && [[ "$CONTEXT_JSON" != "{}" ]]; then
+      if command -v jq >/dev/null 2>&1; then
+        # Verify JSON is parseable before extracting fields
+        if ! echo "$CONTEXT_JSON" | jq empty 2>/dev/null; then
+          CONTEXT_JSON='{}'  # Invalid JSON - reset to empty
+        fi
+      fi
+    fi
+
+    if [[ -n "$CONTEXT_JSON" ]] && [[ "$CONTEXT_JSON" != "{}" ]]; then
+      # Parse JSON with jq
+      if command -v jq >/dev/null 2>&1; then
+        LEVEL=$(echo "$CONTEXT_JSON" | jq -r '.level // 1')
+        PROJECTS=$(echo "$CONTEXT_JSON" | jq -r '.projects | map(.id) | join(", ")' 2>/dev/null || echo "")
+
+        if [[ "$LEVEL" == "2" ]]; then
+          # 2-level structure: include boards
+          BOARDS_JSON=$(echo "$CONTEXT_JSON" | jq -r '.boardsByProject // {}' 2>/dev/null)
+          if [[ -n "$BOARDS_JSON" ]] && [[ "$BOARDS_JSON" != "{}" ]]; then
+            PROJECT_CONTEXT="\\n\\n📦 PROJECT CONTEXT (2-LEVEL STRUCTURE)\\n\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}⚠️ MANDATORY: Each User Story MUST have both:\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}  - **Project**: <project_id>\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}  - **Board**: <board_id>\\n\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}Available projects: ${PROJECTS}\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}Boards by project:\\n"
+
+            # Format boards
+            for proj in $(echo "$CONTEXT_JSON" | jq -r '.projects[].id' 2>/dev/null); do
+              PROJ_BOARDS=$(echo "$CONTEXT_JSON" | jq -r ".boardsByProject[\"$proj\"] | map(.id) | join(\", \")" 2>/dev/null || echo "")
+              [[ -n "$PROJ_BOARDS" ]] && PROJECT_CONTEXT="${PROJECT_CONTEXT}  - ${proj}: ${PROJ_BOARDS}\\n"
+            done
+
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}\\n❌ FORBIDDEN: Comma-separated values (e.g., **Project**: fe, be)\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}✅ REQUIRED: One project + one board per User Story"
+          fi
+        elif [[ -n "$PROJECTS" ]]; then
+          # 1-level structure: projects only
+          PROJECT_COUNT=$(echo "$CONTEXT_JSON" | jq '.projects | length' 2>/dev/null || echo "0")
+
+          if [[ "$PROJECT_COUNT" -gt 1 ]]; then
+            PROJECT_CONTEXT="\\n\\n📦 PROJECT CONTEXT (MULTI-PROJECT)\\n\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}⚠️ MANDATORY: Each User Story MUST have:\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}  - **Project**: <project_id>\\n\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}Available projects: ${PROJECTS}\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}\\n❌ FORBIDDEN: Comma-separated values\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}✅ REQUIRED: One project per User Story"
+          elif [[ "$PROJECT_COUNT" -eq 1 ]]; then
+            # Single project: auto-select
+            SINGLE_PROJECT=$(echo "$CONTEXT_JSON" | jq -r '.projects[0].id' 2>/dev/null)
+            PROJECT_CONTEXT="\\n\\n📦 PROJECT CONTEXT\\n"
+            PROJECT_CONTEXT="${PROJECT_CONTEXT}Single project detected: ${SINGLE_PROJECT} (auto-selected)"
+          fi
+        fi
+      fi
+    fi
+  else
+    # Fallback: Check for multi-project folders
+    if [[ -d "$SPECWEAVE_DIR/docs/internal/specs" ]]; then
+      PROJ_COUNT=$(find "$SPECWEAVE_DIR/docs/internal/specs" -maxdepth 1 -type d | wc -l)
+      if [[ "$PROJ_COUNT" -gt 2 ]]; then
+        PROJ_LIST=$(ls -1 "$SPECWEAVE_DIR/docs/internal/specs" 2>/dev/null | grep -v "_" | tr '\n' ', ' | sed 's/,$//')
+        PROJECT_CONTEXT="\\n\\n📦 PROJECT CONTEXT (MULTI-PROJECT)\\n"
+        PROJECT_CONTEXT="${PROJECT_CONTEXT}⚠️ MANDATORY: Each User Story MUST have **Project**: field\\n"
+        PROJECT_CONTEXT="${PROJECT_CONTEXT}Available folders: ${PROJ_LIST}"
+      fi
+    fi
+  fi
+
+  # WIP LIMITS CHECK (inside same block - no duplicate command detection)
   # Read limits from config.json (respect user's settings!)
   CONFIG_FILE="$SPECWEAVE_DIR/config.json"
   SOFT_LIMIT=1
@@ -226,13 +310,31 @@ if echo "$PROMPT" | grep -q "/specweave:increment"; then
 
   # Above hard cap: strong warning but NOT a block (user decides!)
   if [[ "$ACTIVE_COUNT" -ge "$HARD_CAP" ]]; then
-    printf '{"decision":"approve","systemMessage":"⚠️  WIP LIMIT EXCEEDED (%s/%s)\\n\\nYou have %s active increments (configured maximum: %s)\\n\\nActive increments:\\n%s\\n\\n🧠 Research shows 3+ concurrent tasks = 40%% slower + more bugs\\n\\n💡 Options:\\n  1️⃣  Complete an increment: /specweave:done <id>\\n  2️⃣  Pause an increment: /specweave:pause <id>\\n  3️⃣  Increase limit: Edit .specweave/config.json limits.hardCap\\n  4️⃣  Continue anyway (not recommended)\\n\\n📝 To proceed anyway, just confirm your intent."}\n' "$ACTIVE_COUNT" "$HARD_CAP" "$ACTIVE_COUNT" "$HARD_CAP" "$ACTIVE_LIST"
+    WIP_MSG="⚠️  WIP LIMIT EXCEEDED (${ACTIVE_COUNT}/${HARD_CAP})\\n\\nYou have ${ACTIVE_COUNT} active increments (configured maximum: ${HARD_CAP})\\n\\nActive increments:\\n${ACTIVE_LIST}\\n\\n🧠 Research shows 3+ concurrent tasks = 40%% slower + more bugs\\n\\n💡 Options:\\n  1️⃣  Complete an increment: /specweave:done <id>\\n  2️⃣  Pause an increment: /specweave:pause <id>\\n  3️⃣  Increase limit: Edit .specweave/config.json limits.hardCap\\n  4️⃣  Continue anyway (not recommended)\\n\\n📝 To proceed anyway, just confirm your intent."
+    # Prepend project context if available
+    if [[ -n "$PROJECT_CONTEXT" ]]; then
+      printf '{"decision":"approve","systemMessage":"%s%s"}\n' "$PROJECT_CONTEXT" "$WIP_MSG"
+    else
+      printf '{"decision":"approve","systemMessage":"%s"}\n' "$WIP_MSG"
+    fi
     exit 0
   fi
 
   # At soft limit: mild warning, approve
   if [[ "$ACTIVE_COUNT" -ge "$SOFT_LIMIT" ]]; then
-    printf '{"decision":"approve","systemMessage":"⚠️  WIP LIMIT REACHED (%s/%s)\\n\\nYou have %s active increment(s) (recommended limit: %s)\\n\\nActive increments:\\n%s\\n\\n🧠 Focus Principle: Fewer active increments = maximum productivity\\n\\n💡 Consider:\\n  1️⃣  Complete current work (recommended)\\n  2️⃣  Pause current work (/specweave:pause)\\n  3️⃣  Continue anyway\\n\\n⚠️  Emergency hotfix/bug? Use --type=hotfix or --type=bug"}\n' "$ACTIVE_COUNT" "$SOFT_LIMIT" "$ACTIVE_COUNT" "$SOFT_LIMIT" "$ACTIVE_LIST"
+    WIP_MSG="⚠️  WIP LIMIT REACHED (${ACTIVE_COUNT}/${SOFT_LIMIT})\\n\\nYou have ${ACTIVE_COUNT} active increment(s) (recommended limit: ${SOFT_LIMIT})\\n\\nActive increments:\\n${ACTIVE_LIST}\\n\\n🧠 Focus Principle: Fewer active increments = maximum productivity\\n\\n💡 Consider:\\n  1️⃣  Complete current work (recommended)\\n  2️⃣  Pause current work (/specweave:pause)\\n  3️⃣  Continue anyway\\n\\n⚠️  Emergency hotfix/bug? Use --type=hotfix or --type=bug"
+    # Prepend project context if available
+    if [[ -n "$PROJECT_CONTEXT" ]]; then
+      printf '{"decision":"approve","systemMessage":"%s%s"}\n' "$PROJECT_CONTEXT" "$WIP_MSG"
+    else
+      printf '{"decision":"approve","systemMessage":"%s"}\n' "$WIP_MSG"
+    fi
+    exit 0
+  fi
+
+  # No WIP limit warning, but we may have project context to inject
+  if [[ -n "$PROJECT_CONTEXT" ]]; then
+    printf '{"decision":"approve","systemMessage":"%s"}\n' "$PROJECT_CONTEXT"
     exit 0
   fi
 fi

package/plugins/specweave/hooks/v2/guards/per-us-project-validator.sh

@@ -0,0 +1,281 @@
+#!/bin/bash
+#
+# per-us-project-validator.sh
+#
+# Pre-tool-use hook that validates EACH User Story in spec.md has
+# **Project**: (and **Board**: for 2-level) fields.
+#
+# This is the CRITICAL ENFORCEMENT for increment 0137.
+#
+# Activation:
+# - tool_name: Write (also Edit if full content provided)
+# - file_path matches: .specweave/increments/*/spec.md
+#
+# Rules:
+# - Each ### US-XXX section MUST have **Project**: <value> on next few lines
+# - For 2-level structures, each US MUST also have **Board**: <value>
+# - Project values MUST match configured projects (not generic keywords)
+# - Fallback allowed for existing specs via SPECWEAVE_LEGACY_SPEC=1
+#
+# Returns exit code 1 (block) if validation fails, 0 (allow) otherwise.
+#
+# Bypass:
+# - SPECWEAVE_FORCE_PROJECT=1 - Skip all project validation
+# - SPECWEAVE_LEGACY_SPEC=1 - Allow specs without per-US project (legacy mode)
+#
+
+set -e
+
+# Source common utilities if available
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PLUGIN_ROOT="${SCRIPT_DIR}/../.."
+
+# Logging (silent by default)
+log_debug() {
+  if [ "$SPECWEAVE_DEBUG" = "1" ]; then
+    echo "[per-us-project] $*" >&2
+  fi
+}
+
+# Check for force bypass
+if [ "$SPECWEAVE_FORCE_PROJECT" = "1" ]; then
+  echo '{"decision": "allow", "message": "⚠️  Per-US project validation bypassed (SPECWEAVE_FORCE_PROJECT=1)"}'
+  exit 0
+fi
+
+# Check for legacy mode bypass
+if [ "$SPECWEAVE_LEGACY_SPEC" = "1" ]; then
+  echo '{"decision": "allow", "message": "⚠️  Per-US project validation skipped (SPECWEAVE_LEGACY_SPEC=1 - legacy mode)"}'
+  exit 0
+fi
+
+# Read tool input from stdin
+INPUT=$(cat)
+
+# Extract tool name
+TOOL_NAME=$(echo "$INPUT" | jq -r '.tool_name // ""')
+
+# Only validate Write tool calls (Edit doesn't provide full content)
+if [ "$TOOL_NAME" != "Write" ]; then
+  echo '{"decision": "allow"}'
+  exit 0
+fi
+
+# Extract file path
+FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // ""')
+
+# Only validate spec.md files in increments folder
+if [[ ! "$FILE_PATH" =~ \.specweave/increments/[0-9]{3,4}E?-[^/]+/spec\.md$ ]]; then
+  echo '{"decision": "allow"}'
+  exit 0
+fi
+
+log_debug "Validating per-US project fields for: $FILE_PATH"
+
+# Extract file content
+CONTENT=$(echo "$INPUT" | jq -r '.tool_input.content // ""')
+
+# Count User Stories (### US-XXX pattern)
+US_PATTERN='### US-[0-9]+'
+TOTAL_US=$(echo "$CONTENT" | grep -cE "$US_PATTERN" || echo 0)
+
+log_debug "Total User Stories found: $TOTAL_US"
+
+# If no user stories, allow (might be tasks-only spec or overview)
+if [ "$TOTAL_US" -eq 0 ]; then
+  echo '{"decision": "allow"}'
+  exit 0
+fi
+
+# Extract User Story sections and check for **Project**: field
+# Strategy: For each ### US-XXX, look at next 10 lines for **Project**:
+
+MISSING_PROJECT=()
+MISSING_BOARD=()
+MULTI_PROJECT=()   # USs with multiple projects (comma-separated)
+MULTI_BOARD=()     # USs with multiple boards (comma-separated)
+US_WITH_PROJECT=0
+
+# Use awk to extract US sections and check for Project field
+while IFS= read -r us_line; do
+  # Extract US ID (e.g., US-001)
+  US_ID=$(echo "$us_line" | grep -oE 'US-[0-9]+' | head -1)
+
+  if [ -z "$US_ID" ]; then
+    continue
+  fi
+
+  # Get line number of this US heading
+  LINE_NUM=$(echo "$CONTENT" | grep -nE "^### $US_ID:" | head -1 | cut -d: -f1)
+
+  if [ -z "$LINE_NUM" ]; then
+    continue
+  fi
+
+  # Extract next 10 lines after heading
+  SECTION=$(echo "$CONTENT" | tail -n +$((LINE_NUM + 1)) | head -n 10)
+
+  # Check for **Project**: field
+  PROJECT_LINE=$(echo "$SECTION" | grep -E '^\*\*Project\*\*:\s*\S' | head -1)
+
+  if [ -n "$PROJECT_LINE" ]; then
+    US_WITH_PROJECT=$((US_WITH_PROJECT + 1))
+
+    # Extract project value
+    PROJECT_VALUE=$(echo "$PROJECT_LINE" | sed 's/^\*\*Project\*\*:\s*//' | tr -d '[:space:]')
+
+    # Check for comma (multiple projects - FORBIDDEN)
+    if echo "$PROJECT_VALUE" | grep -q ','; then
+      MULTI_PROJECT+=("$US_ID (has: $PROJECT_VALUE)")
+      log_debug "$US_ID has MULTIPLE projects (FORBIDDEN): $PROJECT_VALUE"
+    else
+      log_debug "$US_ID has **Project**: field ✓ ($PROJECT_VALUE)"
+    fi
+  else
+    MISSING_PROJECT+=("$US_ID")
+    log_debug "$US_ID MISSING **Project**: field ✗"
+  fi
+
+done < <(echo "$CONTENT" | grep -E "^### US-[0-9]+:")
+
+log_debug "User Stories with **Project**: $US_WITH_PROJECT / $TOTAL_US"
+
+# Get project root for structure detection
+PROJECT_ROOT="${FILE_PATH%%/.specweave/*}"
+cd "$PROJECT_ROOT" 2>/dev/null || true
+
+# Get structure level from config
+STRUCTURE_LEVEL=1
+if command -v specweave >/dev/null 2>&1; then
+  CONTEXT_OUTPUT=$(specweave context projects 2>/dev/null || echo '{"level": 1}')
+  STRUCTURE_LEVEL=$(echo "$CONTEXT_OUTPUT" | jq -r '.level // 1')
+  AVAILABLE_PROJECTS=$(echo "$CONTEXT_OUTPUT" | jq -r '.projects[].id // empty' | tr '\n' ', ' | sed 's/,$//')
+else
+  # Fallback: check for 2-level indicators in config
+  if [ -f "$PROJECT_ROOT/.specweave/config.json" ]; then
+    HAS_AREA_PATH=$(jq -r '.sync.profiles | to_entries | map(select(.value.config.areaPathMapping or .value.config.areaPaths)) | length' "$PROJECT_ROOT/.specweave/config.json" 2>/dev/null || echo 0)
+    HAS_BOARD_MAPPING=$(jq -r '.sync.profiles | to_entries | map(select(.value.config.boardMapping.boards | length > 0)) | length' "$PROJECT_ROOT/.specweave/config.json" 2>/dev/null || echo 0)
+
+    if [ "$HAS_AREA_PATH" -gt 0 ] || [ "$HAS_BOARD_MAPPING" -gt 0 ]; then
+      STRUCTURE_LEVEL=2
+    fi
+  fi
+fi
+
+log_debug "Structure level: $STRUCTURE_LEVEL"
+
+# For 2-level structures, also check for **Board**: field
+if [ "$STRUCTURE_LEVEL" = "2" ]; then
+  while IFS= read -r us_line; do
+    US_ID=$(echo "$us_line" | grep -oE 'US-[0-9]+' | head -1)
+
+    if [ -z "$US_ID" ]; then
+      continue
+    fi
+
+    LINE_NUM=$(echo "$CONTENT" | grep -nE "^### $US_ID:" | head -1 | cut -d: -f1)
+
+    if [ -z "$LINE_NUM" ]; then
+      continue
+    fi
+
+    SECTION=$(echo "$CONTENT" | tail -n +$((LINE_NUM + 1)) | head -n 10)
+
+    # Check for **Board**: field
+    BOARD_LINE=$(echo "$SECTION" | grep -E '^\*\*Board\*\*:\s*\S' | head -1)
+
+    if [ -n "$BOARD_LINE" ]; then
+      # Extract board value
+      BOARD_VALUE=$(echo "$BOARD_LINE" | sed 's/^\*\*Board\*\*:\s*//' | tr -d '[:space:]')
+
+      # Check for comma (multiple boards - FORBIDDEN)
+      if echo "$BOARD_VALUE" | grep -q ','; then
+        MULTI_BOARD+=("$US_ID (has: $BOARD_VALUE)")
+        log_debug "$US_ID has MULTIPLE boards (FORBIDDEN): $BOARD_VALUE"
+      else
+        log_debug "$US_ID has **Board**: field ✓ ($BOARD_VALUE)"
+      fi
+    else
+      MISSING_BOARD+=("$US_ID")
+      log_debug "$US_ID MISSING **Board**: field ✗"
+    fi
+
+  done < <(echo "$CONTENT" | grep -E "^### US-[0-9]+:")
+fi
+
+# Build error message if validation fails
+ERRORS=""
+
+if [ ${#MISSING_PROJECT[@]} -gt 0 ]; then
+  MISSING_LIST=$(printf ", %s" "${MISSING_PROJECT[@]}")
+  MISSING_LIST=${MISSING_LIST:2}  # Remove leading ", "
+
+  ERRORS="$ERRORS\n❌ User Stories MISSING **Project**: field:\n   ${MISSING_LIST}\n"
+  ERRORS="$ERRORS\n   Each User Story MUST have **Project**: <project_id> after the heading.\n"
+  ERRORS="$ERRORS\n   Example:\n"
+  ERRORS="$ERRORS   ### US-001: Login Form\n"
+  ERRORS="$ERRORS   **Project**: frontend-app\n"
+
+  if [ -n "$AVAILABLE_PROJECTS" ]; then
+    ERRORS="$ERRORS\n   Available projects: ${AVAILABLE_PROJECTS}\n"
+  fi
+fi
+
+if [ ${#MISSING_BOARD[@]} -gt 0 ]; then
+  MISSING_LIST=$(printf ", %s" "${MISSING_BOARD[@]}")
+  MISSING_LIST=${MISSING_LIST:2}
+
+  ERRORS="$ERRORS\n❌ User Stories MISSING **Board**: field (2-level structure):\n   ${MISSING_LIST}\n"
+  ERRORS="$ERRORS\n   For 2-level structures, each US MUST have **Board**: <board_id>\n"
+  ERRORS="$ERRORS\n   Example:\n"
+  ERRORS="$ERRORS   ### US-001: Login Form\n"
+  ERRORS="$ERRORS   **Project**: acme-corp\n"
+  ERRORS="$ERRORS   **Board**: mobile-team\n"
+fi
+
+# Check for multiple projects (1:1 violation)
+if [ ${#MULTI_PROJECT[@]} -gt 0 ]; then
+  MULTI_LIST=$(printf ", %s" "${MULTI_PROJECT[@]}")
+  MULTI_LIST=${MULTI_LIST:2}
+
+  ERRORS="$ERRORS\n❌ User Stories with MULTIPLE projects (FORBIDDEN - 1:1 mapping required):\n   ${MULTI_LIST}\n"
+  ERRORS="$ERRORS\n   Each User Story MUST map to EXACTLY ONE project.\n"
+  ERRORS="$ERRORS   Split cross-project features into separate USs:\n"
+  ERRORS="$ERRORS\n   WRONG:\n"
+  ERRORS="$ERRORS   ### US-001: OAuth Implementation\n"
+  ERRORS="$ERRORS   **Project**: frontend-app, backend-api  ← FORBIDDEN\n"
+  ERRORS="$ERRORS\n   CORRECT:\n"
+  ERRORS="$ERRORS   ### US-001: OAuth Login Form\n"
+  ERRORS="$ERRORS   **Project**: frontend-app\n"
+  ERRORS="$ERRORS\n   ### US-002: OAuth API Endpoints\n"
+  ERRORS="$ERRORS   **Project**: backend-api\n"
+fi
+
+# Check for multiple boards (1:1 violation)
+if [ ${#MULTI_BOARD[@]} -gt 0 ]; then
+  MULTI_LIST=$(printf ", %s" "${MULTI_BOARD[@]}")
+  MULTI_LIST=${MULTI_LIST:2}
+
+  ERRORS="$ERRORS\n❌ User Stories with MULTIPLE boards (FORBIDDEN - 1:1 mapping required):\n   ${MULTI_LIST}\n"
+  ERRORS="$ERRORS\n   Each User Story MUST map to EXACTLY ONE board.\n"
+  ERRORS="$ERRORS   Split cross-board features into separate USs.\n"
+fi
+
+# If errors found, block the write
+if [ -n "$ERRORS" ]; then
+  # Add bypass instructions
+  ERRORS="$ERRORS\n💡 To bypass this validation:\n"
+  ERRORS="$ERRORS   - SPECWEAVE_FORCE_PROJECT=1 (skip all validation)\n"
+  ERRORS="$ERRORS   - SPECWEAVE_LEGACY_SPEC=1 (allow specs without per-US project)\n"
+
+  # Escape for JSON
+  REASON=$(echo -e "$ERRORS" | jq -Rs .)
+
+  echo "{\"decision\": \"block\", \"reason\": $REASON}"
+  exit 0
+fi
+
+# All validations passed
+log_debug "All $TOTAL_US User Stories have required fields ✓"
+echo '{"decision": "allow"}'
+exit 0

package/plugins/specweave/hooks/v2/handlers/living-specs-handler.sh

@@ -120,6 +120,35 @@ case "$EVENT" in
         run_with_timeout 30 node "$SYNC_SCRIPT" "$INC_ID" >/dev/null 2>&1 &
       fi
     fi
+
+    # ========================================================================
+    # CRITICAL FIX (v0.34.0): Close GitHub/JIRA/ADO issues on increment.done
+    # ========================================================================
+    # Root cause: sync-increment-closure.js was NEVER being called when
+    # increments completed. The EDA architecture routes increment.done to
+    # this handler, but only sync-living-docs.js was called (updates docs).
+    #
+    # sync-increment-closure.js contains the actual logic to:
+    # - Close GitHub issues linked to User Stories
+    # - Close JIRA/ADO issues (with proper gates)
+    # - Post completion comments on external issues
+    #
+    # Without this, GitHub issues stayed OPEN forever after increment closure!
+    # See: GitHub Issues #817-#822 (increment 0136) stayed open until manual fix
+    CLOSURE_SCRIPT=""
+    for path in \
+      "$PROJECT_ROOT/plugins/specweave/lib/hooks/sync-increment-closure.js" \
+      "$PROJECT_ROOT/dist/plugins/specweave/lib/hooks/sync-increment-closure.js" \
+      "${CLAUDE_PLUGIN_ROOT:-}/lib/hooks/sync-increment-closure.js"; do
+      [[ -f "$path" ]] && { CLOSURE_SCRIPT="$path"; break; }
+    done
+
+    if [[ -n "$CLOSURE_SCRIPT" ]]; then
+      cd "$PROJECT_ROOT" || exit 0
+      echo "[$(date '+%Y-%m-%d %H:%M:%S')] Calling sync-increment-closure.js for $INC_ID" >> "$LOG_FILE" 2>/dev/null
+      # Run closure sync in background (can take 10-30s for multiple issues)
+      run_with_timeout 60 node "$CLOSURE_SCRIPT" "$INC_ID" >> "$LOG_FILE" 2>&1 &
+    fi
     ;;
 
   increment.archived)