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.
This is a high-quality reference body: nearly every claim is a non-obvious Netlify-specific fact, code examples are executable and cover the common cases, and risky operations carry explicit warnings plus a Cache-Status verification path. Its weaknesses are structural: ~330 lines of inlined API-reference material with no progressive disclosure into reference files, plus leaked build comments and unpinned-inline version numbers that add noise.
Suggestions
Split the API-reference material into one-level-deep reference files (e.g., references/purge-api.md for the direct API/lambda-compatible purge details and references/cache-helpers.md for the @netlify/cache module), leaving concise usage summaries and clear pointers in SKILL.md.
Remove the leaked build metadata — the "<!-- Gaps: ... -->" and "<!-- system: agent-context/caching/system.md ... -->" comments and the "merged by ctx-gen / edit system.md" preamble — from the rendered skill body.
Move time-sensitive version requirements ("Netlify CLI 20.0.3+", "Next Runtime 5.5.0+") into a dedicated compatibility or legacy-patterns section so they can be maintained without penalizing the main guidance, and add an explicit post-purge verification step (re-request the URL and confirm the Cache-Status header shows a miss).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious platform facts Claude cannot know ("Static assets are fresh for up to a year", "basic-auth on ANY page disables caching for the ENTIRE site", "100 lookups, 20 insertions/deletions") with essentially no padding. Minor deductions: inline version pins ("Netlify CLI 20.0.3+", "Next Runtime 5.5.0+") are time-sensitive and not in a deprecated/old-patterns section, and leaked build metadata ("<!-- Gaps: ... -->", "<!-- system: agent-context/caching/system.md ... merged by ctx-gen ...>", and the "These are org conventions, not docs facts" preamble) is token noise for the skill's consumer. | 4 / 5 |
Actionability | Nearly every section gives copy-paste-ready, complete examples: full TypeScript function snippets for header caching and the Cache API, three purgeCache variants, a complete curl command with headers and JSON body, and executable cacheHeaders/fetchWithCache/needsRevalidation usage. Common cases (opt in via header, purge by tag, cache an expensive fetch, debug via Cache-Status) are each covered with specific code; score 4 would require notable gaps like pseudocode or missing key details, which are absent. | 5 / 5 |
Workflow Clarity | Organization leads the reader well: "Cache-control header to reach for", "Footguns (read first)", directive reference, then task-specific sections, with a Cache-Status debugging section acting as an explicit verification path ("Check it on a deployed URL") and warnings like "an empty cache_tags list purges NOTHING" guarding risky purge operations. It falls short of 5 because there is no explicit validate-and-recover loop (e.g., verify the purge took effect, or confirm headers before/after) and purge — a semi-destructive site-wide operation — has no post-action verification step. | 4 / 5 |
Progressive Disclosure | The ~330-line body has no bundle files at all (no references/, scripts/, or assets/ exist), so all content is inlined in SKILL.md. Sections are well-labeled, but the @netlify/cache helper API reference (cacheHeaders/fetchWithCache/getCacheStatus/needsRevalidation sub-sections) and the direct purge-API details are API-reference material living inline — matching the score-3 anchor of "200 lines of API reference that could be in a separate file". Score 4 would require the bulk detail to be moved out with clear in-body pointers. | 3 / 5 |
Total | 16 / 20 Passed |