diff --git a/.github/workflows/reusable-agents-verifier.yml b/.github/workflows/reusable-agents-verifier.yml index 11e80c67d..6bd7d8465 100644 --- a/.github/workflows/reusable-agents-verifier.yml +++ b/.github/workflows/reusable-agents-verifier.yml @@ -487,7 +487,7 @@ jobs: if: steps.context.outputs.should_run == 'true' && inputs.mode != 'evaluate' run: | set -euo pipefail - npm install -g "@openai/codex@0.101.0" + npm install -g "@openai/codex@0.125.0" echo "Installed Codex CLI version: $(codex --version)" >> "$GITHUB_STEP_SUMMARY" - name: Resolve Codex verifier model diff --git a/scripts/repo_review_evaluator.py b/scripts/repo_review_evaluator.py index c4472dc20..5b75ff755 100644 --- a/scripts/repo_review_evaluator.py +++ b/scripts/repo_review_evaluator.py @@ -972,11 +972,7 @@ def write_packet(output_dir: Path, states: list[dict[str, Any]], generated_on: s review_pending = [ state for state in active if state["review_status"] == "pending standardized review" ] - blocked = [ - state - for state in active - if state["review_status"] != "pending standardized review" - ] + blocked = [state for state in active if state["review_status"] != "pending standardized review"] issue_candidate_repos = [ state for state in active if state["issue_queue_status"] == "draft candidates present" ] diff --git a/tests/scripts/test_repo_review_evaluator.py b/tests/scripts/test_repo_review_evaluator.py index d319eefed..4f44d6d4e 100644 --- a/tests/scripts/test_repo_review_evaluator.py +++ b/tests/scripts/test_repo_review_evaluator.py @@ -193,7 +193,9 @@ def test_collect_repo_state_marks_clean_active_repo_review_pending(tmp_path: Pat (repo_dir / "README.md").write_text("# Manager\n", encoding="utf-8") subprocess = evaluator.subprocess subprocess.run(["git", "-C", str(repo_dir), "init"], check=True, capture_output=True) - subprocess.run(["git", "-C", str(repo_dir), "config", "user.email", "test@example.com"], check=True) + subprocess.run( + ["git", "-C", str(repo_dir), "config", "user.email", "test@example.com"], check=True + ) subprocess.run(["git", "-C", str(repo_dir), "config", "user.name", "Test User"], check=True) subprocess.run(["git", "-C", str(repo_dir), "add", "README.md"], check=True) subprocess.run( @@ -223,7 +225,9 @@ def test_issues_txt_changes_are_helper_inputs_not_review_blockers(tmp_path: Path (repo_dir / "Issues.txt").write_text("# helper\n", encoding="utf-8") subprocess = evaluator.subprocess subprocess.run(["git", "-C", str(repo_dir), "init"], check=True, capture_output=True) - subprocess.run(["git", "-C", str(repo_dir), "config", "user.email", "test@example.com"], check=True) + subprocess.run( + ["git", "-C", str(repo_dir), "config", "user.email", "test@example.com"], check=True + ) subprocess.run(["git", "-C", str(repo_dir), "config", "user.name", "Test User"], check=True) subprocess.run(["git", "-C", str(repo_dir), "add", "README.md", "Issues.txt"], check=True) subprocess.run( diff --git a/tests/workflows/test_verifier_terminal_disposition.py b/tests/workflows/test_verifier_terminal_disposition.py index be0235238..ba30a87bf 100644 --- a/tests/workflows/test_verifier_terminal_disposition.py +++ b/tests/workflows/test_verifier_terminal_disposition.py @@ -16,6 +16,7 @@ def test_reusable_verifier_uploads_terminal_disposition_artifact() -> None: resolve_step = next( step for step in steps if step.get("name") == "Resolve Codex verifier model" ) + install_step = next(step for step in steps if step.get("name") == "Install Codex CLI") run_step = next(step for step in steps if step.get("name") == "Run verifier (checkbox mode)") collect_step = next(step for step in steps if step.get("name") == "Collect verifier metrics") write_step = next( @@ -30,6 +31,7 @@ def test_reusable_verifier_uploads_terminal_disposition_artifact() -> None: resolve_step.get("if") == "steps.context.outputs.should_run == 'true' && inputs.mode != 'evaluate'" ) + assert 'npm install -g "@openai/codex@0.125.0"' in install_step["run"] assert resolve_step["env"]["DEFAULT_CODEX_MODEL"] == "gpt-5.5" assert resolve_step["env"]["FALLBACK_CODEX_MODELS"] == "gpt-5.4 gpt-5.3-codex" assert resolve_step["env"]["VERIFIER_MODE"] == "${{ inputs.mode }}"