Content
78%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 excellent executable reference: complete, runnable, parameter-annotated code throughout, terse comments that carry real three.js-specific gotchas (WebGPU quirks, r163/r176 renames), and a Common Mistakes section that captures deprecated patterns in the right place. The main structural weakness is that all ~500 lines live in SKILL.md with zero progressive disclosure into bundle files, plus some duplicated renderer boilerplate that could be trimmed.
Suggestions
Offload the bulk catalog content — the built-in geometry constructor signatures and the WebGPU/TSL details — into a references/ file (e.g., references/built-in-geometries.md, references/webgpu-tsl.md) and keep SKILL.md as a compact overview with one-level-deep, clearly signaled links, mirroring the See Also pattern already used.
Remove the duplicated renderer/scene/camera setup in the WebGPU section (and trim Quick Start's) to just the lines that differ from WebGL setup, pointing to threejs-fundamentals for the shared boilerplate.
Make the InstancedBufferGeometry WebGL path fully executable — inline a minimal ShaderMaterial vertex-shader snippet or link to a concrete example in threejs-shaders — rather than leaving it as a prose comment.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense commented code where comments carry non-obvious facts (e.g., "setIndex([...]) with a plain array picks Uint16 or Uint32 itself", "WebGPU backend draws Points at one pixel"), not filler. It falls short of score 5 because of minor trimmable over-explanation: the renderer/scene/camera boilerplate is fully duplicated between Quick Start and the WebGPU section, content Claude already knows from threejs-fundamentals. It is well above score 3 ('some unnecessary explanation') since nearly every token carries three.js-specific information. | 4 / 5 |
Actionability | Nearly every section is complete, copy-paste-ready code with parameter-annotated constructors (e.g., "new THREE.CylinderGeometry(1, 1, 2, 32); // radiusTop, radiusBottom, height, radialSegments..." and the full InstancedMesh loop ending "instancedMesh.instanceMatrix.needsUpdate = true"). The one gap — the InstancedBufferGeometry WebGL shader path is deferred via a comment "declare `in vec3 offset;` ... (see threejs-shaders)" — is a minor gap that still leaves an actionable pointer, consistent with the score-5 anchor ('copy-paste ready code; specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | This is a reference/catalog skill, and where multi-step processes exist they are shown in order with explicit flags as checkpoints: "positions.needsUpdate = true; // upload the change", "Build the mesh after assigning morphAttributes so it picks up the targets", "mesh.instanceMatrix.needsUpdate = true; // flag after any batch of setMatrixAt calls". It is not score 5 because there are no explicit validate-and-recover loops, but nothing here is a destructive or batch data operation that would require one, and the rubric's cap for missing validation on destructive/batch workflows does not apply. It exceeds score 3 because sequencing that does exist is explicit and gotcha-flagged. | 4 / 5 |
Progressive Disclosure | Headers are well-organized with easy navigation and a See Also section, but the entire ~495-line API catalog is inlined in SKILL.md with no bundle files at all (no references/, scripts/, or assets/ directories exist). The rubric's rationale ('SKILL.md should be an overview pointing to detailed materials') and its score-3 example ('200 lines of API reference that could be in a separate file') describe exactly this: the built-in constructor catalog and the WebGPU/TSL section are bulk reference content that could live one level deep in a references/ file. It is not score 2 because structure is strong, not 'minimal', and it is not score 4 because no content is offloaded — the score-4 anchor expects 'most content appropriately placed' across files. The simple-skill exception (score 5 for well-organized sections alone) applies only to skills under 50 lines. | 3 / 5 |
Total | 16 / 20 Passed |