Content
77%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 content is exceptionally actionable and the workflows are clearly sequenced with explicit validation and feedback loops, on par with the rubric's best examples. Its weakness is structural: a monolithic ~1000-line file with no reference files means advanced troubleshooting detail (refresh lifecycle, placeholder contracts, attachment recovery) is always in context, hurting both token efficiency and progressive disclosure.
Suggestions
Split the deep-dive operational detail into one-level-deep reference files (e.g., references/provider-refresh.md for the refresh/Vault/lifecycle contract, references/sandbox-lifecycle.md for attachment recovery and restart semantics, references/byoc.md for image and run_as rules), keeping SKILL.md to an overview plus the common workflows with clearly signaled 'See X for details' links.
Trim the multi-paragraph edge-case narratives inline (e.g., the stdin read-only recovery paragraph and the `stable_placeholder` lifecycle contract) to a one-line summary plus a pointer to the relevant reference section, so routine invocations don't pay the full token cost.
When moving content out, preserve the error-string tables verbatim in the references (they are the skill's highest-value, non-inferable knowledge) and keep the trigger-critical quick workflows (create/exec/logs/policy loop) inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~1000 lines of dense, non-generic operational knowledge (exit codes, error-string semantics like `CONFIG_OPERATION_STORAGE_UNCERTAIN` and `attached read-only; retry input...`), so it rarely explains things Claude already knows. However, several deep edge-case sections — the provider refresh/Vault lifecycle, the `stable_placeholder` credential contract, and the 20-line stdin-attachment recovery paragraph — read like a troubleshooting manual dump that all users of the skill pay for on every invocation, even for a simple `sandbox create`. Anchor 3 ("mostly efficient but could be tightened") fits better than 4 given the volume of inline failure-mode detail. | 3 / 5 |
Actionability | Nearly every section gives copy-paste-ready commands with concrete flags (e.g., `openshell sandbox create --from ... --provider claude-code -- claude`, `openshell policy set dev --policy current-policy.yaml --wait`), specific exit codes (0/1/124 for `--wait`), and exact error strings with named remediations. This is fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | Ten workflows are clearly sequenced; Workflow 4 (policy iteration) has numbered steps, an explicit monitor-observe-modify-push-verify feedback loop with a diagram, validation via `--wait` plus `policy list` revision status checks, and a documented retry/repair path (e.g., `ConfigurationInvalid` repair window, `ProvisioningTimedOut` handling). Destructive operations like `sandbox delete` include validation guidance ("deletion accepted" means pending; inspect before assuming completion). | 5 / 5 |
Progressive Disclosure | There are no bundle files at all (no references/, scripts/, or assets/), so everything — including advanced material that clearly belongs in separate files, like the provider-refresh lifecycle, BYOC UID/GID rules, and attachment-recovery semantics — is inlined in one ~1000-line SKILL.md. The body is well-sectioned with headers and does signal external authorities (`openshell --help` tree and published docs links), which lifts it above the anchor-2 'minimal structure' case, but the absence of any one-level-deep reference split for the deep-dive material keeps it at anchor 3. | 3 / 5 |
Total | 16 / 20 Passed |