Content
63%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 dense, expert-level methodology with a clear five-step sequence, concrete evidence rules, a verbatim output contract, and well-signaled one-level-deep references for the material that belongs in separate files. Its two real weaknesses are boilerplate redundancy (routing repeated three times, plus framework/workflow-integration sections that pad a self-declared standalone skill) and ten dangling shared/base and shared/workflow-contract.md links that the bundle does not actually contain.
Suggestions
Resolve the ten missing shared/base/*.md and shared/workflow-contract.md links: either ship the files in the bundle or replace the 'Base Framework' policy-manifest section with the one or two policy rules that actually matter inline.
Deduplicate routing: keep one canonical 'Routing Boundary' section and drop the sibling-skill list from 'Purpose and activation' (and the standalone 'Optional Workflow Integration' boilerplate) — this would lift conciseness from 3 to 4 without losing guidance.
Fix the garbled step-3 heading ('Diagnose package, build, and weight risk' -> 'dependency-weight risk') and add one or two inline example commands (e.g. `npm ls --all`, `cargo tree -i <crate>`) so the body is executable without opening references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense rather than tutorial-style (no 'what is a lockfile' padding), but it contains several padded/boilerplate sections: routing guidance is repeated three times (frontmatter description, 'Purpose and activation' listing sibling skills, and again in 'Routing Boundary'), the 'Base Framework' section carries a policy-link manifest plus precedence boilerplate, and 'Optional Workflow Integration' spends ~8 lines on integration for a skill that declares itself 'fully standalone'. This matches the score-3 anchor 'Mostly efficient but includes some unnecessary explanation or could be tightened'; it is not score 4 because these redundancies are structural sections to trim, not minor instances, and not score 2 because the core guidance itself is terse, expert-level, and free of concept explanations Claude already knows. | 3 / 5 |
Actionability | For an instruction-only analysis skill, the guidance is concrete throughout: precise inspection targets ('manifest-lock alignment, declared/imported/missing/wrong-scope packages, duplicate/conflicting/peer versions'), a verbatim copy-paste-ready output template ('Use this exact structure'), an exact findings-field list ('ID, category, severity, confidence, exact location/path/version/graph edge...'), and executable tool names delegated to the real references/ecosystem-tools.md (npm ls, pip-audit, go mod graph). This matches the score-4 anchor 'Mostly executable guidance... minor gaps'. It falls short of score 5 because no example commands or worked micro-examples appear inline in the body, and one heading is garbled ('Diagnose package, build, and weight risk' appears to mean 'dependency-weight risk'), which slightly muddies execution. | 4 / 5 |
Workflow Clarity | The five workflow steps are clearly sequenced (scope/inventory -> build graphs -> package risk -> architecture/runtime -> supply chain/upgrade readiness) each with concrete inputs, and validation checkpoints exist: 'Verify unused candidates across all non-static use sites before removal', evidence-gating severity ('Use HIGH only with direct manifests/locks/imports/scanners/config/runtime evidence'), 'Unable to determine - evidence needed' as an explicit no-evidence outcome, and a final quality checklist. This matches the score-4 anchor 'Clear sequence with most checkpoints present; minor validation gaps'. Not score 5 because checkpoints are distributed in prose rather than attached as explicit per-step gates, and there is no validate-then-proceed loop inside any step; not score 3 because validation is clearly articulated, and the skill's destructive operations (auto-removal, mass upgrade) are explicitly prohibited with required test/rollback plans. | 4 / 5 |
Progressive Disclosure | Per the guideline to score against the actual bundle structure: the four references/ files (ecosystem-tools.md, risk-and-upgrades.md, architecture-and-runtime.md, examples.md) exist, are well-signaled with markdown links at the relevant workflow step, and are one level deep — good. However, ten referenced paths do not exist in the bundle: all nine shared/base/* policy files linked as 'Required packaged policies' and shared/workflow-contract.md are missing, so navigation breaks for a third of the body's links. This fits the score-3 anchor 'Some structure but could be better organized' — the structure and signaling are present but the bundle does not resolve its own required references. Not score 2 because substantial, genuinely well-split reference files exist and the body itself is sectioned; not score 4 because unresolved required-policy links are more than a minor organization gap. | 3 / 5 |
Total | 14 / 20 Passed |