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 lean, highly actionable release-prep runbook: specific paths, naming conventions, a verbatim scan command, and a mid-workflow validation loop over changelog Unreleased sections. Its main gaps are the absence of a final verification step and a couple of steps that stop just short of copy-paste commands.
Suggestions
Add a final validation step at the end of the workflow (e.g., re-run the rg Unreleased scan, confirm every changelog has a `## {new_version}` section, and verify the package builds/tests pass) to close the workflow-clarity gap.
Make 'Run pub get in /client dir' executable by stating the exact command (e.g., `cd client && dart pub get` or `flutter pub get`) as is already done for the rg scan.
Consider surfacing one verbatim changelog-entry example or pulling item-format detail into the existing write-changelog-entry reference more explicitly, so the drafting steps are as copy-paste ready as the rest of the skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative instruction with zero concept explanation — every bullet states an operation ('Set new version in packages/flet/pubspec.yaml', 'Pull the latest main and create a new branch named prepare-release-{new_version}'). Repeated mentions of the two sibling skills are inline reminders at point of use, so they earn their tokens; not verbose enough to drop to 4. | 5 / 5 |
Actionability | Guidance is concrete: exact file paths (packages/flet/pubspec.yaml, website/sidebars.yml, sdk/python/packages/*/CHANGELOG.md), a copy-paste `rg` scan command, a branch naming pattern, and a fixed section ordering. Not a 5 because a few steps stop short of executable commands — 'Run pub get in /client dir' does not specify the exact command (dart vs flutter pub get), and the changelog item-drafting steps delegate without a verbatim example. | 4 / 5 |
Workflow Clarity | The sequence is clear (version → branch → pubspec → pub get → changelogs → website docs → Unreleased conversion → sorting) and includes real checkpoints: the rg scan across all changelogs with an explicit remediation ('convert that section... Do not leave duplicate release content') and the milestone verify-and-fix loop. Not a 5 because there is no final end-to-end validation of the release state (e.g., verify the build passes, changelog format check, or milestone audit completion). | 4 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist and none are needed: the body is a self-contained ~58-line document organized into Inputs / Related Skills / Steps. The only external links are one-level-deep, clearly signaled references to sibling skills (../flet-deprecation/SKILL.md, ../write-changelog-entry/SKILL.md), matching the simple-skill exception in the rubric notes. | 5 / 5 |
Total | 18 / 20 Passed |