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 strong, highly actionable reference body: concrete commands, complete code samples, and dense tables that assume the reader's competence. Its weaknesses are structural rather than informational — a duplicated merged 'house rules' block, inlined per-framework depth that blurs the overview/reference split, and no explicit verification checkpoints after deploys or config changes.
Suggestions
Deduplicate the merged 'house rules' section: rules 1-6 restate content already in the body (env-var redeploy, SPA catch-all, netlify dev #custom, client-prefix secrets); keep only the net-new rules (Next.js skew-protection version conditions and client fetch options) inline, ideally folded into the existing sections they belong to.
Move deep per-framework detail (e.g., the full Gatsby version matrix, Express serverless-http example, Angular SSR/Image-CDN specifics) into the corresponding references/*.md guides, keeping SKILL.md as a routing layer with the build-settings table and the framework-agnostic gotchas.
Add a short verification step to workflows: after a deploy or env-var change, confirm the change is live (e.g., check the deployed function's env or hit the SSR route) before treating the task as done — currently only the 'redeploy required' rule hints at this.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense tables and terse commands with essentially no concept-teaching padding ('Env values are injected at build time... requires a redeploy'; build-settings tables), so it is well above the midpoint. Not 5: the ~37-line merged 'house rules' block at the bottom restates four rules already stated above (env-var redeploy, SPA catch-all removal, netlify dev #custom, client-prefix secrets), which could be trimmed to only the net-new rules 7-8. | 4 / 5 |
Actionability | Guidance is copy-paste ready throughout: 'npx astro add netlify', 'npm install -D @sveltejs/adapter-netlify' with the exact svelte.config snippet, a complete Express serverless-http example with its netlify.toml, and per-framework build/publish tables. Common cases are covered with specific executable commands; the only placeholders (dist/YOUR_PROJECT_NAME, '(vite)') are inherent to the task, not missing detail. | 5 / 5 |
Workflow Clarity | The skill is reference/lookup-shaped rather than a single linear workflow, and it sequences well: 'Env vars: modern rules (read first)' up front, routing to per-framework guides, then per-framework setup (install → configure → build/deploy) with explicit gotchas. Not 5: there are no explicit validation checkpoints (e.g., verify the deploy picked up the env var or that the adapter built server output) — the redeploy-after-env-change rule is the only checkpoint-like guidance, and it is implicit rather than a step in a sequence. | 4 / 5 |
Progressive Disclosure | The six referenced files (references/astro.md, nextjs.md, nuxt.md, sveltekit.md, tanstack.md, vite.md) all exist, are one level deep, and are clearly signaled both in the opening routing line and in context (e.g., 'Details in references/nextjs.md', 'see references/tanstack.md'). Not 5: substantial per-framework depth (full Gatsby, Angular, Next.js, Express sections with code) is inlined in SKILL.md itself, so the body is a dense reference rather than a concise overview that appropriately splits detail into the guides. | 4 / 5 |
Total | 17 / 20 Passed |