Content
70%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's core strength is an exceptionally clear, well-guarded review workflow: staged scenes, explicit verification with false-positive rejection, and a concrete severity/priority scheme. Its weaknesses are spec-style scaffolding that inflates token cost without adding instruction, and a reference layer that points at bundle files that do not exist alongside the skill.
Suggestions
Ship the referenced bundle files (resources/execution-protocol.md, iso-quality.md, checklist.md, error-playbook.md, verify-ship-protocol.md) with the skill, or remove the references — the body instructs 'follow for the selected task' for a file that is not present.
Cut the non-instructive scaffolding — the 'SSL primitive' column in the Actions table and the 'Resource scope' table — to reclaim tokens; neither changes what Claude does.
Add a short example of an ordered finding (severity, file:line, evidence, fix) so the required report format is concrete rather than implied by guardrails.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body mostly avoids explaining concepts Claude already knows, but a fair share of tokens go to non-actionable spec scaffolding — the 'SSL primitive' action table (READ/SELECT/CALL_TOOL), the 'Resource scope' table, and the Goal/Intent-signature meta sections — that could be tightened or cut. That matches anchor 3 (mostly efficient, some unnecessary content that could be tightened) rather than anchor 4, whose over-explanation is only minor. | 3 / 5 |
Actionability | There is genuinely executable guidance: a canonical command path ('npm audit', 'bandit -r .', 'lighthouse <url>'), a concrete severity rubric (CRITICAL/HIGH/MEDIUM/LOW with definitions), an explicit priority order, and named standards (OWASP, WCAG 2.2 AA, ISO/IEC 25010/29119). It stays at anchor 4 rather than 5 because no example finding/report format is shown and per-stack commands are left as 'add project lint/test/coverage commands', leaving minor gaps. | 4 / 5 |
Workflow Clarity | The PREPARE → ACQUIRE → REASON → VERIFY → FINALIZE sequence is clearly ordered, with an explicit validation stage ('VERIFY: Reproduce findings and reject false positives'), feedback loops ('If a finding cannot be reproduced, do not report it as a finding'), failure-and-recovery handling, and defined exit criteria — matching anchor 5's explicit validation steps, error-recovery loops, and checklists. It is not anchor 4 because checkpoints are explicit throughout rather than having minor gaps. | 5 / 5 |
Progressive Disclosure | The References section is clearly signaled and one level deep, but every referenced bundle path — resources/execution-protocol.md, resources/iso-quality.md, resources/checklist.md, resources/error-playbook.md, resources/verify-ship-protocol.md, and ../_shared/core/* — is absent from the skill bundle (no resources/, references/, scripts/, or assets/ files exist), so navigation dead-ends for files the body says to 'follow'. This matches anchor 3 (structure present but organization impaired) rather than anchor 4's 'minor organization gaps', since core execution content is delegated to missing files. | 3 / 5 |
Total | 15 / 20 Passed |