Content
42%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.
The body is command-rich and well-sectioned but bloated: it inlines a broken duplicate of a script that ships complete in scripts/, embeds CI/hook configuration that belongs in separate files, hardcodes time-sensitive version numbers, and never references the provided CHANGELOG-LATEST.md. Its batch update workflow also lacks any post-update validation, capping workflow clarity at 3.
Suggestions
Remove the inline 'Automatic Update Script' copy and the duplicated sync snippets; point solely to scripts/sync-auto-claude-skills.sh, which already ships complete and correct in the bundle.
Fix or delete the three broken sed commands ('sed -i "s/auto-claude-version: 2.7.2' with no replacement string or closing quote) in the 'Update Specific Skill' and 'Version Mismatch' sections, or replace them with the working sed from scripts/sync-auto-claude-skills.sh:84.
Move the GitHub Actions workflow, pre-commit hook, and version-compatibility matrix into scripts/ or references/ files, link references/CHANGELOG-LATEST.md from the changelog-review step, and add a post-sync validation step (e.g., 'git diff --stat' / verify versions updated) to the Update Workflow since it batch-overwrites skill files.
Replace hardcoded version literals (2.7.2, v2.5.0-v2.7.0 breaking-changes entries) with version-agnostic instructions or a dynamically fetched comparison, so the skill does not decay with each upstream release.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~325-line body is noticeably verbose: it inlines a full ~35-line copy of the sync script that already exists (complete and working) in scripts/sync-auto-claude-skills.sh, plus a ~35-line GitHub Actions workflow and a ~15-line husky pre-commit hook, all of which belong in separate files. It also hardcodes time-sensitive version numbers ('auto-claude-version: 2.7.2', breaking-changes entries for v2.5.0-v2.7.0) outside any 'old patterns'/'deprecated' section. Not 3 because the duplication and bulk go well beyond 'some unnecessary explanation or could be tightened'; not 1 because it rarely explains concepts Claude already knows and is command-focused throughout. | 2 / 5 |
Actionability | Most commands (git fetch/pull, curl to the GitHub releases API, gh release, cp) are concrete and executable, but three sed commands are literally broken syntax — 'sed -i "s/auto-claude-version: 2.7.2' (in the inline sync script, 'Update Specific Skill', and 'Version Mismatch' sections) — with no replacement text and an unclosed quote, making them non-executable. The body also presents this broken inline script instead of the complete, working copy in scripts/sync-auto-claude-skills.sh. Not 4 because broken key commands are more than minor gaps; not 2 because the bulk of the guidance is genuine, runnable commands rather than high-level hints. | 3 / 5 |
Workflow Clarity | The 'Update Workflow' section has a clear four-step sequence (Check Version, Review Changelog, Update Repository, Sync Skills), but the sync/update operation is a batch, semi-destructive operation (cp overwriting reference files, sed -i rewriting SKILL.md files across all skills) with no validation checkpoint afterward — no verification that the sync succeeded, no review of the diff before considering the update done. Per the rubric guideline, a batch/destructive workflow missing validation is capped at 3. Not 4 because the cap applies and validation steps are absent from the workflow; not 2 because the sequence itself is well ordered with concrete commands per step. | 3 / 5 |
Progressive Disclosure | The bundle has scripts/sync-auto-claude-skills.sh and references/CHANGELOG-LATEST.md, but the body references only the script path (then immediately duplicates a divergent, broken copy of it inline) and never links references/CHANGELOG-LATEST.md at all — instead telling the reader to cat or curl the changelog directly. Content that belongs in separate files (the full script, CI workflow, pre-commit hook) is inlined. Not 4 because a provided reference file is orphaned and the script is needlessly duplicated inline; not 2 because the body does have clear section structure and does point to the real script file at least once. | 3 / 5 |
Total | 11 / 20 Passed |