Content
77%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable and its schema-change/troubleshooting workflows are clearly sequenced with validation and feedback loops. The main weaknesses are minor prose padding and a fairly monolithic inline security checklist that could be factored into a reference file.
Suggestions
Extract the 'Security checklist' (principles 6 with its many sub-bullets) into a dedicated reference file and link to it from SKILL.md, keeping only a short pointer and the most critical one-liners inline.
Tighten Principle 3 and the explanatory sentences under bolded checklist headers where the bold header already conveys the rule, to reduce token cost.
Collapse the two-level reference chain by having SKILL.md point directly to the feedback template alongside skill-feedback.md, or inline the small template into skill-feedback.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with Supabase-specific knowledge Claude likely lacks (CLI version requirements, RLS edge cases, deprecated auth.role()), but includes padding such as Principle 3's "Supabase issues are not always solved by retrying the same command, and the answer is not always in the logs..." and rationale restated under already-bolded checklist headers, matching score 2's 'mostly efficient but could be tightened' rather than score 3's 'every token earns its place'. | 2 / 3 |
Actionability | It provides fully executable guidance — concrete commands ("supabase db pull <descriptive-name> --local --yes", "curl -so /dev/null -w %{http_code} https://mcp.supabase.com/mcp"), named MCP tools (execute_sql, get_advisors, search_docs), and copy-paste SQL policy examples contrasting deprecated vs. correct patterns — matching the score-3 'fully executable code/commands; copy-paste ready' anchor. | 3 / 3 |
Workflow Clarity | The schema-change workflow is a clearly sequenced numbered list with explicit validation checkpoints ("Run advisors... Fix any issues", "Verify: supabase migration list --local"), a feedback loop (Principle 3: stop after 2-3 attempts and reconsider), and a checklist for complex processes, matching the score-3 anchor and avoiding the destructive-operation cap since validation is present. | 3 / 3 |
Progressive Disclosure | Section organization is clear and the single reference is well-signaled ("MUST read when the user reports..." → references/skill-feedback.md), but the large inline security checklist is content that could be split into its own reference file, and the reference chain runs two levels deep (SKILL.md → references/skill-feedback.md → assets/feedback-issue-template.md), matching score 2's 'content that should be separate is inline' rather than score 3's 'one-level-deep references'. | 2 / 3 |
Total | 10 / 12 Passed |