Skip to content

Replace local @spree/sdk with NPM package and switch to Turbopack - #1

Merged
damianlegawiec merged 1 commit into
mainfrom
damianlegawiec/npm-sdk-turbopack
Feb 6, 2026
Merged

Replace local @spree/sdk with NPM package and switch to Turbopack#1
damianlegawiec merged 1 commit into
mainfrom
damianlegawiec/npm-sdk-turbopack

Conversation

@damianlegawiec

@damianlegawiec damianlegawiec commented Feb 6, 2026

Copy link
Copy Markdown
Member
  • Change @spree/sdk from file:../spree/sdk to ^0.1.4 from NPM
  • Switch dev script from --webpack to --turbopack
  • Remove --webpack from build script (Turbopack is default)
  • Remove transpilePackages from next.config.ts
  • Fix AccountShellProps types to match NPM SDK User type

Summary by CodeRabbit

  • Performance Improvements

    • Build system updated to use faster bundling technology.
  • Bug Fixes

    • Account page now properly handles cases where user names are not available.
  • Chores

    • SDK dependency updated to a published version.

- Change @spree/sdk from file:../spree/sdk to ^0.1.4 from NPM
- Switch dev script from --webpack to --turbopack
- Remove --webpack from build script (Turbopack is default)
- Remove transpilePackages from next.config.ts
- Fix AccountShellProps types to match NPM SDK User type

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@damianlegawiec
damianlegawiec merged commit 9f66fce into main Feb 6, 2026
@coderabbitai

coderabbitai Bot commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

This PR removes webpack configuration from Next.js setup, transitions build scripts to use turbopack, updates the @spree/sdk dependency from a local file reference to an npm package version, and modifies the AccountShellProps type to allow user name fields to be nullable.

Changes

Cohort / File(s) Summary
Build Configuration
next.config.ts, package.json
Removed transpilePackages entry for @spree/sdk; updated dev script from webpack to turbopack and removed webpack flag from build script.
Dependencies
package.json
Updated @spree/sdk from local file reference (file:../spree/sdk) to npm version (^0.1.4).
Type Definitions
src/app/[country]/[locale]/(storefront)/account/layout.tsx
Modified AccountShellProps to allow first_name and last_name fields to be `string

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Webpack bids farewell, turbopack takes the lead,
From local to npm, @spree/sdk's freed,
Nullable names now dance, more flexible and light,
Configuration streamlined, everything feels right! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch damianlegawiec/npm-sdk-turbopack

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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