CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

xp-code-review

Review pending changes for tests, maintainability, and project rules. Use when the user asks for code review, review pending changes, or alignment with maintainability and project rules.

Quality

80%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/xp-code-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

71%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A lean, well-organized instruction skill with concrete deliverables and precise optional-tool commands. Its weaknesses are redundancy in the checklist bullets (standard review questions Claude would ask anyway) and implicit key details such as how to obtain the diff and where project rules are found.

Suggestions

Trim bullets that restate generic review practice (e.g., "Is the code easy to understand at a glance?", "Are names clear...") and keep only the project-specific criteria to tighten conciseness.

Add the concrete starting step for obtaining pending changes (e.g., run `git status` and `git diff HEAD`) and name where project rules are read from (e.g., CLAUDE.md or docs/standards) to close the actionability and workflow gaps.

DimensionReasoningScore

Conciseness

The body is mostly efficient — no concept explanations, no library tours — but several bullets restate standard review judgment Claude already applies ("Is the code easy to understand at a glance?", "Are functions/classes/modules small, cohesive, and well-factored?") and the Risk & Impact section overlaps the Maintainability section. This fits anchor 3 (could be tightened) rather than anchor 4 (only minor trims) or anchor 2 (no padded sections or unnecessary explanations).

3 / 5

Actionability

Guidance is mostly executable for an instruction-only skill: concrete deliverables ("structured review", "specific, actionable improvements", a ready-to-commit verdict) and exact commands ("tuicr -w", "tuicr review list --repo .", "tuicr review comments --session <slug> --repo ."). Minor gaps — no command for obtaining the pending diff and no pointer to where project rules live (e.g., CLAUDE.md) — keep it at anchor 4 rather than 5.

4 / 5

Workflow Clarity

As a simple single-purpose skill the flow is coherent: evaluation criteria grouped into four labeled areas, then explicit deliverables, with the conditional tuicr path clearly subordinated ("the TUI and its saved comments do not replace that work"). It sits at anchor 4 rather than 5 because the sequence from "pending, uncommitted changes" to output (inspect diff → evaluate → report) is implicit, with no explicit starting step. No destructive or batch operations are involved, so the validation cap does not apply.

4 / 5

Progressive Disclosure

The body is under 50 lines, needs no external references, and is organized into clearly labeled sections (Task, Deliverables, Optional interactive review) with no inlined content that belongs in separate files. Per the rubric's simple-skill guideline this earns anchor 5.

5 / 5

Total

16

/

20

Passed

Description

78%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A well-formed description that explicitly answers both what the skill does and when to use it, with natural trigger phrases. Its main limitations are modest action specificity (one verb) and a missing synonym family (diff, uncommitted changes) that would sharpen triggering.

Suggestions

Enumerate a few distinct review actions (e.g., "inspect the uncommitted diff, evaluate test coverage, flag project-rule violations") instead of the single verb "review" to raise specificity.

Add natural synonyms to the trigger clause such as "diff", "uncommitted changes", or "pre-commit review" to broaden trigger coverage.

DimensionReasoningScore

Specificity

The description names the domain and one concrete action — "Review pending changes for tests, maintainability, and project rules" — but relies on a single verb with three focus areas rather than listing several distinct actions, matching anchor 3. It is not anchor 4 because anchor-4 examples enumerate multiple specific actions (e.g., extracts, fills, converts), while this offers one review action.

3 / 5

Completeness

Both parts are explicit: the what ("Review pending changes for tests, maintainability, and project rules") and a concrete when-clause listing multiple trigger phrases. This matches anchor 5's requirement to clearly and explicitly answer both what AND when; anchor 4 applies when the when-clause could be more explicit, which is not the case here.

5 / 5

Trigger Term Quality

"Use when the user asks for code review, review pending changes, or alignment with maintainability and project rules" includes the natural phrases users would say ("code review", "review pending changes"), giving good keyword coverage. It falls short of anchor 5 because common synonyms like "diff", "uncommitted changes", or "pre-commit review" are missing.

4 / 5

Distinctiveness Conflict Risk

The XP framing (tests, maintainability, project rules, pending changes) carves out a mostly distinct niche with minor overlap risk against a general code-review skill. It is not anchor 5 because "code review" is a broad trigger that a generic review skill would also claim; it is clearly more specific than anchor 3's "Works with document files".

4 / 5

Total

16

/

20

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 — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

saski/arnesto

Path

.agents/skills/xp-code-review/SKILL.md

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.