Skip to content

ci: reduce release-please commit batch size#283

Merged
fan-zhang-sv merged 1 commit intomasterfrom
fix/release-please-graphql-batch-size
Apr 27, 2026
Merged

ci: reduce release-please commit batch size#283
fan-zhang-sv merged 1 commit intomasterfrom
fix/release-please-graphql-batch-size

Conversation

@fan-zhang-sv
Copy link
Copy Markdown
Collaborator

@fan-zhang-sv fan-zhang-sv commented Apr 27, 2026

Summary

  • Configure release-please to fetch commit history one commit per GitHub GraphQL request.
  • Work around repeated GitHub GraphQL internal errors while release-please is collecting merge commits on master.

Why

The release-please workflow is failing before any package build or npm publish step. The logs show release-please successfully loading config, finding the latest releases for both packages, and then failing while paging merge commits through GitHub GraphQL:

✔ Collecting commits since all latest releases
❯ Fetching merge commits on branch master with cursor: ...
Error: release-please failed: Request failed due to following response errors:
 - Something went wrong while executing your query ...

That Something went wrong while executing your query / E001 response is a GitHub GraphQL internal error. In this case, it is happening while release-please is fetching commit history, so reducing the per-request commit batch size makes each GraphQL query smaller and less likely to timeout or hit a GitHub-side execution failure.

Source For The Fix

Release-please v17.3.0 added commit-batch-size specifically to make commit-history pagination configurable:

The upstream PR describes the same failure mode: release-please consistently hitting GitHub GraphQL 502/internal errors while querying repository history, and succeeding only after lowering the number of commits fetched per request.

Tradeoff

Setting commit-batch-size to 1 increases the number of GitHub API calls, so the release-please job may run slower. The benefit is that each GraphQL request is much smaller, which should make the workflow more reliable for the current failure.

Test plan

  • Validated release-please-config.json parses as JSON.
  • Existing CI will run on the PR.

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 27, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@github-actions
Copy link
Copy Markdown
Contributor

PR title: ci: reduce release-please commit batch size — valid conventional commits format, no change needed.

Summary: This is a single-line config change adding "commit-batch-size": 1 to release-please-config.json to work around GitHub GraphQL internal errors during release-please commit history pagination. No code, type, API surface, or bundle size impact.

One minor observation: commit-batch-size: 1 is the most extreme setting — one GraphQL request per commit. If the release-please job becomes noticeably slow (e.g., after many commits accumulate between releases), consider bumping this to 5 or 10 as a middle ground. Not a blocking concern — starting at 1 for reliability and tuning up later is a reasonable approach.

No blocking issues found.

Copy link
Copy Markdown
Contributor

@spencerstock spencerstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fan-zhang-sv fan-zhang-sv merged commit ce3a8cc into master Apr 27, 2026
13 checks passed
@fan-zhang-sv fan-zhang-sv deleted the fix/release-please-graphql-batch-size branch April 27, 2026 17:33
fan-zhang-sv added a commit that referenced this pull request Apr 27, 2026
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.

4 participants