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.
The body is an efficient, actionable six-step workflow with real commands, an explicit label-verification checkpoint, and a well-signaled one-level-deep script reference. Minor room for improvement lies in error-recovery guidance and inlining a concrete title example instead of delegating format entirely to the `pr` skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean six-step workflow with commands, a question-options table, and no explanation of concepts Claude already knows (e.g., no git/gh tutorials); every section earns its tokens. It is not 4 because there are essentially no passages of over-explanation to trim — even the base-detection prose is non-obvious project-specific behavior. | 5 / 5 |
Actionability | Concrete executable commands are given throughout: `git fetch origin`, `bash .agents/skills/open-pr/scripts/detect-base-branch.sh`, and a complete `gh pr create` invocation with flags and heredoc, plus a table of AskQuestion options. It is not 5 because the title format is delegated to the `pr` skill and the body uses placeholders (`<detected-base>`, `<FILLED_TEMPLATE>`) without a filled example; it is not 3 because the commands themselves are real and copy-paste executable, not pseudocode. | 4 / 5 |
Workflow Clarity | A clear numbered 1-6 sequence with most checkpoints present: push-first-if-needed, "Verify the available labels with the PR template", and "Tell the user the result" after base detection. It is not 5 because there are no error-recovery or feedback steps (e.g., what to do if base detection or `gh pr create` fails); it is not 3 because checkpoints are explicit rather than missing, and the operation (draft PR) is neither destructive nor batch, so no cap applies. | 4 / 5 |
Progressive Disclosure | A short, well-sectioned overview that keeps only workflow-level content inline and points one level deep to a real bundle file (`scripts/detect-base-branch.sh`, verified present and matching the described behavior) and to sibling skills (`pr`, `canary`) for details. It is not 4 because navigation is clean and nothing that belongs in a separate file is inlined. | 5 / 5 |
Total | 18 / 20 Passed |