Senior code review
Skill ArcanEdge-AI/codex-agent-playbook/skills/senior-code-review
Tool-agnostic custom instructions, subagent playbooks, skills, and reference docs for production-grade Codex coding agents.
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.
2 things to look at
- 28 days oldThe repository was created 28 days ago. New is not bad, but a brand new repository carrying a familiar-sounding name is the shape a typosquat arrives in, and there has been no time for anyone else to find a problem with it.
- 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.
What its author says it does
Copied from the file, not written here
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.
SKILL.md
1.2 KB, 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]
Gives 0 of the 12 instructions most quality gates skills give
Counted across 1,195 of the 2,094 authors here whose files we hold, read 2026-08-06
- read the output and check the exit codein 55 of 1195, across 14 files
- verify requirements using a line-by-line checklistin 53 of 1195, across 12 files
- identify the verification command proving the claimin 53 of 1195, across 12 files
- run the full verification commandin 51 of 1195, across 11 files
- verify output confirms the claimin 49 of 1195, across 10 files
- check version control diff after agent delegationin 45 of 1195, across 5 files
- state claim with evidencein 43 of 1195, across 3 files
- run the test suitein 32 of 1195, across 24 files
- keep state in memory by defaultin 27 of 1195, across 6 files
- make prototype runnable with one commandin 26 of 1195, across 5 files
- detect the package manager from lockfilesin 24 of 1195, across 5 files
- produce a verification reportin 23 of 1195, across 12 files
Said here and by no other author read
- check generated or vendored files
- check for missing validation
- review naming clarity and consistency
- review for API compatibility
- review safety and performance risk
- fix the issue
Grouped from the skills themselves: near-identical wordings counted once, and counted by distinct author, so one author publishing three of these counts once.