Content
86%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 exemplary procedural skill: copy-paste-ready commands for each recovery scenario, a pre-push verification step, and clean section organization. The only refinements are trimming a few lines of explanation Claude already knows and turning the bad-GPG-code note into an explicit recovery step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean — command blocks with only the caveats that matter ('--force-with-lease is preferred... it refuses if someone else has pushed'). Minor over-explanation a competent agent already knows ('The --signoff (or -s) flag adds this trailer automatically using your user.name and user.email git config', and part of the final Probot section) keeps it just below anchor 5, but it is well above anchor 3's 'some unnecessary explanation'. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands for every case: amend+force-with-lease, a concrete git log command to find unsigned commits with trailer formatting, the rebase edit loop, and a verification command with output interpretation. The common cases (last commit, older commit, cherry-pick) are all covered. | 5 / 5 |
Workflow Clarity | Cases are clearly sequenced with an explicit verification step before push ('Before pushing, verify every commit ahead of origin/main has both DCO and GPG') plus output interpretation ('Each line must start with G or U... N / B / E need investigation'), which satisfies the destructive-operation validation requirement. Short of anchor 5 only because bad GPG codes get a pointer ('See .claude/rules/commit-workflow.md') rather than an explicit fix-and-revalidate feedback loop. | 4 / 5 |
Progressive Disclosure | A compact, self-contained skill (~70 lines) with no bundle files; content is organized into well-labeled sections per case (Case 1/2/3, verification, background), nothing that belongs in a separate file is inlined, and the single external pointer (.claude/rules/commit-workflow.md) is clearly signaled and one level deep. | 5 / 5 |
Total | 18 / 20 Passed |