Skip to content

DRAFT: docs(AGENTS): how to reply to GitHub inline review threads via REST API#2090

Open
enyst wants to merge 3 commits intomainfrom
chore/gh-inline-replies-doc
Open

DRAFT: docs(AGENTS): how to reply to GitHub inline review threads via REST API#2090
enyst wants to merge 3 commits intomainfrom
chore/gh-inline-replies-doc

Conversation

@enyst
Copy link
Collaborator

@enyst enyst commented Feb 16, 2026

Adds a short note to AGENTS.md documenting the correct REST API pattern for replying to existing inline PR review comment threads.

Rationale: GitHub’s API does not provide a .../pulls/comments/{comment_id}/replies endpoint for PR review comments; instead you reply by POSTing a new PR review comment with in_reply_to.

Co-authored-by: openhands openhands@all-hands.dev

@enyst can click here to continue refining the PR


Agent Server images for this PR

GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server

Variants & Base Images

Variant Architectures Base Image Docs / Tags
java amd64, arm64 eclipse-temurin:17-jdk Link
python amd64, arm64 nikolaik/python-nodejs:python3.12-nodejs22 Link
golang amd64, arm64 golang:1.21-bookworm Link

Pull (multi-arch manifest)

# Each variant is a multi-arch manifest supporting both amd64 and arm64
docker pull ghcr.io/openhands/agent-server:de8501e-python

Run

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-de8501e-python \
  ghcr.io/openhands/agent-server:de8501e-python

All tags pushed for this build

ghcr.io/openhands/agent-server:de8501e-golang-amd64
ghcr.io/openhands/agent-server:de8501e-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:de8501e-golang-arm64
ghcr.io/openhands/agent-server:de8501e-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:de8501e-java-amd64
ghcr.io/openhands/agent-server:de8501e-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:de8501e-java-arm64
ghcr.io/openhands/agent-server:de8501e-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:de8501e-python-amd64
ghcr.io/openhands/agent-server:de8501e-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-amd64
ghcr.io/openhands/agent-server:de8501e-python-arm64
ghcr.io/openhands/agent-server:de8501e-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-arm64
ghcr.io/openhands/agent-server:de8501e-golang
ghcr.io/openhands/agent-server:de8501e-java
ghcr.io/openhands/agent-server:de8501e-python

About Multi-Architecture Support

  • Each variant tag (e.g., de8501e-python) is a multi-arch manifest supporting both amd64 and arm64
  • Docker automatically pulls the correct architecture for your platform
  • Individual architecture tags (e.g., de8501e-python-amd64) are also available if needed

\nCo-authored-by: openhands <openhands@all-hands.dev>
@enyst enyst marked this pull request as ready for review February 16, 2026 14:38
Copy link
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Good taste - Pragmatic documentation that solves a real problem. The REST API pattern for replying to inline review threads is non-obvious and worth documenting. LGTM!

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

Comments