Skip to content

ci: consolidate on RELEASE_PAT, remove NPM_TOKEN#48

Merged
mrjasonroy merged 1 commit into
mainfrom
ci/consolidate-tokens
Jun 25, 2026
Merged

ci: consolidate on RELEASE_PAT, remove NPM_TOKEN#48
mrjasonroy merged 1 commit into
mainfrom
ci/consolidate-tokens

Conversation

@mrjasonroy

Copy link
Copy Markdown
Owner

Collapses release automation to one secret (RELEASE_PAT) and removes a dead/expired one.

  • version-bump.yml: PAT_TOKENRELEASE_PAT
  • catch-up-release.yml: same RELEASE_PAT treatment (it had the identical GITHUB_TOKEN merge bug — bot-opened PR runs no CI, so auto-merge never fired)
  • catch-up-release.yml: dropped the NODE_AUTH_TOKEN: NPM_TOKEN line — this workflow doesn't publish (OIDC handles that downstream), so it was dead config on an expired token

NPM_TOKEN and PAT_TOKEN secrets deleted as part of this change.

🤖 Generated with Claude Code

Single token to manage instead of two, and remove a dead/expired secret:

- version-bump.yml: PAT_TOKEN -> RELEASE_PAT (checkout + PR creation).
- catch-up-release.yml: give it the same RELEASE_PAT treatment so its PR
  actually triggers CI and its merge fires tag-on-version-merge (it had the
  same GITHUB_TOKEN bug — bot-opened PR runs no CI, auto-merge never fires).
- catch-up-release.yml: drop the NODE_AUTH_TOKEN/NPM_TOKEN line — this workflow
  doesn't publish (publishing is OIDC via publish.yml downstream), so it was
  dead config referencing an expired token.

NPM_TOKEN and PAT_TOKEN secrets deleted (publishing is OIDC; releases use the
single RELEASE_PAT).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mrjasonroy mrjasonroy enabled auto-merge (squash) June 25, 2026 22:23
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@mrjasonroy mrjasonroy merged commit 18e8d30 into main Jun 25, 2026
7 checks passed
@mrjasonroy mrjasonroy deleted the ci/consolidate-tokens branch June 25, 2026 22:29
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