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 content is highly actionable with strong command coverage, explicit validation for destructive operations, and useful checklists, so destructive-operation caps do not bind. Its main weaknesses are redundancy (repeated commit templates and guardrails) and inlined reference material pointing to non-existent bundle files.
Suggestions
De-duplicate the commit message template: define the HEREDOC form once and reference it from the workflow steps instead of repeating it 3-4 times.
Move the full Conventional Commits spec and CM process map into the referenced resources/*.md files (and create those files), keeping SKILL.md a lean overview.
Consolidate the Guardrails and the closing 'Important notes' section, which restate the same NEVER/ALWAYS rules, to recover tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body largely assumes Git competence (no 'what is a merge' prose) and uses dense tables/commands, but repeats the full HEREDOC commit template 3-4 times and duplicates the guardrails in the closing 'Important notes', plus carries a heavy Scheduling/Scenes/SSL-primitive meta-framework that could be tightened. | 3 / 5 |
Actionability | Many concrete executable commands (git status -sb, git diff --staged, git push --force-with-lease, gitleaks protect --staged, gh pr create) and a full commit-type table, but commit examples stay as placeholders (<type>(<scope>): <description>) rather than a single fully filled-in message, leaving a minor gap below 5. | 4 / 5 |
Workflow Clarity | Clear sequenced workflows (Entry/Scenes, CM workflows 1-6, Conventional Commits Steps 1-5, push/amend procedures) with explicit validation (VERIFY scene, secret scan before staging, git diff --staged before amend, protected-branch check) and checklists, but the dual Quick/Full-path structure and scattered checkpoints leave minor gaps versus the clean 5 anchor. | 4 / 5 |
Progressive Disclosure | Clear section headers and navigation exist, but the full Conventional Commits spec and CM process map are inlined even though resources/*.md files are referenced, and those referenced bundle files do not exist in the skill directory; content that could be separate is inline. | 3 / 5 |
Total | 14 / 20 Passed |