Content
70%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 strong, highly actionable body with a clear six-step workflow, per-engine executable templates, and unusually good validation gates (never-overwrite, ask-before-write, refuse-to-guess the framework API). Its weaknesses are systematic repetition of the assertion-resolution rule, inlining all three engines' templates in one file, and a mocks deliverable promised by the description but never templated.
Suggestions
Move each engine's code templates into references/<engine>.md and load only the one matching the detected engine — the other two engines' templates are dead context for any given project.
State the resolve-the-assertion-API rule once in §2 and mark placeholders with a single short comment; delete the repeated "# FRAMEWORK ASSERT — resolve per the rule above" comments and condense the 17-line blockquote to two or three lines.
Add a mock/stub helper template (or drop "mocks" from the description) so the body delivers everything the description promises.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-forward and mostly efficient, but the §2 no-bare-assert() rule is restated at length in a ~17-line blockquote ("sixty lines below §2 forbidding it…") and the comment "# FRAMEWORK ASSERT — resolve per the rule above; do not emit assert()" is repeated four times within one code block — unnecessary explanation that could be tightened to one mention. | 3 / 5 |
Actionability | Guidance is highly executable — exact Glob patterns per engine, config keys with a fallback chain, a full Grep invocation with -A 30, scripted user prompts, and complete Unity/Unreal templates; the Godot FAIL_IF placeholder is non-executable but explicitly justified, which the rubric exempts. Held at 4 rather than 5 because the description promises "mocks" yet no mock/stub helper template exists, and the system-specific example covers only Godot. | 4 / 5 |
Workflow Clarity | Six numbered steps in coherent order with explicit validation gates: engine-configured check ("Engine not configured. Run /setup-engine first."), never-overwrite check with a scripted report message, ask-before-write permission, and a resolve-the-assertion-API-or-generate-nothing feedback loop with error recovery. | 5 / 5 |
Progressive Disclosure | No bundle files exist and sections are clearly headed, but ~250 lines of per-engine code templates are inlined when only one engine's templates are ever relevant per project — the classic content that belongs in one-level-deep references/<engine>.md files loaded after engine detection. | 3 / 5 |
Total | 15 / 20 Passed |