agentic-loop 3.1.6 → 3.2.1

package/.claude/commands/idea.md

@@ -306,7 +306,7 @@ For frontend stories, acceptance criteria MUST include:
 - "Required elements render" (specify which: header, form, button, etc.)
 - "Works on mobile viewport (375px)"
 
-These get verified by Playwright, not just code review.
+These get verified by Playwright tests and MCP browser tools.
 
 ### Test Steps - CRITICAL
 **Test steps MUST be executable shell commands.** Ralph runs them with bash.
@@ -352,7 +352,7 @@ The Playwright test file can check:
 ]
 ```
 
-**If a step can't be automated**, leave it out of testSteps and put it in acceptanceCriteria instead. Ralph will verify acceptanceCriteria via code review, not by running commands.
+**If a step can't be automated**, leave it out of testSteps and put it in acceptanceCriteria instead. Claude will verify acceptanceCriteria visually using MCP browser tools.
 
 ### Architecture Guidelines
 - **Domain-driven directories** - Group by feature (`src/contact/`) not type (`src/components/`)

package/.claude/commands/prd.md

@@ -283,4 +283,4 @@ The Playwright test file can check:
 ]
 ```
 
-**If a step can't be automated**, leave it out of testSteps and put it in acceptanceCriteria instead. Ralph will verify acceptanceCriteria via code review, not by running commands.
+**If a step can't be automated**, leave it out of testSteps and put it in acceptanceCriteria instead. Claude will verify acceptanceCriteria visually using MCP browser tools.

package/README.md

@@ -45,8 +45,7 @@ claude --dangerously-skip-permissions
 
 **Terminal 2 - Ralph Loop:**
 ```bash
-npx agentic-loop run       # Execute PRDs autonomously
-npx agentic-loop run --fast  # Skip code review (~2x faster)
+npx agentic-loop run         # Execute PRDs autonomously
 ```
 
 > **Tip:** Use two terminals. Plan with Claude in one, run Ralph in the other.
@@ -61,8 +60,8 @@ npx agentic-loop run --fast  # Skip code review (~2x faster)
 ├─────────────────────────────────────────────────────────────┤
 │  1. Read prd.json → find next story where passes=false      │
 │  2. Build prompt (story + context + failures + signs)       │
-│  3. Spawn Claude with prompt                                │
-│  4. Run verification (lint, tests, browser, code review)    │
+│  3. Spawn Claude with prompt + MCP browser tools            │
+│  4. Run verification (lint, tests, testSteps)               │
 │  5. Pass? → commit, next story                              │
 │     Fail? → save error, retry with failure context          │
 │  6. Repeat until all stories pass                           │

package/package.json

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

package/ralph/setup.sh

@@ -340,7 +340,7 @@ EOF
   fi
 }
 
-# Configure MCP (Chrome DevTools)
+# Configure MCP (Browser tools for verification)
 setup_mcp() {
   local claude_json="$HOME/.claude.json"
 
@@ -350,17 +350,82 @@ setup_mcp() {
   # Create claude.json if it doesn't exist
   [[ ! -f "$claude_json" ]] && echo '{}' > "$claude_json"
 
-  # Skip if already configured
-  jq -e '.mcpServers["chrome-devtools"]' "$claude_json" > /dev/null 2>&1 && return 0
+  local added_any=false
+
+  # Add Playwright MCP if not configured
+  if ! jq -e '.mcpServers["playwright"]' "$claude_json" > /dev/null 2>&1; then
+    echo "Configuring MCP servers..."
+    local tmp
+    tmp=$(mktemp)
+    jq '.mcpServers["playwright"] = {
+      "command": "npx",
+      "args": ["-y", "@anthropic-ai/mcp-server-playwright"]
+    }' "$claude_json" > "$tmp" && mv "$tmp" "$claude_json"
+    echo "  Added playwright MCP server (browser automation & testing)"
+    added_any=true
+  fi
 
-  echo "Configuring MCP servers..."
-  local tmp
-  tmp=$(mktemp)
-  jq '.mcpServers["chrome-devtools"] = {
-    "command": "npx",
-    "args": ["-y", "@anthropic-ai/mcp-server-chrome-devtools@0.0.5"]
-  }' "$claude_json" > "$tmp" && mv "$tmp" "$claude_json"
-  echo "  Added chrome-devtools MCP server"
+  # Add Chrome DevTools MCP if not configured
+  if ! jq -e '.mcpServers["chrome-devtools"]' "$claude_json" > /dev/null 2>&1; then
+    [[ "$added_any" == "false" ]] && echo "Configuring MCP servers..."
+    local tmp
+    tmp=$(mktemp)
+    jq '.mcpServers["chrome-devtools"] = {
+      "command": "npx",
+      "args": ["-y", "@anthropic-ai/mcp-server-chrome-devtools@0.0.5"]
+    }' "$claude_json" > "$tmp" && mv "$tmp" "$claude_json"
+    echo "  Added chrome-devtools MCP server (debugging & inspection)"
+    added_any=true
+  fi
+
+  # Ask about test credentials
+  if [[ "$added_any" == "true" ]]; then
+    setup_test_credentials
+  fi
+}
+
+# Set up test credentials for browser automation
+setup_test_credentials() {
+  echo ""
+  echo "  Browser automation often needs login credentials for testing."
+  echo ""
+  read -r -p "  Do you have test user credentials to configure? [y/N] " response
+
+  if [[ "$response" =~ ^[Yy]$ ]]; then
+    echo ""
+    echo "  These will be stored in .env (already in .gitignore - never committed)"
+    echo ""
+
+    # Create .env if it doesn't exist
+    if [[ ! -f ".env" ]]; then
+      touch ".env"
+      echo "  Created .env file"
+    fi
+
+    # Get credentials
+    read -r -p "  Test user email: " test_email
+    read -r -s -p "  Test user password: " test_password
+    echo ""
+
+    # Add to .env (append or update)
+    if grep -q "^RALPH_TEST_USER=" .env 2>/dev/null; then
+      # Update existing
+      local tmp
+      tmp=$(mktemp)
+      sed "s/^RALPH_TEST_USER=.*/RALPH_TEST_USER=$test_email/" .env > "$tmp" && mv "$tmp" .env
+      sed "s/^RALPH_TEST_PASSWORD=.*/RALPH_TEST_PASSWORD=$test_password/" .env > "$tmp" && mv "$tmp" .env
+    else
+      # Append new
+      echo "" >> .env
+      echo "# Test credentials for browser automation (auto-added by agentic-loop)" >> .env
+      echo "RALPH_TEST_USER=$test_email" >> .env
+      echo "RALPH_TEST_PASSWORD=$test_password" >> .env
+    fi
+
+    echo "  Saved credentials to .env"
+    echo ""
+    echo "  Note: .env is gitignored - your password will never be committed to git."
+  fi
 }
 
 # Set up pre-commit hooks

package/ralph/verify.sh

@@ -1,35 +1,14 @@
 #!/usr/bin/env bash
 # shellcheck shell=bash
-# verify.sh - Full UAT verification pipeline for ralph
+# verify.sh - Lightweight verification for ralph
 #
-# This file orchestrates the verification pipeline by sourcing modular components:
-#   - verify/review.sh  - Code review logic
-#   - verify/lint.sh    - Auto-fix + lint checks
-#   - verify/tests.sh   - Unit tests + PRD criteria
-#   - verify/browser.sh - Browser validation
-
-# Validate required source files exist before sourcing
-if [[ ! -f "$RALPH_LIB/playwright.sh" ]]; then
-  echo "Error: Missing $RALPH_LIB/playwright.sh" >&2
-  exit 1
-fi
-if [[ ! -f "$RALPH_LIB/api.sh" ]]; then
-  echo "Error: Missing $RALPH_LIB/api.sh" >&2
-  exit 1
-fi
+# Philosophy: Claude verifies its own work using MCP browser tools.
+# Ralph just runs lint, tests, and testSteps from the PRD.
 
 # Source verification modules
-source "$RALPH_LIB/playwright.sh"
-source "$RALPH_LIB/api.sh"
-
-# Determine the directory where this script lives
 VERIFY_DIR="${RALPH_LIB:-$(dirname "${BASH_SOURCE[0]}")}"
-
-# Source modular verification components
-source "$VERIFY_DIR/verify/review.sh"
 source "$VERIFY_DIR/verify/lint.sh"
 source "$VERIFY_DIR/verify/tests.sh"
-source "$VERIFY_DIR/verify/browser.sh"
 
 run_verification() {
   local story="$1"
@@ -38,142 +17,33 @@ run_verification() {
   print_info "=== Verification: $story ==="
   echo ""
 
-  # 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
-
-  # 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 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
-    story_type="backend"
-  fi
-
   local failed=0
-  local lint_failed=0
-  local test_failed=0
 
   # ========================================
-  # STEP 1: Code review (skip in fast mode or if last failure was lint/test)
+  # STEP 1: Run lint checks
   # ========================================
-  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
-
-  if [[ "$skip_review" == "false" ]]; then
-    echo "  [1/5] Running code review..."
-    if ! run_code_review "$story"; then
-      failed=1
-    fi
+  echo "  [1/3] Running lint checks..."
+  if ! run_configured_checks; then
+    failed=1
   fi
 
   # ========================================
-  # STEP 2+3: Run lint and tests IN PARALLEL
+  # STEP 2: Run unit tests
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
-    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
+    echo "  [2/3] Running tests..."
+    if ! run_unit_tests; then
       failed=1
     fi
   fi
 
   # ========================================
-  # STEP 3: Run Playwright tests (frontend) or API tests (backend)
-  # ========================================
-  if [[ $failed -eq 0 ]]; then
-    echo ""
-    if [[ "$story_type" == "backend" ]]; then
-      echo "  [3/5] Running API tests..."
-      if ! run_api_validation "$story"; then
-        failed=1
-      elif ! run_api_error_tests "$story"; then
-        failed=1
-      fi
-    else
-      echo "  [3/5] Running Playwright tests..."
-      if ! run_playwright_tests "$story"; then
-        failed=1
-      fi
-    fi
-  fi
-
-  # ========================================
-  # STEP 4: Run browser validation (frontend) or API validation (backend)
-  # ========================================
-  if [[ $failed -eq 0 ]]; then
-    echo ""
-    if [[ "$story_type" == "backend" ]]; then
-      echo "  [4/5] Running API validation..."
-      if ! run_api_tests "$story"; then
-        failed=1
-      fi
-    else
-      echo "  [4/5] Running browser validation..."
-      if ! run_browser_validation "$story"; then
-        failed=1
-      fi
-    fi
-  fi
-
-  # ========================================
-  # STEP 5: Run PRD test steps
+  # STEP 3: Run PRD test steps
   # ========================================
   if [[ $failed -eq 0 ]]; then
     echo ""
-    echo "  [5/5] Running PRD test steps..."
+    echo "  [3/3] Running PRD test steps..."
     if ! verify_prd_criteria "$story"; then
       failed=1
     fi
@@ -188,7 +58,6 @@ run_verification() {
     return 0
   else
     print_error "=== Verification failed ==="
-    # Save failure context for next iteration
     save_failure_context "$story"
     return 1
   fi

package/templates/PROMPT.md

@@ -14,36 +14,38 @@ Before writing any code, verify:
 
 For each story, you must:
 
-### 1. Write Tests First
+### 1. Implement the Feature
 
-**For frontend stories:**
-- Write a Playwright test that validates the acceptance criteria
-- Include tests for error handling (API fails, validation errors)
-- Include tests for empty/loading states
-- Include accessibility checks (axe-core)
-- Include mobile viewport test (375px)
-
-**For backend stories:**
-- Write unit tests for the business logic
-- Write API tests that validate all endpoints
-- Test error responses (400, 401, 500)
-- Test validation rules
-
-### 2. Implement the Feature
-
-- Write code to make all tests pass
 - Follow existing patterns in the codebase
 - Handle ALL error cases defined in the story
 - Implement loading states for async operations
 
+### 2. Write Tests
+
+- Write unit tests for the business logic
+- Write tests that validate acceptance criteria
+- Test error cases and edge cases
+
 ### 3. Verify It Actually Works
 
+**You have browser tools - USE THEM to verify your work:**
+
+**Playwright MCP** (testing & automation):
+- `browser_navigate` - Go to a URL and get page content
+- `browser_screenshot` - Take a screenshot to verify UI
+- `browser_click` - Click elements to test interactions
+- `browser_type` - Fill in forms to test inputs
+- `browser_snapshot` - Get accessibility tree for a11y testing
+
+**Chrome DevTools MCP** (debugging & inspection):
+- Inspect DOM, check console for errors
+- Debug network requests
+- Check element styles and computed properties
+
 **Do NOT say you're done until:**
 - All unit tests pass
-- All Playwright tests pass
-- You've opened the browser via MCP and visually verified
+- You've opened the browser and visually verified the feature works
 - Console has no errors
-- It works on mobile (375px viewport)
 - Error states are handled gracefully
 
 ## Rules
@@ -60,33 +62,14 @@ For each story, you must:
 
 Before considering any story complete:
 
-### Code
 - [ ] All acceptance criteria are met
 - [ ] All error handling from story is implemented
-- [ ] Loading states implemented (if frontend)
-- [ ] Validation implemented (if backend)
-- [ ] TypeScript compiles without errors
-
-### Tests
+- [ ] TypeScript/code compiles without errors
 - [ ] Unit tests written and passing
-- [ ] Playwright test written and passing (frontend)
-- [ ] API tests written and passing (backend)
-- [ ] Error cases tested
-- [ ] Edge cases tested (empty state, etc.)
-
-### Browser/API Validation
-- [ ] Browser check passes (frontend) - no console errors
-- [ ] Mobile viewport works (375px)
-- [ ] Accessibility passes (can Tab through, focus visible)
-- [ ] API returns correct responses (backend)
-
-### Documentation
-- [ ] Updated `.ralph/progress.txt` with files created/modified
-- [ ] Noted any key decisions or context for next story
-
-### Quality
+- [ ] **Browser verified** - used Playwright MCP to visually confirm it works
+- [ ] No console errors
 - [ ] Linting passes
-- [ ] Existing tests still pass
+- [ ] Updated `.ralph/progress.txt` with files created/modified
 
 ## If Verification Fails
 

package/ralph/api.sh

@@ -1,216 +0,0 @@
-#!/usr/bin/env bash
-# shellcheck shell=bash
-# api.sh - API validation for backend stories
-
-# Parse an endpoint string into method and path
-# Usage: parse_endpoint "POST /api/users" "http://localhost:3000"
-# Sets: ENDPOINT_METHOD, ENDPOINT_PATH, ENDPOINT_URL
-parse_endpoint() {
-  local endpoint="$1"
-  local base_url="${2:-}"
-
-  # Defaults
-  ENDPOINT_METHOD="GET"
-  ENDPOINT_PATH="$endpoint"
-  ENDPOINT_URL=""
-
-  # Parse method if present (e.g., "POST /api/contact")
-  if [[ "$endpoint" =~ ^(GET|POST|PUT|PATCH|DELETE)[[:space:]]+(.*) ]]; then
-    ENDPOINT_METHOD="${BASH_REMATCH[1]}"
-    ENDPOINT_PATH="${BASH_REMATCH[2]}"
-  fi
-
-  # Build full URL
-  if [[ "$ENDPOINT_PATH" =~ ^https?:// ]]; then
-    ENDPOINT_URL="$ENDPOINT_PATH"
-  elif [[ -n "$base_url" ]]; then
-    ENDPOINT_URL="${base_url}${ENDPOINT_PATH}"
-  fi
-}
-
-# Check if endpoint is a WebSocket (can't be tested with HTTP)
-is_websocket_endpoint() {
-  local endpoint="$1"
-  [[ "$endpoint" =~ ^wss?:// ]] || [[ "$endpoint" =~ ^(GET|POST|PUT|PATCH|DELETE)[[:space:]]+wss?:// ]]
-}
-
-# Validate API endpoints for a backend story
-run_api_validation() {
-  local story="$1"
-
-  # Get API endpoints from story
-  local endpoints
-  endpoints=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .apiEndpoints[]?' "$RALPH_DIR/prd.json" 2>/dev/null)
-
-  if [[ -z "$endpoints" ]]; then
-    echo "  (no apiEndpoints defined, skipping API validation)"
-    return 0
-  fi
-
-  # Get base URL from config or use default
-  local base_url
-  base_url=$(get_config '.api.baseUrl' "http://localhost:3000")
-
-  local failed=0
-
-  echo "  Validating API endpoints..."
-
-  while IFS= read -r endpoint; do
-    [[ -z "$endpoint" ]] && continue
-
-    # Skip WebSocket endpoints - they can't be tested with HTTP curl
-    if is_websocket_endpoint "$endpoint"; then
-      echo "    Skipping WebSocket endpoint: $endpoint (use integration tests)"
-      continue
-    fi
-
-    # Parse endpoint into method, path, and full URL
-    parse_endpoint "$endpoint" "$base_url"
-
-    echo -n "    $ENDPOINT_METHOD $ENDPOINT_PATH... "
-
-    # Make the request
-    local response_code
-    response_code=$(curl -sf -m "$CURL_TIMEOUT_SECONDS" -o /dev/null -w "%{http_code}" -X "$ENDPOINT_METHOD" "$ENDPOINT_URL" 2>/dev/null)
-
-    if [[ "$response_code" =~ ^2[0-9][0-9]$ ]]; then
-      print_success "$response_code"
-    elif [[ "$response_code" == "000" ]]; then
-      print_error "connection failed"
-      failed=1
-    elif [[ "$response_code" == "422" || "$response_code" == "400" ]]; then
-      # 422/400 = endpoint exists but needs params - don't fail verification
-      print_warning "$response_code (needs params)"
-    elif [[ "$response_code" == "401" || "$response_code" == "403" ]]; then
-      # Auth required - endpoint exists
-      print_warning "$response_code (auth required)"
-    else
-      print_error "$response_code"
-      failed=1
-    fi
-  done <<< "$endpoints"
-
-  return $failed
-}
-
-# Run comprehensive API tests for a story
-run_api_tests() {
-  local story="$1"
-
-  # Get test steps that look like API calls
-  local test_steps
-  test_steps=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .testSteps[]?' "$RALPH_DIR/prd.json" 2>/dev/null)
-
-  if [[ -z "$test_steps" ]]; then
-    return 0
-  fi
-
-  local failed=0
-  local log_file
-  log_file=$(create_temp_file ".log") || return 1
-
-  echo "  Running API test steps..."
-
-  while IFS= read -r step; do
-    [[ -z "$step" ]] && continue
-
-    # Check if this looks like a curl command or API test
-    if [[ "$step" =~ ^curl ]]; then
-      echo -n "    $step... "
-
-      if safe_exec "$step" "$log_file"; then
-        print_success "passed"
-      else
-        print_error "failed"
-        echo ""
-        echo "    Response:"
-        tail -"$MAX_OUTPUT_PREVIEW_LINES" "$log_file" | sed 's/^/      /'
-        failed=1
-      fi
-    fi
-  done <<< "$test_steps"
-
-  rm -f "$log_file"
-  return $failed
-}
-
-# Validate error handling for API
-run_api_error_tests() {
-  local story="$1"
-
-  # Get error handling requirements
-  local error_handling
-  error_handling=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .errorHandling[]?' "$RALPH_DIR/prd.json" 2>/dev/null)
-
-  if [[ -z "$error_handling" ]]; then
-    return 0
-  fi
-
-  # Get base URL and endpoints
-  local base_url
-  base_url=$(get_config '.api.baseUrl' "http://localhost:3000")
-
-  local endpoints
-  endpoints=$(jq -r --arg id "$story" '.stories[] | select(.id==$id) | .apiEndpoints[0]?' "$RALPH_DIR/prd.json" 2>/dev/null)
-
-  if [[ -z "$endpoints" ]]; then
-    return 0
-  fi
-
-  # Skip WebSocket endpoints
-  if is_websocket_endpoint "$endpoints"; then
-    echo "  Skipping error tests for WebSocket endpoint"
-    return 0
-  fi
-
-  # Parse endpoint (default to POST for error tests)
-  parse_endpoint "$endpoints" "$base_url"
-  [[ "$ENDPOINT_METHOD" == "GET" ]] && ENDPOINT_METHOD="POST"
-
-  local failed=0
-
-  echo "  Testing API error handling..."
-
-  # Test common error cases
-  while IFS= read -r error_case; do
-    [[ -z "$error_case" ]] && continue
-
-    # Check for 400 tests (bad input)
-    if [[ "$error_case" =~ 400 ]]; then
-      echo -n "    Testing 400 (bad request)... "
-
-      local response_code
-      response_code=$(curl -sf -m "$CURL_TIMEOUT_SECONDS" -o /dev/null -w "%{http_code}" \
-        -X "$ENDPOINT_METHOD" \
-        -H "Content-Type: application/json" \
-        -d '{}' \
-        "$ENDPOINT_URL" 2>/dev/null)
-
-      if [[ "$response_code" == "400" ]]; then
-        print_success "correctly returns 400"
-      else
-        print_warning "got $response_code (expected 400)"
-        failed=1
-      fi
-    fi
-
-    # Check for 401 tests (unauthorized)
-    if [[ "$error_case" =~ 401 ]]; then
-      echo -n "    Testing 401 (unauthorized)... "
-
-      local response_code
-      response_code=$(curl -sf -m "$CURL_TIMEOUT_SECONDS" -o /dev/null -w "%{http_code}" \
-        -X "$ENDPOINT_METHOD" \
-        "$ENDPOINT_URL" 2>/dev/null)
-
-      if [[ "$response_code" == "401" ]]; then
-        print_success "correctly returns 401"
-      else
-        print_warning "got $response_code (expected 401)"
-        failed=1
-      fi
-    fi
-  done <<< "$error_handling"
-
-  return $failed
-}