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.
The body is exceptionally actionable and well-sequenced with strong validation feedback loops and a clean one-level reference structure. Its main weakness is verbosity — duplicated API-mismatch tables and restated rules — that inflates the context cost without adding guidance.
Suggestions
Collapse the two near-identical 'NoSQL/ORM vs PG' method tables (lines ~49–58 and ~203–210) into a single canonical table; the second copy adds no new information.
Trim repeated guardrails — e.g. 'auth.uid() returns text not uuid' and 'do not use current_user in RLS' each appear multiple times — keep one authoritative statement and cross-reference it.
Move the full migration-action enumeration (planMigration/fetchMigration/rollbackMigration/repairMigration) into a reference file, leaving only the core apply/verify loop inline, to reduce the SKILL.md token footprint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and genuinely high-signal, but the body is long (~280 lines) with several padded/ reiterated sections — the NoSQL-vs-PG table appears twice near-identically, migration actions are re-listed, and some guardrails restate rules already covered — so it could be tightened without losing substance. | 3 / 5 |
Actionability | Fully executable guidance throughout: copy-paste CRUD shapes, real MCP tool calls with named parameters (queryPgDatabase(action=...)), exact documented HTTP base URLs, and concrete SQL bundles for GRANT/RLS — covering the common cases copy-paste ready. | 5 / 5 |
Workflow Clarity | A numbered Required Flow (steps 0–12) sequences environment confirmation through DDL/migration, auth, CRUD, and verification, with explicit validation checkpoints (envQuery gate, column-name inspection before CREATE, read-back verification after DDL, RLS error inspection) and feedback loops (MIGRATION_TASK_TIMEOUT → describeMigrationTask, retry-twice-then-inspect). | 5 / 5 |
Progressive Disclosure | SKILL.md acts as a concise overview pointing to eight well-signaled one-level-deep references (references/index.md plus topic files), all of which exist as real files, with a Reference index section and a sibling-skill routing map; detail is appropriately split out and navigation is explicit. | 5 / 5 |
Total | 18 / 20 Passed |