DOP-1171: prepend term to the term's name in the glossary node

Author: madelinezec

CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Support additional ordered list labels (DOP-1129)
 - Support ordered list "start" attribute (DOP-1210)
 - Logging of the parser's version at startup (DOP-1213).
+- Snooty parser yields glossary IDs that match the IDs generated by legacy tooling (DOP-1171)
 
 ### Changed
 

snooty/parser.py

@@ -363,7 +363,7 @@ def dispatch_departure(self, node: docutils.nodes.Node) -> None:
 
             for item in definition_list.get_child_of_type(n.DefinitionListItem):
                 term_text = "".join(term.get_text() for term in item.term)
-                term_identifier = make_id(term_text)
+                term_identifier = make_id(f"term-{term_text}")
                 identifier = n.TargetIdentifier(item.start, [], [term_identifier])
                 identifier.children = item.term[:]
                 target = n.InlineTarget(item.start, [], "std", "term", None)

snooty/test_parser.py

@@ -841,7 +841,7 @@ def test_glossary_node() -> None:
         <term>
           <text>$cmd</text>
           <inline_target domain="std" name="term">
-            <target_identifier ids="['cmd']">
+            <target_identifier ids="['term-cmd']">
               <text>$cmd</text></target_identifier>
           </inline_target>
         </term>
@@ -852,7 +852,7 @@ def test_glossary_node() -> None:
         <term>
           <text>_id</text>
           <inline_target domain="std" name="term">
-            <target_identifier ids="['id']">
+            <target_identifier ids="['term-id']">
               <text>_id</text>
             </target_identifier>
           </inline_target>
@@ -864,7 +864,7 @@ def test_glossary_node() -> None:
         <term>
           <text>aggregate</text>
           <inline_target domain="std" name="term">
-            <target_identifier ids="['aggregate']">
+            <target_identifier ids="['term-aggregate']">
               <text>aggregate</text></target_identifier>
           </inline_target>
         </term>
@@ -875,7 +875,7 @@ def test_glossary_node() -> None:
         <term>
           <text>index</text>
           <inline_target domain="std" name="term">
-            <target_identifier ids="['index']">
+            <target_identifier ids="['term-index']">
               <text>index</text></target_identifier>
           </inline_target>
         </term>