Reconstruct the change
Read the request, diff, surrounding code, call sites, types, tests, configuration, and history needed to understand intended behavior.
Codex Skills for Code Review give Codex a systematic way to understand a change, trace affected behavior, find concrete defects, evaluate tests and security, and write useful review comments. Download the skills for pull requests, patches, refactors, migrations, and pre-merge checks.
task: review payment retry change
inspect:
- change intent
- call sites
- test coverage
verify: findings ranked by impactUseful code review starts by understanding what the change intends to do and which assumptions surround it. The reviewer then follows data, control flow, interfaces, dependencies, and tests beyond the lines shown in the patch.
These skills help Codex prioritize actionable defects over style noise, attach findings to specific evidence, explain impact, and avoid claiming problems that the code does not support.
Read the request, diff, surrounding code, call sites, types, tests, configuration, and history needed to understand intended behavior.
Check boundary cases, state transitions, concurrency, data loss, error handling, compatibility, migrations, resource cleanup, and unintended side effects.
Review trust boundaries, authorization, validation, secrets, logging, dependency changes, injection paths, denial cases, timeouts, and retries.
Rank issues by impact, cite the exact location, describe a realistic failure, separate blockers from suggestions, and note validation gaps.
The process narrows attention toward risks introduced by the change and keeps comments specific enough for the author to act on.
Read the issue and pull request description, identify expected behavior, locate sensitive areas, and note what the change claims not to affect.
Follow callers, data structures, persistence, permissions, error paths, integrations, configuration, and platform-specific behavior.
Test boundaries, malformed input, repeated operations, partial failure, concurrency, null states, migration order, and backward compatibility.
List concrete findings first with severity and evidence, then unanswered questions, test gaps, and a short overall assessment.
A dependency update, schema migration, UI change, and authentication patch require different lines of inquiry.
Review a branch against its base, understand the stated goal, inspect checks, and produce focused inline or summary feedback.
Examine authentication, authorization, trust boundaries, data exposure, validation, dependencies, secrets, and audit events.
Check data compatibility, rollout order, rollback behavior, old callers, invariant preservation, and regression coverage.
Inspect a collected change set for integration risks, configuration gaps, missing tests, deployment concerns, and release blockers.
The review can target one commit, a full branch, a pull request diff, or selected high-risk files.
These skills support reviewers who want clear findings and authors who want a strong self-review before asking teammates.
Trace behavior beyond the diff and focus comments on defects, regressions, security, and maintainability risks.
Run a structured self-review, remove accidental changes, strengthen tests, and provide better context before requesting review.
Apply consistent severity, evidence, security, and quality expectations across repositories and reviewers.
Add code-aware checks around access control, input boundaries, sensitive data, dependencies, logging, and abuse cases.
The skills provide a second set of eyes, not an approval guarantee. They may miss defects and should not replace responsible human review, automated checks, security testing, or specialist assessment.
Install the skills with the repository so Codex can use its architecture, contribution rules, tests, and severity definitions.
Keep SKILL.md with its review stages, risk checklists, severity guidance, and supporting references.
Use project scope for repository-specific rules, or personal scope for a consistent review method across projects.
Record critical domains, trust boundaries, compatibility promises, performance limits, code owners, severity definitions, and ignored generated files.
Specify the base branch or commit plus approved build, test, analysis, dependency, and security checks.
Supply the issue, acceptance criteria, pull request description, rollout plan, and any areas where the author wants deeper review.
Answers about pull requests, false positives, security, large diffs, and suggested fixes.
Yes. They work best with the diff, the intended behavior, and access to surrounding code and tests.
No review tool is perfect. The workflow requires evidence and realistic impact, but the author or reviewer should confirm each finding before acting.
Only when style affects correctness, clarity, consistency, or stated repository rules. Concrete defects and meaningful risks take priority over preferences.
They can inspect many code-level security risks, but they do not replace threat modeling, penetration testing, dependency scanning, or specialist security assessment.
The workflow maps the change first, identifies high-risk paths, reviews file groups in a sensible order, and states any areas that could not be checked deeply.
Understand intent. Trace impact. Support every finding. Keep noise out of the review.