providing-code-review

What problem does it solve?

This Skill prevents superficial or rubber-stamp pull request reviews by enforcing engineering rigor: tracing data flow, checking blast radius, verifying integration boundaries, and validating factual claims before approval.

Core Features & Use Cases

• Data flow analysis: Trace inputs and outputs for changed functions and new fields to ensure values propagate through all intermediaries.
• Blast radius verification: Search the entire repository for callers, detect renames/moves/deletes, and ensure all consumers are updated to avoid silent breakage.
• Integration and CI checks: Confirm API and schema compatibility, verify migrations and config changes, and check that CI and test runs are genuinely passing rather than relying on metadata.
• Use case: Use when reviewing a teammate's PR that introduces a new field, API change, or file rename to avoid runtime errors and cross-repo integration failures.

Quick Start

Open the pull request, trace where each changed value originates and flows to, search the repo for external callers, confirm integration points and migrations, ensure CI is actually green, and validate any factual claims before approving.