Build Modernization

https://github.com/mdx-js/mdx/pull/1269 introduces microbundle to handle the preact package builds and enables node v14 native esm usage for mdx (that is, importing mdx from .mjs files). The usage should expand and result in the removal of @pkgr/rollup.

This issue should be closed after the work in #1269 is expanded to the other packages.

[ChristopherBiscardi]

I replaced pkgr with microbundle for vue and react already: e7e9d46. It’s no longer used.

great! seems like this would’ve been a useful issue to mention in that PR.

MDX is already built using rollup.

Not all of it is: mdx-js/mdx, mdx-js/loader , and mdx-js/vue-loader use CJS.

you are correct in stating that the packages that don’t get built, are also not built with rollup. I will be more clear about that next time.

I was wondering if this issue was also about changing them to MJS?

This issue is about removing pkgr. If pkgr is gone, we can close this issue.

[ChristopherBiscardi]

To clarify the core purpose of this issue: we are currently on an unsupported one-off build toolchain that isn’t used anywhere else afaict, was last released months ago, and has only 257 weekly downloads: https://www.npmjs.com/package/@pkgr/rollup. It is a project risk issue to depend on a niche build pipeline that we don’t control and isn’t maintained. The work was started to get rid of that toolchain and put us onto something more mainstream that is actively maintained (microbundle in this case) that is still based on rollup.

Your concerns about the handling of esm conversion in micromark are heard, but not required for this conversion as the source code is not changing and all existing exports are being maintained as-is. I’ll repeat that we are not changing the underlying build system like micromark did (from browserify to rollup). MDX is already built using rollup.