Content
77%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 highly actionable, well-sequenced body with excellent executable examples and strong validation checkpoints for destructive/database operations. It is held back by verbosity — dated incident narratives and a history-correction block that bloat the token budget — and by inlined setup detail that a references file could absorb.
Suggestions
Move the dated DATABASE_URL incident narrative and the DDL ownership-history correction into a short 'Known pitfalls / past incidents' (or deprecated-claim) section so the main flow stays lean; lead each with the current rule, not the old wrong one.
Pull the SSH-tunnel setup recipes for --dev and --notifications (the 'Ask an infra owner…' boilerplate and the DEV_DATABASE_URL/NOTIFICATION_DB_REPLICA_URL wiring steps) into a references file (e.g. references/tunnels.md) and link to it from a one-line pointer.
Tighten the 'There is a second skill with this name' warning and the write-guard caveat to the single load-bearing sentence each, cutting the redundant restatement of the banner behavior.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and free of basic-concept padding, but the long DATABASE_URL provenance narrative ("That cost an hour and produced a confidently wrong root cause on 2026-08-16…") and the multi-paragraph history-correction block on DDL push it past a lean budget. The dated incident anecdotes are precisely the time-sensitive material the rubric says should be trimmed unless quarantined in a deprecation section. | 3 / 5 |
Actionability | Every guidance is copy-paste-ready: exact `node .claude/skills/postgres-query/query.mjs --prod "…"` invocations, a full target/options flag table, and worked examples covering simple, dev, EXPLAIN, timeout, file, JSON, and notifications cases. Fully executable with no pseudocode. | 5 / 5 |
Workflow Clarity | Sequences are explicit and paired with validation: the mandatory per-run banner ("Read that line before you trust a result"), the read-only default plus `--writable` permission gate, the replica-rejects-writable error rule, and the performance-comparison recipe (same target on both runs). Destructive/batch operations carry clear feedback loops and warnings, hitting the top anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) and the skill is a single 228-line SKILL.md with the one script invoked by path — appropriate for a small skill, but the dev/notifications SSH-tunnel setup and the DATABASE_URL provenance detail are inlined material that could live one level deep. Structure is reasonable with clear section headers, so it sits above the unstructured 2 anchor but not at the cleanly-split 4. | 3 / 5 |
Total | 16 / 20 Passed |