Skip to content

chore(seidroid): take uci v0.0.22 - #559

Merged
bdchatham merged 1 commit into
mainfrom
chore/take-uci-v0-0-22
Sep 15, 2026
Merged

bdchatham merged 1 commit into
mainfrom
chore/take-uci-v0-0-22

Conversation

@bdchatham

Copy link
Copy Markdown
Collaborator

What this is

Repoints this repository's seidroid review caller from a raw commit sha to the uci tag v0.0.22.

seidroid-review.yml landed on uci's main in sei-protocol/uci#108 and shipped as v0.0.22, so for the first time there is a tag to name. Until now every caller pinned a sha on a branch that had to be kept alive by hand.

What changes here

The three uses: lines, and nothing else. No inputs change.

This caller omits driver-version, so it inherits the workflow default, which v0.0.22 sets to sei-agent-driver/v0.21.0. Two things follow from that:

  • The review summary comes first, entries are one or two sentences, and nits are folded into a collapsed block (PLT-1268).
  • The verdict rides in the review that carries the inline comments, instead of a separate comment beside it. A standalone comment is now only the fallback for a run where that review did not land.

Known, non-blocking

Four items are tracked for the next uci release and listed in the release notes — the most visible is that a head which moves mid-review still earns a 👍 on the trigger comment, even though the run correctly declines to approve or to clear a block. None gates a merge.

mode in the concurrency group lets a review and a close run together for one pull request. The trade-off is deliberate and stated in the callee; PLT-1274 tracks the driver-side question.

🤖 Generated with Claude Code

@cursor

cursor Bot commented Sep 15, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Only CI workflow references change; risk is limited to Seidroid review UX and automation behavior from the inherited driver defaults.

Overview
Repoints all three Seidroid reusable-workflow jobs (seidroid-review, seidroid-review-close, seidroid-review-reclaim) from a pinned commit on sei-protocol/uci to the v0.0.22 tag. Workflow inputs, secrets, and permissions are unchanged.

Because this caller does not set driver-version, the bump also picks up the v0.0.22 default driver (sei-agent-driver/v0.21.0), which changes review output behavior: summaries lead with a shorter format, nits collapse into a block, and the verdict is attached to the review that carries inline comments rather than a separate comment when that review posts successfully.

Reviewed by Cursor Bugbot for commit 2707fe8. Bugbot is set up for automated code reviews on this repo. Configure here.

@bdchatham
bdchatham merged commit 7e44e78 into main Sep 15, 2026
12 of 13 checks passed
@seidroid

seidroid Bot commented Sep 15, 2026

Copy link
Copy Markdown

This review did not complete. It reached no verdict, so there is no review of this change to act on.

This review produced no decision that could be read mechanically.

the session reported the turn as failed

The agent's own reply is not published: a reply this driver cannot attribute to its own turn is one it must not repeat here. This run's log names the session that produced it.

Read this run for the rest: https://github.com/sei-protocol/sei-k8s-controller/actions/runs/34958182432

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