dangernoodle-io / TaipanMiner / 25648691477

90%

push

github

refactor(webui): eliminate Svelte 5 binding_property_non_reactive warnings (#388)

* refactor(webui): split PoolEditForm form into per-field bindables

Eliminates Svelte 5 binding_property_non_reactive warnings caused by
bind:value={form.host} dotted access. Each field is now its own
$bindable prop end-to-end (poolState → PoolEditDialog → PoolEditForm).

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

* refactor(webui): use value/onchange instead of bind: against state-machine accessors

Svelte 5 emits binding_property_non_reactive when bind:value is used against
a getter/setter accessor on a plain object (state machines like ws/state
returned by createWifiSetupState/createFlashState). Setters do thread to the
underlying $state cell at runtime, but Svelte's static check can't verify
reactivity, so it warns on every mount.

- WifiSetup: 6 input bindings + WifiSelect prop converted to value/oninput
  and selected/onselect (matches the existing hostname/worker pattern in the
  same file)
- WifiSelect: drop selected $bindable; expose onselect callback
- flasher App: convert Select binding to value/onchange routed through
  state.selectBoard
- Select: drop value $bindable; expose onchange callback

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

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

2381 of 3015 branches covered (78.97%)

Branch coverage included in aggregate %.

82 of 82 new or added lines in 8 files covered. (100.0%)

4643 of 4811 relevant lines covered (96.51%)

171637.74 hits per line