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.
A high-quality pattern reference: executable, engine-aware SQL and code throughout, disciplined tables instead of prose, and explicit validation expectations for review work. Its main structural weakness is that it is a ~400-line monolith — material like the configuration, security, and connection-pool sections would be better split into referenced files so SKILL.md stays an overview.
Suggestions
Split stable reference material (Configuration, Security, Connection Pools, Full-Text Search) into one-level-deep files under references/ and link them from short SKILL.md sections, moving progressive_disclosure from an all-inline monolith to a proper overview.
Remove duplicate guidance — the SKIP LOCKED caveat appears in both Version Check and Transactions — and trim reminders Claude already knows (e.g. why deep OFFSET is slow, that indexes add write cost) to tighten conciseness toward a 5.
Consolidate the scattered validation guidance (EXPLAIN-first indexing, deadlock capture with SHOW ENGINE INNODB STATUS, migration dry run, rollback criteria) into one explicit step-numbered review workflow with a validate-then-fix-then-retry loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense — tables of defaults, anti-patterns, and EXPLAIN signals plus short code blocks, with almost no conceptual padding. It loses a point for a handful of statements Claude already knows ("Read replicas can lag", "Each index increases write cost…", "Do not use deep OFFSET pagination… it makes the server scan and discard rows") and for repeating the SKIP LOCKED caveat in both the Version Check and Transactions sections. Not 5, but far from the verbosity of anchor 3. | 4 / 5 |
Actionability | Nearly every section delivers copy-paste-ready material: complete CREATE TABLE/INDEX statements, the upsert in both engine-compatible forms, a keyset-pagination query with its backing index, SQLAlchemy and mysql2 pool configs, a ready ini configuration block, and diagnostic SQL. This matches anchor 5 — fully executable guidance covering the common cases — and exceeds anchor 4, where only minor gaps would be tolerated. | 5 / 5 |
Workflow Clarity | A coherent flow exists — 'Start by identifying the engine and version' leads in, and the Output Expectations section requires a validation plan (EXPLAIN, migration dry run, lock/deadlock check, rollback criteria) with per-issue guidance like 'Use EXPLAIN before adding or changing an index' and a bounded deadlock retry loop. It falls short of anchor 5 because validation checkpoints are distributed across sections rather than forming one explicit validate-then-fix-then-retry sequence for the risky migration path. Since database operations demand feedback loops, this is not capped at 3 — the loops are present, just not consolidated. | 4 / 5 |
Progressive Disclosure | There are no bundle files at all (no references/, scripts/, or assets/), so ~413 lines of reference material — schema defaults, full-text search, security, connection pools, and a full server configuration block — are all inlined in SKILL.md. Sections are well-headed and navigable, but content that clearly belongs in separate reference files is inline, which is exactly anchor 3; anchor 4 would require most content to be appropriately placed in separate files with clear references. | 3 / 5 |
Total | 16 / 20 Passed |