Strip console warnings before classifying infra failures

.gitignore

@@ -1,4 +1,5 @@
 *.pem
+.claude/scheduled_tasks.lock
 .coverage*
 .env*
 .history

CLAUDE.md

@@ -7,6 +7,10 @@
 
 ## Development Commands
 
+### Shell Environment (Claude already inherits it)
+
+The user starts this session with `source .env && source .venv/bin/activate && claude`, so `.env` vars and the `.venv` are already on PATH for every Bash tool call. Do not prepend `source .env` or `source .venv/bin/activate` to commands. Use `pytest`, `python`, `ruff`, etc. directly. For pytest specifically, use `python -m pytest ...` (not `pytest ...` or `uv run pytest ...`) so cwd is added to `sys.path` and top-level imports like `from constants.models ...` resolve without setting `PYTHONPATH`.
+
 ### Running Locally
 
 ```bash

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "GitAuto"
-version = "1.55.0"
+version = "1.55.2"
 requires-python = ">=3.14"
 dependencies = [
     "annotated-doc==0.0.4",

utils/logs/detect_infra_failure.py

@@ -1,6 +1,7 @@
 # Local imports
 from utils.error.handle_exceptions import handle_exceptions
 from utils.logging.logging_config import logger
+from utils.logs.strip_jest_noise import strip_jest_noise
 
 # Patterns for transient infrastructure failures (not code bugs). Add more as we encounter them.
 INFRA_FAILURE_PATTERNS = [
@@ -33,10 +34,13 @@
 
 @handle_exceptions(default_return_value=None, raise_on_error=False)
 def detect_infra_failure(error_log: str):
+    # Strip console.warn/log blocks and AWS SSM-fallback warnings before scanning so app-level noise (e.g. "AccessDeniedException" from a console.warn that fell back to a default) can't false-positive this classifier.
+    error_log = strip_jest_noise(error_log)
     lower_log = error_log.lower()
     for pattern in INFRA_FAILURE_PATTERNS:
         if pattern.lower() in lower_log:
             logger.info("Infrastructure failure detected: '%s'", pattern)
             return pattern
 
+    logger.info("No infrastructure failure pattern matched")
     return None

utils/logs/test_detect_infra_failure.py

@@ -10,6 +10,15 @@
 # Real CI log from SpiderPlus PR 13615 (segfault during PHPUnit)
 SEGFAULT_LOG_PATH = Path("payloads/github/check_suite/segfault_phpunit_log.txt")
 
+# Real CircleCI `test` job log from Foxquilt/foxden-rating-quoting-backend PR 714.
+# App-level console.warn at the top contains "AccessDeniedException" (SSM fetch fell back to defaults — harmless).
+# Jest ran to completion and reported `Tests: 2 failed, 35 skipped, 4500 passed` from real assertion bugs in the PR.
+# detect_infra_failure used to match "AccessDeniedException" and send the PR down the infra-retry branch, burning 3 empty-commit retries without ever invoking the LLM fix path.
+# Must now classify as None (code bug).
+PR714_FOXDEN_RATING_QUOTING_LOG_PATH = Path(
+    "utils/logs/fixtures/foxden_rating_quoting_pr714_circleci_log.txt"
+)
+
 NORMAL_TEST_FAILURE_LOG = """\
 PHPUnit 9.6.27 by Sebastian Bergmann and contributors.
 
@@ -123,3 +132,11 @@ def test_detect_infra_failure_matches(error_log, expected):
 def test_detect_infra_failure_no_match(error_log, expected):
     result = detect_infra_failure(error_log)
     assert result == expected
+
+
+def test_strip_jest_noise_removes_app_level_access_denied_warn():
+    # Real PR 714 CircleCI log: app-level console.warn at the top prints "AccessDeniedException" from an SSM fetch that fell back to a default.
+    # strip_jest_noise removes that console block inside detect_infra_failure, so the spurious match no longer routes the PR to the infra-retry path.
+    # Jest still ran and reported real assertion failures further down; classifier must return None so the LLM fix path handles them.
+    real_log = PR714_FOXDEN_RATING_QUOTING_LOG_PATH.read_text(encoding="utf-8")
+    assert detect_infra_failure(real_log) is None