Content
85%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.
An unusually actionable, well-sequenced verification runbook with explicit validation gates and evidence checklists — among the strongest content on those dimensions. Its main weaknesses are a broken reference layer (the feature map and execution.md referenced in the body are not in the bundle) and some repeated caveats that could be trimmed.
Suggestions
Ship features/README.md and execution.md in the skill bundle (or correct the links) — the body's first instruction, "Read the feature map first", currently points to a file that does not exist alongside SKILL.md.
Move the per-file evidence listing and the macOS/Windows platform notes into a reference file to slim the main body and make the SKILL.md overview proportionate.
Deduplicate the ownership/cleanup rules repeated across Launch, Cleanup, and Helpers into a single authoritative section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and project-specific throughout ("at-spi2-core is D-Bus activated, not bootstrapped by another accessibility launcher") and assumes Claude's competence, explaining no general concepts. It is not a 5 because ownership/cleanup caveats and negative instructions repeat across sections (e.g. artifact-directory and do-not-kill rules appear in Launch, Cleanup, and Helpers) and several multi-clause sentences could be tightened. | 4 / 5 |
Actionability | Fully executable guidance: an exact apt-get dependency list, copy-paste invocations of verify-linux.sh with flags, the precise gdbus probe, exact binary paths, and the actual MCP tool sequence in Rust (list_windows, snapshot, click via action_target_args). Not 4 because there are no gaps — commands, arguments, and expected outputs are all specified. | 5 / 5 |
Workflow Clarity | The sections run in execution order (Launch, Doctor, Drive, Evidence, Cleanup) with explicit validation gates — the READY criteria requiring "WEB_HARNESS_MARKER_v1 in both AX and the fixture's read-only event journal", check-evidence.sh gating success after cleanup — plus error-recovery feedback ("A timeout or crash may leave incomplete evidence; inspect the launch log and recorded PIDs before retrying") and a delivery-vs-refusal checklist. Not 4 because validation checkpoints are thorough rather than minor-gapped. | 5 / 5 |
Progressive Disclosure | References are clearly signaled and one level deep, but scored against the actual bundle: "Read [the feature map](features/README.md) first" and "See [execution instructions, historical attempts and remaining stand-ins](execution.md)" point to files absent from the bundle (only scripts/verify-linux.sh and scripts/check-evidence.sh exist), so the primary navigation promise is broken and inlined operational detail (per-file evidence listings, platform notes) is not split out. Not 4 because a missing entry-point reference is more than a minor organization gap; not 2 because the body itself is well-sectioned and the script references do resolve. | 3 / 5 |
Total | 17 / 20 Passed |