CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

arn-code-review-pr

This skill should be used when the user says "arn-code-review-pr", "review PR feedback", "analyze PR comments", "validate PR review", "review Bitbucket PR", or wants to validate GitHub or Bitbucket PR review comments, categorize findings, and optionally connect back into the Arness pipeline for fixes. Do NOT use this for creating PRs (use arn-code-ship) or reviewing implementation quality (use arn-code-review-implementation).

Quality

88%

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

85%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 strong procedural skill body: concrete CLI commands for both GitHub and Bitbucket, a well-sequenced six-step workflow with validation checkpoints and an explicit test-fix-revert feedback loop, and proper use of one-level-deep reference files. The main weaknesses are a redundant Error Handling section that duplicates inline guidance and a few spots where commands are implied rather than given.

Suggestions

Trim the Error Handling section to only errors not already covered inline (rate limits, 'bkt' command failures, 'arn-code-architect' agent failure, platform-field fallback), removing the duplicated merged/closed-PR and no-PR-found entries.

Add the concrete label-creation command (e.g., 'gh label create arness-backlog') in Step 6's Create Issues section, and specify the AskUserQuestion options for the priority-level prompt in Step 5.

DimensionReasoningScore

Conciseness

The body is lean procedural guidance with no explanations of concepts Claude already knows ('Filter out bot comments and CI status messages', concrete bash blocks). Not 5 because the 13-line Error Handling section largely restates guidance already given inline in Steps 1-2 (merged/closed PR warnings, 'No PR found for current branch', 'No substantive comments'); not 3 because the duplication is confined to one quick-reference section and everything else earns its place.

4 / 5

Actionability

Nearly every operation has copy-paste commands ('gh api repos/{owner}/{repo}/pulls/{number}/comments --paginate', 'bkt pr comments <id> --json', 'gh pr checkout <number>', 'git checkout main 2>/dev/null || git checkout master 2>/dev/null || git checkout -'). Not 5 due to minor gaps: 'Ensure labels exist (create if missing)' gives no command, and the mechanism for asking the priority level in Step 5 is left unspecified.

4 / 5

Workflow Clarity

Steps 1-6 are clearly sequenced with explicit validation checkpoints (auth checks, MERGED/CLOSED warnings with user confirmation, ask-before-commit/push) and a genuine feedback loop for fixes ('Run targeted tests... If tests fail, attempt to fix (up to 3 attempts). If still failing, revert and inform the user'). Matches the 5 anchor; the 4 anchor's 'minor validation gaps' does not apply.

5 / 5

Progressive Disclosure

The workflow lives in SKILL.md while the report format and deferred-issue template are correctly externalized to real one-level-deep bundle files ('references/pr-report-format.md', 'references/deferred-issue-template.md' — both present), clearly signaled at their point of use in Steps 4 and 5/6. Content is appropriately split with easy navigation; the 5 anchor fits better than the 4 anchor's 'minor organization gaps'.

5 / 5

Total

18

/

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: it states concrete capabilities, gives multiple natural trigger phrases including platform-specific ones, and draws explicit boundaries against sibling skills. The only gaps are a couple of missing trigger synonyms and unmentioned secondary actions like tracking-issue creation.

DimensionReasoningScore

Specificity

Names the domain and several concrete actions ('validate GitHub or Bitbucket PR review comments, categorize findings, and optionally connect back into the Arness pipeline for fixes'). Not 5 because it omits actions the skill actually performs (producing the report, offering local test checkout, creating tracking issues); not 3 because it lists more than 1-2 specific actions.

4 / 5

Completeness

Explicitly answers both what ('validate... PR review comments, categorize findings, and optionally connect back into the Arness pipeline for fixes') and when ('This skill should be used when the user says...') with concrete trigger phrases, matching the 5 anchor. The 4 anchor's weaker, less explicit 'when' does not apply here.

5 / 5

Trigger Term Quality

Includes several natural phrases users would actually say ('review PR feedback', 'analyze PR comments', 'validate PR review', 'review Bitbucket PR'). Not 5 because common variations such as 'address PR comments', 'respond to review feedback', or a plain 'GitHub PR' trigger phrase are missing; clearly above the 3 anchor's 'missing common variations'.

4 / 5

Distinctiveness Conflict Risk

Clear niche with distinct triggers and an explicit negative boundary ('Do NOT use this for creating PRs (use arn-code-ship) or reviewing implementation quality (use arn-code-review-implementation)'), which minimizes wrong-skill triggering. Matches the 5 anchor; the 4 anchor's 'minor overlap risk' is superseded by the explicit disambiguation.

5 / 5

Total

18

/

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

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository

AppsVortex/arness

Path

plugins/arn-code/skills/arn-code-review-pr/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.