Repository navigation
test(codex): isolate runtime selection fixture - #7457
Shreyas652 wants to merge 1 commit into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This PR stays in draft until every box above is ticked. |
|
Scoped review at |
|
Carried in #7552 together with the related flaky-test fixes, keeping your commit as its author and a |
Closes #7403.
Summary
HOME,CODEX_HOME,PATH, installed-runtime enumeration, and the bounded--versionprobe before seeding the process memo.Verification
bun scripts/test.ts --parallel=1 ./tests/codex-integration/codex-runtime.test.ts— 59 passed, 0 failed.bun scripts/test.ts --changed=dev --parallel=1— 59 passed, 0 failed.bun x tsc --noEmit— passed.bun scripts/privacy-scan.ts— passed.Checklist
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
Required local validation passed; commands, results, and any full-suite exception are documented.
I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.