Content
81%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 excels at actionability and workflow clarity — exact CLI invocations, two concrete crash-trace recipes, and an explicit replay-validation checkpoint before proceeding. Its main weaknesses are two sections restating standard GDB commands Claude already knows, and an automation-script pointer that names the file but not how to run it.
Suggestions
Trim or collapse the 'Inspecting Variables and Memory' and 'Source vs Assembly View' sections into a one-line note (e.g., 'All standard gdb commands work in replay') — the current ~20 lines re-explain commands Claude already knows, which is the lowest-scoring dimension (conciseness).
In the 'Automation Script' section, add the actual invocation for scripts/crash_trace.py (e.g., `gdb -x scripts/crash_trace.py <workspace>/rr-trace` or its expected arguments) so the reference is actionable rather than just a pointer.
Consider moving the per-command reverse-execution aliases (`rn`, `rs`, `rc`, `rsi`, `rni`) and GDB cheat sheet into a short reference file, keeping SKILL.md as a lean overview of the record/pack/replay workflow.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core workflow and crash-extraction recipes are dense and non-obvious, but the "Inspecting Variables and Memory" section (`print <var>`, `info locals`, `info args`) and "Source vs Assembly View" section (`list`, `disassemble`, `layout src`) re-explain standard GDB commands Claude already knows. Two full sections of known material goes beyond the 'minor instances of over-explanation' of anchor 4, fitting anchor 3's 'includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | Commands are copy-paste ready with proper placeholders: `libexec/raptor-rr record --workspace <workspace> --trace rr-trace --cwd repo -- ./program <args>` and the two gdb recipes (`bt`, `up`, `break *$pc`, `reverse-continue`) cover the common regular-crash and ASAN cases exactly. Matches anchor 5's 'fully executable; specific examples cover the common cases'; anchor 4 would require missing key details, which there are none. | 5 / 5 |
Workflow Clarity | The Record → Pack → Replay sequence is numbered with an explicit validation checkpoint: "The helper... performs a complete confined replay before reporting a usable recording. Continue only when its replay consistency check passed; retain verification failures for diagnosis, never as confirmed evidence." That is an explicit validate-before-proceeding loop with failure handling, matching anchor 5; it clearly exceeds anchor 4's 'most checkpoints present'. | 5 / 5 |
Progressive Disclosure | Well-organized sections with a real, one-level-deep bundle reference (`scripts/crash_trace.py` exists in the bundle and is clearly signaled in the Automation Script section). Not anchor 5: the Automation Script reference gives no usage detail (arguments, invocation) so the split is slightly under-specified, and the inline GDB cheat-sheet content could live in a reference file. Fits anchor 4's 'most content appropriately placed; minor organization gaps'. | 4 / 5 |
Total | 17 / 20 Passed |