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.
Highly actionable and clearly sequenced, with concrete checks, examples, and a well-defined output format. The main weaknesses are over-explanation of well-known analytical concepts and a monolithic single-file structure that inlines reference material instead of splitting it into bundle files.
Suggestions
Move the Common Data Analysis Pitfalls catalog and Documentation Standards (analysis template, code docstring example, version control notes) into references/ files (e.g., references/pitfalls.md, references/documentation.md), keeping only the pitfall names and a one-line pointer in SKILL.md.
Trim the pitfall entries to name + detection/prevention check — drop 'The problem' explanations and worked arithmetic for concepts Claude already knows like survivorship bias, timezone mismatches, and average of averages.
Fix or remove the dangling [CONNECTORS.md](../../CONNECTORS.md) link, which does not resolve from the skill's location.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The workflow and checklists are tight, but the pitfall catalog spends prose explaining concepts Claude already knows (survivorship bias, timezone mismatches, average-of-averages arithmetic) and ~70 lines of documentation templates/docstring examples pad the file. | 3 / 5 |
Actionability | Fully concrete throughout: runnable SQL for join-explosion detection, a worked weighted-average example, a sanity-check table, specific red flags (e.g., rates at exactly 0% or 100%), and a copy-paste output format template. | 5 / 5 |
Workflow Clarity | Eight clearly sequenced steps with an explicit pre-delivery checklist, a defined 3-level confidence gate, and a stated feedback loop ('If the validation finds issues, fix them and re-validate'). | 5 / 5 |
Progressive Disclosure | No bundle files exist, yet ~380 lines inlines reference-grade material (the pitfall catalog, documentation standards/templates) that belongs in separate files, and the CONNECTORS.md link does not resolve. Section organization itself is good. | 3 / 5 |
Total | 16 / 20 Passed |