Request a design review for a commit and present the results
64
55%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./internal/skills/claude/roborev-design-review/SKILL.mdQuality
Discovery
32%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
The description is too brief and lacks explicit trigger guidance ('Use when...'), making it difficult for Claude to reliably select this skill from a large pool. While 'design review' and 'commit' provide some specificity, the description doesn't explain what a design review entails or enumerate the concrete actions performed. Adding trigger terms and a 'Use when' clause would significantly improve skill selection accuracy.
Suggestions
Add an explicit 'Use when...' clause, e.g., 'Use when the user asks for a design review, visual review, or UI feedback on a commit or set of changes.'
List specific concrete actions, e.g., 'Submits a commit to a design review service, retrieves reviewer feedback, and formats the results as a summary with actionable items.'
Include natural trigger term variations users might say, such as 'UI review', 'visual review', 'review my changes', 'get feedback on commit', or specific tool names if applicable.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Names the domain (design review) and two actions (request a review, present results), but lacks detail on what the review entails, what kind of commit, or what 'present the results' means concretely. | 2 / 3 |
Completeness | Describes what it does at a high level but completely lacks a 'Use when...' clause or any explicit trigger guidance for when Claude should select this skill. Per rubric guidelines, missing 'Use when' caps completeness at 2, and the 'what' is also weak, so this scores a 1. | 1 / 3 |
Trigger Term Quality | Includes 'design review' and 'commit' which are relevant trigger terms, but misses common variations like 'code review', 'PR review', 'review feedback', or tool-specific terms that users might naturally say. | 2 / 3 |
Distinctiveness Conflict Risk | 'Design review for a commit' is somewhat specific and distinguishes it from general code review or document review skills, but could overlap with code review, PR review, or commit analysis skills without clearer scoping. | 2 / 3 |
Total | 7 / 12 Passed |
Implementation
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured skill with clear, actionable steps and good workflow sequencing including validation and error handling. The main weakness is moderate verbosity — the examples section largely restates the instructions, and some sections explain things Claude already knows. The skill effectively guides Claude through a multi-step async process with appropriate checkpoints.
Suggestions
Trim the examples section to only show edge cases or non-obvious behavior rather than restating the main workflow steps.
Remove or condense the 'IMPORTANT' section — Claude doesn't need to be told it must execute bash commands when the instructions already contain bash commands.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Generally efficient but has some unnecessary padding — the 'IMPORTANT' section restates things Claude should know (like executing bash commands), and the 'When NOT to invoke' section could be more concise. The examples section partially duplicates the instructions. | 2 / 3 |
Actionability | Provides concrete, executable commands at each step (git rev-parse, roborev review with specific flags), specifies exact tool usage (Task tool with run_in_background and subagent_type), and gives clear instructions for error handling and output presentation. | 3 / 3 |
Workflow Clarity | Clear 5-step sequence with explicit validation at step 1 (commit ref validation with failure handling), error checking at step 4 with specific recovery suggestions (roborev status, roborev init), and conditional branching based on pass/fail verdict. The workflow has proper feedback loops for error cases. | 3 / 3 |
Progressive Disclosure | The 'See also' section provides good cross-references to related skills, but the examples section is somewhat redundant with the instructions and could be trimmed. The content is reasonably organized but the inline examples largely restate the workflow steps. | 2 / 3 |
Total | 10 / 12 Passed |
Validation
100%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 11 / 11 Passed
Validation for skill structure
No warnings or errors.
2c9749e
Table of Contents
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.