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 skill: executable code for every operation, explicit guards around the real footguns (store scope, region per-call, last-write-wins), and disciplined deferral of full API detail to official docs. Weaknesses are minor: a duplicated house-rules block costs tokens, the destructive region-migration step lacks an explicit verify-before-delete checkpoint, and the monolithic single file could split some detail into a reference file.
Suggestions
Trim or de-duplicate the trailing 'Netlify house rules' section: its rules are already stated in the body (not-a-database, store-scope footgun, access control, CLI, build-plugin read/write), so either remove the merged block or keep only rules not covered above.
Add an explicit verification step to the region-migration workflow (e.g., list/compare entries in the new-region store and only delete from the old store once all entries are confirmed copied) since it ends in a destructive delete.
Consider moving secondary detail — module-version migration, region migration, and file-based upload layout — into a references/ file linked from the main body to reduce the single-file footprint and improve progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and footgun-driven with no re-teaching of known concepts, but the trailing "Netlify house rules" block restates points already made above (not-a-database, store-scope footgun, access control, CLI, build-plugin read/write), so a block of tokens could be trimmed — the minor-trimmable-excess anchor rather than every-token-earns-its-place. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste upload and read Function examples, typed set/setJSON/get signatures, onlyIfNew/onlyIfMatch conditional-write patterns with modified checks, listing/pagination recipes, an expiration recipe, and concrete region configuration — covering the common cases. | 5 / 5 |
Workflow Clarity | Risky operations carry explicit checkpoints (null-check before destructuring, etag conditional reads, if (!modified) feedback responses, expiration check before delete, preview-isolation warnings), but the destructive region-migration sequence ("copy each entry… then delete it from the old one") lacks a verify-copy-before-delete checkpoint — most checkpoints present with a minor validation gap. | 4 / 5 |
Progressive Disclosure | A single well-sectioned file with clear headers and full API detail appropriately deferred to one-level, clearly signaled official-docs links; with no bundle files, ~240 lines in one file (e.g., migration and region detail) is somewhat monolithic, matching the good-structure-with-minor-organization-gaps anchor. | 4 / 5 |
Total | 17 / 20 Passed |