Skip to content

Switch to pnpm and Node.js 24#19

Closed
carlos-poly wants to merge 1 commit intomainfrom
carlos/02-26-switch_to_pnpm
Closed

Switch to pnpm and Node.js 24#19
carlos-poly wants to merge 1 commit intomainfrom
carlos/02-26-switch_to_pnpm

Conversation

@carlos-poly
Copy link
Copy Markdown

@carlos-poly carlos-poly commented Feb 26, 2026

Summary

  • Switch package manager from Yarn to pnpm (lockfile, CI workflow, Makefile, README)
  • Bump minimum Node version from 20.10 to 24
  • Remove ethers from dependencies (never imported anywhere)
  • Move @ethersproject/providers and @ethersproject/wallet to devDependencies (source only uses type imports)
  • Remove 8 unused devDependencies: eslint-config-standard-with-typescript, eslint-plugin-import, eslint-plugin-n, eslint-plugin-node, eslint-plugin-promise, esm, ts-mocha, ts-node
  • Drop unused --require tsconfig-paths/register from test command

Validation

  • pnpm build succeeds
  • All 22 tests pass
  • CI workflow updated to install pnpm and use pnpm commands
  • Verify CI passes on this branch

Note

Medium Risk
Moderate risk because it changes the package manager and bumps the required Node version to 24, which can affect dependency resolution and CI/runtime compatibility. No application logic changes, but lockfile/tooling shifts may surface build/test differences.

Overview
Migrates build/test/lint workflows from Yarn to pnpm: GitHub Actions now installs pnpm (pnpm/action-setup@v4) and runs pnpm install/lint/build/test, and the Makefile test target switches to pnpm nyc.

Updates package metadata for the new toolchain: package.json bumps version to 3.1.0, sets engines to node >=24 and pnpm >=10, and reclassifies some packages between dependencies and devDependencies. Adds pnpm-lock.yaml and removes Yarn-specific ignores/docs references.

Written by Cursor Bugbot for commit 5fcb9a7. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
Author

carlos-poly commented Feb 26, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@carlos-poly carlos-poly marked this pull request as ready for review February 26, 2026 18:06
@carlos-poly carlos-poly requested a review from a team as a code owner February 26, 2026 18:06
@carlos-poly carlos-poly force-pushed the carlos/02-26-switch_to_pnpm branch from beb7ce5 to 5fcb9a7 Compare February 26, 2026 18:45
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread .github/workflows/workflow.yaml
@carlos-poly carlos-poly changed the title Switch to pnpm Switch to pnpm and Node 24 Feb 26, 2026
@carlos-poly carlos-poly changed the title Switch to pnpm and Node 24 Switch to pnpm and Node.js 24 Feb 26, 2026
@carlos-poly carlos-poly closed this Mar 3, 2026
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