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-structured overview for a simple skill: clear task sections and a properly signaled one-level reference containing the full code examples. The main costs are a conceptual intro paragraph duplicated from the reference file and overlapping Check/Code Review sections.
Suggestions
Delete the duplicated 'Why It Matters' prose from SKILL.md — Claude knows why `any` is unsafe, and the paragraph already exists in references/rule.md.
Merge the Check and Code Review sections, which both instruct scanning the file for `any` usages, into one section listing the full set of things to flag (explicit, implicit, unguarded assertions, `any`-inferred returns).
Inline one minimal before/after snippet (`any` → `unknown` + narrowing) in the Fix section so the most common case is executable without opening the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The opening paragraph ('The any type is a local opt-out that becomes a global problem... turning latent runtime errors into compile-time failures') is conceptual background Claude already knows, and it is duplicated verbatim in the 'Why It Matters' section of references/rule.md. The rest is tight, but this unnecessary explanation matches anchor 3 ('Mostly efficient but includes some unnecessary explanation or could be tightened') rather than 4, where only minor trimming would be needed. | 3 / 5 |
Actionability | Each section gives a concrete directive with specific alternatives named per scenario — 'unknown (for values of unknown shape), appropriate generics..., or Zod validation (for external data)' — and the Code Review section enumerates exactly what to flag ('explicit or implicit any, any type assertion that lacks a preceding type guard, any return types inferred as any'). As an instruction-only skill the absence of inline code is acceptable per the rubric notes, but key implementation detail (how to narrow, guard patterns) is deferred entirely to the reference, leaving minor gaps consistent with anchor 4 rather than fully copy-paste-ready guidance at 5. | 4 / 5 |
Workflow Clarity | The four task sections (Check, Fix, Explain, Code Review) each present a single unambiguous directive, and as a simple non-destructive skill no validation checkpoints are required. It falls short of 5 because the Check and Code Review sections substantially overlap ('Scan this TypeScript file for uses of the any type... Report each location' vs 'Flag every explicit or implicit any'), creating mild ambiguity about which procedure applies, matching anchor 4 ('clear sequence with most checkpoints present; minor validation gaps') in structural terms. | 4 / 5 |
Progressive Disclosure | The 45-line body is a well-organized overview (Quick Reference, then per-task sections) with a clearly signaled, one-level-deep pointer at the end: 'For full implementation details, code examples, and framework-specific guidance, see references/rule.md' — and references/rule.md exists and contains exactly that. This matches anchor 5 (clear overview, well-signaled one-level-deep reference, easy navigation); the duplicated prose is a conciseness issue, not a structural one. | 5 / 5 |
Total | 16 / 20 Passed |