PULL REQUEST REVIEW
Review the change against the whole codebase.
Corbulo checks the intent, affected callers and APIs, architecture zones, repository conventions, parallel work, and bugs introduced by the change before returning a merge decision.
INTENT, SCOPE, AND RISK
Understand the change before judging the diff
Corbulo identifies the feature being added, the backend and frontend work involved, the files changed, the blast radius, and the security-sensitive path.
BLOCKING FINDING
A merge block with the code behind it
The review catches an authorization predicate that requires a user to be both team admin and team owner. It shows the changed lines, the valid users who are denied, and the one-operator fix.
WHAT THE REVIEW COVERS
What Corbulo checks before the merge decision.
Intent before impact
Classifies the primary intent, secondary intents, change categories, and risk signals before judging the diff.
Callers and breaking APIs
Follows changed symbols into callers, affected paths, cross-repository consumers, and API surfaces that may break.
Architecture zones
Shows the zones and owners touched by the change, whether the scope fits, and which files fall outside known boundaries.
Findings anchored to the diff
Puts security, runtime, testing, and maintainability findings on the lines that introduced them, with the failure path and a way to fix it.
Conflicts and stacked work
Detects overlapping files, semantic conflicts, parent and child PRs, and the merge order needed to keep the stack coherent.
Projected code health
Shows which health categories move, how many issues are added or fixed, and why the PR is ready, needs review, or blocked.
Convention drift, hotspots, bug magnets, co-change warnings, the full diff, CI checks, GitHub reviews, and discussion stay with the same review.