Fix Codex runner smoke compatibility - #549
Conversation
Code Review ✅ ApprovedRemoves stale Codex smoke flags from the macOS runner auth probe and updates documentation with the supported OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
Claude audit (merge-authority lane)Head SHA: Claude Audit: PASS Summary: Removes stale unsupported codex exec flags and adds documented API-key login path; changes are small, mirrored between code and docs, and covered by new tests. No blocking issues found. Findings:
|
Part of #536. Fixes #541.
Summary
codex login --with-api-keyVerification
scripts/dev-python -m unittest tests.test_doctor_self_hosted_runner.SelfHostedRunnerDoctorTests.test_runner_cli_auth_codex_probe_omits_stale_smoke_flags tests.test_release_hygiene.ReleaseHygieneTests.test_codex_smoke_docs_use_supported_flagsscripts/dev-python -m pytest tests/test_doctor_self_hosted_runner.py tests/test_release_hygiene.pyscripts/dev-python -m ruff check src/code_mower/doctor_checks/self_hosted_runner.py tests/test_doctor_self_hosted_runner.py tests/test_release_hygiene.pyscripts/dev-python -m ruff check .scripts/dev-python -m pytestgit diff --checkscripts/dev-python scripts/privacy_scan.pycodex exec --skip-git-repo-check --sandbox read-only --output-last-message ... "Reply with exactly: ok"returned the expected sentinelPrivacy
Metadata/docs/probe-command only. No source, diffs, transcripts, raw stdout/stderr, auth output, local paths, or secrets are uploaded or persisted by this change.