Content
67%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 lean, single-purpose tool skill with concrete, executable CLI examples and useful fallbacks. The main defects are triplicated input documentation and a primary example that violates the skill's own width/height constraints.
Suggestions
Fix the example to use valid values, e.g. `--input '{"width": 1920, "height": 1080}'`, since the current width/height of 0 fails the stated 'Must be > 0' constraint.
Collapse the 'Inputs', 'Input' table, and 'Input JSON Schema' sections into a single input specification (the table plus one schema block is enough).
Briefly state expected error handling for the documented failure modes (no Scene View open, null camera) so Claude knows how to respond when the tool errors.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (no concept explanations, direct CLI examples), but the two input parameters are stated three times — the 'Inputs' bullet list, the 'Input' table, and the 'Input JSON Schema' block — which is pure duplication. This matches anchor 3 ('mostly efficient but ... could be tightened') rather than 4, since an entire redundant section pair could be cut; it is not 2 because nothing explains concepts Claude already knows. | 3 / 5 |
Actionability | The 'How to Call' section gives copy-paste-ready commands: the direct `unity-mcp-cli run-tool screenshot-scene-view --input '{...}'` form, the `--input-file` variant, the stdin heredoc, plus Troubleshooting for a missing CLI. It falls short of 5 because the primary example passes `"width": 0, "height": 0`, which the skill's own constraint ('Must be > 0') makes invalid — copy-pasting it as shown would fail. | 4 / 5 |
Workflow Clarity | This is a simple single-invocation tool, and the call path is unambiguous (run command, optionally via file/stdin, with a troubleshooting fallback to npx). It is not 5 because there are minor gaps: the example uses invalid values, and no checkpoint covers what to do on failure beyond the CLI-not-found case (e.g., the documented 'no Scene View open' error). It is well above anchor 3, which requires missing checkpoints for a risky multi-step flow that does not apply here. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the single-file content is appropriately short with clear sections (Inputs, Behavior, How to Call, Troubleshooting, Output) and a real pointer to the /unity-initial-setup skill. It scores 4 rather than 5 because the duplicated Inputs/Input/Input JSON Schema sections are an organization gap that should be collapsed into one. | 4 / 5 |
Total | 15 / 20 Passed |