Date libraries are not optional when using @salt-ds/date-adapters with Vite

[origami-z]

Latest version

• I have tested the latest version

Description

When using new lab v56 of date picker with @salt-ds/date-adapters, Vite will fail running local server while trying to resolve all date libraries

Steps to reproduce

Use Vite TS template, install Salt packages with date-fns only (one of 4 date packages), run npm run dev, error below

Plugin: vite:import-analysis
File: xxxxxx/node_modules/.vite/deps/chunk-DZFYDRDP.js?v=a34e13b6:32087:27
32085|
32086| // node_modules/@salt-ds/date-adapters/dist-es/dayjs/index.js
32087| import defaultDayjs from "dayjs";

---

Also tried below vite setting, without any luck

optimizeDeps: {
    exclude: ["dayjs", "luxon", "moment", "moment-timezone"],
  },

According to this vite issue, only way to make it work would be not importing it.

Expected behavior

Only required package should be imported, maybe split 4 adapters into different entry point would work?

Package name(s)

Lab (@salt-ds/lab)

Package version(s)

"@salt-ds/lab": "1.0.0-alpha.56"

Browser

• Chrome
• Safari
• Firefox
• Microsoft Edge

Operating system

• macOS
• Windows
• Linux
• iOS
• Android

Are you a JPMorgan Chase & Co. employee?

• I am an employee of JPMorgan Chase & Co.

[origami-z]

Looks like the build script needs to be updated to support multiple entries

salt-ds/scripts/build.mjs

Lines 36 to 37 in 3d87e17

	const bundle = await rollup({
	input: path.join(cwd, "src/index.ts"),

https://rollupjs.org/configuration-options/#input

exports may potential work, but it's not supported by typescript yet..? https://nodejs.org/api/packages.html#package-entry-points