Skip to content

fix(release): let release PR updates trigger CI - #470

Merged
morluto merged 1 commit into
mainfrom
fix/release-please-ci-auth
Aug 9, 2026
Merged

fix(release): let release PR updates trigger CI#470
morluto merged 1 commit into
mainfrom
fix/release-please-ci-auth

Conversation

@morluto

@morluto morluto commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Description

Release-please currently mutates its pull request with the workflow GITHUB_TOKEN. GitHub deliberately places pull_request workflows caused by that token into action_required, so every release-PR update needs a manual workflow approval before required checks can run.

Use a dedicated RELEASE_PLEASE_TOKEN for both mutations of the release branch: the release-please action and the follow-up canonical catalog commit. Keeping the same identity for both pushes is essential; otherwise the catalog normalization commit reintroduces the approval gate. The existing GITHUB_TOKEN remains a transition fallback, preserving today’s manual-approval behavior until the dedicated credential is provisioned.

The intended credential is a narrowly scoped GitHub App installation token, or a fine-grained PAT with repository contents and pull-request write access. The locally configured broad classic PAT was intentionally not copied into Actions.

Type of Change

  • Bug fix

Testing Performed

  • TypeScript typecheck passes (npm run typecheck)
  • Linting passes (npm run lint)
  • npm run format:check -- .github/workflows/release.yml — passed
  • git diff --check — passed

Tool Contract Impact

None.

Related Issues

Observed on release PR #463.

@cursor

cursor Bot commented Aug 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@morluto
morluto merged commit bf04d1a into main Aug 9, 2026
15 checks passed
@morluto
morluto deleted the fix/release-please-ci-auth branch August 9, 2026 19:46
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.

1 participant