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-structured skill body: exact commands, runnable code, a socket table, and pitfall-driven guidance make it easy to execute correctly. The main costs are duplicated vite.config/manifest warnings across sections, inline API content that duplicates a reference file, and two dangling use-cases links.
Suggestions
Remove the duplicate vite.config.js and manifest path/entrypoint warnings — keep one authoritative location (the callout or the pitfall) and cross-reference it from the other.
Fix or delete the dangling use-cases/detection-enrichment.md and use-cases/first-app.md links, since no use-cases directory exists in the bundle.
Trim the inline Collection/Workflow/LogScale/Cloud Function/Events code blocks to one minimal example each and point to references/foundry-js.md for the full API surface, which already exists.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and high-signal, but the vite.config.js and manifest path/entrypoint warnings each appear twice — once as dedicated callout sections and again nearly verbatim in Common Pitfalls — and the inline Foundry-JS API examples (collections, workflows, LogScale, cloud functions, events) duplicate material in references/foundry-js.md. This fits 'mostly efficient but could be tightened' rather than the 'minor instances' of level 4. | 3 / 5 |
Actionability | Everything is copy-paste ready: exact `foundry ui` commands with required flags (including the `--sockets` EOF hazard), runnable code for every Foundry-JS API, a socket ID table, exact design-token CSS, and pitfalls phrased with the literal error strings to expect ("got string want integer", "Error: EOF", SecurityError). | 5 / 5 |
Workflow Clarity | The sequence is clear (scaffold → build → cd back to app root → `foundry ui run`/`apps run`), with `foundry apps run` validating the manifest on startup and error-recovery guidance in pitfalls ("if a deploy error mentions entrypoint… revert your changes"). Not a 5: there is no explicit validate-after-build or pre-deploy verification step for the UI itself, leaving minor validation gaps. | 4 / 5 |
Progressive Disclosure | Good structure with a Reading Guide table mapping tasks to all eight existing reference files, one level deep. Not a 5: the body links to `use-cases/detection-enrichment.md` and `use-cases/first-app.md`, which do not exist in the bundle, and the full inline API sections belong in the already-existing foundry-js.md reference. | 4 / 5 |
Total | 16 / 20 Passed |