CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-and-commit

(Claude Code only) Have the `staged-reviewer` subagent review the staged changes, resolve its findings with the user, then draft a commit message and commit after approval. Intended for the session that wrote the changes, so the review comes from an agent without the author's context; a fresh session should use `generate-commit-message` instead. Accepts optional arguments - a model name for the commit footer and `--auto-fix` to enable the automatic fix loop (e.g., `/review-and-commit --auto-fix Opus 5.5`).

Quality

91%

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

SKILL.md
Quality
Evals
Security

Quality

Content

88%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 tightly written, well-structured command workflow with strong sequencing, explicit user-approval gates, and a well-bounded automatic fix loop including its own stop conditions. The main weaknesses are minor: the staged-reviewer subagent prompt is only implied, and the referenced `scripts/git-unstaged-diff.sh` is not present in the bundle and is not qualified as belonging to another skill.

Suggestions

Specify what to tell the staged-reviewer subagent beyond the repository root (e.g., a minimal prompt template or a note that the subagent definition already contains its instructions), so step 3 is fully executable without inference.

Qualify the `scripts/git-unstaged-diff.sh` reference with the skill or repository it lives in (or ship it in this skill's `scripts/` directory), since no `scripts/` directory exists in the bundle and the bare path leaves Claude to guess its provenance.

Consider having the automatic fix loop reference the step 5 partial-staging procedure ("apply the same snapshot/Method A procedure as step 5") instead of restating it, to remove the one duplicated block of instructions.

DimensionReasoningScore

Conciseness

The body is lean and assumes Claude's competence: no git or subagent concepts are explained, and every line is an instruction, a condition, or a one-sentence rationale for a rule ("You wrote them; a self-review defeats the purpose of this skill"). The partial-staging rules reappearing in the auto-fix section are a compressed restatement that keeps the alternative mode self-contained, not padding, so it fits the 'lean and efficient; every token earns its place' anchor rather than level 4's 'minor instances that could be trimmed'.

5 / 5

Actionability

Guidance is largely executable: named tools and paths (`scripts/git-unstaged-diff.sh <repo-root>`, `claude-subagents/staged-reviewer.md`, `~/.claude/agents/`), concrete commands (`git add <file>`), a precise footer format, and clear stop conditions. It falls short of fully copy-paste-ready because the prompt to give the `staged-reviewer` subagent is only implied by "Give it the repository root", and the referenced `scripts/git-unstaged-diff.sh` does not exist in this skill's bundle with no skill-qualification on the path — matching level 4's 'concrete commands with minor gaps' rather than level 5's 'specific examples cover the common cases'.

4 / 5

Workflow Clarity

The 7-step workflow is clearly sequenced with explicit validation checkpoints throughout: a guard check before anything else, "Wait for the user" decision points, re-collection of staged changes when state has drifted, a fix-report-re-review feedback loop continuing the same reviewer, and an alternative auto-fix mode with explicit stop conditions (verdict OK, needs human decision, or 3 rounds). This matches the top anchor's 'explicit validation steps; feedback loops for error recovery' — the commit is gated on explicit user approval, so the destructive-operation cap does not apply.

5 / 5

Progressive Disclosure

Sections are well organized (Arguments, Guard, Workflow, Automatic fix loop) and external material is delegated one level deep with clearly signaled paths (`git-workflows`' `references/op_commit_message.md` and `references/partial_staging.md`). It stops short of level 5 because `scripts/git-unstaged-diff.sh` is referenced unqualified but absent from the skill's bundle (no `scripts/` directory), leaving its provenance ambiguous — a minor organization gap matching level 4 rather than level 3, since references are clearly signaled and no content that belongs in a separate file is inlined.

4 / 5

Total

18

/

20

Passed

Description

92%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 strong description: it states a concrete multi-step capability, an explicit and unusually specific usage scope (the authoring session vs. a fresh session, with the alternative skill named), and its arguments. Third-person voice is maintained and there is no fluff. The only minor gap is a few natural trigger phrasings a user might use.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions in sequence — "Have the `staged-reviewer` subagent review the staged changes, resolve its findings with the user, then draft a commit message and commit after approval" — plus documented arguments ("a model name for the commit footer and `--auto-fix`"). This matches the anchor for multiple specific concrete actions with comprehensive coverage; the only unstated capability (the guard behavior) is internal detail, not a capability gap, so it does not fit the level-4 'minor gaps' anchor better.

5 / 5

Completeness

The 'what' is explicit (review staged changes via the staged-reviewer subagent, resolve findings, draft and commit) and the 'when' is explicitly stated: "Intended for the session that wrote the changes, so the review comes from an agent without the author's context; a fresh session should use `generate-commit-message` instead." This is equivalent explicit trigger guidance with a concrete condition, matching the level-5 anchor rather than level 4, where the 'when' would be less specific.

5 / 5

Trigger Term Quality

Natural terms users would say are present: "review", "staged changes", "commit message", "commit", "auto-fix". A few natural phrasings are missing (e.g., "review before committing", "pre-commit review", "review my changes"), which matches 'good keyword coverage; a few natural terms missing' rather than the level-5 anchor requiring comprehensive synonyms, and is clearly above level 3's 'missing common variations' since the core natural vocabulary is all there.

4 / 5

Distinctiveness Conflict Risk

The description occupies a clear niche: it names the specific `staged-reviewer` subagent, scopes usage to the session that wrote the changes, and explicitly routes fresh sessions to a different skill (`generate-commit-message`), giving minimal overlap risk. It clearly fits the 'clear niche with distinct triggers' anchor rather than level 4's 'minor overlap risk with closely related skills'.

5 / 5

Total

19

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

referenced_paths_exist

Referenced path issues: 4 missing

Warning

Total

15

/

16

Passed

Repository

ceshine/ceshine-agent-skills

Path

review-and-commit/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.