feat: update version to 3.6.7 and rename id property to id_ across interfaces and models

README.md

@@ -3,7 +3,7 @@
 [![Python Version](https://img.shields.io/badge/python-3.13+-blue.svg)](https://www.python.org/downloads/)
 [![License](https://img.shields.io/badge/license-GPL--3.0-green.svg)](LICENSE)
 ![Development Status](https://img.shields.io/badge/status-beta-orange.svg)
-![Version](https://img.shields.io/badge/version-3.6.6-blue.svg)
+![Version](https://img.shields.io/badge/version-3.6.7-blue.svg)
 
 **Pyrox** is a Python back-end engine and application framework built on **PyQt6**. It provides a rich set of interfaces, models, services, and abstractions for building industrial automation and desktop applications. Pyrox is designed to be used as a foundation — downstream projects like [ControlRox](https://github.com/iroxusux/ControlRox) build their entire application layer on top of it.
 

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pyrox"
-version = "3.6.6"
+version = "3.6.7"
 authors = [{ name = "Brian LaFond", email = "Brian.L.LaFond@gmail.com" }]
 description = "Python based irox engine."
 readme = "README.md"

pyrox/interfaces/base.py

@@ -53,7 +53,7 @@ def set_id(self, id_: str) -> None: ...
 
     @property
     @abstractmethod
-    def id(self) -> str: ...
+    def id_(self) -> str: ...
 
 
 class INameable(ABC):

pyrox/models/base.py

@@ -165,7 +165,7 @@ def set_id(self, id_: str) -> None:
         self._id = id_
 
     @property
-    def id(self) -> str:
+    def id_(self) -> str:
         """Get the ID of this object."""
         return self.get_id()
 

pyrox/models/test/test_base.py

@@ -146,7 +146,7 @@ def test_set_id_updates_existing(self):
 
     def test_id_property(self):
         obj = HasId(id_="property-id")
-        assert obj.id == "property-id"
+        assert obj.id_ == "property-id"
 
 
 class TestNameable:
@@ -369,7 +369,7 @@ def test_set_all_attributes(self):
 
     def test_properties_access(self):
         obj = CoreMixin(id_="prop-id", name="Prop Name", description="Prop Desc")
-        assert obj.id == "prop-id"
+        assert obj.id_ == "prop-id"
         assert obj.name == "Prop Name"
         assert obj.description == "Prop Desc"
 

pyrox/models/test/test_connection.py

@@ -8,18 +8,18 @@
 class _Sensor(IConnectable):
     def __init__(self, id_: str):
         super().__init__()
-        self.id_ = id_
+        self._id = id_
         self.on_activate_callbacks: list = []
         self.on_deactivate_callbacks: list = []
 
     def get_id(self) -> str:
-        return self.id_
+        return self._id
 
     def set_id(self, id_: str) -> None:
-        self.id_ = id_
+        self._id = id_
 
     @property
-    def id(self) -> str:
+    def id_(self) -> str:
         return self.get_id()
 
     def get_inputs(self) -> dict[str, Any]:
@@ -35,19 +35,19 @@ def get_outputs(self) -> dict[str, Any]:
 class _Motor(IConnectable):
     def __init__(self, id_: str):
         super().__init__()
-        self.id_ = id_
+        self._id = id_
         self.speed = 0.0
         self.start_called = False
         self.stop_called = False
 
     def get_id(self) -> str:
-        return self.id_
+        return self._id
 
     def set_id(self, id_: str) -> None:
-        self.id_ = id_
+        self._id = id_
 
     @property
-    def id(self) -> str:
+    def id_(self) -> str:
         return self.get_id()
 
     def start(self):
@@ -881,7 +881,7 @@ class TestConnectable:
     def test_initialization_with_id(self):
         """Test initialization with ID."""
         obj = Connectable(id_="test_obj_1")
-        assert obj.id == "test_obj_1"
+        assert obj.id_ == "test_obj_1"
 
     def test_inherits_from_hasid(self):
         """Test that Connectable inherits from HasId."""