{"id":"acea5724-7c98-458e-a0ee-05697e91fa6c","shortId":"KSEAk4","kind":"skill","title":"receiving-code-review","tagline":"Code review requires technical evaluation, not emotional performance.","description":"# Code Review Reception\n\n## Overview\n\nCode review requires technical evaluation, not emotional performance.\n\n**Core principle:** Verify before implementing. Ask before assuming. Technical correctness over social comfort.\n\n## The Response Pattern\n\n```\nWHEN receiving code review feedback:\n\n1. READ: Complete feedback without reacting\n2. UNDERSTAND: Restate requirement in own words (or ask)\n3. VERIFY: Check against codebase reality\n4. EVALUATE: Technically sound for THIS codebase?\n5. RESPOND: Technical acknowledgment or reasoned pushback\n6. IMPLEMENT: One item at a time, test each\n```\n\n## Forbidden Responses\n\n**NEVER:**\n- \"You're absolutely right!\" (explicit CLAUDE.md violation)\n- \"Great point!\" / \"Excellent feedback!\" (performative)\n- \"Let me implement that now\" (before verification)\n\n**INSTEAD:**\n- Restate the technical requirement\n- Ask clarifying questions\n- Push back with technical reasoning if wrong\n- Just start working (actions > words)\n\n## Handling Unclear Feedback\n\n```\nIF any item is unclear:\n  STOP - do not implement anything yet\n  ASK for clarification on unclear items\n\nWHY: Items may be related. Partial understanding = wrong implementation.\n```\n\n**Example:**\n```\nyour human partner: \"Fix 1-6\"\nYou understand 1,2,3,6. Unclear on 4,5.\n\n❌ WRONG: Implement 1,2,3,6 now, ask about 4,5 later\n✅ RIGHT: \"I understand items 1,2,3,6. Need clarification on 4 and 5 before proceeding.\"\n```\n\n## Source-Specific Handling\n\n### From your human partner\n- **Trusted** - implement after understanding\n- **Still ask** if scope unclear\n- **No performative agreement**\n- **Skip to action** or technical acknowledgment\n\n### From External Reviewers\n```\nBEFORE implementing:\n  1. Check: Technically correct for THIS codebase?\n  2. Check: Breaks existing functionality?\n  3. Check: Reason for current implementation?\n  4. Check: Works on all platforms/versions?\n  5. Check: Does reviewer understand full context?\n\nIF suggestion seems wrong:\n  Push back with technical reasoning\n\nIF can't easily verify:\n  Say so: \"I can't verify this without [X]. Should I [investigate/ask/proceed]?\"\n\nIF conflicts with your human partner's prior decisions:\n  Stop and discuss with your human partner first\n```\n\n**your human partner's rule:** \"External feedback - be skeptical, but check carefully\"\n\n## YAGNI Check for \"Professional\" Features\n\n```\nIF reviewer suggests \"implementing properly\":\n  grep codebase for actual usage\n\n  IF unused: \"This endpoint isn't called. Remove it (YAGNI)?\"\n  IF used: Then implement properly\n```\n\n**your human partner's rule:** \"You and reviewer both report to me. If we don't need this feature, don't add it.\"\n\n## Implementation Order\n\n```\nFOR multi-item feedback:\n  1. Clarify anything unclear FIRST\n  2. Then implement in this order:\n     - Blocking issues (breaks, security)\n     - Simple fixes (typos, imports)\n     - Complex fixes (refactoring, logic)\n  3. Test each fix individually\n  4. Verify no regressions\n```\n\n## When To Push Back\n\nPush back when:\n- Suggestion breaks existing functionality\n- Reviewer lacks full context\n- Violates YAGNI (unused feature)\n- Technically incorrect for this stack\n- Legacy/compatibility reasons exist\n- Conflicts with your human partner's architectural decisions\n\n**How to push back:**\n- Use technical reasoning, not defensiveness\n- Ask specific questions\n- Reference working tests/code\n- Involve your human partner if architectural\n\n**Signal if uncomfortable pushing back out loud:** \"Strange things are afoot at the Circle K\"\n\n## Acknowledging Correct Feedback\n\nWhen feedback IS correct:\n```\n✅ \"Fixed. [Brief description of what changed]\"\n✅ \"Good catch - [specific issue]. Fixed in [location].\"\n✅ [Just fix it and show in the code]\n\n❌ \"You're absolutely right!\"\n❌ \"Great point!\"\n❌ \"Thanks for catching that!\"\n❌ \"Thanks for [anything]\"\n❌ ANY gratitude expression\n```\n\n**Why no thanks:** Actions speak. Just fix it. The code itself shows you heard the feedback.\n\n**If you catch yourself about to write \"Thanks\":** DELETE IT. State the fix instead.\n\n## Gracefully Correcting Your Pushback\n\nIf you pushed back and were wrong:\n```\n✅ \"You were right - I checked [X] and it does [Y]. Implementing now.\"\n✅ \"Verified this and you're correct. My initial understanding was wrong because [reason]. Fixing.\"\n\n❌ Long apology\n❌ Defending why you pushed back\n❌ Over-explaining\n```\n\nState the correction factually and move on.\n\n## Common Mistakes\n\n| Mistake | Fix |\n|---------|-----|\n| Performative agreement | State requirement or just act |\n| Blind implementation | Verify against codebase first |\n| Batch without testing | One at a time, test each |\n| Assuming reviewer is right | Check if breaks things |\n| Avoiding pushback | Technical correctness > comfort |\n| Partial implementation | Clarify all items first |\n| Can't verify, proceed anyway | State limitation, ask for direction |\n\n## Real Examples\n\n**Performative Agreement (Bad):**\n```\nReviewer: \"Remove legacy code\"\n❌ \"You're absolutely right! Let me remove that...\"\n```\n\n**Technical Verification (Good):**\n```\nReviewer: \"Remove legacy code\"\n✅ \"Checking... build target is 10.15+, this API needs 13+. Need legacy for backward compat. Current impl has wrong bundle ID - fix it or drop pre-13 support?\"\n```\n\n**YAGNI (Good):**\n```\nReviewer: \"Implement proper metrics tracking with database, date filters, CSV export\"\n✅ \"Grepped codebase - nothing calls this endpoint. Remove it (YAGNI)? Or is there usage I'm missing?\"\n```\n\n**Unclear Item (Good):**\n```\nyour human partner: \"Fix items 1-6\"\nYou understand 1,2,3,6. Unclear on 4,5.\n✅ \"Understand 1,2,3,6. Need clarification on 4 and 5 before implementing.\"\n```\n\n## GitHub Thread Replies\n\nWhen replying to inline review comments on GitHub, reply in the comment thread (`gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies`), not as a top-level PR comment.\n\n## The Bottom Line\n\n**External feedback = suggestions to evaluate, not orders to follow.**\n\nVerify. Question. Then implement.\n\nNo performative agreement. Technical rigor always.\n\n## When to Use\nThis skill is applicable to execute the workflow or actions described in the overview.\n\n## Limitations\n- Use this skill only when the task clearly matches the scope described above.\n- Do not treat the output as a substitute for environment-specific validation, testing, or expert review.\n- Stop and ask for clarification if required inputs, permissions, safety boundaries, or success criteria are missing.","tags":["receiving","code","review","antigravity","awesome","skills","sickn33","agent-skills","agentic-skills","ai-agent-skills","ai-agents","ai-coding"],"capabilities":["skill","source-sickn33","skill-receiving-code-review","topic-agent-skills","topic-agentic-skills","topic-ai-agent-skills","topic-ai-agents","topic-ai-coding","topic-ai-workflows","topic-antigravity","topic-antigravity-skills","topic-claude-code","topic-claude-code-skills","topic-codex-cli","topic-codex-skills"],"categories":["antigravity-awesome-skills"],"synonyms":[],"warnings":[],"endpointUrl":"https://skills.sh/sickn33/antigravity-awesome-skills/receiving-code-review","protocol":"skill","transport":"skills-sh","auth":{"type":"none","details":{"cli":"npx skills add sickn33/antigravity-awesome-skills","source_repo":"https://github.com/sickn33/antigravity-awesome-skills","install_from":"skills.sh"}},"qualityScore":"0.700","qualityRationale":"deterministic score 0.70 from registry signals: · indexed on github topic:agent-skills · 34583 github stars · SKILL.md body (6,412 chars)","verified":false,"liveness":"unknown","lastLivenessCheck":null,"agentReviews":{"count":0,"score_avg":null,"cost_usd_avg":null,"success_rate":null,"latency_p50_ms":null,"narrative_summary":null,"summary_updated_at":null},"enrichmentModel":"deterministic:skill-github:v1","enrichmentVersion":1,"enrichedAt":"2026-04-22T18:52:07.324Z","embedding":null,"createdAt":"2026-04-18T21:43:25.584Z","updatedAt":"2026-04-22T18:52:07.324Z","lastSeenAt":"2026-04-22T18:52:07.324Z","tsv":"'-13':718 '-6':167,758 '/comments':805 '/pulls':803 '/replies':807 '1':46,166,170,180,194,237,383,757,761,770 '10.15':697 '13':701 '2':52,171,181,195,244,388,762,771 '3':61,172,182,196,249,406,763,772 '4':67,176,187,201,255,411,767,777 '5':74,177,188,203,261,768,779 '6':81,173,183,197,764,773 'absolut':95,516,680 'acknowledg':77,231,486 'act':624 'action':130,228,533,850 'actual':336 'add':374 'afoot':481 'agreement':225,619,672,834 'alway':837 'anyth':144,385,526 'anyway':663 'api':699,799 'apolog':598 'applic':844 'architectur':448,470 'ask':30,60,117,146,185,219,459,666,888 'assum':32,640 'avoid':648 'back':121,273,418,420,453,475,567,603 'backward':705 'bad':673 'batch':631 'blind':625 'block':394 'bottom':817 'boundari':896 'break':246,396,423,646 'brief':494 'build':694 'bundl':711 'call':344,736 'care':322 'catch':500,522,548 'chang':498 'check':63,238,245,250,256,262,321,324,575,644,693 'circl':484 'clarif':148,199,775,890 'clarifi':118,384,655 'claude.md':98 'clear':863 'code':3,5,13,17,43,513,539,677,692 'codebas':65,73,243,334,629,734 'comfort':37,652 'comment':790,796,815 'common':614 'compat':706 'complet':48 'complex':402 'conflict':295,442 'context':267,429 'core':25 'correct':34,240,487,492,561,588,609,651 'criteria':899 'csv':731 'current':253,707 'databas':728 'date':729 'decis':302,449 'defend':599 'defens':458 'delet':554 'describ':851,867 'descript':495 'direct':668 'discuss':305 'drop':716 'easili':280 'emot':11,23 'endpoint':341,738 'environ':879 'environment-specif':878 'evalu':9,21,68,823 'exampl':161,670 'excel':102 'execut':846 'exist':247,424,441 'expert':884 'explain':606 'explicit':97 'export':732 'express':529 'extern':233,316,819 'factual':610 'featur':327,371,433 'feedback':45,49,103,134,317,382,488,490,545,820 'filter':730 'first':310,387,630,658 'fix':165,399,403,409,493,503,507,536,558,596,617,713,755 'follow':827 'forbidden':90 'full':266,428 'function':248,425 'gh':798 'github':782,792 'good':499,688,721,751 'grace':560 'gratitud':528 'great':100,518 'grep':333,733 'handl':132,209 'heard':543 'human':163,212,298,308,312,354,445,467,753 'id':712,806 'impl':708 'implement':29,82,107,143,160,179,215,236,254,331,351,376,390,581,626,654,723,781,831 'import':401 'incorrect':435 'individu':410 'initi':590 'inlin':788 'input':893 'instead':112,559 'investigate/ask/proceed':293 'involv':465 'isn':342 'issu':395,502 'item':84,137,151,153,193,381,657,750,756 'k':485 'lack':427 'later':189 'legaci':676,691,703 'legacy/compatibility':439 'let':105,682 'level':813 'limit':665,855 'line':818 'locat':505 'logic':405 'long':597 'loud':477 'm':747 'match':864 'may':154 'metric':725 'miss':748,901 'mistak':615,616 'move':612 'multi':380 'multi-item':379 'need':198,369,700,702,774 'never':92 'noth':735 'one':83,634 'order':377,393,825 'output':873 'over-explain':604 'overview':16,854 'owner':801 'partial':157,653 'partner':164,213,299,309,313,355,446,468,754 'pattern':40 'perform':12,24,104,224,618,671,833 'permiss':894 'platforms/versions':260 'point':101,519 'pr':804,814 'pre':717 'principl':26 'prior':301 'proceed':205,662 'profession':326 'proper':332,352,724 'push':120,272,417,419,452,474,566,602 'pushback':80,563,649 'question':119,461,829 're':94,515,587,679 'react':51 'read':47 'real':669 'realiti':66 'reason':79,124,251,276,440,456,595 'receiv':2,42 'receiving-code-review':1 'recept':15 'refactor':404 'refer':462 'regress':414 'relat':156 'remov':345,675,684,690,739 'repli':784,786,793 'repo':800,802 'report':362 'requir':7,19,55,116,621,892 'respond':75 'respons':39,91 'restat':54,113 'review':4,6,14,18,44,234,264,329,360,426,641,674,689,722,789,885 'right':96,190,517,573,643,681 'rigor':836 'rule':315,357 'safeti':895 'say':282 'scope':221,866 'secur':397 'seem':270 'show':510,541 'signal':471 'simpl':398 'skeptic':319 'skill':842,858 'skill-receiving-code-review' 'skip':226 'social':36 'sound':70 'sourc':207 'source-sickn33' 'source-specif':206 'speak':534 'specif':208,460,501,880 'stack':438 'start':128 'state':556,607,620,664 'still':218 'stop':140,303,886 'strang':478 'substitut':876 'success':898 'suggest':269,330,422,821 'support':719 'target':695 'task':862 'technic':8,20,33,69,76,115,123,230,239,275,434,455,650,686,835 'test':88,407,633,638,882 'tests/code':464 'thank':520,524,532,553 'thing':479,647 'thread':783,797 'time':87,637 'top':812 'top-level':811 'topic-agent-skills' 'topic-agentic-skills' 'topic-ai-agent-skills' 'topic-ai-agents' 'topic-ai-coding' 'topic-ai-workflows' 'topic-antigravity' 'topic-antigravity-skills' 'topic-claude-code' 'topic-claude-code-skills' 'topic-codex-cli' 'topic-codex-skills' 'track':726 'treat':871 'trust':214 'typo':400 'unclear':133,139,150,174,222,386,749,765 'uncomfort':473 'understand':53,158,169,192,217,265,591,760,769 'unus':339,432 'usag':337,745 'use':349,454,840,856 'valid':881 'verif':111,687 'verifi':27,62,281,287,412,583,627,661,828 'violat':99,430 'without':50,289,632 'word':58,131 'work':129,257,463 'workflow':848 'write':552 'wrong':126,159,178,271,570,593,710 'x':290,576 'y':580 'yagni':323,347,431,720,741 'yet':145","prices":[{"id":"18031d04-a53b-4350-bfa5-613ebcde7b20","listingId":"acea5724-7c98-458e-a0ee-05697e91fa6c","amountUsd":"0","unit":"free","nativeCurrency":null,"nativeAmount":null,"chain":null,"payTo":null,"paymentMethod":"skill-free","isPrimary":true,"details":{"org":"sickn33","category":"antigravity-awesome-skills","install_from":"skills.sh"},"createdAt":"2026-04-18T21:43:25.584Z"}],"sources":[{"listingId":"acea5724-7c98-458e-a0ee-05697e91fa6c","source":"github","sourceId":"sickn33/antigravity-awesome-skills/receiving-code-review","sourceUrl":"https://github.com/sickn33/antigravity-awesome-skills/tree/main/skills/receiving-code-review","isPrimary":false,"firstSeenAt":"2026-04-18T21:43:25.584Z","lastSeenAt":"2026-04-22T18:52:07.324Z"}],"details":{"listingId":"acea5724-7c98-458e-a0ee-05697e91fa6c","quickStartSnippet":null,"exampleRequest":null,"exampleResponse":null,"schema":null,"openapiUrl":null,"agentsTxtUrl":null,"citations":[],"useCases":[],"bestFor":[],"notFor":[],"kindDetails":{"org":"sickn33","slug":"receiving-code-review","github":{"repo":"sickn33/antigravity-awesome-skills","stars":34583,"topics":["agent-skills","agentic-skills","ai-agent-skills","ai-agents","ai-coding","ai-workflows","antigravity","antigravity-skills","claude-code","claude-code-skills","codex-cli","codex-skills","cursor","cursor-skills","developer-tools","gemini-cli","gemini-skills","kiro","mcp","skill-library"],"license":"mit","html_url":"https://github.com/sickn33/antigravity-awesome-skills","pushed_at":"2026-04-22T06:40:00Z","description":"Installable GitHub library of 1,400+ agentic skills for Claude Code, Cursor, Codex CLI, Gemini CLI, Antigravity, and more. Includes installer CLI, bundles, workflows, and official/community skill collections.","skill_md_sha":"5f5d39d606172d552e905d17e98e1ba8050801a7","skill_md_path":"skills/receiving-code-review/SKILL.md","default_branch":"main","skill_tree_url":"https://github.com/sickn33/antigravity-awesome-skills/tree/main/skills/receiving-code-review"},"layout":"multi","source":"github","category":"antigravity-awesome-skills","frontmatter":{"name":"receiving-code-review","description":"Code review requires technical evaluation, not emotional performance."},"skills_sh_url":"https://skills.sh/sickn33/antigravity-awesome-skills/receiving-code-review"},"updatedAt":"2026-04-22T18:52:07.324Z"}}