Content
78%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.
Highly actionable reference content with unusually good real-world pitfalls (silent SDK validation failures, no compare-and-set, schema immutability after deploy) and genuine validation checkpoints. Its weaknesses are structural: some duplicated sections, no ordered end-to-end workflow, and two referenced use-case files that are missing from the bundle.
Suggestions
Fix the dead links: either add the `use-cases/collections.md` and `use-cases/lookup-table-enrichment.md` files or remove/replace those references — a reader following them hits nothing.
Deduplicate: the collection name/description constraints and workflow_integration YAML each appear twice (dedicated sections plus Common Pitfalls / Manifest Configuration); keep one canonical table and cross-reference it.
Add a short ordered "New collection checklist" (write schema to /tmp → check description constraints → `collections create` → verify manifest entry → wire access) so the existing validation checkpoints form an explicit sequence rather than scattered guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly all content is earned, non-generic platform knowledge (limits tables, draft-7-only rule, upsert semantics, paging-by-start-key) with no tutorials on concepts Claude already knows, but there is duplication: collection name/description constraints and the workflow_integration YAML each appear in both a dedicated section and Common Pitfalls, and the 60-line TypeScript IncidentCollection class could be trimmed. Fits anchor 4 (efficient with minor trimming possible) rather than 5 or 3. | 4 / 5 |
Actionability | Copy-paste-ready throughout: complete `foundry collections create` commands with flags, exact REST endpoints, two full valid JSON schemas, working TypeScript and falconpy Python snippets, an FQL syntax table, and manifest YAML blocks. Covers the common cases and includes error-path specifics (404 vs 429, result.errors messages); matches the anchor-5 example. | 5 / 5 |
Workflow Clarity | Validation checkpoints exist — "Check the description against this table before running `collections create`", "Always check `result?.errors?.length` after write operations", "Verify persistence with a follow-up read or list call" — so the destructive/batch cap of 3 does not apply. However, there is no explicit end-to-end sequence (create → wire manifest → implement access → verify); the steps and checkpoints are scattered across reference sections rather than ordered, fitting anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | Section organization is good and the Reading Guide table clearly signals `references/advanced-patterns.md` (verified present, one level deep), but the body also points to `use-cases/collections.md` and `use-cases/lookup-table-enrichment.md` which do not exist in the bundle — broken navigation paths. Combined with substantial content inlined in SKILL.md (~390 lines including a full CRUD class that belongs in a reference), this sits between anchors 3 and 4, and the dead references are more than a minor gap. | 3 / 5 |
Total | 16 / 20 Passed |