Skip to content

Add a review action that reviews one PR with the Banzai harness#210

Open
petergam wants to merge 3 commits into
mainfrom
add-review-action
Open

Add a review action that reviews one PR with the Banzai harness#210
petergam wants to merge 3 commits into
mainfrom
add-review-action

Conversation

@petergam

Copy link
Copy Markdown

What

Adds the review composite action — a sibling of the harness action that reviews a single pull request with the Codex agent. Backed by the new banzai-harness review subcommand (PR: framna-dk/banzai-codes-harness#19).

How it works

On a pull_request event it shells banzai-harness preflight --pr-number … then banzai-harness review …, which boots Codex on the PR head, reads the diff, and posts a concise static review: 👀 reaction, inline comments, and one summary (COMMENT/REQUEST_CHANGES, never APPROVE). No file edits or commits, so it never re-triggers itself.

Runs on the same self-hosted pool as harness (banzai-harness, codex, gh, git on PATH). Secrets (GH_TOKEN, OPENAI_API_KEY) are read from the environment.

Notes

  • Replaces the external `@codex review` bot; the harness PR drops the re-ping fallback from its workflow prompt since the review now runs automatically on every push.
  • Example workflow with PR trigger + per-PR `concurrency` is in review/README.md.
  • Independent of the unmerged harness-action PR (WIP: Harness #201) — different files; this branches off `main`.

🤖 Generated with Claude Code

A thin composite over `banzai-harness review` (sibling of the harness
action): on a PR event it boots Codex on the PR head, reads the diff, and
posts a concise static review (inline comments + one summary, never
APPROVE). Runs on the same self-hosted pool; secrets come from the env.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
petergam and others added 2 commits June 15, 2026 15:31
Don't name the underlying coding agent (so it can change later) and stop
documenting an agent API token in the action — the runner provides the
review agent's credentials. Only GH_TOKEN is supplied by the caller.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The review command requires the repo slug; validating it at preflight (now
that it's a preflight input) fails fast instead of failing in the run step.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant