Skip to content

Upgrade UCI AI review to latest and allow review of PRs from seidroid - #4059

Merged
masih merged 2 commits into
mainfrom
masih/bum-uci-ai-20
Sep 1, 2026
Merged

Upgrade UCI AI review to latest and allow review of PRs from seidroid#4059
masih merged 2 commits into
mainfrom
masih/bum-uci-ai-20

Conversation

@masih

@masih masih commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

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.

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

cursor Bot commented Aug 31, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CI workflow pin and bot-allowlist only; no application or security-sensitive runtime code changes.

Overview
Bumps the reusable UCI workflows for AI Assistant and AI Review from v0.0.19 to v0.0.20, updating both the uses pin and matching uci-ref to commit 68010d0a.

On AI Review, adds allowed-bots: '["seidroid[bot]"]' so automated backport PRs from seidroid can get AI review under the new UCI behavior (human PRs remain gated by allowed-team).

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

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedAug 31, 2026, 1:01 PM

@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.

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.

Comment thread .github/workflows/ai-review.yml Outdated
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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[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

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.27%. Comparing base (95f6bfb) to head (d8f7635).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            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     
Flag Coverage Δ
sei-db 69.80% <ø> (ø)
sei-db-state-db ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 102 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@masih
masih added this pull request to the merge queue Aug 31, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 31, 2026
@masih
masih added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit 0846501 Sep 1, 2026
73 of 75 checks passed
@masih
masih deleted the masih/bum-uci-ai-20 branch September 1, 2026 10:18
@seidroid

seidroid Bot commented Sep 1, 2026

Copy link
Copy Markdown

Successfully created backport PR for release/v6.7:

masih added a commit that referenced this pull request Sep 1, 2026
…view of PRs from seidroid (#4063)

Backport of #4059 and #4067 to `release/v6.7`.

---------

Co-authored-by: Masih H. Derkani <m@derkani.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants