Content
100%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 dense, rule-driven instruction set with executable code, a complete workflow with explicit validation and error-recovery checkpoints, and a clean one-level-deep reference structure whose three bundle files all exist and match their in-body descriptions. It contains no known-concept padding, no time-sensitive unversioned API guidance, and no buried references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body never explains concepts Claude already knows; every line is a project-specific rule ('A mock that differs from the DTO tests a contract that does not exist', 'a bare \'/api/room-types\' stops matching once the base URL is set'), and each code snippet illustrates one specific rule — the only repetition is the deliberate DO NOT checklist, not padding. | 5 / 5 |
Actionability | It provides fully executable guidance: the HttpTestingController match/flush snippet, the DOM interaction snippet with `whenStable()`, a concrete DI override ('providers: [{ provide: I18nService, useClass: MockI18nService }]'), the run command 'ng test --watch=false', an assertions table with real examples, and two complete copy-paste-ready spec templates in references/. | 5 / 5 |
Workflow Clarity | The 9-step workflow has pre-flight checks (existing spec reconciliation, package.json/app.config.ts inspection), an explicit validation step (run the tests), a detailed failure-recovery policy (fix test mechanics vs. leave specification disagreements failing, never weaken an assertion), and a structured final report with expected vs. actual values. Test writing is non-destructive, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | All three bundle files exist and are referenced one level deep with clear signals of their content: the Templates section describes references/page.spec.ts.md and references/service.spec.ts.md, and the Markers section links references/markers.md; the bulk template code is correctly split out of SKILL.md. | 5 / 5 |
Total | 20 / 20 Passed |