From 48245337914712e6320152d2defd8ee3db9bf987 Mon Sep 17 00:00:00 2001
From: Michael Chow <mc_al_github@fastmail.com>
Date: Fri, 11 Oct 2024 12:57:56 -0400
Subject: [PATCH] fix: description list no longer escapes quotes inside code

---
 quartodoc/renderers/base.py                   |  7 ++--
 quartodoc/renderers/md_renderer.py            |  4 +--
 .../tests/__snapshots__/test_renderers.ambr   | 32 +++++++++----------
 quartodoc/tests/test_renderers.py             |  2 +-
 4 files changed, 24 insertions(+), 21 deletions(-)

diff --git a/quartodoc/renderers/base.py b/quartodoc/renderers/base.py
index d8a8968..7dea074 100644
--- a/quartodoc/renderers/base.py
+++ b/quartodoc/renderers/base.py
@@ -13,12 +13,15 @@ def escape(val: str):
     return f"`{val}`"
 
 
-def sanitize(val: str, allow_markdown=False):
+def sanitize(val: str, allow_markdown=False, escape_quotes=False):
     # sanitize common tokens that break tables
     res = val.replace("\n", " ").replace("|", "\\|")
 
     # sanitize elements that get turned into smart quotes
-    res = res.replace("'", r"\'").replace('"', r"\"")
+    # this is to avoid defaults that are strings having their
+    # quotes screwed up.
+    if escape_quotes:
+        res = res.replace("'", r"\'").replace('"', r"\"")
 
     # sanitize elements that can get interpreted as markdown links
     # or citations
diff --git a/quartodoc/renderers/md_renderer.py b/quartodoc/renderers/md_renderer.py
index 1193a2b..51cbd87 100644
--- a/quartodoc/renderers/md_renderer.py
+++ b/quartodoc/renderers/md_renderer.py
@@ -60,7 +60,7 @@ def to_definition_list(self):
         name = self.name
         anno = self.annotation
         desc = sanitize(self.description, allow_markdown=True)
-        default = sanitize(str(self.default))
+        default = sanitize(str(self.default), escape_quotes=True)
 
         part_name = (
             Span(Strong(name), Attr(classes=["parameter-name"]))
@@ -219,7 +219,7 @@ def render_annotation(self, el: str) -> str:
         el:
             An object representing a type annotation.
         """
-        return sanitize(el)
+        return sanitize(el, escape_quotes=True)
 
     @dispatch
     def render_annotation(self, el: None) -> str:
diff --git a/quartodoc/tests/__snapshots__/test_renderers.ambr b/quartodoc/tests/__snapshots__/test_renderers.ambr
index d6de891..0d99ba9 100644
--- a/quartodoc/tests/__snapshots__/test_renderers.ambr
+++ b/quartodoc/tests/__snapshots__/test_renderers.ambr
@@ -504,60 +504,60 @@
       :   A description.
   '''
 # ---
-# name: test_render_numpydoc_section_return[name: int\n    A description.]
+# name: test_render_numpydoc_section_return[name: int\n    A `"description"`.]
   '''
   Code
       Parameters
       ---
       name: int
-          A description.
+          A `"description"`.
   
       Returns
       ---
       name: int
-          A description.
+          A `"description"`.
   
       Attributes
       ---
       name: int
-          A description.
+          A `"description"`.
   
   Default
       # Parameters {.doc-section .doc-section-parameters}
   
-      | Name   | Type   | Description    | Default    |
-      |--------|--------|----------------|------------|
-      | name   |        | A description. | _required_ |
+      | Name   | Type   | Description        | Default    |
+      |--------|--------|--------------------|------------|
+      | name   |        | A `"description"`. | _required_ |
   
       # Returns {.doc-section .doc-section-returns}
   
-      | Name   | Type   | Description    |
-      |--------|--------|----------------|
-      | name   | int    | A description. |
+      | Name   | Type   | Description        |
+      |--------|--------|--------------------|
+      | name   | int    | A `"description"`. |
   
       # Attributes {.doc-section .doc-section-attributes}
   
-      | Name   | Type   | Description    |
-      |--------|--------|----------------|
-      | name   | int    | A description. |
+      | Name   | Type   | Description        |
+      |--------|--------|--------------------|
+      | name   | int    | A `"description"`. |
   
   List
       # Parameters {.doc-section .doc-section-parameters}
   
       <code>[**name**]{.parameter-name} [:]{.parameter-annotation-sep} []{.parameter-annotation}</code>
   
-      :   A description.
+      :   A `"description"`.
   
       # Returns {.doc-section .doc-section-returns}
   
       <code>[**name**]{.parameter-name} [:]{.parameter-annotation-sep} [int]{.parameter-annotation}</code>
   
-      :   A description.
+      :   A `"description"`.
   
       # Attributes {.doc-section .doc-section-attributes}
   
       <code>[**name**]{.parameter-name} [:]{.parameter-annotation-sep} [int]{.parameter-annotation}</code>
   
-      :   A description.
+      :   A `"description"`.
   '''
 # ---
diff --git a/quartodoc/tests/test_renderers.py b/quartodoc/tests/test_renderers.py
index 922318d..8117fcf 100644
--- a/quartodoc/tests/test_renderers.py
+++ b/quartodoc/tests/test_renderers.py
@@ -223,7 +223,7 @@ def test_render_doc_signature_name_alias_of_alias(snapshot, renderer):
 @pytest.mark.parametrize(
     "doc",
     [
-        """name: int\n    A description.""",
+        """name: int\n    A `"description"`.""",
         """int\n    A description.""",
     ],
 )