Skip to content

refactor: simplify PR review to primary verifier flow - #8

Merged
jaibhasin merged 1 commit into
mainfrom
jaibhasin/analyze-slow-pr-reviews
Jul 27, 2026
Merged

refactor: simplify PR review to primary verifier flow#8
jaibhasin merged 1 commit into
mainfrom
jaibhasin/analyze-slow-pr-reviews

Conversation

@jaibhasin

@jaibhasin jaibhasin commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

Replaces router-driven specialist fan-out with one primary reviewer across selected bundles, followed by the existing verifier.
Removes obsolete specialist routing and task code, and reports a single Precision review status.
Preserves AGENTS.md prompt and tool protections, while updating tests, Action metadata, README, and the future-work checklist.
Validated with cargo fmt, cargo clippy, and 56 passing tests.

Summary by CodeRabbit

  • New Features

    • Introduced a precision-first review workflow that evaluates all selected changes together.
    • Added bounded, read-only repository inspection for more focused reviews.
    • Prevented review tools from accessing agent-instruction files.
  • Improvements

    • Consolidated results into a single Precision review section.
    • Updated configuration and action descriptions to reflect the primary review model.
    • Added guidance for future review optimizations and evaluation.
  • Documentation

    • Revised architecture, configuration, output, and source-boundary documentation.

@cursor

cursor Bot commented Jul 27, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2ffd2235-d0be-403e-a0ff-a75d77aaeb50

📥 Commits

Reviewing files that changed from the base of the PR and between 5634190 and 750e1bf.

📒 Files selected for processing (22)
  • README.md
  • action.yml
  • docs/research/future_checklist.md
  • src/agents/integration_tests.rs
  • src/agents/mod.rs
  • src/agents/prompts/mod.rs
  • src/agents/prompts/primary.rs
  • src/agents/prompts/router.rs
  • src/agents/prompts/specialists.rs
  • src/agents/router.rs
  • src/agents/tasks.rs
  • src/main.rs
  • src/reporting/anchors.rs
  • src/reporting/summary.rs
  • src/reporting/summary_tests.rs
  • src/repository/mod.rs
  • src/repository/tests.rs
  • src/repository/tools.rs
  • src/review/contextual.rs
  • src/review/contextual_tests.rs
  • src/review/legacy.rs
  • src/types.rs

📝 Walkthrough

Walkthrough

Changes

Primary Reviewer Consolidation

Layer / File(s) Summary
Primary review contracts and boundaries
src/types.rs, src/agents/prompts/*, src/repository/*
Review execution now uses one Primary agent, a dedicated prompt, and reviewer tool validation that blocks direct AGENTS.md access.
Single-review execution and verification
src/agents/*, src/review/*
Selected bundles are submitted in one primary-review request, findings are normalized and verified independently, and failures or empty selections update run status accordingly.
Precision reporting and validation
src/reporting/*, src/review/contextual_tests.rs
Output now contains a single Precision review section, with tests updated for the Primary agent and removed router-fallback state.
Product and workflow documentation
README.md, action.yml, src/main.rs, docs/research/*
Project descriptions, configuration text, CLI help, review-output behavior, and future workflow notes describe the primary-review architecture.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant ReviewBundles
  participant PrimaryReviewer
  participant Verifier
  participant ReviewPublisher
  ReviewBundles->>PrimaryReviewer: submit selected bundles
  PrimaryReviewer-->>ReviewBundles: return candidate findings
  ReviewBundles->>Verifier: verify findings
  Verifier-->>ReviewPublisher: provide verified findings
  ReviewPublisher-->>ReviewPublisher: render Precision review
Loading

Possibly related PRs

  • jaibhasin/PRBot#3: Directly overlaps with review orchestration, contrasting this single-primary flow with a multi-agent contextual pipeline.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jaibhasin/analyze-slow-pr-reviews

Comment @coderabbitai help to get the list of available commands.

@jaibhasin
jaibhasin merged commit 493f7e0 into main Jul 27, 2026
3 of 4 checks passed
@jaibhasin
jaibhasin deleted the jaibhasin/analyze-slow-pr-reviews branch July 27, 2026 21:22
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