Content
63%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a well-structured interactive workflow with explicit approval and conflict-resolution checkpoints, but it suffers from noticeable duplication across the Constraints, 'What is covered', and 'When to use' sections. Its biggest structural defect is at the bundle level: the referenced plan template is a dangling xi:include pointing to a nonexistent assets file, so the deferred detail the workflow depends on cannot actually be reached.
Suggestions
Consolidate duplicated guidance: state the sanitized-summary and source-authority rules once (either in Constraints or Step 1) and drop the verbatim trigger list from 'When to use this skill' since the description already carries it.
Fix the bundle's broken reference chain: create assets/java-design-plan-template.md or inline the template into references/041-planning-plan-mode.md, since the reference's xi:include currently points to a nonexistent file.
Add an explicit post-generation validation step (e.g., verify the generated plan records all source artifacts, derivation direction, and Execution Instructions) to close the workflow's feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient with no explanations of concepts Claude already knows, but content is duplicated across sections: the sanitized-summary rule appears in both Constraints ('Use maintainer-provided sanitized summaries for issue, PR, wiki...') and Step 1; 'When to use this skill' repeats the frontmatter trigger list verbatim; and the 'What is covered' bullets overlap the Constraints. This is more than the 'minor instances that could be trimmed' of the anchor above. | 3 / 5 |
Actionability | Instruction-only skill with mostly concrete guidance: 'Run `date` before starting', 'Ask one or two focused questions at a time', an enumerated plan structure in Step 4, and explicit MUST/MUST NOT constraints. Minor gaps remain — the plan template and filename conventions are deferred entirely to the reference file. | 4 / 5 |
Workflow Clarity | Clear numbered sequence (steps 0–5) with most checkpoints present: 'Validate the summary and wait for approval before writing' and 'Do not propagate conflicting changes until the user makes an explicit decision'. Falls short of the top anchor because there is no validation or feedback loop after the plan is generated. | 4 / 5 |
Progressive Disclosure | The body itself is a clean overview with one clearly signaled, one-level-deep reference (references/041-planning-plan-mode.md, which exists). However, scored against the actual bundle structure, the reference contains '<xi:include href="assets/java-design-plan-template.md" parse="text"/>' and no assets/ directory exists — a dangling second-level reference that breaks the bundle's navigation chain, keeping it below the 'references mostly clear' anchor. | 3 / 5 |
Total | 14 / 20 Passed |