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.
A dense, disciplined runbook for an autonomous PR-landing loop: exact gh/GraphQL commands, deterministic situation ordering, explicit test-first validation loops, and a hard invariants checklist. The only soft spots are a few over-packed sentences in the ready-note rules and a single-file layout that could surface some detail in its own sections or reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and operational throughout: it assumes Claude's competence, gives commands without explaining git/GitHub basics, and almost every line is a rule or an exact command. A few dense passages could still be tightened, e.g. the ready-note rules in situation 4 ("then one line for each check that does not hold on the current head: it failed outside this PR's diff, it was not run, or it ran on a narrower command..."), which keeps it just below the every-token-earns-its-place anchor. | 4 / 5 |
Actionability | Fully executable guidance: exact invocations like `gh pr view <n> --json number,state,headRefName,headRefOid,baseRefName,mergeable,labels,body,comments,reviews`, the complete reviewThreads GraphQL query, `gh pr checks <n> --json name,bucket,link`, `addPullRequestReviewThreadReply(input: {pullRequestReviewThreadId, body})`, and `gh api repos/<owner>/<repo>/actions/jobs/<job id>/logs`, with exact input file paths, env vars, and marker strings. The common cases are covered copy-paste ready. | 5 / 5 |
Workflow Clarity | The round is sequenced deterministically (inputs, reading the record, "Take the first that applies" situation ordering, invariants, doing the work) with explicit validation and feedback loops: "Reproduce the finding or the failure as a failing test, see it fail, fix it, and run the tests again", "Read both again before every write", "confirm HEAD is the head you read", and exit-code handling ("exits with 1 when a check failed and 8 when checks are pending; those are results, not failed reads"). The 14-item invariants list acts as a checklist, matching the top anchor. | 5 / 5 |
Progressive Disclosure | A single-file skill with clean section structure (inputs, reading the record, situations, invariants, doing the work, judgment); no bundle files exist and the body references none, so all content sits one level deep. It is not a 5: at roughly 170 lines it exceeds the well-organized-single-file size, and dense operational rules (e.g. the ready-note spec) are buried inside situation 4 rather than split out, which is a minor organization gap. | 4 / 5 |
Total | 18 / 20 Passed |