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, actionable instruction skill with concrete commands, a clear sync workflow, and explicit drift validation. Minor room to tighten illustrative version examples and to spell out an error-recovery loop after `version:check` fails.
Suggestions
Add an explicit 'If `task version:check` fails: re-run `task version:sync`, then re-check' feedback loop to lift workflow clarity to the top anchor.
Trim or generalize the illustrative version numbers (e.g., `1.0.0-rc.32` → `1.0.0-rc.N`) to reduce time-sensitive-looking detail.
Consider a one-line summary of the sync order at the top of the propagation section so the two-family ordering is scannable before the numbered list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and assumes Claude's competence throughout; the PEP 440 rationale and Do/Don't lists earn their place, with only minor instances (e.g., the illustrative version-number examples) that could be trimmed. | 4 / 5 |
Actionability | Fully executable guidance with concrete commands (`task version:sync`, `task version:bump:*`, `task version:set`, `task version:check`), exact file paths, and named config keys (`VERSION_TARGETS`, `XBERG_DEP_MANIFESTS`, `NPM_MANIFESTS`) covering the common cases. | 5 / 5 |
Workflow Clarity | Clear ordered sequence (numbered sync steps, chaining bump helpers) with an explicit validation checkpoint (`task version:check` dry-runs both and 'fails on drift', run in CI), though no explicit fix-and-retry feedback loop is described. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly labeled sections (Single source of truth, propagation, lockstep integrations, Do, Don't) with no need for external references for a self-contained skill of this scope. | 5 / 5 |
Total | 18 / 20 Passed |