Content
57%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 a dense, actionable SQL cookbook with strong code examples, but it is over-long for SKILL.md, inlines reference material that should live in separate files, and—critically—references six bundle files that do not exist. Migration workflows also lack explicit validation feedback loops.
Suggestions
Create the missing bundle files (scripts/query_optimizer.py, migration_generator.py, schema_explorer.py and references/query_patterns.md, optimization_guide.md, orm_patterns.md) or remove the references and inline only what is needed.
Move the dialect-differences, transaction-isolation, and backup-recipe tables out of SKILL.md into reference files so the main skill stays a lean overview with one-level-deep pointers.
Add explicit validate→fix→retry checkpoints to the migration and backfill workflows (e.g., verify row counts and integrity after each batch, halt and fix on mismatch) to satisfy the database-operations feedback-loop expectation.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly concrete code and tables with little prose padding, but ~450 lines inline reference-grade material (dialect-differences table, transaction-isolation table, backup recipes for three engines, a 10-row anti-patterns table) that the body itself implies belongs in the referenced files. | 3 / 5 |
Actionability | Abundant copy-paste-ready SQL/bash/Python/Prisma/TypeScript examples, but the three invoked scripts (query_optimizer.py, migration_generator.py, schema_explorer.py) do not exist in any scripts/ directory, so those commands are not actually runnable. | 4 / 5 |
Workflow Clarity | Multi-step sequences are present (translation patterns, EXPLAIN workflow, expand-contract migration) but validation checkpoints are mostly missing or implicit; per the rubric, a database/migration skill lacking validate-fix-retry feedback loops is capped at 3. | 3 / 5 |
Progressive Disclosure | Section structure is clean and references are clearly signaled one level deep ("> See references/..."), but no references/ or scripts/ bundle directories exist, so all six referenced files are broken, and bulk reference material is inlined rather than split out. | 3 / 5 |
Total | 13 / 20 Passed |