Skip to content

mark-pr-ready: fall back to GraphQL when the CCR draft route is missing - #3793

Merged
luhenry merged 1 commit into
mainfrom
mark-pr-ready-convert-draft
Oct 23, 2026
Merged

luhenry merged 1 commit into
mainfrom
mark-pr-ready-convert-draft

Conversation

@luhenry

@luhenry luhenry commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Summary

convert_to_draft in ci_scripts/mark-pr-ready.py posted to a /ccr/convert_to_draft endpoint that only exists inside Claude Code sessions. In GitHub Actions it returns 404 (seen on PR #3791 in run 39969829885), which aborted the whole run and left the remaining PRs unprocessed.

The function now tries the CCR route first and, on a 404, falls back to PyGithub's pr.convert_to_draft() (GraphQL). Other HTTP errors still raise. The docstring is updated, since it wrongly claimed the CCR route works everywhere.

Testing

Only checked that the script compiles; not run against GitHub. The next scheduled run will confirm the GraphQL fallback works with the Actions GITHUB_TOKEN.

The CCR convert_to_draft route only exists inside Claude Code sessions; in
GitHub Actions it returns 404 and aborted the whole run.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WLrKmVNbTh8QfP71a78k8b
@luhenry
luhenry merged commit 8cc4f8f into main Oct 23, 2026
4 of 6 checks passed
@luhenry
luhenry deleted the mark-pr-ready-convert-draft branch October 23, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants