chore: migrate from yarn to pnpm#115
Merged
Merged
Conversation
- Update packageManager field to pnpm@11.9.0 - Replace yarn.lock with pnpm-lock.yaml - Update all script to use pnpm run - Update CI workflow to use pnpm/action-setup@v4 and pnpm cache Claude-Session: https://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv
- Upgrade pnpm/action-setup from v4 (Node 20) to v6 (Node 24) - Upgrade CI Node.js version from 18.x to 24.x - Upgrade action.yml runtime from node16 to node20 Fixes CI failure caused by Node 20 deprecation on GitHub Actions runners. Claude-Session: https://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv
pnpm/action-setup@v6 reads the version from packageManager in package.json automatically; specifying both causes an error. Claude-Session: https://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv
Replace default import `yaml` with named import `{ load as yamlLoad }` in readConfig.ts to resolve the import/no-named-as-default-member lint error.
Claude-Session: https://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv
- Add @jest/globals to devDependencies (was a phantom dep under yarn's hoisted node_modules, not available under pnpm's strict isolation)
- Base the fs mock in readConfig.test.ts on jest.requireActual('fs') to preserve fs.constants (O_RDONLY etc.) needed by @actions/core transitive deps at init time
Claude-Session: https://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
yarnwithpnpm@11.9.0as the package manageryarn.lock, addpnpm-lock.yaml(generated viapnpm install)packageManagerfield inpackage.jsontopnpm@11.9.0allscript to usepnpm runinstead ofnpm runpnpm/action-setup@v4, switchsetup-nodecache fromyarntopnpm, replaceyarn install/yarn allwithpnpm install/pnpm allTest plan
pnpm installinstalls cleanly with no errorspnpm all(build → format → lint → package → test → copy-matcher) succeedsdist/is up to date after the build stephttps://claude.ai/code/session_011RYBwYUi1wcQL2atYneNAv