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 dense, high-signal skill body that assumes competence, gives real commands for every step, and sequences both ordinary and post-sync push workflows with explicit validation and repair loops. The only notable gaps are the absence of a fully concrete example invocation and the lack of any reference splitting despite a ~110-line body.
Suggestions
Include one filled-in example of the change-scope and vitest coverage invocations (real package and base ref from a recent change) so a common case is fully copy-paste ready.
Move the coverage-selection mechanics and the post-sync validation detail into a one-level-deep reference file, keeping SKILL.md as a leaner overview with clearly signaled links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence throughout: it never explains what rebase, Vitest, or CI are, and every sentence carries repo-specific, non-inferable guidance (e.g., "pre-commit fixes staged lint, checks staged whitespace, and guards vendored-source metadata"; "The configured per-file 100% thresholds still apply inside the selected source scope"). This matches the 5 anchor — every token earns its place — and shows none of the 4 anchor's trimmable over-explanation. | 5 / 5 |
Actionability | Nearly all guidance is executable: `git status --short --branch`, `pnpm --silent run change-scope --base <verified-base-ref>`, the full `vitest run ... --coverage.include=...` invocation, `--force-with-lease=<branch>:<observed-oid>`, and `git rev-parse HEAD origin/$(git branch --show-current)` are real, copy-shape-ready commands. It sits at 4 rather than 5 because the vitest and change-scope examples use structural placeholders (`packages/<group>/<package>/...`) with no single fully concrete example invocation covering a common case, and the flexibility is only implicitly justified rather than demonstrated. | 4 / 5 |
Workflow Clarity | Both multi-step workflows are explicitly sequenced with validation checkpoints and feedback loops: the push procedure (run checks once → commit and inspect fixer-touched files → push → "Verify the remote ref matches local HEAD" → "Report pending checks as pending") and the post-sync sequence (re-query heads → inspect each rewritten layer → run selected evidence → "Keep every PR unmerged... until all selected checks pass"). Failure handling includes a repair loop ("repair the failure, validate the repair, and publish the correction"), and the destructive force-push path is guarded by lease verification — matching the 5 anchor including its feedback-loop requirement for risky operations. | 5 / 5 |
Progressive Disclosure | The body is well-structured with clear section headers (Inspect the outgoing change, Select relevant evidence, Full local rehearsal, Protect history-rewriting pushes, Handle failures, Push procedure) and every command inline is core to the flow. It does not reach 5 because the skill exceeds the ~50-line simple-skill case with no bundle files at all: detailed subsections like the coverage-selection mechanics and post-sync validation procedure could be split into one-level-deep references to keep SKILL.md a leaner overview. | 4 / 5 |
Total | 18 / 20 Passed |