Content
86%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-structured, highly actionable CLI skill: executable commands throughout, sequenced example workflows with verification steps, and clean one-level-deep offloading to references/common-workflows.md and references/permissions.md. The main gaps are trimmable duplicated command listings and missing error-recovery guidance around destructive operations.
Suggestions
Trim or move the full glab command listings (issues/MRs/ci/repo/release) into a references file, since each section already links the official command docs.
Add a brief error-recovery note to the workflows involving destructive actions (merge/close/delete), e.g. re-check state via the wrapper script or re-authenticate with `glab auth login` on auth failure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean one-line command blocks with almost no concept explanation Claude already knows, matching anchor 4 ("efficient; minor instances of over-explanation that could be trimmed"). Not 5 because the full glab command listings for five subsystems duplicate the per-section official-doc links, and the Aliases/Configuration/"Model Guidance" sections are trimmable; not 3 because there is no genuinely padded or unnecessary explanation block. | 4 / 5 |
Actionability | Every section gives copy-paste-ready, fully executable commands with real arguments — "$SKILL_DIR/scripts/gitlab.py issues list --repo GROUP/REPO", "glab issue create --title "Bug: Login fails" --description "Description" --label bug", "glab api projects/:id/issues -X POST -f title="Issue"" — and the Examples section covers the common cases (MR review, issue+MR creation, pipeline monitoring). Matches anchor 5. | 5 / 5 |
Workflow Clarity | The three example workflows are clearly sequenced with explicit verification checkpoints ("Verify approval was recorded", "Verify MR was created and linked", "Verify pipeline restarted"), fitting anchor 4. Not 5 because there are no error-recovery/feedback loops inside the workflows (Troubleshooting is a separate unlinked section) and destructive actions ("glab mr merge", "glab issue close", "glab release delete") appear only in bare listings without any verification workflow; not 3 because the batch/destructive workflows that are shown do include validation steps, so the cap does not apply. | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned overview, and both bundle references are real files, clearly signaled, and exactly one level deep — "See [permissions.md](references/permissions.md) for read/write classification of each command" and "See [common-workflows.md](references/common-workflows.md) for more examples" — with the detailed 505-line workflow material correctly split out. Matches anchor 5 (clear overview, well-signaled one-level-deep references, easy navigation). | 5 / 5 |
Total | 18 / 20 Passed |