fix(gmail): add reply-all recipients for draft replies#805
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 15, 2026, 1:27 AM ET / 05:27 UTC. Summary Reproducibility: yes. Source inspection shows current main prepares draft reply metadata but builds draft recipients only from explicit To/Cc/Bcc while allowing missing To, matching the linked report. Review metrics: 3 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land the latest head after maintainer review and green checks if the opt-in draft Do we have a high-confidence way to reproduce the issue? Yes. Source inspection shows current main prepares draft reply metadata but builds draft recipients only from explicit To/Cc/Bcc while allowing missing To, matching the linked report. Is this the best way to solve the issue? Yes for the latest PR shape, subject to maintainer approval of the new CLI flag. The explicit opt-in mode preserves no-recipient drafts by default while matching the existing send/reply-all recipient derivation model. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 23b847ca53cf. Label changesLabel justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
|
@clawsweeper stop |
|
🦞✅ I added |
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
cf1edbf to
c89a054
Compare
Summary
--reply-alltogog gmail drafts createandgog gmail drafts update.--reply-allis set, reusing the existing reply recipient logic.--reply-all.Fixes #804
Testing
make docs-commandsgo test -vet=off ./internal/cmd -run 'Test(GmailDrafts(Create|Update)Cmd|BuildReplyRecipients|ReplyInfo|FetchReplyInfo)'make ciNotes
Reported by @privatenumber in #804.
ClawSweeper 🐠 replacement reef notes:
fish notes: reasoning high; reviewed against 6582fc7.