Content
50%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.
The body is highly actionable with concrete code for App Router migration and patterns, but it is padded with repetitive generateStaticParams examples and re-explained concepts, and it monolithically inlines content that should be split into reference files. The migration workflow lacks integrated validation checkpoints despite destructive cleanup steps.
Suggestions
Collapse the generateStaticParams section: keep one canonical example plus the migration comparison, and remove the redundant Common Patterns / Multiple Dynamic Segments / Dynamic Behavior Configuration restatements to cut substantial padding.
Move the detailed generateStaticParams, routing-pattern, and data-fetching deep dives into separate reference files under references/ and keep SKILL.md as a concise overview with one-level-deep links.
Integrate validation checkpoints into the migration workflow — e.g., after Step 4 run `npm run build` and only proceed to the destructive Step 5 cleanup once the build passes — rather than leaving validation as a detached final checklist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Noticeably verbose: the generateStaticParams section restates the same function across Basic Usage, Common Patterns (3 variants), Multiple Dynamic Segments, Dynamic Behavior Configuration, and Migration — heavily padded — and several sections re-explain Next.js concepts Claude already knows (e.g., Server Component benefits). | 2 / 5 |
Actionability | Mostly executable guidance with concrete, copy-paste TypeScript/TSX and file paths for layouts, metadata, routing, and migration; minor gaps from undefined helpers (getPost, getCategories, db.post.findMany) and a `return (/* render */)` placeholder. | 4 / 5 |
Workflow Clarity | The migration guide has a clear Step 1–5 sequence, but validation lives in a separate end-of-document checklist rather than as inline checkpoints, and the destructive Step 5 (deleting pages/ files) lacks a verify-before-delete feedback loop — capping this at 3 per the destructive-operation rule. | 3 / 5 |
Progressive Disclosure | Section headers are well organized, but this is a ~780-line monolithic SKILL.md with no bundle files; large blocks that belong in separate references (the generateStaticParams deep-dive, data-fetching patterns) are inlined, and cross-skill pointers at the end are not bundle references. | 3 / 5 |
Total | 12 / 20 Passed |