Skip to content
Back to skills

Receiving Feedback

ASecurity

Use when receiving code review comments, reviewer suggestions, or technical corrections — before implementing any of them

  • 2 stars
  • 0 votes
  • 0 copies
  • 2 views
  • Added September 6, 2026
ai-agentsreactgitapi

Works with

  • api

Security analysis

A100/100

Scanned September 6, 2026

npx -y skills add yuchi-chang/no-cape --skill receiving-feedback --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Receiving Feedback?

Add the live security badge to your README. It updates with every re-scan.

Security grade badge for Receiving Feedback
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/yuchi-chang-receiving-feedback/badge)](https://www.skillsdirectory.com/skills/yuchi-chang-receiving-feedback)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

Download with Pro
SKILL.md
---
name: receiving-feedback
description: Use when receiving code review comments, reviewer suggestions, or technical corrections — before implementing any of them
---

# Receiving Review Feedback

Feedback is input to evaluate, not orders to execute. Verify before implementing; technical correctness over social comfort.

## The pattern

1. Read all the feedback before reacting to any of it.
2. Verify each claim against the actual codebase — reviewers can lack context.
3. If ANY item is unclear, clarify all unclear items before implementing any of them (items may be related; partial understanding produces wrong implementations).
4. Implement in order: blocking issues → simple fixes → complex fixes. Test each one separately.

## No performative agreement

Never "You're absolutely right!" / "Great point!". Restate the requirement, ask the clarifying question, or just fix it — the diff is the acknowledgment. If you pushed back and turn out to be wrong: state the correction factually ("Checked X — you're correct, fixing") and move on. No apology spiral.

## When to push back

- The suggestion breaks existing functionality or violates platform/version constraints
- The reviewer lacks context the current code depends on
- YAGNI: asked to "implement X properly" → grep for actual usage first; if nothing calls it, propose removing it instead
- It conflicts with an architectural decision the user already made → surface the conflict, don't silently comply

Push back with evidence (code, tests, grep output), not defensiveness.

## GitHub

Reply to inline review comments in their thread (`gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies`), not as a top-level PR comment.

Attribution

Is this your skill, or is something wrong with this listing? Request removal or report an issue. Author removals are honored within 72 hours.

Comments

Loading comments…