Skip to content

test: validate Codex PR review bootstrap #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
robbycochran wants to merge 2 commits into
base: main
Choose a base branch
from
Open

test: validate Codex PR review bootstrap #227

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2b45666
docs: clarify Codex review bootstrap
robbycochran Sep 19, 2026
9bb377c
docs: note review artifact diagnostics
robbycochran Sep 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions scripts/review/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,10 @@ not a generic agent-result protocol.
The PR-specific lifecycle remains in `scripts/pr-review.sh`; future agent
removal should delete the corresponding profile and task bundle without
changing PR eligibility, diff integrity, sandbox cleanup, or artifact handling.

The reusable CI workflow enables the Codex profile's ephemeral provider
bootstrap for each review run; local managed workspaces can leave that setup
disabled and use their pre-provisioned providers instead.

The workflow uploads the bounded review directory as an artifact so a failed
run can be diagnosed without exposing provider credentials.
Loading