zwave-js / zwave-js-ui / 22577597557

19%

push

github

fix(ui): show port field as disabled when controlled by ZWAVE_PORT env var (#4568)

- [x] Explore codebase to understand the issue
- [x] Modify Settings.vue to show port as disabled field with external
env var hint instead of hiding it when `ZWAVE_PORT` is set
- [x] Add `isPortManagedExternally` computed property to avoid repeating
the check
- [x] Run code review and addressed feedback
- [x] Fix lint issue: collapse `<v-col>` attrs to single line as
required by prettier

<!-- START COPILOT ORIGINAL PROMPT -->



<details>

<summary>Original prompt</summary>

> 
> ----
> 
> *This section details on the original issue you should resolve*
> 
> <issue_title>11.11+ tries to connect to TCP and no UI option exist to
change back to /dev/zwave</issue_title>
> <issue_description>### Checklist
> 
> - [x] I am not using Home Assistant. Or: a developer has told me to
come here.
> - [x] I have checked the troubleshooting section and my problem is not
described there.
> - [x] I have read the changelog and my problem is not mentioned there.
> 
> ### Deploy method
> 
> Kubernetes 
> 
> ### Z-Wave JS UI version
> 
> 11.11
> 
> ### ZwaveJS version
> 
> 15.20.1
> 
> ### Describe the bug
> 
> When changing to v11.11 it tries to connect to 
> ```
> 2026-02-28 11:22:25.857 WARN BACKUP: Nvm backup is disabled
> 2026-02-28 11:22:25.888 INFO Z-WAVE: Connecting to
tcp://10.99.134.31:80
> ```
> Swapping back to 11.10 and it connects again to `/dev/zwave` as
configured.
> 
> Also, in 11.11 there is no UI settings to change the "port" any more.
> 
> <img width="998" height="217" alt="Image"
src="https://github.com/user-attachments/assets/efbd9a20-99f3-4855-aea4-f68d4d3e1a0a"
/>
> 
> Compared to 11.10
> 
> <img width="944" height="127" alt="Image"
src="https://github.com/user-attachments/assets/2ea32341-87c9-47ca-97e5-0b4737644889"
/>
> 
> In settings.json, the port is still configured as `/dev/zwave` when
running 11.11.
> 
> When reloading the UI, and before "New version of page
a... (continued)

334 of 444 branches covered (75.23%)

Branch coverage included in aggregate %.

4052 of 22581 relevant lines covered (17.94%)

1.12 hits per line