Content
100%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.
This is an exceptionally well-written operational skill: lean, dense with hard-won CLI specifics, fully executable commands, and explicit confirmation gates around destructive operations. The only trifling gap is that step 6's release command uses <deployment-id> without stating where it comes from (the deploy output), which a competent operator will infer.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every sentence carries non-obvious operational knowledge Claude could not infer — e.g. "list-deployments only runs next to a manifest.yml with an app_id and a name, even with --app-id" and "Never retry by stripping IDs: that creates a second app next to the customer's". Nothing explains concepts Claude already knows, and no dates or version numbers pad the body, matching anchor 5's "every token earns its place". | 5 / 5 |
Actionability | All commands are copy-paste ready with exact flags (the stub-manifest workaround, "foundry apps sync --app-id <app-id> --deployment-version <version> -d $WORK/deployed --replace-all --no-git --no-prompt"), decision logic is driven by concrete exit codes ("Exit code 2 means the plan removes an artifact..."), and failure signatures are quoted verbatim ("Error: ... has entries with IDs but no name", "The templates API returned 403") with remedies — anchor 5. | 5 / 5 |
Workflow Clarity | The 7-step procedure is explicitly sequenced with validation checkpoints throughout: step 1 confirms profile/CID ownership, step 5 gates destructive removals on exit code 2 with user confirmation, step 6 runs "foundry apps validate" before deploy, and "When it fails" provides error-recovery paths (stop, keep $WORK, use debugging-workflows) — anchor 5's explicit validation with feedback loops; the destructive-operation cap does not apply since validation is present. | 5 / 5 |
Progressive Disclosure | The ~60-line body keeps the procedure inline while offloading verification detail to a single, clearly signaled, one-level-deep reference — "See [references/live-verification.md](references/live-verification.md) for what was checked against a live CID and the checklist for re-testing" — and the script lives in scripts/upgrade_manifest.py as cited; both bundle files exist and contain exactly what is claimed, matching anchor 5. | 5 / 5 |
Total | 20 / 20 Passed |