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 clear, well-structured tool reference with executable commands, complete schemas, and explicit error behavior. The main inefficiencies are the near-verbatim restatement of the frontmatter description in the intro and the generic AssetObjectRef boilerplate inlined in the output schema.
Suggestions
Drop or compress the intro paragraph since it duplicates the frontmatter description almost word-for-word; the Behavior section already covers it.
Trim the generic AssetObjectRef/$defs descriptions (e.g. 'It could be Material, ScriptableObject, Prefab...') to the fields actually needed to interpret the result, or move the shared schema to a referenced file.
Replace the placeholder input example ("nothing": "string_value") with a minimal realistic invocation, and add one line on handling the throw when no prefab stage is open.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (command example, compact input table, troubleshooting in two lines), but includes content that could be tightened: the intro paragraph restates the description almost verbatim ("Save a prefab. Use it when you are in prefab editing mode... Use 'assets-prefab-open' tool to open a prefab first"), and ~45 lines of generic AssetObjectRef/$defs boilerplate descriptions ("Reference to UnityEngine.Object asset instance. It could be Material, ScriptableObject, Prefab...") are duplicated padding for a single-tool doc. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the minor-trimming of anchor 4, but is far from the concept-explaining verbosity of anchor 1-2. | 3 / 5 |
Actionability | Provides a concrete, executable CLI invocation ("unity-mcp-cli run-tool assets-prefab-save --input '{...}'"), complete input and output JSON schemas, and actionable troubleshooting ("npm install -g unity-mcp-cli" / "npx unity-mcp-cli"). It falls short of fully copy-paste-ready (anchor 5) because the primary example uses a placeholder payload ("nothing": "string_value") and the stdin variant shows generic {"param": "value"} rather than a realistic invocation — minor gaps consistent with anchor 4. | 4 / 5 |
Workflow Clarity | For a simple single-action skill the sequence is unambiguous: it states the precondition ("Use it when you are in prefab editing mode", "Use 'assets-prefab-open' tool to open a prefab first") and an explicit failure checkpoint ("Throws when no prefab stage is currently open"), with the returned AssetObjectRef serving as the success signal. It is not score 5 because there is no guidance on what to do after the call (e.g. verifying the returned asset path or handling the throw), a minor validation gap matching anchor 4; the destructive/batch cap does not apply since this is a single user-requested save, not a batch operation. | 4 / 5 |
Progressive Disclosure | Well-organized single-file tool doc with clear sections (Behavior, How to Call, Troubleshooting, Input, Output) and no nested or buried references; the one cross-skill pointer ("Read the /unity-initial-setup skill") is clearly signaled. It is not score 5 because the ~50-line output schema block with generic type definitions is inline content that arguably belongs in a shared reference, and the body runs ~100 lines rather than the lean overview the under-50-line exception rewards — minor organization gaps consistent with anchor 4. | 4 / 5 |
Total | 15 / 20 Passed |