Content
45%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 skill has a genuinely well-structured tool workflow with an explicit verification loop, but it is undermined by three referenced scripts that are missing from the bundle, roughly half the body being SQL textbook material Claude already knows, and three of four reference files never being pointed to. The core guidance is trustworthy in shape but not currently executable as written.
Suggestions
Add the missing schema_analyzer.py, index_optimizer.py, and migration_generator.py to the bundle (e.g., in scripts/) or rewrite the workflow to use commands that actually exist — as written, none of the four workflow steps can be executed.
Move the Query Generation Patterns, Sharding & Replication, and Connection Pooling sections into the existing reference files and replace them with one-line pointers; this material is standard SQL knowledge and costs context without adding skill-specific value.
Reference database_selection_decision_tree.md, index_strategy_patterns.md, and normalization_guide.md from the relevant body sections (e.g., point the Multi-Database Decision Matrix and Indexing Strategies sections at them) so the bundled references are discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly half the body restates standard SQL knowledge Claude already has — the JOIN/CTE/window-function examples under "Query Generation Patterns", the Connection Pooling and Read Replicas sections, the Sharding & Replication tables, and Best Practices bullets like "Use meaningful names" and "Validate inputs: Prevent SQL injection attacks". That is several padded sections of unnecessary explanation, matching anchor 2 rather than 3, where only isolated instances would remain after trimming. | 2 / 5 |
Actionability | The tool-workflow commands are formatted as copy-paste ready ("python3 schema_analyzer.py --input schema.sql --generate-erd --output-format json -o analysis.json"), but the three scripts they invoke — schema_analyzer.py, index_optimizer.py, migration_generator.py — do not exist anywhere in the bundle, so the core workflow cannot actually be executed; the remaining SQL snippets are executable but generic. This is anchor 3: concrete-looking guidance with a critical missing piece, not the executable-and-complete anchor 4. | 3 / 5 |
Workflow Clarity | The four-step tool workflow is clearly sequenced and includes a genuine validation loop ("Re-run step 1 on the *target* schema and assert the issues found in the first pass are gone; run migration_generator.py --validate-only before handing over"), satisfying the database-operations feedback-loop requirement. It falls short of anchor 5 because there is no error-recovery guidance (what to do when the analyzer flags issues that cannot be fixed, or when a migration fails --validate-only). | 4 / 5 |
Progressive Disclosure | Only one of the four reference files is signaled from the body ("→ See references/database-design-reference.md for details"); database_selection_decision_tree.md, index_strategy_patterns.md, and normalization_guide.md are never mentioned and are effectively undiscoverable, while ~120 lines of sharding, replication, and query-pattern content that belongs in those references is inlined in SKILL.md. Buried references plus inlined reference-grade content matches anchor 2; structure does exist, but the reference wiring failure is well below the midpoint. | 2 / 5 |
Total | 11 / 20 Passed |