Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion apps/auth/nextjs-ssr/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"@crossmint/server-sdk": "workspace:*",
"react": "^18",
"react-dom": "^18",
"next": "15.5.15"
"next": "15.5.18"
},
"devDependencies": {
"typescript": "^5",
Expand Down
2 changes: 1 addition & 1 deletion apps/auth/remix-ssr/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
"postcss": "^8.4.38",
"tailwindcss": "^3.4.4",
"typescript": "^5.1.6",
"vite": "^5.1.0",
"vite": "^6.4.2",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Vite major version bump (5→6) may require migration steps

vite is bumped from ^5.1.0 to ^6.4.2, which is a breaking major version change. Vite 6 introduced the new Environment API and several breaking changes that affect plugins, SSR configuration, and module resolution behavior. Please verify the Remix app's vite.config.ts is compatible with Vite 6 (check plugin API changes, resolve.conditions defaults, and any usage of the legacy CJS Node.js API).

Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/auth/remix-ssr/package.json
Line: 38

Comment:
**Vite major version bump (5→6) may require migration steps**

`vite` is bumped from `^5.1.0` to `^6.4.2`, which is a breaking major version change. Vite 6 introduced the new Environment API and several breaking changes that affect plugins, SSR configuration, and module resolution behavior. Please verify the Remix app's `vite.config.ts` is compatible with Vite 6 (check plugin API changes, `resolve.conditions` defaults, and any usage of the legacy CJS Node.js API).

How can I resolve this? If you propose a fix, please make it concise.

"vite-tsconfig-paths": "^4.2.1"
},
"engines": {
Expand Down
2 changes: 1 addition & 1 deletion apps/payments/nextjs/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
"@dynamic-labs/ethereum": "4.8.3",
"@dynamic-labs/sdk-react-core": "4.8.3",
"@dynamic-labs/solana": "4.8.3",
"next": "15.5.15",
"next": "15.5.18",
"react": "18.3.1",
"react-dom": "18.3.1"
},
Expand Down
2 changes: 1 addition & 1 deletion apps/wallets/quickstart-devkit/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"clsx": "2.1.1",
"firebase": "^10.3.1",
"firebaseui": "6.1.0",
"next": "15.5.15",
"next": "15.5.18",
"react": "^18",
"react-dom": "^18",
"tailwind-merge": "2.4.0",
Expand Down
2 changes: 1 addition & 1 deletion apps/wallets/react/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"react-native-web": "~0.21.2",
"@solana/web3.js": "1.98.1",
"@stellar/stellar-sdk": "^14.6.1",
"next": "^16.2.3",
"next": "^15.5.18",
"react": "^19.0.0",
"react-dom": "^19.0.0",
"viem": "^2.33.1"
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
"rimraf": "6.0.1",
"ts-jest": "29.1.1",
"tsup": "8.1.0",
"turbo": "2.2.3",
"turbo": "2.9.14",
"typescript": "5.5.3",
"vitest": "1.6.1",
"vitest-mock-extended": "2.0.2"
Expand Down
2 changes: 1 addition & 1 deletion packages/client/base/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"@crossmint/common-sdk-base": "workspace:*",
"@datadog/browser-logs": "6.24.1",
"exponential-backoff": "3.1.1",
"uuid": "9.0.1",
"uuid": "14.0.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 uuid v14 requires Node.js 20+ and removes CJS support

uuid v14 drops Node.js 18 support and uuid v12 removed CommonJS entirely. The package exports a CJS artifact ("require": "./dist/index.cjs" and "main": "./dist/index.cjs"), which means any consumer loading the CJS bundle will attempt to require('uuid'). If the bundler (tsup) does not inline uuid into the CJS output, this will fail at runtime on environments still using CJS imports or Node.js 18. Additionally, searching the source tree reveals uuid is not directly imported anywhere under packages/client/base/src — worth verifying whether this dependency is still needed at all.

Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/client/base/package.json
Line: 27

Comment:
**`uuid` v14 requires Node.js 20+ and removes CJS support**

`uuid` v14 drops Node.js 18 support and `uuid` v12 removed CommonJS entirely. The package exports a CJS artifact (`"require": "./dist/index.cjs"` and `"main": "./dist/index.cjs"`), which means any consumer loading the CJS bundle will attempt to `require('uuid')`. If the bundler (tsup) does not inline `uuid` into the CJS output, this will fail at runtime on environments still using CJS imports or Node.js 18. Additionally, searching the source tree reveals `uuid` is not directly imported anywhere under `packages/client/base/src` — worth verifying whether this dependency is still needed at all.

How can I resolve this? If you propose a fix, please make it concise.

"zod": "3.22.4"
},
"devDependencies": {
Expand Down
2 changes: 1 addition & 1 deletion packages/client/wallets/walletconnect/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
"@types/react": "^18.3.0",
"@types/react-dom": "^18.3.0",
"autoprefixer": "10.4.17",
"postcss": "8.4.35",
"postcss": "8.5.10",
"react": "^18.3.0",
"react-dom": "^18.3.0",
"tailwindcss": "3.4.1"
Expand Down
Loading
Loading