CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

qodo-review-resolver

Read or resolve a pull request's Qodo review using Qodo managed tools through MCP or the CLI — fetch structured status, reviewed commit SHA, and findings for ANY PR as JSON, with optional extended details and citation evidence for audits, then optionally resolve open findings and record outcomes, once or until clean. Use this — never `gh`/`curl` scraping of review comments — for "is the review clean on PR

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

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 highly actionable, well-sequenced operational skill: exact commands, explicit validation gates, and careful error-state handling make it executable end to end, and reference files are correctly scoped and conditionally loaded. Its main weakness is redundancy — the freshness, no-scraping, and authorization rules are each restated multiple times, roughly doubling the body length — plus operational detail inlined in SKILL.md that belongs in reference files.

Suggestions

State the freshness rule (completed review with commit_sha == PR head) once in a single authoritative section and reference it elsewhere instead of restating it in the Description, Instructions callout, Read the session state FIRST, Guardrails, and watch-mode steps.

Move the Runtime compatibility gate and Sandbox auth diagnostic sections into files under references/ (like the existing windows-shells.md pattern), keeping only the trigger conditions and a link in SKILL.md.

Consolidate the repeated authorization rules (dismiss/mark-implemented require explicit authorization; autofix covers only code edits) into the Record the outcome section and point to it from Two modes, Resolve a finding, and Guardrails.

DimensionReasoningScore

Conciseness

The prose is dense and domain-specific — no time is spent on concepts Claude already knows — but key rules are restated nearly verbatim across sections: the freshness rule ("`completed` AND its `commit_sha` equals that head SHA") appears in at least four places (Description, the Instructions callout, Read the session state FIRST, Guardrails, plus watch-mode steps and Configuration), and both the don't-scrape and dismissal-needs-explicit-authorization rules each recur 3+ times. This sits between anchors 2 and 3: the writing is tight rather than padded (arguing 3), but ~430 lines could plausibly be halved by consolidating the repetition (arguing 2); the best fit is 'mostly efficient but could be tightened'.

3 / 5

Actionability

Fully executable guidance throughout: copy-paste-ready commands with exact flags ("qodo read pr-review-session findings --pr-url <PR_URL> --extended --json", "qodo pr-review-session dismiss --finding-ids <id> --reason intentional --explanation \"...\" --json"), field-by-field response schemas, per-error-code handling ("`not_found` (wrong id or wrong workspace) and `conflict` ... are terminal — don't retry them"), and a worked end-to-end example. Matches the 5 anchor: copy-paste ready commands with specific examples covering the common cases.

5 / 5

Workflow Clarity

The multi-step process is clearly sequenced (compatibility gate → preflight → fetch → read session state FIRST → present → triage → scope filter → resolve → record outcome) with explicit validation checkpoints: the freshness gate, repo-identity binding before edits, per-finding result reads after batch writes ("Read `results` per finding, don't assume the call succeeded as a whole"), and a bounded watch loop with terminal error states. The batch/dismissive-write operations all carry mandatory verification, so the destructive/batch cap at 3 does not apply; this matches the 5 anchor's explicit validation steps and error-recovery feedback loops.

5 / 5

Progressive Disclosure

All three referenced files are real, one level deep, and well signaled with conditional load guidance ("read [shell guidance](references/windows-shells.md) before the first call; [launcher recovery](references/windows-launcher.md) only if invocation fails"), and the references themselves contain no nested chains. However, large self-contained operational sections (the Runtime compatibility gate, the Sandbox auth diagnostic, the dismiss/mark-implemented semantics) are inlined in a ~430-line body where the rubric envisions a leaner overview, so structure is good but not fully split — matching the 4 anchor with minor organization gaps rather than the 5 anchor's clean overview.

4 / 5

Total

17

/

20

Passed

Description

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

An exemplary description: it states concrete capabilities across read/resolve/record modes, gives explicit 'Use this' trigger guidance with a rich list of natural user phrases, and carves out a distinct niche while explicitly steering away from the gh/curl scraping anti-pattern. All four dimensions sit at the top anchor.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions with comprehensive coverage — "fetch structured status, reviewed commit SHA, and findings for ANY PR as JSON, with optional extended details and citation evidence for audits, then optionally resolve open findings and record outcomes, once or until clean" — spanning read, resolve, and record across MCP and CLI. It matches the 5 anchor (multiple specific concrete actions, comprehensive) rather than 4, which requires minor gaps in coverage; none are evident.

5 / 5

Completeness

It explicitly answers both questions: the "what" ("Read or resolve a pull request's Qodo review using Qodo managed tools through MCP or the CLI — fetch structured status...") and the "when" ("Use this... for" followed by concrete quoted trigger phrases). This is a direct match to the 5 anchor ("Clearly and explicitly answers both what AND when with concrete trigger phrases"); the 4 anchor requires the 'when' to be less explicit, which it is not.

5 / 5

Trigger Term Quality

It includes a comprehensive set of natural user phrases with synonyms: "is the review clean on PR #N", "get Qodo's findings for <pr> as JSON", "audit the review evidence", "what did Qodo flag", "is this review up to date with head", "check before merging", "resolve my PR review", "fix the review findings", "babysit this PR until it's clean". This matches the 5 anchor's comprehensive natural-term coverage including synonyms; nothing a user would naturally say is missing.

5 / 5

Distinctiveness Conflict Risk

It occupies a clear niche — reading/resolving a pull request's Qodo review through Qodo's own managed tools — and actively distinguishes itself from adjacent behavior ("never `gh`/`curl` scraping of review comments"), with branded trigger terms ("Qodo findings", "babysit this PR"). Minimal conflict risk with other skills, matching the 5 anchor.

5 / 5

Total

20

/

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

qodo-ai/qodo-skills

Path

codex-packages/qodo/skills/qodo-review-resolver/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.