Content
82%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 high-quality, dense instruction body: nearly everything is executable and repo-specific, with concrete templates and an explicit local-verification checklist. Its main costs are length (the component catalog and templates could be split into reference files) and a workflow whose steps are scattered across sections rather than presented as one sequence.
Suggestions
Consolidate the page-creation workflow (create file -> register in docs.yml -> verify with npm run dev -> confirm checklist) into a single numbered sequence near the top, adding a fix-and-retry loop for broken links/components, instead of spreading it across 'Where new pages live', 'Routing', and 'Local verification'.
Move the full Fern MDX component catalog and the changelog/PR entry templates into a references/ file (e.g. references/mdx-components.md), keeping one compact example per component inline in SKILL.md to reduce token load.
Trim low-value meta commentary such as '<Info> — informational, interchangeable with <Note> in practice' and other rationale sentences that add tokens without changing what Claude would do.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is long (~300 lines) but almost every line carries repo-specific facts Claude cannot know (Fern MDX component syntax, callout selection rules, changelog surfaces, CI-enforced PR headings), so it does not explain concepts Claude already knows. Minor over-explanation could be trimmed (e.g. '<Info>' being 'interchangeable with <Note> in practice', some rationale sentences), placing it below the lean anchor 5 but well above the noticeably-verbose anchor 3. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste-ready frontmatter template, real MDX component examples, the docs.yml page-entry snippet, the changelog entry template, local verification commands ('npm run dev'), and the exact PR headings CI enforces. Specific examples cover the common authoring cases (new page, images, cross-links, changelog, PR description). | 5 / 5 |
Workflow Clarity | The new-page workflow is clearly sequenced — create the file, register it under navigation: in docs.yml, then run the local preview and confirm rendering, links, components, and images — and it warns against inferring routing from folder layout. Checkpoints are present but distributed across sections rather than one numbered sequence, and no explicit fix-and-retry loop is spelled out, so it falls just short of anchor 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the body is organized into clear, navigable sections that push detail to real repo files (fern/docs.yml, .github/pull_request_template.md, anchor pages) — appropriate one-level-deep pointers. However, the full component catalog and the changelog/PR templates are inlined in a ~300-line body that could partly live in a reference file, which keeps it below the ideal-split anchor 5. | 4 / 5 |
Total | 17 / 20 Passed |