Content
57%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 delivers concrete CLI usage and complete parameter documentation, making it usable as-is. It is held back by duplication (match levels, SerializedMember schema), placeholder-only examples that are not runnable, an implicit rather than explicit find-then-call workflow, and inlined JSON schemas that belong in separate reference files.
Suggestions
Move the full Input/Output JSON schemas into references/ (e.g. references/schema.md) and keep only a compact parameter table plus one realistic example in SKILL.md, removing the duplicated SerializedMember block.
Replace the all-placeholder example with at least one real invocation, e.g. a filter object like {"typeName":"PlayerController","methodName":"TakeDamage"} with actual inputParameters, so the command is copy-paste runnable.
Make the workflow explicit as numbered steps: 1) run 'reflection-method-find' to get the method schema, 2) construct filter/inputParameters from that schema, 3) call this tool, 4) inspect the returned result field — and deduplicate the match-level documentation into one section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The prose sections are lean, but content is duplicated: match levels are explained both in "## Match levels" and again row-by-row in the Input table, and the entire com.IvanMurzak.ReflectorNet.Model.SerializedMember schema block appears verbatim in both the Input and Output JSON schemas (~120 combined lines of schema). This could be tightened to a single authoritative copy. | 3 / 5 |
Actionability | Concrete, executable commands are provided (unity-mcp-cli run-tool with --input, --input-file, and stdin heredoc variants) plus a troubleshooting note for a missing CLI. Not 5 because the example is all placeholders ("string_value" for every field, including "filter" which should be a MethodRef object), so it is not copy-paste runnable for a real call. | 4 / 5 |
Workflow Clarity | The find-then-call sequence is present but only implicit ("Use 'reflection-method-find' to find available method before using it") — it is never laid out as ordered steps. There are no validation checkpoints, such as confirming the method schema was found before calling or interpreting a failed match, though the operation is a single non-destructive call. | 3 / 5 |
Progressive Disclosure | The body has real section structure (Match levels, Inputs, How to Call, Input, Output) aiding navigation, but there are no bundle files at all and ~120 lines of JSON schema are inlined in SKILL.md — reference material that clearly belongs in a separate references/ file per the rubric's rationale. | 3 / 5 |
Total | 13 / 20 Passed |