Content
55%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 has an excellent, validated multi-step workflow but is weighed down by verbose inlined reference material and commands that point to scripts missing from the bundle. Most detailed tables and SQL patterns should move into the existing reference files and be linked.
Suggestions
Move the query-generation patterns, multi-database decision matrix, and sharding/replication tables into the existing reference files (e.g. index_strategy_patterns.md, database_selection_decision_tree.md) and link them, rather than inlining them.
Provide the referenced scripts (schema_analyzer.py, index_optimizer.py, migration_generator.py) in a scripts/ directory, or replace the bash commands with a description of what the scripts should do so the workflow is honest about its current state.
Link all four reference files from the body (currently only database-design-reference.md is referenced) so the normalization guide, index patterns, and decision tree are discoverable instead of duplicating their content inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~310-line body inlines large tutorial-style material Claude largely already knows — JOIN/CTE/window-function/aggregation patterns, multi-database decision matrices, sharding and replication tables — that reads as padded reference content rather than skill-specific guidance. | 2 / 5 |
Actionability | Concrete `python3` commands with flags and sample asset paths look executable, but the referenced scripts (schema_analyzer.py, index_optimizer.py, migration_generator.py) are absent from the bundle with no scripts/ directory, so the commands do not run as written; the inline SQL is copy-paste ready. | 3 / 5 |
Workflow Clarity | The four-step Tool Workflow (analyze → optimize → generate → verify) is clearly sequenced, and step 4 is an explicit verification loop that re-runs analysis on the target schema and runs --validate-only, satisfying the destructive/batch validation requirement. | 5 / 5 |
Progressive Disclosure | Four reference files and three asset files exist and are real, but only database-design-reference.md is linked from the body while the other three references go unused, and substantial content that belongs in those references is inlined as tables and SQL tutorials. | 3 / 5 |
Total | 13 / 20 Passed |