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.
An exceptionally well-sequenced workflow contract: every phase has concrete commands, templates, or exact tool-call structures, and validation with feedback loops is built into verify, review, and finish. The only room for improvement is minor token redundancy from rules stated twice and the absence of any reference-file split for the template and worktree details.
Suggestions
State the explicit-request gating once in the body (it currently appears in the intro paragraph and again as a standalone paragraph, in addition to the description) and merge the two merge-confirmation statements in Finish into a single rule.
Consider moving the spec template and amendment rules to a references/spec-template.md so the main body stays a lean phase contract — or, if the one-load design is intentional, say so where the template appears to justify its inline length.
Trim lawyered parentheticals like '(detached HEAD, an unrelated feature branch, unknown default branch)' to the single most common case, letting the 'ask the user' fallback cover the rest.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and prescriptive throughout — git worktree mechanics are given as commands with no explanation of what a worktree is, and the template is shown rather than described. Minor redundancy keeps it below anchor 5: the explicit-request gating appears in both the intro ('Enter this workflow only on an explicit user request') and a standalone paragraph, and the merge-confirmation rule is stated twice in Finish ('Do not auto-finish: confirm with the user before any irreversible action' and 'the merge always needs its own confirmation'). This fits anchor 4 ('efficient; minor instances that could be trimmed'), not anchor 3, since there is no padding or concept teaching. | 4 / 5 |
Actionability | Copy-paste-ready commands and templates throughout: 'Run `git check-ignore -q "$path"`... Then run `git worktree add "$path" -b "$branch" "$base"`', concrete dependency modes ('bun ci', 'uv sync --frozen'), a complete fill-in spec template with frontmatter, and exact Question-tool option structures. Even judgment-based sections give precise structures (the reviewer-provision list and the three required review conclusions), matching anchor 5's 'fully executable... specific examples cover the common cases' for an instruction-only skill. | 5 / 5 |
Workflow Clarity | The full sequence is explicit ('grill → workspace → spec → implement → verify → review → finalize → finish') with decision routing in Step 0 ('Fully constrained mechanical change... → skip Grill and Spec'), explicit validation ('Verification and review are strictly sequential... Do not substitute prior output or a subagent report for fresh evidence'), error-recovery loops ('After two failed fixes, stop patching and re-derive the cause'; 'Fix critical findings, re-verify, and re-review affected areas'), non-convergence stop conditions, and destructive-action gating. This matches anchor 5's 'clear sequence with explicit validation steps; feedback loops for error recovery'. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), so this is a deliberately self-contained single file ('One skill load, no internal skill hand-offs') with clear per-phase section headers and no nested references. At ~200 lines it exceeds the under-50-lines exception that would allow a 5 without references, and material like the spec template, amendment rules, and worktree mechanics could be split into reference files — fitting anchor 4 ('good structure; most content is appropriately placed; minor organization gaps') rather than anchor 5, and clearly above anchor 3 since nothing is buried or poorly signaled. | 4 / 5 |
Total | 18 / 20 Passed |