Skip to content

fix(sdk): skip empty REST update when only toggling PR draft - #101

Merged
HugoRCD merged 2 commits into
mainfrom
fix/sdk-update-pr-draft-only
Aug 25, 2026
Merged

fix(sdk): skip empty REST update when only toggling PR draft#101
HugoRCD merged 2 commits into
mainfrom
fix/sdk-update-pr-draft-only

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Aug 25, 2026

Copy link
Copy Markdown
Member

Octokit serializes a PATCH with all REST fields undefined to an empty body, which GitHub rejects before the GraphQL draft mutation runs.

🔗 Linked issue

Closes #89

📚 Description

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
github-tools-docs Ready Ready Preview, v0 Aug 25, 2026 1:26pm
github-tools-test-agent Ready Ready Preview, v0 Aug 25, 2026 1:26pm

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

Octokit serializes a PATCH with all REST fields undefined to an empty
body, which GitHub rejects before the GraphQL draft mutation runs.
provider-utils 4.x resolves undici 5.29.0, which dependency review flags as high (GHSA-v9p9-hfj2-hcw8).
@HugoRCD
HugoRCD force-pushed the fix/sdk-update-pr-draft-only branch from 6325a68 to e15e105 Compare August 25, 2026 13:25
@HugoRCD HugoRCD self-assigned this Aug 25, 2026
@HugoRCD
HugoRCD merged commit ade87da into main Aug 25, 2026
14 checks passed
@HugoRCD
HugoRCD deleted the fix/sdk-update-pr-draft-only branch August 25, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

updatePullRequest fails with "Body should be a JSON object" when only toggling draft

1 participant