LearnNewsExamplesServices
Frontmatter
id10333
titleImplement since_comment_id and comment_id parameters for get_conversation
stateClosed
labels
enhancementaiarchitectureperformance
assigneesneo-gemini-pro
createdAtApr 25, 2026, 9:19 PM
updatedAtApr 25, 2026, 10:11 PM
githubUrlhttps://github.com/neomjs/neo/issues/10333
authorneo-gemini-pro
commentsCount2
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtApr 25, 2026, 10:11 PM

Implement since_comment_id and comment_id parameters for get_conversation

Closed v13.0.0/archive-v13-0-0-chunk-6 enhancementaiarchitectureperformance
neo-gemini-pro
neo-gemini-pro commented on Apr 25, 2026, 9:19 PM

Context

#10272 highlighted that A2A PR reviews suffer from quadratic token scaling because get_conversation fetches the entire thread every time. We agreed exact-match fetching is needed to solve this.

The Problem

The read-path for PR conversations currently lacks selective fetch. Agents must parse the full history to find new comments, wasting context tokens.

The Architectural Reality

The get_conversation tool in ai/mcp/server/github-workflow/ uses the gh CLI. GitHub's REST API supports per-ID fetch, but our wrapper doesn't expose it.

The Fix

Add optional comment_id, since_comment_id, and last_n parameters to get_conversation with proper precedence (comment_id > since_comment_id > last_n > full).

Acceptance Criteria

  • get_conversation accepts comment_id, since_comment_id, last_n optional parameters.
  • Backward compatibility is maintained if no new parameters are passed.

Out of Scope

New tools like get_comment and list_comments are explicitly out of scope to respect the tool-surface cap.

Related

Origin: #10272

Origin Session ID: 0b29a8fa-c6b0-42e2-ab3b-8015a99db2d8 Retrieval Hint: "selective fetch since_comment_id get_conversation a2a"