Content
100%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.
An exemplary operational skill: a dense, eight-step release workflow with executable commands, per-step verification checkpoints, error-recovery loops, and appropriately split reference files. It teaches only project-specific constraints (branch topology, SignPath approval, asset-signing rules) and assumes Claude knows the underlying tooling.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence throughout — no explanations of git tagging, GPG, or CI concepts — and every sentence is operational (e.g. "Tag the merge commit on the release branch, not the PR branch", "SignPath refuses a run that was re-run more than three times"). Only project-specific facts are included, and each earns its place; nothing here is padded or time-conditional guidance. | 5 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover every stage: `git checkout -b release/X.Y.Z origin/main`, `qmake6 && make -j4 && make check`, `git tag -s vX.Y.Z -m "QtPass vX.Y.Z"`, `gh release edit vX.Y.Z --notes-file <(sed -n '/^## \[X.Y.Z\]/,/^## \[/p' CHANGELOG.md | sed '$d')`, and a dry-run-then-run pattern for signing. The version-bump table names each exact file and field, and both branch variants (main and 1.8) are covered. | 5 / 5 |
Workflow Clarity | Steps 1–8 are clearly sequenced with an explicit "**Verify:**" checkpoint after nearly every stage, plus error-recovery guidance ("If the release date slips, land a follow-up 'Release X.Y.Z: dated YYYY-MM-DD' PR before tagging", "Do not tag on a red run"). Destructive/publishing operations are guarded by `--dryrun` and tag verification (`git tag -v`), so the batch/destructive validation cap does not apply. | 5 / 5 |
Progressive Disclosure | The core release procedure lives in SKILL.md while the two detail-heavy domains (downstream packaging, local builds) are split into real, one-level-deep bundle files, each clearly signaled with a when-to-read condition ("Read [references/downstream-packaging.md](references/downstream-packaging.md) when bumping FreeBSD, OpenBSD, MacPorts…"). Both referenced files exist in the bundle, and lint/doc details are correctly delegated to sibling skills (qtpass-docs, qtpass-linting) rather than inlined. | 5 / 5 |
Total | 20 / 20 Passed |