Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix playground AST views w/ clone #2315

Merged
merged 1 commit into from
Jun 26, 2023
Merged

Fix playground AST views w/ clone #2315

merged 1 commit into from
Jun 26, 2023

Conversation

Jokcy
Copy link
Contributor

@Jokcy Jokcy commented Jun 26, 2023

const capture = (name) => () => (tree) => {
        file.data[name] = tree
      }

In our playground page, we use capture in remarkPlugins and rehypePlugins to capture the ast on different step. But after generate the mdast or hast, the evaluate function will keep going and later plugins will do some mutation on the ast object, which result to what we showed to user of mdast/hast in not correct on that step. So I just structuredClone the ast as a snapshot when capture and then save to file.

When compile <A b={<B><C /></B>}></A>, this mr generate <C /> as JSXElement, the old version generate as CallExpression with callee of _jsx.

@vercel
Copy link

vercel bot commented Jun 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mdx ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 4:33am

@codecov-commenter
Copy link

Codecov Report

Patch and project coverage have no change.

Comparison is base (7504cfb) 100.00% compared to head (94026da) 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #2315   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines         2177      2177           
=========================================
  Hits          2177      2177           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wooorm wooorm changed the title clone ast to prevent later process Fix playground AST views w/ clone Jun 26, 2023
@wooorm wooorm merged commit dbe9f44 into mdx-js:main Jun 26, 2023
@wooorm
Copy link
Member

wooorm commented Jun 26, 2023

Thank you! :)

@wooorm wooorm added 💪 phase/solved Post is done 🕸 area/website This affects the website labels Jun 26, 2023
kodiakhq bot referenced this pull request in X-oss-byte/Nextjs Oct 24, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@mdx-js/loader](https://mdxjs.com) ([source](https://github.com/mdx-js/mdx)) | [`2.3.0` -> `3.0.0`](https://renovatebot.com/diffs/npm/@mdx-js%2floader/2.3.0/3.0.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@mdx-js%2floader/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@mdx-js%2floader/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@mdx-js%2floader/2.3.0/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@mdx-js%2floader/2.3.0/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |
| [@mdx-js/react](https://mdxjs.com) ([source](https://github.com/mdx-js/mdx)) | [`2.3.0` -> `3.0.0`](https://renovatebot.com/diffs/npm/@mdx-js%2freact/2.3.0/3.0.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@mdx-js%2freact/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@mdx-js%2freact/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@mdx-js%2freact/2.3.0/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@mdx-js%2freact/2.3.0/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>mdx-js/mdx (@&#8203;mdx-js/loader)</summary>

### [`v3.0.0`](https://github.com/mdx-js/mdx/releases/tag/3.0.0)

[Compare Source](https://github.com/mdx-js/mdx/compare/2.3.0...3.0.0)

(see <https://mdxjs.com/migrating/v3/> on how to migrate)

##### Change

-   [`e08b759`](https://github.com/mdx-js/mdx/commit/e08b7596) [`5afa48e`](https://github.com/mdx-js/mdx/commit/5afa48e6) Change to require Node 16
-   [`5a13d73`](https://github.com/mdx-js/mdx/commit/5a13d73b) Change to use export maps
-   [`cbc2822`](https://github.com/mdx-js/mdx/commit/cbc2822f) Update `unified`, types, plugins, etc
-   [`96b51f9`](https://github.com/mdx-js/mdx/commit/96b51f93) Remove inferral of development from `NODE_ENV`

##### Change (unlikely to affect you)

-   [`c961af8`](https://github.com/mdx-js/mdx/commit/c961af80) Remove `useDynamicImport` option
-   [`9cb26fd`](https://github.com/mdx-js/mdx/commit/9cb26fd1) `@mdx-js/register`: remove package
-   [`0d1558a`](https://github.com/mdx-js/mdx/commit/0d1558a3) `@mdx-js/esbuild`: remove experimental `allowDangerousRemoteMdx`
-   [`0f62bce`](https://github.com/mdx-js/mdx/commit/0f62bce9) `@mdx-js/node-loader`: remove `fixRuntimeWithoutExportMap`
-   [`4f92422`](https://github.com/mdx-js/mdx/commit/4f924227) `@mdx-js/preact`: remove deprecated `MDXContext`, `withMDXComponents`
-   [`a362bb4`](https://github.com/mdx-js/mdx/commit/a362bb43) `@mdx-js/react`: remove deprecated `MDXContext`, `withMDXComponents`

##### Add

-   [`e12f307`](https://github.com/mdx-js/mdx/commit/e12f3079) Add support for passing `baseUrl` when running
-   [`2c511a4`](https://github.com/mdx-js/mdx/commit/2c511a40) Add support for `baseUrl` as a `URL`
-   [`1863914`](https://github.com/mdx-js/mdx/commit/1863914c) Add deprecation warning for classic runtime
-   [`a34177c`](https://github.com/mdx-js/mdx/commit/a34177c3) Add support for ES2024 in MDX, adjacent JSX and expression blocks
-   [`44fd9ca`](https://github.com/mdx-js/mdx/commit/44fd9cac) Add support for `await` in MDX
-   [`3a7f194`](https://github.com/mdx-js/mdx/commit/3a7f1947) Add `tableCellAlignToStyle` option, to use `align`
-   [`fdfe17b`](https://github.com/mdx-js/mdx/commit/fdfe17b8) `@mdx-js/rollup`: add support for Vite development mode
    by [@&#8203;remcohaszing](https://github.com/remcohaszing) in [https://github.com/mdx-js/mdx/pull/2376](https://github.com/mdx-js/mdx/pull/2376)

##### Misc

-   [`f48d038`](https://github.com/mdx-js/mdx/commit/f48d038b) Remove unneeded pragma comment after transform
-   [`8f3b292`](https://github.com/mdx-js/mdx/commit/8f3b2920) Add a `use strict` directive to function bodies
-   [`172e519`](https://github.com/mdx-js/mdx/commit/172e5190) `@mdx-js/react`: fix to classify `@types/react` as a peer dependency
    by [@&#8203;remcohaszing](https://github.com/remcohaszing) in [https://github.com/mdx-js/mdx/pull/2281](https://github.com/mdx-js/mdx/pull/2281)
-   [`a7bd79b`](https://github.com/mdx-js/mdx/commit/a7bd79bb) Refactor output to immediately export default
-   [`e525db9`](https://github.com/mdx-js/mdx/commit/e525db9b) [`dae82ae`](https://github.com/mdx-js/mdx/commit/dae82ae4) Refactor some errors
-   [`ce173f2`](https://github.com/mdx-js/mdx/commit/ce173f28) Refactor to add types for JSX runtimes
-   [`8a56312`](https://github.com/mdx-js/mdx/commit/8a563128) Refactor output to use spread, not `Object.assign`
    by [@&#8203;remcohaszing](https://github.com/remcohaszing) in [https://github.com/mdx-js/mdx/pull/2328](https://github.com/mdx-js/mdx/pull/2328)
-   [`825717f`](https://github.com/mdx-js/mdx/commit/825717fd) Refactor to sort default components
    by [@&#8203;remcohaszing](https://github.com/remcohaszing) in [https://github.com/mdx-js/mdx/pull/2318](https://github.com/mdx-js/mdx/pull/2318)
-   [`d8a62d2`](https://github.com/mdx-js/mdx/commit/d8a62d20) Add missing type dependencies
    by [@&#8203;arcanis](https://github.com/arcanis) in [https://github.com/mdx-js/mdx/pull/2256](https://github.com/mdx-js/mdx/pull/2256)

##### Docs

-   [`a9f0c04`](https://github.com/mdx-js/mdx/commit/a9f0c046) Add guide on injecting components
-   [`24e3d8d`](https://github.com/mdx-js/mdx/commit/24e3d8d1) Add compat sections to readmes
-   [`30e4a5d`](https://github.com/mdx-js/mdx/commit/30e4a5d5) Add sponsor
-   [`07503a5`](https://github.com/mdx-js/mdx/commit/07503a5f) Update link to KaTeX CSS in docs
    by [@&#8203;victor23k](https://github.com/victor23k) in [https://github.com/mdx-js/mdx/pull/2360](https://github.com/mdx-js/mdx/pull/2360)
-   [`74aee56`](https://github.com/mdx-js/mdx/commit/74aee569) [`bc1d9e5`](https://github.com/mdx-js/mdx/commit/bc1d9e56) [`765310c`](https://github.com/mdx-js/mdx/commit/765310ca) [`6d1e64d`](https://github.com/mdx-js/mdx/commit/6d1e64d9) Refactor docs
-   [`7fd1d9a`](https://github.com/mdx-js/mdx/commit/7fd1d9a4) Fix docs on how to use solid
    by [@&#8203;BeiyanYunyi](https://github.com/BeiyanYunyi) in [https://github.com/mdx-js/mdx/pull/2300](https://github.com/mdx-js/mdx/pull/2300)
-   [`4129f90`](https://github.com/mdx-js/mdx/commit/4129f90e) Fix a couple typos
    by [@&#8203;deining](https://github.com/deining) in [https://github.com/mdx-js/mdx/pull/2266](https://github.com/mdx-js/mdx/pull/2266)
-   [`bb902f8`](https://github.com/mdx-js/mdx/commit/bb902f83) Fix typo
    by [@&#8203;ChristianMurphy](https://github.com/ChristianMurphy) in [https://github.com/mdx-js/mdx/pull/2380](https://github.com/mdx-js/mdx/pull/2380)

##### Site

-   [`78a1eb5`](https://github.com/mdx-js/mdx/commit/78a1eb52) Add v3 blog post
-   [`2b1948c`](https://github.com/mdx-js/mdx/commit/2b1948c0) Add v3 migration guide
-   [`d6bb70c`](https://github.com/mdx-js/mdx/commit/d6bb70ca) Add improved error display in playground
-   [`89097e4`](https://github.com/mdx-js/mdx/commit/89097e4c) Remove unmaintained dev-dependency
-   [`3e23ba9`](https://github.com/mdx-js/mdx/commit/3e23ba90) Add more options to playground
-   [`d92128b`](https://github.com/mdx-js/mdx/commit/d92128ba) Fix links in docs
-   [`ab3aa96`](https://github.com/mdx-js/mdx/commit/ab3aa966) Add GitHub pages
    by [@&#8203;remcohaszing](https://github.com/remcohaszing) in [https://github.com/mdx-js/mdx/pull/2377](https://github.com/mdx-js/mdx/pull/2377)
-   [`a2c8693`](https://github.com/mdx-js/mdx/commit/a2c86936) Fix site
    by [@&#8203;wooorm](https://github.com/wooorm) in [https://github.com/mdx-js/mdx/pull/2358](https://github.com/mdx-js/mdx/pull/2358)
-   [`dbe9f44`](https://github.com/mdx-js/mdx/commit/dbe9f44b) Fix playground AST views w/ clone
    by [@&#8203;Jokcy](https://github.com/Jokcy) in [https://github.com/mdx-js/mdx/pull/2315](https://github.com/mdx-js/mdx/pull/2315)
-   [`7504cfb`](https://github.com/mdx-js/mdx/commit/7504cfb8) Add more options to playground, directives, format, etc
    by [@&#8203;slorber](https://github.com/slorber) in [https://github.com/mdx-js/mdx/pull/2295](https://github.com/mdx-js/mdx/pull/2295)
-   [`57301df`](https://github.com/mdx-js/mdx/commit/57301dfa) Add resizable editor/layout to playground
    by [@&#8203;slorber](https://github.com/slorber) in [https://github.com/mdx-js/mdx/pull/2296](https://github.com/mdx-js/mdx/pull/2296)
-   [`9eb747d`](https://github.com/mdx-js/mdx/commit/9eb747d6) Add info on how to build site locally
    by [@&#8203;slorber](https://github.com/slorber) in [https://github.com/mdx-js/mdx/pull/2297](https://github.com/mdx-js/mdx/pull/2297)

**Full Changelog**: mdx-js/mdx@2.3.0...3.0.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/X-oss-byte/Nextjs).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕸 area/website This affects the website 💪 phase/solved Post is done
Development

Successfully merging this pull request may close these issues.

3 participants