Skip to content

Add Failing tests for empty shells with dynamic params#95872

Open
gnoff wants to merge 2 commits into
canaryfrom
jstory/fix-allow-query-empty-shell
Open

Add Failing tests for empty shells with dynamic params#95872
gnoff wants to merge 2 commits into
canaryfrom
jstory/fix-allow-query-empty-shell

Conversation

@gnoff

@gnoff gnoff commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

params that are always dynamic (no generateStaticParam) in Cache Components should never be prerendered. This commit adds tests that pass in self hosted but fail in Vercel adapter and vercel's legacy mode.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Stats from current PR

Warning

No stats were collected for Webpack because its stats job did not complete (it failed, was cancelled, or timed out). The results below only cover the bundlers that finished.

✅ No significant changes detected

📊 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) 812ms 811ms █▁▂█▂
Cold (Ready in log) 778ms 768ms █▁▁█▂
Cold (First Request) 1.282s 1.270s ▇▁▁█▃
Warm (Listen) 812ms 811ms █▁▂▇▂
Warm (Ready in log) 769ms 775ms █▁▁█▂
Warm (First Request) 1.285s 1.286s █▁▁█▃

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 5.136s 4.970s █▁▁█▃
Cached Build 5.127s 5.057s █▁▁█▄
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
node_modules Size 517 MB 517 MB █████
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles
Canary PR Change
01e_imjacf07z.js gzip 154 B N/A -
03gmyxr7dukvs.js gzip 10.3 kB N/A -
040t1q17lo593.js gzip 8.77 kB N/A -
0bb0vtp8dwsez.js gzip 450 B N/A -
0cz1d0mv5g_q7.js gzip 39.4 kB 39.4 kB
0eit-_nonzj2s.js gzip 156 B N/A -
0l0sr6gau5xjn.js gzip 3.53 kB N/A -
0nxqz3q165m2l.js gzip 220 B N/A -
0qr-1tnb7_hg5.js gzip 5.72 kB N/A -
0uilzvklme4ea.js gzip 8.76 kB N/A -
0z0y83mfx-l-h.js gzip 10 kB N/A -
1_-g_r52k540c.js gzip 7.4 kB N/A -
1_an7sn6d05rs.js gzip 71 kB N/A -
15igz6de8tbl1.js gzip 156 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 -
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 -
1nd7cq5in04rl.js gzip 156 B N/A -
1o34vh44ndvtu.js gzip 13.2 kB N/A -
222aicw6zfjee.js gzip 168 B N/A -
22adlxvtko43-.js gzip 156 B N/A -
27pui1--s83v1.js gzip 161 B N/A -
2hmvy2eezqhve.js gzip 155 B N/A -
2rtsd8k_9_3wp.js gzip 65.6 kB 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 -
3-l1ry5jvjih5.js gzip 156 B N/A -
34ymlfb51t7yk.js gzip 160 B N/A -
3c244akvx_-be.js gzip 8.74 kB N/A -
3dmlemitx3ef3.js gzip 2.29 kB N/A -
3jiffnngz-na-.js gzip 153 B N/A -
3ju4nfc0yn2eu.js gzip 153 B N/A -
3qoudnhxxelar.js gzip 157 B N/A -
3vfl1fxa4me-g.js gzip 45.2 kB N/A -
4237vdj6eyslt.js gzip 8.7 kB N/A -
turbopack-0m..7odi.js gzip 3.8 kB N/A -
turbopack-0x..hrn0.js gzip 3.8 kB N/A -
turbopack-1b..x1e0.js gzip 3.8 kB N/A -
turbopack-1i..vmrl.js gzip 3.82 kB N/A -
turbopack-1y..b-7-.js gzip 3.8 kB N/A -
turbopack-2d..17x1.js gzip 3.8 kB N/A -
turbopack-2y..dlny.js gzip 3.78 kB N/A -
turbopack-2z..eq0n.js gzip 3.8 kB N/A -
turbopack-37..nh-g.js gzip 3.8 kB N/A -
turbopack-37..om2h.js gzip 3.8 kB N/A -
turbopack-39..z6f_.js gzip 3.8 kB N/A -
turbopack-3c..pabh.js gzip 3.8 kB N/A -
turbopack-3g..ijvr.js gzip 3.8 kB N/A -
turbopack-3h..yd91.js gzip 3.8 kB N/A -
0030hp-a_j9-y.js gzip N/A 156 B -
00tc16j5-eos9.js gzip N/A 1.46 kB -
02m7iodv32z8g.js gzip N/A 154 B -
0d7pewi7wgo4-.js gzip N/A 155 B -
0eruqpjj0h_ao.js gzip N/A 161 B -
0gaw1gmpwuwkn.js gzip N/A 8.8 kB -
0mejki6x1gy5u.js gzip N/A 160 B -
0pcox5e50zag8.js gzip N/A 65.6 kB -
1-mrm3ivjbndn.js gzip N/A 157 B -
12gwyaf8lv86s.js gzip N/A 157 B -
12zyjjmq4oad1.js gzip N/A 5.72 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 -
1zp2ipjglr15r.js gzip N/A 166 B -
2-phc9obd3nyd.js gzip N/A 8.77 kB -
24gok_u2-8uee.js gzip N/A 153 B -
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 -
2k7havh6955t0.js gzip N/A 154 B -
2n5fp99a5xt9s.js gzip N/A 7.4 kB -
2oza28kdqvoay.js gzip N/A 8.74 kB -
2zh8-mmhujy8o.js gzip N/A 2.29 kB -
30h_m0irj-q6p.js gzip N/A 221 B -
37ejbfx1u9lpo.js gzip N/A 8.75 kB -
38c63ct4i1cy7.js gzip N/A 449 B -
3ge7xyod8yvys.js gzip N/A 154 B -
3jimpal3lvz5e.js gzip N/A 9.44 kB -
3psggucft0se9.js gzip N/A 71 kB -
3su6a4l8jbfth.js gzip N/A 3.53 kB -
3vn00s3x0zs4q.js gzip N/A 155 B -
3wb4pe4z4-aim.js gzip N/A 8.77 kB -
40ll_plfwk8s3.js gzip N/A 155 B -
turbopack-0r..9osd.js gzip N/A 3.8 kB -
turbopack-0t..5bzs.js gzip N/A 3.8 kB -
turbopack-0x..20qr.js gzip N/A 3.8 kB -
turbopack-10..78dl.js gzip N/A 3.81 kB -
turbopack-10..nxvf.js gzip N/A 3.8 kB -
turbopack-13..0wb8.js gzip N/A 3.78 kB -
turbopack-1a..ccvc.js gzip N/A 3.8 kB -
turbopack-1j..e8j8.js gzip N/A 3.8 kB -
turbopack-21..mb7z.js gzip N/A 3.8 kB -
turbopack-2g..jzua.js gzip N/A 3.8 kB -
turbopack-2k..c_qw.js gzip N/A 3.8 kB -
turbopack-3c..alqz.js gzip N/A 3.8 kB -
turbopack-3x..vw7k.js gzip N/A 3.8 kB -
turbopack-40..rxjx.js gzip N/A 3.8 kB -
Total 448 kB 448 kB ⚠️ +39 B

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 780 B 768 B 🟢 12 B (-2%)
Total 780 B 768 B ✅ -12 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 432 B 434 B
Total 432 B 434 B ⚠️ +2 B

🔄 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 ⚠️ +2 B
📎 Tarball URL
https://vercel-packages.vercel.app/next/commits/13bd38d095592d3259f410bbc76a6a9c12fcd545/next

Commit: 13bd38d

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Failing test suites

Commit: 13bd38d | About building and testing Next.js

pnpm test-deploy test/e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts (job)

  • cache-components-prerender-matrix > fully-static-param/with-root-param/non-empty-shell > serves the whole document from the cache at lang=en category=fresh id=fresh (DD)
  • cache-components-prerender-matrix > fully-static-param/with-root-param/non-empty-shell > serves the whole document from the cache at lang=en category=shoes id=fresh (DD)
  • cache-components-prerender-matrix > fully-static-param/without-root-param/non-empty-shell > serves the whole document from the cache at lang=fresh category=fresh id=fresh (DD)
  • cache-components-prerender-matrix > fully-static-param/without-root-param/non-empty-shell > serves the whole document from the cache at lang=en category=fresh id=fresh (DD)
  • cache-components-prerender-matrix > fully-static-param/without-root-param/non-empty-shell > serves the whole document from the cache at lang=en category=shoes id=fresh (DD)
  • cache-components-prerender-matrix > partial-static-param/with-root-param/non-empty-shell > does not share the entry at lang=en category=fresh id=fresh across category values (category included in the cache key) (DD)
Expand output

● cache-components-prerender-matrix › partial-static-param/with-root-param/non-empty-shell › does not share the entry at lang=en category=fresh id=fresh across category values (category included in the cache key)

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  632 |           ? 'PRERENDER'
  633 |           : 'MISS'
> 634 |         expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                              ^
  635 |           expectedStatus
  636 |         )
  637 |       }

  at toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:634:62)
  at retry (lib/next-test-utils.ts:867:14)
  at expectServedFromDifferentEntry (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:615:5)
  at Object.<anonymous> (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:724:13)

● cache-components-prerender-matrix › fully-static-param/without-root-param/non-empty-shell › serves the whole document from the cache at lang=fresh category=fresh id=fresh

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  688 |               ? 'PRERENDER'
  689 |               : 'MISS'
> 690 |             expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                                  ^
  691 |               expectedStatus
  692 |             )
  693 |           }

  at Object.toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:690:66)

● cache-components-prerender-matrix › fully-static-param/without-root-param/non-empty-shell › serves the whole document from the cache at lang=en category=fresh id=fresh

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  688 |               ? 'PRERENDER'
  689 |               : 'MISS'
> 690 |             expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                                  ^
  691 |               expectedStatus
  692 |             )
  693 |           }

  at Object.toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:690:66)

● cache-components-prerender-matrix › fully-static-param/without-root-param/non-empty-shell › serves the whole document from the cache at lang=en category=shoes id=fresh

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  688 |               ? 'PRERENDER'
  689 |               : 'MISS'
> 690 |             expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                                  ^
  691 |               expectedStatus
  692 |             )
  693 |           }

  at Object.toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:690:66)

● cache-components-prerender-matrix › fully-static-param/with-root-param/non-empty-shell › serves the whole document from the cache at lang=en category=fresh id=fresh

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  688 |               ? 'PRERENDER'
  689 |               : 'MISS'
> 690 |             expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                                  ^
  691 |               expectedStatus
  692 |             )
  693 |           }

  at Object.toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:690:66)

● cache-components-prerender-matrix › fully-static-param/with-root-param/non-empty-shell › serves the whole document from the cache at lang=en category=shoes id=fresh

expect(received).toBe(expected) // Object.is equality

Expected: "PRERENDER"
Received: "REVALIDATED"

  688 |               ? 'PRERENDER'
  689 |               : 'MISS'
> 690 |             expect(first.response.headers.get('x-vercel-cache')).toBe(
      |                                                                  ^
  691 |               expectedStatus
  692 |             )
  693 |           }

  at Object.toBe (e2e/app-dir/cache-components-prerender-matrix/cache-components-prerender-matrix.test.ts:690:66)

@gnoff
gnoff force-pushed the jstory/fix-allow-query-empty-shell branch 2 times, most recently from 55eb582 to 1328eb6 Compare July 17, 2026 20:57
Comment thread .claude/launch.json Outdated
@gnoff
gnoff force-pushed the jstory/fix-allow-query-empty-shell branch 2 times, most recently from 4c47681 to ab8fb05 Compare July 17, 2026 21:12
gnoff added 2 commits July 17, 2026 15:23
params that are always dynamic (no generateStaticParam) in Cache Components should never be prerendered. This commit adds tests that pass in self hosted but fail in Vercel adapter and vercel's legacy mode.
@gnoff
gnoff force-pushed the jstory/fix-allow-query-empty-shell branch from ab8fb05 to 13bd38d Compare July 17, 2026 22:23
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