Content
17%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 thin templated scaffold: it repeats the description, fills sections with boilerplate, gives only abstract instructions, and points to a resource file that does not exist. It provides little executable guidance and no real workflow with validation.
Suggestions
Replace the generic instruction bullets with a concrete, sequenced workflow (triage -> reproduce -> write failing test -> branch -> implement -> validate -> open PR) with explicit validation checkpoints.
Remove the duplicated opening paragraph and the templated use/don't-use boilerplate, or rewrite them with specific triggers and boundaries.
Either create `resources/implementation-playbook.md` with the promised patterns/examples or remove the broken reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The opening paragraph duplicates the frontmatter description verbatim, and the "Use this skill when" / "Do not use this skill when" sections are templated boilerplate that mechanically repeats "github issue resolution expert" five times, making it noticeably padded rather than lean. | 2 / 5 |
Actionability | The Instructions are universal abstractions ("Clarify goals, constraints, and required inputs", "Apply relevant best practices and validate outcomes") with no concrete commands, code, or domain-specific steps, matching the anchor for entirely vague guidance that only describes rather than instructs. | 1 / 5 |
Workflow Clarity | There is no real sequenced workflow, only a generic bullet list; for operations involving branch management and PR creation the only validation mention is the abstract "validate outcomes", with no checkpoints, fitting the rough-sequence-with-many-gaps-and-no-validation anchor. | 2 / 5 |
Progressive Disclosure | The body references `resources/implementation-playbook.md` in both Instructions and Resources, but no such file exists in the bundle, so the signaled reference is broken; the section structure is minimal and the only disclosure pointer is non-existent. | 2 / 5 |
Total | 7 / 20 Passed |