Review Flow
Use this skill to keep reviews concise, severity-driven, and aligned with repository rules.
Goals
- Lead with the most serious problem first.
- Keep findings proportional to user or maintainer impact.
- Separate repository contract issues from style preferences.
Workflow
- Identify the touched contract surface: API, schema, MCP behavior, UI, release flow, or workflow rules.
- Check whether validation depth matches the blast radius.
- Report findings in severity order, or say clearly when no findings remain.
- Keep merge-readiness notes narrow: unresolved risk, missing validation, or stale workflow state.
- Treat "merge-ready" as a recommendation for Loocor/Codex review unless Loocor explicitly asks you to merge the PR in the current session.
- Prefer Codex Review (
@codex reviewon the PR) over local-only signoff; do not treat Copilot as the default review path.
When to read more
- Read
references/review-rubric.mdwhen the review needs the full decision and reporting rubric. - Cross-reference the
validationskill when a finding depends on missing or weak validation evidence.