Content
48%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 body is well-organized as a tool reference with genuinely useful operational details (CLI variants, defaults, mutual exclusivity, troubleshooting), but it functions as a monolithic dump: duplicated toggle documentation and ~280 lines of JSON schema inlined where a reference file should carry them, and no executable example showing a real invocation with actual values.
Suggestions
Move the Input/Output JSON schemas into a references/ file (e.g. references/schema.md) and keep only the parameter table and a short example in SKILL.md, removing the triple duplication of toggle descriptions.
Replace the "string_value" placeholder template with one or two real executable examples — e.g. an actual assetRef with assetPath/assetGuid and boolean values for the include* toggles — so the command is copy-paste ready.
Show how to chain the prerequisite step: a short end-to-end snippet running assets-find with 't:Shader' and feeding the result's reference into assets-shader-get-data.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly 280 of ~450 lines are raw JSON schema dumps, and the toggle descriptions are repeated three times (Toggles section, Input table, and schema descriptions). This is noticeably verbose with several padded/duplicated sections; it is not a 3 because the schema duplication and inlining go beyond minor tightening, though the prose sections themselves are lean. | 2 / 5 |
Actionability | There is real concrete guidance — the actual CLI command, --input-file and stdin heredoc variants, path syntax, defaults, and a troubleshooting section with executable npm commands — but the primary example is a non-executable skeleton ("assetRef": "string_value", "includeMessages": "string_value") and the Input table lists every type as "any". This sits between 'mostly executable' and 'minimal concrete guidance', matching the incomplete-guidance anchor rather than 4. | 3 / 5 |
Workflow Clarity | As a single read-only tool call there is no destructive/batch validation cap, and the sequence is clearly signaled (locate the shader via assets-find/assets-shader-list-all, then call with the appropriate toggles), with defaults and mutual-exclusivity rules stated. It is not a 5 because no concrete example shows how to construct the required assetRef from the found shader, leaving the final invocation step implicit. | 4 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist, and ~280 lines of API schema that clearly belongs in a separate reference file are inlined in SKILL.md — matching the anchor 'content that should be separate is inline'. It is not a 2 because the body is well-sectioned (Toggles, Path-scoped reads, Path syntax, How to Call, Input, Output) making it navigable; it is not a 4 because the bulk of the file is reference material rather than an overview. | 3 / 5 |
Total | 12 / 20 Passed |