Create separate CJS and UMD builds

[benmccann]

Marked version: 4.0.1

Markdown flavor: n/a

Description

Fixes #2265

This comment pointed out that it would be very weird to use a CJS file in the browser: #2265 (comment)

That's true, but it's not actually CJS. It's UMD. And I think UMD was specified for the CDNs, which makes a lot more sense. It's just really confusing the way we have it setup now because the file extension doesn't match the actual file type.

Putting a UMD file as main is breaking people in 4.0.1. I didn't realize we were doing that in the last PR because I was thrown off by the file extensions. Bundlers generally know how to deal with CJS and ESM, but not UMD. It's reported that this change fixes things: #2265 (comment)

Contributor

• Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR); or,
• no tests required for this PR.
• If submitting new feature, it has been documented in the appropriate places.

Committer

In most cases, this should be a different person than the contributor.

• CI is green (no forced merge required).
• Squash and Merge PR following conventional commit guidelines.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/markedjs/markedjs/Cvg8tgQHQVgfWDbN9XKNiiAVpWWH
✅ Preview: https://markedjs-git-fork-benmccann-browser-esm-markedjs.vercel.app

[UziTech]

@benmccann You have been helping out a lot recently. Would you like to join @markedjs and help maintain marked?

[benmccann]

Thanks for the offer, but I'll probably disappear once this conversion to ESM has settled down 😄 I've got a fair amount of open source projects on my plate already

[styfle]

Its an unfortunate state of the JS ecosystem but this change looks correct, thanks!

[github-actions]

🎉 This PR is included in version 4.0.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[splch]

Just to understand this correctly, if we're using marked on the client side, we now will be using lib/marked.umd.js?

[UziTech]

@splch if you are using marked in a script tag you should use /marked.min.js. if you use marked in node or webpack you should use import { marked } from 'marked'; and the loader should choose the correct file.

The cjs and umd files should be the exact same the only difference should be the file name which some loaders require to work.