More helpful errors instead of crash on invalid docs syntax

robojumper · Oct 22, 2023 · 6ab1859 · 6ab1859
1 parent 339a38c
commit 6ab1859
Show file tree

Hide file tree

Showing 5 changed files with 43 additions and 2 deletions.
diff --git a/.scripts/make_docs.py b/.scripts/make_docs.py
@@ -145,8 +145,18 @@ def make_doc_item(sess, lines: List[str], file: str, span: Tuple[int, int],
     """
     item = {}
     # first line: meta info
-    for pair in lines[0].split(';'):
-        k, v = pair.strip().split(':')
+    first_line = lines[0].strip()
+    if first_line.endswith(';'):
+        first_line = first_line[:-1]
+    for pair in first_line.split(';'):
+        if pair == "":
+            sess.err(f"{file}:{span[0]+1}: too many semicolons in {HL_DOCS_KEYWORD} line")
+            continue
+        kvs = pair.strip().split(':')
+        if len(kvs) != 2:
+            sess.err(f"{file}:{span[0]+1}: not a key-value pair in {HL_DOCS_KEYWORD} line (expected key:value, got {pair})")
+            continue
+        k, v = kvs
         if k in item:
             sess.err(f"{file}:{span[0]+1}: duplicate key `{k}`")
         if k == "feature" or k == "ref":

diff --git a/test/docs/test_output/docs/misc/ILikeSemicolons.md b/test/docs/test_output/docs/misc/ILikeSemicolons.md
@@ -0,0 +1,11 @@
+Title: ILikeSemicolons
+
+# ILikeSemicolons
+
+Tracking Issue: [#53](https://github.com/X2CommunityCore/X2WOTCCommunityHighlander/issues/53)
+
+Deny this for aesthetic reasons
+
+## Source code references
+
+* [UnrealScriptFile2.uc:104-105](https://github.com/X2CommunityCore/X2WOTCCommunityHighlander/blob/master/test_src/UnrealScriptFile2.uc#L104-L105)
diff --git a/test/docs/test_output/docs/misc/TrailingSemi.md b/test/docs/test_output/docs/misc/TrailingSemi.md
@@ -0,0 +1,11 @@
+Title: TrailingSemi
+
+# TrailingSemi
+
+Tracking Issue: [#99999](https://github.com/X2CommunityCore/X2WOTCCommunityHighlander/issues/99999)
+
+Trailing semicolons are allowed
+
+## Source code references
+
+* [UnrealScriptFile2.uc:98-99](https://github.com/X2CommunityCore/X2WOTCCommunityHighlander/blob/master/test_src/UnrealScriptFile2.uc#L98-L99)
diff --git a/test/docs/test_output/stdout.log b/test/docs/test_output/stdout.log
diff --git a/test/docs/test_src/UnrealScriptFile2.uc b/test/docs/test_src/UnrealScriptFile2.uc
@@ -95,6 +95,15 @@ class UnrealScriptFile2 extends Object;
 /// NewGameState: none,
 /// ```
 
+/// HL-Docs: feature:TrailingSemi;
+/// Trailing semicolons are allowed
+
+/// HL-Docs: feature;NotAKVP; issue:777
+/// Semicolon instead of colon...
+
+/// HL-Docs: feature:ILikeSemicolons;;; issue:53
+/// Deny this for aesthetic reasons
+
 function Abc()
 {
 	/// HL-Docs: feature:IffyInclude; issue:9; tags: