Content
78%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 tight, instruction-only router skill with concrete paths, a runnable detection command, and an explicit failure checkpoint. Its weaknesses are minor: redundant restatement of SDK.md's scope between the intro and the Scope section, and no guidance for the authenticated-bucket branch after the status check.
Suggestions
Cut the duplicated scope sentence ('SDK.md owns how DataChain code is written — API usage, UDF signatures...') and 'It is self-sufficient on its own'; the intro already tells the reader to read SDK.md in full and what it contains.
Add one line for the success branch of the bucket check, e.g. what to do when `anon: false` or when `datachain bucket status` succeeds (which credentials/settings to check before writing pipeline code).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with no concept over-explanation, but the intro paragraph and the 'Scope of this skill' section state nearly the same thing twice ("It holds the SDK rules: API usage, UDF signatures..." vs "`SDK.md` owns how DataChain code is written — API usage, UDF signatures...") and "It is self-sufficient on its own" is filler — a small trim, matching the 'minor instances that could be trimmed' anchor rather than the fully lean anchor. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: named paths (`dc-knowledge/index.md`, `dc-knowledge/datasets/`, `dc-knowledge/buckets/`), a concrete check (`anon: true/false` in frontmatter), and a runnable command (`datachain bucket status <uri>`) with an explicit failure action. It falls short of fully executable because the authenticated-bucket path after detection is unspecified — no command or next step for the non-anon case. | 4 / 5 |
Workflow Clarity | The 'Before writing any pipeline code' sequence is clearly numbered with an explicit error checkpoint ("If `denied` or `not found`, stop and ask the user") and a hard safety rule ("Never create or modify files under `dc-knowledge/`"). Anchor 5 is not met because the checkpoint only covers the failure branch — there is no validation of the success branch (what to do differently when `anon` is true vs false), leaving a minor validation gap. | 4 / 5 |
Progressive Disclosure | A short, well-sectioned overview that delegates all SDK detail to a single one-level-deep, clearly signaled reference ("Read `{skill_dir}/SDK.md` in full") with an accurate scope summary of its contents, and clearly partitions responsibility with the sibling knowledge skill. No bundle files are present to inline incorrectly; navigation is unambiguous. | 5 / 5 |
Total | 17 / 20 Passed |