DOP-2975: Correctly handle non-list elements in list context

Author: i80and

snooty/parser.py

@@ -205,7 +205,6 @@ def dispatch_visit(self, node: docutils.nodes.Node) -> None:
                             util.get_line(node.children[0]),
                         )
                     )
-                    raise docutils.nodes.SkipNode()
             else:
                 # Handle case where :field: does not appear in a directive
                 self.diagnostics.append(
@@ -214,6 +213,7 @@ def dispatch_visit(self, node: docutils.nodes.Node) -> None:
                         util.get_line(node.children[0]),
                     )
                 )
+            raise docutils.nodes.SkipNode()
         elif isinstance(node, docutils.nodes.field_name):
             raise docutils.nodes.SkipNode()
         elif isinstance(node, docutils.nodes.field_body):

snooty/test_parser.py

@@ -820,7 +820,7 @@ def test_codeblock() -> None:
         """
 .. code-block:: java
    :copyable: false
-   :source: 
+   :source:
 
    foo""",
     )
@@ -3228,3 +3228,23 @@ def test_directive_line_offset() -> None:
     page.finish(diagnostics)
     assert len(diagnostics) == 1
     assert diagnostics[0].start[0] == 13
+
+
+def test_field_list_in_list() -> None:
+    """Tests DOP-2975"""
+    path = ROOT_PATH.joinpath(Path("test.rst"))
+    project_config = ProjectConfig(ROOT_PATH, "")
+    parser = rstparser.Parser(project_config, JSONVisitor)
+
+    page, diagnostics = parse_rst(
+        parser,
+        path,
+        r"""
+a. :hash: json
+
+#. :hash: json
+""",
+    )
+
+    page.finish(diagnostics)
+    assert len(diagnostics) == 2