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 highly actionable, well-organized scaffold guide with executable commands and complete code patterns, but it is a monolithic file that inlines a full pattern library (much of it boilerplate Claude already knows) instead of splitting it into reference files, and a few referenced artifacts (data-source.ts, .env.example) are never defined.
Suggestions
Split the nine inlined code patterns into reference files (e.g. references/patterns.md for guard/interceptor/filter, references/orm.md for TypeORM/Prisma) and keep SKILL.md as an overview with one-level-deep pointers.
Define or show the missing artifacts referenced by commands and steps: 'src/data-source.ts' for TypeORM migrations and the '.env.example' template contents.
Trim textbook boilerplate Claude already knows (the pass-through JwtAuthGuard, standard interceptor/filter bodies) down to the project-specific conventions that differ from NestJS defaults.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Prose sections are lean and instructional, but roughly 250 lines are textbook NestJS boilerplate Claude already knows — e.g. the trivial JwtAuthGuard ('export class JwtAuthGuard extends AuthGuard(\'jwt\')' with a pass-through canActivate) and the standard logging interceptor and exception filter. This is 'mostly efficient but includes some unnecessary explanation or could be tightened', not the lean anchor 5, yet it avoids concept-explanation padding that would drop it to 2. | 3 / 5 |
Actionability | The scaffold commands ('npx @nestjs/cli@latest new <project-name> --strict') and code files are concrete and copy-paste ready, covering controller, service, entity, DTO, module wiring. Minor gaps keep it below 'fully executable': migration commands reference 'src/data-source.ts' which is never defined or shown, and '.env.example' is required by First Steps but its contents are not provided. | 4 / 5 |
Workflow Clarity | A clear sequence runs Scaffold Command → Project Structure → patterns → First Steps After Scaffold, ending with a smoke check ('Test the API: curl http://localhost:3000/api/docs'). This matches 'clear sequence with most checkpoints present; minor validation gaps' — there is one verification step but no deeper checkpoints (e.g. confirming the build or type-check passes after wiring), so not 5. | 4 / 5 |
Progressive Disclosure | There are no bundle files at all (no references/, scripts/, assets/), so nine full reference implementations (guard, interceptor, filter, entity, etc.) are inlined in a ~420-line monolithic SKILL.md. Section headers are clear and navigation is one-level, but 'content that should be separate is inline' — the pattern library belongs in reference files — matching anchor 3 rather than the well-split anchor 4. | 3 / 5 |
Total | 14 / 20 Passed |