receiving-code-review

Clarify code review feedback and implement items one at a time.

504|66|Updated Jul 16, 2024
One-click install
npx skills add https://github.com/foxminchan/BookWorm --skill receiving-code-review-foxminchan
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: receiving-code-review
Source: https://github.com/foxminchan/BookWorm/tree/main/.github/skills/receiving-code-review
Command: npx skills add https://github.com/foxminchan/BookWorm --skill receiving-code-review-foxminchan

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Code review requires technical evaluation, not emotional performance. Clarifying requirements and verifying changes before implementation reduces risk and rework.

Core Features & Use Cases

  • Restate and clarify unclear items to ensure mutual understanding.
  • Verify proposed changes against the actual codebase and tests.
  • Push back with technical reasoning when suggestions are unsafe or misaligned.
  • Implement items one at a time, validating each change and tracking impact.
  • Escalate ambiguity or scope conflicts to the human partner for alignment.

Quick Start

Read the feedback in full, restate the requirement, verify against the codebase, and implement items one at a time after testing.

Frequently Asked Questions about receiving-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I handle code review feedback without introducing new bugs?

Handling code review feedback safely requires reading feedback in full, restating unclear requirements, and implementing changes item-by-item with verification and testing before final integration.

What is the best way to push back on a code review suggestion?

Pushing back during a code review requires technical rigor, meaning you verify proposed changes against the actual codebase and provide technical reasoning for why unsafe or misaligned suggestions should not be implemented.

How do I manage ambiguous code review comments from external reviewers?

Managing ambiguous code review comments involves restating and clarifying the unclear items to ensure mutual understanding, and escalating scope conflicts or ambiguity to your human partner for alignment before implementation.

Can I implement multiple code review changes at the same time?

Implementing multiple code review changes simultaneously is discouraged; you should implement items one at a time, validating each change through testing, tracking impact, and updating documentation to maintain technical rigor.

When should I escalate code review feedback to a human partner?

You should escalate code review feedback to a human partner when you encounter scope conflicts, unresolved ambiguity, or unclear items that cannot be resolved through codebase verification alone, ensuring proper alignment before proceeding.

What does technical rigor in code review actually involve?

Technical rigor in code review involves structured evaluation of feedback, verifying proposed changes against the codebase and tests, reasoning through pushback, and executing implementations stepwise to reduce risk and rework.