Skip to content

Add AI-powered PR review workflow - #4529

Draft
kingsleyzissou wants to merge 2 commits into
osbuild:mainfrom
kingsleyzissou:ai-review
Draft

Add AI-powered PR review workflow#4529
kingsleyzissou wants to merge 2 commits into
osbuild:mainfrom
kingsleyzissou:ai-review

Conversation

@kingsleyzissou

@kingsleyzissou kingsleyzissou commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add an AI-powered PR review workflow that runs Claude on Vertex AI to provide automated feedback on pull requests. Reviewers are built from first principles and existing project documentation rather than contributor personas.

Changes

  • Add GitHub Actions workflow that triggers on PRs to main, authenticates with GCP, and runs the review script
  • Implement four parallel reviewers (code quality, UX, security, testing) with project-specific guidelines derived from AGENTS.md and established best practices
  • Post critical findings as inline review comments and an aggregated summary comment (upserted across runs)
  • Include guard rails: skip draft PRs, non-code changes, trivial diffs, and oversized diffs
  • Validate all findings against the parsed diff to filter out hallucinated file paths or lines outside the changeset

Uses Claude on Vertex AI to review PRs with three parallel reviewers
(code quality, UX, security). Findings are posted as inline comments
and an aggregated summary.

Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.90%. Comparing base (9f4afa4) to head (7900b1b).
⚠️ Report is 219 commits behind head on main.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4529   +/-   ##
=======================================
  Coverage   75.90%   75.90%           
=======================================
  Files         225      225           
  Lines        7209     7209           
  Branches     2668     2668           
=======================================
  Hits         5472     5472           
  Misses       1494     1494           
  Partials      243      243           

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f4afa4...7900b1b. Read the comment docs.

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

Reviewer guidelines are derived from first principles and existing
project documentation (AGENTS.md).

Changes:
- Code quality reviewer: add Redux/RTK Query best practices section
  and prefer API-derived types over hand-maintained enums
- Add fourth testing reviewer with Vitest/RTL conventions
- Add dual-environment project context (hosted + Cockpit) to shared
  task prompt so all reviewers are aware
- Add lightweight tone guidance: questions over directives, label
  nitpicks, acknowledge trade-offs

Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it had no activity for the past 30 days. Remove the "Stale" label or add a comment, otherwise this PR will be closed in 7 days.

@github-actions github-actions Bot added Stale and removed Stale labels Jul 13, 2026
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