Content
88%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 is an exemplary lean orchestration skill: a clearly sequenced recheck→fix→report workflow with explicit validation checkpoints, feedback loops, and a well-defined terminal-state vocabulary, all written without a wasted token. The main weaknesses are the total absence of concrete commands (execution is delegated to AGENTS.md and $audit-asc-pr, which are outside the bundle) and the reliance on that unverifiable external file for critical rules.
Suggestions
Inline one or two concrete example commands for the core recheck (e.g., the gh CLI calls for fetching checks and review threads) so the primary loop is executable without consulting external files.
Either inline the essential branch-update/history rules from AGENTS.md or state clearly in SKILL.md where AGENTS.md lives and that it must be read first, so critical constraints are discoverable even when the external file is unavailable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Quotes: "status reads once; watch repeats until a terminal state; fix-and-watch also applies authorized fixes" and "Re-fetch the PR after pushing and confirm the live head, checks, and thread state" — the body is lean and dense; every sentence carries an instruction, no concept Claude already knows is explained, and there is no padding. It matches the 5 anchor ('every token earns its place'); the 4 anchor's 'minor instances of over-explanation' does not apply. | 5 / 5 |
Actionability | Quotes: "Resolve the exact PR and compare its current head SHA with the last audited or pushed SHA", "Fetch checks, reviews, top-level comments, and GraphQL review threads in parallel", and the checkpoint field list ("objective, PR, head and base SHAs, authority, validation results and inputs...") — the decision guidance is concrete and specific, with exact states and required checkpoint fields. Not a 5 because no literal commands or code (e.g., gh CLI invocations) are given and key execution details are deferred to AGENTS.md and $audit-asc-pr, leaving minor gaps; it is well above the 3 anchor's pseudocode-level vagueness. | 4 / 5 |
Workflow Clarity | Quotes: the numbered recheck sequence ("1. Resolve the exact PR... 2. Fetch checks... 5. If `main` advanced, refresh the merge-base diff"), explicit validation checkpoints ("Re-fetch the PR after pushing and confirm the live head, checks, and thread state", "Reproduce a valid defect and add or update a focused test before changing behavior"), and error-recovery loops ("After an uncertain write, reconcile remote state before a bounded retry"). The feedback loops and terminal-state definitions (observed/changed/pending/clean/blocked) fully match the 5 anchor. | 5 / 5 |
Progressive Disclosure | Quotes: sections "## Recheck current state", "## Address actionable feedback", "## Return one state", "## Automation contract" — the single-file body is well organized with no bundle files to split, and the simple-skill note would allow a 5. It scores 4 instead because critical rules are repeatedly deferred to an external `AGENTS.md` ("follow the branch-update rules in `AGENTS.md`") and a `$audit-asc-pr` skill that are not part of the bundle, so navigation depends on files whose existence and depth cannot be verified here. | 4 / 5 |
Total | 18 / 20 Passed |