Content
86%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, dense reference: fully executable code, terse gotcha-bearing comments, no padding, and deprecated/version-sensitive material correctly quarantined in a 'Common Mistakes' section. The main weakness is structure: everything lives inline in SKILL.md with no references/ bundle, so rarely-needed detail (per-format loaders, the mistakes catalog) loads into context even when only the core glTF path is needed.
Suggestions
Move the 'Common Mistakes' catalog and the per-format sections (OBJ, FBX, STL, PLY, USD) into references/ files (e.g. references/formats.md, references/common-mistakes.md), keeping SKILL.md as a concise overview with one-level-deep, clearly signaled links.
Add an explicit post-load processing sequence (load → traverse/enable shadows → scale to unit box → verify attributes → add to scene) as a single ordered workflow, rather than scattering the steps across per-format sections.
In the Quick Start, call out the LoadingManager/error-handling step explicitly so first-time users get a validation checkpoint before rendering.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and comment-dense with no primer material (no explanation of what three.js or loaders are); every code block carries non-obvious gotchas as terse comments (e.g. 'glTF PBR materials render black without lights or an environment'). Version-sensitive details (r179/r180/r181 renames) are confined to the deprecated-patterns 'Common Mistakes' section the rubric exempts, matching the 5 anchor 'every token earns its place'. | 5 / 5 |
Actionability | Fully executable, copy-paste-ready examples cover the common cases: glTF with Draco/meshopt/KTX2, textures with color-space rules, HDR/EXR/UltraHDR environments, OBJ/FBX/STL/PLY/USD formats, SkeletonUtils cloning, disposal traversal, and WebGPU setup. Not 4: there are no gaps — even edge cases like PLY Float64 conversion come with working fixes. | 5 / 5 |
Workflow Clarity | The Quick Start and async-patterns sections give a clear load→process→render sequence, and the LoadingManager section shows a real feedback loop (timeout → abort → catch → finally; error callbacks on texture loads). It sits at 4 rather than 5 because the body is organized as a topic reference rather than a checkpointed pipeline — processing and verification steps (traverse, scale-to-unit-box, Float64 conversion) are shown per-format rather than as one explicit ordered workflow. | 4 / 5 |
Progressive Disclosure | Sections are well-organized with clear headers and a 'See Also' list of sibling skills, but the ~500-line body has no bundle files at all: the 17-item 'Common Mistakes' catalog and the per-format loader details (OBJ, FBX, STL, PLY, USD) are content that clearly belongs in reference files. This matches the 3 anchor ('content that should be separate is inline') rather than 4, because there are no one-level-deep reference files to signal. | 3 / 5 |
Total | 17 / 20 Passed |