Skip to content

Comments

feat: add codemod and publishing workflow updates via GitHub Actions OIDC closes #112#124

Merged
bjohansebas merged 5 commits intoexpressjs:mainfrom
skyeyeye:feat/trusted-publisher
Feb 22, 2026
Merged

feat: add codemod and publishing workflow updates via GitHub Actions OIDC closes #112#124
bjohansebas merged 5 commits intoexpressjs:mainfrom
skyeyeye:feat/trusted-publisher

Conversation

@skyeyeye
Copy link
Contributor

@skyeyeye skyeyeye commented Feb 3, 2026

📦 Publish codemods using GitHub Actions Trusted Publisher (OIDC) closes #112

This PR migrates codemod publishing to GitHub Actions trusted publishing using OIDC, removing the need for long-lived API keys.

✨ What changed

Removed API-key based authentication (codemod login --api-key)

Publishing now relies on the workflow’s verified identity

Existing publish and test steps remain unchanged

🔒 Why this is better

Eliminates stored secrets in GitHub

Improves supply-chain and release security

Aligns with Codemod’s recommended publishing approach

📖 Reference

Trusted publishers documentation:
https://docs.codemod.com/publishing#trusted-publishers

Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

see nodejs/userland-migrations@df28625

You have to use the codemod action

Signed-off-by: Ayush Saha <ayushsaha.work@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM! thanks!

@bjohansebas
Copy link
Member

Note: do not merge. I need to talk with the codemod team about this because I see that I have to configure something from the platform

@bjohansebas
Copy link
Member

Okay, no changes are necessary. @expressjs/codemod-captains, I’m going to remove the token from the environment and also delete the token on the platform.

@bjohansebas bjohansebas merged commit 2992380 into expressjs:main Feb 22, 2026
29 checks passed
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.

publish: use trusted published

2 participants