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 highly actionable, information-dense command reference with copy-paste-ready examples and well-chosen, Depot-specific content that assumes Claude's competence. Its main weaknesses are the 272-line monolithic structure with specialized sections (Registry, output formats, pricing) inlined instead of split into reference files, and minor redundancy (the gha cache warning appears twice).
Suggestions
Move the specialized Depot Registry section (pull-through cache configuration, token scoping, pull-token/push commands) into a references/registry.md file and keep a short overview with common `--save`/`depot pull` commands in SKILL.md.
Move the special output formats (estargz, zstd, SOCI) into a references/output-formats.md file, keeping only a one-line pointer plus the most common case inline.
Deduplicate the `--cache-from type=gha` / `--cache-to type=gha` guidance, which appears both in the Migration section and the Common Mistakes table — state it once and cross-reference it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with executable commands and Depot-specific facts Claude would not know (builder pricing, autoscaling behavior, registry subdomains), with almost no explanation of familiar concepts. Minor trimmable redundancy remains: the `--cache-from type=gha` warning appears both in the Migration section and again in the Common Mistakes table, and the Registry prose (pull-through cache details, token scoping) runs long — so it sits at "efficient with minor over-explanation" rather than "every token earns its place". | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: `depot build -t repo/image:tag . --push`, a complete HCL bake file example, per-service compose YAML, and error-specific fixes such as `DOCKER_API_VERSION=1.52 depot build --load .` and `--provenance=false` for unknown/unknown. Specific examples cover the common cases (build, load, push, save, multi-platform, secrets, migration). | 5 / 5 |
Workflow Clarity | Sequences are clear where they matter — compose integration ("depot bake -f docker-compose.yml --load" then "docker compose up") and the migration steps — and verification signals exist (`--lint`, `--print`, "look for [depot] prefix in logs", `depot image list --output json`). It falls short of a 5 because there are no explicit validate-then-proceed feedback loops, though the skill is a command reference rather than a fragile multi-step pipeline, so the missing-loop cap for destructive/batch operations does not apply. | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned 272-line monolith with no bundle files and no references: specialized content that clearly belongs in separate reference files — the ~40-line Depot Registry section (pull-through cache, token scopes), special output formats (estargz/zstd/SOCI), and the pricing/autoscaling details — is inlined in SKILL.md. This matches the 3 anchor ("content that should be separate is inline"); the under-50-line simple-skill exception does not apply at this length, and it is not a 4 because there are no references at all to be "mostly clear". | 3 / 5 |
Total | 16 / 20 Passed |