[BUGFIX] Treat terms with textroles as definition list

packages/guides-restructured-text/src/RestructuredText/Parser/Productions/DefinitionListRule.php

@@ -32,6 +32,7 @@
 use function ltrim;
 use function mb_strlen;
 use function mb_substr;
+use function preg_match;
 use function str_starts_with;
 use function trim;
 
@@ -44,8 +45,10 @@ final class DefinitionListRule implements Rule
 {
     public const PRIORITY = 30;
 
-    public function __construct(private readonly InlineMarkupRule $inlineMarkupRule, private readonly RuleContainer $bodyElements)
-    {
+    public function __construct(
+        private readonly InlineMarkupRule $inlineMarkupRule,
+        private readonly RuleContainer $bodyElements,
+    ) {
     }
 
     public function applies(BlockContext $blockContext): bool
@@ -156,8 +159,8 @@ private function isDefinitionTerm(string|null $currentLine, string|null $nextLin
             return false;
         }
 
-        // This a field list
-        if (str_starts_with(trim($currentLine), ':')) {
+        // This is a field list
+        if (preg_match('/^:([^:]+):( (.*)|)$/mUsi', $currentLine) > 0) {
             return false;
         }
 

packages/guides-restructured-text/src/RestructuredText/Parser/Productions/FieldListRule.php

@@ -45,19 +45,19 @@ public function __construct(private readonly RuleContainer $productions, private
 
     public function applies(BlockContext $blockContext): bool
     {
-        return $this->isFieldLine($blockContext->getDocumentIterator()->current());
+        return self::isFieldLine($blockContext->getDocumentIterator()->current());
     }
 
     public function apply(BlockContext $blockContext, CompoundNode|null $on = null): Node|null
     {
         $iterator = $blockContext->getDocumentIterator();
         $fieldListItemNodes = [];
-        while ($iterator->valid() && $this->isFieldLine($iterator->current())) {
+        while ($iterator->valid() && self::isFieldLine($iterator->current())) {
             $fieldListItemNodes[] = $this->createListItem($blockContext);
             $iterator->next();
             while ($iterator->valid() && LinesIterator::isEmptyLine($iterator->current())) {
                 $peek = $iterator->peek();
-                if (!LinesIterator::isEmptyLine($peek) && !$this->isFieldLine($peek)) {
+                if (!LinesIterator::isEmptyLine($peek) && !self::isFieldLine($peek)) {
                     break;
                 }
 
@@ -133,7 +133,7 @@ private function createDefinition(
         $buffer = new Buffer();
         $documentIterator = $blockContext->getDocumentIterator();
         $nextLine = $documentIterator->getNextLine();
-        if ($nextLine !== null && !$this->isFieldLine($nextLine)) {
+        if ($nextLine !== null && !self::isFieldLine($nextLine)) {
             $indenting = mb_strlen($nextLine) - mb_strlen(trim($nextLine));
             if ($indenting > 0) {
                 $buffer->push(mb_substr($documentIterator->getNextLine() ?? '', $indenting));
@@ -181,7 +181,7 @@ private function createDefinition(
         }
     }
 
-    private function isFieldLine(string|null $currentLine): bool
+    private static function isFieldLine(string|null $currentLine): bool
     {
         if ($currentLine === null) {
             return false;

psalm-baseline.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.17.0@c620f6e80d0abfca532b00bda366062aaedf6e5d">
+<files psalm-version="5.18.0@b113f3ed0259fd6e212d87c3df80eec95a6abf19">
   <file src="packages/guides-cli/src/Config/Configuration.php">
     <UndefinedMethod>
       <code>ignoreExtraKeys</code>

tests/Integration/tests/lists/definition-lists/expected/index.html

@@ -0,0 +1,18 @@
+<!-- content start -->
+    <div class="section" id="definition-list-with-textrole-in-term">
+    <h1>Definition list with textrole in term</h1>
+
+            <dl>
+                        <dt><code>One definition</code></dt>
+
+        <dd><p>Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam</p><p>Nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat.</p></dd>                        <dt><code>Another definition</code></dt>
+
+        <dd><div class="admonition note">
+            <p>sed diam voluptua. At vero eos et accusam et justo duo dolores
+et ea rebum. Stet clita kasd gubergren, no sea takimata sanctus est.</p>
+</div>
+<p>Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet, consetetur.</p></dd>    </dl>
+
+    </div>
+
+<!-- content end -->

tests/Integration/tests/lists/definition-lists/input/index.rst

@@ -0,0 +1,15 @@
+=====================================
+Definition list with textrole in term
+=====================================
+
+:code:`One definition`
+    Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam
+
+    Nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat.
+
+:code:`Another definition`
+    ..  note::
+        sed diam voluptua. At vero eos et accusam et justo duo dolores
+        et ea rebum. Stet clita kasd gubergren, no sea takimata sanctus est.
+
+    Lorem ipsum dolor sit amet. Lorem ipsum dolor sit amet, consetetur.