Intercept node name and location changes from outside UI

[spudwebb]

I'm creating a ZWave driver for HomeSeer based on ZWave JS UI and using the ZWaveJS.NET library to communicate with the ZWave JS server.
I want to be able to update the name and location of a node so that when those values are updated within HomSseer they are also updated in ZWaveJS UI.
To do that I tried to use the ZWaveNode.SetName() and ZWaveNode.SetLocation() methods of the .NET library which correctly set the values in the ZWaveJS cache (in store/xxxxxxxx.values.jsonl), but it seems that ZWave JS UI completely ignore those values and uses its own instances of name and location that are saved in store/nodes.json.

So my question is: is there a way to update the name and location of a node displayed by ZWave JS UI using a command sent to the ZWave JS server?
Alternatively would it be possible to modify ZWave JS UI behavior so that either
(a) if there is no name/location in nodes.json then it uses the name and location from the ZWave JS cache
or
(b) add a setting that will force ZWave JS UI to always use the name and location from the ZWave JS cache

Related discussion zwave-js/ZWaveJS.NET#60

[robertsLando]

@spudwebb Problem is that actually I have no way to detect that node name and location changed on driver so that's why you don't see it there, if you restart the application you probably will see it correctly

[spudwebb]

@robertsLando
The driver uses the ValueID from the "Node Naming and Location" command class even if the node does not supports this class, so when I update the values the following events are emitted by the driver:

2024-02-09 09:00:16.640 INFO Z-WAVE: [Node 005] Value added 5-119-0-name => My Test Dimmer
2024-02-09 09:00:16.641 INFO Z-WAVE: [Node 005] Value updated: 119-0-name MyDimmer => My Test Dimmer

and it seems that ZWave JS UI get this info because it creates a "Node Naming and Location v0" CC with the name and location values:

but after a restart of the application you can see that this CC is gone, and the name and location fields are still not populated:

The device used for these screenshots does NOT support the "Node Naming and Location" command class

[robertsLando]

The driver uses the ValueID from the "Node Naming and Location" command class even if the node does not supports this class

I didn't know about that. If so I could certainly catch those events to upate cached node name and location. Let me get a confirmation from @AlCalzone

[AlCalzone]

That's correct. The bit about the value IDs vanishing is awkward though, I may have to rethink that.

[robertsLando]

@spudwebb Could you give a try to #3592?

[spudwebb]

@robertsLando I tested the SYNC-NODE-NAME-LOC branch, and the fix only works for node that supports the "Node Naming and Location" CC. Name and Location of nodes that do not support it remain blank after a name or location update on the driver.

[robertsLando]

@AlCalzone What is the behaviour when node doens't support name/location CC? Do you still list that cc in defined valueIds or do you emit the 'node added' event? If none of this that's the problem, I actually subscribe valueId observers on node ready and on value added events, if you only emit value changed I don't subscribe to it

[AlCalzone]

getDefinedValueIDs should only return value IDs for CCs that are supported by the device.

I think you should ignore the Node Naming and Location CC in the list of value IDs (since you already have a dedicated UI for those properties) and instead read node.name / node.location for populating and subscribe to value added/updated/removed events for NodeNamingAndLocationCCValues.name.id and NodeNamingAndLocationCCValues.location.id for updating the naming UI.