Content
56%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.
The body delivers a genuinely executable optimization workflow with concrete compile/profile/verify commands and a well-defined iteration budget, but it is significantly overweight. Generic best-practice and pitfall lists, duplicated report formats, and the absence of any bundle files to offload the large templates make it both verbose and monolithic.
Suggestions
Move the report template and worked example report into a references/report-template.md file and keep only a short pointer plus the required format fields inline.
Delete or drastically compress the generic "Best Practices", "Common Pitfalls", and stock profiling/compiler-flag sections, which restate knowledge Claude already has.
State the 2-iteration limit once prominently (e.g., in the constraints section) and remove the ~7 repeated restatements throughout, and add an explicit recovery step for when verification FAILs.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~380-line body includes several padded sections that restate knowledge Claude already has: generic "Best Practices" ("Measure First", "Consider Trade-offs"), generic pitfalls ("premature optimization"), stock profiling commands (perf, valgrind, -O3 -Wall), a full worked example report plus a near-duplicate blank template (~100 lines combined), and the 2-iteration limit repeated roughly eight times. This is noticeably verbose rather than merely having a few spots to tighten. | 2 / 5 |
Actionability | Mostly executable guidance: concrete g++/rustc/javac invocations, sanitizer builds, and output-comparison commands (diff <(./reference) <(./optimized)). Steps 1 and 5, however, rely on pseudocode (read_file("topk_benchmark.cpp"), write_file(...)) that is not runnable, which keeps it below the fully copy-paste-ready 5 anchor. | 4 / 5 |
Workflow Clarity | Five clearly sequenced steps with explicit stopping criteria and per-round correctness verification plus a "revert and stop" rule for regressions. It falls short of the 5 anchor because there is no error-recovery path when verification FAILs (e.g., fix and re-run), only performance-regression handling. | 4 / 5 |
Progressive Disclosure | Sections are well organized with headers, but nothing is split out: a ~100-line report template and a full worked example report are inlined, and there are no bundle references at all. This matches the anchor for content that should be separate sitting inline despite reasonable structure, not the good-structure-with-clear-references anchor above. | 3 / 5 |
Total | 13 / 20 Passed |