Content
81%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-engineered workflow skill: it delegates the fragile git/gh automation to a verified bundle script and keeps the body as a clear orchestration guide with explicit gating steps, exit-code-driven branching, and a genuine conflict-resolution feedback loop. Remaining weaknesses are small — a duplicated constraints block, a couple of unresolved placeholders, and slightly awkward reference formatting.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is operational throughout — commands, exit-code branches, and gating instructions with almost no explanation of concepts Claude already knows. Minor trimmable material keeps it below the score-5 'lean and efficient' anchor: the Constraints section restates the frontmatter description nearly verbatim, and the template before/after examples in Phase 4 are somewhat long, though they encode genuinely non-obvious instructions; it is clearly above score 3, where padding would be more than incidental. | 4 / 5 |
Actionability | Guidance is mostly executable: the orchestrator invocation with --pr/--channel/--action flags (verified against scripts/flutter_cp.dart, whose interface, exit codes 0/1/2, and SUCCESS:AUTOMATED:/SUCCESS:MANUAL: markers all match), `git status --porcelain`, `gh issue view <ISSUE_NUMBER> --json title,body`, and a complete `gh pr edit` command with title format and label. Below the score-5 anchor because a few placeholders are never resolved — `<DART_EXECUTABLE>` has no guidance on how to determine it, and `--body "<FINAL_TEMPLATE>"` gives no handling for multi-line bodies in shell. | 4 / 5 |
Workflow Clarity | The four-phase workflow has an explicit sequence, validation checkpoints (clean-repo check via `git status --porcelain`, exit-code-driven branching at each script run), a full error-recovery feedback loop (exit 2 → analyze conflicts → resolve or hand to user → `git add` → `--action continue`), two explicit gating steps ('MUST STOP CALLING TOOLS AND ASK' for the channel choice and for user approval of the drafted template), and a fallback for `gh pr edit` GraphQL failures. This matches the score-5 anchor: clear sequence, explicit validation, and feedback loops for error recovery. | 5 / 5 |
Progressive Disclosure | Structure is good and appropriately split: the 460-line automation logic correctly lives in scripts/flutter_cp.dart (a real bundle file, correctly referenced and accurately described), the template lives in the repo's .github directory (one level deep), and the body stays a lean orchestration guide with clear phase headers. Below the score-5 anchor due to minor organization gaps: the template is referenced via an awkward inline-code-plus-markdown-link hybrid ('[PULL_REQUEST_CP_TEMPLATE](../../../.github/PR_TEMPLATE/PULL_REQUEST_CP_TEMPLATE.md) relative to this skill') and the script path appears only inside bash commands with no explicit listing or description of the bundle contents. | 4 / 5 |
Total | 17 / 20 Passed |