From a653e8adda9e5b71b7cca6242e0427590666eb0c Mon Sep 17 00:00:00 2001
From: spotikhanov <spotikhanov@yandex.ru>
Date: Wed, 8 Jan 2025 00:21:18 +0300
Subject: [PATCH] Remove ENABLE_MATH option from pulldown_cmark to fix links
 which contain dollar sign (#22647)

This pr closes #21466 issue by disabling math in pulldown_cmark Parser.

The dollar sign symbol is used in pulldown_cmark Math extension, see
"Math in links" section for more details:
https://pulldown-cmark.github.io/pulldown-cmark/specs/math.html

I've tried another approach at first, without disabling math extension:

```
let iterator = TextMergeWithOffset::new(Parser::new_ext(text, options));
```

instead of current implementation

```
Parser::new_ext(text, options).into_offset_iter()
```

This way pulldown_cmark merges consecutive text events and this helps to
correctly parse links from plain text:
https://svelte.dev/docs/svelte/$state

But in this case the dollar sign still breaks markdown links:
\[https://svelte.dev/docs/svelte/$state](https://svelte.dev/docs/svelte/$state)

So in the end I disabled the math extension, it fixes both link formats.
See markdown/examples/markdown.rs to reproduce.

Release Notes:
- N/A
---
 crates/markdown/examples/markdown.rs | 6 ++++++
 crates/markdown/src/parser.rs        | 1 +
 2 files changed, 7 insertions(+)

diff --git a/crates/markdown/examples/markdown.rs b/crates/markdown/examples/markdown.rs
index 30cbc64736bd400c9c5716abe1e928bbb0432a38..643a54b54e12e0f25e8f2789ce557623d6eaddef 100644
--- a/crates/markdown/examples/markdown.rs
+++ b/crates/markdown/examples/markdown.rs
@@ -51,6 +51,12 @@ Links are created using the format [http://zed.dev](https://zed.dev).
 
 They can also be detected automatically, for example https://zed.dev/blog.
 
+They may contain dollar signs:
+
+[https://svelte.dev/docs/svelte/$state](https://svelte.dev/docs/svelte/$state)
+
+https://svelte.dev/docs/svelte/$state
+
 ## Images
 Images are like links, but with an exclamation mark `!` in front.
 
diff --git a/crates/markdown/src/parser.rs b/crates/markdown/src/parser.rs
index d21892b7ded83a956484e6ed169cc0e3bc7f330c..b605a35af5ee6e8e8b3a4582b216f7eac90ff82b 100644
--- a/crates/markdown/src/parser.rs
+++ b/crates/markdown/src/parser.rs
@@ -8,6 +8,7 @@ pub fn parse_markdown(text: &str) -> Vec<(Range<usize>, MarkdownEvent)> {
     let mut options = Options::all();
     options.remove(pulldown_cmark::Options::ENABLE_DEFINITION_LIST);
     options.remove(pulldown_cmark::Options::ENABLE_YAML_STYLE_METADATA_BLOCKS);
+    options.remove(pulldown_cmark::Options::ENABLE_MATH);
 
     let mut events = Vec::new();
     let mut within_link = false;