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 well-built, highly actionable skill: exact rule configurations, executable verification commands, and a clear five-phase workflow organized almost entirely as dense tables with negligible padding. Weaknesses are minor — a fragmentary CI example, one vague rule-mapping row, no explicit fix-and-retry checkpoints, and reference tables that could be moved into bundle files to slim the always-loaded body.
Suggestions
Replace the CI fragment with a complete, copy-paste-ready GitHub Actions workflow (or explicitly label it as a skeleton), and resolve the 'Custom or import/extensions rule' row to the exact rule configuration.
Add an explicit validation checkpoint after implementation, e.g. 'Run the three verification commands; fix and re-run until lint/format/typecheck all pass before wiring up CI.'
Move the large rule-mapping tables and the language quick reference into a references/ file (e.g. references/rule-mappings.md) to reduce the always-loaded token footprint, keeping only the workflow and principles in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and table-driven with almost no padding — rule mappings, config principles, and commands are delivered as compact reference tables rather than prose, and the 'Mindset' section is a short two-line framing rather than a lecture. Not 5 because a few rows restate defaults Claude already knows (e.g. 'tabWidth: 2 (default)', 'strict: true' alongside the two flags it subsumes) and the import-sorting plugin is explained twice (sections 2 and 3), which could be trimmed. | 4 / 5 |
Actionability | Mostly executable guidance: rule tables give exact config strings with severities and options ('@typescript-eslint/no-unused-vars: ["warn", { argsIgnorePattern: "^_"... }]'), and verification ships copy-paste commands ('npx eslint 'web/src/**/*.{ts,js,vue}'', 'npx vue-tsc --noEmit') plus ready package.json scripts. Not 5 because of two minor gaps: the row 'Explicit .js extensions in imports | Custom or import/extensions rule' defers the actual rule to pseudocode-level guidance, and the CI snippet is a fragment ('name: Lint & Typecheck / steps:') rather than a complete workflow file. | 4 / 5 |
Workflow Clarity | A clear five-phase sequence (Discovery → Mapping → Implementation → Verification → CI Integration) with an explicit verification phase of runnable commands, so the destructive/batch cap at 3 does not apply. Not 5 because there are no error-recovery checkpoints or gating — nothing like 'if lint reports errors, fix and re-run before adding CI' — and the workflow proceeds straight from implementation to CI without an explicit pass/fail checkpoint. | 4 / 5 |
Progressive Disclosure | Good structure: well-organized sections, external links are real, clearly signaled, and one level deep (vuejs.org style guide, ESLint flat config docs, typescript-eslint rules, Prettier options, Biome), and the References section doubles as a navigation index including a related-skill pointer. Not 5 because no bundle files exist and ~130 lines of lookup tables (the rule-mapping tables and the language quick reference) are inline reference material that a references/ file could offload from the always-loaded body; not 3 because the inline content is genuinely used by the core workflow and organization is clean. | 4 / 5 |
Total | 16 / 20 Passed |