[btea]

The inline branch of vite:worker (?worker&inline, ?sharedworker&inline) embeds the worker entry as a blob URL, but assets the worker references are still emitted as separate files. Those files normally reach the output through generateBundle, which an HMR patch skips. When the worker is re-bundled for a patch, the referenced asset gets a new content-hash filename that was never emitted, so requests for it fall through to the SPA fallback and return text/html.

Emit the cached worker assets during load in the inline branch as well, matching the non-inline branch from #23068.

[sapphi-red]

Instead of adding tests to hmr-full-bundle-mode playground, is it possible to add tests to worker playground? Maybe we need to fix other issues in the playground first though. If that turns out to be hard, we can merge this PR as-is first as well.

[sapphi-red]

cc @h-a-n-a

[h-a-n-a]

I don't feel like there's anything that blocks us from adding this to the worker playground.

Maybe we need to fix other issues in the playground first though.

What're the issues you were referring to here?

[sapphi-red]

the worker playground is now disabled, so I assumed there are many tests failing

vite/vitest.config.e2e.ts

Lines 29 to 32 in 02cffa9

	'./playground/worker/__tests__/es/worker-es.spec.ts',
	'./playground/worker/__tests__/iife/worker-iife.spec.ts',
	'./playground/worker/__tests__/relative-base-iife/worker-relative-base-iife.spec.ts',
	'./playground/worker/__tests__/relative-base/worker-relative-base.spec.ts',

[h-a-n-a]

the worker playground is now disabled, so I assumed there are many tests failing

vite/vitest.config.e2e.ts

Lines 29 to 32 in 02cffa9

	'./playground/worker/__tests__/es/worker-es.spec.ts',
	'./playground/worker/__tests__/iife/worker-iife.spec.ts',
	'./playground/worker/__tests__/relative-base-iife/worker-relative-base-iife.spec.ts',
	'./playground/worker/__tests__/relative-base/worker-relative-base.spec.ts',

I see. I haven't checked these yet. Maybe we should try to enable the test and see if anything fails and use isBundledDev to skip them. But at least we can add the current test there.

[h-a-n-a]

I tried to enable worker bundled-dev test cases. Unlike other enabled test cases in Vite playground. These cases were not triaged. But I feel like we can remove it from the exclude list first and gradually enable them. #23289 cc @sapphi-red

[sapphi-red]

@btea Would you see if you can enable some tests that is skipped in #23289 after this PR?

[btea]

I've verified this locally, and it seems I can't enable any of the test cases skipped in #23289.