From 96a54b809c201605b0416b17b03ad208def33b73 Mon Sep 17 00:00:00 2001
From: kevin9327 <5299031+kevin9327@users.noreply.github.com>
Date: Mon, 14 Sep 2026 20:19:20 +0900
Subject: [PATCH] fix(linetypes): load .lin definitions whose header has no
 description

In a .lin file the description is optional and the comma goes with it, so
"*PLAIN" followed by "A,.5,-.25" is a complete definition. parse only kept
a header that contained a comma, so such a linetype was dropped silently
and a CENTERLTYPE naming it then reported that it was not loaded.

Take the whole header as the name when there is no comma.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 src/io/linetypes.rs | 37 +++++++++++++++++++++++++++++++++----
 1 file changed, 33 insertions(+), 4 deletions(-)

diff --git a/src/io/linetypes.rs b/src/io/linetypes.rs
index 5ce413489..92f47ed05 100644
--- a/src/io/linetypes.rs
+++ b/src/io/linetypes.rs
@@ -299,10 +299,10 @@ fn parse(src: &str) -> Vec<LineType> {
         }
 
         if let Some(rest) = line.strip_prefix('*') {
-            // New pattern header: *NAME,Description
-            if let Some((name, desc)) = rest.split_once(',') {
-                current = Some((name.trim().to_string(), desc.trim().to_string()));
-            }
+            // New pattern header: *NAME[,Description]. The description is
+            // optional, and the comma goes with it.
+            let (name, desc) = rest.split_once(',').unwrap_or((rest, ""));
+            current = Some((name.trim().to_string(), desc.trim().to_string()));
         } else if line.to_ascii_uppercase().starts_with('A') {
             // Element line: A,v1,v2,...
             let Some((name, desc)) = current.take() else {
@@ -608,3 +608,32 @@ fn push_lt_segment(token: &str, out: &mut Vec<LtSegment>) {
         out.push(seg);
     }
 }
+
+#[cfg(test)]
+mod header_tests {
+    use super::{parse, populate_document_from_source};
+    use acadrust::CadDocument;
+
+    /// A `.lin` header's description is optional, and without one the comma
+    /// goes too: `*PLAIN` followed by its `A,` line is a whole definition.
+    #[test]
+    fn a_linetype_without_a_description_is_loaded() {
+        let mut doc = CadDocument::new();
+        let added = populate_document_from_source(&mut doc, "*PLAIN\nA,.5,-.25\n");
+        assert_eq!(added, 1, "the definition was dropped");
+        assert!(doc.line_types.contains("PLAIN"));
+
+        let types = parse("*PLAIN\nA,.5,-.25\n");
+        assert_eq!(types[0].description, "");
+        let lengths: Vec<f64> = types[0].elements.iter().map(|e| e.length).collect();
+        assert_eq!(lengths, vec![0.5, -0.25]);
+    }
+
+    /// A header with a description still splits at the first comma.
+    #[test]
+    fn a_description_after_the_comma_is_kept() {
+        let types = parse("*DASHED,Dashed __ __\nA,.5,-.25\n");
+        assert_eq!(types[0].name, "DASHED");
+        assert_eq!(types[0].description, "Dashed __ __");
+    }
+}