dcmspec 0.2.2__tar.gz → 0.3.0__tar.gz

{dcmspec-0.2.2 → dcmspec-0.3.0}/CHANGELOG.md

@@ -2,6 +2,35 @@
 
 These release notes summarize key changes, improvements, and breaking updates for each version of **dcmspec**.
 
+## [0.3.0] - 2025-11-27
+
+### Added
+
+- CSV output mode in `SpecPrinter` via `print_csv` method
+- Excel OOXML output mode in `SpecPrinter` via `print_xlsx` method
+- Optional `output` parameter in `SpecPrinter` for writing to files
+- Optional `column_width` parameter in `print_table` and `print_xlsx`
+- `--print-mode` option in `modattributes` CLI for CSV and OOXML output
+- `--output` option in `modattributes` CLI for directing output to files
+
+### Changed
+
+- Switch PyPI version badge in `README` from `badge.fury.io` to `shields.io` for faster updates
+- Update modattributes CLI example for CSV and XLSX output
+- Improved nesting level color schemes for consistency and better contrast and readability.
+
+### Fixed
+
+- Correct handling of `include_table` names in `DOMTableSpecParser`
+- Correct handling of empty node attributes in `SpecPrinter`
+
+## [0.2.3] - 2025-09-29
+
+### Fixed
+
+- Hotfix: Force UTF-8 decoding for DICOM standard XHTML downloads to prevent mojibake when server omits charset ([#85](https://github.com/dwikler/dcmspec/issues/85)).
+- Hotfix: Add missing `progress_observer` argument to `CSVTableSpecParser.parse` for interface compatibility and to prevent `TypeError` when used with `SpecFactory` ([#86](https://github.com/dwikler/dcmspec/issues/86)).
+
 ## [0.2.2] - 2025-09-25
 
 ### Fixed

{dcmspec-0.2.2 → dcmspec-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dcmspec
-Version: 0.2.2
+Version: 0.3.0
 Summary: Toolkit for extracting, parsing, and processing DICOM specifications.
 License: Apache-2.0
 License-File: LICENSE
@@ -25,8 +25,10 @@ Provides-Extra: pdf
 Requires-Dist: anytree (>=2.13.0,<3.0.0)
 Requires-Dist: bs4 (>=0.0.2,<0.0.3)
 Requires-Dist: camelot-py (>=1.0.0,<2.0.0) ; extra == "pdf"
+Requires-Dist: html2text (>=2025.4.15,<2026.0.0)
 Requires-Dist: lxml (>=5.4.0,<6.0.0)
 Requires-Dist: opencv-python-headless (>=4.12.0.88,<5.0.0) ; extra == "pdf"
+Requires-Dist: openpyxl (>=3.1.5,<4.0.0)
 Requires-Dist: openpyxl (>=3.1.5,<4.0.0) ; extra == "pdf"
 Requires-Dist: pandas (>=2.3.1,<3.0.0) ; extra == "pdf"
 Requires-Dist: pdfplumber (>=0.11.7,<0.12.0) ; extra == "pdf"
@@ -43,9 +45,9 @@ Project-URL: Source, https://github.com/dwikler/dcmspec
 Description-Content-Type: text/markdown
 
 [![tests](https://github.com/dwikler/dcmspec/actions/workflows/test.yml/badge.svg)](https://github.com/dwikler/dcmspec/actions/workflows/test.yml)
-[![PyPI version](https://badge.fury.io/py/dcmspec.svg)](https://badge.fury.io/py/dcmspec)
+[![PyPI version](https://img.shields.io/pypi/v/dcmspec.svg)](https://pypi.org/project/dcmspec/)
 [![Python versions](https://img.shields.io/pypi/pyversions/dcmspec.svg)](https://pypi.org/project/dcmspec/)
-[![DOI](https://zenodo.org/badge/DOI/10.5281/zenodo.17206999.svg)](https://doi.org/10.5281/zenodo.17206999)
+[![DOI](https://zenodo.org/badge/DOI/10.5281/zenodo.17742287.svg)](https://doi.org/10.5281/zenodo.17742287)
 
 # dcmspec
 

{dcmspec-0.2.2 → dcmspec-0.3.0}/README.md

@@ -1,7 +1,7 @@
 [![tests](https://github.com/dwikler/dcmspec/actions/workflows/test.yml/badge.svg)](https://github.com/dwikler/dcmspec/actions/workflows/test.yml)
-[![PyPI version](https://badge.fury.io/py/dcmspec.svg)](https://badge.fury.io/py/dcmspec)
+[![PyPI version](https://img.shields.io/pypi/v/dcmspec.svg)](https://pypi.org/project/dcmspec/)
 [![Python versions](https://img.shields.io/pypi/pyversions/dcmspec.svg)](https://pypi.org/project/dcmspec/)
-[![DOI](https://zenodo.org/badge/DOI/10.5281/zenodo.17206999.svg)](https://doi.org/10.5281/zenodo.17206999)
+[![DOI](https://zenodo.org/badge/DOI/10.5281/zenodo.17742287.svg)](https://doi.org/10.5281/zenodo.17742287)
 
 # dcmspec
 

{dcmspec-0.2.2 → dcmspec-0.3.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "dcmspec"
-version = "0.2.2"
+version = "0.3.0"
 description = "Toolkit for extracting, parsing, and processing DICOM specifications."
 authors = [{ name = "David Wikler", email = "david.wikler@ulb.be" }]
 license = { text = "Apache-2.0" }
@@ -29,6 +29,8 @@ dependencies = [
     "requests (>=2.32.3,<3.0.0)",
     "lxml (>=5.4.0,<6.0.0)",
     "rich (>=14.0.0,<15.0.0)",
+    "html2text (>=2025.4.15,<2026.0.0)",
+    "openpyxl (>=3.1.5,<4.0.0)",
 ]
 
 [project.urls]

{dcmspec-0.2.2 → dcmspec-0.3.0}/src/dcmspec/apps/cli/iodattributes.py

@@ -58,12 +58,25 @@ def main():
     parser.add_argument("--config", help="Path to the configuration file")
     parser.add_argument(
         "--print-mode", 
-        choices=["table", "tree", "none"],
+        choices=["table", "tree", "csv", "xlsx", "none"],
         default="table",
-        help="Print as 'table' (default), 'tree', or 'none' to skip printing"
+        help="Print as 'table' (default), 'tree', 'csv', 'xlsx' (requires --output), or 'none' to skip printing"
+    )
+    parser.add_argument(
+        "--no-color",
+        action="store_true",
+        help="Disable colorized output (default: color enabled for terminal, disabled for file output)"
+    )
+    parser.add_argument(
+        "--output",
+        type=str,
+        help="Path to output file. If not specified (and not xlsx), prints to stdout."
     )
     args = parser.parse_args()
 
+    if args.print_mode == "xlsx" and not args.output:
+        parser.error("--output is required when --print-mode xlsx")
+
     cache_file_name = "Part3.xhtml"
     model_file_name = f"Part3_{args.table}_expanded.json"
     table_id = args.table 
@@ -114,11 +127,21 @@ def main():
     )
 
     # Print the model
-    printer = IODSpecPrinter(model)
+    printer = IODSpecPrinter(model, output=getattr(args, "output", None))
+
+    # Only disable color if --no-color is set; SpecPrinter disables color for file outputs automatically
+    use_color = not args.no_color
+
     if args.print_mode == "tree":
-        printer.print_tree(colorize=True)
+        printer.print_tree(
+            attr_names=["elem_tag", "elem_type", "elem_name"], attr_widths=[11, 2, 100], colorize=use_color
+            )
     elif args.print_mode == "table":
-        printer.print_table(colorize=True)
+        printer.print_table(colorize=use_color, column_widths=[30, 11, 4, 60])
+    elif args.print_mode == "csv":
+        printer.print_csv(colorize=use_color)
+    elif args.print_mode == "xlsx":
+        printer.print_xlsx(column_widths=[60, 16, 10, 100], colorize=use_color)
     # else: do not print anything if print_mode == "none"
 
 if __name__ == "__main__":

{dcmspec-0.2.2 → dcmspec-0.3.0}/src/dcmspec/apps/cli/modattributes.py

@@ -4,7 +4,7 @@ Features:
 - Download and parse DICOM Module Attributes tables from Part 3 of the DICOM standard.
 - Optionally merge additional information (VR, VM, Keyword, Status) from Part 6.
 - Cache the resulting model as a JSON file for future runs and as a structured representation of the standard.
-- Print the resulting model as a table or tree.
+- Print the resulting model as a table, tree, csv, or xlsx.
 - Supports caching, configuration files, and command-line options for flexible workflows.
 
 Usage:
@@ -113,12 +113,12 @@ def main():
 
     The tool parses the specified Module Attributes table to extract all attributes, tags, types,
     and descriptions for the module. Optionally, it can merge in VR, VM, Keyword, or Status information
-    from Part 6. The output can be printed as a table or tree.
+    from Part 6. The output can be printed as a table, tree, csv, or xlsx (xlsx requires --output).
 
     The resulting model is cached as a JSON file. The primary purpose of this cache file is to provide
-    a structured, machine-readable representation of the module's attributes, which can be used for further processing
-    or integration in other tools. As a secondary benefit, the cache file is also used to speed up subsequent runs
-    of the CLI scripts.
+    a structured, machine-readable representation of the module's attributes, which can be used for
+    further processing or integration in other tools. As a secondary benefit, the cache file is also used
+    to speed up subsequent runs of the CLI scripts.
 
     Usage:
         poetry run python -m src.dcmspec.apps.cli.modattributes <table_id> [options]
@@ -129,14 +129,17 @@ def main():
         --include-depth (int): Depth to which included tables should be parsed (default: unlimited).
         --force-parse: Force reparsing of the DOM and regeneration of the JSON model.
         --force-download: Force download of the input file and regeneration of the model.
-        --print-mode (str): Print as 'table' (default), 'tree', or 'none' to skip printing.
+        --print-mode (str): Print as 'table' (default), 'tree', 'csv', 'xlsx', or 'none' to skip printing.
+                            'xlsx' requires --output.
+        --output (str): Path to output file. If not specified (and not xlsx), prints to stdout.
         --add-part6 (list): Specification(s) to merge from Part 6 (e.g., --add-part6 VR VM).
         --force-update: Force update of the specifications merged from part 6, even if cached.
         -d, --debug: Enable debug logging to the console.
         -v, --verbose: Enable verbose (info-level) logging to the console.
 
-    Example:
+    Examples:
         poetry run python -m src.dcmspec.apps.cli.modattributes table_C.7-1 --add-part6 VR VM
+        poetry run python -m src.dcmspec.apps.cli.modattributes table_C.7-1 --print-mode xlsx --output module.xlsx
 
     """
     # Parse command-line arguments
@@ -164,9 +167,19 @@ def main():
     )
     parser.add_argument(
         "--print-mode", 
-        choices=["table", "tree", "none"],
+        choices=["table", "tree", "csv", "xlsx", "none"],
         default="table",
-        help="Print as 'table' (default), 'tree', or 'none' to skip printing"
+        help="Print as 'table' (default), 'tree', 'csv', 'xlsx' (requires --output), or 'none' to skip printing"
+    )
+    parser.add_argument(
+        "--no-color",
+        action="store_true",
+        help="Disable colorized output (default: color enabled for terminal, disabled for file output)"
+    )
+    parser.add_argument(
+        "--output",
+        type=str,
+        help="Path to output file. If not specified (and not xlsx), prints to stdout."
     )
     parser.add_argument(
         "--add-part6",
@@ -189,9 +202,12 @@ def main():
         action="store_true",
         help="Enable verbose (info-level) logging to the console"
     )
-    
+
     args = parser.parse_args()
 
+    if args.print_mode == "xlsx" and not args.output:
+        parser.error("--output is required when --print-mode xlsx")
+
     # Set up logger
     logger = logging.getLogger("modattributes")
     handler = logging.StreamHandler()
@@ -254,11 +270,18 @@ def main():
         model = module_model
 
     logger.debug("Model ready for printing/output")
-    printer = SpecPrinter(model)
+    printer = SpecPrinter(model, output=args.output)
+
+    # Only disable color if --no-color is set; SpecPrinter disables color for file outputs automatically
+    use_color = not args.no_color
     if args.print_mode == "tree":
-        printer.print_tree(colorize=True)
+        printer.print_tree(attr_names=["elem_tag", "elem_type", "elem_name"], attr_widths=[11, 2, 100], colorize=use_color)
     elif args.print_mode == "table":
-        printer.print_table(colorize=True)
+        printer.print_table(colorize=use_color, column_widths=[30, 11, 4, 60])
+    elif args.print_mode == "csv":
+        printer.print_csv(colorize=use_color)
+    elif args.print_mode == "xlsx":
+        printer.print_xlsx(column_widths=[60, 16, 10, 100], colorize=use_color)
     # else: do not print anything if print_mode == "none"
 
 if __name__ == "__main__":

{dcmspec-0.2.2 → dcmspec-0.3.0}/src/dcmspec/csv_table_spec_parser.py

@@ -3,10 +3,11 @@
 Provides the CSVTableSpecParser class for parsing DICOM specification tables in CSV format,
 converting them into structured in-memory representations using anytree.
 """
-from typing import Tuple
+from typing import Dict, List, Tuple, Optional
 from anytree import Node
 
 from dcmspec.spec_parser import SpecParser
+from dcmspec.progress import ProgressObserver
 
 class CSVTableSpecParser(SpecParser):
     """Base parser for DICOM Specification IHE tables in CSV-like format."""
@@ -14,10 +15,11 @@ class CSVTableSpecParser(SpecParser):
     def parse(
         self,
         table: dict,
-        column_to_attr,
-        name_attr="elem_name",
-        table_id=None,
-        include_depth=None,
+        column_to_attr: dict,
+        name_attr: str = "elem_name",
+        table_id: Optional[str] = None,
+        include_depth: Optional[int] = None,
+        progress_observer: Optional[ProgressObserver] = None,
     ) -> Tuple[Node, Node]:
         """Parse specification metadata and content from a single table dict.
 
@@ -27,11 +29,18 @@ class CSVTableSpecParser(SpecParser):
             name_attr (str): The attribute to use for node names.
             table_id (str, optional): Table identifier for model parsing.
             include_depth (int, optional): The depth to which included tables should be parsed.
+            progress_observer (Optional[ProgressObserver]): 
+                Accepted for interface compatibility, but ignored in this parser.
+                Included so that this method can be called with the same arguments as other table parsers.
 
         Returns:
             tuple: (metadata_node, content_node)
 
         """
+        if progress_observer is not None and hasattr(self, "logger"):
+            self.logger.debug(
+                "Progress reporting is not supported yet for CSV parsing and will be ignored."
+            )
         # Use the header and data from the grouped table dict
         header = table.get("header", [])
         data = table.get("data", [])
@@ -47,8 +56,8 @@ class CSVTableSpecParser(SpecParser):
 
     def parse_table(
         self,
-        tables: list,  # List of tables, each a list of rows (list of str)
-        column_to_attr: dict,
+        tables: List[List[List[str]]],  # List of tables, each a list of rows (list of str)
+        column_to_attr: Dict[int, str],
         name_attr: str = "elem_name",
     ) -> Node:
         """Build a tree from tables using column mapping and '>' nesting logic.

{dcmspec-0.2.2 → dcmspec-0.3.0}/src/dcmspec/doc_handler.py

@@ -94,6 +94,8 @@ class DocHandler:
         try:
             with requests.get(url, timeout=30, stream=True, headers={"Accept-Encoding": "identity"}) as response:
                 response.raise_for_status()
+                self._set_response_encoding(response)
+
                 total = int(response.headers.get('content-length', 0))
                 chunk_size = 8192
                 if binary:
@@ -109,6 +111,19 @@ class DocHandler:
             self.logger.error(f"Failed to save file {file_path}: {e}")
             raise RuntimeError(f"Failed to save file {file_path}: {e}") from e
 
+    def _set_response_encoding(self, response):
+        """Set response.encoding to UTF-8 only if the Content-Type header does not specify a charset.
+        
+        Force utf-8 decoding if the web server does not specify the charset in the HTTP response
+        Content-Type header (DICOM standard XHTML files are always UTF-8 encoded).  
+        """
+        content_type = response.headers.get("Content-Type", "")
+        if "charset=" not in content_type.lower():
+            response.encoding = "utf-8"
+            self.logger.debug("No charset in Content-Type header; forcing UTF-8 decoding.")
+        else:
+            self.logger.debug(f"Using server-specified encoding from Content-Type: {content_type}")
+            
     def _report_progress(self, downloaded, total, progress_observer, last_percent):
         """Report progress if percent changed.
 

{dcmspec-0.2.2 → dcmspec-0.3.0}/src/dcmspec/dom_table_spec_parser.py

@@ -3,17 +3,20 @@
 Provides the DOMSpecParser class for parsing DICOM specification tables from XHTML documents,
 converting them into structured in-memory representations using anytree.
 """
+
 from contextlib import contextmanager
 import re
 import unicodedata
+from typing import Any, Dict, Optional, Union
+
 from unidecode import unidecode
 from anytree import Node
 from bs4 import BeautifulSoup, Tag
-from typing import Any, Dict, Optional, Union
-from dcmspec.spec_parser import SpecParser
+import html2text
 
+from dcmspec.spec_parser import SpecParser
 from dcmspec.dom_utils import DOMUtils
-from dcmspec.progress import Progress, ProgressStatus, calculate_percent
+from dcmspec.progress import Progress, ProgressObserver, ProgressStatus, calculate_percent
 
 class DOMTableSpecParser(SpecParser):
     """Parser for DICOM specification tables in XHTML DOM format.
@@ -43,7 +46,7 @@ class DOMTableSpecParser(SpecParser):
         column_to_attr: Dict[int, str],
         name_attr: str,
         include_depth: Optional[int] = None,  # None means unlimited
-        progress_observer: Optional[Any] = None,
+        progress_observer: Optional[ProgressObserver] = None,
         skip_columns: Optional[list[int]] = None,
         unformatted: Optional[Union[bool, Dict[int, bool]]] = True,
     ) -> tuple[Node, Node]:
@@ -59,7 +62,7 @@ class DOMTableSpecParser(SpecParser):
             name_attr (str): The attribute name to use for building node names.
             include_depth (Optional[int], optional): The depth to which included tables should be parsed. 
                 None means unlimited.
-            progress_observer (Optional[object], optional): Optional observer to report download progress.
+            progress_observer (Optional[ProgressObserver]): Optional observer to report parsing progress.
             skip_columns (Optional[list[int]]): List of column indices to skip if the row is missing a column.
                 This argument is typically set via `parser_kwargs` when using SpecFactory.
             unformatted (Optional[Union[bool, Dict[int, bool]]]): 
@@ -132,7 +135,7 @@ class DOMTableSpecParser(SpecParser):
         name_attr: str,
         table_nesting_level: int = 0,
         include_depth: Optional[int] = None,  # None means unlimited
-        progress_observer: Optional[Any] = None,
+        progress_observer: Optional[ProgressObserver] = None,
         skip_columns: Optional[list[int]] = None,
         visited_tables: Optional[set] = None,
         unformatted_list: Optional[list[bool]] = None,
@@ -151,7 +154,7 @@ class DOMTableSpecParser(SpecParser):
             name_attr: tree node attribute name to use to build node name
             table_nesting_level: The nesting level of the table (used for recursion call only).
             include_depth: The depth to which included tables should be parsed.
-            progress_observer (Optional[object], optional): Optional observer to report download progress.
+            progress_observer (Optional[ProgressObserver]): Optional observer to report parsing progress.
             skip_columns (Optional[list[int]]): List of column indices to skip if the row is missing a column.
             visited_tables (Optional[set]): Set of table IDs that have been visited to prevent infinite recursion.
             unformatted_list (Optional[list[bool]]): List of booleans indicating whether to extract each column as 
@@ -287,7 +290,7 @@ class DOMTableSpecParser(SpecParser):
         unformatted_list: list[bool],
         level_nodes: Dict[int, Node],
         root: Node,
-        progress_observer: Optional[Any] = None
+        progress_observer: Optional[ProgressObserver] = None
     ) -> None:
         """Process all rows in the table, handling recursion, nesting, and node creation."""
         rows = table.find_all("tr")[1:]
@@ -506,6 +509,7 @@ class DOMTableSpecParser(SpecParser):
 
         return logical_cells, logical_col_idx, physical_col_idx
 
+
     def _extract_cell_value(
         self,
         cell: Tag,
@@ -518,11 +522,27 @@ class DOMTableSpecParser(SpecParser):
             if unformatted_list and logical_col_idx < len(unformatted_list)
             else True
         )
-        if use_unformatted:
-            return self._clean_extracted_text(cell.get_text(separator="\n", strip=True))
-        else:
+
+        # Guard clause: if formatted HTML is required, return content as-is
+        if not use_unformatted:
+            # Keep original HTML content
             return self._clean_extracted_text(cell.decode_contents())
 
+        # Use html2text for better readability of unformatted text extraction
+        converter = self._create_html2text_converter()
+        raw_text = converter.handle(str(cell))
+        return self._clean_extracted_text(raw_text)
+
+
+    def _create_html2text_converter(self) -> html2text.HTML2Text:
+        """Create and configure an html2text converter for consistent text extraction."""
+        converter = html2text.HTML2Text()
+        converter.ignore_links = True       # Remove URLs
+        converter.ignore_images = True      # Remove image references
+        converter.ignore_emphasis = True    # Remove Markdown emphasis
+        converter.body_width = 0            # Disable word wrapping
+        return converter
+
     def _update_rowspan_trackers(
         self,
         physical_col_idx: int,
@@ -781,15 +801,7 @@ class DOMTableSpecParser(SpecParser):
         return header
 
     def _clean_extracted_text(self, text: str) -> str:
-        """Clean extracted text using Unicode normalization and regex.
-
-        Args:
-            text (str): The text to be cleaned.
-
-        Returns:
-            str: The cleaned text.
-
-        """
+        """Clean extracted text using Unicode normalization and regex."""
         # Normalize unicode characters to compatibility form
         cleaned = unicodedata.normalize('NFKC', text)
 
@@ -805,27 +817,48 @@ class DOMTableSpecParser(SpecParser):
         # Remove stray  character
         cleaned = cleaned.replace('\u00c2', '')
 
+        # Collapse multiple newlines (including those separated by spaces/tabs) into a single newline
+        cleaned = re.sub(r'(\n\s*){2,}', '\n', cleaned)
+
         return cleaned.strip()
 
-    def _sanitize_string(self, input_string: str) -> str:
-        """Sanitize string to use it as a node attribute name.
+    @staticmethod
+    def _sanitize_string(input_string: str) -> str:
+        r"""Sanitize a string to make it safe for use as a node attribute name.
 
-        - Convert non-ASCII characters to closest ASCII equivalents
-        - Replace space characters and slashes with underscores
-        - Replace parentheses characters with dashes
+        Transformations applied:
+        - Convert to lowercase.
+        - Transliterate non-ASCII characters to ASCII.
+        - Replace spaces, slashes, newlines, and dots with underscores ("_").
+        - Replace parentheses with dashes ("-").
+        - Remove all characters except letters, digits, underscores, and dashes.
+        - Collapse multiple consecutive underscores into a single underscore.
+        - Remove leading and trailing underscores for cleanliness.
+        - Return a default name if the result is empty after sanitization.
 
         Args:
-            input_string (str): The string to be sanitized.
+            input_string (str): The original string to sanitize.
 
         Returns:
-            str: The sanitized string.
+            str: A sanitized version of the input string, suitable for use as an identifier.
+                    or "unnamed_node" if sanitization results in an empty string.
+
+        Example:
+            >>> DOMTableSpecParser._sanitize_string(
+            '>>Include\\nTable C.36.2.2.19-1 "RT Beam Limiting Device Definition Macro Attributes"\\n.'
+            )
+            'include_table_c_36_2_2_19-1_rt_beam_limiting_device_definition_macro_attributes'
+            >>> DOMTableSpecParser._sanitize_string('...')
+            'unnamed_node'
 
         """
-        # Normalize the string to NFC form and transliterate to ASCII
         normalized_str = unidecode(input_string.lower())
-        # Replace spaces and slashes with underscores, parentheses with dashes, and single quotes with underscores
-        return re.sub(
-            r"[ /\-()']",
-            lambda match: "-" if match.group(0) in "()" else "_",
-            normalized_str,
-        )
+        sanitized = re.sub(r"[ /\n\\.]", "_", normalized_str)  # spaces, slashes, newlines, dots → _
+        sanitized = re.sub(r"[()]", "-", sanitized)            # parentheses → -
+        sanitized = re.sub(r"[^a-z0-9_-]", "", sanitized)      # remove other chars
+        sanitized = re.sub(r"_+", "_", sanitized)              # collapse multiple underscores
+        sanitized = sanitized.strip("_")                       # remove leading/trailing underscores
+
+        # Fallback to default name if sanitization resulted in empty string
+        return sanitized or "unnamed_node"
+