Repository navigation
ci: add seidroid review workflow - #55
Conversation
Wire the seidroid reviewer the same way sei-k8s-controller does: an automatic review on pull request events, `@seidroid review` / `@seidroid review close` on comment, and a reclaim of the sandbox when the pull request closes. Reviews against CLAUDE.md, since this repository keeps no REVIEW.md. Refs PLT-1554 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
PR SummaryLow Risk Overview Review runs on PR open/update/reopen and on Reviewed by Cursor Bugbot for commit abddd81. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Adds a thin caller workflow that wires automatic and comment-triggered seidroid review, plus session close and reclaim, to the pinned sei-protocol/uci reusable workflow at v0.0.27. The triggers, permissions and secret passing match what the pinned workflow expects, and its guard still enforces the team, fork and draft checks, so nothing blocks; codex's one finding was checked and dropped because the reusable workflow never opens a session for a fork PR, so there is nothing left to reclaim.
1 nit, not posted on the code
.github/workflows/seidroid.yml:67— This comment says to run@seidroid review closeon a fork PR before merging, and the PR description says the same. That step should never be needed: the pinned uci guard refuses to review a fork-originated PR on both the automatic and the comment path, so no sandbox is ever created to leave running. The only fork-related effect is a failed reclaim run, which has nothing to clean up.
seidroid review · decision approve · session ad61d92acb8542febbf46c84a52ace18 · turn resp_claude_ce85a2d45c583334a1cbcef47f9f8905 · item fd4448d5be2c54d299642dfb243ed67e
Findings: 0 blocking | 0 non-blocking | 0 posted inline
Resolves PLT-1554.
Adds
.github/workflows/seidroid.yml. It's copied unchanged fromsei-k8s-controllerand callssei-protocol/uci/.github/workflows/seidroid-review.yml, pinned at v0.0.27. It sets up three jobs:@seidroid reviewcomment. It reviews againstCLAUDE.mdand approves when the result is clean.@seidroid review closecomment tears down the PR's session.Commenting is limited to
sei-protocol/sei-core, and Dependabot runs are skipped.Before this works
PLATFORM_CODE_AGENT_APP_IDandPLATFORM_CODE_AGENT_APP_PKvisible to sei-config.OMNIGENT_MACHINE_CLIENT_SECRETas a repo secret.Until these are done, the seidroid jobs on this PR will fail.