Vite fixes (default exports, circular dependency)

[ryanclark]

This fixes an issue with Vite production builds where certain libraries that export things a bit differently don't work (vitejs/vite#2139).

To do this, we're using the ESModule interop (checking for .default) instead of the Vite config change as it doesn't work with node v16

Also this removes a circular dependency that Vite has been detecting in production builds

Export "SelectResource" of module "../../../web/packages/teleport/src/Discover/SelectResource/SelectResource.tsx" was reexported through module "../../../web/packages/teleport/src/Discover/SelectResource/index.ts" while both modules are dependencies of each other and will end up in different chunks by current Rollup settings. This scenario is not well supported at the moment as it will produce a circular dependency between chunks and will likely lead to broken execution order.
Either change the import in "../../../web/packages/teleport/src/Discover/Discover.tsx" to point directly to the exporting module or reconfigure "output.manualChunks" to ensure these modules end up in the same chunk.

[gzdunek]

as RollupCommonJSOptions is not needed, commonjsOptions is already typed properly.

[ryanclark]

Removed