Unterminated template when parsing template literals with multiple newlines (regression from v1)

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions

@mdx-js/mdx

Link to runnable example

https://stackblitz.com/edit/github-rumgbj?file=index.js

Steps to reproduce

Add an extra \n between .foo and .bar styles in the linked stackblitz example

Expected behavior

It should compile successfully. We are using this pattern in Storybook’s starter template:

https://raw.githubusercontent.com/storybookjs/storybook/next/lib/cli/src/frameworks/common/Introduction.stories.mdx

Actual behavior

It will crash with

Could not parse expression with acorn: Unterminated template

Or perhaps this is an intentional breaking change in MDXv2?

Runtime

No response

Package manager

No response

OS

No response

Build and bundle tools

No response

[shilman]

Hi @wooorm thanks so much for getting back to me.

I updated the example to make it a little more readable and realized it was not exactly what I wanted to convey (though the behavior is the same).

Original (broken)

<style>{`
  .foo {}

  .bar {}
`}</style>

The “original” version worked with MDXv1 and we shipped it to tens of thousands of users as part of the example template in Storybook Docs. Now in MDXv2 it’s broken. It feels like a regression to me, but if that’s the expected behavior I’m totally fine with it – I realize that’s what major versions are for.

Updated1 (works)

<style>{`
  .foo {}
  .bar {}
`}</style>

Updated2 (also works)

<style>
  {`
    .foo {}
  
    .bar {}
  `}
</style>

Discussion

Both updated versions work. I could update our template accordingly, and provide migration instructions or potentially even ship a codemod to update users’ code accordingly.

Before I do that, however, I want to confirm:

• That this original example should not work in MDXv2, i.e. it’s expected behavior and not a regression
• Do you have a preference between Updated1 and Updated2? (I’m guessing Updated2 is more idiomatic MDX, but not entirely sure)

Thanks again for your help! And congratulations on the v2 release–super excited to get this into users’ hands!!! 🎉

[wooorm]

@brendonco you’re probably better off asking questions about storybook betas in storybook repos