feat(inverter): add full EG4 lineup across SNA + 18KPV families

- SNA family (capture-validated): 6000XP, 12000XP, 6500EX-48, 3000EHV-48
- 18KPV family (document-derived, untested): 18KPV, 12kPV
- Add dedicated alias driver modules for clean labels (eg4_12000xp,
  eg4_12kpv, eg4_6500ex, eg4_3000ehv)
- Update settings dropdown with full lineup (legacy models marked)
- Document FlexBOSS/GridBOSS as a NEW platform needing a dedicated map
  (NOT aliased to SNA/18KPV — would silently decode garbage)
- Update README (Supported inverter models table) + TODO #7
- 21 tests pass

Author: jmewing

README.md

@@ -42,6 +42,28 @@ This project is actively running on private hardware monitoring an EG4 6000XP in
 | Grafana | ✅ Built | Pre-loaded dashboards and data source |
 | Automation rules | ✅ Built | Rule-table model (time + sensor conditions) that writes holding registers |
 
+### Supported inverter models
+
+lux-mon speaks the **Luxpower SNA** and **18KPV** register families. EG4
+inverters are Luxpower rebadges, so the full EG4 lineup is supported via two
+families:
+
+| Family | Models | Register map | Status |
+|--------|--------|--------------|--------|
+| **SNA** (2-MPPT, single-phase off-grid) | EG4 6000XP, 12000XP, 6500EX-48, 3000EHV-48, Luxpower SNA | `collector/registers.py` | ✅ Capture-validated on the 6000XP |
+| **18KPV** (3-MPPT, split-phase 120/240V, AFCI, generator) | EG4 18KPV, 12kPV | `collector/drivers/eg4_18kpv.py` | ⚠️ Document-derived, not yet capture-validated |
+
+- **SNA family** is the validated reference — the 6000XP map was
+  reverse-engineered from live captures. The 12000XP, 6500EX-48, and
+  3000EHV-48 (legacy, discontinued) share the exact same register layout.
+- **18KPV family** is derived from the official EG4 Modbus protocol document
+  and has **not** been validated against a live unit. The 12kPV shares the
+  18KPV map. Community feedback will identify any gaps.
+- **FlexBOSS18/21 and GridBOSS** are a *new* platform (not a Luxpower SNA
+  rebadge) with an unknown register map. They are intentionally **not**
+  aliased to either family — doing so would silently decode garbage. A
+  dedicated driver is needed once their protocol is documented.
+
 ### In progress / near-term
 
 - **Inverter Edit Mode page** — manual Read/Set for every editable EG4 6000XP holding register, mirroring the EG4 Monitor Maintenance tab.
@@ -52,7 +74,7 @@ This project is actively running on private hardware monitoring an EG4 6000XP in
 
 - Generator and AC-coupled charge support (generator-charge register path is stubbed)
 - Battery protection automations (SOC threshold + restore) — rule-table model supports this, needs UI
-- More inverter models via pluggable Modbus RTU drivers
+- More inverter models via pluggable Modbus RTU drivers (FlexBOSS/GridBOSS need a dedicated register map)
 - Forecast.Solar provider (listed in settings, not yet wired)
 
 ## Architecture

TODO.md

@@ -39,9 +39,11 @@
 ### 6. Holding register map corrections — ⬜ (explicitly "in progress")
 - Align `collector/protocol.py` with `LXP_REGISTERS.txt`, add missing registers, fix ranges, correct AC charge current to register 168.
 
-### 7. More inverter models — 🚧 (aliases only)
-- `collector/drivers/registry.py` maps many model names (`eg4_3000ehv`, `eg4_6500ex`, `luxpower_12k`, `growatt`, `solis`, etc.) to the `eg4_6000xp` driver as a fallback.
-- **To do:** write dedicated drivers for models that differ from the 6000XP register family.
+### 7. More inverter models — 🚧 (SNA + 18KPV families done; FlexBOSS/GridBOSS pending)
+- `collector/drivers/registry.py` now maps the full EG4 lineup across two families:
+  - **SNA family** (capture-validated): `eg4_6000xp`, `eg4_12000xp`, `eg4_6500ex`, `eg4_3000ehv`, `luxpower_sna`.
+  - **18KPV family** (document-derived, untested): `eg4_18kpv`, `eg4_12kpv`.
+- **To do:** FlexBOSS18/21 and GridBOSS are a *new* platform (not a Luxpower SNA rebadge) with an unknown register map — need a dedicated driver once their protocol is documented. Do NOT alias them to SNA/18KPV.
 
 ### 8. RS-485 transport for the main collector — 🚧 (separate daemon only)
 - The main collector's `transport` options are `tcp_passive`, `tcp_active`, `replay`. RS-485 is a **separate** daemon (`lux-mon-rs485`), not a pluggable transport of the main collector.

collector/drivers/eg4_12000xp.py

@@ -0,0 +1,17 @@
+"""EG4 12000XP inverter driver.
+
+The EG4 12000XP is a 12 kW off-grid inverter in the Luxpower SNA family
+(2-MPPT, single-phase). This driver is an alias of the canonical
+`luxpower_sna` driver — same register map, batches, and decode path.
+"""
+from __future__ import annotations
+
+from . import ModelDriver
+from .luxpower_sna import create_driver as _sna
+
+
+def create_driver() -> ModelDriver:
+    drv = _sna()
+    drv.name = "eg4_12000xp"
+    drv.label = "EG4 12000XP"
+    return drv

collector/drivers/eg4_12kpv.py

@@ -0,0 +1,17 @@
+"""EG4 12kPV inverter driver.
+
+The EG4 12kPV is the 12 kW sibling of the 18KPV — same 3-MPPT, split-phase
+120/240V, AFCI, generator-input register family. This driver is an alias of
+the `eg4_18kpv` driver (document-derived, not yet capture-validated).
+"""
+from __future__ import annotations
+
+from . import ModelDriver
+from .eg4_18kpv import create_driver as _18kpv
+
+
+def create_driver() -> ModelDriver:
+    drv = _18kpv()
+    drv.name = "eg4_12kpv"
+    drv.label = "EG4 12kPV"
+    return drv

collector/drivers/eg4_3000ehv.py

@@ -0,0 +1,17 @@
+"""EG4 3000EHV-48 inverter driver.
+
+The EG4 3000EHV-48 is a legacy small off-grid inverter in the Luxpower SNA
+family (2-MPPT, single-phase). Discontinued but still in the field. This
+driver is an alias of the canonical `luxpower_sna` driver.
+"""
+from __future__ import annotations
+
+from . import ModelDriver
+from .luxpower_sna import create_driver as _sna
+
+
+def create_driver() -> ModelDriver:
+    drv = _sna()
+    drv.name = "eg4_3000ehv"
+    drv.label = "EG4 3000EHV-48"
+    return drv

collector/drivers/eg4_6500ex.py

@@ -0,0 +1,17 @@
+"""EG4 6500EX-48 inverter driver.
+
+The EG4 6500EX-48 is a legacy 6.5 kW off-grid inverter in the Luxpower SNA
+family (2-MPPT, single-phase). Discontinued but still in the field. This
+driver is an alias of the canonical `luxpower_sna` driver.
+"""
+from __future__ import annotations
+
+from . import ModelDriver
+from .luxpower_sna import create_driver as _sna
+
+
+def create_driver() -> ModelDriver:
+    drv = _sna()
+    drv.name = "eg4_6500ex"
+    drv.label = "EG4 6500EX-48"
+    return drv

collector/drivers/registry.py

@@ -7,6 +7,10 @@
 from . import ModelDriver
 from .eg4_6000xp import create_driver as _eg4_6000xp
 from .eg4_18kpv import create_driver as _eg4_18kpv
+from .eg4_12kpv import create_driver as _eg4_12kpv
+from .eg4_12000xp import create_driver as _eg4_12000xp
+from .eg4_6500ex import create_driver as _eg4_6500ex
+from .eg4_3000ehv import create_driver as _eg4_3000ehv
 from .luxpower_sna import create_driver as _luxpower_sna
 
 logger = logging.getLogger(__name__)
@@ -21,13 +25,25 @@
 # so would silently decode garbage. They are intentionally absent until a
 # dedicated driver with a validated map is written.
 DRIVERS: Dict[str, Callable[[], ModelDriver]] = {
+    # ── SNA family (capture-validated on the 6000XP) ──
+    # 2-MPPT, single-phase off-grid inverters. All share the exact same
+    # register family; the 6000XP is the validated reference.
     "eg4_6000xp": _eg4_6000xp,
     "luxpower_sna": _luxpower_sna,
+    "eg4_12000xp": _eg4_12000xp,
+    "eg4_6500ex": _eg4_6500ex,
+    "eg4_3000ehv": _eg4_3000ehv,
+
+    # ── 18KPV family (document-derived, NOT yet capture-validated) ──
+    # 3-MPPT, split-phase 120/240V, AFCI, generator input. The 18KPV map is
+    # derived from the official EG4 Modbus protocol doc; the 12kPV shares it.
     "eg4_18kpv": _eg4_18kpv,
-    # Rebadges of the Luxpower SNA that share the exact same register family.
-    "eg4_3000ehv": _luxpower_sna,
-    "eg4_6500ex": _luxpower_sna,
-    "luxpower_12k": _luxpower_sna,
+    "eg4_12kpv": _eg4_12kpv,
+
+    # ── FlexBOSS / GridBOSS ──
+    # NEW platform (not a Luxpower SNA rebadge). Register map unknown; do NOT
+    # alias to either family above. Intentionally absent until a dedicated
+    # driver with a validated map is written.
 }
 
 DEFAULT_MODEL = "eg4_6000xp"

collector/settings.py

@@ -227,13 +227,14 @@ def effective_value(name: str, db_value: Optional[str]):
         "section": "inverter",
         "options": [
             ("eg4_6000xp", "EG4 6000XP"),
+            ("eg4_12000xp", "EG4 12000XP"),
             ("eg4_18kpv", "EG4 18KPV"),
+            ("eg4_12kpv", "EG4 12kPV"),
             ("luxpower_sna", "Luxpower SNA"),
-            ("eg4_3000ehv", "EG4 3000EHV"),
-            ("eg4_6500ex", "EG4 6500EX"),
-            ("luxpower_12k", "Luxpower 12K"),
+            ("eg4_6500ex", "EG4 6500EX-48 (legacy)"),
+            ("eg4_3000ehv", "EG4 3000EHV-48 (legacy)"),
         ],
-        "hint": "Inverter model for protocol selection (Luxpower SNA family only)",
+        "hint": "Inverter model (Luxpower SNA / 18KPV families only)",
     },
     "pv_max_power": {
         "label": "Max Solar PV Power",

tests/test_drivers.py

@@ -9,6 +9,27 @@ def test_registry_has_sna_family():
     assert "eg4_6000xp" in DRIVERS
     assert "luxpower_sna" in DRIVERS
     assert "eg4_18kpv" in DRIVERS
+    assert "eg4_12kpv" in DRIVERS
+    assert "eg4_12000xp" in DRIVERS
+    assert "eg4_6500ex" in DRIVERS
+    assert "eg4_3000ehv" in DRIVERS
+
+
+def test_sna_aliases_share_register_family():
+    # 12000XP / 6500EX / 3000EHV are SNA rebadges → same map as 6000XP
+    base = get_driver("eg4_6000xp")
+    for model in ("eg4_12000xp", "eg4_6500ex", "eg4_3000ehv"):
+        d = get_driver(model)
+        assert d.input_registers is base.input_registers
+        assert d.batches == base.batches
+
+
+def test_12kpv_is_18kpv_family():
+    kpv = get_driver("eg4_18kpv")
+    pv = get_driver("eg4_12kpv")
+    assert pv.input_registers is kpv.input_registers
+    assert pv.batches == kpv.batches
+    assert pv.label == "EG4 12kPV"
 
 
 def test_unsupported_model_raises():