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 strong operational skill: every stage is driven by concrete, executable commands with exact API endpoints and templates, and the workflow is clearly sequenced with genuine validation checkpoints and feedback loops. The only real weaknesses are a monolithic inline body with no reference files for detailed material and some repetition of the safety guardrails.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative with no explanations of concepts Claude already knows, but safety rules like 'never merge, approve, or close' are restated across the Hard Guardrails, Stage C, and Stage F sections — minor repetition that could be trimmed, placing it at 'efficient; minor instances of over-explanation' rather than 'every token earns its place'. | 4 / 5 |
Actionability | Fully executable throughout: runnable `node .agents/skills/pr-gardening/scripts/find-candidates.mjs`-style commands with flags, exact API endpoints (`GET /api/companies/:companyId/search/extract`, `POST /api/issues/:issueId/comments`), idempotency-key formats, and a copy-paste-ready marker-comment template covering the common cases. | 5 / 5 |
Workflow Clarity | Stages A through F are clearly sequenced with explicit validation checkpoints and feedback loops: 'Always rerun this stage after any wake', 'Rerun Stage B immediately before this stage', and a Verification section with `node --test` plus a live dry-run sanity checklist (community PRs must appear only under `droppedCommunityPullRequests`). The destructive/batch cap does not apply since validation is present. | 5 / 5 |
Progressive Disclosure | Script logic is appropriately externalized to real bundle files that are all correctly referenced inline (find-candidates.mjs, check-readiness.mjs, render-report.mjs, pr-gardening.test.mjs all exist in ./scripts/), and sections are well organized. However, the body is a ~180-line monolith with no one-level-deep reference docs for detailed material like the deduplication rules and API contracts, matching 'good structure; minor organization gaps' rather than a reference-pointing overview. | 4 / 5 |
Total | 18 / 20 Passed |