What problem does it solve? Reviewing pull requests for architectural risk is time-consuming and inconsistent, and it is easy to miss lifecycle timing bugs, state invariant violations, or integration boundary issues hidden in a diff. This Skill applies a fixed set of architectural review principles to a pull request and produces a calibrated approve, comment, or request-changes decision with concrete inline comments. ## Core Features & Use Cases - Principles-based PR inspection: Reads the pull request description, diff, existing discussion, and CI status, then traces the change through lifecycle timing, state invariants, integration boundaries, and test coverage. - Calibrated decision ladder: Classifies the outcome as approve, approve with notes, comment only, request changes, or needs deeper thought, with explicit rules separating blockers from taste. - Safe GitHub posting: Drafts the review body and inline comments, prints a chat summary, and only posts a pending review or submits it after explicit user confirmation. - Use Case: Given a pull request that claims to fix an end-to-end locator issue, the Skill checks whether CI actually reaches the browser assertion, flags the gap as a blocker, and drafts a request-changes review with the smallest fix. ## Quick Start Ask the assistant to run /ben-architect-review on a specific pull request URL and wait for the drafted review summary before confirming whether to post it as pending or submit it on GitHub.
