fix(review): explain-change gathers PR facts through REST so it works in cloud sessions #6750

Activity

1. github-actions

   added

   needs-triageNot yet classified. Floor until a type and one priority tier are set.Not yet classified. Floor until a type and one priority tier are set.

   on Oct 10, 2026
2. github-actions commented on Oct 10, 2026

   

   github-actionsbot

   on Oct 10, 2026 – with GitHub Actions

   Contributor

   More actions

   Thanks for the report. Automated triage handed this issue to a maintainer.

   _{Automated triage. A maintainer reviews it.}
3. github-actions

   added

   needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.Human-in-the-loop required; autonomous sessions must not resolve items carrying this.

   on Oct 10, 2026
4. kyle-sexton commented on Oct 11, 2026

   

   kyle-sexton

   on Oct 11, 2026 – with Claude

   ContributorAuthor

   More actions

   Draft PR: #7081 (branch fix/6750-read-pr-over-rest).

   explain-change already reads PR facts through /source-control:pull-request view (#7026, per ADR 0060), so the fix sits in that action's script: view now reads the PR, its files and its diff through gh api (REST) with the same output fields, and finds a branch's PR through the pulls head filter. Verified with the updated read-pr.test.sh (45 pass), the explain-change node tests, and a live end-to-end run of explain-change's fact path (facts, diff, digest policy, publish gate) in a cloud session where gh pr view returns the GraphQL 403.

   Left for a human: the list action still uses gh pr list (GraphQL); explain-change does not call it, but quality-gate and improve do. A follow-up issue may be worth filing. A head branch on a fork is not found by branch name; pass the number.

   The needs-human label was left for Kyle to clear.

   ---

   Generated by Claude Code
5. kyle-sexton commented on Oct 11, 2026

   

   kyle-sexton

   on Oct 11, 2026 – with Claude

   ContributorAuthor

   More actions

   Handoff note for whoever picks this up next. The overnight Claude run has stopped, and Kyle is moving issue work to Cursor.

   • Branch: fix/6750-read-pr-over-rest
   • PR: fix(source-control): read pull-request facts and diffs over REST in view #7081 (draft)
   • State: PR opened, verified PASS after 2 fixes (e69549f; live REST view of fix(planning): word interview questions so yes accepts the recommendation #7036 works where gh pr view gets the GraphQL 403). Live fact path checked against fix(planning): word interview questions so yes accepts the recommendation #7036
   • Notes: Fixed in source-control read-pr.sh view (REST only); explain-change unchanged. Follow-up: list action still uses gh pr list (GraphQL), used by quality-gate and improve. Fork PRs need a number
   • Next step: Review the draft PR; decisions for Kyle are listed in its body. Mark it ready once Kyle agrees (that starts the CI test and review lanes), then drive CI green.

   Where a branch is listed it is pushed, so it can be taken over as is. A draft PR shows a red ci-status by design until it is marked ready.

   ---

   Generated by Claude Code
6. kyle-sexton

   added the

   Bug

   issue type on Oct 12, 2026
7. kyle-sexton

   removed

   needs-triageNot yet classified. Floor until a type and one priority tier are set.Not yet classified. Floor until a type and one priority tier are set.

   on Oct 12, 2026
8. kyle-sexton

   closed this as completedin #7081on Oct 13, 2026