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.
A highly actionable, code-dense reference whose commands and examples are executable and cover the main Supabase Edge Function use cases. Its weaknesses are structural: significant duplication between the body and the existing patterns reference file, a missing referenced file (deno-imports.md), and the absence of debugging/verification steps around the deploy workflow.
Suggestions
Create the missing references/deno-imports.md or remove the dead link; verify every referenced path exists in the bundle.
Move the extended pattern examples (Stripe webhook, database webhook, cron, shared code) into references/function-patterns.md and keep SKILL.md as a concise overview, eliminating the CORS/client-init duplication.
Add verification steps to the workflow: 'supabase functions logs <name>' for debugging and a post-deploy curl smoke test against the production functions URL.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-forward with almost no conceptual padding — sections like 'Create Function' and 'Deploy Function' are just commands and complete examples. It does not reach 5 because of redundancy: the hello-world handler appears in three variations (basic, CORS, shared), Supabase client initialization is repeated across three sections, and pinned version/date strings ('npm:stripe@14', apiVersion '2023-10-16') are time-sensitive details not isolated in a deprecated section. | 4 / 5 |
Actionability | Every section provides copy-paste-ready, executable code or commands covering the common cases: creating and deploying functions, serving locally, curl testing with auth headers, CORS preflight handling, JWT user extraction, secrets management, Stripe webhook verification, pg_cron scheduling, and config.toml settings. This matches the top anchor fully; only appropriate placeholders (<ANON_KEY>, <project>) remain. | 5 / 5 |
Workflow Clarity | The create → serve locally → test with curl → deploy sequence is laid out in logical section order with concrete commands per phase and an explicit local test checkpoint ('curl -i --request POST .../functions/v1/hello-world'). It falls short of 5 because there is no post-deploy verification step, no error-recovery guidance, and no mention of 'supabase functions logs' for debugging failures. | 4 / 5 |
Progressive Disclosure | A References section signals two files, but 'references/deno-imports.md' does not exist in the bundle — a dead link — while 'references/function-patterns.md' exists but covers pattern material duplicated inline (CORS, client creation, API endpoints). The ~400-line body inlines substantial pattern content that belongs in the existing patterns reference. This matches 'some structure but could be better organized; content that should be separate is inline' rather than 4, since one reference is broken and the split between overview and reference file is not clean. | 3 / 5 |
Total | 16 / 20 Passed |