dangernoodle-io / TaipanMiner / 28250151820

90%

push

github

fix: fleet logs early-bail on no-data + single-worker --follow guard (TA-458) (#583)

* feat: add single_worker profile flag for esp32-s2/esp32-c3 (TA-458)

- add single_worker: bool field to Profile dataclass (default false)
- set single_worker=True in profile_for() for esp32-s2 and esp32-c3 prefixes
- mirror single_worker: true in config/profiles.yaml for both classes
- these single-core, heap-tight boards expose only one httpd worker; --follow
  can saturate it and block all other endpoints

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* fix: fleet logs early-bail on no-data + single-worker --follow guard (TA-458)

early-bail (all boards, all modes):
- add SSEIdleTimeout exception and SSE_IDLE_TIMEOUT=10.0s constant to sse.py
- stream_lines gains idle_timeout param (default SSE_IDLE_TIMEOUT); uses the
  shorter of idle_timeout/timeout as the socket read timeout so resp.read()
  does not block past the idle window
- on socket.timeout the idle deadline is checked; SSEIdleTimeout raised if expired
- any non-empty line (data: or comment) disables the idle check for that stream
- tail_lines passes idle_timeout=None to preserve TA-447 bounded-collect behavior
- _stream_one_device catches SSEIdleTimeout: prints clear stderr warning and
  returns 0 (graceful bail); for multi-host only the silent host disconnects

single-worker --follow warning:
- cmd_logs checks profile_for(d.board).single_worker when --follow is set
- prints prominent WARNING naming host, board, and recommending --lines/--duration
- proceeds after warning (warn-but-proceed, per user decision)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* test: offline tests for idle-bail and single-worker warn; docs (TA-458)

test_sse.py:
- TestStreamLinesIdleTimeout: idle fires when no data (socket.timeout mock),
  not fired when data arrives, disabled when idle_timeout=None, comment resets clock
- TestTailLinesNoIdleTimeout: tail_lines with idle_timeout=None never raises
- upda... (continued)

2886 of 3627 branches covered (79.57%)

Branch coverage included in aggregate %.

5385 of 5540 relevant lines covered (97.2%)

149095.41 hits per line