source-control: read-pr.sh calls GraphQL before its REST read, so it fails in cloud sessions
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 76/100
Research direction
Read plugins/source-control/skills/pull-request/scripts/read-pr.sh around line 113, then compare the fallback behavior in babysit_gh.py and morning-brief.sh mentioned in the issue. Verify that the REST read runs before any failing GraphQL call, or that the specified GraphQL refusal triggers the REST fallback; the script should complete its pull request visibility read in cloud sessions.
Written by the indexing model from the issue text.
Description
Problem
plugins/source-control/skills/pull-request/scripts/read-pr.sh:113 runs gh pr view --json (GraphQL) before its REST visibility read. The cloud GitHub proxy refuses every GraphQL request with a 403 (https://code.claude.com/docs/en/cloud-environments#github-proxy, read 2026-10-11), so in a cloud session the script exits before the REST read runs.
Fix direction
Do the REST read first, or fall back to gh api repos/{owner}/{repo}/pulls/{n} when the GraphQL call returns the proxy's "GitHub GraphQL is not available from Claude Code sessions" refusal, matching the fallback babysit_gh.py and morning-brief.sh use after #7219.
Related
- #7219 (updated the GraphQL-refusal matchers)
🤖 Generated with Claude Code
- Dominant language
- Shell
- Stars
- 22
- Forks
- 2
- Avg merge
- 6h 42m
- Merged PRs (30d)
- 902
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from melodic-software/claude-code-plugins
-
needs-triage priority: medium
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
melodic-software/claude-code-plugins#7288 · 1 comment ·
Maintainers usually reply within 1 day
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdOpenneeds-human needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
melodic-software/claude-code-plugins#7286 · 1 comment ·
Maintainers usually reply within 1 day
-
needs-triage priority: low
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
melodic-software/claude-code-plugins#7282 · 1 comment ·
Maintainers usually reply within 1 day
-
needs-triage priority: medium
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
melodic-software/claude-code-plugins#7273 · 1 comment ·
Maintainers usually reply within 1 day
-
needs-human needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
melodic-software/claude-code-plugins#7232 · 1 comment ·
Maintainers usually reply within 1 day
All issues in melodic-software/claude-code-plugins
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
sclorg/s2i-ruby-container#660 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
wazuh/wazuh-docker#2732 ·
Maintainers usually reply within 1 day
-
Route 53 DNS analytics/insights adds an observability signal for the DNS layerPossibly taken A pull request linked to this issue is open or already merged. Opendocumentation enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100