Provide ESM bundle

[marvin-j97]

Most popular packages have moved on to ESM-only or dual export packages.
Moo is still using the old-fashioned UMD format, being not-really-compatible with modern tooling.

[tjvr]

Hi, can you give an example of tooling that is not compatible with Moo today?

[nathan]

I imagine "not-really-compatible" here means "it works completely fine and my packaging script can't exclude compileStates" or "it works completely fine and my editor can't autocomplete the entry point names"

[mb21]

Yeah, without ESM bundle it cannot be tree-shaken... but looking at the source, I'm not sure how much there could be shaken away anyway..?

[johndeighan]

ESM would allow this, which is a style I use all the time:

import {compile} from 'moo';

[weotch]

Hi, can you give an example of tooling that is not compatible with Moo today?

I brought it into a Remix app via gqlmin and I am getting the following error:

TypeError: Cannot read properties of undefined (reading 'states')
    at Module.minify [as default] (/node_modules/gqlmin/dist/index.js:4:21)

When I add moo to the optimizeDeps of Vite, this issue goes away.

[salaun-marion]

When I add moo to the optimizeDeps of Vite, this issue goes away.

Can you explain in detail how you did it please ? Because when I add the code above to my vite.config.js file, I still have an error message in my console Uncaught TypeError: import_moo.default.states is not a function.

My vite.config.js :

export default defineConfig(({ mode }) => {
  const env = loadEnv(mode, process.cwd(), '')

  return {
     ...
    optimizeDeps: {
      include: ['moo'],
    },
    ...
}

[weotch]

Yeah, here's the relevant bit from my vite.config.ts:

export default defineConfig({
  ssr: {
    optimizeDeps: {
      include: [
        'moo', // Used by gqlmin
      ],
    },
  },
  optimizeDeps: {
    include: [
      'moo', // Used by gqlmin
    ],
  }
});

This is for a Remix / Hydrogen app

[salaun-marion]

Yeah, here's the relevant bit from my vite.config.ts:
...
This is for a Remix / Hydrogen app

Thank you for your answer but it did not work for me, I still got the same error. At the end, I have just changed the parent library, citation-js from where moo was a dependency to @citation-js/core where moo is no longer a dependency.

Edit :

• at the end I have just realised that I have totally forgot to add "type":"module" to package.json during my Vite migration (see more here : Import moo failed citation-js/citation-js#244 )