agentic-loop 3.12.1 → 3.12.3

package/package.json

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

package/ralph/init.sh

@@ -593,8 +593,8 @@ auto_configure_project() {
     if [[ "$require_tests" == "true" ]]; then
       echo ""
       print_warning "No test directory or test files found."
-      echo "  Without tests, Ralph can only verify syntax and API responses."
-      echo "  Import errors and integration issues won't be caught."
+      echo "  Without tests, Ralph relies on lint, type-checking, and PRD test steps."
+      echo "  Consider adding tests or PRD testCommands for better verification."
       echo ""
       echo "  To fix: Add tests, or set in .ralph/config.json:"
       echo "    {\"tests\": {\"directory\": \"src\", \"patterns\": \"*.test.ts\"}}"

package/ralph/prd-check.sh

@@ -174,8 +174,8 @@ validate_prd() {
     if [[ "$require_tests" == "true" && -z "$test_dir" ]]; then
       echo ""
       print_warning "No test directory configured in .ralph/config.json"
-      echo "  Without tests, Ralph can only verify syntax and API responses."
-      echo "  Import errors and integration issues won't be caught."
+      echo "  Without tests, Ralph relies on lint, type-checking, and PRD test steps."
+      echo "  Consider adding tests or PRD testCommands for better verification."
       echo ""
       echo "  To fix: Add tests, or set in .ralph/config.json:"
       echo "    {\"tests\": {\"directory\": \"src\", \"patterns\": \"*.test.ts\"}}"
@@ -284,7 +284,7 @@ _validate_and_fix_stories() {
   local cnt_no_tests=0 cnt_backend_curl=0 cnt_backend_contract=0
   local cnt_frontend_tsc=0 cnt_frontend_url=0 cnt_frontend_context=0
   local cnt_auth_security=0 cnt_list_pagination=0 cnt_prose_steps=0
-  local cnt_migration_prereq=0 cnt_naming_convention=0
+  local cnt_migration_prereq=0 cnt_naming_convention=0 cnt_bare_pytest=0
 
   echo "  Checking test coverage..."
 
@@ -331,6 +331,18 @@ _validate_and_fix_stories() {
           cnt_frontend_tsc=$((cnt_frontend_tsc + 1))
         fi
       fi
+
+      # Check for bare pytest/python in projects using uv/poetry/pipenv
+      local py_runner
+      py_runner=$(detect_python_runner ".")
+      if [[ -n "$py_runner" ]]; then
+        # Project uses a Python runner - check for bare pytest/python commands
+        # Match: "pytest " at start or after space/semicolon, but not preceded by "run "
+        if echo "$test_steps" | grep -qE '(^|[; ])pytest ' && ! echo "$test_steps" | grep -qE "(uv run|poetry run|pipenv run) pytest"; then
+          story_issues+="use '$py_runner pytest' not bare 'pytest', "
+          cnt_bare_pytest=$((cnt_bare_pytest + 1))
+        fi
+      fi
     fi
 
     # Check 2: Backend needs apiContract
@@ -434,6 +446,7 @@ _validate_and_fix_stories() {
     [[ $cnt_list_pagination -gt 0 ]] && echo "    ${cnt_list_pagination}x list: add pagination"
     [[ $cnt_migration_prereq -gt 0 ]] && echo "    ${cnt_migration_prereq}x migration: add prerequisites (DB reset)"
     [[ $cnt_naming_convention -gt 0 ]] && echo "    ${cnt_naming_convention}x API consumer: add camelCase transformation note"
+    [[ $cnt_bare_pytest -gt 0 ]] && echo "    ${cnt_bare_pytest}x use 'uv run pytest' not bare 'pytest'"
 
     # Check if Claude is available for auto-fix
     if command -v claude &>/dev/null; then