agentic-loop 3.12.2 → 3.13.0

package/package.json

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

package/ralph/init.sh

@@ -603,6 +603,36 @@ auto_configure_project() {
     fi
   fi
 
+  # 8. Detect Python runner (uv/poetry/pipenv) and update test command
+  if [[ -f "pyproject.toml" || -f "requirements.txt" ]]; then
+    local py_runner=""
+    py_runner=$(detect_python_runner ".")
+
+    if [[ -n "$py_runner" ]]; then
+      # Check if commands.test exists and needs updating
+      local current_test
+      current_test=$(jq -r '.commands.test // ""' "$tmpfile" 2>/dev/null)
+
+      # Update if test command is bare pytest (no runner prefix)
+      if [[ "$current_test" == "pytest"* ]] && [[ "$current_test" != "$py_runner"* ]]; then
+        local new_test="${py_runner} ${current_test}"
+        jq --arg test "$new_test" '.commands.test = $test' "$tmpfile" > "${tmpfile}.new" && mv "${tmpfile}.new" "$tmpfile"
+        echo "  Auto-updated commands.test: $new_test"
+        updated=true
+      fi
+
+      # Also update commands.dev if it's bare python
+      local current_dev
+      current_dev=$(jq -r '.commands.dev // ""' "$tmpfile" 2>/dev/null)
+      if [[ "$current_dev" == "python "* ]] && [[ "$current_dev" != "$py_runner"* ]]; then
+        local new_dev="${py_runner} ${current_dev}"
+        jq --arg dev "$new_dev" '.commands.dev = $dev' "$tmpfile" > "${tmpfile}.new" && mv "${tmpfile}.new" "$tmpfile"
+        echo "  Auto-updated commands.dev: $new_dev"
+        updated=true
+      fi
+    fi
+  fi
+
   # Save if updated
   if [[ "$updated" == "true" ]]; then
     mv "$tmpfile" "$config"

package/ralph/prd-check.sh

@@ -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

package/templates/config/fastmcp.json

@@ -25,12 +25,12 @@
   },
 
   "commands": {
-    "dev": "python -m ${PROJECT_MODULE}.server",
+    "dev": "uv run python -m ${PROJECT_MODULE}.server",
     "install": "uv pip install -e .",
-    "lint": "ruff check src/",
-    "format": "ruff format src/",
-    "typecheck": "mypy src/",
-    "test": "pytest",
+    "lint": "uv run ruff check src/",
+    "format": "uv run ruff format src/",
+    "typecheck": "uv run mypy src/",
+    "test": "uv run pytest",
     "seed": "",
     "resetDb": ""
   },

package/templates/hooks/warn-uv-python.sh

@@ -0,0 +1,38 @@
+#!/usr/bin/env bash
+# warn-uv-python.sh - Warn about bare pytest/python instead of uv run
+# Hook: PostToolUse matcher: "Bash"
+#
+# Copy to .ralph/hooks/ and Claude Code will warn when bare pytest/python
+# commands are used in projects that should use 'uv run'.
+
+set -euo pipefail
+
+INPUT=$(cat)
+COMMAND=$(echo "$INPUT" | jq -r '.tool_input.command // ""')
+
+# Skip if empty
+[[ -z "$COMMAND" ]] && { echo '{"continue": true}'; exit 0; }
+
+WARNINGS=""
+
+# Check for bare pytest (not preceded by "uv run")
+if echo "$COMMAND" | grep -qE '(^|\s|;|&&|\|)pytest\s' && ! echo "$COMMAND" | grep -qE 'uv run pytest'; then
+  WARNINGS="Use 'uv run pytest' instead of bare 'pytest' to ensure correct environment."
+fi
+
+# Check for bare python (not preceded by "uv run")
+if echo "$COMMAND" | grep -qE '(^|\s|;|&&|\|)python\s' && ! echo "$COMMAND" | grep -qE 'uv run python'; then
+  WARNINGS="${WARNINGS:+$WARNINGS\n}Use 'uv run python' instead of bare 'python' to ensure correct environment."
+fi
+
+# Output warning as additional context (non-blocking)
+if [[ -n "$WARNINGS" ]]; then
+  jq -n --arg warn "$WARNINGS" '{
+    "continue": true,
+    "hookSpecificOutput": {
+      "additionalContext": $warn
+    }
+  }'
+else
+  echo '{"continue": true}'
+fi