CtrlK
BlogDocsLog inGet started
Tessl Logo

receiving-code-review

Use when receiving code review feedback, before implementing suggestions, especially if feedback seems unclear or technically questionable - requires technical rigor and verification, not performative agreement or blind implementation

64

Quality

80%

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 ./.agency/plugins/nori/skills/receiving-code-review/SKILL.md
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 process skill with concrete commands, explicit validation gates, and strong dialogue examples for the trickiest step (pushing back on feedback). Its main weakness is redundancy: key rules are repeated across the process, checklist, tone, and red-flag sections, which inflates token cost without adding information.

Suggestions

State the YAGNI check once (Step 3) and drop the near-verbatim duplicate in Response Tone Guidelines.

Consolidate the repeated 'clarify all before implementing any' rule into a single authoritative statement; the Quick Reference Checklist, Response Tone Guidelines, and Red Flags sections each restate it.

Merge or trim the Red Flags section, which overlaps the Common Mistakes table and the Response Tone Guidelines, keeping one consolidated do/don't reference.

DimensionReasoningScore

Conciseness

The body is directive and assumes Claude's competence (no concept over-explanation), but contains notable repetition: the YAGNI check appears nearly verbatim in both Step 3 and Response Tone Guidelines, 'clarify ALL unclear items before implementing ANY' is stated roughly four times across Steps 2, the checklist, and Red Flags, and the Quick Reference Checklist plus Red Flags sections largely restate the process steps. Mostly efficient but could be tightened — anchor 3, not 4, since the redundancy exceeds minor trimming.

3 / 5

Actionability

Provides copy-paste-ready commands throughout: 'gh pr view --json number -q .number', 'gh pr view [PR-NUMBER] --comments', 'grep -r "endpointName" .', 'npm run lint:*-types', 'git diff --stat', 'git push', plus a concrete TodoWrite template and good/bad dialogue examples for the clarify step. Placeholders are appropriately parameterized and cover the common cases — matches anchor 5.

5 / 5

Workflow Clarity

Steps 0-6 are clearly sequenced with explicit validation checkpoints: a STOP-and-clarify gate before any implementation, individual testing per fix, 'If tests fail, fix before proceeding' retry loops, and full verification (tests, type checks, lint, format, diff) before pushing. For this batch operation (multi-item feedback) the validate-fix-retry loops are all present, so the batch-operation cap does not apply — matches anchor 5.

5 / 5

Progressive Disclosure

No bundle files exist, so this is a single-file skill; section headers make navigation easy and the cross-reference to the finishing-a-development-branch skill is clearly signaled. However, at ~190 lines some content (Response Tone Guidelines, Common Mistakes, Red Flags) repeats and could be consolidated or split, and the referenced path '.claude/skills/finishing-a-development-branch/SKILL.md' is not part of any bundle here — anchor 4, not 5.

4 / 5

Total

17

/

20

Passed

Description

70%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 well-targeted description with an explicit and conditional 'Use when' trigger clause and a distinctive receiving-vs-giving framing. Its main weakness is that the 'what' is expressed as behavioral posture rather than concrete actions, and it lacks synonym coverage for common phrasings like 'PR comments' or 'review feedback'.

Suggestions

Enumerate 2-3 concrete actions after the trigger clause, e.g. 'Fetches PR comments, verifies suggestions against the codebase, and implements fixes with testing' to raise specificity.

Add natural trigger synonyms such as 'PR comments', 'review feedback', or 'addressing review comments' so the description matches how users actually phrase the request.

State the core capability affirmatively (what the skill does) before the 'not performative agreement or blind implementation' contrast, so 'what' is concrete rather than only implied by negation.

DimensionReasoningScore

Specificity

Names the domain ('receiving code review feedback', 'implementing suggestions') and 1-2 concrete behaviors ('technical rigor and verification', not 'performative agreement or blind implementation'), but never enumerates the actual actions the skill performs (fetch comments, clarify, implement, test, push). Fits anchor 3: domain plus 1-2 concrete actions, not comprehensive — not anchor 4, which requires several specific actions.

3 / 5

Completeness

The 'when' is explicit ('Use when receiving code review feedback, before implementing suggestions, especially if...') and the 'what' is present but stated as a stance ('requires technical rigor and verification, not performative agreement or blind implementation') rather than concrete capabilities. Both present with room for a more specific 'what' — anchor 4, not 5.

4 / 5

Trigger Term Quality

Contains the natural central phrase 'code review feedback' plus triggering conditions like 'before implementing suggestions', 'feedback seems unclear', 'technically questionable'. Good coverage, but common variations users would say ('PR comments', 'review comments', 'address review feedback') are missing, so it falls between anchor 4 and anchor 5 — closer to 4.

4 / 5

Distinctiveness Conflict Risk

'Receiving code review feedback' carves out a clear niche distinct from skills that perform code reviews, with minimal but real overlap risk: a request like 'handle this review' could route to a reviewing skill. Mostly distinct with minor overlap risk against closely related skills — anchor 4, not 5.

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
microsoft/FluidFramework
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.