Skip to content

EEBus: fix GridGuard entity type removed from LPC/LPP - #31568

Merged
andig merged 2 commits into
masterfrom
fix/eebus-gridguard-31561
Jul 7, 2026
Merged

EEBus: fix GridGuard entity type removed from LPC/LPP#31568
andig merged 2 commits into
masterfrom
fix/eebus-gridguard-31561

Conversation

@andig

@andig andig commented Jul 7, 2026

Copy link
Copy Markdown
Member

fixes #31561

Bumps the enbility/eebus-go replace to a new evcc-io/eebus-go branch (rollup-gridguard) that adds enbility/eebus-go#235 on top of the current pin. That upstream PR restores EntityTypeTypeGridGuard to the CsLPC/CsLPP valid entity types, which enbility/eebus-go#224 (pulled in via #31550) had accidentally dropped. Without it, a ControlBox's GridGuard entity was silently discarded, CsLPC never bound to it, HEMS never received a heartbeat, and the LPC failsafe tripped deterministically about two minutes after boot.

  • go.mod/go.sum: enbility/eebus-go replace now points at evcc-io/eebus-go@rollup-gridguard

🤖 Generated with Claude Code

@andig andig added the bug Something isn't working label Jul 7, 2026
@andig andig changed the title EEBus: bump eebus-go to restore GridGuard entity type in CsLPC/CsLPP EEBus: fix GridGuard entity type removed from LPC/LPP Jul 7, 2026
@andig
andig enabled auto-merge (squash) July 7, 2026 17:11

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@andig
andig disabled auto-merge July 7, 2026 17:16
@andig

andig commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

Added TestControlBoxGridGuardHeartbeat (server/eebus/test/gridguard_test.go): a ControlBox test double exposing only a GridGuard entity connects, and the test asserts CsLPCInterface.IsHeartbeatWithinDuration() becomes true. Verified it fails deterministically against the pre-fix eebus-go commit (heartbeatDiag never gets set because deviceConnected filters the ControlBox's DeviceDiagnosis entity out via validEntityTypes) and passes against the fix.

🤖 Generated with Claude Code

@andig
andig enabled auto-merge (squash) July 7, 2026 17:31
@andig
andig merged commit 82a3f64 into master Jul 7, 2026
9 checks passed
@andig
andig deleted the fix/eebus-gridguard-31561 branch July 7, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

controlbox: missing heartbeat- entering failsafe mode

1 participant