fix(sunsynk,deye): own the power sign flags instead of inheriting them

[springfall2008]

The bug

base.args is a single shared dict and is not namespaced per inverter type, so whichever component sets grid_power_invert sets it for every inverter index on the install.

teslemetry.py:646 and fox.py:2235 both set it True for their own hardware — quite correctly. Sunsynk and Deye never claimed the key at all, so on an install running either of those alongside them, they inherited the flip:

sensor.predbat_sunsynk_<sn>_grid_power  = +506  (exporting, correct)
  → inverter.grid_power                 = +506
  → grid_power_invert, left True by another component → −506
  → Predbat reads negative as importing → power-flow arrow reversed

Verified live that the component itself publishes the correct sign — raw pac −506 with the day's counters at 0.0 kWh imported against 10.7 kWh exported, published as +506, which is Predbat's documented convention (negative import, positive export). So the sensor was right and the flag was flipping it afterwards.

The fix

Both components now set grid_power_invert, battery_power_invert and load_power_invert explicitly to False in automatic_config.

Setting them rather than relying on the default is the whole point: a default only applies when nothing else has written the key, which is exactly the case that was failing.

False is right for all three because publish_data already emits Predbat's conventions — grid negative on import (SUNSYNK_TELEMETRY_NEGATE / DEYE_TELEMETRY_NEGATE), battery positive on discharge, load positive — each confirmed against live hardware.

Testing

A regression test per component asserting all three flags are claimed with one entry per inverter, so neither can silently go back to inheriting. Full --quick suite green, pre-commit clean.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR fixes a cross-component configuration leak where grid_power_invert/battery_power_invert/load_power_invert could be unintentionally inherited via the shared base.args dict, causing Sunsynk and Deye power-flow signs to be flipped after publishing already-correct sensor values.

Changes:

• Explicitly sets grid_power_invert, battery_power_invert, and load_power_invert to False in automatic_config() for Sunsynk and Deye so they always “own” these flags.
• Adds regression tests for both components to ensure all three flags are always set and are per-inverter lists (one entry per inverter index).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
apps/predbat/sunsynk.py	Sets all three invert flags explicitly to False during Sunsynk auto-discovery to prevent inherited sign flips.
apps/predbat/deye.py	Sets all three invert flags explicitly to False during Deye auto-discovery to prevent inherited sign flips.
apps/predbat/tests/test_sunsynk_config.py	Adds a regression test asserting the invert flags are claimed and correctly sized per inverter.
apps/predbat/tests/test_deye_publish.py	Adds a regression test asserting the invert flags are claimed and correctly sized per inverter.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.