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.
A dense, highly actionable operational contract with an exemplary verify-then-escalate workflow and concrete code throughout. Its main weakness is conciseness: the invariant is restated several times and some sections (cursor overlay cosmetics, historical reproducers) pad an already long file.
Suggestions
State the snapshot-before-action / verify-after invariant once in 'The core invariant' section and reference it elsewhere instead of re-deriving it in the intro, the canonical loop, and 'Verify after every action'.
Move the agent-cursor-overlay theme/animation detail (glow, action marks, session chips) into a dedicated companion file or trim it to the one operational caveat (AX-run seeding) that actually affects driving.
Ship the referenced companion files (MACOS.md, WINDOWS.md, LINUX.md, BROWSER.md, RECORDING.md) in the bundle so the one-level-deep navigation the body promises actually resolves.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly operational and novel (driver-specific contract Claude does not already know), but at ~1050 lines it repeats the snapshot-before-action invariant in four places and includes a long cursor-overlay cosmetics section and a historical IINA reproducer that could be trimmed. | 3 / 5 |
Actionability | Provides copy-paste-ready bash CLI invocations with exact JSON args, a Python crosshair snippet, and a tool dispatch table with precise signatures covering the common cases — fully executable guidance. | 5 / 5 |
Workflow Clarity | The verify-then-escalate ladder is an explicit numbered algorithm with a validation checkpoint after every action ('verify after every action — mandatory'), feedback loops (verify → fix → re-snapshot → retry), and clear escalation rungs. | 5 / 5 |
Progressive Disclosure | Section headers are clear and companion files (MACOS/WINDOWS/LINUX/BROWSER/RECORDING.md) are signaled one level deep, but no bundle files are actually present in the directory and the core SKILL.md is a long monolith inlining detail that the structure implies lives elsewhere. | 4 / 5 |
Total | 17 / 20 Passed |