Phase review
Engineering workflows and standards for Codex and Claude Code: plan, implement, test, review, and deliver repository changes.
npx -y skills add yarlson/yarstack --skill phase-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
- 3 stars3 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
Review one validated phase for behavior-preserving structural regressions introduced or materially worsened by the change. Use after phase validation and before final delivery.
SKILL.md
1.6 KB, as published. Nobody here has run it
Phase Structural Review
Own a narrow maintainability pass after correctness has been established.
Workflow
- Read the selected contract, applicable repository rules, final diff, and changed units.
- Identify only complexity, duplication, unclear ownership, loose types, or unnecessary indirection introduced or materially worsened by the selected change.
- Use
crap-index-assesswhen the repository's configured CRAP check covers changed methods or the phase was intended to reduce their CRAP risk. Treat score changes as supporting evidence, not a structural verdict. - Fix a finding only when the parent implementation request authorizes scoped edits and the correction is clearly behavior-preserving.
- Report a worthwhile correction when it requires non-trivial restructuring or broader authority; do not expand scope during the review.
- Rerun affected checks after every edit.
Correctness belongs to phase-validate; general read-only defect review to code-review; test sufficiency to test-gap-review; documentation accuracy to docs-drift-review; and specialized risks to their owning reviewers.
Do not add behavior, pursue unrelated cleanup, use subjective file-size thresholds, or continue until no imaginable simplification remains.
Finish when structural regressions introduced by the change are fixed or reported with their scope and risk.