Fix dash conversion corrupting tables inside blockquotes

Dash conversion scans line by line, so a blockquoted table separator
arrives as "> | -- | -- |", which never matched the table separator
pattern (anchored at "|"). The row was treated as prose and smart-
punctuated into "| - | - |", breaking table rendering in Obsidian.
Fenced code blocks inside blockquotes had the same blind spot, since
fence tracking only stripped whitespace, and blockquoted thematic
breaks were missed because that check ran against the raw line.

Add _strip_block_markers() to remove leading blockquote markers and
indentation, and use it wherever block-level constructs are detected
in _convert_dash_sequences() and _strip_trailing_whitespace(). Prose
dashes inside blockquotes still convert.

Tests exercise _convert_dash_sequences() directly rather than through
mdformat.text(), so they do not pull in the gfm parser extension,
which conflicts with this plugin's list_item renderer.

Author: jdmonaco

README.md

@@ -165,7 +165,9 @@ The result—unexpected as it was—changed everything.
 Pages 10–20 of the report.
 ```
 
-Dashes are preserved inside fenced code blocks, inline code spans, HTML comments, and HTML tags. Thematic breaks (`---`) and frontmatter delimiters are not affected. Sequences of 4+ dashes are left unchanged.
+Dashes are preserved inside fenced code blocks, inline code spans, HTML comments, and HTML tags. Thematic breaks (`---`), GFM table separator rows (`| -- | -- |`), and frontmatter delimiters are not affected. Sequences of 4+ dashes are left unchanged.
+
+These exclusions apply equally inside blockquotes, at any nesting depth: leading `>` markers are stripped before block-level patterns are matched, so tables and code fences in a blockquote or Obsidian callout are preserved just as they are at the top level.
 
 ### Wikilink Preservation
 

mdformat_space_control/__init__.py

@@ -1,6 +1,6 @@
 """An mdformat plugin for space control: EditorConfig indentation, tight lists, frontmatter spacing, smart dash conversion, soft break joining, and wikilinks."""
 
-__version__ = "0.4.9"
+__version__ = "0.4.10"
 
 from .config import (
     get_current_file,

mdformat_space_control/plugin.py

@@ -425,6 +425,19 @@ def _normalize_frontmatter_spacing(text: str) -> str:
     return before_content + after_content
 
 
+# Leading blockquote markers ('>' with optional spaces), possibly nested and
+# possibly indented. Block-level constructs inside a blockquote -- code fences,
+# GFM table separator rows, thematic breaks -- are recognized only after these
+# markers are stripped, so line-scanning postprocessors must strip them first
+# or they mis-handle every blockquoted table and fenced block.
+_BLOCKQUOTE_PREFIX_RE = re.compile(r"^[ \t]*(?:>[ \t]?)+")
+
+
+def _strip_block_markers(line: str) -> str:
+    """Return ``line`` without leading blockquote markers and indentation."""
+    return _BLOCKQUOTE_PREFIX_RE.sub("", line).lstrip()
+
+
 def _strip_trailing_whitespace(text: str) -> str:
     """Strip trailing whitespace, preserving code blocks.
 
@@ -436,8 +449,8 @@ def _strip_trailing_whitespace(text: str) -> str:
     in_code_block = False
 
     for line in lines:
-        # Track fenced code block state
-        stripped = line.lstrip()
+        # Track fenced code block state (blockquoted fences included)
+        stripped = _strip_block_markers(line)
         if stripped.startswith("```") or stripped.startswith("~~~"):
             in_code_block = not in_code_block
             result.append(line.rstrip())  # Strip fence line itself
@@ -494,7 +507,9 @@ def _convert_dash_sequences(text: str) -> str:
     in_html_comment = False
 
     for line in lines:
-        stripped = line.lstrip()
+        # Strip blockquote markers so block constructs nested in a blockquote
+        # (fences, table separator rows, thematic breaks) are recognized.
+        stripped = _strip_block_markers(line)
         if stripped.startswith("```") or stripped.startswith("~~~"):
             in_code_block = not in_code_block
             result.append(line)
@@ -505,7 +520,8 @@ def _convert_dash_sequences(text: str) -> str:
             if "-->" in line:
                 in_html_comment = False
             result.append(line)
-        elif (only_dashes_re.match(line) or separator_line_re.match(stripped)
+        elif (only_dashes_re.match(stripped)
+              or separator_line_re.match(stripped)
               or table_sep_re.match(stripped)):
             result.append(line)
         else:

tests/test_dash_conversion.py

@@ -385,3 +385,91 @@ def test_dashes_outside_html_still_convert(self):
         expected = "word\u2013word and word\u2014word\n"
         result = mdformat.text(input_text, extensions={"space_control"})
         assert result == expected
+
+
+class TestDashConversionInBlockquotes:
+    """Block constructs nested in a blockquote must be recognized.
+
+    Dash conversion scans line by line, so it sees '> | -- | -- |', not
+    '| -- | -- |'. Blockquote markers must be stripped before block-level
+    patterns are matched; otherwise GFM table separator rows inside a
+    blockquote or Obsidian callout get smart-punctuated into '| - | - |',
+    which breaks table rendering in Obsidian.
+
+    These test _convert_dash_sequences directly rather than through
+    mdformat.text() so they do not depend on the gfm parser extension,
+    which is deliberately not a dependency of this plugin.
+    """
+
+    def test_blockquote_table_separator_preserved(self):
+        """A table separator row inside a blockquote must keep its dashes."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "> | A | B |\n> | -- | -- |\n> | 1 | 2 |"
+        assert _convert_dash_sequences(text) == text
+
+    def test_obsidian_callout_table_separator_preserved(self):
+        """Tables inside an Obsidian callout must survive conversion."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = (
+            "> [!note] Landscape\n>\n> | Model | Size |\n"
+            "> | -- | -- |\n> | Qwen3 | 27B |"
+        )
+        assert _convert_dash_sequences(text) == text
+
+    def test_nested_blockquote_table_separator_preserved(self):
+        """Separator rows survive at any blockquote nesting depth."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = ">> | A | B |\n>> | -- | -- |\n>> | 1 | 2 |"
+        assert _convert_dash_sequences(text) == text
+
+    def test_indented_blockquote_separator_preserved(self):
+        """Indented blockquote markers are stripped before matching."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "  > | A | B |\n  > | -- | -- |"
+        assert _convert_dash_sequences(text) == text
+
+    def test_blockquote_alignment_separator_preserved(self):
+        """Alignment colons in a blockquoted separator row are preserved."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "> | A | B |\n> | :-- | --: |\n> | 1 | 2 |"
+        assert _convert_dash_sequences(text) == text
+
+    def test_blockquote_code_fence_preserved(self):
+        """Dashes in a fenced code block inside a blockquote are preserved."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "> ```bash\n> mdfmt -- file.md\n> a -- b\n> ```"
+        assert _convert_dash_sequences(text) == text
+
+    def test_blockquote_thematic_break_preserved(self):
+        """A thematic break inside a blockquote is not converted."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        assert _convert_dash_sequences("> ---") == "> ---"
+
+    def test_blockquote_prose_dash_still_converts(self):
+        """Genuine prose dashes inside a blockquote still convert."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        result = _convert_dash_sequences("> quoted prose -- with a dash")
+        assert result == "> quoted prose \u2013 with a dash"
+
+    def test_blockquote_conversion_is_idempotent(self):
+        """Repeated conversion of a blockquoted table is a fixed point."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "> | A | B |\n> | -- | -- |\n> | 1 | 2 |"
+        once = _convert_dash_sequences(text)
+        assert _convert_dash_sequences(once) == once == text
+
+    def test_plain_table_separator_still_preserved(self):
+        """The non-blockquoted case keeps working."""
+        from mdformat_space_control.plugin import _convert_dash_sequences
+
+        text = "| A | B |\n| -- | -- |\n| 1 | 2 |"
+        assert _convert_dash_sequences(text) == text