Skip to content

fix(staged): use robot emoji for agent comment suffix - #930

Merged
matt2e merged 2 commits into
mainfrom
github-suffix-should-be-emoji
Aug 26, 2026
Merged

fix(staged): use robot emoji for agent comment suffix#930
matt2e merged 2 commits into
mainfrom
github-suffix-should-be-emoji

Conversation

@matt2e

@matt2e matt2e commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Agent-authored review comments pushed to GitHub were suffixed with \n\nGenerated by AI, which reads as a full paragraph of boilerplate under every comment. Replace it with a compact \n🤖 suffix so the attribution is a single trailing glyph instead of a separate block.

Changes

  • github_single_comment_body now appends \n🤖 to agent-authored comments (single newline, no blank line).
  • Added a doc comment explaining the agent/human split.
  • Updated the existing agent-suffix test and added coverage asserting human-authored comments are posted verbatim.

matt2e and others added 2 commits August 26, 2026 10:46
Agent-authored review comments posted to GitHub were suffixed with the
text "Generated by AI". Replace it with the 🤖 emoji so the attribution
reads as a compact marker rather than a sentence.

Human-authored comments continue to post verbatim with no suffix; add a
test pinning that behaviour alongside the updated agent-suffix test.

Signed-off-by: Matt Toohey <contact@matttoohey.com>
Review feedback on the agent-suffix change: the blank line between the
comment body and the 🤖 marker was unnecessary. Separate them with a
single newline instead, and update the agent-attribution test to match.

Re: #review:0659177f-8fc4-4ba7-b21a-44473ee508d1

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e
matt2e merged commit 67d96ef into main Aug 26, 2026
2 checks passed
@matt2e
matt2e deleted the github-suffix-should-be-emoji branch August 26, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant