Skip to content

fix: posting UAT as a new comment, not update the original issue body (#43) - #46

Merged
ngthluu merged 1 commit into
mainfrom
ai/issue-43
Jul 25, 2026
Merged

fix: posting UAT as a new comment, not update the original issue body (#43)#46
ngthluu merged 1 commit into
mainfrom
ai/issue-43

Conversation

@ngthluu

@ngthluu ngthluu commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Closes #43

Automated by loope (bug flow). Spec and plan, if any, are committed in this branch under docs/.

The UAT step appended its checklist to the issue body via a
read-modify-write `gh issue edit --body`, rewriting the human's own report
to carry bot output. It now posts the section as a new issue comment
instead, leaving the body untouched.

- UATTarget is now UATSurfaces + Comment. UATSurfaces is one
  `gh issue view --json body,comments` returning the body followed by every
  comment; the marker check scans all of them, so an issue that already got
  a body checklist from an older loope does not gain a duplicate comment.
- IssueBody/AppendIssueBody are gone with their last caller.
- maxIssueBodyChars and its skip are gone: maxUATChars (8000) already keeps
  a comment far clear of GitHub's 65536 limit, so the guard was dead.

Judgment calls made headless:
- isBotStatusComment is left alone, so the UAT comment is still fed back as
  issue content on a rework run. That matches the previous behavior (the
  body was always included) and keeps any boxes a human ticked in context.
- The uat-section template is unchanged; the marker and heading read the
  same in a comment as they did in the body.

Closes #43

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ngthluu ngthluu self-assigned this Jul 25, 2026
@ngthluu
ngthluu merged commit 2ce1d6a into main Jul 25, 2026
1 check passed
@ngthluu
ngthluu deleted the ai/issue-43 branch July 25, 2026 11:19
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.

fix: posting UAT as a new comment, not update the original issue body

1 participant