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.
An exceptionally actionable, well-validated workflow document — every step is executable, the clean-state definition is airtight, and validation/checkpoint discipline is exemplary. The main cost is repetition: the Hard Rules section and several inline rationales restate invariants already spelled out in the loop, which inflates token cost without adding new information.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — nearly all content is repo-specific operational knowledge (bot behaviors, exact commands, query gotchas) rather than concepts Claude already knows. But the same invariants are restated repeatedly: the 'trigger both reviewers' warning appears in the table, step 8, and Hard Rules; the sync check is explained at length in steps 6, 7, and Hard Rules; and several long rationale sentences ('A babysit loop spanning a long session is exactly the scenario where...') could be tightened. This fits the 3 anchor ('mostly efficient but could be tightened') better than 4 ('minor instances'), since the duplication spans multiple sections. | 3 / 5 |
Actionability | Every step carries copy-paste-ready commands: exact `gh pr view`/`gh api graphql` queries with jq filters, the reply and resolve mutations with the correct id fields, the exact trigger wordings ('@greptile', '@cubic-dev-ai review this PR'), and the post-push verify commands. Fully executable with specific gotchas (thread-level author field breaks the query, thread id vs comment databaseId) covering the common failure cases — the 5 anchor. | 5 / 5 |
Workflow Clarity | A clearly sequenced 10-step loop with explicit validation checkpoints: the three-part 'clean' definition checked freshly every round, pagination handled via pageInfo before evaluating clean, post-push commit-sync verification after every push, confirmation that both bots went 'pending' after re-triggering, an explicit wait state (step 9) for pending checks, and defined stop conditions with a two-round escalation. This is a feedback-loop-rich workflow for public/batch actions — squarely the 5 anchor, not 4, because validation and error-recovery loops are explicit at every stage rather than mostly present. | 5 / 5 |
Progressive Disclosure | Well-sectioned single-file skill (no bundle files exist), with one-level-deep, clearly signaled cross-references to `.agents/skills/ship/SKILL.md` steps instead of inlining that material. It stays at 4 rather than 5 because the skill is well over 50 lines and leans on paraphrased descriptions of another skill's steps ('re-run the full sync check from /ship step 2 — not just the log command, the whole check-and-recover flow'), which is a minor organization gap versus either a self-contained reference or tighter pointers. | 4 / 5 |
Total | 17 / 20 Passed |