Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content is highly actionable with concrete commands, regexes, and templates, and is well sectioned, but it suffers from label-mapping redundancy across tables, an only-implied failure-recovery loop, and a monolithic single-file structure with inline reference material that could be externalized.
Suggestions
Consolidate the three type-to-label mappings into a single canonical table and have the Branch Naming and PR Body sections reference it, removing redundant prose restatements.
Make the failure-recovery loop explicit in the Workflow, e.g. step 7: "If checks fail, fix the reported issues, commit, and re-push until all checks pass."
Move the lengthy branch-naming and conventional-commit specifications into reference files (e.g. references/branch-naming.md, references/conventional-commits.md) and keep SKILL.md as a concise overview that links to them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with specs, regexes, and tables rather than concept filler, but the type-to-label mapping recurs across three tables (Branch Naming, PR Body Format, Conventional Commits) and the commit type list is restated in prose after the regex, so it could be tightened. | 2 / 3 |
Actionability | It provides executable regexes for branch and commit naming, a copy-paste-ready Commands section (git checkout, shellcheck, gh pr create/edit with real flags), and concrete PR body templates with exact markdown, satisfying fully executable guidance. | 3 / 3 |
Workflow Clarity | A clear 7-step sequence with validation checkpoints (shellcheck before push, wait for CI) is present, but the error-recovery feedback loop (check fails -> fix -> re-push) is only implied by "Wait for automated checks to pass" rather than explicitly spelled out. | 2 / 3 |
Progressive Disclosure | Sections are well-organized and there are no broken or deeply nested references, but the ~190-line body is monolithic with large inline reference tables (branch/commit specs, label mappings) that could be split into one-level-deep reference files, missing the score-3 "content appropriately split" bar. | 2 / 3 |
Total | 9 / 12 Passed |