Content
70%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.
The body is a strong, domain-specific developer guide with a genuinely well-engineered workflow (explicit validation loops, tool guidance, an extensive pitfalls checklist). Its main weaknesses are repeated content that could be deduplicated, the absence of any concrete resource-file example, and references to instruction files that are not clearly signaled or present in the bundle.
Suggestions
State the duplicate-explore rule once (e.g. in the pitfalls section) and reference it from the other two locations to remove the triple repetition and tighten conciseness.
Include one short, representative example resource file (e.g. a minimal model YAML or metrics view with an inline explore block) to push actionability toward copy-paste ready.
Turn the vague deferrals ('see the metrics view instructions') into explicit, verifiable references with paths, and move the per-resource-type detail into separate reference files so SKILL.md stays an overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is overwhelmingly domain-specific knowledge Claude cannot be assumed to know (Rill's resource types, DAG semantics, default OLAP connector behavior), but it could be tightened: the duplicate-explore warning is repeated three times (Explores section, workflow step 7, Common pitfalls) and the 'expensive resources' caution recurs across sections. This fits anchor 3, 'Mostly efficient but includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | Concrete, executable guidance throughout for an instruction-only skill: exact properties ('olap_connector:', 'managed: true', 'mode: read', 'timeseries:'), named tools ('project_status', 'query_sql', 'wait_until_idle', 'rill validate'), and specific limits ('up to 10 dimensions and up to 5 measures'). This matches anchor 4 ('Mostly executable guidance... minor gaps'); it stops short of anchor 5 because there is no copy-paste-ready example of a resource file (e.g. a sample model or metrics view YAML). | 4 / 5 |
Workflow Clarity | The 'Recommended workflow' provides a clearly sequenced 8-step pipeline with an explicit validation and iterate-until-fixed loop, a blocking 'wait_until_idle' checkpoint for reconciliation, an 'rill validate' fallback when MCP tools are unavailable, and a detailed pitfalls checklist. This matches anchor 5 (clear sequence with explicit validation steps, feedback loops for error recovery, and a checklist). | 5 / 5 |
Progressive Disclosure | Section organization is reasonable, but the body repeatedly defers to 'the metrics view instructions', 'instructions for model development', and skills like 'rill-metrics-view' without any paths or links, and no bundle files exist to back these references. Much of the ~300 lines of per-resource-type detail could also live in separate files. This fits anchor 3 ('references present but not clearly signaled; content that should be separate is inline'). | 3 / 5 |
Total | 15 / 20 Passed |