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.
The body is a well-structured, largely actionable tool reference with a copy-paste CLI invocation, input/output schemas, and built-in validation semantics. Its main weaknesses are redundant duplicate documentation of the two input parameters and a trivial default-valued example that doesn't demonstrate a meaningful call.
Suggestions
Merge the duplicate "## Inputs" list, "## Input" table, and input JSON schema into a single input section to remove the triple documentation of isPlaying/isPaused.
Show a meaningful example invocation, e.g. --input '{"isPlaying": true}' to start playmode, instead of the no-op all-defaults example.
Add one or two explicit error-recovery steps after the Behavior section (fix the compilation error, re-run the call) to close the feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The two parameters are documented twice in prose ("## Inputs" — "`isPlaying` (default `false`) — sets `EditorApplication.isPlaying`" and again in the "## Input" table "If true, the 'playmode' will be started..."), a third time in the input JSON schema, and the opening sentence restates the frontmatter description. Mostly efficient reference material with some clear redundancy — anchor 3, not 2 because the bulk (CLI invocation, schemas) earns its tokens. | 3 / 5 |
Actionability | "unity-mcp-cli run-tool editor-application-set-state --input '{...}'" is a fully executable copy-paste command with `--input-file`, stdin-piping variants, and a concrete troubleshooting path ("npm install -g unity-mcp-cli" / "npx unity-mcp-cli"). Minor gap: the shown example passes both default values ("isPlaying": false, "isPaused": false), a no-op invocation rather than a meaningful common case like starting playmode — anchor 4, not 5. | 4 / 5 |
Workflow Clarity | For this simple single-purpose skill the action is unambiguous, with a precondition state-check referenced ("Use 'editor-application-get-state' tool to get the current state first"), built-in validation ("Refuses any state change while `EditorUtility.scriptCompilationFailed` is true"), and a post-change snapshot returned. It stops short of anchor 5 because error recovery is only hinted ("so the caller can fix them first") with no explicit fix-and-retry steps in the body. | 4 / 5 |
Progressive Disclosure | A single-file skill with no bundle files, organized into clear sections (Inputs, Behavior, How to Call, Troubleshooting, Input, Output) and one clearly signaled cross-reference ("Read the /unity-initial-setup skill for detailed installation instructions"). The ~60-line inline output schema is reasonable for a tool doc with no reference files — anchor 4, not 5 because there are no one-level-deep reference files and the schema block could arguably live in a separate reference. | 4 / 5 |
Total | 15 / 20 Passed |