kicad-sch-api 0.4.1__tar.gz → 0.4.3__tar.gz

{kicad_sch_api-0.4.1/kicad_sch_api.egg-info → kicad_sch_api-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kicad-sch-api
-Version: 0.4.1
+Version: 0.4.3
 Summary: Professional KiCAD schematic manipulation library with exact format preservation
 Author-email: Circuit-Synth <shane@circuit-synth.com>
 Maintainer-email: Circuit-Synth <shane@circuit-synth.com>

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api/__init__.py

@@ -42,7 +42,7 @@ Advanced Usage:
         print(f"Found {len(issues)} validation issues")
 """
 
-__version__ = "0.4.0"
+__version__ = "0.4.3"
 __author__ = "Circuit-Synth"
 __email__ = "info@circuit-synth.com"
 

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api/core/components.py

@@ -448,11 +448,25 @@ class ComponentCollection(BaseCollection[Component]):
         Remove component by reference.
 
         Args:
-            reference: Component reference to remove
+            reference: Component reference to remove (e.g., "R1")
 
         Returns:
-            True if component was removed
+            True if component was removed, False if not found
+
+        Raises:
+            TypeError: If reference is not a string
+
+        Examples:
+            sch.components.remove("R1")
+            sch.components.remove("C2")
+
+        Note:
+            For removing by UUID or component object, use remove_by_uuid() or remove_component()
+            respectively. This maintains a clear, simple API contract.
         """
+        if not isinstance(reference, str):
+            raise TypeError(f"reference must be a string, not {type(reference).__name__}")
+
         component = self._reference_index.get(reference)
         if not component:
             return False
@@ -466,6 +480,70 @@ class ComponentCollection(BaseCollection[Component]):
         logger.info(f"Removed component: {reference}")
         return True
 
+    def remove_by_uuid(self, component_uuid: str) -> bool:
+        """
+        Remove component by UUID.
+
+        Args:
+            component_uuid: Component UUID to remove
+
+        Returns:
+            True if component was removed, False if not found
+
+        Raises:
+            TypeError: If UUID is not a string
+        """
+        if not isinstance(component_uuid, str):
+            raise TypeError(f"component_uuid must be a string, not {type(component_uuid).__name__}")
+
+        if component_uuid not in self._uuid_index:
+            return False
+
+        component = self._items[self._uuid_index[component_uuid]]
+
+        # Remove from component-specific indexes
+        self._remove_from_indexes(component)
+
+        # Remove from base collection
+        super().remove(component_uuid)
+
+        logger.info(f"Removed component by UUID: {component_uuid}")
+        return True
+
+    def remove_component(self, component: "Component") -> bool:
+        """
+        Remove component by component object.
+
+        Args:
+            component: Component object to remove
+
+        Returns:
+            True if component was removed, False if not found
+
+        Raises:
+            TypeError: If component is not a Component instance
+
+        Examples:
+            comp = sch.components.get("R1")
+            sch.components.remove_component(comp)
+        """
+        if not isinstance(component, Component):
+            raise TypeError(
+                f"component must be a Component instance, not {type(component).__name__}"
+            )
+
+        if component.uuid not in self._uuid_index:
+            return False
+
+        # Remove from component-specific indexes
+        self._remove_from_indexes(component)
+
+        # Remove from base collection
+        super().remove(component.uuid)
+
+        logger.info(f"Removed component: {component.reference}")
+        return True
+
     def get(self, reference: str) -> Optional[Component]:
         """Get component by reference."""
         return self._reference_index.get(reference)

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api/core/factories/element_factory.py

@@ -126,7 +126,7 @@ class ElementFactory:
             uuid=label_dict.get("uuid", str(uuid.uuid4())),
             position=pos,
             text=label_dict.get("text", ""),
-            label_type=LabelType(label_dict.get("label_type", "local")),
+            label_type=LabelType(label_dict.get("label_type", "label")),
             rotation=label_dict.get("rotation", 0.0),
             size=label_dict.get("size", 1.27),
             shape=(

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api/core/formatter.py

@@ -125,6 +125,10 @@ class ExactFormatter:
         self.rules["global_label"] = FormatRule(inline=False, quote_indices={1})
         self.rules["hierarchical_label"] = FormatRule(inline=False, quote_indices={1})
 
+        # Text elements
+        self.rules["text"] = FormatRule(inline=False, quote_indices={1})
+        self.rules["text_box"] = FormatRule(inline=False, quote_indices={1})
+
         # Effects and text formatting
         self.rules["effects"] = FormatRule(inline=False)
         self.rules["font"] = FormatRule(inline=False)
@@ -279,6 +283,8 @@ class ExactFormatter:
             "junction",
             "label",
             "hierarchical_label",
+            "text",
+            "text_box",
             "polyline",
             "rectangle",
         ):
@@ -384,7 +390,8 @@ class ExactFormatter:
                 result += f"\n{next_indent}{self._format_element(element, indent_level + 1)}"
             else:
                 if i in rule.quote_indices and isinstance(element, str):
-                    result += f' "{element}"'
+                    escaped_element = self._escape_string(element)
+                    result += f' "{escaped_element}"'
                 else:
                     result += f" {self._format_element(element, 0)}"
 
@@ -396,7 +403,8 @@ class ExactFormatter:
         indent = "\t" * indent_level
         next_indent = "\t" * (indent_level + 1)
 
-        result = f"({lst[0]}"
+        tag = str(lst[0])
+        result = f"({tag}"
 
         for i, element in enumerate(lst[1:], 1):
             if isinstance(element, list):
@@ -425,9 +433,18 @@ class ExactFormatter:
         return True
 
     def _escape_string(self, text: str) -> str:
-        """Escape quotes in string for S-expression formatting."""
-        # Replace double quotes with escaped quotes
-        return text.replace('"', '\\"')
+        """Escape special characters in string for S-expression formatting."""
+        # Escape backslashes first (must be done before other replacements)
+        text = text.replace('\\', '\\\\')
+        # Escape double quotes
+        text = text.replace('"', '\\"')
+        # Escape newlines (convert actual newlines to escaped representation)
+        text = text.replace('\n', '\\n')
+        # Escape carriage returns
+        text = text.replace('\r', '\\r')
+        # Escape tabs
+        text = text.replace('\t', '\\t')
+        return text
 
     def _needs_quoting(self, text: str) -> bool:
         """Check if string needs to be quoted."""

kicad_sch_api-0.4.3/kicad_sch_api/core/parsing_utils.py

@@ -0,0 +1,63 @@
+"""
+Utility functions for parsing S-expression data.
+
+This module contains helper functions used by various parsers
+to handle common parsing patterns safely and consistently.
+"""
+
+import logging
+from typing import Any
+
+import sexpdata
+
+logger = logging.getLogger(__name__)
+
+
+def parse_bool_property(value: Any, default: bool = True) -> bool:
+    """
+    Parse a boolean property from S-expression data.
+
+    Handles both sexpdata.Symbol and string types, converting yes/no to bool.
+    This is the canonical way to parse boolean properties from KiCad files.
+
+    Args:
+        value: Value from S-expression (Symbol, str, bool, or None)
+        default: Default value if parsing fails or value is None
+
+    Returns:
+        bool: Parsed boolean value
+
+    Examples:
+        >>> parse_bool_property(sexpdata.Symbol('yes'))
+        True
+        >>> parse_bool_property('no')
+        False
+        >>> parse_bool_property(None, default=False)
+        False
+        >>> parse_bool_property('YES')  # Case insensitive
+        True
+
+    Note:
+        This function was added to fix a critical bug where Symbol('yes') == 'yes'
+        returned False, causing properties like in_bom and on_board to be parsed
+        incorrectly.
+    """
+    # If value is None, use default
+    if value is None:
+        return default
+
+    # Convert Symbol to string
+    if isinstance(value, sexpdata.Symbol):
+        value = str(value)
+
+    # Handle string values (case-insensitive)
+    if isinstance(value, str):
+        return value.lower() == "yes"
+
+    # Handle boolean values directly
+    if isinstance(value, bool):
+        return value
+
+    # Unexpected type - use default
+    logger.warning(f"Unexpected type for boolean property: {type(value)}, using default={default}")
+    return default

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api/parsers/elements/symbol_parser.py

@@ -10,6 +10,7 @@ from typing import Any, Dict, List, Optional
 
 import sexpdata
 
+from ...core.parsing_utils import parse_bool_property
 from ...core.types import Point
 from ..base import BaseElementParser
 
@@ -74,9 +75,15 @@ class SymbolParser(BaseElementParser):
                                 prop_value = str(prop_value).replace('\\"', '"')
                             symbol_data["properties"][prop_name] = prop_value
                 elif element_type == "in_bom":
-                    symbol_data["in_bom"] = sub_item[1] == "yes" if len(sub_item) > 1 else True
+                    symbol_data["in_bom"] = parse_bool_property(
+                        sub_item[1] if len(sub_item) > 1 else None,
+                        default=True
+                    )
                 elif element_type == "on_board":
-                    symbol_data["on_board"] = sub_item[1] == "yes" if len(sub_item) > 1 else True
+                    symbol_data["on_board"] = parse_bool_property(
+                        sub_item[1] if len(sub_item) > 1 else None,
+                        default=True
+                    )
 
             return symbol_data
 

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3/kicad_sch_api.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kicad-sch-api
-Version: 0.4.1
+Version: 0.4.3
 Summary: Professional KiCAD schematic manipulation library with exact format preservation
 Author-email: Circuit-Synth <shane@circuit-synth.com>
 Maintainer-email: Circuit-Synth <shane@circuit-synth.com>

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/kicad_sch_api.egg-info/SOURCES.txt

@@ -53,6 +53,7 @@ kicad_sch_api/core/labels.py
 kicad_sch_api/core/nets.py
 kicad_sch_api/core/no_connects.py
 kicad_sch_api/core/parser.py
+kicad_sch_api/core/parsing_utils.py
 kicad_sch_api/core/pin_utils.py
 kicad_sch_api/core/schematic.py
 kicad_sch_api/core/texts.py

{kicad_sch_api-0.4.1 → kicad_sch_api-0.4.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "kicad-sch-api"
-version = "0.4.1"
+version = "0.4.3"
 description = "Professional KiCAD schematic manipulation library with exact format preservation"
 readme = "README.md"
 license = "MIT"