CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-local-branch

Deep code review of the branch you are standing on, before it becomes a PR. Scope is merge-base..working-tree, so it covers commits, staged and unstaged changes together. Runs the same review lanes as review-pr — implementation, verification, approach, security — against the local checkout through the sandbox CLI, and saves a draft to ~/.nx-branch-reviews/<branch>.md. Use when asked to review "this branch", "my changes", or work that is not yet on GitHub.

Quality

86%

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

81%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 well-engineered workflow skill: rigorous sequencing, abundant validation gates, and deliberate reuse of the sibling review-pr skill to avoid duplicating security-critical blocks. The deductions are for justificatory prose that could be trimmed and for the fact that several critical steps are only executable by consulting the sibling skill.

Suggestions

Trim the multi-line rationalizations (the `|| true` bash-vs-zsh verification story, the trust-model framing prose) to one line each; the instruction survives without the argument.

Inline at least a skeleton of the verification block (or ship it as a bundled reference file) so Step 5 is executable without opening the sibling review-pr skill.

Move the charter template and Agent dispatch prompt into a referenced file if the sibling skill already defines them, keeping SKILL.md to the deltas.

DimensionReasoningScore

Conciseness

The body is dense and mostly earns its tokens — every step carries non-obvious, calibrated detail ('Do not "fix" it to `$BASE...HEAD`', 'a dirty tree has no stable identity... can never be deduped'). But there are minor instances of over-explanation that could be trimmed, e.g., the multi-line justification of `|| true` shell behavior and trust-model prose like 'Be honest about that rather than implying a boundary that does not exist'. Fits the 4 anchor; not 5 because not every token earns its place, and not 3 because the padding is minor rather than 'several unnecessary explanations'.

4 / 5

Actionability

Mostly executable guidance: complete bash blocks with variables and error handling, a fill-in charter template, the full Agent dispatch prompt, and the draft frontmatter spec. Not 5 because critical parts are delegated by name to the sibling skill rather than being copy-paste ready — 'Use `review-pr`'s verification block exactly... the single-`verdict` shell gate' and 'the prompt shape from `review-pr` Step 5' require reading another skill to execute. Not 3 because everything actually present is concrete and executable, not pseudocode.

4 / 5

Workflow Clarity

Seven clearly sequenced steps with explicit validation checkpoints and feedback loops: `test -n "$BASE" || FATAL`, the empty-diff `test -s` gate, write-then-verify-then-move, the 'Refuse to run on the default branch' check, Step 5's per-agent evidence verification with a retry-once rule ('re-dispatch once, demanding a line in the far half of the file'), and sandbox cleanup. Matches the 5 anchor — clear sequence, explicit validation, error-recovery loops.

5 / 5

Progressive Disclosure

Good structure: clean step sections, and it deliberately delegates bulk to the sibling skill with clearly signaled references ('Read it, don't reimplement it', 'Same shape as `review-pr` Step 5') instead of duplicating security-critical shell — sound one-level-deep disclosure. Not 5 because no bundle files exist and the references point at another skill's sections rather than bundled files, and the ~215-line body keeps the charter and dispatch templates inline; not 3 because organization is clearly better than 'references present but not clearly signaled'.

4 / 5

Total

17

/

20

Passed

Description

87%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: concrete capabilities, an explicit and natural trigger clause, and a clearly staked-out niche versus the sibling PR-review skill. The only deductions are a second-person phrase and a few missing trigger synonyms.

Suggestions

Rewrite 'the branch you are standing on' in third person (e.g., 'the current checked-out branch') to satisfy the third-person voice rule and recover the specificity point.

Add one or two more natural trigger synonyms such as 'local changes' or 'review my diff' to push trigger-term coverage toward comprehensive.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions with comprehensive coverage — 'Deep code review of the branch you are standing on', 'Scope is merge-base..working-tree, so it covers commits, staged and unstaged changes together', the four named lanes 'implementation, verification, approach, security', and 'saves a draft to ~/.nx-branch-reviews/<branch>.md' — which matches the 5 anchor. Reduced by 1 per the rubric's voice rule because it uses second person ('the branch you are standing on').

4 / 5

Completeness

Explicitly answers both questions: the 'what' is concrete (scope definition, four review lanes, sandbox CLI, draft path) and the 'when' is an explicit trigger clause with concrete phrases ('Use when asked to review "this branch", "my changes", or work that is not yet on GitHub'). Matches the 5 anchor exactly.

5 / 5

Trigger Term Quality

Good natural keyword coverage: 'review "this branch"', '"my changes"', 'work that is not yet on GitHub', 'before it becomes a PR' — phrases a user would naturally say. Not 5 because a few natural synonyms are missing (e.g., 'local changes', 'review my diff', 'pre-push'); not 3 because coverage clearly goes beyond 'some relevant keywords'.

4 / 5

Distinctiveness Conflict Risk

Clear niche with distinct triggers: it explicitly separates itself from the sibling PR-review skill ('before it becomes a PR', 'work that is not yet on GitHub'), so 'review this branch / my changes' routes here rather than to a PR skill. Conflict risk is minimal; not 4 because the boundary with the closest related skill is stated explicitly rather than left to inference.

5 / 5

Total

18

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

14

/

16

Passed

Repository

nrwl/nx

Path

.claude/skills/review-local-branch/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.