worktrunk-review
Reviews a pull request for idiomatic Rust, project conventions, and code quality. Use when asked to review a PR or when running as an automated PR reviewer.
95
95%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Advisory
Suggest reviewing before use
Quality
Discovery
89%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
This is a well-structured skill description that clearly communicates its purpose and when to use it. The explicit 'Use when...' clause with both manual and automated triggers is excellent. The main weakness is that the capabilities could be more specific about what aspects of Rust code it actually reviews.
Suggestions
Add more specific concrete actions to improve specificity, e.g., 'checks ownership patterns, validates error handling, suggests clippy improvements, reviews unsafe usage'
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Names the domain (Rust PR review) and mentions what it reviews for ('idiomatic Rust, project conventions, and code quality'), but doesn't list specific concrete actions like 'checks for unsafe blocks', 'validates error handling patterns', or 'suggests clippy fixes'. | 2 / 3 |
Completeness | Clearly answers both what ('Reviews a pull request for idiomatic Rust, project conventions, and code quality') and when ('Use when asked to review a PR or when running as an automated PR reviewer') with explicit trigger guidance. | 3 / 3 |
Trigger Term Quality | Includes natural keywords users would say: 'review', 'PR', 'pull request', 'Rust', and 'automated PR reviewer'. These are terms developers naturally use when requesting code reviews. | 3 / 3 |
Distinctiveness Conflict Risk | Clearly scoped to Rust-specific PR reviews with distinct triggers. The combination of 'Rust' + 'PR review' creates a clear niche that wouldn't conflict with general code review skills or other language-specific skills. | 3 / 3 |
Total | 11 / 12 Passed |
Implementation
100%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is an excellent skill that demonstrates best practices across all dimensions. It provides concrete, executable commands for every step, maintains a clear workflow with validation checkpoints, and efficiently communicates complex PR review logic without unnecessary verbosity. The skill appropriately delegates to other skills for general review principles while focusing on project-specific conventions and GitHub API interactions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is lean and efficient, assuming Claude's competence with Rust, git, and GitHub APIs. No unnecessary explanations of basic concepts; every section provides actionable, specific guidance. | 3 / 3 |
Actionability | Provides fully executable bash commands and gh CLI examples throughout. Commands are copy-paste ready with clear variable substitution patterns and specific API endpoints. | 3 / 3 |
Workflow Clarity | Clear 5-step sequence with explicit validation checkpoints (pre-flight checks before review, incremental change detection, thread resolution before submission). Includes decision points for early exit and feedback loops. | 3 / 3 |
Progressive Disclosure | Well-structured with clear sections. Appropriately references external skills (/reviewing-code, /developing-rust) at the start without deep nesting. Content is organized logically from setup through submission. | 3 / 3 |
Total | 12 / 12 Passed |
Validation
81%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 13 / 16 Passed
Validation for skill structure
| Criteria | Description | Result |
|---|---|---|
metadata_version | 'metadata' field is not a dictionary | Warning |
license_field | 'license' field is missing | Warning |
frontmatter_unknown_keys | Unknown frontmatter key(s) found; consider removing or moving to metadata | Warning |
Total | 13 / 16 Passed | |
- Repository
- max-sixty/worktrunk
- Commit
a45fa7f
- Reviewed
Table of Contents
Is this your skill?
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.