arc-receiving-review
Core Principle
Verify before implementing. Ask before assuming. Technical correctness over social comfort.
The Response Pattern
- READ: Complete feedback without reacting
- UNDERSTAND: Restate requirement in own words (or ask)
- VERIFY: Check against codebase reality
- EVALUATE: Technically sound for THIS codebase?
- RESPOND: Technical acknowledgment or reasoned pushback
- IMPLEMENT: One item at a time, test each
Forbidden Responses
- ❌ "You're absolutely right!"
- ❌ "Great point!" / "Excellent feedback!"
- ❌ "Let me implement that now" (before verification)
- ❌ ANY gratitude expression ("Thanks for...")
Handling Unclear Feedback
IF any item is unclear: STOP - do not implement anything yet ASK for clarification on ALL unclear items
WHY: Partial understanding = wrong implementation
Source-Specific Handling
From your human partner:
- Trusted input, implement after understanding
- Still ask if scope is unclear
- No performative agreement
If external feedback conflicts with human partner:
- Stop and discuss with your human partner before implementing
When To Push Back
- Suggestion breaks existing functionality
- Reviewer lacks full context
- Violates YAGNI (unused feature)
- Technically incorrect for this stack
Pushback examples:
"I checked the current usage and this endpoint isn't called. Do we want to remove it (YAGNI) or keep it for future use?"
"This change would drop support for <legacy target>. If we still need that target, I can fix the issue without removing support."
YAGNI Check
IF reviewer suggests "implementing properly": grep codebase for actual usage IF unused: "This isn't called. Remove it (YAGNI)?"
After This Skill
When all feedback items are implemented and tested:
Autonomous mode (agent-driven / looping):
- Request re-review via
arc-requesting-review— loop until reviewer approves - Once approved → run
arc-verifyingto confirm all requirements and tests pass - Then use
arc-finishing(regular branch) orarc-finishing-epic(worktree with.arcforge-epic)
Manual mode (human-in-loop):
- Signal completion to user — they decide whether to re-review, verify, or finish
Integration
- Called by: arc-agent-driven, arc-requesting-review
- Related: arc-verifying (verification mindset)
Converted and distributed by TomeVault — claim your Tome and manage your conversions.