Content
63%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 highly actionable and accurate, with executable code, commands, and thorough examples for every commit type. Its main weaknesses are verbosity — duplicating spec content Claude already knows and repeating examples across sections — and a monolithic single-file layout where the spec rules, FAQ, and tool integrations belong in one-level-deep reference files.
Suggestions
Deduplicate: the breaking-change examples (lines ~176–194 and ~300–321), the revert example (~340–343 and ~479–483), and the BREAKING CHANGE rules (stated in three sections) should each appear once.
Move the verbatim 16-rule spec transcription, the FAQ, and the tool integration configs (commitlint/semantic-release/git-cliff) into one-level-deep reference files (e.g., references/spec.md, references/tools.md), keeping SKILL.md as a lean overview.
Replace the spec transcription with a short "how to compose a message from a change" procedure (determine type → check breaking change → write imperative description ≤72 chars → optional body/footers), since Claude already knows the underlying spec.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 500+ line body transcribes the entire Conventional Commits v1.0.0 spec verbatim (rules 1–16), a Benefits section, and an FAQ — material Claude already knows — and duplicates content: the three breaking-change examples appear twice, the revert example appears twice, and BREAKING CHANGE rules are stated in three separate sections. It is not a 1 because the tables, regex, and tool configs are dense reference material rather than fluffy explanation, but several padded and redundant sections are clearly unnecessary. | 2 / 5 |
Actionability | Fully executable, copy-paste-ready guidance throughout: a working header-validation regex, complete Python validate/parse functions, runnable commitlint install-and-test bash commands, semantic-release and git-cliff config snippets, and concrete good/bad example messages covering all common cases. | 5 / 5 |
Workflow Clarity | The single action (compose a conforming commit message) is unambiguous and fully supported — structure, types, rules, best practices, and examples are all specified. It is not a 5 because the body is a reference document rather than a procedure; no explicit sequence guides the reader from a change/diff to a finished message, and the reader must synthesize the workflow from the spec rules. | 4 / 5 |
Progressive Disclosure | Section headers are clear and well-organized, but the entire skill is a single ~515-line monolith with no bundle files; content that clearly belongs in separate reference files — the full 16-rule spec transcription, the FAQ, tool integration configs, and the References/Related Skills sections — is inlined. This matches the anchor "content that should be separate is inline" with some structure present. | 3 / 5 |
Total | 14 / 20 Passed |