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.
An unusually dense, expert-level body: every rule pairs a specific constant with the failure it prevents, and the Verify section is a genuine checklist. The main defects are structural: the core instruction to copy functions from demo.html (and the Kibori source it is extracted from) cannot be followed because neither file ships with the skill, leaving the reader to reconstruct the module from the inline numbers.
Suggestions
Ship demo.html in the bundle (e.g., references/ or assets/) or replace the 'Reuse demo.html / Copy updateLean, openInspect...' instruction with self-contained inline code for those functions, since the file is not present.
Either include the Kibori source reference (kibori.html) in the bundle or restate the 'What is and is not in the source' section as a plain ported-vs-new list that does not depend on an unavailable file.
Add a short ordered implementation sequence (scene setup, layers and two-pass render, lean, lift, close, a11y) so a reader rebuilding from scratch has an execution order rather than only concept-ordered rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and dense throughout: every rule carries a constant and the failure it prevents ('Otherwise the camera keeps easing during a 100 ms click and the raycast at pointerup misses a piece that moved 6 px'), and the Numbers table packs exact values ('turn | 0.0085 / 0.0065 rad/px, damp 2.4/s, idle 0.26 rad/s after 2.2 s'). No section explains concepts Claude already knows; every token earns its place. | 5 / 5 |
Actionability | The concrete inline guidance is excellent ('fit = min(0.92 bandH, 0.88 W) / (2 R ppu)', 'light.layers.enable(1)', 'shadowMap.autoUpdate = false', exact timings and easing curves), but the headline instruction — 'Reuse `demo.html`... Copy `updateLean`, `updateHand`, `openInspect`, `closeInspect`, the pointer block and `draw`' — points at a file that is not present in the bundle, so the primary reuse path cannot be executed. The missing source file is a key detail gap beyond the 'minor gaps' of the 4 anchor, while the inline constants keep it above the vague/high-level-hints level of 2. | 3 / 5 |
Workflow Clarity | There is a clear porting sequence (reuse the demo module, copy the named functions, 'Swap the meshes for your own object; nothing else names the joint except FROM, RR and Q_BENCH') and an explicit '## Verify' validation checklist covering click-vs-drag, all close paths, focus behavior, reduced motion, and console cleanliness. Not a 5: the 14 rules are concept-ordered rather than an execution sequence, and there is no fix-and-retry feedback loop; not a 3: validation checkpoints are explicit and specific, not merely implied. | 4 / 5 |
Progressive Disclosure | The single file is well-sectioned with clear headers (Rules, Numbers, Room, Reduced motion, Expensive gotchas, Verify), but the body cites 'demo.html' and 'kibori.html' and neither exists in references/, scripts/, or assets/ — the references are dangling, so a reader cannot navigate to the promised source. This sits between the 'references present but not clearly signaled / should be separate' anchor and the broken-structure anchor. | 3 / 5 |
Total | 15 / 20 Passed |