Upgrade UCI AI review to latest and allow review of PRs from seidroid - #4059
Conversation
Upgrade to the latest UCI version for AI review and comments, which most notably: reduces cancellation noise further, and optionally allows review of PRs created by non-humans. This repo will only accept PRs created by `seidroid`. Note that the only CI workflow that allows seidroid to create PRs is auto backporting workflow.
PR SummaryLow Risk Overview On AI Review, adds Reviewed by Cursor Bugbot for commit d8f7635. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
There was a problem hiding this comment.
Straightforward bump of the pinned UCI reusable-workflow SHA from v0.0.19 to v0.0.20 plus a new allowed-bots: '[\"seidroid[bot]\"]' input on the AI review workflow; the SHA is applied consistently across both uses: and uci-ref: in both files, and scoping the bot allowlist to seidroid[bot] matches the repo's only bot PR source (the UCI backport workflow, whose PRs contain code already merged to main). One comment-only typo (v0.0.220) is the sole finding.
Findings: 0 blocking | 1 non-blocking | 1 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- None at the file/PR level.
- 1 suggestion(s)/nit(s) flagged inline on specific lines.
| with: | ||
| # See: https://github.com/sei-protocol/uci/releases/tag/v0.0.19 | ||
| uci-ref: eb65d6d3f967c661f6af91cfac599fd355445f71 | ||
| # See: https://github.com/sei-protocol/uci/releases/tag/v0.0.220 |
There was a problem hiding this comment.
[suggestion] Typo in the release-tag reference: v0.0.220 should be v0.0.20. Every other occurrence in this PR (line 12 here, and both lines in ai-assist.yml) says v0.0.20, and https://github.com/sei-protocol/uci/releases/tag/v0.0.220 is a dead link. It's comment-only so nothing breaks at runtime, but these comments are the only human-readable mapping from the pinned SHA to a release, so a wrong one defeats the purpose of the pin annotation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4059 +/- ##
==========================================
- Coverage 61.33% 60.27% -1.06%
==========================================
Files 2157 2055 -102
Lines 188986 176894 -12092
==========================================
- Hits 115918 106628 -9290
+ Misses 62235 60421 -1814
+ Partials 10833 9845 -988
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Successfully created backport PR for |
Upgrade to the latest UCI version for AI review and comments, which most notably: reduces cancellation noise further, and optionally allows review of PRs created by non-humans.
This repo will only accept PRs created by
seidroid. Note that the only CI workflow that allows seidroid to create PRs is auto backporting workflow.