Skip to content

Fix npm publish by re-enabling corepack #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jkebinger merged 15 commits into from
Nov 20, 2025
Merged

Fix npm publish by re-enabling corepack #78

jkebinger merged 15 commits into from
Nov 20, 2025

Conversation

@jkebinger
Copy link
Contributor

@jkebinger jkebinger commented Nov 20, 2025

The prepack script calls 'yarn readme', but npm upgrade disables corepack. Re-enable it before publishing so the lifecycle script works.

@jkebinger
Re-enable corepack after npm upgrade for prepack script
190b847
@jkebinger
Enable workflow to run on pull requests for testing
5b6ad25
@jkebinger
Run yarn readme before npm upgrade and skip lifecycle scripts during …
0413aff 
…publish
@jkebinger
Upgrade to Yarn 4.11.0 for better OIDC support and use provenance
631de6a
@jkebinger
Add yarn version output before publishing
b3fcb1c
@jkebinger
Add explicit scope configuration for @reforge-com packages
0d9e2d5
@jkebinger
Add npm publish registry too
897348e
@jkebinger
Remove --provenance flag to test basic OIDC auth first
f356c9a
@jkebinger
Add comprehensive OIDC debug output based on Yarn implementation
8ee5be1
@jkebinger
Remove GitLab-specific debug output
0e2fae3
@jkebinger
Add OIDC token fetch test to debug silent failures
b45ed45
@jkebinger
Add full OIDC flow test: ID token fetch + NPM token exchange
f2ebb46
@jkebinger
Test both URL encodings for OIDC token exchange
27e1910
@jkebinger
Remove debug code - OIDC working after fixing repo name case
04645c0
@jkebinger
Remove pull_request trigger - only run on main branch
1b942b3
jdwyah
jdwyah approved these changes Nov 20, 2025
View reviewed changes
Copy link
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

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

thanks!

@jkebinger jkebinger merged commit 9be9143 into main Nov 20, 2025
6 checks passed
@jkebinger jkebinger deleted the fix-npm-publish-corepack branch November 20, 2025 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jdwyah jdwyah jdwyah approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jkebinger @jdwyah