Content
77%Weight 40%Scale 1-3Reviews 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 with concrete commands and well-sequenced validation-backed workflows, but it is verbose and monolithic — a giant inline YAML configuration block with duplicated command listings and no progressive disclosure into bundle files. Splitting configuration detail into referenced files and deduplicating the command list would improve token efficiency and navigability.
Suggestions
Move the large CodeRabbit integration configuration (workflow pseudocode, platform notes, execution_guidelines) into a referenced bundle file and keep SKILL.md to a concise overview with a clearly signaled link.
Remove the duplicated command listing — keep commands in one place (the YAML 'commands' block or the 'Quick Commands' section) and reference the other rather than repeating both.
Add a short top-of-body overview/Quick Start that orients the reader before the full YAML definition, improving progressive disclosure for a skill well over 50 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational rather than padded with basic-concept fluff, but it carries notable redundancy — the command list appears both in the YAML block and again in the 'Quick Commands' section — and the CodeRabbit execution_guidelines could be tightened. | 2 / 3 |
Actionability | It provides concrete, executable guidance: specific CLI invocations ('~/.local/bin/coderabbit --prompt-only -t committed --base ${DEFAULT_BRANCH:-main}'), explicit git operations, command syntax with args, and concrete file paths, matching the copy-paste-ready anchor. | 3 / 3 |
Workflow Clarity | Multi-step processes are clearly sequenced with validation checkpoints — the activation STEP 1-5 sequence and the CodeRabbit self-healing loop (parse severity, break/continue, FAIL after max iterations) provide an explicit validate-fix-retry feedback loop. | 3 / 3 |
Progressive Disclosure | Section headings and a one-level-deep dependency listing give some structure, but the core agent definition is a monolithic inline YAML block (~360 lines) with no bundle files to offload detail, so content that should be separate remains inline. | 2 / 3 |
Total | 10 / 12 Passed |