Skip to content

source-control:babysit-prs: fetch-all-pr-comments.sh reports in_reply_to_id=None for properly-threaded inline review replies #587

Description

@kyle-sexton

Lane: babysit (safe/gate tier, /loop /source-control:babysit-prs)

Triggering example: worker processing #563 needed to confirm whether kyle-sexton's classification-table replies were threaded under the correct codex inline review comments (P1→#572 defer, P2→#573 defer) before deciding resolve-vs-leave-open. fetch-all-pr-comments.sh reported in_reply_to_id=None for these replies even though GraphQL thread membership confirmed they are properly threaded inline replies.

Observed: in_reply_to_id field from the script is unreliable for thread linkage — a consumer following the script's own contract would wrongly conclude these are top-level/unthreaded comments.
Expected: the field either reflects true reply-parent linkage, or the script's docs explicitly warn not to use it for thread-membership decisions (forcing callers to a GraphQL cross-check, which is what the worker did as a workaround this cycle).

Category: bug
Impact: any babysit worker or the orchestrator itself that trusts in_reply_to_id for "is this reply threaded under finding X" decisions can silently misclassify addressed-vs-unaddressed findings.

Work-class: C3 (bug-fix-shaped) — attended triage 2026-07-23, operator-ratified. 🤖

Metadata

Metadata

Assignees

Labels

agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.priority: mediumReal value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions