aether-robotics 3.7.2__tar.gz → 3.7.3__tar.gz

{aether_robotics-3.7.2/aether_robotics.egg-info → aether_robotics-3.7.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aether-robotics
-Version: 3.7.2
+Version: 3.7.3
 Summary: Autonomous multi-agent robotics system with DRL-First Hybrid FDIR
 Author: Chahel Paatur
 License: MIT
@@ -57,7 +57,7 @@ Dynamic: license-file
     ████████████████████████████████
 ```
 
-# AETHER v3.7.2 — Autonomous Robotics Operating System
+# AETHER v3.7.3 — Autonomous Robotics Operating System
 
 > AETHER is the autonomous operating system for robots. Plug in and talk to your robot in plain English and ask it to do anything you want.
 

{aether_robotics-3.7.2 → aether_robotics-3.7.3}/README.md

@@ -16,7 +16,7 @@
     ████████████████████████████████
 ```
 
-# AETHER v3.7.2 — Autonomous Robotics Operating System
+# AETHER v3.7.3 — Autonomous Robotics Operating System
 
 > AETHER is the autonomous operating system for robots. Plug in and talk to your robot in plain English and ask it to do anything you want.
 

aether_robotics-3.7.3/VERSION

@@ -0,0 +1 @@
+3.7.3

{aether_robotics-3.7.2 → aether_robotics-3.7.3}/aether/__init__.py

@@ -1,2 +1,2 @@
-__version__ = "3.7.2"
+__version__ = "3.7.3"
 __author__ = "Chahel Paatur"

{aether_robotics-3.7.2 → aether_robotics-3.7.3}/aether/core/tool_discovery.py

@@ -97,13 +97,49 @@ def _probe_gpio_gpiozero() -> Tuple[bool, Dict]:
 
 
 def _probe_i2c() -> Tuple[bool, Dict]:
-    """Try board + busio for I2C (Adafruit Blinka)."""
+    """Enumerate I2C devices on bus 1 so adapter signature_matches can
+    detect HATs by address (e.g. SunFounder Robot HAT at 0x14).
+
+    smbus2 (shipped in the `pi` install extra) is the authoritative
+    enumerator: opening SMBus(1) proves the kernel I2C bus is enabled and
+    the user has permission, and read_byte() ACK-probes each address.
+    This replaces the old Blinka-only check, which required board+busio
+    and so reported "not detected" on real Pi HATs that work fine over
+    smbus2.
+
+    Returns (True, {"type", "available": True, "bus": 1, "devices": [...]})
+    on success, or (False, {"available": False, "devices": []}) on any
+    failure. The "devices" field is ALWAYS a list, never None. The whole
+    body is wrapped so a bug here can never crash boot.
+    """
     try:
-        import board  # noqa: F401
-        import busio  # noqa: F401
-        return True, {"type": "blinka_i2c"}
-    except (ImportError, RuntimeError, NotImplementedError):
-        return False, {"detail": "board/busio not available"}
+        try:
+            import smbus2
+        except ImportError:
+            # No enumerator available — report I2C absent rather than
+            # crash. (Blinka was the old path; smbus2 is required for the
+            # actual device scan, so its absence means no enumeration.)
+            return False, {"available": False, "devices": []}
+
+        bus = smbus2.SMBus(1)
+        devices: List[int] = []
+        try:
+            for addr in range(0x03, 0x78):
+                try:
+                    bus.read_byte(addr)
+                    devices.append(addr)
+                except OSError:
+                    pass  # no device ACKed at this address
+        finally:
+            bus.close()
+
+        print(f"  [I2C] Scanned bus 1: found {len(devices)} device(s): "
+              f"{[hex(d) for d in devices]}")
+        return True, {"type": "smbus2_i2c", "available": True,
+                      "bus": 1, "devices": devices}
+    except Exception:
+        # No /dev/i2c-1, permission error, or any unexpected failure.
+        return False, {"available": False, "devices": []}
 
 
 def _probe_mavlink_serial() -> Tuple[bool, Dict]:
@@ -1203,6 +1239,14 @@ class ToolDiscovery:
                 if key == "oled":
                     iface = info.get("interface", "spi").upper()
                     detail = f"{hw_type} {resolution} {iface}"
+                if key == "i2c":
+                    devs = info.get("devices", [])
+                    bus = info.get("bus", 1)
+                    if devs:
+                        addrs = ", ".join(hex(d) for d in devs)
+                        detail = f"bus {bus}, {len(devs)} device(s) at {addrs}"
+                    else:
+                        detail = f"bus {bus}, no devices found"
             else:
                 if key == "oled":
                     if info.get("spi_not_enabled"):

{aether_robotics-3.7.2 → aether_robotics-3.7.3/aether_robotics.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aether-robotics
-Version: 3.7.2
+Version: 3.7.3
 Summary: Autonomous multi-agent robotics system with DRL-First Hybrid FDIR
 Author: Chahel Paatur
 License: MIT
@@ -57,7 +57,7 @@ Dynamic: license-file
     ████████████████████████████████
 ```
 
-# AETHER v3.7.2 — Autonomous Robotics Operating System
+# AETHER v3.7.3 — Autonomous Robotics Operating System
 
 > AETHER is the autonomous operating system for robots. Plug in and talk to your robot in plain English and ask it to do anything you want.
 

{aether_robotics-3.7.2 → aether_robotics-3.7.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "aether-robotics"
-version = "3.7.2"
+version = "3.7.3"
 description = "Autonomous multi-agent robotics system with DRL-First Hybrid FDIR"
 readme = "README.md"
 license = {text = "MIT"}

aether_robotics-3.7.3/tests/test_tool_discovery.py

@@ -0,0 +1,133 @@
+"""Tests for ToolDiscovery: platform detection, manifest structure."""
+import os
+import sys
+from types import ModuleType
+from unittest.mock import patch
+
+import pytest
+
+sys.path.insert(0, os.path.join(os.path.dirname(__file__), ".."))
+
+from aether.core.tool_discovery import ToolDiscovery, _probe_i2c
+
+
+@pytest.fixture(scope="module")
+def manifest():
+    d = ToolDiscovery()
+    d.discover()
+    return d.manifest
+
+
+class TestToolDiscovery:
+    def test_platform_detected(self, manifest):
+        assert "platform" in manifest
+        assert manifest["platform"] != ""
+
+    def test_camera_found_or_missing(self, manifest):
+        hw = manifest.get("hardware", {})
+        cam = hw.get("camera", {})
+        # camera key must exist with an 'available' field
+        assert "available" in cam
+
+    def test_manifest_has_capability_score(self, manifest):
+        assert "capability_score" in manifest
+        score = manifest["capability_score"]
+        assert isinstance(score, (int, float))
+        assert 0 <= score <= 100
+
+    def test_available_tools_non_empty(self, manifest):
+        tools = manifest.get("available_tools", [])
+        assert isinstance(tools, list)
+        assert len(tools) > 0
+
+    def test_missing_capabilities_is_list(self, manifest):
+        missing = manifest.get("missing_capabilities", [])
+        assert isinstance(missing, list)
+        # On a dev machine at least some capabilities will be missing
+        # (e.g., GPIO on macOS), so the list should be non-empty
+        assert len(missing) > 0
+
+
+# ── I2C device enumeration (v3.7.3) ────────────────────────────────────
+
+
+def _fake_smbus2(ack_addrs=None, instantiation_error=None):
+    """Build a fake smbus2 module whose SMBus(1).read_byte ACKs (returns)
+    only at ack_addrs and raises OSError elsewhere. If instantiation_error
+    is set, SMBus(bus) raises it (simulates a missing /dev/i2c-N)."""
+    ack = set(ack_addrs or [])
+    mod = ModuleType("smbus2")
+
+    class _SMBus:
+        def __init__(self, bus):
+            if instantiation_error is not None:
+                raise instantiation_error
+            self.bus = bus
+
+        def read_byte(self, addr):
+            if addr in ack:
+                return 0
+            raise OSError("no device ACKed")
+
+        def close(self):
+            pass
+
+    mod.SMBus = _SMBus
+    return mod
+
+
+def test_probe_i2c_returns_devices_list_on_success():
+    fake = _fake_smbus2(ack_addrs=[0x14, 0x40])
+    with patch.dict(sys.modules, {"smbus2": fake}):
+        ok, info = _probe_i2c()
+    assert ok is True
+    assert info["available"] is True
+    assert info["bus"] == 1
+    assert info["type"] == "smbus2_i2c"
+    assert info["devices"] == [0x14, 0x40]  # ascending scan order
+
+
+def test_probe_i2c_returns_empty_list_on_no_devices():
+    fake = _fake_smbus2(ack_addrs=[])  # nothing ACKs
+    with patch.dict(sys.modules, {"smbus2": fake}):
+        ok, info = _probe_i2c()
+    assert ok is True           # bus opened fine, just no peripherals
+    assert info["devices"] == []
+
+
+def test_probe_i2c_returns_false_on_no_bus():
+    # SMBus(1) raises FileNotFoundError — /dev/i2c-1 absent / I2C disabled
+    fake = _fake_smbus2(instantiation_error=FileNotFoundError("/dev/i2c-1"))
+    with patch.dict(sys.modules, {"smbus2": fake}):
+        ok, info = _probe_i2c()
+    assert ok is False
+    assert info["available"] is False
+    assert info["devices"] == []
+
+
+def test_probe_i2c_never_crashes_boot():
+    # smbus2 import raises ImportError — must return False cleanly, no crash
+    with patch.dict(sys.modules, {"smbus2": None}):
+        ok, info = _probe_i2c()
+    assert ok is False
+    assert info["available"] is False
+    assert info["devices"] == []
+
+
+def test_capability_discovery_printer_shows_i2c_with_devices(capsys):
+    """With a device enumerated, the summary's i2c line must read OK with
+    the address — never the old '--- not detected'."""
+    with patch("aether.core.tool_discovery._probe_i2c",
+               return_value=(True, {"type": "smbus2_i2c", "available": True,
+                                    "bus": 1, "devices": [0x14]})):
+        d = ToolDiscovery()
+        d.discover()
+        d.print_summary()
+
+    out = capsys.readouterr().out
+    i2c_line = next(ln for ln in out.splitlines()
+                    if ln.strip().startswith(("[+] i2c", "[-] i2c")))
+    assert "[+]" in i2c_line
+    assert "OK" in i2c_line
+    assert "0x14" in i2c_line
+    assert "not detected" not in i2c_line

aether_robotics-3.7.2/VERSION

@@ -1 +0,0 @@
-3.7.2

aether_robotics-3.7.2/tests/test_tool_discovery.py

@@ -1,46 +0,0 @@
-"""Tests for ToolDiscovery: platform detection, manifest structure."""
-import os
-import sys
-
-import pytest
-
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), ".."))
-
-from aether.core.tool_discovery import ToolDiscovery
-
-
-@pytest.fixture(scope="module")
-def manifest():
-    d = ToolDiscovery()
-    d.discover()
-    return d.manifest
-
-
-class TestToolDiscovery:
-    def test_platform_detected(self, manifest):
-        assert "platform" in manifest
-        assert manifest["platform"] != ""
-
-    def test_camera_found_or_missing(self, manifest):
-        hw = manifest.get("hardware", {})
-        cam = hw.get("camera", {})
-        # camera key must exist with an 'available' field
-        assert "available" in cam
-
-    def test_manifest_has_capability_score(self, manifest):
-        assert "capability_score" in manifest
-        score = manifest["capability_score"]
-        assert isinstance(score, (int, float))
-        assert 0 <= score <= 100
-
-    def test_available_tools_non_empty(self, manifest):
-        tools = manifest.get("available_tools", [])
-        assert isinstance(tools, list)
-        assert len(tools) > 0
-
-    def test_missing_capabilities_is_list(self, manifest):
-        missing = manifest.get("missing_capabilities", [])
-        assert isinstance(missing, list)
-        # On a dev machine at least some capabilities will be missing
-        # (e.g., GPIO on macOS), so the list should be non-empty
-        assert len(missing) > 0