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 exceptionally disciplined operational policy: fully executable commands, approval gates on every write, explicit validation checkpoints around CI and merging, and a well-signaled bundled watcher script whose described behavior matches the actual code. Only minor tightening (repeated head-change caveats, possible offloading of the CI policy detail to a reference file) keeps it from full marks.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious operational policy (credential selection, login-only-on-confirmed-rejection, watcher semantics) and wastes no tokens explaining what GitHub or git is — 'Efficient; minor instances of over-explanation that could be trimmed'. It is not a 5 because the CI section repeats the head-change discipline ('changed heads ... stop', then again 'After any push or head change, discard old success/review evidence') and some clauses ('Its name is an admin label, not a verified upstream username') could be tightened. | 4 / 5 |
Actionability | Every section carries copy-paste-ready commands with explicit placeholders: 'gh repo view OWNER/REPO --json name,description,url,defaultBranchRef', 'glab mr create --repo GROUP/PROJECT --title "..." --description-file mr.md', and the full watcher invocation 'node "<skill dir>/scripts/watch-ci.mjs" --repo OWNER/REPO --pr NUMBER --head REVIEWED_HEAD_SHA' whose flags match the bundled script exactly. It is not a 4 because there are no real gaps — even the '<skill dir>' placeholder is explicitly told how to resolve. | 5 / 5 |
Workflow Clarity | The multi-step flow is clearly sequenced (choose account → login only after confirmed rejection → inspect → checkout/edit → approval-gated writes → CI wait → merge) with explicit validation checkpoints: capture the head SHA before review, re-read checks and threads immediately before merging, discard stale evidence after any head change, and merge only with 'gh pr merge NUMBER --repo OWNER/REPO --squash --match-head-commit REVIEWED_HEAD_SHA'. Destructive operations (push, merge) are approval-gated and error-recovery semantics (which failures stop the watcher, read the status record not the exit code) are spelled out, matching the top anchor. | 5 / 5 |
Progressive Disclosure | The single bundled file (scripts/watch-ci.mjs) is real, referenced one level deep with its exact invocation, and the body appropriately summarizes the script's semantics rather than inlining its logic — 'Good structure; most content is appropriately placed; references mostly clear'. It is not a 5 because the account-selection and CI-wait policy prose is long enough that a references/ file could carry part of it, and the sibling skills ('use-shared-credential', 'interactive-login') are cited by name only without paths. | 4 / 5 |
Total | 18 / 20 Passed |