Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with a clear, validated multi-stage workflow, but it leans long with some over-explained passages and a monolithic structure. The most concrete defect is a broken reference: edit_dictionary.py is cited as a bundle file beside SKILL.md but does not exist in the skill directory.
Suggestions
Provide the missing edit_dictionary.py (Stage 2.5 and Guardrails reference it as "beside this SKILL.md" / "$SKILL_DIR/edit_dictionary.py", but no such file exists in the skill bundle) or remove the references.
Move the ~80-line inline GeoJSON feature-id-key inspector into a scripts/ file and reference it, reducing the SKILL.md body length and improving progressive disclosure.
Tighten the repo-local packaging rationale and de-duplicate the out-of-band TUI warning that appears in both Stage 2.5 and the Guardrails section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Per-line it is tight and free of basic-concept padding, but the ~480-line body and several over-explained passages — the repo-local packaging rationale, the inline ~80-line GeoJSON inspector, and the out-of-band TUI warning repeated in both Stage 2.5 and Guardrails — could be tightened, matching the "mostly efficient but could be tightened" anchor. | 2 / 3 |
Actionability | Provides fully executable, copy-paste-ready bash and python — the denull/describegpt/viz-smart invocations with real flags, inline verification scripts, and concrete remediation branches — matching the fully-executable anchor rather than pseudocode. | 3 / 3 |
Workflow Clarity | A clearly ordered five-stage pipeline ("in this order and no other") with explicit validation checkpoints (Stage 0 preconditions, the dictionary coverage check, Stage 5's test -s and python verification) and feedback loops for the destructive denull step, satisfying the highest anchor including for batch/destructive operations. | 3 / 3 |
Progressive Disclosure | It has clear section/stage structure, but no references/, scripts/, or assets/ bundle directories exist and the body's referenced edit_dictionary.py ("beside this SKILL.md") is absent — a broken reference — while the large GeoJSON inspector and TUI helper are inlined rather than split into referenced files, matching the "content that should be separate is inline" anchor and falling short of the well-split overview. | 2 / 3 |
Total | 10 / 12 Passed |