Content
27%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill is an action-rich persona definition but is delivered as a monolithic, heavily duplicated inline YAML that inflates token cost. Workflow scaffolding is present but undermined by stale command references and a lack of progressive disclosure.
Suggestions
Move the large YAML agent definition into a separate bundled reference file and keep SKILL.md as a lean overview with one-level-deep, clearly signaled links.
Deduplicate the command list — keep it in one canonical location and reference it elsewhere instead of repeating the table, YAML block, and Quick Commands verbatim.
Update the 'Typical Workflow' to use current command names (*security-audit, *analyze-performance) instead of the deprecated *rls-audit and *explain.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body embeds a ~400-line inline YAML agent-definition block and repeats the command list three times (Star Commands table, commands YAML, Quick Commands), plus persona/emoji/zodiac padding Claude does not need. It is not 2 because the verbosity is pervasive and structural, not a few tighten-able spots. | 1 / 3 |
Actionability | Concrete, copy-paste commands are listed (e.g., '*apply-migration {path}', '~/.local/bin/coderabbit --prompt-only -t uncommitted'), but the actual task workflows live in external dependency files (db-apply-migration.md, security-audit.md) that are only named, not shown. It is not 3 because key execution detail is deferred and not inline, and not 1 because many specific commands and one full workflow are executable. | 2 / 3 |
Workflow Clarity | A migration sequence with validation checkpoints exists (snapshot baseline → dry-run → apply → smoke-test) and the CodeRabbit workflow has explicit gates, but the 'Typical Workflow' references deprecated command names ('*rls-audit', '*explain') absent from the current command table. It is not 3 because that inconsistency undermines clarity, and not 1 because real validation steps are present for destructive operations. | 2 / 3 |
Progressive Disclosure | No references/, scripts/, or assets/ bundle files exist, and the entire agent-definition YAML is inlined as a monolithic block rather than split into navigable, well-signaled references. It is not 2 because content that clearly should be separate is inline with no real file-level navigation. | 1 / 3 |
Total | 6 / 12 Passed |