fix: add missing state_class and device_class to sensor entities

[michikrug]

Several sensor entities were missing state_class, preventing Home Assistant from recording long-term statistics and excluding them from the Energy Dashboard.

Affected sensors across b2500Base, b2500V1, and venus devices:

• Battery percentage / SoC — device_class: battery sensors were missing state_class: measurement, so they did not appear as selectable options in HA's Energy Dashboard battery configuration.
• Voltage sensors — cell voltages (min, max, diff, avg, individual cells) and solar input voltages had device_class: voltage but no state_class.
• Temperature sensors — temperature_min/temperature_max and BMS cell temperatures had device_class: temperature but no state_class.
• BMS physical measurements (venus) — voltage, current, temperature, charge voltage, and MOSFET temperature in the BMS field loop were missing state_class: measurement.
• Off-grid power (venus) — had device_class: apparent_power but was missing state_class: measurement.

The Jupiter device already had these correctly set and served as the reference. All added values are state_class: measurement (instantaneous, non-cumulative readings). No logic changes — discovery payload fields only.

Tested: manually published a modified MQTT discovery payload with state_class: measurement via MQTT Explorer and confirmed the entity appeared in HA Developer Tools with the correct attribute and became selectable in the Energy Dashboard.

AI Transparency Note: Change was prepared and implemented with Claude Code

Summary by CodeRabbit

• Improvements
  • Home Assistant sensors for battery level, capacity, SoC, and off‑grid power now use measurement state classification for more accurate telemetry and history.
  • Per‑cell voltages, per‑cell temperatures, and aggregate cell voltage metrics (min/max/avg/diff) now expose measurement classification and improved metadata for clearer display and recording.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 38ade9f4-79b6-4c6a-a0ae-9de2a31f935e

📥 Commits

Reviewing files that changed from the base of the PR and between e411523 and c2ec0d2.

📒 Files selected for processing (1)

• src/device/venus.ts

🚧 Files skipped from review as they are similar to previous changes (1)

• src/device/venus.ts

---

Walkthrough

This PR adds state_class: 'measurement' metadata to Home Assistant sensor configurations in B2500 Base, B2500 V1, and Venus device modules; Venus BMS field metadata also includes stateClass for conditional advertising.

Changes

B2500 Base, B2500 V1, and Venus Home Assistant Sensor Definitions

Layer / File(s)	Summary
B2500 Base device sensor definitions src/device/b2500Base.ts	Battery Percentage, Battery Capacity, Temperature Min/Max, Host Battery SoC, Extra Battery SoC (1 and 2), and cell-voltage aggregate (min, max, diff, avg) and per-cell sensors now include state_class: 'measurement'.
B2500 V1 device extra battery sensor src/device/b2500V1.ts	Extra battery input voltage sensor component now includes state_class: 'measurement'.
Venus device runtime sensors and BMS metadata src/device/venus.ts	Runtime sensors (batteryCapacity, batterySoc, offGridPower) and BMS per-cell voltage/temperature sensors now include state_class: 'measurement'. bmsFields mapping was extended with camel-cased stateClass: 'measurement' for multiple BMS properties and advertising sets state_class from info.stateClass when present.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• tomquist/hm2mqtt#309: Both PRs modify the bmsFields mapping and BMS sensor advertising logic in src/device/venus.ts.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and accurately summarizes the main changes: adding missing state_class and device_class fields to sensor entities for Home Assistant discovery across multiple device files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[tomquist]

Thank you