Skip to content

Conversation

@timneutkens
Copy link
Member

@timneutkens timneutkens commented Jan 9, 2026

What?

Adds the docs-only change check to the stats-aggregate job.

Why?

The stats job already skips work for docs-only changes using scripts/run-for-change.mjs, but the stats-aggregate job was missing this check. This caused unnecessary job runs when PRs only contain documentation changes.

How?

  • Added the "Check non-docs only change" step (same pattern as the stats job)
  • Added conditional if checks to all subsequent steps
  • Changed fetch-depth from 1 to 25 (required for the change detection script to check commit history)

@nextjs-bot nextjs-bot added the created-by: Turbopack team PRs by the Turbopack team. label Jan 9, 2026
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Jan 9, 2026

Tests Passed

Copy link
Contributor

@vercel vercel bot left a comment

Choose a reason for hiding this comment

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

Additional Suggestion:

When there are docs-only changes, the stats-aggregate job runs unnecessarily and attempts to download non-existent artifacts because it only checks step-level conditions instead of job-level conditions.

Fix on Vercel

@nextjs-bot
Copy link
Collaborator

Stats from current PR

✅ 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) 455ms 455ms ▁▁▁▁▁
Cold (First Request) 1.124s 1.085s ▁██▁█
Warm (Listen) 456ms 456ms ▁▁▁▁▁
Warm (First Request) 330ms 338ms ▅▁▁▄▆
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 456ms 455ms █▁▁▁█
Cold (First Request) 1.821s 1.855s █▅▄▅▁
Warm (Listen) 456ms 456ms ▅▅▁▅▅
Warm (First Request) 1.867s 1.844s █▅▄▅▁

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 3.999s 4.011s ▅▁▁▃▂
Cached Build 4.078s 4.060s ▃▁▂▂▁
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 13.957s 13.980s █▁▁▂▁
Cached Build 14.062s 13.987s █▁▁▂▂
node_modules Size 457 MB 457 MB ▇▇███
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles: **430 kB** → **430 kB** ✅ -4 B

82 files with content-based hashes (individual files not comparable between builds)

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 793 B 796 B
Total 793 B 796 B ⚠️ +3 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 449 B 450 B
Total 449 B 450 B ⚠️ +1 B

📦 Webpack

Client

Main Bundles
Canary PR Change
2086.HASH.js gzip 169 B N/A -
2161-HASH.js gzip 5.41 kB N/A -
2747-HASH.js gzip 4.48 kB N/A -
4322-HASH.js gzip 52.7 kB N/A -
ec793fe8-HASH.js gzip 62.3 kB N/A -
framework-HASH.js gzip 59.8 kB 59.8 kB
main-app-HASH.js gzip 251 B 254 B 🔴 +3 B (+1%)
main-HASH.js gzip 38.6 kB 39 kB
webpack-HASH.js gzip 1.68 kB 1.68 kB
1596.HASH.js gzip N/A 169 B -
2658-HASH.js gzip N/A 52.5 kB -
6349-HASH.js gzip N/A 4.46 kB -
7019-HASH.js gzip N/A 5.43 kB -
b17a3386-HASH.js gzip N/A 62.3 kB -
Total 225 kB 225 kB ⚠️ +157 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 193 B
_error-HASH.js gzip 182 B 182 B
css-HASH.js gzip 336 B 335 B
dynamic-HASH.js gzip 1.8 kB 1.8 kB
edge-ssr-HASH.js gzip 256 B 256 B
head-HASH.js gzip 352 B 349 B
hooks-HASH.js gzip 385 B 384 B
image-HASH.js gzip 580 B 580 B
index-HASH.js gzip 259 B 258 B
link-HASH.js gzip 2.5 kB 2.51 kB
routerDirect..HASH.js gzip 319 B 317 B
script-HASH.js gzip 385 B 387 B
withRouter-HASH.js gzip 316 B 315 B
1afbb74e6ecf..834.css gzip 106 B 106 B
Total 7.97 kB 7.96 kB ✅ -8 B

Server

Edge SSR
Canary PR Change
edge-ssr.js gzip 125 kB 125 kB
page.js gzip 239 kB 239 kB
Total 364 kB 364 kB ⚠️ +255 B
Middleware
Canary PR Change
middleware-b..fest.js gzip 651 B 651 B
middleware-r..fest.js gzip 155 B 156 B
middleware.js gzip 32.9 kB 33 kB
edge-runtime..pack.js gzip 842 B 842 B
Total 34.6 kB 34.7 kB ⚠️ +119 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 738 B 738 B
Total 738 B 738 B
Build Cache
Canary PR Change
0.pack gzip 3.6 MB 3.61 MB 🔴 +7.26 kB (+0%)
index.pack gzip 98.5 kB 98.3 kB
index.pack.old gzip 99.2 kB 98.3 kB
Total 3.8 MB 3.81 MB ⚠️ +6.25 kB

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 303 kB 303 kB
app-page-exp..prod.js gzip 157 kB 157 kB
app-page-tur...dev.js gzip 302 kB 302 kB
app-page-tur..prod.js gzip 157 kB 157 kB
app-page-tur...dev.js gzip 299 kB 299 kB
app-page-tur..prod.js gzip 155 kB 155 kB
app-page.run...dev.js gzip 299 kB 299 kB
app-page.run..prod.js gzip 155 kB 155 kB
app-route-ex...dev.js gzip 68.2 kB 68.2 kB
app-route-ex..prod.js gzip 46.9 kB 46.9 kB
app-route-tu...dev.js gzip 68.2 kB 68.2 kB
app-route-tu..prod.js gzip 46.9 kB 46.9 kB
app-route-tu...dev.js gzip 67.8 kB 67.8 kB
app-route-tu..prod.js gzip 46.7 kB 46.7 kB
app-route.ru...dev.js gzip 67.8 kB 67.8 kB
app-route.ru..prod.js gzip 46.7 kB 46.7 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 41.1 kB 41.1 kB
pages-api-tu..prod.js gzip 31.2 kB 31.2 kB
pages-api.ru...dev.js gzip 41.1 kB 41.1 kB
pages-api.ru..prod.js gzip 31.2 kB 31.2 kB
pages-turbo....dev.js gzip 50.8 kB 50.8 kB
pages-turbo...prod.js gzip 38.2 kB 38.2 kB
pages.runtim...dev.js gzip 50.8 kB 50.8 kB
pages.runtim..prod.js gzip 38.2 kB 38.2 kB
server.runti..prod.js gzip 62 kB 62 kB
Total 2.67 MB 2.67 MB ✅ -3 B

@timneutkens timneutkens marked this pull request as ready for review January 11, 2026 20:58
@timneutkens timneutkens merged commit 78c102f into canary Jan 11, 2026
291 of 293 checks passed
@timneutkens timneutkens deleted the fix-stats-aggregate-docs-check branch January 11, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

created-by: Turbopack team PRs by the Turbopack team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants