Address No "exports" main defined in package json for vite

[moshie]

Summary

Package JSON is missing the main property

Fixes: #16751

Resolves:

Test plan

I applied this fix directly in my local node modules and this fixed the error

[moshie]

@RodEsp Good catch thanks, updated

[moshie]

@philipp-spiess Sorry to ping you but this bug is affecting us and was wondering if you wouldn't mind reviewing this PR for me please?

[thecrypticace]

Doesn't this imply that you have to use a version of node where require(esm) is enabled?

afaik the only reason this would fail is because require(…) is being used instead of import "" or await import(…) to load a file which feels weird in this case since Vite is basically all in on ESM and Vite's CJS build has been deprecated for quite a while. Afaik it's going away in Vite 7 too so I'm curious to know how/why this is happening / still a problem.

[moshie]

Yeah I get the feeling something is wrong in playwright's component testing which uses Vite under the hood to create the components bundle and serve it.

// playwright-ct.config.ts
import { default as tailwindcss } from "@tailwindcss/vite";
import { defineConfig } from "@playwright/experimental-ct-react";

export default defineConfig({
	use: {
		ctViteConfig: {
			plugins: [react(), tailwindcss()],
		},
	},
});

I can see from the error tract that they are using the CJS loader.

I'll try and make time to create a reproduction which I'll share in a new issue