Content
77%Reviews 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 performance workflow with concrete code and a measure-before/after validation loop. Its weaknesses are length/repetition across steps and a lack of progressive disclosure — the monolithic body keeps detail inline rather than splitting it into reference files.
Suggestions
Move the larger BAD/FIX reference tables and code patterns (e.g. DMS query rewriting, bundle-size fixes) into separate reference files and link to them from the main steps to reduce inline length.
De-duplicate repeated guidance (staleTime 30_000, 'Add limit: 100', shared-hook extraction) into a single canonical mention referenced from each step.
Either create the referenced 'semantic-knowledge/' directory or remove the conditional 'if available' pointer so navigation signals point to real, clearly-signaled references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~500-line body avoids explaining basic concepts but repeats similar guidance across steps (staleTime 30_000, 'Add limit', BAD/FIX patterns), so it is mostly efficient yet could be tightened; not 3 due to length and redundancy, not 1 since it does not pad with known concepts. | 2 / 3 |
Actionability | Provides concrete grep commands and copy-paste-ready executable code (useMemo/useCallback, useInfiniteQuery, useVirtualizer, AbortController, lodash-es), matching the fully-executable anchor. | 3 / 3 |
Workflow Clarity | Clear 11-step sequence with an explicit validation feedback loop (Step 1 baseline measurement, Step 11 'Re-run...report the delta' and 'Do not fabricate numbers'), so the destructive/batch cap at 2 does not apply. | 3 / 3 |
Progressive Disclosure | No bundle files exist and the 500-line body is monolithic with all detail inline; the single reference to a non-existent 'semantic-knowledge/' directory is conditional and weak, so it has structure but content that should be separate is inline. Not 3 (the under-50-line exception does not apply), not 1 because the step sections are well organized. | 2 / 3 |
Total | 10 / 12 Passed |