code-review-expertlisted
Install: claude install-skill rocky2431/ultra-builder-pro-cli
# Review a bounded code change
Prioritize defects that can change behavior, safety, operability, or delivery. Do not
fill the report with style preferences or generic best-practice commentary.
## Workflow
1. Establish the exact diff, stated intent, acceptance evidence, current checkout, and
relevant repository guidance. If the scope is empty or ambiguous, report that
boundary instead of reviewing unrelated history.
2. Trace changed behavior from a live entry point through state and side effects to a
consumer. Read callers, contracts, tests, and error paths needed to prove or refute
a finding.
3. Load focused references only when their scope is present:
- `security-rules` for trust or authorization boundaries;
- `integration-rules` for cross-component behavior;
- `references/correctness-reliability.md` for a concrete behavioral risk;
- `references/design-boundaries.md` for a concrete ownership or coupling problem;
- `references/removal-plan.md` for code proposed for deletion.
4. For every candidate finding, state the violated contract, triggering condition,
user or system impact, and evidence in the current source. Discard speculative
concerns that lack a plausible execution path.
5. Assign severity from impact and likelihood:
- P0: exploitable security issue, data loss, or deterministic critical failure;
- P1: material correctness, authorization, reliability, or delivery failure;
- P2: bounded maintainability or edge-case defect wi