Add connect/disconnect status for real heater/stirrer hardware #3

2026-07-03T19:21:17+02:00

jens commented

2026-07-03 19:21:17 +02:00

Heater and Stirrer can be real serial hardware (hendi, Pololu1376), but there was no connection concept at all - the constructors opened the port and crashed the whole server if the device was missing, with no way to see connection status or firmware version and no way to reconnect without a restart.

Adds an observable Connectable mixin (components/connectable.py) shared by AHeater/AStirrer; real devices defer opening the serial port to an explicit connect(), auto-connect on server startup, and surface Connected/FirmwareVersion/Simulated plus manual Connect/Disconnect over the web GUI. Heating/stirring and Sud Start are all gated on connection state, and a disconnect mid-brew force-stops the run via the same path as a manual Stop.

Test plan

Ran server with sim config: both panels show Connected, Start/ramp works as before.
Ran server with a config pointing heater/stirrer at nonexistent serial ports: server starts without crashing, panels show Disconnected, Start is refused with an error.
Manual Connect/Disconnect commands round-trip correctly over the websocket.

Heater and Stirrer can be real serial hardware (hendi, Pololu1376), but there was no connection concept at all - the constructors opened the port and crashed the whole server if the device was missing, with no way to see connection status or firmware version and no way to reconnect without a restart. Adds an observable `Connectable` mixin (`components/connectable.py`) shared by `AHeater`/`AStirrer`; real devices defer opening the serial port to an explicit `connect()`, auto-connect on server startup, and surface Connected/FirmwareVersion/Simulated plus manual Connect/Disconnect over the web GUI. Heating/stirring and Sud Start are all gated on connection state, and a disconnect mid-brew force-stops the run via the same path as a manual Stop. ## Test plan - [x] Ran server with sim config: both panels show Connected, Start/ramp works as before. - [x] Ran server with a config pointing heater/stirrer at nonexistent serial ports: server starts without crashing, panels show Disconnected, Start is refused with an error. - [x] Manual Connect/Disconnect commands round-trip correctly over the websocket.

jens added 2 commits 2026-07-03 19:21:18 +02:00

refactor: make ATemperatureSensor's temp observable directly on the sensor 698c019581

TempSensorSim/TempSensor_max31865's temperature() now stores its
reading on self.temp, so ATemperatureSensor's inherited AttributeChange
(previously never triggered by anything) actually fires. TempSensorTask
no longer keeps its own shadow copy of the reading - it registers its
websocket-push callback on self.sensor directly and just drives the
read each tick; server/brewpi.py's TC-feeding registration moved from
sensor_task to sensor for the same reason.

Priming read happens before registering the callback (not after) since
all tasks are built synchronously at module level, before the asyncio
event loop starts - registering first would fire on_temp_changed's
asyncio.create_task() with no running loop.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GpePKZiEZWbGo9HrfuML6U

feat: add connect/disconnect status for real heater/stirrer hardware f20e617d81

Heater and Stirrer can be real serial hardware (hendi, Pololu1376), but
there was no connection concept at all - the constructors opened the
port and crashed the whole server if the device was missing, with no
way to see connection status or firmware version and no way to
reconnect without a restart.

Adds an observable Connectable mixin (components/connectable.py) shared
by AHeater/AStirrer; real devices defer opening the serial port to an
explicit connect(), auto-connect on server startup, and surface
Connected/FirmwareVersion/Simulated plus manual Connect/Disconnect over
the web GUI. Heating/stirring and Sud Start are all gated on connection
state, and a disconnect mid-brew force-stops the run via the same path
as a manual Stop.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YaPLuRPpyjWcwhMvCvpHCL

jens closed this pull request

2026-07-03 19:24:31 +02:00

jens reopened this pull request

2026-07-03 19:25:56 +02:00

jens merged commit 1db0e66ab7 into master

2026-07-03 19:26:18 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jens/brewpi#3