Content
71%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, well-checkpointed onboarding procedure whose executable templates and validation loops are excellent. Its weaknesses are structural: a monolithic single-file layout with no references/ bundle, duplicated troubleshooting content, and a step-numbering error (Step 10 missing) that slightly mars navigation.
Suggestions
Move the configuration field reference, the CI workflow template, and the Troubleshooting/.dockerignore section into references/ files (e.g. references/config.md, references/ci.md, references/troubleshooting.md) and link them from the body, keeping SKILL.md as an overview.
Fix the step numbering: Step 9 jumps directly to Step 11 (no Step 10), and Step 13's decline branch says "skip Steps 13 and 14" while inside Step 13 — renumber and reword.
Deduplicate troubleshooting: Step 8's "Common issues and fixes" list and the Troubleshooting Reference table cover the same errors — keep one and reference it from the other.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and command-driven, but troubleshooting items ("Exec format error", "Token validation failed", "No such file or directory") are repeated verbatim in both Step 8's fix list and the Troubleshooting Reference table, and the Dockerfile git-comment blocks are duplicated across the Python and Rust templates. Mostly efficient, but several sections could be tightened. | 3 / 5 |
Actionability | Fully executable throughout: complete Dockerfile templates, offload.toml variants for pytest/cargo/default frameworks, a copy-paste scripts/offload-tests.sh, a full GitHub Actions workflow, and a symptom/cause/fix troubleshooting table. Placeholders are explicitly flagged ("values in <angle-brackets> are placeholders you must substitute"), and common cases (pytest, cargo, vitest, monorepo) each get concrete examples. | 5 / 5 |
Workflow Clarity | Steps are clearly sequenced with strong validation checkpoints and feedback loops ("Do not continue until all prerequisites are confirmed", "Repeat until offload collect succeeds", baseline comparison before optimization, "Do not proceed until they match"). Not 5 because step numbering skips Step 10 entirely (Step 9 jumps to Step 11) and Step 13's "If they decline, skip Steps 13 and 14" self-references confusingly — minor coherence gaps, not missing validation. | 4 / 5 |
Progressive Disclosure | No references/ bundle exists; the full configuration field reference, CI workflow template, and troubleshooting guide are all inlined in a single ~585-line SKILL.md, with only a vague external pointer ("see the Offload README") instead of well-signaled bundle files. Section headers give reasonable structure, so this sits at "some structure, content that should be separate is inline". | 3 / 5 |
Total | 15 / 20 Passed |