Content
72%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, executable SQL reference with good in-file organization and minimal padding. Its weaknesses are the absence of explicit validation checkpoints (feedback loops) around database changes and the fully inlined reference material with no progressive disclosure into bundle files.
Suggestions
Add an explicit validation workflow (e.g., 1) create table + constraints, 2) EXPLAIN ANALYZE to confirm index usage, 3) if not used, adjust index/operator and re-check) so feedback loops are sequenced, not scattered.
Move the SRID tables, index-selection guide, and full schema examples into references/ files (one level deep, clearly linked from SKILL.md) to enable progressive disclosure.
De-duplicate the SQL-injection guidance (keep it once, either in the intro note or in Common Pitfalls) and trim the "Use X for:" lists that restate the inline code comments.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly lean SQL blocks, tables, and bullet lists that assume PostGIS knowledge, with only minor trims available (the SQL-injection note is restated in "Common Pitfalls", and the "Use X for:" lists repeat the code comments). Not 5 due to that duplication; not 3 because there is no padding explaining concepts Claude already knows. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready DDL and DML: complete CREATE TABLE statements with constraints and generated columns, index creation for each type, and realistic queries covering the common cases (POI radius search, parcel intersect, GPS track assembly, zone containment). | 5 / 5 |
Workflow Clarity | "Before You Start (5 Questions)" and "Core Rules" provide ordering, and validation is mentioned (run EXPLAIN ANALYZE, ST_IsValidReason -> ST_MakeValid), but these are scattered tips rather than explicit validate-then-fix-then-retry checkpoints; the rubric caps workflow clarity at 3 for database-operation skills lacking feedback loops. Not 2 because a decision sequence and validation content do exist. | 3 / 5 |
Progressive Disclosure | A single ~495-line file with clear section headers but zero external references: the SRID tables, index-selection guide, and four full schema examples are reference material that belongs in separate files, matching the some-structure-but-should-be-split anchor. Not 4 because nothing is offloaded to a references/ bundle; not 2 because in-file organization and navigation are good. | 3 / 5 |
Total | 15 / 20 Passed |