Content
75%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 strong, well-structured single-file skill: concrete rule/config syntax throughout, executable verification commands, and a clear five-step workflow with a dedicated verification phase and a valuable 'Never' guardrail section. The main gaps are the fragmentary CI YAML, the absence of an explicit feedback loop after verification, and the lack of any bundle split for the detailed mapping tables.
Suggestions
Complete the CI snippet (add 'on:' and 'jobs:' keys) or replace it with the exact runnable commands so it is copy-paste ready.
Sequence verification as an explicit feedback loop: run lint/format:check/typecheck, fix failures, and re-run until all checks pass before moving to CI integration.
Move the detailed ESLint/Vue/Prettier/TypeScript mapping tables and the Language Quick Reference into a references/ file, leaving SKILL.md as a lean overview with one-level-deep pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with project-specific values ('singleQuote: true', 'printWidth: 100', 'strict: true') that earn their tokens, but the Language Quick Reference table ('Python | Ruff | mypy / pyright') restates knowledge Claude already has and could be trimmed. Not a 5 because a few tokens are spent re-teaching known material; not a 3 because the padding is minor and localized. | 4 / 5 |
Actionability | Exact rule syntax ('@typescript-eslint/consistent-type-imports: ["error", { prefer: "type-imports" }]') and copy-paste commands ('npx eslint \'web/src/**/*.{ts,js,vue}\'', full package.json scripts) cover the common cases. Not a 5 because the CI block is a fragment — it shows 'name: Lint & Typecheck' and 'steps:' without the 'on:'/'jobs:' hierarchy, so it is not runnable as written. | 4 / 5 |
Workflow Clarity | A clear five-phase sequence (Discovery → Mapping → Implementation → Verification → CI Integration) with an explicit verification step listing commands and '--fix' recovery commands. Not a 5 because error recovery is not sequenced as an explicit run → fix → re-run-until-clean feedback loop, only implied by the presence of --fix variants. | 4 / 5 |
Progressive Disclosure | Well-sectioned single file with a clearly signaled References section of external links; no bundle files exist, so all detail is inline. Not a 5 because roughly 80 lines of detailed rule-mapping tables (and the language quick reference) are inlined in SKILL.md rather than split into one-level-deep references/ files; not a 3 because the structure and navigation are good, with only minor organization gaps. | 4 / 5 |
Total | 16 / 20 Passed |