specweave 0.23.16 → 0.24.0

package/plugins/specweave/hooks/lib/migrate-increment-work.sh.bak

@@ -0,0 +1,245 @@
+#!/bin/bash
+
+# SpecWeave Increment Work Migration Utility
+# Purpose: Smart migration options when WIP limit is reached
+#
+# Options:
+#   1. Transfer work (move incomplete tasks to new increment)
+#   2. Adjust WIP limit (temporarily allow 3 active)
+#   3. Force-close (mark as complete without transferring work)
+
+set -euo pipefail
+
+# ============================================================================
+# CONFIGURATION
+# ============================================================================
+
+SPECWEAVE_DIR="${SPECWEAVE_DIR:-.specweave}"
+CONFIG_FILE="$SPECWEAVE_DIR/config.json"
+
+# ============================================================================
+# HELPER FUNCTIONS
+# ============================================================================
+
+# Get incomplete tasks from increment
+get_incomplete_tasks() {
+  local increment_id="$1"
+  local tasks_file="$SPECWEAVE_DIR/increments/$increment_id/tasks.md"
+
+  if [[ ! -f "$tasks_file" ]]; then
+    echo ""
+    return
+  fi
+
+  # Extract unchecked tasks (lines starting with "- [ ]")
+  grep -E "^\s*-\s*\[ \]" "$tasks_file" 2>/dev/null || echo ""
+}
+
+# Count incomplete tasks
+count_incomplete_tasks() {
+  local increment_id="$1"
+  local tasks=$(get_incomplete_tasks "$increment_id")
+
+  if [[ -z "$tasks" ]]; then
+    echo "0"
+  else
+    echo "$tasks" | wc -l | xargs
+  fi
+}
+
+# Transfer work from old increment to new increment
+transfer_work() {
+  local source_increment="$1"
+  local target_increment="$2"
+
+  echo "🔄 Transferring incomplete work from $source_increment to $target_increment..."
+
+  # Get incomplete tasks
+  local incomplete_tasks=$(get_incomplete_tasks "$source_increment")
+
+  if [[ -z "$incomplete_tasks" ]]; then
+    echo "✅ No incomplete tasks to transfer"
+    return 0
+  fi
+
+  local task_count=$(echo "$incomplete_tasks" | wc -l | xargs)
+  echo "   📋 Found $task_count incomplete tasks"
+
+  # Append to target increment's tasks.md
+  local target_tasks="$SPECWEAVE_DIR/increments/$target_increment/tasks.md"
+
+  if [[ ! -f "$target_tasks" ]]; then
+    echo "❌ Target increment tasks.md not found: $target_tasks"
+    return 1
+  fi
+
+  # Add separator and transferred tasks
+  cat >> "$target_tasks" <<EOF
+
+---
+
+## 🔄 Transferred from $source_increment
+
+$incomplete_tasks
+
+EOF
+
+  echo "   ✅ Transferred $task_count tasks to $target_tasks"
+
+  # Mark source increment as completed-with-work-transfer
+  local source_metadata="$SPECWEAVE_DIR/increments/$source_increment/metadata.json"
+
+  if [[ -f "$source_metadata" ]]; then
+    node -e "
+      const fs = require('fs');
+      const path = '$source_metadata';
+      const data = JSON.parse(fs.readFileSync(path, 'utf-8'));
+      data.status = 'completed';
+      data.completionNote = 'Work transferred to $target_increment';
+      data.completedAt = new Date().toISOString();
+      data.workTransferredTo = '$target_increment';
+      fs.writeFileSync(path, JSON.stringify(data, null, 2));
+    "
+    echo "   ✅ Updated $source_increment metadata (status: completed, work transferred)"
+  fi
+
+  # Update target increment metadata
+  local target_metadata="$SPECWEAVE_DIR/increments/$target_increment/metadata.json"
+
+  if [[ -f "$target_metadata" ]]; then
+    node -e "
+      const fs = require('fs');
+      const path = '$target_metadata';
+      const data = JSON.parse(fs.readFileSync(path, 'utf-8'));
+      data.workTransferredFrom = data.workTransferredFrom || [];
+      data.workTransferredFrom.push('$source_increment');
+      fs.writeFileSync(path, JSON.stringify(data, null, 2));
+    "
+    echo "   ✅ Updated $target_increment metadata (work received from $source_increment)"
+  fi
+
+  echo "✅ Work transfer complete!"
+  return 0
+}
+
+# Adjust WIP limit temporarily
+adjust_wip_limit() {
+  local new_limit="$1"
+
+  echo "⚠️  Temporarily adjusting WIP limit to $new_limit..."
+
+  if [[ ! -f "$CONFIG_FILE" ]]; then
+    echo "❌ Config file not found: $CONFIG_FILE"
+    return 1
+  fi
+
+  # Update config.json (create backup first)
+  cp "$CONFIG_FILE" "$CONFIG_FILE.bak"
+
+  node -e "
+    const fs = require('fs');
+    const path = '$CONFIG_FILE';
+    const data = JSON.parse(fs.readFileSync(path, 'utf-8'));
+
+    // Ensure limits object exists
+    if (!data.limits) {
+      data.limits = {};
+    }
+
+    // Store original limit
+    if (!data.limits.originalHardCap) {
+      data.limits.originalHardCap = data.limits.hardCap || 2;
+    }
+
+    // Set new limit
+    data.limits.hardCap = parseInt('$new_limit', 10);
+    data.limits.wipAdjustedAt = new Date().toISOString();
+
+    fs.writeFileSync(path, JSON.stringify(data, null, 2));
+  "
+
+  echo "   ✅ WIP limit adjusted to $new_limit"
+  echo "   ⚠️  Remember to revert after completing one increment!"
+  echo "   💡 Run: specweave revert-wip-limit"
+
+  return 0
+}
+
+# Force-close increment
+force_close() {
+  local increment_id="$1"
+
+  echo "⚠️  Force-closing $increment_id..."
+
+  local metadata_file="$SPECWEAVE_DIR/increments/$increment_id/metadata.json"
+
+  if [[ ! -f "$metadata_file" ]]; then
+    echo "❌ Metadata file not found: $metadata_file"
+    return 1
+  fi
+
+  node -e "
+    const fs = require('fs');
+    const path = '$metadata_file';
+    const data = JSON.parse(fs.readFileSync(path, 'utf-8'));
+    data.status = 'completed';
+    data.completionNote = 'Force-closed to start new increment (WIP limit reached)';
+    data.completedAt = new Date().toISOString();
+    data.forceCloseReason = 'wip-limit-reached';
+    fs.writeFileSync(path, JSON.stringify(data, null, 2));
+  "
+
+  echo "   ✅ $increment_id marked as completed (force-closed)"
+  echo "   ⚠️  Incomplete work was not transferred!"
+
+  return 0
+}
+
+# ============================================================================
+# MAIN CLI
+# ============================================================================
+
+case "${1:-}" in
+  transfer)
+    if [[ $# -ne 3 ]]; then
+      echo "Usage: $0 transfer <source_increment> <target_increment>"
+      exit 1
+    fi
+    transfer_work "$2" "$3"
+    ;;
+
+  adjust-wip)
+    if [[ $# -ne 2 ]]; then
+      echo "Usage: $0 adjust-wip <new_limit>"
+      exit 1
+    fi
+    adjust_wip_limit "$2"
+    ;;
+
+  force-close)
+    if [[ $# -ne 2 ]]; then
+      echo "Usage: $0 force-close <increment_id>"
+      exit 1
+    fi
+    force_close "$2"
+    ;;
+
+  count-incomplete)
+    if [[ $# -ne 2 ]]; then
+      echo "Usage: $0 count-incomplete <increment_id>"
+      exit 1
+    fi
+    count_incomplete_tasks "$2"
+    ;;
+
+  *)
+    echo "SpecWeave Increment Work Migration Utility"
+    echo ""
+    echo "Usage:"
+    echo "  $0 transfer <source> <target>    # Transfer work from source to target"
+    echo "  $0 adjust-wip <limit>             # Adjust WIP limit temporarily"
+    echo "  $0 force-close <increment>        # Force-close increment"
+    echo "  $0 count-incomplete <increment>   # Count incomplete tasks"
+    exit 1
+    ;;
+esac

package/plugins/specweave/hooks/lib/sync-spec-content.sh

@@ -15,7 +15,7 @@
 #   sync-spec-content.sh <spec-path>
 #
 
-set -euo pipefail
+set +e  # EMERGENCY FIX: Changed from set -euo pipefail to prevent Claude Code crashes
 
 # Get script directory
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
@@ -131,7 +131,7 @@ echo ""
 set +e
 node "$SYNC_CLI" --spec "$SPEC_PATH" --provider "$PROVIDER"
 SYNC_EXIT_CODE=$?
-set -e
+set +e  # EMERGENCY FIX: Changed from set -e to prevent Claude Code crashes
 
 if [ $SYNC_EXIT_CODE -eq 0 ]; then
   echo ""

package/plugins/specweave/hooks/lib/sync-spec-content.sh.bak

@@ -0,0 +1,149 @@
+#!/bin/bash
+#
+# Sync Spec Content to External Tools
+#
+# This script automatically syncs spec CONTENT (title, description, user stories)
+# to external tools (GitHub Issues, JIRA Epics, Azure DevOps Features).
+#
+# CRITICAL: This does NOT sync STATUS - that's managed by external tools.
+#
+# Sync Direction:
+# - Title/Description/User Stories: SpecWeave → External Tool (we update)
+# - Status/State: External Tool → SpecWeave (we read)
+#
+# Usage:
+#   sync-spec-content.sh <spec-path>
+#
+
+set -euo pipefail
+
+# Get script directory
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(cd "$SCRIPT_DIR/../../../.." && pwd)"
+
+# Input validation
+if [ $# -lt 1 ]; then
+  echo "Usage: $0 <spec-path>" >&2
+  exit 1
+fi
+
+SPEC_PATH="$1"
+
+# Validate spec file exists
+if [ ! -f "$SPEC_PATH" ]; then
+  echo "❌ Spec file not found: $SPEC_PATH" >&2
+  exit 1
+fi
+
+# Check if sync is enabled
+CONFIG_FILE="$PROJECT_ROOT/.specweave/config.json"
+if [ ! -f "$CONFIG_FILE" ]; then
+  echo "ℹ️  No config file found, skipping spec content sync" >&2
+  exit 0
+fi
+
+# Check if sync is enabled in config
+SYNC_ENABLED=$(node -p "
+  try {
+    const config = require('$CONFIG_FILE');
+    config.sync?.enabled ?? false;
+  } catch {
+    false;
+  }
+" 2>/dev/null || echo "false")
+
+if [ "$SYNC_ENABLED" != "true" ]; then
+  echo "ℹ️  Sync disabled in config, skipping spec content sync" >&2
+  exit 0
+fi
+
+# Check if spec content sync is enabled
+SPEC_SYNC_ENABLED=$(node -p "
+  try {
+    const config = require('$CONFIG_FILE');
+    config.sync?.settings?.syncSpecContent ?? true;
+  } catch {
+    true;
+  }
+" 2>/dev/null || echo "true")
+
+if [ "$SPEC_SYNC_ENABLED" != "true" ]; then
+  echo "ℹ️  Spec content sync disabled in config" >&2
+  exit 0
+fi
+
+# Get active provider
+ACTIVE_PROFILE=$(node -p "
+  try {
+    const config = require('$CONFIG_FILE');
+    config.sync?.activeProfile ?? null;
+  } catch {
+    null;
+  }
+" 2>/dev/null || echo "null")
+
+if [ "$ACTIVE_PROFILE" = "null" ]; then
+  echo "ℹ️  No active sync profile, skipping spec content sync" >&2
+  exit 0
+fi
+
+# Get provider from active profile
+PROVIDER=$(node -p "
+  try {
+    const config = require('$CONFIG_FILE');
+    config.sync?.profiles?.['"$ACTIVE_PROFILE"']?.provider ?? null;
+  } catch {
+    null;
+  }
+" 2>/dev/null || echo "null")
+
+if [ "$PROVIDER" = "null" ]; then
+  echo "ℹ️  No provider configured for profile '$ACTIVE_PROFILE'" >&2
+  exit 0
+fi
+
+# Validate provider
+if [[ ! "$PROVIDER" =~ ^(github|jira|ado)$ ]]; then
+  echo "❌ Invalid provider: $PROVIDER (must be github, jira, or ado)" >&2
+  exit 1
+fi
+
+# Check if sync CLI command exists
+SYNC_CLI="$PROJECT_ROOT/dist/src/cli/commands/sync-spec-content.js"
+if [ ! -f "$SYNC_CLI" ]; then
+  echo "ℹ️  Sync CLI not built, skipping spec content sync" >&2
+  echo "   Run: npm run build" >&2
+  exit 0
+fi
+
+# Check if Node.js is available
+if ! command -v node &> /dev/null; then
+  echo "❌ Node.js not found, cannot sync spec content" >&2
+  exit 1
+fi
+
+echo ""
+echo "🔄 Syncing spec content to $PROVIDER..."
+echo "   Spec: $(basename "$SPEC_PATH")"
+echo ""
+
+# Run sync
+set +e
+node "$SYNC_CLI" --spec "$SPEC_PATH" --provider "$PROVIDER"
+SYNC_EXIT_CODE=$?
+set -e
+
+if [ $SYNC_EXIT_CODE -eq 0 ]; then
+  echo ""
+  echo "✅ Spec content synced successfully"
+  echo ""
+else
+  echo ""
+  echo "⚠️  Spec content sync failed (exit code: $SYNC_EXIT_CODE)" >&2
+  echo "   This is non-blocking. You can sync manually later:" >&2
+  echo "   node dist/src/cli/commands/sync-spec-content.js --spec \"$SPEC_PATH\" --provider $PROVIDER" >&2
+  echo ""
+  # Non-blocking: continue execution even if sync fails
+fi
+
+exit 0

package/plugins/specweave/hooks/lib/update-status-line.sh

@@ -14,8 +14,12 @@
 # 6. Write to cache
 #
 # Performance: 50-100ms (runs async, user doesn't wait)
+#
+# EMERGENCY FIX (v0.24.4): Changed from set -euo pipefail to set +e
+# CRITICAL: Hooks MUST use set +e to prevent Claude Code crashes!
+# See: CLAUDE.md section 9a - Hook Performance & Safety
 
-set -euo pipefail
+set +e
 
 # Find project root
 find_project_root() {
@@ -149,8 +153,20 @@ INCREMENT_ID=$(echo "$CURRENT_INCREMENT" | grep -oE '^[0-9]{4}')
 INCREMENT_NAME_ONLY=$(echo "$CURRENT_INCREMENT" | sed 's/^[0-9]\{4\}-//')
 INCREMENT_NAME="$INCREMENT_ID-$INCREMENT_NAME_ONLY"
 
-# Step 6: Write cache (now includes AC metrics)
-jq -n \
+# Step 6: Write cache with atomic validation (v0.24.4 - prevents corruption)
+TMP_CACHE_FILE="$PROJECT_ROOT/.specweave/state/.status-line-tmp.json"
+
+# Validate all numeric inputs before jq (prevents jq failures)
+[[ "$TOTAL_TASKS" =~ ^[0-9]+$ ]] || TOTAL_TASKS=0
+[[ "$COMPLETED_TASKS" =~ ^[0-9]+$ ]] || COMPLETED_TASKS=0
+[[ "$PERCENTAGE" =~ ^[0-9]+$ ]] || PERCENTAGE=0
+[[ "$TOTAL_ACS" =~ ^[0-9]+$ ]] || TOTAL_ACS=0
+[[ "$COMPLETED_ACS" =~ ^[0-9]+$ ]] || COMPLETED_ACS=0
+[[ "$OPEN_ACS" =~ ^[0-9]+$ ]] || OPEN_ACS=0
+[[ "$OPEN_COUNT" =~ ^[0-9]+$ ]] || OPEN_COUNT=0
+
+# Generate cache to temp file first (atomic operation)
+if jq -n \
   --arg id "$CURRENT_INCREMENT" \
   --arg name "$INCREMENT_NAME" \
   --argjson completed "$COMPLETED_TASKS" \
@@ -171,6 +187,20 @@ jq -n \
     },
     openCount: $openCount,
     lastUpdate: (now | strftime("%Y-%m-%dT%H:%M:%SZ"))
-  }' > "$CACHE_FILE"
+  }' > "$TMP_CACHE_FILE" 2>/dev/null; then
+
+  # Validate generated JSON before replacing cache (corruption prevention)
+  if jq empty "$TMP_CACHE_FILE" 2>/dev/null; then
+    mv "$TMP_CACHE_FILE" "$CACHE_FILE"
+  else
+    # Invalid JSON generated - keep old cache
+    echo "[$(date)] ERROR: Generated invalid JSON, keeping old cache" >> "$DEBUG_LOG" 2>/dev/null || true
+    rm -f "$TMP_CACHE_FILE"
+  fi
+else
+  # jq generation failed - keep old cache
+  echo "[$(date)] ERROR: jq failed to generate status cache (exit $?)" >> "$DEBUG_LOG" 2>/dev/null || true
+  rm -f "$TMP_CACHE_FILE"
+fi
 
 exit 0

package/plugins/specweave/hooks/lib/validate-spec-status.sh

@@ -16,7 +16,7 @@
 #
 # Requires: bash 4.0+ (for associative arrays)
 
-set -euo pipefail
+set +e  # EMERGENCY FIX: Changed from set -euo pipefail to prevent Claude Code crashes
 
 # Find project root
 find_project_root() {

package/plugins/specweave/hooks/lib/validate-spec-status.sh.bak

@@ -0,0 +1,163 @@
+#!/bin/bash
+#
+# validate-spec-status.sh
+#
+# Validates that spec.md status values match IncrementStatus enum.
+# Prevents vocabulary drift and ensures status line hook compatibility.
+#
+# Usage:
+#   bash validate-spec-status.sh <increment-id>
+#   bash validate-spec-status.sh --all  # Validate all increments
+#
+# Exit codes:
+#   0 = Valid
+#   1 = Invalid status found
+#   2 = Error (file not found, etc.)
+#
+# Requires: bash 4.0+ (for associative arrays)
+
+set -euo pipefail
+
+# Find project root
+find_project_root() {
+  local dir="$PWD"
+  while [[ "$dir" != "/" ]]; do
+    if [[ -d "$dir/.specweave" ]]; then
+      echo "$dir"
+      return 0
+    fi
+    dir=$(dirname "$dir")
+  done
+  echo "$PWD"
+}
+
+PROJECT_ROOT=$(find_project_root)
+INCREMENTS_DIR="$PROJECT_ROOT/.specweave/increments"
+
+# Valid IncrementStatus enum values (from src/core/types/increment-metadata.ts)
+VALID_STATUSES=("planning" "active" "backlog" "paused" "completed" "abandoned")
+
+# Get suggested correction for invalid status
+get_correction() {
+  local status="$1"
+  case "$status" in
+    "planned") echo "planning" ;;
+    "in-progress"|"in_progress") echo "active" ;;
+    "done") echo "completed" ;;
+    "cancelled"|"canceled") echo "abandoned" ;;
+    *) echo "" ;;
+  esac
+}
+
+# Validate a single spec file
+validate_spec() {
+  local spec_file="$1"
+  local increment_id=$(basename "$(dirname "$spec_file")")
+
+  if [[ ! -f "$spec_file" ]]; then
+    echo "❌ Error: spec.md not found for increment $increment_id"
+    return 2
+  fi
+
+  # Extract status from YAML frontmatter
+  local status=$(grep -m1 "^status:" "$spec_file" 2>/dev/null | cut -d: -f2 | tr -d ' "' || echo "")
+
+  if [[ -z "$status" ]]; then
+    echo "⚠️  Warning: No status field in $increment_id/spec.md"
+    return 0  # Not an error, just a warning
+  fi
+
+  # Check if status is valid
+  local is_valid=0
+  for valid_status in "${VALID_STATUSES[@]}"; do
+    if [[ "$status" == "$valid_status" ]]; then
+      is_valid=1
+      break
+    fi
+  done
+
+  if [[ $is_valid -eq 1 ]]; then
+    echo "✅ $increment_id: status '$status' is valid"
+    return 0
+  else
+    # Invalid status found - suggest correction
+    echo ""
+    echo "❌ Invalid status in $increment_id/spec.md"
+    echo "   Found: '$status'"
+    echo ""
+
+    # Check if we have a suggested correction
+    local correction=$(get_correction "$status")
+    if [[ -n "$correction" ]]; then
+      echo "   💡 Did you mean: '$correction'?"
+      echo ""
+      echo "   Fix:"
+      echo "   sed -i '' 's/^status: $status/status: $correction/' $spec_file"
+    else
+      echo "   Valid statuses: ${VALID_STATUSES[*]}"
+    fi
+    echo ""
+
+    return 1
+  fi
+}
+
+# Validate all increments
+validate_all() {
+  local exit_code=0
+  local total=0
+  local valid=0
+  local invalid=0
+
+  echo "Validating all increments in $INCREMENTS_DIR"
+  echo ""
+
+  for spec_file in "$INCREMENTS_DIR"/*/spec.md; do
+    if [[ -f "$spec_file" ]]; then
+      total=$((total + 1))
+      if validate_spec "$spec_file"; then
+        valid=$((valid + 1))
+      else
+        invalid=$((invalid + 1))
+        exit_code=1
+      fi
+    fi
+  done
+
+  echo ""
+  echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+  echo "Summary: $total increments checked"
+  echo "  ✅ Valid: $valid"
+  echo "  ❌ Invalid: $invalid"
+  echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+
+  if [[ $invalid -gt 0 ]]; then
+    echo ""
+    echo "⚠️  Please fix invalid status values to use official IncrementStatus enum."
+    echo ""
+    echo "Valid statuses:"
+    for status in "${VALID_STATUSES[@]}"; do
+      echo "  - $status"
+    done
+  fi
+
+  return $exit_code
+}
+
+# Main logic
+if [[ $# -eq 0 ]]; then
+  echo "Usage: $0 <increment-id> | --all"
+  echo ""
+  echo "Examples:"
+  echo "  $0 0042-test-infrastructure-cleanup"
+  echo "  $0 --all"
+  exit 2
+fi
+
+if [[ "$1" == "--all" ]]; then
+  validate_all
+else
+  INCREMENT_ID="$1"
+  SPEC_FILE="$INCREMENTS_DIR/$INCREMENT_ID/spec.md"
+  validate_spec "$SPEC_FILE"
+fi

package/plugins/specweave/hooks/post-first-increment.sh

@@ -6,7 +6,7 @@
 #
 # NON-INTERACTIVE: Just shows a message (hooks run in background)
 
-set -euo pipefail
+set +e  # EMERGENCY FIX: Changed from set -euo pipefail to prevent Claude Code crashes
 
 # Get project root (where .specweave/ lives)
 PROJECT_ROOT="$(pwd)"