Content
92%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is lean, highly actionable, and sequences a multi-step batch operation with strong validation checkpoints. Its only real defect is a referenced bundle script (run-all-templates.sh) that is absent from the skill directory.
Suggestions
Ship run-all-templates.sh alongside SKILL.md (or remove the reference in step 6), since the body claims it is 'bundled next to this file' but no such file exists in the bundle.
Make the broken-reference risk structural: list bundled scripts in an explicit 'Bundle files' section so missing files are obvious at a glance.
Confirm the PR body placeholder instructs escaping JSON special characters, as the inline curl -d payload currently shows unquoted template placeholders.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and assumes Claude's competence - executable bash is given without explaining what nx or git are, and the explanatory context (e.g. 'how the react template broke') is load-bearing rationale for verification rather than padding. | 3 / 3 |
Actionability | Copy-paste-ready commands for every phase (clone, migrate, verify, commit/PR) with concrete env flags and a real curl PR payload, plus a parameter table of all template repos. | 3 / 3 |
Workflow Clarity | A clearly sequenced 7-step procedure with explicit validation checkpoints and feedback loops for this batch/risky operation, e.g. 'git status --porcelain # MUST be empty' and 'If any target fails... revert that repo (git checkout .) and report - never open a red PR'. | 3 / 3 |
Progressive Disclosure | Sections are well-organized, but the body references 'run-all-templates.sh (bundled next to this file)' which does not exist in the bundle - a broken one-level reference, so navigation is not fully sound. | 2 / 3 |
Total | 11 / 12 Passed |