agentic-loop 3.1.5 → 3.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-loop",
-  "version": "3.1.5",
+  "version": "3.1.6",
   "description": "Autonomous AI coding loop - PRD-driven development with Claude Code",
   "author": "Allie Jones <allie@allthrive.ai>",
   "license": "MIT",

package/ralph/loop.sh

@@ -337,25 +337,17 @@ run_loop() {
 
     # 6. Run verification pipeline
     echo ""
-    if run_verification "$story"; then
+    # Capture verification output for failure context
+    local verify_log="$RALPH_DIR/last_verification.log"
+    set -o pipefail
+    if run_verification "$story" 2>&1 | tee "$verify_log"; then
       # Mark story as complete
       update_json "$RALPH_DIR/prd.json" \
         --arg id "$story" '(.stories[] | select(.id==$id) | .passes) = true'
 
       # Clear failure context on success
       rm -f "$RALPH_DIR/last_failure.txt"
-      rm -f "$RALPH_DIR/last_migration_failure.log"
-      rm -f "$RALPH_DIR/last_review_failure.json"
-      rm -f "$RALPH_DIR/last_test_failure.log"
-      rm -f "$RALPH_DIR/last_lint_failure.log"
-      rm -f "$RALPH_DIR/last_typescript_failure.log"
-      rm -f "$RALPH_DIR/last_build_failure.log"
-      rm -f "$RALPH_DIR/last_go_failure.log"
-      rm -f "$RALPH_DIR/last_rust_failure.log"
-      rm -f "$RALPH_DIR/last_playwright_failure.log"
-      rm -f "$RALPH_DIR/last_browser_failure.json"
-      rm -f "$RALPH_DIR/last_precommit_failure.log"
-      rm -f "$RALPH_DIR/last_prd_failure.log"
+      rm -f "$RALPH_DIR/last_verification.log"
 
       # Auto-commit if git is available
       if command -v git &>/dev/null && [[ -d ".git" ]]; then
@@ -601,24 +593,11 @@ _inject_failure_context() {
   echo ""
   echo "## Previous Iteration Failed"
   echo ""
-  echo "**IMPORTANT:** The previous attempt at this story failed verification. Review the errors below and fix them."
+  echo "Fix the errors below. If a PRD test step is broken, you can fix it in .ralph/prd.json."
   echo ""
   echo '```'
   echo "$failure_context"
   echo '```'
-  echo ""
-  echo "### What to do:"
-  echo "1. Read the error messages carefully"
-  echo "2. Identify the root cause - is it your implementation OR a flawed test step?"
-  echo "3. Fix the issue (do not just retry the same approach)"
-  echo "4. Run verification again"
-  echo ""
-  echo "### If the test step itself is broken:"
-  echo "If the error is in the PRD test step (not your implementation), you CAN fix it:"
-  echo "1. Read .ralph/prd.json to find the story index and broken testStep"
-  echo "2. Use jq to update: \`jq '.stories[IDX].testSteps[N] = \"fixed\"' .ralph/prd.json > tmp && mv tmp .ralph/prd.json\`"
-  echo "3. Common test step bugs: missing imports, str() triggering lazy loading, wrong paths"
-  echo "4. ONLY fix the test command - do NOT change acceptance criteria or story scope"
 }
 
 # Helper: Inject signs (learned patterns)

package/ralph/verify.sh

@@ -197,151 +197,14 @@ run_verification() {
 # Save failure context for next iteration
 save_failure_context() {
   local story="$1"
-
   local context_file="$RALPH_DIR/last_failure.txt"
 
   {
-    echo "=== Failure Context for $story ==="
-    echo "Timestamp: $(date -Iseconds 2>/dev/null || date)"
+    echo "=== Verification failed for $story ==="
     echo ""
-
-    if [[ -f "$RALPH_DIR/last_migration_failure.log" ]]; then
-      echo "--- Migration Failure ---"
-      echo "Database migrations failed. Fix the migration or the code causing it:"
-      tail -50 "$RALPH_DIR/last_migration_failure.log"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_review_failure.json" ]]; then
-      echo "--- Code Review Failure ---"
-      echo "Issues found by code review:"
-      jq -r '.issues[] | "- [\(.severity)] \(.category): \(.message)\n  File: \(.file // "unknown"):\(.line // "?")\n  Fix: \(.suggestion // "See above")"' "$RALPH_DIR/last_review_failure.json" 2>/dev/null
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_test_failure.log" ]]; then
-      echo "--- Test Failure ---"
-      tail -50 "$RALPH_DIR/last_test_failure.log"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_playwright_failure.log" ]]; then
-      echo "--- Playwright Failure ---"
-      tail -50 "$RALPH_DIR/last_playwright_failure.log"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_browser_failure.json" ]]; then
-      echo "--- Browser Validation Failure ---"
-      jq -r '"Errors: " + (.errors | join(", "))' "$RALPH_DIR/last_browser_failure.json" 2>/dev/null
-      jq -r '"Console errors: " + (.consoleErrors | join(", "))' "$RALPH_DIR/last_browser_failure.json" 2>/dev/null
-      jq -r '"Missing elements: " + (.elementsMissing | join(", "))' "$RALPH_DIR/last_browser_failure.json" 2>/dev/null
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_lint_failure.log" ]]; then
-      echo "--- Lint Failure ---"
-      echo "Fix these lint errors. The auto-fixer couldn't resolve them, but you can:"
-      echo ""
-      cat "$RALPH_DIR/last_lint_failure.log"
-      echo ""
-      echo "Common fixes:"
-      echo "  - E722 bare except: Change 'except:' to 'except Exception:'"
-      echo "  - F401 unused import: Remove the unused import"
-      echo "  - F841 unused variable: Remove or use the variable"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_typescript_failure.log" ]]; then
-      echo "--- TypeScript Failure ---"
-      echo "Fix these TypeScript type errors:"
-      echo ""
-      cat "$RALPH_DIR/last_typescript_failure.log"
-      echo ""
-      echo "Common fixes:"
-      echo "  - TS2322: Type mismatch - ensure assigned value matches expected type"
-      echo "  - TS2339: Property not found - check spelling or add to interface"
-      echo "  - TS2345: Argument type mismatch - cast or fix the argument type"
-      echo "  - TS7006: Implicit any - add explicit type annotation"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_build_failure.log" ]]; then
-      echo "--- Build Failure ---"
-      echo "Fix these build errors:"
-      echo ""
-      cat "$RALPH_DIR/last_build_failure.log"
-      echo ""
-      echo "Common causes:"
-      echo "  - Import errors: Check file paths and exports"
-      echo "  - SSR issues (Next.js): Use dynamic imports for client-only code"
-      echo "  - Missing dependencies: Run npm install"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_go_failure.log" ]]; then
-      echo "--- Go Failure ---"
-      echo "Fix these Go errors:"
-      echo ""
-      cat "$RALPH_DIR/last_go_failure.log"
-      echo ""
-      echo "Common fixes:"
-      echo "  - Unused variable: Remove or use with _ prefix"
-      echo "  - Undefined: Check imports and spelling"
-      echo "  - Type mismatch: Ensure types align or use type assertion"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_rust_failure.log" ]]; then
-      echo "--- Rust Failure ---"
-      echo "Fix these Rust errors:"
-      echo ""
-      cat "$RALPH_DIR/last_rust_failure.log"
-      echo ""
-      echo "Common fixes:"
-      echo "  - Unused variable: Prefix with _ or remove"
-      echo "  - Borrow checker: Check ownership and lifetimes"
-      echo "  - Missing trait: Add #[derive(...)] or impl"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_precommit_failure.log" ]]; then
-      echo "--- Pre-commit / Lint Failure ---"
-      echo "Fix these errors before the story can be completed:"
-      echo ""
-      # Extract actual error lines (not warnings-only or file modification messages)
-      grep -E "^error:|: error:|Error:|SyntaxError|✖ [0-9]+ problems|^[^:]+:[0-9]+:[0-9]+: [EF][0-9]+" "$RALPH_DIR/last_precommit_failure.log" | head -30
-      # If no errors shown, show the full log tail
-      if ! grep -qE "^error:|: error:|Error:|SyntaxError|✖ [0-9]+ problems" "$RALPH_DIR/last_precommit_failure.log"; then
-        echo "(Full output):"
-        tail -40 "$RALPH_DIR/last_precommit_failure.log"
-      fi
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_fastapi_response_check.log" ]]; then
-      echo "--- FastAPI Response Model Failure ---"
-      echo "Add Pydantic response_model to these endpoints for proper Swagger docs:"
-      echo ""
-      cat "$RALPH_DIR/last_fastapi_response_check.log"
-      echo ""
-      echo "Fix by adding response_model parameter or return type annotation:"
-      echo '  @router.get("/items", response_model=list[ItemSchema])'
-      echo "  async def get_items() -> list[ItemSchema]:"
-      echo ""
-    fi
-
-    if [[ -f "$RALPH_DIR/last_prd_failure.log" ]]; then
-      echo "--- PRD Test Step Failure ---"
-      echo "A test step in the PRD failed. Either fix your implementation OR fix the test step:"
-      echo ""
-      cat "$RALPH_DIR/last_prd_failure.log"
-      echo ""
-      echo "If the test step itself is broken (not your code), you can fix it:"
-      echo "  1. Read .ralph/prd.json to find the story and testSteps array"
-      echo "  2. Fix the command (e.g., add missing imports for SQLAlchemy models)"
-      echo "  3. Update with: jq '.stories[IDX].testSteps[N] = \"fixed command\"' .ralph/prd.json > tmp && mv tmp .ralph/prd.json"
-      echo ""
+    # Just include the verification output - Claude can figure out what went wrong
+    if [[ -f "$RALPH_DIR/last_verification.log" ]]; then
+      tail -100 "$RALPH_DIR/last_verification.log"
     fi
   } > "$context_file"
 }