Skip to content

DALI: host capabilities for hosts that are not a controller - #1226

Closed
evgeny-boger wants to merge 1 commit into
masterfrom
split/dali-host-seams
Closed

DALI: host capabilities for hosts that are not a controller#1226
evgeny-boger wants to merge 1 commit into
masterfrom
split/dali-host-seams

Conversation

@evgeny-boger

Copy link
Copy Markdown
Member

Первый из серии PR, на которые разбит #1223 (он закрывается в пользу этих). Каждый — самостоятельная единица от master; остальные не зависят друг от друга, кроме отмеченного.

Что сделано

У страницы DALI второй дом — автономный WASM-редактор устройств — без syslog и без брокера, который видит кто-то ещё. daliHostCapabilities (stores/dali/host-capabilities.ts) — неглубокий набор флагов, которые такой хост выставляет до первого рендера:

  • syslogMonitor — переключатель «Сохранять в syslog» на вкладке шины показывается только там, где syslog есть;
  • externalBroker — редактор MQTT-id устройства скрывается там, где id никому не виден (демон оставляет id по умолчанию; схема демона его требует — прячется только редактор).

Копирование топика по клику на контрол и его тултип «скопировано» — за topicCopyPolicy, которую хост без брокера очищает; путь записи значения контрола сведён в один хелпер sendCellValueUpdate.

Контроллерный homeui сохраняет все умолчания — поведение не меняется.

Тесты

device-store-mqtt-id-hidden.test.ts — редактор id скрыт при externalBroker = false и не тронут иначе. tsc --noEmit, eslint, vitest по затронутым каталогам — чисто.

Связанные: wirenboard/wb-wasm-device-editor#96 (хост, который эти флаги выставляет).

🤖 Generated with Claude Code

The DALI page has a second home — the standalone WASM device editor — with
no syslog and no broker anyone else can see. daliHostCapabilities lets such
a host switch off what it cannot honour before the first render: the
"Save to syslog" toggle, and the MQTT id editor (the daemon keeps its default
id). Topic copy on cell click, and its "copied" tooltip, sit behind a policy
the same host clears; the cell value update path is one helper. Controller
homeui keeps every default.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@evgeny-boger

Copy link
Copy Markdown
Member Author

Эти швы (daliHostCapabilities, topicCopyPolicy, хелпер sendCellValueUpdate) нужны только автономному WASM-редактору устройств — на контроллере они ничего не меняют, отдельным PR им быть незачем.

Они уже целиком входят в #1223 (ветка feat/dali-device-controls), там им и место.

PR закрываю без мержа, ветку split/dali-host-seams оставляю. #1230 переделан так, чтобы не зависеть от этого PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant