Content
75%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-sequenced skill body whose value is its density of non-obvious platform quirks and copy-paste commands. Its main costs are triple-stated warnings that inflate the token budget, a missing error-recovery loop around validation, and a Use Cases section referencing files absent from the bundle.
Suggestions
State each recurring warning once (e.g., the `--description` 50-character cap, 'don't trim vendor specs', 'skip x-cs-operation-config unless asked') and let the Common Pitfalls section be the single recap location instead of repeating them in the intro, workflow, and pitfalls sections.
Remove or ship the `use-cases/*.md` files referenced in the Use Cases section — as written they are dangling references that fail navigation.
Add a short error-recovery step after `foundry apps validate --no-prompt` (what to check and fix when validation fails) to close the feedback loop in the main workflow.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with genuinely non-obvious, Foundry-specific knowledge (50-char description limit, https:// stripping, clientCredentials-only), but several points are repeated three times — the `--description` 50-char cap appears in the intro quote, step 3, and Common Pitfalls; 'Foundry handles large specs fine / don't trim' and 'skip x-cs-operation-config unless asked' each recur three times. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the 'minor instances' of the 4 anchor. | 3 / 5 |
Actionability | Nearly everything is copy-paste executable: exact `foundry api-integrations create` invocations, a complete venv bootstrap for the adapt script, JSON/YAML config snippets, working JavaScript for calling integrations, and grep/python3 recipes for patching large specs without reading them. Path placeholders like `/path/to/foundry-skills/` are explicitly justified by the 'resolve relative to this SKILL.md' instruction, so this is fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | The three-step Download → Adapt → Register workflow is explicitly ordered, with a validation checkpoint ('Validate immediately after registering (`foundry apps validate --no-prompt`)') and a hook safety net for a missed step. It falls short of the 5 anchor because there is no error-recovery loop — what to do when validation or the import fails is not addressed (only the auth-field exception is), so checkpoints are present but feedback loops are incomplete. | 4 / 5 |
Progressive Disclosure | The body is a well-signaled overview with details pushed one level deep into three real, substantive reference files (spec-adaptation-examples.md, auth-examples.md, calling-patterns.md), plus a Reading Guide table and a scripts/ helper. The gap keeping it from 5: the Use Cases section points to `use-cases/http-actions.md` and two sibling files that are not present in the bundle — dangling references that break navigation. | 4 / 5 |
Total | 16 / 20 Passed |