Content
85%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, well-structured integration skill: ordered workflow with explicit validation and a thorough verification checklist, expert-level density with no concept padding, and clean progressive disclosure to three well-signaled reference files. The two deductions are the non-compiling useCoreDm reference in the minimal example and minor verbosity from inline version pins and repeated rules across sections.
Suggestions
Make the Minimal Example compile as written: pass useCoreDm (and its determination) as a prop or define it in the component, e.g. viewerOptions={{ sdk, useCoreDm: props.useCoreDm }}, so the snippet is copy-paste ready.
Consolidate the time-sensitive version pins into a single place (the Dependencies section framed with an old-patterns note for the stale 0.3.x/4.35.3 range) and drop the duplicated pin/disposal statements between the workflow steps, Critical Rules, and checklist.
Trim tangential background such as the legacy --classic / useDune() scaffold parenthetical in Prerequisites, or move it to a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and expert-level with no basic-concept padding, but it is not fully lean: exact version pins ("^1.48535.0", "4.36.0", "10.17.0") are time-sensitive and sit inline in the Dependencies table and prose rather than a deprecated/old-patterns section, the reveal pin and disposal rules are stated three times (workflow step, Critical Rules, checklist), and the legacy "--classic" scaffolds parenthetical is tangential. Efficient overall with minor trims possible — the score-4 anchor, not the every-token-earns-its-place of score 5. | 4 / 5 |
Actionability | Guidance is concrete and near copy-paste ready: a complete typed controller class and RevealWidget mount, an exact install command ("pnpm add @cognite/reveal-widget @cognite/reveal@4.36.0..."), and specific inspect/dedupe/CSP actions. However, the minimal example's viewerOptions={{ sdk, useCoreDm }} references useCoreDm, which is not in scope in the snippet (neither prop nor variable), so the example fails typecheck as written — a minor gap that keeps it off the fully-executable score-5 anchor. | 4 / 5 |
Workflow Clarity | The Integration Workflow gives 10 ordered, concrete steps ending in an explicit validation gate ("Run typecheck and build (tsc --noEmit, pnpm build, etc.) and fix any dependency/peer-version issues"), reinforced by a 16-item Verification Checklist and troubleshooting references for error recovery. This is the clear-sequence-with-explicit-validation-and-checklist pattern of the score-5 anchor; this is an integration task, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | The body is a true overview (workflow, minimal example, critical rules) with bulk detail correctly split into three real, one-level-deep reference files (implementation.md, vite-config.md, csp-and-fixes.md — all verified present, with the html-overlays anchor matching an actual heading and no nested second-level references). Each pointer clearly signals its contents (e.g. "For CSP/manifest.json allowances, the useCoreDm/StrictMode gotchas, the point-cloud app-side fix, and 360-collection troubleshooting, read csp-and-fixes.md"), matching the well-signaled one-level-deep score-5 anchor. | 5 / 5 |
Total | 18 / 20 Passed |