specweave 0.17.0 → 0.17.1

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

@@ -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/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/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/post-increment-planning.sh

@@ -646,49 +646,71 @@ EOF
 
   translate_living_docs_specs "$increment_id"
 
-  # 7. Auto-create GitHub issue (if configured)
+  # 7. Increment-level GitHub sync (DISABLED - See architecture note below)
   log_info ""
-  log_info "🔗 Checking GitHub issue auto-creation..."
+  log_info "ℹ️  Increment-level GitHub sync disabled (by design)"
+  log_debug "Increments are INTERNAL work units, not synced to external tools"
+  log_debug "External sync happens at SPEC level (.specweave/docs/internal/specs/)"
+  log_debug "See: .specweave/increments/0025-per-project-resource-config/reports/CORRECT-SYNC-ARCHITECTURE.md"
+
+  # TODO: Implement spec-level sync instead
+  # Architecture:
+  #   - SPECS (.specweave/docs/internal/specs/) → GitHub Projects/JIRA Epics/ADO Features
+  #   - User Stories → GitHub Issues
+  #   - Increments → INTERNAL ONLY (no external sync)
+  #
+  # Commands to implement:
+  #   - /specweave-github:sync-spec spec-001
+  #   - /specweave-jira:sync-spec spec-001
+  #   - /specweave-ado:sync-spec spec-001
+  #
+  # See: plugins/specweave-github/commands/specweave-github-sync-spec.md
+
+  # COMMENTED OUT: Increment-level GitHub issue creation (architecturally wrong)
+  # # Check if auto-create is enabled in config
+  # local auto_create=$(cat "$CONFIG_FILE" 2>/dev/null | grep -A 5 '"sync"' | grep -A 2 '"settings"' | grep -o '"autoCreateIssue"[[:space:]]*:[[:space:]]*\(true\|false\)' | grep -o '\(true\|false\)' || echo "false")
+  #
+  # if [ "$auto_create" = "true" ]; then
+  #   ... (increment-level sync code removed)
+  # fi
+
+  # 8. Sync spec content to external tools (if configured)
+  log_info ""
+  log_info "🔗 Checking spec content sync..."
 
-  # Check if auto-create is enabled in config
-  local auto_create=$(cat "$CONFIG_FILE" 2>/dev/null | grep -A 5 '"sync"' | grep -A 2 '"settings"' | grep -o '"autoCreateIssue"[[:space:]]*:[[:space:]]*\(true\|false\)' | grep -o '\(true\|false\)' || echo "false")
+  # Check if sync is enabled in config
+  local sync_enabled=$(cat "$CONFIG_FILE" 2>/dev/null | grep -o '"enabled"[[:space:]]*:[[:space:]]*\(true\|false\)' | head -1 | grep -o '\(true\|false\)' || echo "false")
 
-  if [ "$auto_create" = "true" ]; then
-    log_info "  📦 Auto-create enabled, checking for GitHub CLI..."
+  if [ "$sync_enabled" = "true" ]; then
+    log_info "  📦 Sync enabled, syncing spec content to external tool..."
 
-    # Check if gh CLI is available
-    if command -v gh &> /dev/null; then
-      log_info "  ✓ GitHub CLI found"
+    # Find the helper script
+    local sync_script="${CLAUDE_PLUGIN_ROOT}/hooks/lib/sync-spec-content.sh"
 
-      # Check if metadata.json already has GitHub issue
-      local metadata_file="$increment_dir/metadata.json"
-      local existing_issue=""
+    # Fallback to relative path if CLAUDE_PLUGIN_ROOT not set
+    if [ -z "$CLAUDE_PLUGIN_ROOT" ]; then
+      local hook_dir="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+      sync_script="$hook_dir/lib/sync-spec-content.sh"
+    fi
 
-      if [ -f "$metadata_file" ]; then
-        existing_issue=$(cat "$metadata_file" 2>/dev/null | grep -o '"issue"[[:space:]]*:[[:space:]]*[0-9]*' | grep -o '[0-9]*' || echo "")
-      fi
+    if [ -f "$sync_script" ] && [ -x "$sync_script" ]; then
+      log_debug "Calling sync-spec-content.sh for $spec_file"
 
-      if [ -n "$existing_issue" ]; then
-        log_info "  ℹ️  GitHub issue already exists: #$existing_issue"
+      # Call the sync script (non-blocking)
+      if "$sync_script" "$spec_file" 2>&1 | while read -r line; do echo "  $line"; done; then
+        log_info "  ✅ Spec content sync completed successfully!"
       else
-        log_info "  🚀 Creating GitHub issue for $increment_id..."
-
-        # Call the create_github_issue function (defined below)
-        if create_github_issue "$increment_id" "$increment_dir"; then
-          log_info "  ✅ GitHub issue created successfully!"
-        else
-          log_info "  ⚠️  Failed to create GitHub issue (non-blocking)"
-          log_debug "Issue creation failed, continuing..."
-        fi
+        log_info "  ⚠️  Spec content sync failed (non-blocking)"
+        log_debug "Spec sync failed, continuing..."
       fi
     else
-      log_debug "GitHub CLI not found, skipping issue creation"
+      log_debug "Sync script not found or not executable: $sync_script"
     fi
   else
-    log_debug "GitHub issue auto-create disabled in config"
+    log_debug "Spec content sync disabled in config"
   fi
 
-  # 8. Final summary
+  # 9. Final summary
   log_info ""
   local total_translated=$((increment_success_count))
 

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

@@ -102,10 +102,23 @@ EOF
       if [[ -n "$INCOMPLETE_INCREMENTS" ]]; then
         COUNT=$(echo "$INCOMPLETE_INCREMENTS" | wc -l | xargs)
 
+        # Get incomplete task count for migration guidance
+        MIGRATION_SCRIPT="$(dirname "${BASH_SOURCE[0]}")/lib/migrate-increment-work.sh"
+        INCOMPLETE_TASKS=""
+
+        for increment in $INCOMPLETE_INCREMENTS; do
+          if [[ -x "$MIGRATION_SCRIPT" ]]; then
+            TASK_COUNT=$("$MIGRATION_SCRIPT" count-incomplete "$increment" 2>/dev/null || echo "?")
+            INCOMPLETE_TASKS="${INCOMPLETE_TASKS}\n  - $increment ($TASK_COUNT incomplete tasks)"
+          else
+            INCOMPLETE_TASKS="${INCOMPLETE_TASKS}\n  - $increment"
+          fi
+        done
+
         cat <<EOF
 {
   "decision": "block",
-  "reason": "❌ Cannot create new increment! You have $COUNT incomplete increment(s):\n\n$(echo "$INCOMPLETE_INCREMENTS" | sed 's/^/  - /')\n\n💡 Complete or close them first:\n  - /specweave:done <id>     # Mark as complete\n  - /specweave:pause <id>    # Pause for later\n  - /specweave:abandon <id>  # Abandon if obsolete\n\nℹ️ The discipline exists for a reason:\n  ✓ Prevents scope creep\n  ✓ Ensures completions are tracked\n  ✓ Maintains living docs accuracy\n  ✓ Keeps work focused"
+  "reason": "❌ Cannot create new increment! You have $COUNT incomplete increment(s):$INCOMPLETE_TASKS\n\n💡 **SMART MIGRATION OPTIONS:**\n\n1️⃣  **Transfer Work** (Recommended)\n   Move incomplete tasks to new increment:\n   \`\`\`bash\n   # After creating new increment, run:\n   bash plugins/specweave/hooks/lib/migrate-increment-work.sh transfer <old-id> <new-id>\n   \`\`\`\n   ✅ Clean closure + work continues\n\n2️⃣  **Adjust WIP Limit** (Emergency Only)\n   Temporarily allow 3 active increments:\n   \`\`\`bash\n   bash plugins/specweave/hooks/lib/migrate-increment-work.sh adjust-wip 3\n   \`\`\`\n   ⚠️  20% productivity cost, revert ASAP\n\n3️⃣  **Force-Close** (Quick Fix)\n   Mark increment as complete (work lost):\n   \`\`\`bash\n   bash plugins/specweave/hooks/lib/migrate-increment-work.sh force-close <increment-id>\n   \`\`\`\n   ⚠️  Incomplete work NOT transferred!\n\n📝 **Traditional Options:**\n  - /specweave:done <id>     # Complete properly\n  - /specweave:pause <id>    # Pause for later\n  - /specweave:abandon <id>  # Abandon if obsolete\n\nℹ️ The discipline exists for a reason:\n  ✓ Prevents scope creep\n  ✓ Ensures completions are tracked\n  ✓ Maintains living docs accuracy\n  ✓ Keeps work focused"
 }
 EOF
         exit 0
@@ -114,6 +127,76 @@ EOF
   fi
 fi
 
+# ==============================================================================
+# PRE-FLIGHT SYNC CHECK: Ensure living docs are fresh before operations
+# ==============================================================================
+
+# Detect increment operations that need fresh data
+if echo "$PROMPT" | grep -qE "/(specweave:)?(done|validate|progress|do)"; then
+  # Extract increment ID from prompt (if provided)
+  INCREMENT_ID=$(echo "$PROMPT" | grep -oE "[0-9]{4}[a-z0-9-]*" | head -1)
+
+  # If no ID in prompt, try to find active increment
+  if [[ -z "$INCREMENT_ID" ]] && [[ -d ".specweave/increments" ]]; then
+    INCREMENT_ID=$(find .specweave/increments -mindepth 1 -maxdepth 1 -type d | while read increment_dir; do
+      metadata="$increment_dir/metadata.json"
+      if [[ -f "$metadata" ]]; then
+        status=$(node -e "
+          try {
+            const data = JSON.parse(require('fs').readFileSync('$metadata', 'utf-8'));
+            if (data.status === 'active') {
+              console.log('$(basename "$increment_dir")');
+            }
+          } catch (e) {}
+        " 2>/dev/null)
+
+        if [[ -n "$status" ]]; then
+          echo "$status"
+          break
+        fi
+      fi
+    done)
+  fi
+
+  # If we have an increment ID, check freshness
+  if [[ -n "$INCREMENT_ID" ]]; then
+    INCREMENT_SPEC=".specweave/increments/$INCREMENT_ID/spec.md"
+    LIVING_DOCS_SPEC=".specweave/docs/internal/specs/spec-$INCREMENT_ID.md"
+
+    # Check if increment spec exists
+    if [[ -f "$INCREMENT_SPEC" ]]; then
+      # Get modification times
+      if [[ "$(uname)" == "Darwin" ]]; then
+        # macOS
+        INCREMENT_MTIME=$(stat -f %m "$INCREMENT_SPEC" 2>/dev/null || echo 0)
+        LIVING_DOCS_MTIME=$(stat -f %m "$LIVING_DOCS_SPEC" 2>/dev/null || echo 0)
+      else
+        # Linux
+        INCREMENT_MTIME=$(stat -c %Y "$INCREMENT_SPEC" 2>/dev/null || echo 0)
+        LIVING_DOCS_MTIME=$(stat -c %Y "$LIVING_DOCS_SPEC" 2>/dev/null || echo 0)
+      fi
+
+      # Check if increment is newer than living docs (or living docs doesn't exist)
+      if [[ "$INCREMENT_MTIME" -gt "$LIVING_DOCS_MTIME" ]]; then
+        # Sync needed - run sync-living-docs
+        PLUGIN_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+        SYNC_SCRIPT="$PLUGIN_ROOT/lib/hooks/sync-living-docs.js"
+
+        if [[ -f "$SYNC_SCRIPT" ]]; then
+          # Run sync (capture output but don't block on errors)
+          if node "$SYNC_SCRIPT" "$INCREMENT_ID" >/dev/null 2>&1; then
+            # Success - sync completed
+            :
+          else
+            # Sync failed - log but continue
+            echo "[WARNING] Pre-flight sync failed for $INCREMENT_ID" >&2
+          fi
+        fi
+      fi
+    fi
+  fi
+fi
+
 # ==============================================================================
 # CONTEXT INJECTION: Add current increment status
 # ==============================================================================
@@ -142,20 +225,44 @@ if [[ -d ".specweave/increments" ]]; then
   done)
 
   if [[ -n "$ACTIVE_INCREMENT" ]]; then
-    # Get task completion percentage
-    TASKS_FILE=".specweave/increments/$ACTIVE_INCREMENT/tasks.md"
-    if [[ -f "$TASKS_FILE" ]]; then
-      TASK_STATS=$(grep -E "^\s*-\s*\[[ x]\]" "$TASKS_FILE" 2>/dev/null | wc -l | xargs || echo "0")
-      COMPLETED_TASKS=$(grep -E "^\s*-\s*\[x\]" "$TASKS_FILE" 2>/dev/null | wc -l | xargs || echo "0")
-
-      if [[ "$TASK_STATS" -gt 0 ]]; then
-        PERCENTAGE=$(( COMPLETED_TASKS * 100 / TASK_STATS ))
-        CONTEXT="📍 Active Increment: $ACTIVE_INCREMENT ($PERCENTAGE% complete, $COMPLETED_TASKS/$TASK_STATS tasks)"
+    # Use status line cache for consistent display
+    STATUS_LINE_CACHE=".specweave/state/status-line.json"
+    if [[ -f "$STATUS_LINE_CACHE" ]]; then
+      # Get status line from cache (ultra-fast)
+      STATUS_LINE=$(node -e "
+        try {
+          const { StatusLineManager } = require('./dist/core/status-line/status-line-manager.js');
+          const manager = new StatusLineManager(process.cwd());
+          const result = manager.render();
+          console.log(result || '');
+        } catch (e) {
+          // Fallback: show increment name only
+          console.log('');
+        }
+      " 2>/dev/null || echo "")
+
+      if [[ -n "$STATUS_LINE" ]]; then
+        CONTEXT="✓ $STATUS_LINE"
       else
-        CONTEXT="📍 Active Increment: $ACTIVE_INCREMENT"
+        # Fallback: parse tasks.md manually
+        TASKS_FILE=".specweave/increments/$ACTIVE_INCREMENT/tasks.md"
+        if [[ -f "$TASKS_FILE" ]]; then
+          TASK_STATS=$(grep -E "^\s*-\s*\[[ x]\]" "$TASKS_FILE" 2>/dev/null | wc -l | xargs || echo "0")
+          COMPLETED_TASKS=$(grep -E "^\s*-\s*\[x\]" "$TASKS_FILE" 2>/dev/null | wc -l | xargs || echo "0")
+
+          if [[ "$TASK_STATS" -gt 0 ]]; then
+            PERCENTAGE=$(( COMPLETED_TASKS * 100 / TASK_STATS ))
+            CONTEXT="✓ Active Increment: $ACTIVE_INCREMENT ($PERCENTAGE% complete, $COMPLETED_TASKS/$TASK_STATS tasks)"
+          else
+            CONTEXT="✓ Active Increment: $ACTIVE_INCREMENT"
+          fi
+        else
+          CONTEXT="✓ Active Increment: $ACTIVE_INCREMENT"
+        fi
       fi
     else
-      CONTEXT="📍 Active Increment: $ACTIVE_INCREMENT"
+      # No cache, fall back to increment name only
+      CONTEXT="✓ Active Increment: $ACTIVE_INCREMENT"
     fi
   fi
 fi