Senior code review
Skill ArcanEdge-AI/codex-agent-playbook/skills/senior-code-review
Use before finalizing a meaningful code change. Reviews the final diff for correctness, scope control, maintainability, validation gaps, safety risk, performance risk, accessibility risk, and subagent claims that still need verification.From its SKILL.md
npx -y skills add ArcanEdge-AI/codex-agent-playbook --skill senior-code-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
1.2 KB, 219 tokens by cl100k_base, as published. Nobody here has run it
Senior Code Review Skill
Use this skill before finalizing meaningful code changes.
Review the final diff for:
- unrelated changes
- accidental formatting churn
- generated, vendored, compiled, or package-owned files
- missing or weak validation
- unused imports, variables, types, functions, or files caused by the change
- naming clarity
- consistency with existing patterns
- over-abstraction
- speculative configurability
- behavior changes beyond the request
- API compatibility
- migration risk
- safety risk
- performance risk
- accessibility regressions
- subagent claims that were not independently verified
Ask:
Would I approve this in code review?
If not, fix the issue or report the remaining risk clearly.
Final report format:
Summary:
- Changed X to do Y.
Verification:
- Ran: [command or check]
- Result: [passed/failed/not run, with reason]
Notes:
- [Assumptions, unrelated issues, subagent usage, follow-ups, or risk notes if any]
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.