Repository navigation
deps: floor ebus-mqtt-client at 0.6.0 - #86
Merged
Merged
Conversation
0.6.0 holds every publish issued while the link is down and flushes it before the SDK's on-connect republish. 0.5.0 left QoS 1 and 2 publishes to paho, which replayed them after that republish and past mosquitto's 20-message receive quota; mosquitto discarded the excess QoS 2 messages, so simple-device and simple-tree-device ended with the root's retained $state on init instead of ready (electrification-bus/ebus-mqtt-client#20). tests/conftest.py: the mocked paho now delivers CONNACK when the network loop starts, as real paho does. From 0.6.0 a publish reaches paho only after on_connect has flushed the hold, so without it four tests saw no publishes. The suite passes with the fixture against both 0.5.0 and 0.6.0. Also: CHANGELOG [Unreleased] compare link pointed at v0.20.1; the mdns extra is used by utility-meter --discover as well. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01T32pvgqCzp1qCz7K5gS9Wm
dcj
added a commit
that referenced
this pull request
Oct 4, 2026
Bumps __version__ to 0.24.0, promotes Unreleased to [0.24.0], adds the tag link definition. #86 Floor ebus-mqtt-client at 0.6.0, which holds every publish issued while the link is down and flushes it before the SDK's on-connect republish. With 0.5.0, simple-device and simple-tree-device ended with the root's retained $state on init instead of ready. #87 Device.as_dict() raised TypeError for any device with a node. Minor rather than patch: the floor changes what a construction-time publish does on the wire, the same reasoning that made ebus-mqtt-client 0.6.0 minor. Also in this release: examples documentation corrections (#89). Claude-Session: https://claude.ai/code/session_01T32pvgqCzp1qCz7K5gS9Wm Co-authored-by: Claude Opus 5.5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Raises the
ebus-mqtt-clientfloor to 0.6.0. That release holds every publish issued while the link is down and flushes it before the SDK's on-connect republish.Why the floor matters here: a root built with
mqtt_cfg=publishes during construction, before CONNACK. With 0.5.0, QoS 1 and 2 publishes from that window were left to paho. paho replayed them after the republish and past mosquitto's 20-message receive quota, and mosquitto discarded the excess QoS 2 messages. As a result,simple-deviceandsimple-tree-deviceended with the root's retained$stateoninitinstead ofready(electrification-bus/ebus-mqtt-client#20). With 0.6.0 both endready. Broker quota drops fall from 3, 48 and 201 (simple-device, simple-tree-device, utility-meter) to 0.tests/conftest.py: the mocked paho now delivers CONNACK when the network loop starts, as real paho does. Without it, four tests see no publishes under 0.6.0. The suite passes with the fixture against both 0.5.0 and 0.6.0: 706 tests, with 0.6.0 installed from PyPI into a fresh venv.Also fixed:
[Unreleased]compare link pointed at v0.20.1.mdnsextra is also used byutility-meter --discover.🤖 Generated with Claude Code
https://claude.ai/code/session_01T32pvgqCzp1qCz7K5gS9Wm