Content
71%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.
A lean, well-organized overview that defers implementation detail to a real, one-level-deep reference file. The main weaknesses are the near-duplicate 'Check'/'Code Review' sections and opening-paragraph rationale that cost tokens without adding guidance Claude lacks.
Suggestions
Merge 'Check' and 'Code Review' (or frame 'Code Review' explicitly as the PR-context variant of 'Check') to remove the duplicated audit instructions.
Trim the GDPR rationale in the opening paragraph to one sentence — Claude already knows Article 5(1)(c) — and keep the persuasive detail in references/rule.md.
Surface one or two verification steps from references/rule.md (e.g., search the codebase for localStorage.setItem, inspect request bodies in the Network tab) so post-fix validation is visible without opening the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | 'Check' and 'Code Review' give nearly identical audit instructions ('Audit the form fields, API request bodies, and client-side storage keys...' vs 'Review form components, fetch/axios calls, and storage utilities... Flag any field or key that is collected but not read'), and the opening GDPR rationale is motivation Claude already knows — mostly efficient but could be tightened, anchor 3 rather than anchor 4's 'minor instances'. | 3 / 5 |
Actionability | Concrete, executable direction for an instruction-only skill: named audit targets, a precise criterion ('collected but not read by an active feature'), and specific remediation ('Replace raw personal identifiers in analytics and logs with pseudonymous IDs and define a retention window for each stored value'). The minor gap — no inline example of a violation before the reference pointer — keeps it below anchor 5. | 4 / 5 |
Workflow Clarity | A clear Check → Fix → Explain sequence with the audit acting as an explicit pre-fix gate; minor validation gaps remain — no post-fix verification step is surfaced in the body (it lives in references/rule.md), and the redundant 'Code Review' section muddies where it fits in the sequence — so anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | A ~30-line overview with a clearly signaled, one-level-deep reference ('For full implementation details, code examples, and framework-specific guidance, see references/rule.md'); the file exists and holds exactly those details with no nested references, so content is appropriately split with easy navigation. | 5 / 5 |
Total | 16 / 20 Passed |