Content
71%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 dense, expert recipe: exact constants, per-species tables, and a strong failure-mode catalogue make it highly actionable for reconstructing the shader. The main weaknesses are the missing demo.html/PROMPT.md bundle files the text depends on, and the absence of a complete copy-paste shader snippet.
Suggestions
Ship `demo.html` and `PROMPT.md` in the bundle (e.g. under assets/) or remove the "Run `demo.html` first" instruction, since neither file exists next to SKILL.md and the references currently dangle.
Inline the complete `onBeforeCompile` patch (the W struct plus the `map_fragment`, `roughnessmap_fragment`, and `normal_fragment_maps` replacements) as one copy-paste GLSL block so the shader doesn't have to be assembled from prose.
Add inline checkpoints to the recipe (e.g. after step 5, "verify at a grazing angle and 3 m that no layer shimmers") so validation is sequenced into the workflow rather than only available in the separate failures catalogue.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is telegraphic and never explains concepts Claude already knows ("Patch, do not replace", "Bake one field", exact constants throughout), but the "Self-score of this material" section is evaluative meta rather than instruction and could be trimmed, matching the minor-trim anchor at 4 rather than every-token-earns-its-place at 5. | 4 / 5 |
Actionability | Guidance is exact and near-executable ("`lateWidth=0.065+annual*0.21`", "`#include <roughnessmap_fragment>` sets `roughnessFactor=clamp(roughness*W.rough,0.35,1)`", per-species hex colors, chamfer radius formula), but there is no complete copy-paste shader block and the generators live in `demo.html`, which is not in the bundle — concrete with gaps, not fully executable. | 4 / 5 |
Workflow Clarity | The 10-step recipe is clearly sequenced and the "Anchored failures" section provides symptom→cause→fix recovery loops ("A repeated tile is visible at the second board … Sample object-space position, seed per material"), but validation checkpoints sit in a separate catalogue instead of inline verify-after-step-N steps. | 4 / 5 |
Progressive Disclosure | The single file is well-sectioned (Recipe, Species constants, Anchored failures, Budgets), but the body instructs "Run `demo.html` first" and "`PROMPT.md` recreates it" while neither file exists in the bundle (no references/, scripts/, or assets/ directories) — clearly signaled references that don't resolve. | 3 / 5 |
Total | 15 / 20 Passed |