Content
96%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.
An excellent single-file skill body: lean, platform-specific reference material with fully actionable MCP tool guidance, a gated validate-before-trigger workflow, and explicit partial-failure recovery. The only refinement opportunity is splitting the dense credential-type and mirror-config field tables into a reference file to sharpen the overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, table-driven, and platform-specific throughout — exact credential fields, defaults inline ("runScheduledEvery (hours, default 0 = off)"), terse lifecycle steps — and never explains concepts Claude already knows (no 'what S3 is' or 'how globs work' padding). It matches anchor 5 (lean, assumes Claude's competence, every token earns its place); the only borderline passages (the provenance note and batch-processing routing callout) are actionable routing guidance, not padding, so it does not fall to anchor 4. | 5 / 5 |
Actionability | As an instruction-only MCP-tool skill, the guidance is fully executable: exact tool names ("connect_cloud_storage", "datasource_validate"), complete parameter specs with types and defaults ("namingStrategy (fullPath|fileName|eTag|metadata, default fullPath)"), return shapes ("{credentialId, credentialCreated, datasourceId, validation, triggered, batchIds?}"), a concrete glob example ("[\"images/**/*.jpg\", \"batch-*/**\"]"), and explicit failure recovery ("status: 'partial' with the ids so you can clean up with credentials_delete / datasource_delete"). This matches anchor 5's copy-paste-ready specificity covering the common cases; per the scoring notes, absence of code is not penalized for an instruction-only skill when guidance is this actionable. | 5 / 5 |
Workflow Clarity | The lifecycle is an explicitly numbered sequence — Create → Validate ("Run it to self-diagnose a bad credential/bucket **before** triggering") → Trigger → Poll — with the validation gate built into the fast path ("trigger (default true) starts a mirror run **only if validation passes**") and an explicit error-recovery loop for partial failures (cleanup via credentials_delete / datasource_delete). This matches anchor 5 (clear sequence, explicit validation, feedback loops); the batch/destructive-operation cap of 3 does not apply because validation is present and prominent. | 5 / 5 |
Progressive Disclosure | Structure is good: clear sections, a tool index table, and well-signaled one-level-deep pointers to related skills ("roboflow://skills/roboflow-data-management/SKILL"), with no nested references. It sits at anchor 4 rather than 5 because field-level reference material — the credential-types table and the mirror_configs settings enumeration (~40 lines) — is inlined in SKILL.md where a references/ split could keep the overview leaner; that is a minor organization gap, not the buried or misplaced content of lower anchors. | 4 / 5 |
Total | 19 / 20 Passed |