Content
88%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 high-quality instruction skill: concrete commands, executable SQL patterns, explicit validation loops, and genuinely Supabase-specific traps that Claude would not reliably know. The main improvement opportunities are moving the long security checklist to a reference file for on-demand loading and trimming the few passages that restate general Postgres or generic agent knowledge.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with Supabase-specific knowledge Claude cannot be assumed to know (Data API exposure vs RLS, 'auth.role() deprecation', 'TO authenticated alone is BOLA', storage upsert needing INSERT+SELECT+UPDATE), and commands/URLs are given without padding. It stays below 5 because a few items re-explain general Postgres/agent knowledge Claude already has — e.g., 'SECURITY DEFINER functions bypass RLS' and principle 3's generic 'recover from errors, don't loop' advice — which could be trimmed. | 4 / 5 |
Actionability | Guidance is copy-paste executable throughout: 'curl -so /dev/null -w "%{http_code}" https://mcp.supabase.com/mcp', 'supabase migration new <name>', 'supabase db pull <descriptive-name> --local --yes', 'supabase migration list --local', complete SQL policy examples with USING/WITH CHECK, and concrete doc-fetch URLs. Priority-ordered documentation methods and version-gated fallbacks (MCP execute_sql if CLI < v2.79.0) cover the common cases fully. | 5 / 5 |
Workflow Clarity | Multi-step processes are explicitly sequenced with validation checkpoints and feedback loops: the migration commit workflow runs advisors ('Fix any issues') before generating, then verifies with 'supabase migration list --local'; MCP troubleshooting is an ordered 1-2-3 diagnostic sequence; principle 2 mandates post-fix verification; and the security checklist anchors risky database work. This matches the top anchor including the checklist and error-recovery guidance. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned and its one bundle reference is clearly signaled ('MUST read when', references/skill-feedback.md, verified to exist and be one level deep). It is not a 5 because the ~45-line inlined security checklist is exactly the kind of detailed material that could live in a references/ file loaded on demand, and the reference chain reaches two hops (SKILL.md → references/skill-feedback.md → assets/feedback-issue-template.md), leaving minor organization gaps against the 'everything appropriately split' anchor. | 4 / 5 |
Total | 18 / 20 Passed |