Skip to content

v0.0.22

Latest

Choose a tag to compare

@bdchatham bdchatham released this 15 Sep 10:26
4bd0b78

What's Changed

  • feat(seidroid-review): add the agentic pull request reviewer as a reusable workflow by @bdchatham in #108

Full Changelog: v0.0.21...v0.0.22


seidroid-review.yml

The agentic pull request reviewer, as a reusable workflow. It reviews a pull request from inside the Actions runner and posts inline comments on the changed lines, a check run, and a review position. One file, and it needs no other file in this repository.

A caller wires the triggers and calls it with uses::

uses: sei-protocol/uci/.github/workflows/seidroid-review.yml@v0.0.22

Driver floor is sei-agent-driver/v0.21.0, in both the driver-version default and MIN_DRIVER_VERSION. A caller that omits the input gets that release. The floor and the default name one version on purpose: the conclusion a review reaches for a given set of findings is specific to the driver that reached it.

Migrating

It replaces ai-review.yml rather than running beside it. A repository that wires the automatic path here should retire that caller in the same change, or every pull request is reviewed twice.

Callers currently pinning the commit 81e975c should move to this tag.

Known, non-blocking

Tracked for the next release:

  • On a head that moved mid-review the workflow declines to approve and declines to clear a block, but the trigger comment still receives a 👍.
  • On most paths nothing on the pull request says the head moved — the warning is in the run log only.
  • One comment in the file claims a wrong approval is corrected by the next review. It is not, when re-review-on-push is off and no block stands.

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 file; whether a close deleting a session mid-review is safe is tracked as PLT-1274.