specweave 1.0.174 → 1.0.175

package/CLAUDE.md

@@ -1,4 +1,4 @@
-<!-- SW:META template="claude" version="1.0.172" sections="header,start,autodetect,metarule,rules,workflow,reflect,context,structure,taskformat,secrets,syncing,testing,tdd,api,limits,troubleshooting,lazyloading,principles,linking,mcp,auto,docs" -->
+<!-- SW:META template="claude" version="1.0.174" sections="header,start,autodetect,metarule,rules,workflow,reflect,context,structure,taskformat,secrets,syncing,testing,tdd,api,limits,troubleshooting,lazyloading,principles,linking,mcp,auto,docs" -->
 
 <!-- SW:SECTION:hook-priority version="1.0.171" -->
 ## ⛔ ABSOLUTE PRIORITY: Hook Instructions Are Mandatory
@@ -50,7 +50,7 @@ Hooks exist to enforce workflow discipline. If you ignore them:
 **This is non-negotiable. No exceptions. No "just this once".**
 <!-- SW:END:hook-priority -->
 
-<!-- SW:SECTION:header version="1.0.172" -->
+<!-- SW:SECTION:header version="1.0.174" -->
 **Framework**: SpecWeave | **Truth**: `spec.md` + `tasks.md`
 <!-- SW:END:header -->
 
@@ -108,7 +108,7 @@ plugins/specweave/
 **Old "commands" are just skills with `disable-model-invocation: true`** - they only respond to explicit `/name` invocation, not keyword detection.
 <!-- SW:END:claude-code-concepts -->
 
-<!-- SW:SECTION:start version="1.0.172" -->
+<!-- SW:SECTION:start version="1.0.174" -->
 ## Getting Started
 
 **Initial increment**: `0001-project-setup` (auto-created by `specweave init`)
@@ -118,7 +118,7 @@ plugins/specweave/
 2. **Customize**: Edit spec.md and use for setup tasks
 <!-- SW:END:start -->
 
-<!-- SW:SECTION:autodetect version="1.0.172" -->
+<!-- SW:SECTION:autodetect version="1.0.174" -->
 ## Auto-Detection
 
 SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
@@ -128,7 +128,7 @@ SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
 **Opt-out phrases**: "Just brainstorm first" | "Don't plan yet" | "Quick discussion" | "Let's explore ideas"
 <!-- SW:END:autodetect -->
 
-<!-- SW:SECTION:metarule version="1.0.172" -->
+<!-- SW:SECTION:metarule version="1.0.174" -->
 ## Meta-Rule: Think-Before-Act
 
 **Satisfy dependencies BEFORE dependent operations.**
@@ -139,7 +139,7 @@ SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
 ```
 <!-- SW:END:metarule -->
 
-<!-- SW:SECTION:rules version="1.0.172" -->
+<!-- SW:SECTION:rules version="1.0.174" -->
 ## Rules
 
 1. **Files** → `.specweave/increments/####-name/` (see Structure section for details)
@@ -150,7 +150,7 @@ SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
 6. **⛔ Marketplace refresh**: Use `specweave refresh-marketplace` CLI (not `scripts/refresh-marketplace.sh`)
 <!-- SW:END:rules -->
 
-<!-- SW:SECTION:workflow version="1.0.172" -->
+<!-- SW:SECTION:workflow version="1.0.174" -->
 ## Workflow
 
 `/sw:increment "X"` → `/sw:do` → `/sw:progress` → `/sw:done 0001`
@@ -170,7 +170,7 @@ SpecWeave auto-detects product descriptions and routes to `/sw:increment`:
 **Natural language**: "Let's build X" → `/sw:increment` | "What's status?" → `/sw:progress` | "We're done" → `/sw:done` | "Ship while sleeping" → `/sw:auto`
 <!-- SW:END:workflow -->
 
-<!-- SW:SECTION:reflect version="1.0.172" -->
+<!-- SW:SECTION:reflect version="1.0.174" -->
 ## Skill Memories
 
 SpecWeave learns from corrections. Learnings saved here automatically. Edit or delete as needed.
@@ -183,7 +183,7 @@ SpecWeave learns from corrections. Learnings saved here automatically. Edit or d
 <!-- Auto-captured by SpecWeave reflect. Edit or delete as needed. -->
 <!-- Learnings are organized by skill name. User edits override SpecWeave defaults. -->
 
-<!-- SW:SECTION:context version="1.0.172" -->
+<!-- SW:SECTION:context version="1.0.174" -->
 ## Context
 
 **Before implementing**: Check ADRs at `.specweave/docs/internal/architecture/adr/`
@@ -191,7 +191,7 @@ SpecWeave learns from corrections. Learnings saved here automatically. Edit or d
 **Load context**: `/sw:context <topic>` loads relevant living docs into conversation
 <!-- SW:END:context -->
 
-<!-- SW:SECTION:structure version="1.0.172" -->
+<!-- SW:SECTION:structure version="1.0.174" -->
 ## Structure
 
 ```
@@ -206,7 +206,7 @@ SpecWeave learns from corrections. Learnings saved here automatically. Edit or d
 **Everything else → subfolders**: `reports/` | `logs/` | `scripts/` | `backups/`
 <!-- SW:END:structure -->
 
-<!-- SW:SECTION:taskformat version="1.0.172" -->
+<!-- SW:SECTION:taskformat version="1.0.174" -->
 ## Task Format
 
 ```markdown
@@ -216,7 +216,7 @@ SpecWeave learns from corrections. Learnings saved here automatically. Edit or d
 ```
 <!-- SW:END:taskformat -->
 
-<!-- SW:SECTION:secrets version="1.0.172" -->
+<!-- SW:SECTION:secrets version="1.0.174" -->
 ## Secrets Check
 
 **BEFORE CLI tools**: Check existing config first!
@@ -230,7 +230,7 @@ gh auth status
 **SECURITY**: NEVER use `grep TOKEN .env` without `-q` flag - it exposes credentials in terminal!
 <!-- SW:END:secrets -->
 
-<!-- SW:SECTION:syncing version="1.0.172" -->
+<!-- SW:SECTION:syncing version="1.0.174" -->
 ## External Sync (GitHub/JIRA/ADO)
 
 **Commands**: `/sw-github:sync {id}` (issues) | `/sw:sync-specs` (living docs only)
@@ -240,7 +240,7 @@ gh auth status
 **Config**: Set `sync.github.enabled: true` + `canUpdateExternalItems: true` in config.json
 <!-- SW:END:syncing -->
 
-<!-- SW:SECTION:testing version="1.0.172" -->
+<!-- SW:SECTION:testing version="1.0.174" -->
 ## Testing
 
 BDD in tasks.md | Unit >80% | `.test.ts` (Vitest)
@@ -252,7 +252,7 @@ vi.mock('./module', () => ({ func: mockFn }));
 ```
 <!-- SW:END:testing -->
 
-<!-- SW:SECTION:tdd version="1.0.172" -->
+<!-- SW:SECTION:tdd version="1.0.174" -->
 ## TDD Mode (Test-Driven Development)
 
 **When `testing.defaultTestMode: "TDD"` is configured**, follow RED-GREEN-REFACTOR discipline:
@@ -313,7 +313,7 @@ When TDD is enabled, tasks include phase markers:
 **Rule**: Complete dependencies BEFORE dependent tasks (RED before GREEN).
 <!-- SW:END:tdd -->
 
-<!-- SW:SECTION:api version="1.0.172" -->
+<!-- SW:SECTION:api version="1.0.174" -->
 ## API Development (OpenAPI-First)
 
 **For API projects only.** Commands: `/sw:api-docs --all` | `--openapi` | `--postman` | `--validate`
@@ -321,13 +321,13 @@ When TDD is enabled, tasks include phase markers:
 Enable in config: `{"apiDocs":{"enabled":true,"openApiPath":"openapi.yaml"}}`
 <!-- SW:END:api -->
 
-<!-- SW:SECTION:limits version="1.0.172" -->
+<!-- SW:SECTION:limits version="1.0.174" -->
 ## Limits
 
 **Max 1500 lines/file** — extract before adding
 <!-- SW:END:limits -->
 
-<!-- SW:SECTION:troubleshooting version="1.0.172" -->
+<!-- SW:SECTION:troubleshooting version="1.0.174" -->
 ## Troubleshooting
 
 | Issue | Fix |
@@ -343,7 +343,7 @@ Enable in config: `{"apiDocs":{"enabled":true,"openApiPath":"openapi.yaml"}}`
 | Marketplace shows 0 | Normal with auto-load; `/plugin list` shows actual |
 <!-- SW:END:troubleshooting -->
 
-<!-- SW:SECTION:lazyloading version="1.0.172" -->
+<!-- SW:SECTION:lazyloading version="1.0.174" -->
 ## Plugin Auto-Loading
 
 Plugins load automatically based on project type and keywords. Manual install if needed:
@@ -357,7 +357,7 @@ export SPECWEAVE_DISABLE_AUTO_LOAD=1         # Disable auto-load
 **Token savings**: Core ~3-5K tokens vs all plugins ~60K+
 <!-- SW:END:lazyloading -->
 
-<!-- SW:SECTION:principles version="1.0.172" -->
+<!-- SW:SECTION:principles version="1.0.174" -->
 ## Principles
 
 1. **Spec-first**: `/sw:increment` before coding
@@ -366,7 +366,7 @@ export SPECWEAVE_DISABLE_AUTO_LOAD=1         # Disable auto-load
 4. **Traceable**: All work → specs → ACs
 <!-- SW:END:principles -->
 
-<!-- SW:SECTION:linking version="1.0.172" -->
+<!-- SW:SECTION:linking version="1.0.174" -->
 ## Bidirectional Linking
 
 Tasks ↔ User Stories auto-linked via AC-IDs: `AC-US1-01` → `US-001`
@@ -374,7 +374,7 @@ Tasks ↔ User Stories auto-linked via AC-IDs: `AC-US1-01` → `US-001`
 Task format: `**AC**: AC-US1-01, AC-US1-02` (CRITICAL for linking)
 <!-- SW:END:linking -->
 
-<!-- SW:SECTION:mcp version="1.0.172" -->
+<!-- SW:SECTION:mcp version="1.0.174" -->
 ## External Services
 
 **Priority**: CLI tools first (simpler) → MCP for complex integrations
@@ -396,7 +396,7 @@ claude mcp add --transport stdio postgres -- npx -y @modelcontextprotocol/server
 MCP supports lazy-loading (auto mode) - tools load on-demand when >10% context.
 <!-- SW:END:mcp -->
 
-<!-- SW:SECTION:auto version="1.0.172" -->
+<!-- SW:SECTION:auto version="1.0.174" -->
 ## Auto Mode
 
 **Commands**: `/sw:auto` (start) | `/sw:auto-status` (check) | `/sw:cancel-auto` (emergency only)
@@ -413,7 +413,7 @@ MCP supports lazy-loading (auto mode) - tools load on-demand when >10% context.
 **STOP & ASK** if: Spec conflicts | Task unnecessary | Requirement ambiguous
 <!-- SW:END:auto -->
 
-<!-- SW:SECTION:docs version="1.0.172" -->
+<!-- SW:SECTION:docs version="1.0.174" -->
 ## Docs
 
 [spec-weave.com](https://spec-weave.com)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specweave",
-  "version": "1.0.174",
+  "version": "1.0.175",
   "description": "Spec-driven development framework for Claude Code. AI-native workflow with living documentation, intelligent agents, and multilingual support (9 languages). Enterprise-grade traceability with permanent specs and temporary increments.",
   "type": "module",
   "main": "dist/index.js",

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

@@ -1,20 +1,28 @@
 #!/bin/bash
 
-# SpecWeave UserPromptSubmit Hook (v1.0.169 - Direct Skill Invocation)
+# SpecWeave UserPromptSubmit Hook (v1.0.175 - Reliable Plugin Detection)
 # Fires BEFORE user's command executes (prompt-based hook)
 # Purpose: Auto-load plugins, discipline validation, context injection, instant command execution
 #
 # FEATURES:
+# - v1.0.175: CRITICAL FIX - Use installed_plugins.json as SOURCE OF TRUTH
+#   * Reads ~/.claude/plugins/installed_plugins.json directly (eliminates false restart warnings)
+#   * `claude plugin list` can have timing/buffering issues → unreliable for detection
+#   * Primary: check_plugin_installed_from_json() using jq (fast, accurate)
+#   * Fallback: `claude plugin list` only if jq unavailable
+#   * Post-install verification: re-checks registry after install to confirm success
+#   * Increased timeouts: 5s → 10s for CLI operations (reduces timing issues)
+#   * Guard against false positives: if install says "success" but not in registry → treat as already installed
 # - v1.0.169: DIRECT SKILL INVOCATION - Call sw:increment-planner directly (not wrapper)
 #   * Skips 2-level indirection (hook → sw:increment command → sw:increment-planner skill)
 #   * Passes FULL user prompt as args (not just extracted name)
 #   * Uses <system><rules> tags (Claude-trained) instead of custom <mandatory_instruction>
 #   * More concise, imperative instruction text
-# - v1.0.167: FIX PLUGIN RESTART WARNING - Use `claude plugin list` BEFORE install
+# - v1.0.167: FIX PLUGIN RESTART WARNING - Use `claude plugin list` BEFORE install (DEPRECATED - had timing issues)
 #   * Claude CLI always outputs "Successfully installed" even when already installed
-#   * Now calls `claude plugin list` once to get current plugins, then checks against that
-#   * Skips install for already-installed plugins (faster + no false restart warnings)
-#   * Only shows restart warning for TRULY new plugin installs
+#   * Called `claude plugin list` once to get current plugins, then checked against that
+#   * Skipped install for already-installed plugins (faster + no false restart warnings)
+#   * ISSUE: `claude plugin list` output unreliable → false positives → fixed in v1.0.175
 # - v1.0.147: SYNC PLUGIN INSTALL - Plugins available for CURRENT prompt!
 #   * Replaced 20s async LLM detection with ~200ms sync `claude plugin install`
 #   * Claude Code hot-reload picks up plugins immediately
@@ -138,6 +146,36 @@ output_approve_with_context() {
   printf '{"hookSpecificOutput":{"hookEventName":"UserPromptSubmit","additionalContext":"%s"}}\n' "$escaped"
 }
 
+# Helper: Check if plugin is installed by reading installed_plugins.json (v1.0.175)
+# This is the SOURCE OF TRUTH - more reliable than `claude plugin list` which can have timing issues.
+# Args: $1=plugin name (e.g., "sw-frontend"), $2=marketplace (e.g., "specweave")
+# Returns: 0 if installed, 1 if not installed
+check_plugin_installed_from_json() {
+  local plugin="$1"
+  local marketplace="$2"
+  local registry_path="${HOME}/.claude/plugins/installed_plugins.json"
+
+  # File must exist
+  [[ ! -f "$registry_path" ]] && return 1
+
+  # Must have jq for reliable JSON parsing
+  if ! command -v jq >/dev/null 2>&1; then
+    return 1  # Fallback to CLI check if jq not available
+  fi
+
+  # Check if plugin exists in registry
+  # Format: {"plugins": {"sw-frontend@specweave": [...], ...}}
+  local full_name="${plugin}@${marketplace}"
+  local has_plugin
+  has_plugin=$(jq -r --arg key "$full_name" '.plugins[$key] // null' "$registry_path" 2>/dev/null)
+
+  if [[ "$has_plugin" != "null" ]] && [[ -n "$has_plugin" ]]; then
+    return 0  # Installed
+  else
+    return 1  # Not installed
+  fi
+}
+
 # ==============================================================================
 # KEYWORD-BASED PLUGIN DETECTION REMOVED (v1.0.159)
 # ==============================================================================
@@ -291,16 +329,6 @@ if [[ "${SPECWEAVE_DISABLE_AUTO_LOAD:-0}" != "1" ]] && [[ "${SPECWEAVE_DISABLE_H
                   PLUGINS_INSTALLED=""
                   PLUGINS_ALREADY=""
 
-                  # v1.0.167: Get list of already-installed plugins ONCE via `claude plugin list`
-                  # This is more reliable than parsing internal JSON files
-                  # Format: "sw-frontend@specweave", "context7@claude-plugins-official", etc.
-                  CURRENT_PLUGINS=""
-                  if command -v timeout >/dev/null 2>&1; then
-                    CURRENT_PLUGINS=$(timeout 5 claude plugin list 2>/dev/null | grep -E "^  ❯ " | sed 's/^  ❯ //' || true)
-                  else
-                    CURRENT_PLUGINS=$(claude plugin list 2>/dev/null | grep -E "^  ❯ " | sed 's/^  ❯ //' || true)
-                  fi
-
                   for plugin in $DETECTED_PLUGINS; do
                     [[ -z "$plugin" ]] && continue
 
@@ -312,11 +340,28 @@ if [[ "${SPECWEAVE_DISABLE_AUTO_LOAD:-0}" != "1" ]] && [[ "${SPECWEAVE_DISABLE_H
                       MARKETPLACE="claude-plugins-official"
                     fi
 
-                    # v1.0.167: Check if plugin is ALREADY installed (from cached list)
+                    # v1.0.175: Check if plugin is ALREADY installed (SOURCE OF TRUTH)
+                    # Primary: Check installed_plugins.json (reliable, fast, no timing issues)
+                    # Fallback: Check `claude plugin list` if jq not available
                     FULL_PLUGIN_NAME="${plugin}@${MARKETPLACE}"
                     ALREADY_INSTALLED=false
-                    if echo "$CURRENT_PLUGINS" | grep -q "^${FULL_PLUGIN_NAME}$"; then
+
+                    # Try JSON registry first (most reliable)
+                    if check_plugin_installed_from_json "$plugin" "$MARKETPLACE"; then
                       ALREADY_INSTALLED=true
+                    else
+                      # Fallback to CLI check (if jq not available or registry file missing)
+                      # Give it a longer timeout (10s) to reduce timing issues
+                      CURRENT_PLUGINS=""
+                      if command -v timeout >/dev/null 2>&1; then
+                        CURRENT_PLUGINS=$(timeout 10 claude plugin list 2>/dev/null | grep -E "^  ❯ " | sed 's/^  ❯ //' || true)
+                      else
+                        CURRENT_PLUGINS=$(claude plugin list 2>/dev/null | grep -E "^  ❯ " | sed 's/^  ❯ //' || true)
+                      fi
+
+                      if echo "$CURRENT_PLUGINS" | grep -q "^${FULL_PLUGIN_NAME}$"; then
+                        ALREADY_INSTALLED=true
+                      fi
                     fi
 
                     if [[ "$ALREADY_INSTALLED" == "true" ]]; then
@@ -325,15 +370,26 @@ if [[ "${SPECWEAVE_DISABLE_AUTO_LOAD:-0}" != "1" ]] && [[ "${SPECWEAVE_DISABLE_H
                       PLUGINS_ALREADY="${PLUGINS_ALREADY}${plugin}"
                     else
                       # Plugin not installed - install it
+                      # Use longer timeout (10s) to ensure installation completes
                       if command -v timeout >/dev/null 2>&1; then
-                        OUT=$(timeout 5 claude plugin install "${FULL_PLUGIN_NAME}" 2>&1) || true
+                        OUT=$(timeout 10 claude plugin install "${FULL_PLUGIN_NAME}" 2>&1) || true
                       else
                         OUT=$(claude plugin install "${FULL_PLUGIN_NAME}" 2>&1) || true
                       fi
 
-                      if echo "$OUT" | grep -qi "success"; then
-                        [[ -n "$PLUGINS_INSTALLED" ]] && PLUGINS_INSTALLED="$PLUGINS_INSTALLED, "
-                        PLUGINS_INSTALLED="${PLUGINS_INSTALLED}${plugin}"
+                      # Only mark as installed if we see "success" or "installed" in output
+                      if echo "$OUT" | grep -qiE "(success|installed)"; then
+                        # Double-check: verify it actually got installed
+                        # Re-check the registry to confirm (guard against false positives)
+                        sleep 0.5  # Brief delay for registry to update
+                        if check_plugin_installed_from_json "$plugin" "$MARKETPLACE"; then
+                          [[ -n "$PLUGINS_INSTALLED" ]] && PLUGINS_INSTALLED="$PLUGINS_INSTALLED, "
+                          PLUGINS_INSTALLED="${PLUGINS_INSTALLED}${plugin}"
+                        else
+                          # Install claimed success but plugin not in registry - treat as already installed
+                          [[ -n "$PLUGINS_ALREADY" ]] && PLUGINS_ALREADY="$PLUGINS_ALREADY, "
+                          PLUGINS_ALREADY="${PLUGINS_ALREADY}${plugin}"
+                        fi
                       fi
                     fi
                   done