read task cell should only need Data #88339

sokra · 2026-01-09T17:52:50Z

No description provided.

sokra · 2026-01-09T17:53:13Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

nextjs-bot · 2026-01-09T17:53:43Z

Allow CI Workflow Run

approve CI run for commit: 736c5da

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

vercel

Additional Suggestion:

Code loads a task with TaskDataCategory::Data but then tries to access the InProgress field which belongs to TaskDataCategory::All, causing the field access to fail and silently miss the intended modification

vercel · 2026-01-09T18:16:13Z

turbopack/crates/turbo-tasks-backend/src/backend/mod.rs

+            let (task, reader) = ctx.task_pair(task_id, reader_id, TaskDataCategory::Data);
            (task, Some(reader))
        } else {
-            (ctx.task(task_id, TaskDataCategory::All), None)
+            (ctx.task(task_id, TaskDataCategory::Data), None)


Task is incorrectly re-scheduled when InProgress state is not loaded due to loading with Data instead of All category

codspeed-hq · 2026-01-12T09:31:33Z

Merging this PR will not alter performance

✅ 17 untouched benchmarks
⏩ 3 skipped benchmarks¹

_{Comparing sokra/read-cell-data (d2375a3) with sokra/task-prio (17d5b7b)}

3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

nextjs-bot · 2026-01-12T09:52:10Z

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 (Ready in log)	435ms	435ms	✓	▁█▂▇▇
Cold (First Request)	1.104s	1.157s	✓	▁█▂▇█
Warm (Listen)	457ms	457ms	✓	▃▆▃▆▃
Warm (Ready in log)	439ms	437ms	✓	▂▁█▁▁
Warm (First Request)	334ms	332ms	✓	▁▆██▁

📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric	Canary	PR	Change	Trend
Cold (Listen)	456ms	457ms	✓	▁████
Cold (Ready in log)	436ms	438ms	✓	▅▁▂█▅
Cold (First Request)	1.852s	1.861s	✓	▄▄▄█▄
Warm (Listen)	456ms	456ms	✓	▁▅▅▅▅
Warm (Ready in log)	437ms	435ms	✓	▃▇▁▅▅
Warm (First Request)	1.831s	1.831s	✓	▄▆▄█▅

⚡ Production Builds

Metric	Canary	PR	Change	Trend
Fresh Build	4.052s	4.004s	✓	▄▁█▁▁
Cached Build	4.067s	4.039s	✓	▄▁█▁▂

📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric	Canary	PR	Change	Trend
Fresh Build	13.868s	13.942s	✓	▁▂▂█▁
Cached Build	14.023s	14.005s	✓	▁▂▁█▁
node_modules Size	457 MB	457 MB	✓	█████

📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

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

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

Server

Middleware

	Canary	PR	Change
middleware-b..fest.js gzip	788 B	788 B	✓
Total	788 B	788 B	✓

Build Details

Build Manifests

	Canary	PR	Change
_buildManifest.js gzip	451 B	450 B	✓
Total	451 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.8 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.7 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	226 kB	⚠️ +244 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	240 kB	✓
Total	364 kB	364 kB	⚠️ +287 B

Middleware

	Canary	PR	Change
middleware-b..fest.js gzip	652 B	651 B	✓
middleware-r..fest.js gzip	155 B	156 B	✓
middleware.js gzip	33 kB	33.1 kB	✓
edge-runtime..pack.js gzip	842 B	842 B	✓
Total	34.6 kB	34.8 kB	⚠️ +146 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.62 MB	3.62 MB	✓
index.pack gzip	98.8 kB	98.3 kB	✓
index.pack.old gzip	98.9 kB	99.3 kB	✓
Total	3.82 MB	3.82 MB	⚠️ +2.85 kB

🔄 Shared (bundler-independent)

Runtimes

	Canary	PR	Change
app-page-exp...dev.js gzip	303 kB	303 kB	✓
app-page-exp..prod.js gzip	158 kB	158 kB	✓
app-page-tur...dev.js gzip	303 kB	303 kB	✓
app-page-tur..prod.js gzip	158 kB	158 kB	✓
app-page-tur...dev.js gzip	299 kB	299 kB	✓
app-page-tur..prod.js gzip	156 kB	156 kB	✓
app-page.run...dev.js gzip	300 kB	300 kB	✓
app-page.run..prod.js gzip	156 kB	156 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.68 MB	2.68 MB	⚠️ +3 B

nextjs-bot added created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js. labels Jan 9, 2026

This was referenced Jan 9, 2026

Turbopack: prefetch children too #88324

Merged

Turbopack: snapshot the ModuleGraph to avoid (eventual) inconsistent graphs #88340

Open

vercel bot reviewed Jan 9, 2026

View reviewed changes

read task cell should only need Data

d2375a3

sokra force-pushed the sokra/read-cell-data branch from 736c5da to d2375a3 Compare January 12, 2026 09:23

sokra force-pushed the sokra/task-prio branch from bba22c0 to 17d5b7b Compare January 12, 2026 09:23

This was referenced Jan 12, 2026

Consistency for compute_async_module_info #88409

Draft

Turbopack: follow tracing spans through aggregation update queue #88410

Draft

Turbopack: avoid chunk_item_id task and use selective cell access instead #88411

Draft

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

read task cell should only need Data #88339

read task cell should only need Data #88339

sokra commented Jan 9, 2026

Uh oh!

sokra commented Jan 9, 2026 •

edited

Loading

Uh oh!

nextjs-bot commented Jan 9, 2026

Uh oh!

vercel bot left a comment

Uh oh!

vercel bot Jan 9, 2026 •

edited

Loading

Uh oh!

codspeed-hq bot commented Jan 12, 2026 •

edited

Loading

Uh oh!

nextjs-bot commented Jan 12, 2026

⚡ Dev Server

📦 Dev Server (Webpack)

⚡ Production Builds

📦 Production Builds (Webpack)

Bundle Sizes

⚡ Turbopack

📦 Webpack

🔄 Shared (bundler-independent)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

read task cell should only need Data #88339

Are you sure you want to change the base?

read task cell should only need Data #88339

Conversation

sokra commented Jan 9, 2026

Uh oh!

sokra commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nextjs-bot commented Jan 9, 2026

Uh oh!

vercel bot left a comment

Choose a reason for hiding this comment

Uh oh!

vercel bot Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

codspeed-hq bot commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merging this PR will not alter performance

Footnotes

Uh oh!

nextjs-bot commented Jan 12, 2026

Stats from current PR

✅ No significant changes detected

⚡ Dev Server

📦 Dev Server (Webpack)

⚡ Production Builds

📦 Production Builds (Webpack)

Bundle Sizes

⚡ Turbopack

📦 Webpack

🔄 Shared (bundler-independent)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

sokra commented Jan 9, 2026 •

edited

Loading

vercel bot Jan 9, 2026 •

edited

Loading

codspeed-hq bot commented Jan 12, 2026 •

edited

Loading