CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

pr-review

Review Nemotron pull requests for code correctness, instructions that match the implementation, first-user setup, and reproducible notebooks and cookbooks.

Quality

82%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Quality

Content

90%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.

This is a strongly written instruction-only skill: the body is exceptionally tight, every section delivers concrete review heuristics rather than general knowledge, and validation/reporting rules are explicit. The only structural limitation is that the workflow lives in thematic prose sections rather than a clearly sequenced process with feedback loops.

DimensionReasoningScore

Conciseness

The body is lean and insight-dense with zero padding or explanation of concepts Claude already knows — e.g., "Saved outputs or a previous interactive session do not establish that Run All works" and "a glob matching zero files or a fully mocked execution path cannot establish reproducibility" are pure task-specific heuristics. Every token earns its place, matching anchor 5.

5 / 5

Actionability

The guidance enumerates exactly what to inspect and how to report: "Verify flags, defaults, config keys, environment variables, working directories, and host/container paths", "Record relevant seeds, data splits, model revisions, software versions, and output provenance", and explicit verdict rules ("Return `request_changes` for defects that should prevent merge, `approve` when there are none, and `comment` for advisory feedback"). For an instruction-only skill this fully covers the common cases, matching anchor 5.

5 / 5

Workflow Clarity

A clear sequence is present (context and delivery mode → three review priorities → evidence validation → reporting) with an explicit validation checkpoint: "Validate candidate findings against the inspected revisions, relevant callers, and tests... reproduce the concrete failure through the approved execution path. Otherwise use source and supplied test evidence and state the limitation." Not a 5 because the workflow is organized as thematic sections rather than a sequenced process with an explicit validate-fix-retry loop, and the runner-invoked vs. standalone review paths are interleaved across prose.

4 / 5

Progressive Disclosure

No bundle files (references/, scripts/, assets/) exist and none are needed; the body is well-organized under clear headers (Review context, Review priorities, Evidence and reporting) with all content appropriately inline for a ~100-line policy skill. Not a 5 because the top anchor requires well-signaled one-level-deep references and the simple-skill 5 exception applies only to skills under 50 lines.

4 / 5

Total

18

/

20

Passed

Description

66%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.

The description clearly states what the skill does with concrete, domain-specific review concerns, and the Nemotron focus keeps it fairly distinct. Its main weakness is the complete absence of any "when to use" trigger guidance, which caps completeness and leaves natural trigger synonyms like "PR" or "diff" uncovered.

Suggestions

Add an explicit trigger clause such as "Use when reviewing Nemotron PRs, diffs, or changesets, or when the user asks for a code review of model training or notebook changes."

Include natural synonyms and abbreviations users would actually say — "PR", "diff", "code review" — alongside "pull requests" to broaden trigger coverage.

Sharpen the action list from abstract categories toward concrete operations (e.g., "check documented commands against entry points and argument parsers, trace setup from README to inference") to close the gap to a fully comprehensive description.

DimensionReasoningScore

Specificity

Lists several concrete review actions — "code correctness, instructions that match the implementation, first-user setup, and reproducible notebooks and cookbooks" — which matches anchor 4's several specific actions with minor gaps. Not a 5 because the actions are somewhat abstract review categories rather than the fully concrete operation list of the top anchor.

4 / 5

Completeness

Has a clear "what" (reviews Nemotron PRs for four named concerns) but no "Use when..." clause or equivalent explicit trigger guidance, which the guidelines state caps completeness at 3. Not a 4 because the "when" is entirely absent rather than merely implicit-but-present.

3 / 5

Trigger Term Quality

Includes natural terms users would say such as "review", "pull requests", "notebooks", and "cookbooks", matching anchor 4's good coverage. Not a 5 because common variations like "PR", "diff", or "code review" are missing; not a 3 since the present keywords go beyond the minimal coverage of the mid anchor.

4 / 5

Distinctiveness Conflict Risk

"Nemotron pull requests" with the notebook/cookbook focus carves a mostly distinct niche, matching anchor 4's minor overlap risk with closely related skills. Not a 5 because it could still trigger for generic code-review or PR-review requests where no Nemotron specificity exists.

4 / 5

Total

15

/

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

NVIDIA-NeMo/Nemotron

Path

skills/pr-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.