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 well-structured, navigable skill body with excellent progressive disclosure into verified reference files and two clearly sequenced workflows with useful checkpoints. The main cost is token weight: the generic Maintainability/Security checklists pay tokens for knowledge Claude already has instead of reserving the body for the genuinely non-obvious patterns (bug patterns, stack references).
Suggestions
Trim the Maintainability dimension to a one-line pointer of project-specific bars — naming, comment, and magic-constant principles are knowledge Claude already has and each bullet spends tokens restating it.
Compress Common bug pattern entries to pattern + fix (e.g., "CDN serving old asset under same filename → always use a new filename or cache-bust query string"), cutting the trailing rationale sentences.
Reserve the body's checklist budget for what is distinctive to this skill (the stack-agnostic bug patterns and stack reference routing) and move the generic security/performance checklist items into a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and unpadded, but sections like Maintainability ("Naming. Functions and variables named for what they do", "Comments. Explain why, not what", "No literal `60000` in code. Use named constants") and much of the Security dimension restate principles Claude already knows. This fits anchor 3 ("mostly efficient but includes some unnecessary explanation") better than anchor 4, where over-explanation would be only a minor instance rather than recurring sections. | 3 / 5 |
Actionability | Concrete instruction-only guidance throughout: specific checks ("Session cookies have `Secure`, `HttpOnly`, `SameSite` attributes set"), a runnable command ("`git log --oneline`"), and an explicit output artifact ("a markdown document at `code-review-[date].md`" with a six-section structure). Minor gaps keep it below anchor 5 — e.g., few executable commands for verifying findings in the review workflow. | 4 / 5 |
Workflow Clarity | Two clearly numbered sequences with several checkpoints ("Reproduce locally. Confirms it's a code issue", "Check cache state... before concluding it's a code bug", "Verify in production. Check the actual fix worked"). Not capped at 3 since this is a review skill, not a destructive/batch operation; it stays at 4 rather than 5 because there is no explicit fix-and-retry feedback loop and the review workflow lacks a verify-findings step. | 4 / 5 |
Progressive Disclosure | The ~200-line body is a genuine overview: stack-specific detail is split into three real, one-level-deep reference files (verified on disk; they contain no references beyond a back-link to SKILL.md), clearly signaled both inline ("Use the template in references/review-template.md") and in a dedicated "Reference files" section. This matches anchor 5's clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 16 / 20 Passed |