Content
85%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 highly actionable, well-sequenced pipeline skill with excellent validation and feedback loops for batch delivery. Its main weakness is structure: everything lives in one inlined file with project-specific traps and gate details that would be better split into well-signaled reference files.
Suggestions
Move the project-specific trap list (Bazel deps, commitlint line lengths, lint-markdown limits, mise env-var interference) out of Phase 2 step 5 into a references/traps.md file, linked with a one-line pointer.
Split the release and deploy procedures in "After the batch" into a references/release-deploy.md, keeping only a brief trigger condition inline.
Move the omp-specific plan-mode mechanics (`xd://propose`, Alt+Shift+P fallback) into a reference so the core two-phase pipeline stays tool-agnostic and leaner.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence with no concept explanations ("One owner per worktree", the traps list), but project-specific minutiae such as omp commands, lint-docs section-ordering rules, and commitlint line-length details could be trimmed or split out. It sits between the 4 anchor (minor trimming possible) and the 5 anchor (every token earns its place). | 4 / 5 |
Actionability | Concrete, executable commands appear throughout: `git ls-remote --heads origin '*<slug>'`, `GIT_SEQUENCE_EDITOR=true git rebase -i --autosquash origin/main`, `gh pr checks <n> --watch --fail-fast=false`, plus exact commit message formats (`decision: propose <id>`) and PR title constraints ("at most 70 characters"). This matches the fully-executable top anchor. | 5 / 5 |
Workflow Clarity | Two clearly sequenced phases with 12 numbered steps, pinned stop conditions ("A stop condition means stop and ask, not improvise"), and validation checkpoints at every stage: CI-form local gates, smoke of the changed path, validation of every reviewer finding, and CI watch with a feedback loop ("On failure, run `ci-fix`"). Batch operations are well protected, matching the top anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist and no references are linked; all content is inlined in a single ~85-line file. Section headers are clear, but content that plausibly belongs in separate reference files (the project-specific traps list, gate commands, release/deploy details) is inline, matching the 3 anchor ("content that should be separate is inline") rather than the 4 anchor's "references mostly clear". | 3 / 5 |
Total | 17 / 20 Passed |