ENH: stricter handling of empty comment lines in description parsers

Table description parser now stops collecting at an empty comment line
(bare `#`), so only the actual description is captured. Column
description parser raises ValueError if an empty comment line appears
inside a `# description:` block, catching accidental paragraph breaks
early.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: fedorov

scripts/python/idc_index_data_manager.py

@@ -62,11 +62,8 @@ def parse_table_description(sql_query: str) -> str:
                         if desc_text:
                             description_lines.append(desc_text)
                         j += 1
-                    elif next_stripped.startswith("#"):
-                        # Empty comment line, skip
-                        j += 1
                     else:
-                        # Non-comment line, stop collecting
+                        # Empty comment line or non-comment line, stop collecting
                         break
                 break
 
@@ -116,9 +113,14 @@ def parse_column_descriptions(sql_query: str) -> dict[str, str]:
                         if desc_text:
                             description_lines.append(desc_text)
                         i += 1
-                    elif next_stripped.startswith("#"):
-                        # Empty comment line, skip
-                        i += 1
+                    elif next_stripped == "#":
+                        msg = (
+                            "Empty comment line found inside a '# description:' block "
+                            f"at line {i + 1}. Column descriptions must not contain "
+                            "empty comment lines. Either remove the blank line or "
+                            "end the description before it."
+                        )
+                        raise ValueError(msg)
                     else:
                         # Non-comment line - this should contain the column definition
                         break

tests/test_column_description_parser.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import pytest
 from scripts.python.idc_index_data_manager import IDCIndexDataManager
 
 
@@ -163,7 +164,7 @@ def test_no_descriptions(self):
         assert len(descriptions) == 0
 
     def test_empty_description_lines(self):
-        """Test handling of empty comment lines in descriptions."""
+        """Test that empty comment lines inside a description block raise an error."""
         sql_query = """
 SELECT
   # description:
@@ -173,12 +174,8 @@ def test_empty_description_lines(self):
   collection_name,
 FROM table
 """
-        descriptions = IDCIndexDataManager.parse_column_descriptions(sql_query)
-        assert "collection_name" in descriptions
-        # Empty comment lines should be skipped
-        assert (
-            descriptions["collection_name"] == "name of the collection additional info"
-        )
+        with pytest.raises(ValueError, match="Empty comment line found inside"):
+            IDCIndexDataManager.parse_column_descriptions(sql_query)
 
     def test_nested_array_select_with_if(self):
         """Test parsing complex nested ARRAY/SELECT/IF statements."""