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 well-crafted workflow skill: copy-paste-ready commands, a complete subagent prompt template, and explicit validation and failure-handling checkpoints for every risky operation including the submodule edge cases. The only improvements available are trimming the triple-stated workflow summary and moving the subagent prompt template into a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and every non-obvious fact (the .source submodule, diff.ignoreSubmodules=all, lint-staged/biome hooks, gitlink bump rules) earns its tokens with no general-concept padding. It is not 5 because the workflow sequence is restated three times (intro paragraph, checklist, step sections) and could be tightened; not 3 because there is no explanation of concepts Claude already knows. | 4 / 5 |
Actionability | Guidance is fully executable: exact commands such as 'git worktree add -b review/<branch>-<BASE_SHA:0:7> ../review-<BASE_SHA:0:7> <BASE_SHA>', 'git -c diff.ignoreSubmodules=none commit', a complete copy-paste subagent prompt template, a HEREDOC commit message, and the ff-only merge plus teardown commands. It is not 4 because commands and templates cover the common cases end-to-end with only intentional SHA/branch placeholders. | 5 / 5 |
Workflow Clarity | The five steps are clearly sequenced with explicit validation checkpoints and error-recovery loops: capturing BASE_SHA as the sole review target, 'Verify before deleting' with Grep, ReadLints on touched files, 'git log --oneline -3' history confirmation, the --ff-only divergence handling ('do not force anything; report the divergence'), the worktree-path-exists abort fallback, and the clean-result path when the review finds nothing. It is not 4 because both the checkpoints and the failure paths are explicit rather than implicit. | 5 / 5 |
Progressive Disclosure | No bundle files exist and the single ~155-line file is organized into clear sections (Submodule, Workflow, Guardrails, Related skills) with well-signaled links to sibling skills. It is not 5 because the inlined ~25-line subagent prompt template is reference-shaped content that could live in a separate file and the body exceeds the simple under-50-line case; not 3 because nothing is buried and the structure is genuinely good. | 4 / 5 |
Total | 18 / 20 Passed |