thrivekit 2.0.21 → 2.0.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thrivekit",
-  "version": "2.0.21",
+  "version": "2.0.22",
   "description": "Tools to thrive with agentic coding - RALPH autonomous loop, Claude Code hooks, PRD-driven development",
   "author": "Allie Jones <allie@allthrive.ai>",
   "license": "MIT",

package/ralph/init.sh

@@ -482,6 +482,7 @@ Commands:
   prd --file <file>       Generate PRD from file
   run                     Run autonomous loop until all stories pass
   run --max <n>           Run with max iterations (default: 20)
+  run --fast              Skip code review for faster iterations
   status                  Show current feature and story status
   check                   Run verification checks only
   verify <story-id>       Verify a specific story

package/ralph/loop.sh

@@ -99,6 +99,7 @@ preflight_checks() {
 run_loop() {
   local max_iterations="$DEFAULT_MAX_ITERATIONS"
   local specific_story=""
+  local fast_mode=false
 
   # Parse arguments
   while [[ $# -gt 0 ]]; do
@@ -111,12 +112,19 @@ run_loop() {
         specific_story="$2"
         shift 2
         ;;
+      --fast)
+        fast_mode=true
+        shift
+        ;;
       *)
         shift
         ;;
     esac
   done
 
+  # Export for use in verification
+  export RALPH_FAST_MODE="$fast_mode"
+
   # Validate prerequisites
   check_dependencies
 
@@ -236,10 +244,8 @@ run_loop() {
         continue
       fi
 
-      # Exponential backoff before retry
-      local backoff=$((2 ** (consecutive_failures - 1)))
-      print_warning "Retry $consecutive_failures/$max_story_retries for $story (waiting ${backoff}s...)"
-      sleep "$backoff"
+      # Quick retry - no delay needed (Claude API isn't rate-limited)
+      print_warning "Retry $consecutive_failures/$max_story_retries for $story"
     else
       consecutive_failures=1
       last_story="$story"

package/ralph/verify.sh

@@ -38,15 +38,25 @@ run_verification() {
   print_info "=== Verification: $story ==="
   echo ""
 
-  # Determine story type
-  local story_type
-  story_type=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .type // "frontend"' "$RALPH_DIR/prd.json" 2>/dev/null)
+  # Clear old failure logs (so smart-skip logic uses fresh state)
+  rm -f "$RALPH_DIR/last_precommit_failure.log" \
+        "$RALPH_DIR/last_test_failure.log" \
+        "$RALPH_DIR/last_review_failure.json" 2>/dev/null
 
-  local has_test_url
-  has_test_url=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .testUrl // empty' "$RALPH_DIR/prd.json" 2>/dev/null)
+  # Check for fast mode
+  local fast_mode="${RALPH_FAST_MODE:-false}"
+  if [[ "$fast_mode" == "true" ]]; then
+    echo "  (fast mode - skipping code review)"
+  fi
+
+  # Determine story type (single jq call for all story info)
+  local story_json
+  story_json=$(jq --arg id "$story" '.stories[] | select(.id==$id)' "$RALPH_DIR/prd.json" 2>/dev/null)
 
-  local has_api_endpoints
-  has_api_endpoints=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .apiEndpoints[0] // empty' "$RALPH_DIR/prd.json" 2>/dev/null)
+  local story_type has_test_url has_api_endpoints
+  story_type=$(echo "$story_json" | jq -r '.type // "frontend"')
+  has_test_url=$(echo "$story_json" | jq -r '.testUrl // empty')
+  has_api_endpoints=$(echo "$story_json" | jq -r '.apiEndpoints[0] // empty')
 
   # Auto-detect type if not specified
   if [[ -n "$has_api_endpoints" && -z "$has_test_url" ]]; then
@@ -54,52 +64,86 @@ run_verification() {
   fi
 
   local failed=0
+  local lint_failed=0
+  local test_failed=0
 
   # ========================================
-  # STEP 1: Code review (catch issues before running tests)
+  # STEP 1: Code review (skip in fast mode or if last failure was lint/test)
   # ========================================
-  echo "  [1/6] Running code review..."
-  if ! run_code_review "$story"; then
-    failed=1
+  local skip_review=false
+  if [[ "$fast_mode" == "true" ]]; then
+    skip_review=true
+  elif [[ -f "$RALPH_DIR/last_precommit_failure.log" ]] || [[ -f "$RALPH_DIR/last_test_failure.log" ]]; then
+    # Skip review if last failure was lint/test - review won't help
+    skip_review=true
+    echo "  [1/5] Skipping code review (last failure was lint/test)"
   fi
 
-  # ========================================
-  # STEP 2: Run configured checks (lint, build, etc.)
-  # ========================================
-  if [[ $failed -eq 0 ]]; then
-    echo ""
-    echo "  [2/6] Running configured checks..."
-    if ! run_configured_checks; then
+  if [[ "$skip_review" == "false" ]]; then
+    echo "  [1/5] Running code review..."
+    if ! run_code_review "$story"; then
       failed=1
     fi
   fi
 
   # ========================================
-  # STEP 3: Run unit tests
+  # STEP 2+3: Run lint and tests IN PARALLEL
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
-    echo "  [3/6] Running unit tests..."
-    if ! run_unit_tests; then
+    echo "  [2/5] Running lint + tests (parallel)..."
+
+    # Create temp files for results
+    local lint_log test_log
+    lint_log=$(mktemp)
+    test_log=$(mktemp)
+
+    # Run lint in background
+    (run_configured_checks > "$lint_log" 2>&1; echo $? > "${lint_log}.exit") &
+    local lint_pid=$!
+
+    # Run tests in background
+    (run_unit_tests > "$test_log" 2>&1; echo $? > "${test_log}.exit") &
+    local test_pid=$!
+
+    # Wait for both
+    wait $lint_pid 2>/dev/null
+    wait $test_pid 2>/dev/null
+
+    # Check results
+    lint_failed=$(cat "${lint_log}.exit" 2>/dev/null || echo "1")
+    test_failed=$(cat "${test_log}.exit" 2>/dev/null || echo "1")
+
+    # Show lint output
+    echo "    Lint:"
+    cat "$lint_log" | sed 's/^/      /'
+
+    # Show test output
+    echo "    Tests:"
+    cat "$test_log" | sed 's/^/      /'
+
+    # Cleanup
+    rm -f "$lint_log" "${lint_log}.exit" "$test_log" "${test_log}.exit"
+
+    if [[ "$lint_failed" != "0" ]] || [[ "$test_failed" != "0" ]]; then
       failed=1
     fi
   fi
 
   # ========================================
-  # STEP 4: Run Playwright tests (frontend) or API tests (backend)
+  # STEP 3: Run Playwright tests (frontend) or API tests (backend)
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
     if [[ "$story_type" == "backend" ]]; then
-      echo "  [4/6] Running API tests..."
+      echo "  [3/5] Running API tests..."
       if ! run_api_validation "$story"; then
         failed=1
       elif ! run_api_error_tests "$story"; then
-        # Only run error tests if validation passed
         failed=1
       fi
     else
-      echo "  [4/6] Running Playwright tests..."
+      echo "  [3/5] Running Playwright tests..."
       if ! run_playwright_tests "$story"; then
         failed=1
       fi
@@ -107,17 +151,17 @@ run_verification() {
   fi
 
   # ========================================
-  # STEP 5: Run browser validation (frontend) or API validation (backend)
+  # STEP 4: Run browser validation (frontend) or API validation (backend)
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
     if [[ "$story_type" == "backend" ]]; then
-      echo "  [5/6] Running API validation..."
+      echo "  [4/5] Running API validation..."
       if ! run_api_tests "$story"; then
         failed=1
       fi
     else
-      echo "  [5/6] Running browser validation..."
+      echo "  [4/5] Running browser validation..."
       if ! run_browser_validation "$story"; then
         failed=1
       fi
@@ -125,11 +169,11 @@ run_verification() {
   fi
 
   # ========================================
-  # STEP 6: Run PRD test steps
+  # STEP 5: Run PRD test steps
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
-    echo "  [6/6] Running PRD test steps..."
+    echo "  [5/5] Running PRD test steps..."
     if ! verify_prd_criteria "$story"; then
       failed=1
     fi