Handle multiple device updates in OTA check

[markvp]

ControllerCommandHandler.ts only handles the first device update when checking for OTA updates. Should handle the case where multiple devices are included.

Location: ControllerCommandHandler.ts:926

Extracted from TODO comment as part of #420.

[markvp]

@Apollon77 are you ok if I submit a PR for this?

[Apollon77]

Yes, but verify that it also works. code wise this should be fine and I somewhen also tested parallel updates in matter.js itself but mainly in test environments, so needs to be verified then with the server ... I assume HA will never send multiple updates

[markvp]

Implementation update: the fix for #determineCommissionOptions is done locally — QR codes with multiple *-separated payloads now commission each device sequentially, with node_added events emitted for each.

However, the WS protocol (commission_with_code) currently defines its response as a single MatterNode, which means the response only reports the first device. Additional nodes are communicated via events, but this is a protocol-level limitation.

Upstream coordination needed

This affects three repos:

Repo	What needs to change
matter-js/python-matter-server	Server + client library response type
matter-js/matterjs-server	WS protocol types + ws-client (this repo)
home-assistant/core	Matter integration consumer (homeassistant/components/matter/api.py)

Logged upstream: matter-js/python-matter-server#1366
Will also file against home-assistant/core for the downstream consumer.

[markvp]

@Apollon77 Change requests have been filed across all three affected repos:

• commission_with_code: support multiple devices in a single QR code python-matter-server#1366 (upstream protocol)
• Matter: commission_with_code may produce multiple nodes from a single QR code home-assistant/core#168174 (downstream consumer)

Would you like me to pursue PRs for these, or would you prefer to handle the coordination?

[Apollon77]

Please do not mix different ideas and tasks into one. the OTA change has nothing to do with the Commissioning topic. Please do separately. I also commented for the multi qr code because it will likely not work that way

[markvp]

it was because that was where the todo was...

https://github.com/matter-js/matterjs-server/pull/532/changes#diff-77f244b9aa87cba248ac658fa5e6608509141a9bec84de6ca0d7400f6a325715L875

[Apollon77]

@markvp yes but this has nothing to do with OTA!

[markvp]

Yah I totally get that, I'm just saying I missed the OTA part and saw the multiple devices part.. sorry...