From ed5e98b643c11531deee4d9764a5ecb830390a92 Mon Sep 17 00:00:00 2001 From: cliffhall Date: Sun, 6 Sep 2026 12:28:01 -0400 Subject: [PATCH] docs: paginate the review-comments fetch MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Copilot review round 2 on #2273: the list endpoint pages at 30, so a large review round would silently drop later comments — and a workflow whose whole point is replying to every comment cannot be documented with a command that returns only the first thirty. --paginate combines with --jq fine (only --slurp does not), verified against a live review. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_017nLLLTGc37hQiC8s3mNyhq Signed-off-by: cliffhall --- .claude/skills/pr-flow/SKILL.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.claude/skills/pr-flow/SKILL.md b/.claude/skills/pr-flow/SKILL.md index d7d1a06e..08947b67 100644 --- a/.claude/skills/pr-flow/SKILL.md +++ b/.claude/skills/pr-flow/SKILL.md @@ -291,7 +291,9 @@ arrive late (see step 8). ```sh # Fetch the round's comments by REVIEW id — the unpaginated /reviews listing # hides later rounds behind your own replies. - gh api repos/modelcontextprotocol/inspector/pulls//reviews//comments \ + # --paginate: this endpoint returns 30 per page, and a round you only half + # fetch is a round you only half answer. + gh api --paginate repos/modelcontextprotocol/inspector/pulls//reviews//comments \ --jq '.[]|"\(.id) \(.path):\(.line)\n\(.body)"' # Reply into one thread, keyed by the comment id from above.