Content
82%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 highly actionable, well-sequenced debugging procedure with genuinely non-obvious environment knowledge and correct use of its bundle script. Weaknesses are minor: some Claude-known material in the root-cause patterns section, no error-recovery guidance around daemon/PID failures, and a broken pointer to a nonexistent worked-example file.
Suggestions
Fix or remove the dangling reference to `examples/react-reconciler-performance-measure-leak.md` — either add the file to the bundle or drop the 'Worked Example' section, since following the link yields nothing.
Trim the Step 5 'Common patterns' list to the Qwen-specific case (the `performance.measure()` → `measureEntryBuffer` example) and cut the generic leak archetypes Claude already knows.
Add a brief error-recovery note for the two most likely failure points: PID discovery returning an unexpected process (verify with `ps -p $NODE_PID`) and the chrome-devtools daemon not being running when memory tools are called.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean — every command block earns its place, and the Notes sections explain genuinely non-obvious facts (process chain walking, QWEN_CODE_NO_RELAUNCH rationale, shell-state non-persistence). Two mild over-explanations keep it from a 5: the opening paragraph restates the frontmatter description, and the Step 5 'Common patterns' list (event listener leaks, closure capture, module-level caches) is largely knowledge Claude already has. | 4 / 5 |
Actionability | Guidance is fully executable end to end: copy-paste tmux/npm/env commands, the exact `kill -USR2` snapshot trigger, exact `chrome-devtools` subcommands with documented outputs ('uid, className, count, selfSize, maxRetainedSize'), and clear provenance for each placeholder ('Where `<uid>` is from the `get_memory_snapshot_details` output'). The one bundle script is invoked with a concrete, correct path. This matches 'fully executable; copy-paste ready; covers the common cases'. | 5 / 5 |
Workflow Clarity | The six-step sequence is explicit and well-ordered, with real validation checkpoints (three-snapshot trend confirmation in Step 2, post-fix re-run with 'confirm the leaking class count stabilizes' in Step 6). It is not a 5 because there are no error-recovery feedback loops — nothing tells Claude what to do when the daemon fails to start, a snapshot file doesn't appear, or PID discovery returns the wrong process — which the score-5 anchor requires and the score-4 anchor ('most checkpoints present; minor validation gaps') describes exactly. | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned ~155-line workflow that appropriately keeps procedure content inline, and its one bundle reference (`scripts/find-leaf-node.sh`) is real, correctly pathed, and clearly signaled. It drops from 5 because the closing 'Worked Example' points to `examples/react-reconciler-performance-measure-leak.md`, which does not exist in the bundle (no `examples/` directory) — a dangling one-level reference that fails navigation; the score-3 anchor's 'references present but not clearly signaled' is a worse fit since the signal is clear, just broken. | 4 / 5 |
Total | 17 / 20 Passed |