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 well-crafted instruction-only skill body: concrete, sequenced procedures with validation and user-confirmation around destructive operations, a genuinely instructive worked example, and delegation of detail to external files instead of duplication. Its main lever is length — the three subcommand sections could be split into reference files to make the overview leaner.
Suggestions
Split each subcommand's detailed procedure ('compact', 'rls scenarios', 'align') into a reference file, keeping SKILL.md to the three contracts, the classification table, and the when-to-invoke summaries — the body is ~275 lines where a leaner overview would suffice.
Tighten the 'Spell out the silent edges' bullet list and the rhetorical asides ('A test failing because the policy is wrong is the test doing its job') into terse checklists — several sentences restate the same injunction.
Add an explicit error-recovery step after 'supabase db reset' (e.g. 'if reset or tests fail, fix the consolidated SQL and re-run') to close the feedback loop the validation checkpoints imply.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and prescriptive — no padded explanation of what migrations, RLS, or Supabase are, and judgment calls are compressed into table form (the migration classification table, the schemas-vs-migrations concern table). A few rhetorical flourishes ('A test failing because the policy is wrong is the test doing its job', 'Read both') and the long prose enumeration of RLS silent edges could be trimmed, but most tokens earn their place — matching anchor 4's 'efficient; minor instances of over-explanation'. | 4 / 5 |
Actionability | For an instruction-only skill the guidance is fully concrete: numbered procedures for all three subcommands, a classification table with explicit signals and allowed actions, a worked example with real filenames showing both the right and the wrong move, exact commands ('supabase db reset', 'supabase test db', 'supabase migration new'), named personas, and specific assertion anti-patterns ('Replace is(count, 0) with ok(count >= 0)'). The rubric's scoring note confirms instruction-only skills are judged on guidance concreteness, which this fully meets — matching anchor 5's 'specific examples cover the common cases'. | 5 / 5 |
Workflow Clarity | Each subcommand has a clearly sequenced numbered procedure with explicit validation checkpoints: compact requires classifying every migration, verifying the classification with the user before any destructive step, then a post-merge 'supabase db reset' plus 'supabase test db' check; align ends with a 'db reset' smoke check and forbids touching migrations; rls scenarios mandates both positive and negative assertions. Because destructive operations (deleting migration files) are guarded by classification, user confirmation, and post-hoc resets, the missing-validation cap at 3 does not apply — this matches anchor 5's clear sequence with explicit validation for risky operations. | 5 / 5 |
Progressive Disclosure | The body is well organized into three clearly headed subcommand sections with per-section 'When to invoke' and procedure subheadings, and it delegates rather than duplicates — 'Skeleton + fixture/session conventions live in supabase/AGENTS.md § RLS testing — point readers there rather than re-list', plus one-level-deep links to AGENTS.md, seed.sql, and the opt-library skill. It falls short of anchor 5 only because at ~275 lines covering three distinct workflows, the per-subcommand detail is a natural candidate for reference files that would keep the top-level SKILL.md a leaner overview. | 4 / 5 |
Total | 18 / 20 Passed |