Content
75%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 body with executable commands, clear sequencing, and good organization. The main weakness is token efficiency: the same script invocation is repeated ~8 times and the overview duplicates the description, and the SQL pattern library is inlined rather than split into references.
Suggestions
Consolidate the repeated analyze.py invocation blocks: state the command template once with the parameter table, and in the Complete Example and Multi-file Example show only the flag values that change, cutting roughly a third of the body.
Move the Analysis Patterns section (basic exploration, aggregation, window functions, pivot-style SQL) into a one-level-deep reference file (e.g., references/patterns.md) and keep a short pointer plus 2-3 signature examples inline.
Add a brief error-recovery loop for the workflow: what to do when a query fails, when a table name doesn't match the sanitized naming rules, or when a sheet/file is not found — e.g., re-run --action inspect to confirm table names and retry.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body mostly respects Claude's intelligence — it never explains what SQL or Excel is — but the identical `analyze.py` invocation block is repeated ~8 times across Workflow, Complete Example, and Multi-file Example, and the Overview/Core Capabilities sections restate the frontmatter description. Not 4: the repetition is a genuine tightening opportunity, not just minor trimmable instances. | 3 / 5 |
Actionability | Every command is copy-paste-ready with concrete SQL (joins, window functions, pivots), a parameter table, explicit output formats, and two worked end-to-end examples covering the common cases. Not 4: guidance is fully executable with no pseudocode or missing key details. | 5 / 5 |
Workflow Clarity | The sequence (understand requirements → inspect schema → query/summarize → export) is clear, with schema inspection acting as a checkpoint before query construction, and the "Do NOT read the Python file" note sets boundaries. Not 5: there is no error-recovery guidance — no feedback loop for a failed query, table-name mismatch, or missing-sheet case. Not 3: the sequence and checkpoints are present and explicit; queries are read-only so the destructive/batch validation cap does not apply. | 4 / 5 |
Progressive Disclosure | Sections are well-organized with clear headers, the referenced script (scripts/analyze.py) is a real bundle file, and the parameter table and naming rules are appropriately placed inline. Not 5: at ~245 lines, the Analysis Patterns SQL library and the two worked examples could be split into one-level-deep reference files rather than fully inlined in SKILL.md. Not 3: structure is good and nothing that clearly belongs elsewhere is buried. | 4 / 5 |
Total | 16 / 20 Passed |