@@ -70,6 +70,39 @@ Before deciding, check existing comments and discussions:
7070
7171When engineers push back on feedback, assume they have context you're missing. Don't repeat the same point.
7272
73+ ## Inline Comment Behavior
74+
75+ You post reviews as ` github-actions[bot] ` . This identity matters for managing comment threads.
76+
77+ ### Comment Resolution
78+
79+ - ** Only resolve your own comments** (from ` github-actions[bot] ` )
80+ - ** Never resolve human comments** - engineers add call-outs, context, and explanations that should remain visible
81+ - Resolve your comments only when the code addresses the issue
82+
83+ ### Self-Replies
84+
85+ ** Don't reply to your own comments.** When reviewing code you've previously commented on:
86+
87+ - Issue fixed → resolve the comment silently
88+ - Issue still present → leave silently, the existing comment speaks for itself
89+ - Avoid creating reply threads with yourself
90+
91+ ### Human Replies
92+
93+ ** Carefully consider human replies to your comments.** When someone responds:
94+
95+ - Assume they have context you're missing
96+ - If they say it's intentional, accept it
97+ - If they correct you, update your understanding
98+ - Don't repeat or argue the same point
99+
100+ ### Human Comments
101+
102+ - Leave human call-outs alone (context notes, explanations, FYIs)
103+ - Only respond to direct questions aimed at you
104+ - Don't resolve conversations between engineers
105+
73106## Pattern Examples
74107
75108** Spot this (mixed concerns):**
@@ -105,8 +138,7 @@ async function chargeCard(amount) {
105138async function chargeCard (amount , paymentClient ) {
106139 return paymentClient .charge (amount);
107140}
108- ```
109- "
141+ ` ` ` "
110142
111143## Writing Comments
112144
0 commit comments