Skip to content

docs(claude): note review-reply threading hygiene#6421

Merged
g-talbot merged 1 commit into
mainfrom
gtt/claude-md-review-replies
May 12, 2026
Merged

docs(claude): note review-reply threading hygiene#6421
g-talbot merged 1 commit into
mainfrom
gtt/claude-md-review-replies

Conversation

@g-talbot
Copy link
Copy Markdown
Contributor

@g-talbot g-talbot commented May 12, 2026

Adds one entry to the Known Pitfalls table in CLAUDE.md.

I'd been replying to Codex review comments by posting new inline comments at the same line rather than as threaded replies under the original. They look similar in some views but are NOT the same — the reply chain is broken, and the "Resolved conversations" view doesn't roll them up together.

Correct technique with gh:

gh api repos/{owner}/{repo}/pulls/{pr}/comments/{codex_comment_id}/replies --input reply.json

or pass in_reply_to_id in the POST body to pulls/{pr}/comments. Verify via the API that in_reply_to_id is set on the new comment.

Test plan

  • Doc-only change, no code paths affected

@g-talbot g-talbot requested a review from a team as a code owner May 12, 2026 13:03
When responding to PR review comments (Codex bot or human reviewers),
use the `replies` endpoint (`POST .../pulls/{pr}/comments/{id}/replies`)
or pass `in_reply_to_id` so the reply is **threaded under** the
original comment, not posted as a standalone inline comment at the
same line. Threaded replies show in the "Resolved conversations"
view; standalone inline comments don't. Verify via the GitHub API
that `in_reply_to_id` is set on your new comment.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@g-talbot g-talbot force-pushed the gtt/claude-md-review-replies branch from 277d237 to a43bf75 Compare May 12, 2026 13:04
@g-talbot g-talbot changed the title docs(claude): note review-reply threading + project name = Quickwit docs(claude): note review-reply threading hygiene May 12, 2026
@g-talbot g-talbot merged commit c9873f5 into main May 12, 2026
9 checks passed
@g-talbot g-talbot deleted the gtt/claude-md-review-replies branch May 12, 2026 13:48
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.

2 participants