Skip to content

Migrate npm publishing to OIDC - #693

Merged
pomek merged 1 commit into
masterfrom
ci/4651-oidc
Jul 28, 2026
Merged

Migrate npm publishing to OIDC#693
pomek merged 1 commit into
masterfrom
ci/4651-oidc

Conversation

@pomek

@pomek pomek commented Jul 28, 2026

Copy link
Copy Markdown
Member

🚀 Summary

Migrates the release workflow from the long-lived CKE5_NPM_TOKEN to npm Trusted Publishing with CircleCI OIDC.

The PR also:

  • Installs npm ^11.11.0 in the release job because the cimg/node:24.11.0 image ships an npm version whose OIDC implementation does not recognize CircleCI (support was added in feat: adds circleci to oidc npm/cli#8925, first released in npm 11.11.0).
  • Configures releaseTools.publishPackages() to use OIDC (useOidc: true) and removes the login-based npmOwner option, as npm whoami does not support OIDC authentication.
  • Triggers the trusted GitHub App pipeline definition during releases (--pipeline-definition-id).
  • Switches the origin remote to SSH and pins GitHub's published Ed25519 host key so the release commit and tag can be pushed from the GitHub App checkout without an interactive prompt.
  • Adds a focused script test verifying the OIDC option and the absence of npmOwner.

The changes mirror the Mrgit and Umberto migrations driven by ckeditor/ckeditor5-internal#4432.


📌 Related issues


💡 Additional information

This is an internal CI change, so no changelog entry is required. ESLint and script tests pass locally, and the trusted publisher configuration inputs (organization, project, and pipeline definition IDs) were verified against the CircleCI API. Before the first release on this flow, a trusted publisher for @ckeditor/ckeditor5-react must be configured on npm, and a release after merge is required to validate the OIDC exchange with npm end to end.

@pomek
pomek merged commit efc70e5 into master Jul 28, 2026
10 checks passed
@pomek
pomek deleted the ci/4651-oidc branch July 28, 2026 05:45
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