Conversation
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Docker builds report
|
Storybook's webpack5 builder still needs raw-loader which was removed during the rspack migration. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
talissoncosta
left a comment
There was a problem hiding this comment.
Nice migration — the build speed improvement is massive. One thought for a future follow-up: now that the main build is on rspack, we could migrate Storybook to @storybook/builder-vite to drop webpack from the dependency tree entirely. Both builders are available for Storybook 10.
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Migrates webpack to rspack, a Rust based build system by ByteDance.
Improves local developer experience and build time from roughly 36 seconds to 4.5-5s
Note: It seems we will still need a lot of the original webpack plugins due to storybook / chromatic
How did you test this code?
Mainly builds (local + vercel) and then also E2E running on docker images