Content
88%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 tight, highly actionable workflow with strong validation checkpoints and feedback loops appropriate to a destructive/batch binary-rebuild task. Structure is clean and references are one level deep; the only minor gap is a few guardrail sentences that could be trimmed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — concrete commands with no padding or concept explanations — though a few guardrail sentences ('Do not rebase, reset, or overwrite unrelated work', 'Do not substitute a native-only build') could be slightly tightened. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout — 'cargo update -p ffmpeg-next', 'bun install --frozen-lockfile', 'bun run build-all', 'git diff --check', 'objdump -p packages/compositor-win32-x64-msvc/remotion.exe', 'gh pr create --draft --body-file' — covering the common cases concretely. | 5 / 5 |
Workflow Clarity | A clearly sequenced multi-step process (prepare worktree → update Cargo/rebuild → commit/push/PR → finish) with explicit validation checkpoints and stop conditions — 'Stop on any pre-existing change', 'Verify in Cargo.lock that...', 'Stop if unrelated tracked files changed', 'require msvcrt.dll and reject api-ms-win-crt-*', 'If the push is rejected, stop and report it'. | 5 / 5 |
Progressive Disclosure | Well-organized into clear section headers with one-level-deep external references to sibling skills ([../pr/SKILL.md], [../pr-name/SKILL.md]); no bundle files exist so content is appropriately self-contained, with only minor organization gaps. | 4 / 5 |
Total | 18 / 20 Passed |