CtrlK
BlogDocsLog inGet started
Tessl Logo

ce-resolve-pr-feedback

Resolve PR review feedback. Use when addressing feedback already left on a PR. Not for reviewing the code before feedback exists; that is ce-code-review.

64

Quality

79%

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

Fix and improve this skill with Tessl

tessl review fix ./skills/ce-resolve-pr-feedback/SKILL.md
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.

An exemplar orchestration skill: a lean, self-disciplining overview that routes by explicit mode table to verified, self-contained references, with validation and termination criteria stated up front. The only real cost is prose density in the pipeline/authority paragraphs, which trades readability for unattended-run safety.

DimensionReasoningScore

Conciseness

The body is dense with operational content Claude does not already know (escalation policy, authority inheritance, mode routing, the `## Unapplied review findings` ticking rule) and explains no generic concepts. It sits at anchor 4 ('efficient; minor instances that could be trimmed') because the `mode:pipeline` and 'Authority in pipeline mode' paragraphs are long, parenthetical-heavy sentences that could be tightened or partly deferred to `references/pipeline-mode.md`; it is above anchor 3 since there is no padded or unnecessary explanation.

4 / 5

Actionability

Guidance is concrete and executable: a mode-detection table keyed on exact argument forms (`pull/123#discussion_r...` vs `#issuecomment-...`), a named positive signal (`gh repo view` succeeding), the exact API path (`repos/OWNER/REPO/pulls/comments/COMMENT_ID`) with the 404 rationale, and GraphQL resolution. It is mostly anchor 4 ('mostly executable guidance; concrete commands with minor gaps') rather than anchor 5 because the actual fetch/reply/resolve commands live in the references and scripts rather than as copy-paste-ready examples in the body — acceptable for an orchestration skill, but a hair below 'fully executable'.

4 / 5

Workflow Clarity

The full-mode 9-step pipeline is enumerated inline (fetch, triage, consolidate & decide, parallel fix, validate, commit/push, reply/resolve, verify, summary), targeted mode is given as 2 explicit steps, and a Success Criteria checklist closes the loop with a concrete termination check ('Empty result from get-pr-comments on verify'). This is a batch operation with explicit validation and verification steps plus error-recovery convergence rules (`invariant_rounds`, `needs-human`), matching anchor 5; it exceeds anchor 4 because validation checkpoints are explicit, not implicit.

5 / 5

Progressive Disclosure

The 59-line body is a lean router: each mode points to a self-contained, one-level-deep reference (all of `references/full-mode.md`, `targeted-mode.md`, `pipeline-mode.md`, `evaluation-rubric.md`, and `agents/pr-comment-resolver.md` exist as real files), each with a stated purpose and even step counts. Scripts sit alongside as bundled tools. This matches anchor 5 ('clear overview with well-signaled one-level-deep references, content appropriately split').

5 / 5

Total

18

/

20

Passed

Description

73%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 tight, well-structured description with an explicit 'Use when' trigger and an unusually clear disambiguation clause against the sibling review skill. The main gap is breadth: the 'what' covers only 'resolve feedback' and trigger synonyms are thin, leaving it just short of the top anchors.

Suggestions

Expand the 'what' with the skill's other concrete actions, e.g. "Evaluate and fix PR review feedback, reply to threads, commit and push fixes, and resolve the threads" — this would lift both specificity and completeness toward the top anchors.

Add natural trigger synonyms such as "review comments", "pull request comments", or "code review feedback" so users phrasing the need as 'address the comments on my PR' also match.

Optionally name the platform (GitHub) in the description so users on other forges can self-exclude before invoking.

DimensionReasoningScore

Specificity

The description names the domain and one concrete action ("Resolve PR review feedback") plus an explicit negative boundary, but does not enumerate the skill's other concrete actions (reply to threads, commit/push fixes, resolve via GraphQL). It matches anchor 3 ('names domain and 1-2 concrete actions, but not comprehensive'); it is above anchor 2 because the action is specific rather than generic, and below anchor 4 because several specific actions are missing.

3 / 5

Completeness

Both parts are explicitly present: what ("Resolve PR review feedback") and when ("Use when addressing feedback already left on a PR"), matching anchor 4's 'has both what and when'. It falls short of anchor 5 because the 'what' is a single terse action without the reply/resolve/commit scope of the skill, whereas anchor-5 examples answer 'what' with multiple concrete actions; it is clearly above anchor 3 since the 'when' is explicit, not implied.

4 / 5

Trigger Term Quality

Natural phrases users would say are present: "PR review feedback", "addressing feedback", "feedback already left on a PR". Common variations like "review comments", "pull request", "inline comments", or "code review" are missing, which matches anchor 4 ('good keyword coverage; a few natural terms missing') rather than anchor 5's comprehensive synonym coverage, and is well above anchor 3's 'missing common variations'.

4 / 5

Distinctiveness Conflict Risk

"Not for reviewing the code before feedback exists; that is ce-code-review" draws an explicit boundary naming the sibling skill, giving a clear niche with distinct triggers and minimal conflict risk (anchor 5). The feedback-exists-vs-not state qualifier makes mis-triggering against a pre-review skill unlikely.

5 / 5

Total

16

/

20

Passed

Validation

81%

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

Validation — 13 / 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

referenced_paths_exist

Referenced path issues: 1 deeper-than-1-level

Warning

Total

13

/

16

Passed

Repository
EveryInc/compound-engineering-plugin
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.