projecta-rrr 1.18.7 → 1.18.8

package/CHANGELOG.md

@@ -4,6 +4,25 @@ All notable changes to RRR will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [1.18.8] - 2026-02-03
+
+### Fixed
+
+- **HUD scripts actually removed** - v1.18.7 said they were removed but files still existed
+  - Deleted `scripts/rrr-hud.js`, `scripts/rrr-hud-dynamic.js`, and `hooks/rrr-sync-hud-state.sh`
+  - These were stated as removed in v1.18.7 but only removed from being called, not physically deleted
+- **Intent gate now works without HUD** - Reads directly from `.planning/STATE.md` instead of `hud-state.json`
+  - Fixes drift detection after HUD removal
+  - No longer depends on removed `rrr-sync-hud-state.sh` sync script
+- **Version mismatch false positives** - Simplified version check in `/rrr:new-milestone`
+  - No longer compares package.json version (npm release) with milestone version (feature delivery)
+  - These are different things and don't need to match (0.1.0 working on v1.0 is valid)
+  - Only checks: does CHANGELOG match package.json?
+
+### Changed
+
+- **OhMyPosh docs deprecated** - Added warnings that automated sync no longer works
+
 ## [1.18.7] - 2026-02-03
 
 ### Removed

package/commands/rrr/new-milestone.md

@@ -108,83 +108,56 @@ Milestone name: $ARGUMENTS (optional - will prompt if not provided)
    **If no patches (PATCH_COUNT = 0 or file doesn't exist):**
    - Skip silently, continue to step 1.5
 
-1.5. **Version consistency check (MANDATORY):**
+1.5. **Version consistency check (SIMPLIFIED):**
 
-   Before writing any milestone files, verify version consistency.
+   Verify CHANGELOG matches package.json (if CHANGELOG exists).
 
    **Important context:**
-   - `package.json` is the CANONICAL source of truth for the current version
-   - `/rrr:check-version` compares against npm registry (different concern — checks if release is published)
-   - Documentation version drift is normal after updates — auto-fix is almost always the right choice
+   - `package.json` version = npm release version (e.g., 0.1.0)
+   - Milestone version = feature delivery version (e.g., v1.0)
+   - These are DIFFERENT and don't need to match!
+   - Only check: does CHANGELOG match package.json?
 
    ```bash
-   # Read canonical version from package.json
-   CANONICAL_VERSION=$(node -p "require('./package.json').version" 2>/dev/null)
-   echo "Canonical version (package.json): $CANONICAL_VERSION"
-
-   # Read CHANGELOG version
-   CHANGELOG_VERSION=$(grep -E "^## \[[0-9]" CHANGELOG.md 2>/dev/null | head -1 | sed 's/## \[\([^]]*\)\].*/\1/')
-   echo "CHANGELOG version: $CHANGELOG_VERSION"
-
-   # Read planning doc versions
-   ROADMAP_VERSION=$(grep -oE "v[0-9]+\.[0-9]+(\.[0-9]+)?" .planning/ROADMAP.md 2>/dev/null | head -1)
-   STATE_VERSION=$(grep -oE "v[0-9]+\.[0-9]+(\.[0-9]+)?" .planning/STATE.md 2>/dev/null | head -1)
+   # Read package.json version
+   PKG_VERSION=$(node -p "require('./package.json').version" 2>/dev/null || echo "")
+
+   # Read CHANGELOG version (first entry)
+   if [ -f "CHANGELOG.md" ]; then
+     CHANGELOG_VERSION=$(grep -E "^## \[[0-9]" CHANGELOG.md 2>/dev/null | head -1 | sed 's/## \[\([^]]*\)\].*/\1/')
+
+     if [ -n "$CHANGELOG_VERSION" ] && [ "$PKG_VERSION" != "$CHANGELOG_VERSION" ]; then
+       echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+       echo "VERSION MISMATCH DETECTED"
+       echo "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━"
+       echo "package.json: $PKG_VERSION"
+       echo "CHANGELOG.md: $CHANGELOG_VERSION"
+       echo ""
+       echo "CHANGELOG should match package.json for release tracking."
+       # Ask user if they want to fix this or continue
+     fi
+   else
+     echo "No CHANGELOG.md found - skipping version check"
+   fi
    ```
 
-   **Version mismatch detection:**
-
-   Extract major.minor from canonical version and compare:
-   - If CHANGELOG major.minor != canonical major.minor → MISMATCH
-   - If planning docs reference different milestone version → MISMATCH
-
    **If mismatch detected:**
 
-   ```
-   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-   VERSION SYNC NEEDED
-   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-
-   Canonical (package.json): {CANONICAL_VERSION}
-   CHANGELOG.md:             {CHANGELOG_VERSION}
-   Planning docs:            {ROADMAP_VERSION}
-
-   This is normal after updates. Auto-fix will sync docs to package.json.
-   ```
-
    Use AskUserQuestion:
    - header: "Version"
-   - question: "Documentation versions need sync. How would you like to proceed?"
+   - question: "CHANGELOG version doesn't match package.json. Fix this?"
    - options:
-     - "Auto-fix now (Recommended)" — Update docs to match package.json
-     - "Run /rrr:check-version" — Check npm registry status (different concern — won't fix doc drift)
-     - "Continue anyway" — Proceed with mismatched versions (documentation debt)
-     - "Abort" — Cancel milestone creation
-
-   **If "Auto-fix now (Recommended)":**
-   - Update CHANGELOG.md with [Unreleased] header if needed
-   - Update planning docs milestone references to match canonical
-   - Continue to step 2
-
-   **If "Run /rrr:check-version":**
-   ```
-   Note: /rrr:check-version compares package.json against the npm registry.
-   This is useful for checking publish status, but won't fix documentation drift.
-   You may still need to run "Auto-fix now" after.
+     - "Yes, update CHANGELOG (Recommended)" — Update CHANGELOG to match package.json
+     - "No, continue anyway" — Proceed with mismatch (can fix later with /rrr:check-version)
 
-   Run `/rrr:check-version` to check registry status, then return to `/rrr:new-milestone`
-   ```
-   Exit command.
-
-   **If "Continue anyway":**
-   ```
-   Continuing with version mismatch. This creates documentation debt.
-   Consider running `/rrr:new-milestone` again later with "Auto-fix now" to sync versions.
-   ```
-   Continue to step 2.
+   **If "Yes, update CHANGELOG":**
+   - Add/update CHANGELOG entry to match package.json version
+   - Continue to step 2
 
-   **If "Abort":** Exit command.
+   **If "No, continue anyway":**
+   - Continue to step 2 (user can fix later)
 
-   **If versions match:** Continue to step 2.
+   **If no mismatch:** Continue to step 2.
 
 2. **Gather milestone goals:**
 

package/docs/OHMPOSH-SETUP.md

@@ -1,5 +1,9 @@
 # OhMyPosh HUD Setup Guide
 
+> **⚠️ DEPRECATED**: As of v1.18.7, the automated HUD sync scripts have been removed.
+> This integration requires manual state updates via `rrr-state-manager.js`.
+> Consider using inline status updates instead (built into RRR v1.18.4+).
+
 This guide explains how to set up the RRR HUD using OhMyPosh with **Kushal theme** styling (recommended) or Atomic theme.
 
 ## Theme Options

package/docs/ohmyposh-config.md

@@ -1,5 +1,8 @@
 # OhMyPosh Integration for RRR HUD
 
+> **⚠️ DEPRECATED**: As of v1.18.7, automated HUD sync scripts have been removed.
+> OhMyPosh integration requires manual state updates. Use RRR's inline status display instead.
+
 Display RRR context (intent, drift status, session duration) in your terminal prompt using OhMyPosh.
 
 ## Prerequisites

package/hooks/rrr-intent-gate.sh

@@ -20,8 +20,8 @@ fi
 # SKIP CONDITIONS - Let these through without gate
 # ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-# Skip empty or very short messages (lowered from 15 to 8 to catch "fix a bug")
-[[ -z "$USER_MESSAGE" || ${#USER_MESSAGE} -lt 8 ]] && exit 0
+# Skip empty or very short messages (5 chars catches "add X", "fix Y", etc.)
+[[ -z "$USER_MESSAGE" || ${#USER_MESSAGE} -lt 5 ]] && exit 0
 
 # Skip slash commands (already intentional)
 [[ "$USER_MESSAGE" == /* ]] && exit 0
@@ -39,22 +39,44 @@ echo "$msg_lower" | grep -qE "^(what|why|how|where|when|which|can you explain|ex
 echo "$msg_lower" | grep -qE "\?$" && exit 0
 
 # ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-# CHECK STATE - Only gate if no active plan
+# CHECK STATE - Read directly from .planning/ files
 # ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-HUD_FILE="${HOME}/.claude/rrr/hud-state.json"
-[[ ! -f "$HUD_FILE" ]] && exit 0
-
-status=$(jq -r '.status // ""' "$HUD_FILE" 2>/dev/null)
-plan=$(jq -r '.plan // ""' "$HUD_FILE" 2>/dev/null)
-phase=$(jq -r '.phase // ""' "$HUD_FILE" 2>/dev/null)
-milestone=$(jq -r '.milestone // ""' "$HUD_FILE" 2>/dev/null)
+# Find RRR project root (look for .planning directory)
+RRR_PROJECT=""
+current_dir="$(pwd)"
+for i in 0 1 2 3 4 5; do
+    if [ -d ".planning" ]; then
+        RRR_PROJECT="$(pwd)"
+        break
+    fi
+    cd .. 2>/dev/null || break
+done
+cd "$current_dir" 2>/dev/null
+
+# Not in an RRR project - skip gate
+[[ -z "$RRR_PROJECT" ]] && exit 0
+
+# Check STATE.md for active work
+STATE_FILE="${RRR_PROJECT}/.planning/STATE.md"
+if [ -f "$STATE_FILE" ]; then
+    # Extract current position from STATE.md
+    milestone=$(grep -E "^Milestone:" "$STATE_FILE" 2>/dev/null | sed 's/Milestone:[[:space:]]*//' | tr -d '\r\n' || echo "")
+    phase=$(grep -E "^Phase:" "$STATE_FILE" 2>/dev/null | sed 's/Phase:[[:space:]]*//' | tr -d '\r\n' || echo "")
+    plan=$(grep -E "^Plan:" "$STATE_FILE" 2>/dev/null | sed 's/Plan:[[:space:]]*//' | tr -d '\r\n' || echo "")
+    status=$(grep -E "^Status:" "$STATE_FILE" 2>/dev/null | sed 's/Status:[[:space:]]*//' | tr -d '\r\n' || echo "")
+else
+    milestone=""
+    phase=""
+    plan=""
+    status=""
+fi
 
-# If we have an active plan, let through
-[[ -n "$plan" && "$plan" != "null" ]] && exit 0
+# If we have an active plan (not "—" or empty), let through
+[[ -n "$plan" && "$plan" != "—" && "$plan" != "null" ]] && exit 0
 
-# If status is active (working on something), let through
-[[ "$status" == "active" ]] && exit 0
+# If status contains "Executing", let through
+echo "$status" | grep -qi "executing" && exit 0
 
 # ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 # DETECT WORK REQUESTS - Only gate substantial work
@@ -67,18 +89,19 @@ if echo "$msg_lower" | grep -qiE "$work_patterns"; then
     echo ""
     echo "<MANDATORY-INTENT-GATE>"
     echo ""
-    echo "I detected a work request but you have no active plan."
+    echo "⚠️  DRIFT DETECTED: Work request without active plan"
     echo ""
-    echo "  Current: ${milestone:-no milestone} / ${phase:-no phase} / ${plan:-no plan}"
-    echo "  Status:  ${status:-unknown}"
+    echo "  Current: ${milestone:-no milestone} / ${phase:-no phase} / ${plan:-—}"
+    echo "  Status:  ${status:-Defining requirements}"
     echo ""
-    echo "Before I proceed, where should this work go?"
+    echo "Where should this work be tracked?"
     echo ""
-    echo "  1) /rrr:plan-phase   - Create a tracked phase for this work"
-    echo "  2) /rrr:add-todo     - Save to backlog for later"
-    echo "  3) Type 'adhoc'      - Do it now without tracking"
+    echo "  1. /rrr:plan-phase   - Create a tracked phase"
+    echo "  2. /rrr:add-todo     - Save to backlog"
+    echo "  3. Type 'adhoc'      - Quick fix (untracked)"
     echo ""
     echo "</MANDATORY-INTENT-GATE>"
+    echo ""
     exit 1  # BLOCK until user classifies
 fi
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "projecta-rrr",
-  "version": "1.18.7",
+  "version": "1.18.8",
   "description": "A meta-prompting, context engineering and spec-driven development system for Claude Code by Projecta.ai",
   "bin": {
     "projecta-rrr": "bin/install.js"

package/scripts/jarvis.sh

@@ -103,123 +103,6 @@ load_libraries() {
 
 load_libraries
 
-# ═══════════════════════════════════════════════════════════════════════════════
-# Phase Paths Library (supports both old and new structures)
-# ═══════════════════════════════════════════════════════════════════════════════
-
-PHASE_PATHS_LOADED=false
-
-load_phase_paths() {
-    local lib_paths=("$SCRIPT_DIR/../rrr/lib" "${HOME}/.claude/rrr/lib")
-
-    for lib_dir in "${lib_paths[@]}"; do
-        if [[ -d "$lib_dir" ]] && [[ -f "$lib_dir/phase-paths.md" ]]; then
-            # Source the phase-paths library functions
-            source /dev/stdin <<'PHASE_FUNCTIONS' 2>/dev/null || true
-# Get the base directory for phases (supports both structures)
-get_phase_base_dir() {
-    local planning_dir="${1:-.planning}"
-    local roadmap_file="$planning_dir/ROADMAP.md"
-
-    # Strategy 1: Look for ## Current Milestone: header
-    local active_milestone=""
-    if [ -f "$roadmap_file" ]; then
-        active_milestone=$(grep -E "^## Current Milestone:" "$roadmap_file" 2>/dev/null | \
-            grep -oE "v[0-9]+\.[0-9]+" | head -1)
-    fi
-
-    # Strategy 2: Look for :construction: marker
-    if [ -z "$active_milestone" ]; then
-        active_milestone=$(grep -E ":construction:" "$roadmap_file" 2>/dev/null | \
-            grep -oE "v[0-9]+\.[0-9]+" | head -1)
-    fi
-
-    # Use milestone phases dir if found
-    if [ -n "$active_milestone" ]; then
-        local phases_dir="$planning_dir/milestones/$active_milestone/phases"
-        echo "$phases_dir"
-        return 0
-    fi
-
-    # Legacy fallback
-    echo "$planning_dir/phases"
-    return 0
-}
-
-# Find a phase directory by number
-find_phase_dir() {
-    local phase_number="$1"
-    local planning_dir="${2:-.planning}"
-    local roadmap_file="$planning_dir/ROADMAP.md"
-
-    if [ -z "$phase_number" ]; then
-        return 1
-    fi
-
-    # Normalize phase number
-    local phase_pattern
-    if [[ "$phase_number" =~ \\. ]]; then
-        phase_pattern="${phase_number}-*"
-    else
-        local padded=$(printf "%02d" "$phase_number" 2>/dev/null || echo "$phase_number")
-        phase_pattern="${padded}-*"
-    fi
-
-    # Get active milestone
-    local active_milestone=""
-    if [ -f "$roadmap_file" ]; then
-        active_milestone=$(grep -E "Current Milestone|:construction:" "$roadmap_file" 2>/dev/null | head -1 | grep -oE "v[0-9]+\.[0-9]+" | head -1)
-    fi
-
-    # Search in current milestone
-    if [ -n "$active_milestone" ]; then
-        local milestone_phases="$planning_dir/milestones/$active_milestone/phases"
-        if [ -d "$milestone_phases" ]; then
-            local found
-            found=$(find "$milestone_phases" -maxdepth 1 -type d -name "$phase_pattern" 2>/dev/null | head -1)
-            if [ -n "$found" ] && [ -d "$found" ]; then
-                echo "$found"
-                return 0
-            fi
-        fi
-    fi
-
-    # Fallback to old structure
-    if [ -d "$planning_dir/phases" ]; then
-        local found
-        found=$(find "$planning_dir/phases" -maxdepth 1 -type d -name "$phase_pattern" 2>/dev/null | head -1)
-        if [ -n "$found" ] && [ -d "$found" ]; then
-            echo "$found"
-            return 0
-        fi
-    fi
-
-    return 1
-}
-PHASE_FUNCTIONS
-            PHASE_PATHS_LOADED=true
-            break
-        fi
-    done
-}
-
-load_phase_paths
-
-# Resolve PHASES_DIR dynamically (support both old and new structures)
-if [ "$PHASE_PATHS_LOADED" = true ]; then
-    PHASES_DIR=$(get_phase_base_dir "$PLANNING_DIR")
-else
-    # Fallback: check if new structure exists
-    if [ -d "$PLANNING_DIR/milestones" ]; then
-        # Find active milestone from roadmap
-        local active_milestone
-        active_milestone=$(grep -E ":construction:|Current Milestone" "$ROADMAP_FILE" 2>/dev/null | grep -oE "v[0-9]+\.[0-9]+" | head -1)
-        if [ -n "$active_milestone" ]; then
-            PHASES_DIR="$PLANNING_DIR/milestones/$active_milestone/phases"
-        fi
-    fi
-fi
-
 # ═══════════════════════════════════════════════════════════════════════════════
 # jq Resolution (optional - graceful fallback)
 # ═══════════════════════════════════════════════════════════════════════════════
@@ -673,18 +556,6 @@ get_target_phases() {
 
 phase_has_plan() {
     local phase="$1"
-
-    # Use find_phase_dir if library loaded
-    if [ "$PHASE_PATHS_LOADED" = true ]; then
-        local phase_dir
-        phase_dir=$(find_phase_dir "$phase" "$PLANNING_DIR")
-        if [ -n "$phase_dir" ] && ls "$phase_dir"/*-PLAN.md 1>/dev/null 2>&1; then
-            return 0
-        fi
-        return 1
-    fi
-
-    # Fallback to old logic
     local phase_major="${phase%%.*}"
     local phase_major_padded
     phase_major_padded=$(printf "%02d" "$phase_major")
@@ -704,18 +575,6 @@ phase_has_plan() {
 
 phase_has_summary() {
     local phase="$1"
-
-    # Use find_phase_dir if library loaded
-    if [ "$PHASE_PATHS_LOADED" = true ]; then
-        local phase_dir
-        phase_dir=$(find_phase_dir "$phase" "$PLANNING_DIR")
-        if [ -n "$phase_dir" ] && ls "$phase_dir"/*-SUMMARY.md 1>/dev/null 2>&1; then
-            return 0
-        fi
-        return 1
-    fi
-
-    # Fallback to old logic
     local phase_major="${phase%%.*}"
     local phase_major_padded
     phase_major_padded=$(printf "%02d" "$phase_major")
@@ -737,17 +596,6 @@ phase_has_summary() {
 
 get_phase_name() {
     local phase="$1"
-
-    # Use find_phase_dir if library loaded
-    if [ "$PHASE_PATHS_LOADED" = true ]; then
-        local phase_dir
-        phase_dir=$(find_phase_dir "$phase" "$PLANNING_DIR")
-        if [ -n "$phase_dir" ]; then
-            basename "$phase_dir" | sed "s/^[0-9]\+\. *[0-9]*-//" | sed "s/^[0-9]\+-[^-]*-//"
-        fi
-    fi
-
-    # Fallback to old logic
     local phase_major="${phase%%.*}"
     local phase_major_padded
     phase_major_padded=$(printf "%02d" "$phase_major")
@@ -764,19 +612,6 @@ get_phase_name() {
 
 phase_requires_hitl() {
     local phase="$1"
-
-    # Use find_phase_dir if library loaded
-    if [ "$PHASE_PATHS_LOADED" = true ]; then
-        local phase_dir
-        phase_dir=$(find_phase_dir "$phase" "$PLANNING_DIR")
-        if [ -z "$phase_dir" ]; then
-            return 1
-        fi
-        grep -rq "HITL_REQUIRED: true\|HUMAN_VERIFICATION_REQUIRED\|MANUAL_VERIFICATION" "$phase_dir"/*-PLAN.md 2>/dev/null
-        return $?
-    fi
-
-    # Fallback to old logic
     local phase_major="${phase%%.*}"
     local phase_major_padded
     phase_major_padded=$(printf "%02d" "$phase_major")
@@ -846,23 +681,6 @@ git_commit_phase() {
 # Execution
 # ═══════════════════════════════════════════════════════════════════════════════
 
-# ═══════════════════════════════════════════════════════════════════════════════
-# Claude Command Runner (supports infisical for MiniMax)
-# ═══════════════════════════════════════════════════════════════════════════════
-
-INFISICAL_AVAILABLE=false
-CLAUDE_CMD="claude"
-
-check_infiisical() {
-    if command -v infisical &>/dev/null; then
-        INFISICAL_AVAILABLE=true
-        # Use infisical to inject MiniMax API key
-        CLAUDE_CMD="infisical run --"
-    fi
-}
-
-check_infiisical
-
 run_claude_command() {
     local command="$1"
 
@@ -888,46 +706,23 @@ run_claude_command() {
 
     local output_file="$LOGS_DIR/claude_${RUN_ID}_$(date +%s).log"
 
-    # Use infisical if available, otherwise direct claude
-    if $INFISICAL_AVAILABLE; then
-        log INFO "Using MiniMax via infisical"
-        if $CLAUDE_CMD claude -p "$command" >> "$output_file" 2>&1; then
-            log OK "Command completed"
-            reset_consecutive_failures
-            [[ "$CIRCUIT_BREAKER_LOADED" == "true" ]] && circuit_breaker_record_success
-            return 0
-        else
-            local exit_code=$?
-            log ERROR "Command failed (exit $exit_code). See $output_file"
-
-            local signature
-            signature=$(grep -i "error\|failed\|exception" "$output_file" 2>/dev/null | head -1 | cut -c1-100 || echo "unknown_error")
-
-            if ! record_failure "$signature" "$exit_code"; then
-                return 1
-            fi
-
-            return 1
-        fi
+    if claude -p "$command" >> "$output_file" 2>&1; then
+        log OK "Command completed"
+        reset_consecutive_failures
+        [[ "$CIRCUIT_BREAKER_LOADED" == "true" ]] && circuit_breaker_record_success
+        return 0
     else
-        if claude -p "$command" >> "$output_file" 2>&1; then
-            log OK "Command completed"
-            reset_consecutive_failures
-            [[ "$CIRCUIT_BREAKER_LOADED" == "true" ]] && circuit_breaker_record_success
-            return 0
-        else
-            local exit_code=$?
-            log ERROR "Command failed (exit $exit_code). See $output_file"
-
-            local signature
-            signature=$(grep -i "error\|failed\|exception" "$output_file" 2>/dev/null | head -1 | cut -c1-100 || echo "unknown_error")
+        local exit_code=$?
+        log ERROR "Command failed (exit $exit_code). See $output_file"
 
-            if ! record_failure "$signature" "$exit_code"; then
-                return 1
-            fi
+        local signature
+        signature=$(grep -i "error\|failed\|exception" "$output_file" 2>/dev/null | head -1 | cut -c1-100 || echo "unknown_error")
 
+        if ! record_failure "$signature" "$exit_code"; then
             return 1
         fi
+
+        return 1
     fi
 }