Content
78%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 highly actionable, well-structured scaffold skill whose code examples are complete and executable with only minor token waste in trivial plugin wrappers and editorial asides. The main structural weakness is that all file templates live inline in a single 376-line SKILL.md instead of being split into a bundle, and the workflow has one validation checkpoint but no error-recovery loops.
Suggestions
Move the complete file templates (app.ts, plugins/*, routes/*, services/*) into a templates/ or references/ bundle directory and keep SKILL.md as a concise overview with clearly signaled one-level-deep references — this also makes templates directly copyable rather than retyped.
Trim the trivial full-code sections (sensible plugin wrapper, health check) to one-line mentions, and cut marketing asides ("killer testing feature", "~15k req/s") that add tokens without guidance.
Add brief error-recovery guidance to the first-steps workflow (e.g., what to check when `pnpm tsc --noEmit` fails or the port is already in use) to close the validation feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean, purposeful code with terse conventions ("Everything is a plugin", "No `any`"), assuming Claude's competence — e.g., comments are reduced to one-line pointers like "# Provides: fastify.httpErrors.notFound(), .badRequest(), etc.". Not anchor 5 because a few sections could be trimmed: trivial wrapper plugins (sensible.ts, health check) are shown in full, and editorial asides like "This is Fastify's killer testing feature" and "~15k req/s ... ideal for containerized microservices" add tokens without guidance. | 4 / 5 |
Actionability | Everything is copy-paste executable: a complete scaffold command (pnpm add list, tsc --init flags, directory creation), full implementations for app.ts, server.ts, env/swagger/sensible plugins, TypeBox schemas, routes, service, auth decorator, and an error handler. The few placeholders ("Replace with real JWT verification", "Replace with real database client") are explicitly justified, and "First Steps After Scaffold" gives runnable commands. | 5 / 5 |
Workflow Clarity | The sequence is clear and well-ordered — scaffold command → project structure → key conventions → essential patterns → numbered "First Steps After Scaffold" (copy .env, install, start dev server, test with `curl http://localhost:3000/api/health`) → common commands. The curl health check is an explicit validation checkpoint, but the workflow lacks error-recovery guidance (what to do on validation/type failures, port conflicts), which keeps it below anchor 5. No destructive or batch operations, so no cap applies. | 4 / 5 |
Progressive Disclosure | Section structure is clear (## scaffold / structure / conventions / patterns / commands / integrations), but the entire ~250 lines of file templates are inlined in SKILL.md with no bundle files at all — no references/, scripts/, or assets/ exist. Content that clearly belongs in separate files (the complete plugin and route templates, and the Integration Notes) is inline; a templates/ or references/ bundle would let SKILL.md stay an overview and make the templates directly copyable. | 3 / 5 |
Total | 16 / 20 Passed |