Skip to content

Turbopack: Simplify parent directory creation retry loop logic#95835

Merged
bgw merged 1 commit into
canaryfrom
bgw/simplify-parent-directory-creation
Jul 20, 2026
Merged

Turbopack: Simplify parent directory creation retry loop logic#95835
bgw merged 1 commit into
canaryfrom
bgw/simplify-parent-directory-creation

Conversation

@bgw

@bgw bgw commented Jul 15, 2026

Copy link
Copy Markdown
Member

Noticed while reading through the symlink code.

The symlink logic had some clever ErrorKind::AlreadyExists logic that folded the retry into the main retry loop, but we still had a separate retry loop for after handling ErrorKind::NotFound. This folds the loops together.

I hand-implemented the symlink version of this and had opus port it to the non-symlink version.

@bgw
bgw requested a review from lukesandberg July 15, 2026 21:22
@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Stats from current PR

🔴 1 regression

Metric Canary PR Change Trend
Turbo Build Time 5.238s 5.727s 🔴 +489ms (+9%) █▃▁▁▁
📊 All Metrics
📖 Metrics Glossary

Dev Server Metrics:

  • Listen = TCP port starts accepting connections
  • First Request = HTTP server returns successful response
  • Cold = Fresh build (no cache)
  • Warm = With cached build artifacts

Build Metrics:

  • Fresh = Clean build (no .next directory)
  • Cached = With existing .next directory

Change Thresholds:

  • Time: Changes < 50ms AND < 10%, OR < 2% are insignificant
  • Size: Changes < 1KB AND < 1% are insignificant
  • All other changes are flagged to catch regressions

⚡ Dev Server

Metric Canary PR Change Trend
Cold (Listen) 814ms 812ms █▂▂▁▂
Cold (Ready in log) 807ms 778ms █▂▁▁▁
Cold (First Request) 1.376s 1.287s █▃▁▁▁
Warm (Listen) 814ms 812ms ▇▂▂▁▂
Warm (Ready in log) 806ms 792ms █▂▁▁▁
Warm (First Request) 1.378s 1.329s █▃▁▁▁
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 812ms 812ms █▂▁▁▂
Cold (Ready in log) 783ms 783ms █▂▁▁▂
Cold (First Request) 3.296s 3.370s █▂▁▁▂
Warm (Listen) 812ms 813ms █▂▁▁▂
Warm (Ready in log) 777ms 777ms █▂▁▁▂
Warm (First Request) 3.347s 3.362s █▁▁▁▂

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 5.238s 5.727s 🔴 +489ms (+9%) █▃▁▁▁
Cached Build 5.635s 5.598s █▄▁▁▁
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 25.298s 25.017s █▁▁▁▂
Cached Build 25.295s 25.413s █▁▁▁▂
node_modules Size 517 MB 517 MB ▁▁▁▁▁
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles
Canary PR Change
0273j2-i6qkw4.js gzip 162 B N/A -
02frxk4kfgzv_.js gzip 155 B N/A -
03gmyxr7dukvs.js gzip 10.3 kB N/A -
040t1q17lo593.js gzip 8.77 kB N/A -
05m2nb2_o-jlv.js gzip 155 B N/A -
0bb0vtp8dwsez.js gzip 450 B N/A -
0cz1d0mv5g_q7.js gzip 39.4 kB 39.4 kB
0kqq9k7djlyvz.js gzip 157 B N/A -
0l0sr6gau5xjn.js gzip 3.53 kB N/A -
0nxqz3q165m2l.js gzip 220 B N/A -
0on5bfg96uqxk.js gzip 65.6 kB N/A -
0p1r-feajb5jb.js gzip 71 kB N/A -
0qr-1tnb7_hg5.js gzip 5.72 kB N/A -
0uilzvklme4ea.js gzip 8.76 kB N/A -
0xiw1unpygi1c.js gzip 157 B N/A -
0xyuy39dq2pzd.js gzip 156 B N/A -
0z0y83mfx-l-h.js gzip 10 kB N/A -
14ff853ktwa9b.js gzip 154 B N/A -
15ok8ydf274z0.js gzip 10.6 kB N/A -
16cqqcigykfea.js gzip 13.6 kB N/A -
17h18zxt1915l.js gzip 1.46 kB N/A -
1bs04l2ed4r7a.js gzip 153 B N/A -
1bxtbnlhloeph.js gzip 169 B N/A -
1elt1qium-r2m.css gzip 115 B 115 B
1fq1h7dxad4cv.js gzip 9.44 kB N/A -
1fym-adbk_69_.js gzip 13.1 kB N/A -
1gav7nfh4y8ln.js gzip 8.76 kB N/A -
1o34vh44ndvtu.js gzip 13.2 kB N/A -
1r4p-e_k9ckke.js gzip 157 B N/A -
27kmxbcd9zzhi.js gzip 160 B N/A -
2lpy00b303z50.js gzip 158 B N/A -
2rypqffktcxqh.js gzip 8.69 kB N/A -
2uqsizxtehgqf.js gzip 8.79 kB N/A -
2vzktdqwbijqc.js gzip 8.74 kB N/A -
3bhf_ndbc3iz2.js gzip 158 B N/A -
3c244akvx_-be.js gzip 8.74 kB N/A -
3dmlemitx3ef3.js gzip 2.29 kB N/A -
3lk7xgw97hbb3.js gzip 7.4 kB N/A -
3vfl1fxa4me-g.js gzip 45.2 kB N/A -
4237vdj6eyslt.js gzip 8.7 kB N/A -
turbopack-01..-1_6.js gzip 3.8 kB N/A -
turbopack-0j..iuww.js gzip 3.8 kB N/A -
turbopack-13..am56.js gzip 3.8 kB N/A -
turbopack-13..4w7t.js gzip 3.82 kB N/A -
turbopack-15..1saa.js gzip 3.81 kB N/A -
turbopack-1f..n2-o.js gzip 3.79 kB N/A -
turbopack-24..w6pd.js gzip 3.8 kB N/A -
turbopack-2l..18oa.js gzip 3.8 kB N/A -
turbopack-2p..qm-4.js gzip 3.8 kB N/A -
turbopack-2x..9yw0.js gzip 3.8 kB N/A -
turbopack-36..z3of.js gzip 3.81 kB N/A -
turbopack-3f..htxq.js gzip 3.8 kB N/A -
turbopack-3p..qki7.js gzip 3.8 kB N/A -
turbopack-43..nmng.js gzip 3.8 kB N/A -
00tc16j5-eos9.js gzip N/A 1.46 kB -
02jud9pe218fc.js gzip N/A 7.4 kB -
0gaw1gmpwuwkn.js gzip N/A 8.8 kB -
1-oaf1uo7f5x3.js gzip N/A 154 B -
12zyjjmq4oad1.js gzip N/A 5.72 kB -
18jvbdyomjd0w.js gzip N/A 151 B -
19eeqn1mjv0vt.js gzip N/A 156 B -
1dl-0_vny_nsm.js gzip N/A 155 B -
1gkxn3kvy0eri.js gzip N/A 154 B -
1j_7iw52-48fa.js gzip N/A 65.6 kB -
1k9cc4-gpa2cx.js gzip N/A 13.1 kB -
1kfiwrcr725kl.js gzip N/A 13.2 kB -
1n59uogejo81s.js gzip N/A 10.3 kB -
1npei3hlexdv3.js gzip N/A 9.99 kB -
1ssuiwj_rnpc5.js gzip N/A 8.77 kB -
1twnfn519w0_u.js gzip N/A 155 B -
2-phc9obd3nyd.js gzip N/A 8.77 kB -
24utq57r3l1cw.js gzip N/A 8.7 kB -
2fhjq6ek3srmr.js gzip N/A 8.69 kB -
2g0t7upz_v-hf.js gzip N/A 10.6 kB -
2h5nkoal80pda.js gzip N/A 45.2 kB -
2ia9qqy99aie9.js gzip N/A 13.6 kB -
2oza28kdqvoay.js gzip N/A 8.74 kB -
2t3-1eadj7woi.js gzip N/A 160 B -
2ui0dsrierofb.js gzip N/A 154 B -
2zh8-mmhujy8o.js gzip N/A 2.29 kB -
30h_m0irj-q6p.js gzip N/A 221 B -
32k2cm_v3snl5.js gzip N/A 154 B -
32s5d72o9mhg7.js gzip N/A 154 B -
33awzvk7o3i9x.js gzip N/A 71 kB -
37ejbfx1u9lpo.js gzip N/A 8.75 kB -
38c63ct4i1cy7.js gzip N/A 449 B -
3ggk1lzl49p05.js gzip N/A 165 B -
3jimpal3lvz5e.js gzip N/A 9.44 kB -
3sparlp-b3c_j.js gzip N/A 161 B -
3su6a4l8jbfth.js gzip N/A 3.53 kB -
3wb4pe4z4-aim.js gzip N/A 8.77 kB -
3zzrhjpzxeze1.js gzip N/A 153 B -
turbopack-02..vehg.js gzip N/A 3.79 kB -
turbopack-03..6f4n.js gzip N/A 3.8 kB -
turbopack-09..fbap.js gzip N/A 3.8 kB -
turbopack-0b..8zgv.js gzip N/A 3.8 kB -
turbopack-0b..9_57.js gzip N/A 3.8 kB -
turbopack-0j..7d6e.js gzip N/A 3.8 kB -
turbopack-1_..76lm.js gzip N/A 3.8 kB -
turbopack-13..4f4z.js gzip N/A 3.8 kB -
turbopack-1i..z722.js gzip N/A 3.8 kB -
turbopack-1i..flax.js gzip N/A 3.8 kB -
turbopack-1z..enel.js gzip N/A 3.81 kB -
turbopack-2i..h7js.js gzip N/A 3.8 kB -
turbopack-2r..kuk6.js gzip N/A 3.8 kB -
turbopack-3_..dvpc.js gzip N/A 3.81 kB -
Total 448 kB 448 kB ⚠️ +25 B

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 785 B 774 B 🟢 11 B (-1%)
Total 785 B 774 B ✅ -11 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 433 B 433 B
Total 433 B 433 B

📦 Webpack

Client

Main Bundles
Canary PR Change
2486.HASH.js gzip 169 B N/A -
3146-HASH.js gzip 64.2 kB N/A -
39fcf99b-HASH.js gzip 62.8 kB N/A -
8443-HASH.js gzip 4.68 kB N/A -
9431-HASH.js gzip 5.62 kB N/A -
framework-HASH.js gzip 59.8 kB 59.8 kB
main-app-HASH.js gzip 255 B 251 B 🟢 4 B (-2%)
main-HASH.js gzip 39.7 kB 40.1 kB 🔴 +452 B (+1%)
webpack-HASH.js gzip 1.68 kB 1.68 kB
6105-HASH.js gzip N/A 5.63 kB -
764.HASH.js gzip N/A 169 B -
8898-HASH.js gzip N/A 63.6 kB -
9597-HASH.js gzip N/A 4.65 kB -
e1ccab69-HASH.js gzip N/A 62.8 kB -
Total 239 kB 239 kB ✅ -211 B
Polyfills
Canary PR Change
polyfills-HASH.js gzip 39.4 kB 39.4 kB
Total 39.4 kB 39.4 kB
Pages
Canary PR Change
_app-HASH.js gzip 194 B 194 B
_error-HASH.js gzip 183 B 182 B
css-HASH.js gzip 335 B 335 B
dynamic-HASH.js gzip 1.8 kB 1.8 kB
edge-ssr-HASH.js gzip 255 B 254 B
head-HASH.js gzip 351 B 349 B
hooks-HASH.js gzip 384 B 384 B
image-HASH.js gzip 580 B 580 B
index-HASH.js gzip 259 B 259 B
link-HASH.js gzip 2.49 kB 2.49 kB
routerDirect..HASH.js gzip 319 B 319 B
script-HASH.js gzip 386 B 386 B
withRouter-HASH.js gzip 315 B 313 B
1afbb74e6ecf..834.css gzip 106 B 106 B
Total 7.96 kB 7.95 kB ✅ -7 B

Server

Edge SSR
Canary PR Change
edge-ssr.js gzip 128 kB 128 kB
page.js gzip 283 kB 285 kB 🔴 +2.13 kB (+1%)
Total 411 kB 413 kB ⚠️ +1.81 kB
Middleware
Canary PR Change
middleware-b..fest.js gzip 616 B 618 B
middleware-r..fest.js gzip 156 B 155 B
middleware.js gzip 45 kB 45.3 kB
edge-runtime..pack.js gzip 842 B 842 B
Total 46.6 kB 46.9 kB ⚠️ +285 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 719 B 720 B
Total 719 B 720 B ⚠️ +1 B
Build Cache
Canary PR Change
0.pack gzip 4.82 MB 4.81 MB 🟢 6.19 kB (0%)
index.pack gzip 118 kB 118 kB
index.pack.old gzip 119 kB 119 kB
Total 5.05 MB 5.05 MB ✅ -6.27 kB

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 364 kB 364 kB
app-page-exp..prod.js gzip 201 kB 201 kB
app-page-tur...dev.js gzip 364 kB 364 kB
app-page-tur..prod.js gzip 201 kB 201 kB
app-page-tur...dev.js gzip 360 kB 360 kB
app-page-tur..prod.js gzip 199 kB 199 kB
app-page.run...dev.js gzip 361 kB 361 kB
app-page.run..prod.js gzip 199 kB 199 kB
app-route-ex...dev.js gzip 81.5 kB 81.5 kB
app-route-ex..prod.js gzip 55.4 kB 55.4 kB
app-route-tu...dev.js gzip 81.5 kB 81.5 kB
app-route-tu..prod.js gzip 55.4 kB 55.4 kB
app-route-tu...dev.js gzip 81.1 kB 81.1 kB
app-route-tu..prod.js gzip 55.1 kB 55.1 kB
app-route.ru...dev.js gzip 81.1 kB 81.1 kB
app-route.ru..prod.js gzip 55.1 kB 55.1 kB
dist_client_...dev.js gzip 324 B 324 B
dist_client_...dev.js gzip 326 B 326 B
dist_client_...dev.js gzip 318 B 318 B
dist_client_...dev.js gzip 317 B 317 B
pages-api-tu...dev.js gzip 45.2 kB 45.2 kB
pages-api-tu..prod.js gzip 33.9 kB 33.9 kB
pages-api.ru...dev.js gzip 45.2 kB 45.2 kB
pages-api.ru..prod.js gzip 33.9 kB 33.9 kB
pages-turbo....dev.js gzip 54.6 kB 54.6 kB
pages-turbo...prod.js gzip 39.6 kB 39.6 kB
pages.runtim...dev.js gzip 54.6 kB 54.6 kB
pages.runtim..prod.js gzip 39.5 kB 39.5 kB
server.runti..prod.js gzip 67.5 kB 67.5 kB
use-cache-pr...dev.js gzip 71.4 kB 71.4 kB
use-cache-pr...dev.js gzip 71.4 kB 71.4 kB
use-cache-pr...dev.js gzip 69.7 kB 69.7 kB
use-cache-pr...dev.js gzip 69.7 kB 69.7 kB
Total 3.49 MB 3.49 MB
📎 Tarball URL
https://vercel-packages.vercel.app/next/commits/fd2acfc2891e1166b79e9f72a651d3be569e29f8/next

Commit: fd2acfc

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Tests Passed

Commit: fd2acfc

Comment thread turbopack/crates/turbo-tasks-fs/src/retry.rs
@bgw
bgw force-pushed the bgw/simplify-parent-directory-creation branch from 5a0e4d9 to 8e50050 Compare July 15, 2026 21:38
@bgw
bgw marked this pull request as ready for review July 15, 2026 21:38
@bgw
bgw force-pushed the bgw/simplify-parent-directory-creation branch from 8e50050 to fd2acfc Compare July 20, 2026 07:25
@bgw
bgw merged commit 5fe8c69 into canary Jul 20, 2026
128 checks passed
@bgw
bgw deleted the bgw/simplify-parent-directory-creation branch July 20, 2026 21:10
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.

2 participants