log_bold

Author: kellerza

.cursor/skills/release/SKILL.md

@@ -38,20 +38,27 @@ Run from the **sunsynk** repo root. Stop with a clear message unless **all** pas
 3. Use the new version string and change the `## Unreleased` heading in
    `hass-addon-sunsynk-edge/CHANGELOG.md` to `## Release x.x.x`
 4. Copy `hass-addon-sunsynk-edge/CHANGELOG.md` to `hass-addon-sunsynk-multi/CHANGELOG.md`
-5. Commit using the release number (`0.0.0` form) as the message and push to GitHub
-6. Create a draft GitHub release with tag `v0.0.0` (semantic version preceded by `v`)
-7. Notify the user to review and **publish** the draft release on GitHub to start the release
+5. In `www/docs/reference/multi-options.md`, remove every edge-only option prefix
+   `<i-mdi-dev-to class="vp-edge-option-icon" />` (list items and table cells). Leave the `::: tip`
+   block unchanged — it stays for future edge-only options.
+6. Commit using the release number (`0.0.0` form) as the message and push to GitHub
+7. Create a draft GitHub release with tag `v0.0.0` (semantic version preceded by `v`)
+8. Notify the user to review and **publish** the draft release on GitHub to start the release
    process
 
 ## Implementation notes
 
 - Read current version from `[project]` → `version = "..."` in `pyproject.toml`.
 - Heading match is case-insensitive for `## Unreleased`; write `## Release <new_version>` (no `v`
   prefix).
-- Stage only `pyproject.toml`, `hass-addon-sunsynk-edge/CHANGELOG.md`, and
-  `hass-addon-sunsynk-multi/CHANGELOG.md`. Include `uv.lock` if the version bump changed it. The
-  multi CHANGELOG copy is the one allowed hand-edit under `hass-addon-sunsynk-multi/` in this skill
-  (overwrite; do not merge).
+- Stage only `pyproject.toml`, `hass-addon-sunsynk-edge/CHANGELOG.md`,
+  `hass-addon-sunsynk-multi/CHANGELOG.md`, and `www/docs/reference/multi-options.md` (if the edge
+  prefixes were present). Include `uv.lock` if the version bump changed it. The multi CHANGELOG copy
+  is the one allowed hand-edit under `hass-addon-sunsynk-multi/` in this skill (overwrite; do not
+  merge).
+- When stripping edge prefixes, delete the HTML tag and the space after it so bullets stay
+  `- \`OPTION\`` (or the table cell starts with the backtick). Do not touch the tip, and do not
+  rewrite surrounding option text.
 - Do not copy `config.yaml` here. On a published release, `deployer2.yml` copies edge `config.yaml`,
   `translations/`, and `apparmor.txt` onto stable except `name`/`slug`, then sets `version` to the
   tag.

src/ha_addon_sunsynk_multi/a_inverter.py

@@ -183,9 +183,9 @@ def serial_matches_config(self) -> bool:
         expected_ser = self.opt.serial_nr.replace("_", "")
         actual_ser = str(self.identity.serial)
         if expected_ser != actual_ser:
-            raise ValueError(
-                f"Serial number mismatch. Expected {expected_ser}, got {actual_ser}"
-            )
+            msg = f"Serial number mismatch. Expected {expected_ser}, got {actual_ser}"
+            self.log_bold(msg)
+            raise ValueError(msg)
         return True
 
     async def read_sensors(self, *, sensors: Iterable[Sensor], msg: str = "") -> bool:
@@ -282,13 +282,7 @@ async def connect(self) -> None:
             ) from exc
 
         assert self.identity is not None
-        _LOG.info(
-            "Identity: device_type=%s (%s) protocol=%s serial=****%s",
-            self.identity.device_type,
-            hex(self.identity.device_type_code),
-            self.identity.protocol,
-            str(self.identity.serial)[-5:],
-        )
+        self.log_bold(str(self.identity))
         self.warn_device_type_config()
         self.serial_matches_config()
 

src/sunsynk/identity.py

@@ -39,6 +39,13 @@ class Identity(Component):
     protocol: str = NumberField(2, signed=False, convert=_protocol_version)  # type: ignore[assignment]
     serial: str = string(3, length=5)  # type: ignore[assignment]
 
+    def __str__(self) -> str:
+        """Return a string representation of the identity."""
+        return (
+            f"Identity: device_type={self.device_type} ({hex(self.device_type_code)}) "
+            f"protocol={self.protocol} serial=****{str(self.serial)[-5:]}"
+        )
+
 
 def suggested_sensor_definitions(device_type_code: int) -> str:
     """Return the SENSOR_DEFINITIONS profile for ``device_type_code`` (default single-phase)."""

src/tests/sunsynk/test_identity.py

@@ -37,6 +37,9 @@ async def test_identity_decode() -> None:
     assert identity.device_type == "Single-phase hybrid"
     assert identity.protocol == "1.2"
     assert identity.serial == "ABCDE12345"
+    assert str(identity) == (
+        "Identity: device_type=Single-phase hybrid (0x3) protocol=1.2 serial=****12345"
+    )
     assert suggested_sensor_definitions(3) == "single-phase"
     assert suggested_sensor_definitions(0x601) == "three-phase-hv"
     assert suggested_sensor_definitions(4) == "single-phase"