Repository navigation
feat(seidroid-review): drive sei-agent-driver v0.22.0, which waits out a server restart - #112
Conversation
…t a server restart Move the driver-version default and MIN_DRIVER_VERSION to v0.22.0 together, as the file requires. v0.22.0 waits out a server restart instead of ending a live review on the first gateway 503 in the rollout window (sei-protocol/sei-internal-skills#458). The floor's rationale now names it as the third load-bearing release. Verified that v0.22.0 installs from an empty module cache: `go install …/sei-agent-driver/cmd/sei-agent-driver@v0.22.0` reports "sei-agent-driver version v0.22.0". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
PR SummaryLow Risk Overview The inline rationale for the minimum version is extended: v0.22.0 is documented as a third load-bearing release because it retries transient gateway 502/503/504 (no error envelope) during omnigent’s ~two-minute single-replica restart window, instead of ending the review with transport failure (exit 6) and forcing a manual re-run. Reviewed by Cursor Bugbot for commit 1b998a5. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
This PR changes the driver-version default and MIN_DRIVER_VERSION together from v0.21.0 to v0.22.0. It also updates the input's format example and the comment explaining the minimum version. No issues were found; the other version references left in the file describe past releases and don't need to change.
Findings: 0 blocking | 0 non-blocking | 0 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- None at the file/PR level.
…2.0 (#460) Bumps the three `seidroid-review.yml` pins in `.github/workflows/seidroid.yml` to **uci v0.0.23**. The pin style is unchanged. uci v0.0.23 (sei-protocol/uci#112) moves the review driver to **sei-agent-driver v0.22.0** (#458), which waits out an omnigent server restart. Before it, a review whose turn overlapped a server rollout failed with exit 6, "the transport to the agent failed before a reply arrived", on the first gateway 503 in the window. The review's sandbox and turn survived, and a person had to re-run the review. sei-protocol/platform#1831 failed this way on 2026-09-28. This PR's own seidroid review runs on the new pin, so it exercises v0.0.23 and driver v0.22.0 end to end. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…2.0 (#259) Bumps the three `seidroid-review.yml` pins in `.github/workflows/seidroid.yml` to **uci v0.0.23**. The pin style is unchanged. uci v0.0.23 (sei-protocol/uci#112) moves the review driver to **sei-agent-driver v0.22.0** (sei-protocol/sei-internal-skills#458), which waits out an omnigent server restart. Before it, a review whose turn overlapped a server rollout failed with exit 6, "the transport to the agent failed before a reply arrived", on the first gateway 503 in the window. The review's sandbox and turn survived, and a person had to re-run the review. sei-protocol/platform#1831 failed this way on 2026-09-28. This PR's own seidroid review runs on the new pin, so it exercises v0.0.23 and driver v0.22.0 end to end. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Summary
Moves
seidroid-review.ymlto sei-agent-driver v0.22.0. Both thedriver-versiondefault andMIN_DRIVER_VERSIONmove together, as the file requires.v0.22.0 (sei-protocol/sei-internal-skills#458) waits out a server restart. Today, a review whose turn overlaps an omnigent server rollout fails with exit 6: "the transport to the agent failed before a reply arrived". The server runs one replica with
strategy: Recreate, so it has no server for about two minutes per rollout. The driver's first session read in that window gets the gateway's503 no healthy upstreamand ends the run, although the sandbox and the turn survive. That is how sei-protocol/platform#1831 failed on 2026-09-28.The driver now retries a 502, 503 or 504 that carries no error envelope for up to
SEIDROID_SERVER_RESTART_BUDGET_S(180 s). A 503 the server writes itself, such asrunner_unavailable, stays final. A run deadline or cancellation during the wait reports a timeout or a cancellation, not exit 6.Changes
driver-versiondefault:v0.21.0→v0.22.0MIN_DRIVER_VERSION:v0.21.0→v0.22.0, and its rationale names v0.22.0 as the third load-bearing release.v0.22.0.Verification
go install github.com/sei-protocol/sei-internal-skills/sei-agent-driver/cmd/sei-agent-driver@v0.22.0from an empty module cache:sei-agent-driver version v0.22.0, module hashh1:TVczpvLS87KqJwSaZ6MoNgTN8x3aciQMCpV7wBJetNE=.actionlint: 6 findings, the same 6 as onmain, none on the changed lines.After merge
Cut uci v0.0.23, then bump the
@<sha> # v0.0.22pin in the callers: platform, sei-internal-skills, sei-k8s-controller, sei-load and seictl.🤖 Generated with Claude Code