ossia / score / 35347425313

42%

Build Type

push

github

Committed by jcelerier

Commit Message

bitfocus: reload the protocol settings of a saved device

The configuration a module was set up with never came back: the settings
widget only restored it when the settings still carried a live handler,
which never survives a save, and it appended the widget values to the
stored list instead of replacing them. A document therefore accumulated
duplicate keys whose trailing - empty - copy won in reconnect(), so a
configured webhook url was lost both in the dialog and in the module.

Restore the values once the module answers with its fields, whichever
path the settings came in through, and write each key once.
Deduplicating on load repairs the documents that already doubled.

The module path was also stored absolute, which only resolves on the
machine the device was created on: relativize it against the library
like every other file score references.

Along the way: keep the name of a device that already has one instead of
resetting it to the module's short name on every edit, keep the values of
fields the widget does not know how to render, and stop restarting the
module process when its own entry is picked again in the device list.

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

Coverage Stats

12 of 75 new or added lines in 4 files covered. (16.0%)

39 existing lines in 6 files now uncovered.

114929 of 287991 relevant lines covered (39.91%)

77343.85 hits per line