refactor(turbopack): Only perform strongly consistent reads/resolves on VcOperation

[bgw]

This PR only allows strongly consistent reads to happen on VcOperation:

• Moves Vc::strongly_consistent to OperationVc::read_strongly_consistent.
• Moves Vc::resolve_strongly_consistent to OperationVc::resolve_strongly_consistent (and makes it return a ResolvedVc instead of a Vc).

Both these use OperationVc::connect() to connect before reading/resolving.

Why do this?

• For resolved Vcs, the task id associated with the Vc may be unintuitive, as it could be the task that constructs the cell, not the function you most recently called.

• For local Vcs, the task id is our own task. We can't strongly resolve ourselves.

So, the solution is to force strong resolution to occur on OperationVcs, where the task pointed to is as obvious as possible.

Unit Tests

There are still some unit tests depending on Vc::strongly_consistent/Vc::resolve_strongly_consistent. I'll fix those and completely remove these methods in a subsequent PR.

Closes PACK-3777

[bgw]

• refactor(turbopack): Only perform strongly consistent reads/resolves on VcOperation #75016 : 2 dependent PRs (#75051 , #75059 ) 👈 (View in Graphite)
• perf(turbo-tasks): Optionally schedule ResolveNative/ResolveTrait tasks as local tasks #69126
• refactor(turbopack/next-api): Make VcArc use OperationVc #74479
• fix(turbopack-swc-utils): Don't panic if issues aren't emitted, use #[must_use] annotation #74906
• canary

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

[ijjk]

Stats from current PR

Default Build (Increase detected ⚠️)

General

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
buildDuration	18.5s	15.5s	N/A
buildDurationCached	14.7s	12.3s	N/A
nodeModulesSize	418 MB	418 MB	N/A
nextStartRea..uration (ms)	397ms	397ms	✓

Client Bundles (main, webpack)

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
5306-HASH.js gzip	54 kB	54 kB	N/A
8276.HASH.js gzip	169 B	168 B	N/A
8377-HASH.js gzip	5.44 kB	5.44 kB	N/A
bccd1874-HASH.js gzip	52.9 kB	52.9 kB	✓
framework-HASH.js gzip	57.5 kB	57.5 kB	N/A
main-app-HASH.js gzip	240 B	242 B	N/A
main-HASH.js gzip	34.4 kB	34.4 kB	N/A
webpack-HASH.js gzip	1.71 kB	1.71 kB	N/A
Overall change	52.9 kB	52.9 kB	✓

Legacy Client Bundles (polyfills)

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
polyfills-HASH.js gzip	39.4 kB	39.4 kB	✓
Overall change	39.4 kB	39.4 kB	✓

Client Pages

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
_app-HASH.js gzip	193 B	193 B	✓
_error-HASH.js gzip	193 B	193 B	✓
amp-HASH.js gzip	512 B	510 B	N/A
css-HASH.js gzip	343 B	342 B	N/A
dynamic-HASH.js gzip	1.84 kB	1.84 kB	✓
edge-ssr-HASH.js gzip	265 B	265 B	✓
head-HASH.js gzip	363 B	362 B	N/A
hooks-HASH.js gzip	393 B	392 B	N/A
image-HASH.js gzip	4.57 kB	4.57 kB	N/A
index-HASH.js gzip	268 B	268 B	✓
link-HASH.js gzip	2.35 kB	2.34 kB	N/A
routerDirect..HASH.js gzip	328 B	328 B	✓
script-HASH.js gzip	397 B	397 B	✓
withRouter-HASH.js gzip	323 B	326 B	N/A
1afbb74e6ecf..834.css gzip	106 B	106 B	✓
Overall change	3.59 kB	3.59 kB	✓

Client Build Manifests

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
_buildManifest.js gzip	749 B	747 B	N/A
Overall change	0 B	0 B	✓

Rendered Page Sizes

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
index.html gzip	523 B	523 B	✓
link.html gzip	538 B	537 B	N/A
withRouter.html gzip	519 B	520 B	N/A
Overall change	523 B	523 B	✓

Edge SSR bundle Size

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
edge-ssr.js gzip	129 kB	129 kB	N/A
page.js gzip	208 kB	208 kB	N/A
Overall change	0 B	0 B	✓

Middleware size

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
middleware-b..fest.js gzip	671 B	666 B	N/A
middleware-r..fest.js gzip	155 B	156 B	N/A
middleware.js gzip	31.3 kB	31.3 kB	N/A
edge-runtime..pack.js gzip	844 B	844 B	✓
Overall change	844 B	844 B	✓

Next Runtimes

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
274-experime...dev.js gzip	322 B	322 B	✓
274.runtime.dev.js gzip	314 B	314 B	✓
app-page-exp...dev.js gzip	375 kB	375 kB	✓
app-page-exp..prod.js gzip	130 kB	130 kB	✓
app-page-tur..prod.js gzip	143 kB	143 kB	✓
app-page-tur..prod.js gzip	139 kB	139 kB	✓
app-page.run...dev.js gzip	363 kB	363 kB	✓
app-page.run..prod.js gzip	126 kB	126 kB	✓
app-route-ex...dev.js gzip	37.6 kB	37.6 kB	✓
app-route-ex..prod.js gzip	25.6 kB	25.6 kB	✓
app-route-tu..prod.js gzip	25.6 kB	25.6 kB	✓
app-route-tu..prod.js gzip	25.4 kB	25.4 kB	✓
app-route.ru...dev.js gzip	39.2 kB	39.2 kB	✓
app-route.ru..prod.js gzip	25.4 kB	25.4 kB	✓
pages-api-tu..prod.js gzip	9.69 kB	9.69 kB	✓
pages-api.ru...dev.js gzip	11.6 kB	11.6 kB	✓
pages-api.ru..prod.js gzip	9.68 kB	9.68 kB	✓
pages-turbo...prod.js gzip	21.9 kB	21.9 kB	✓
pages.runtim...dev.js gzip	27.7 kB	27.7 kB	✓
pages.runtim..prod.js gzip	21.9 kB	21.9 kB	✓
server.runti..prod.js gzip	916 kB	916 kB	✓
Overall change	2.47 MB	2.47 MB	✓

build cache Overall increase ⚠️

	vercel/next.js canary	vercel/next.js bgw/operation-strong-resolve	Change
0.pack gzip	2.1 MB	2.1 MB	⚠️ +2.01 kB
index.pack gzip	74.9 kB	75.1 kB	⚠️ +132 B
Overall change	2.17 MB	2.17 MB	⚠️ +2.14 kB

Diff details

Diff for main-HASH.js

Diff too large to display

Commit: 57af58e

[ijjk]

Tests Passed