chore(remix-dev): replace xdm with @mdx-js/mdx v2

[jacobwgillespie]

This PR addresses issue #1802 - xdm is now deprecated, and all functionality has been ported to @mdx-js/mdx.

Closes: #1802

• Docs
• Tests

Testing Strategy:

All existing tests pass with yarn test. Additionally, I created a playground and verified locally that mdx files still build as expected.

[changeset-bot]

🦋 Changeset detected

Latest commit: 43d47a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages

Name	Type
@remix-run/dev	Patch
create-remix	Patch
remix	Patch
@remix-run/architect	Patch
@remix-run/cloudflare	Patch
@remix-run/cloudflare-pages	Patch
@remix-run/cloudflare-workers	Patch
@remix-run/css-bundle	Patch
@remix-run/deno	Patch
@remix-run/eslint-config	Patch
@remix-run/express	Patch
@remix-run/node	Patch
@remix-run/react	Patch
@remix-run/serve	Patch
@remix-run/server-runtime	Patch
@remix-run/testing	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[remix-cla-bot]

Hi @jacobwgillespie,

Welcome, and thank you for contributing to Remix!

Before we consider your pull request, we ask that you sign our Contributor License Agreement (CLA). We require this only once.

You may review the CLA and sign it by adding your name to contributors.yml.

Once the CLA is signed, the CLA Signed label will be added to the pull request.

If you have already signed the CLA and received this response in error, or if you have any questions, please contact us at hello@remix.run.

Thanks!

- The Remix team

[remix-cla-bot]

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

[belgattitude]

Thanks, deasync is very problematic on my side and even more with vercel and node 18.

Would be very nice to include in the next release 🙏

[machour]

@jacobwgillespie could you please rebase this on latest dev and fix conflicts? I'll try to get eyes on this PR.

[jacobwgillespie]

@machour yep, rebased, thank you!

[MichaelDeBoey]

@jacobwgillespie I didn't look into it, but there's also @mdx-js/esbuild
https://mdxjs.com/packages/esbuild

Could this replace our own esbuild plugin or do we need to keep this custom one?

[jacobwgillespie]

Perhaps! I'm not super familiar with Remix's use of MDX internally, mostly just that xdm is deprecated and directly replaceable with @mdx-js/mdx.

[MichaelDeBoey]

Hi @jacobwgillespie!

Are you still interested in getting this one merged?

If so, please rebase onto latest dev & resolve conflicts

[jacobwgillespie]

@MichaelDeBoey Thanks for the ping, I have rebased onto the latest dev 👍

[jacobwgillespie]

I'm not sure why the tests are failing, it's upset about packages/remix-dev/modules.ts which I don't think is affected by this PR.

[jacobwgillespie]

Oh I see it, it's the new *.mdx types, let me take a look

[github-actions]

🤖 Hello there,

We just published version v0.0.0-nightly-b1149bb-20230810 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!