Content
75%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 solid, code-rich skill body with executable examples for every provider and a clear deploy sequence. The main gaps are a missing streaming code example (despite streaming being mandated) and redundancy between the 'Never do these' list and the appended house-rules section.
Suggestions
Add a concrete streaming example (SDK streaming + ReadableStream) and/or a background-function pattern — the body mandates streaming in two places but shows no streaming code, and a self-identified GAP comment confirms this gap.
Remove or de-duplicate the appended 'Netlify house rules' section, which restates the 'Never do these' warnings verbatim and adds ~25 redundant tokens; keep a single canonical guardrail list.
Consider moving the per-provider SDK code blocks or the full env-var/constraints reference into a bundled reference file so SKILL.md stays a lean overview with one-level-deep pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly lean and code-driven with purposeful prose, but the appended 'Netlify house rules' section restates the top 'Never do these' warnings (browser-callable, runtime-only credentials, 60s timeout, dynamic model list), adding redundant tokens; not 5 because of that duplication, not 3 because the core body is efficient. | 4 / 5 |
Actionability | Provides copy-paste-ready executable code for every provider SDK with install commands, a full Function example, an env-var table, a vite.config snippet, and deploy commands; held to 4 rather than 5 because streaming is mandated ('Never leave a slow generation unstreamed') yet no streaming code example is present — a noted GAP. | 4 / 5 |
Workflow Clarity | The 'Enable & deploy' section gives a clear numbered sequence (credit plan → netlify init → deploy --prod → don't disable AI Features) with guardrails in 'Never do these'; not 5 because there are no explicit validation checkpoints or error-recovery feedback loops, though the operation is not destructive/batch so no cap applies. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections (Function example, Provider SDKs, Injected env vars, Local dev, Enable & deploy, Constraints & gotchas, Reference) with one-level-deep external doc links in the Reference section; not 5 because all content is inlined in a ~170-line SKILL.md with no bundle files, and the appended house-rules section is a structural oddity rather than a separate file. | 4 / 5 |
Total | 16 / 20 Passed |