[ptr727]

Summary

• Adds scripts/gh_graphql.py, one GraphQL transport (request, raising GraphQLError, never exiting) and one cursor walker (walk, connection path as a parameter).
• pr_review.py keeps its exit 1 and messages by mapping GraphQLError to its existing SystemExit. handoff.py maps it to Execution, exit 2.
• No new exit code. All existing tests pass unchanged. Added tests/test_gh_graphql.py and a handoff test that a transport failure exits 2 (the "errors beside rows" case fails on the old code, which answered from the rows).

Intended behavior changes

• pr_review.py: a null pullRequest now ends in a clean exit 1 with a message, where it used to raise a traceback.
• pr_review.py: a page claiming another without an advancing cursor now ends in a clean exit 1, where it used to loop forever.
• handoff.py: a response carrying errors, or lacking pageInfo.hasNextPage, is now exit 2, where it used to answer from the rows it had.

Closes #2903

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: d6295bd8-e203-4455-a536-5215c546bda9

---

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

---

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

🟡 Changes recommended

Transport failures can still escape the intended error mapping or lose the GraphQL error details.

3 open findings

• Map gh launch failures to GraphQLError · New
• Validate GraphQL response data is an object · New
• Preserve GraphQL errors from non-zero gh responses · New

What changed in this PR

This PR shares GraphQL request and pagination code between the review and handoff scripts while keeping each script's exit codes.

Changes:

• Adds a shared GraphQL transport and cursor walker.
• Moves both scripts onto the shared code and adds failure-path tests.

File	Description
tests/​test_handoff.py	Tests handoff GraphQL failures.
tests/​test_gh_graphql.py	Tests the shared transport and walker.
scripts/​README.md	Documents the new library and tests.
scripts/​pr_review.py	Uses the shared transport and walker.
scripts/​handoff.py	Adapts handoff reads to the shared code.
scripts/​gh_graphql.py	Implements the shared transport and walker.

🧠 Review effort: Lite

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[ptr727]

A recorded local strict-review pass covers head 9db325158d46a83184a17f7363d869a755f2ecb0, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[codecov]

Codecov Report

❌ Patch coverage is 96.22642% with 4 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@26b8da7). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
scripts/pr_review.py	86.66%	2 Missing ⚠️
scripts/gh_graphql.py	98.59%	1 Missing ⚠️
scripts/handoff.py	95.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             develop    #2914   +/-   ##
==========================================
  Coverage           ?   60.00%           
==========================================
  Files              ?       17           
  Lines              ?     8773           
  Branches           ?        0           
==========================================
  Hits               ?     5264           
  Misses             ?     3509           
  Partials           ?        0           

Flag	Coverage Δ
python-3.13	60.00% <96.22%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.