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 well-sequenced workflow with validation checkpoints, but it ships none of the referenced tools or sample assets, making the core "run these" workflow non-executable, and it bloats the context window with generic database knowledge that should live in reference files.
Suggestions
Ship the referenced scripts (schema_analyzer.py, index_optimizer.py, migration_generator.py) and assets (sample_schema.sql/json, sample_query_patterns.json) referenced in the workflow, or remove the "run these — do not analyze by hand" directive if the tools are unavailable.
Move the generic SQL patterns, index/EXPLAIN/N+1, connection pooling, sharding, replication, and multi-DB comparison sections into references/ files, keeping only the tool workflow and migration/expansion-contract guidance in SKILL.md.
Provide a concrete rollback-verification step in the workflow (e.g., re-run `schema_analyzer.py` on the post-migration schema and assert the first-pass findings are resolved) rather than leaving rollback as prose advice.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~310-line body is noticeably verbose, inlining large amounts of general database knowledge Claude already has — generic SQL JOIN/CTE/window-function/aggregation examples, index-type tables, EXPLAIN reading, N+1 detection, connection pooling, and sharding/replication matrices — padded against a smaller core of genuinely skill-specific content. | 2 / 5 |
Actionability | The tool workflow gives detailed, concrete commands (e.g., `python3 schema_analyzer.py --input schema.sql --generate-erd ...`), but the referenced scripts and sample assets (schema_analyzer.py, index_optimizer.py, migration_generator.py, assets/*.json) do not exist in the bundle, so the commands are not actually runnable as written. | 3 / 5 |
Workflow Clarity | A clear 4-step sequence (analyze → optimize → migrate → verify) with an explicit validation/feedback loop in step 4 and a `--validate-only` checkpoint; the destructive/batch cap does not apply since validation is present, though the verify step and rollback checklists could be more concrete. | 4 / 5 |
Progressive Disclosure | Despite a clearly signaled one-level reference ("→ See references/database-design-reference.md") and section headers, roughly 200 lines of general reference material that belongs in separate files is inlined in SKILL.md, and the referenced bundle files (scripts/, assets/, references/) do not exist, leaving navigation broken. | 2 / 5 |
Total | 11 / 20 Passed |