Content
88%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 dense, highly actionable reference with complete JSON/SQL examples and an explicit verification feedback loop for the destructive mutation tools. The main weakness is repetition of the top silent-failure rule and length that could be split into a separate reference file.
Suggestions
Consolidate the "groupBy ignored unless addOns.groupBy is true" warning to one canonical location and cross-reference it, instead of restating it in the silent-failures list, the group-by tokens section, and the addOns section.
Consider moving the large lookup tables (panel types, aggregations per source, group-by tokens, display config) into a references/ file generated by the same mcp:docs command, keeping SKILL.md as the overview plus key traps.
Add a short top-of-file table of contents so readers can jump to the relevant section in the ~480-line body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense and assumes Claude's competence (no preamble on what dashboards/ClickHouse/percentiles are) and leads with Maple-specific traps. It sits below 5 because the "groupBy ignored unless addOns.groupBy is true" rule is restated four times across sections and the file runs ~480 lines with some reinforcement that could be trimmed, matching the score-4 anchor of efficient content with minor over-explanation. | 4 / 5 |
Actionability | Multiple complete copy-paste-ready JSON examples (query, raw_sql, static, breakdown, reduceToValue, full assembled widget) plus a complete macro-based SQL example, alongside tables of valid aggregations, group-by tokens, panel types, and units, and concrete commands (bun run --cwd apps/api mcp:docs, describe_warehouse_tables). This matches the score-5 anchor of fully executable, copy-paste-ready guidance covering common cases. | 5 / 5 |
Workflow Clarity | The mutation workflow has an explicit validate-fix-resubmit loop ("MCP success is not chart correctness… Read the verdict: suspicious or broken means fix and resubmit") and a prerequisite checkpoint ("Call describe_warehouse_tables first"), plus a clear simplified-vs-raw entry decision. The destructive/batch cap-at-3 rule does not apply because validation checkpoints are present, matching the score-5 anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the skill is a single well-sectioned file (When to use, Panel types, Data sources, Units, Queries, Display config, Raw SQL) with no nested references. It is below 5 because at ~480 lines of dense reference tables (panel types, aggregations, group-by tokens, display config) some material could plausibly live in a separate file, matching the score-4 anchor of good structure with minor organization gaps; cohesion from a single generated schema module keeps it from dropping to 3. | 4 / 5 |
Total | 18 / 20 Passed |