skills/planner-review/SKILL.md
PLANNER REVIEW SKILL — Review a completed backlog item and update design artifacts accordingly. USE FOR: PLANNER reviewing an EXECUTOR's completed work; updating FEATURES.md after a feature lands; removing completed items from BACKLOG.md; checking for drift. DO NOT USE FOR: implementing backlog items (EXECUTOR work); initial planning (read GOALS and DESIGN directly).
npx skillsauth add zcross00/ProcessDocumentation planner-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Use this skill after an EXECUTOR reports that a backlog item is complete. The PLANNER verifies the work and updates design artifacts.
Open design/BACKLOG.md and find the reported BL-N. Read the full detail block, including:
Check each Done-when criterion against the actual codebase:
build-verify (or confirm the EXECUTOR already did).Verdict: All done-when criteria pass → proceed. Any criterion fails → report the gap; do not update artifacts.
If the completed item was type Feature or Rework that adds visible capability:
design/FEATURES.md.| [Feature name] | [Brief description] | BL-N | YYYY-MM-DD |
If the feature doesn't clearly map to a goal, add it to the Features Without Goal Link section and consider logging drift.
Delete both the table row and the detail block for the completed BL-N. Do not renumber other items.
If priority section becomes empty after removal, leave the section header in place.
Review the implementation for:
design/DESIGN.md need to reflect new decisions, constraints, or changes that emerged during implementation? If yes, update the relevant section.Report the review result to the user:
development
TEST WRITING SKILL — Write correctly structured JUnit 5 tests that match this project's conventions. USE FOR: adding tests for a new backlog item; flagging missing test coverage; writing tests for engine classes, domain logic, or persistence. Covers class placement, naming conventions, setup patterns, assertion style, and what not to test. DO NOT USE FOR: running tests (use build-verify skill); diagnosing test failures (read the Surefire report directly).
development
STANDARDS CHECK JAVA SKILL — Apply standards/general.md and standards/java.md systematically to a Java file or diff. USE FOR: pre-commit standards compliance check; reviewing a file for standards adherence before or after a change. Produces a categorized violation list with the specific rule cited for each. DO NOT USE FOR: general code review (use planner-review skill for implementation review).
testing
REFINEMENT SKILL — Analyze design, features, backlog, and drift to identify and plan needed work. USE FOR: PLANNER producing new backlog items from gaps between design and current state; evaluating tech debt and drift for resolution; structured backlog growth when the user asks the PLANNER to refine or plan more work. DO NOT USE FOR: implementing backlog items (EXECUTOR work); reviewing completed work (use planner-review skill).
testing
GIT WORKFLOW SKILL — Exact command sequences for all branch lifecycle operations in this project. USE FOR: creating a work branch for a backlog item; committing and merging work; tagging design versions or releases. Eliminates the need to look up git-workflow.md during implementation. DO NOT USE FOR: deciding what to work on (use backlog-entry or agent-orientation).