Skip to content

feat(seidroid-review): drive sei-agent-driver v0.22.0, which waits out a server restart - #112

Merged
bdchatham merged 1 commit into
mainfrom
chore/seidroid-review-driver-v0.22.0
Sep 28, 2026
Merged

bdchatham merged 1 commit into
mainfrom
chore/seidroid-review-driver-v0.22.0

Conversation

@bdchatham

Copy link
Copy Markdown
Contributor

Summary

Moves seidroid-review.yml to sei-agent-driver v0.22.0. Both the driver-version default and MIN_DRIVER_VERSION move 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's 503 no healthy upstream and 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 as runner_unavailable, stays final. A run deadline or cancellation during the wait reports a timeout or a cancellation, not exit 6.

Changes

  • driver-version default: v0.21.0 → v0.22.0
  • MIN_DRIVER_VERSION: v0.21.0 → v0.22.0, and its rationale names v0.22.0 as the third load-bearing release.
  • The input's format example now names v0.22.0.

Verification

  • go install github.com/sei-protocol/sei-internal-skills/sei-agent-driver/cmd/sei-agent-driver@v0.22.0 from an empty module cache: sei-agent-driver version v0.22.0, module hash h1:TVczpvLS87KqJwSaZ6MoNgTN8x3aciQMCpV7wBJetNE=.
  • The workflow YAML parses. actionlint: 6 findings, the same 6 as on main, none on the changed lines.

After merge

Cut uci v0.0.23, then bump the @<sha> # v0.0.22 pin in the callers: platform, sei-internal-skills, sei-k8s-controller, sei-load and seictl.

🤖 Generated with Claude Code

…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>
@cursor

cursor Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Workflow-only pin and comment updates; behavior change lives in the external driver version callers will install.

Overview
Bumps the seidroid-review workflow to sei-agent-driver v0.22.0 by moving both the driver-version input default and MIN_DRIVER_VERSION from v0.21.0 to v0.22.0, including the install example in the input description.

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.

@seidroid seidroid Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@bdchatham
bdchatham merged commit 5675a17 into main Sep 28, 2026
11 checks passed
@bdchatham
bdchatham deleted the chore/seidroid-review-driver-v0.22.0 branch September 28, 2026 17:26
bdchatham added a commit to sei-protocol/sei-internal-skills that referenced this pull request Sep 28, 2026
…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>
bdchatham added a commit to sei-protocol/seictl that referenced this pull request Sep 28, 2026
…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>
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