Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update swc_core to v0.89.x #61086

Merged
merged 10 commits into from
Jan 27, 2024
Merged

Update swc_core to v0.89.x #61086

merged 10 commits into from
Jan 27, 2024

Conversation

kdy1
Copy link
Member

@kdy1 kdy1 commented Jan 24, 2024

Turbopack

What?

Update SWC crates

Why?

To fix some issues

How?

Fixes #59805
Fixes #56144

Closes PACK-2222
Closes PACK-2285

@kdy1 kdy1 self-assigned this Jan 24, 2024
@ijjk ijjk added Turbopack Related to Turbopack with Next.js. created-by: Turbopack team PRs by the turbopack team type: next labels Jan 24, 2024
@ijjk ijjk added the Font (next/font) Related to Next.js Font Optimization. label Jan 25, 2024
@ijjk
Copy link
Member

ijjk commented Jan 25, 2024

Stats from current PR

Default Build
General
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
buildDuration 11.7s 11.8s N/A
buildDurationCached 6.2s 4.9s N/A
nodeModulesSize 200 MB 200 MB
nextStartRea..uration (ms) 427ms 435ms N/A
Client Bundles (main, webpack)
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
3f784ff6-HASH.js gzip 53.4 kB 53.4 kB
423.HASH.js gzip 185 B 181 B N/A
68-HASH.js gzip 29.8 kB 29.8 kB N/A
framework-HASH.js gzip 45.2 kB 45.2 kB
main-app-HASH.js gzip 238 B 240 B N/A
main-HASH.js gzip 31.8 kB 31.8 kB N/A
webpack-HASH.js gzip 1.7 kB 1.7 kB
Overall change 100 kB 100 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
polyfills-HASH.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
_app-HASH.js gzip 194 B 195 B N/A
_error-HASH.js gzip 182 B 181 B N/A
amp-HASH.js gzip 502 B 502 B
css-HASH.js gzip 320 B 322 B N/A
dynamic-HASH.js gzip 2.5 kB 2.5 kB N/A
edge-ssr-HASH.js gzip 255 B 256 B N/A
head-HASH.js gzip 350 B 349 B N/A
hooks-HASH.js gzip 368 B 369 B N/A
image-HASH.js gzip 4.18 kB 4.18 kB N/A
index-HASH.js gzip 257 B 256 B N/A
link-HASH.js gzip 2.61 kB 2.61 kB N/A
routerDirect..HASH.js gzip 310 B 311 B N/A
script-HASH.js gzip 384 B 383 B N/A
withRouter-HASH.js gzip 306 B 308 B N/A
1afbb74e6ecf..834.css gzip 106 B 106 B
Overall change 608 B 608 B
Client Build Manifests
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
_buildManifest.js gzip 484 B 484 B
Overall change 484 B 484 B
Rendered Page Sizes
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
index.html gzip 528 B 527 B N/A
link.html gzip 541 B 540 B N/A
withRouter.html gzip 523 B 522 B N/A
Overall change 0 B 0 B
Edge SSR bundle Size
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
edge-ssr.js gzip 94 kB 94 kB N/A
page.js gzip 149 kB 149 kB N/A
Overall change 0 B 0 B
Middleware size
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
middleware-b..fest.js gzip 626 B 622 B N/A
middleware-r..fest.js gzip 151 B 149 B N/A
middleware.js gzip 37.6 kB 37.6 kB N/A
edge-runtime..pack.js gzip 1.92 kB 1.92 kB
Overall change 1.92 kB 1.92 kB
Next Runtimes
vercel/next.js canary vercel/next.js kdy1/swc-core-89-1 Change
app-page-exp...dev.js gzip 170 kB 170 kB
app-page-exp..prod.js gzip 95.7 kB 95.7 kB
app-page-tur..prod.js gzip 96.4 kB 96.4 kB
app-page-tur..prod.js gzip 90.9 kB 90.9 kB
app-page.run...dev.js gzip 142 kB 142 kB
app-page.run..prod.js gzip 90.3 kB 90.3 kB
app-route-ex...dev.js gzip 22.2 kB 22.2 kB
app-route-ex..prod.js gzip 14.9 kB 14.9 kB
app-route-tu..prod.js gzip 14.9 kB 14.9 kB
app-route-tu..prod.js gzip 14.5 kB 14.5 kB
app-route.ru...dev.js gzip 21.6 kB 21.6 kB
app-route.ru..prod.js gzip 14.5 kB 14.5 kB
pages-api-tu..prod.js gzip 9.42 kB 9.42 kB
pages-api.ru...dev.js gzip 9.69 kB 9.69 kB
pages-api.ru..prod.js gzip 9.42 kB 9.42 kB
pages-turbo...prod.js gzip 22 kB 22 kB
pages.runtim...dev.js gzip 22.6 kB 22.6 kB
pages.runtim..prod.js gzip 22 kB 22 kB
server.runti..prod.js gzip 49.7 kB 49.7 kB
Overall change 933 kB 933 kB
Commit: bc4c696

@ijjk
Copy link
Member

ijjk commented Jan 25, 2024

Tests Passed

@kdy1 kdy1 force-pushed the kdy1/swc-core-89-1 branch 2 times, most recently from 029f69e to 458ee0c Compare January 26, 2024 02:59
@kdy1 kdy1 mentioned this pull request Jan 26, 2024
@kdy1
Copy link
Member Author

kdy1 commented Jan 26, 2024

@kdy1 kdy1 force-pushed the kdy1/swc-core-89-1 branch 3 times, most recently from 790d0a7 to 4eaa68e Compare January 26, 2024 09:59
@kdy1
Copy link
Member Author

kdy1 commented Jan 27, 2024

swc-project/swc#8530 was the cause of the regression

@@ -642,7 +642,10 @@ pub fn server_components<C: Comments>(
as_folder(ReactServerComponents {
is_react_server_layer,
comments,
filepath: filename.to_string(),
filepath: match filename {
FileName::Custom(path) => format!("<{}>", path),
Copy link
Member Author

Choose a reason for hiding this comment

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

This is to partially revert swc-project/swc#8530

kdy1 added a commit to vercel/turbo that referenced this pull request Jan 27, 2024
### Description

Update SWC crates.

### Testing Instructions

 - next.js counterpart: vercel/next.js#61086

Closes PACK-2286
@kdy1 kdy1 marked this pull request as ready for review January 27, 2024 09:26
@kdy1 kdy1 enabled auto-merge (squash) January 27, 2024 09:26
@kdy1 kdy1 merged commit 7b9d90b into canary Jan 27, 2024
66 of 68 checks passed
@kdy1 kdy1 deleted the kdy1/swc-core-89-1 branch January 27, 2024 09:43
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Turbopack team PRs by the turbopack team Font (next/font) Related to Next.js Font Optimization. locked Turbopack Related to Turbopack with Next.js. type: next
Projects
None yet
3 participants