Skip to content

ci: Sync claude-review workflow with systemd - #4479

Merged
behrmann merged 1 commit into
systemd:mainfrom
daandemeyer:push-wourusopzqvl
Oct 8, 2026
Merged

behrmann merged 1 commit into
systemd:mainfrom
daandemeyer:push-wourusopzqvl

Conversation

@daandemeyer

Copy link
Copy Markdown
Contributor

The claude-review workflow still runs a coordinating agent that spawns one
review subagent per lens. systemd replaced that with one bounded Claude run
per lens in parallel, followed by a run that validates the candidate
findings. Each completed review raises the severity bar for new inline
comments, and re-reviews focus on the changes since the last reviewed head.
The review runs on Opus 5.5 from a pinned Claude Code release with a
checked checksum. Port that rework.

The lens descriptions are rewritten for mkosi. The check that skips reviews
while README.md contains the AI review notice is dropped, because mkosi has
no such notice.

Signed-off-by: Daan De Meyer daan@amutable.com

The claude-review workflow still runs a coordinating agent that spawns one
review subagent per lens. systemd replaced that with one bounded Claude run
per lens in parallel, followed by a run that validates the candidate
findings. Each completed review raises the severity bar for new inline
comments, and re-reviews focus on the changes since the last reviewed head.
The review runs on Opus 5.5 from a pinned Claude Code release with a
checked checksum. Port that rework.

The lens descriptions are rewritten for mkosi. The check that skips reviews
while README.md contains the AI review notice is dropped, because mkosi has
no such notice.

Signed-off-by: Daan De Meyer <daan@amutable.com>
@behrmann

behrmann commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

No point in wasting CI time for this, it doesn't affect any currently running workflow.

@behrmann
behrmann merged commit 2ca0d9c into systemd:main Oct 8, 2026
37 of 53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants