fix(manifest): include assets referenced in html

[ArnaudBarre]

Discovered by hasard while doing #14643 (required to make tests pass)

[sapphi-red]

Looks good to me 👍

Would you add a test here that ensures the asset only referenced in html is included in the manifest?

vite/playground/assets/__tests__/assets.spec.ts

Lines 413 to 424 in 96a4ce3

	test.runIf(isBuild)('manifest', async () => {
	const manifest = readManifest('foo')
	const entry = manifest['index.html']
	for (const file of listAssets('foo')) {
	if (file.endsWith('.css')) {
	expect(entry.css).toContain(`assets/${file}`)
	} else if (!file.endsWith('.js')) {
	expect(entry.assets).toContain(`assets/${file}`)
	}
	}
	})

[bluwy]

tiny nit: could we use an svg so it's smaller?

[ArnaudBarre]

My next PR will make SVG be inline.
But I can use a new SVG and reduce the inlineSize now, but this requires a new config test because the default one is already increase to 8kb (why I used ~12kb one)

[bluwy]

Ah ok if it's specifically to pass the 8kb limit, and there's no existing files to use I'm fine with it then. No need to change 👍

[ArnaudBarre]

@sapphi-red you're suggesting adding a new test with a html + asset file only and duplicate this test?
The linked test is actually failing with the playground change without the code

[sapphi-red]

My bad, I misunderstood the test.