Content
92%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 tight, highly actionable instruction-only skill: concrete commands, exact section and label names, an explicit post-edit validation step with a recovery loop, and a closing checklist. The only gap is the missing gh command for editing an existing PR body, which keeps actionability just below full marks.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean ~40 lines with no explanation of concepts Claude already knows; the only elaboration ("pkg means the TiDB module area, not a literal Go package path") is non-obvious domain knowledge that earns its place. Matches the 'every token earns its place' anchor, not the trimmable anchor 4. | 5 / 5 |
Actionability | Provides a copy-paste command ("gh pr create -T .github/pull_request_template.md"), exact label names ("do-not-merge/needs-linked-issue"), and exact mutable targets. Not 5 because the existing-PR edit path says to call gh without giving the specific edit command (e.g., "gh pr edit --body-file"), leaving a minor gap. | 4 / 5 |
Workflow Clarity | A clear 7-step sequence with an explicit validation checkpoint (step 7: re-read the PR and check bot-gated labels) and a feedback loop ("If a label remains unexpectedly, diff the current body against .github/pull_request_template.md"), plus a Quick Checks checklist — matching the anchor with validation steps, error recovery, and checklists. | 5 / 5 |
Progressive Disclosure | The skill is under 50 lines, has no bundle files, and needs none: sections (Overview, Workflow, Quick Checks) are well-organized and the only external pointers (.github/pull_request_template.md, tidb-issue-metadata-guard) are clearly signaled. The simple-skill exception applies. | 5 / 5 |
Total | 19 / 20 Passed |