feat(device): «Устройство на плане» по референсу — серия 4 из четырёх (#600)

§7 SPEC.md референса. Пять карточек #598 остались, менялось содержимое; все ключи _markerDialog и все записи в них прежние (§7.2): - Basics — баннеры привязки стали callout'ами; Name с подсказкой «Shown on the plan»; привязка — сегмент [Virtual | From the HA list] (те же радио bmode) и кнопка выбора #marker-binding с панелью в потоке (sourcePicker набора), чекбокс «Show entities» — внутри панели рядом с поиском (решение владельца Q8); Room селектом с подсказкой «по зоне / переопределяет»; секции радара и пылесоса — своими модулями без изменений; - Tap action — селект с подсказкой Target/Now, «Ask for confirmation» — строка-тумблер только при toggle/run (Q8), Entity to toggle и What to run теми же условиями (список целей — панель набора), «Controls other light sources» — чипы набора и поиск с панелью в потоке; - Light and glow — «Include the device temperature…» строкой-тумблером; роль — сегмент [Auto | Always | Never] с подсказкой результата Auto; Leading light entity по условиям; блок свечения (.hpf-block): подзаголовок с «?», сегмент [From source | Set colour | Colour and brightness] с отключёнными вариантами и aria-describedby на объяснение, плашка цвета вокруг прежнего hp-color-opacity (Q5), яркость и радиус строками с числом и единицей, при Never блок неактивен целиком (aria-disabled) с callout'ом причины; - Appearance — поле иконки с превью, очисткой и ссылкой Pin в подсказке; Display селектом с подсказкой режима; пульс — плашка цвета и строка размера; Value source по условиям; подраздел Value badge — компактный тумблер, Value селектом с техническим идентификатором, Position сегментом [Above | Right | Below | Left]; предупреждения бейджа — заметки и callout'ы; Display preview — прежний hp-device-preview целиком в блоке на тинте с тегом Now (решение владельца Q3); «Icon size and rotation» — две строки range-line в сетке; - Details — Model и Link в ряд, Description textarea набора, Manuals чипами и строкой действий Attach…, «Additional actions» подзаголовком набора. Футер: Hide и Delete слева, статус, Cancel и Save справа. К10: снимок черновика при _openMarkerDialog (marker-form-state.ts поверх dialog-baseline; транзиентные ключи — раскрытие и фильтры списков, busy, объявление подсказки, autoIcon, uploadId); в edit Save только при изменениях, закрытие с изменениями спрашивает через _confirmDanger. Прежние запреты Save (busy, калибровка радара, привязка не выбрана/не подтверждена) остались; привязка и радиус свечения названы словами под полем (marker.error_binding, marker.error_glow_radius), «Review N fields» ведёт к полю. Тумблеры диалога — строки набора (нативный чекбокс, К7), а не ha-switch; слайдеры — прежний _rangeInput (ha-slider в HA). Набор: sourcePicker получил toolbar и disabled; сегмент — value у радио, disabled и describedBy у варианта; callout — role note и id; unitInput — describedBy; rangeLine — id; новые классы .hpf-block/.hpf-disabled, .hpf-toolbar/.hpf-check, .hpf-iconfield/.hpf-iconpreview, .hpf-mono, .hpf-note, .hpf-hint-line, textarea.hpf-input. settingsCopy вынесен в editors/settings-copy.ts (общий для пространства, комнаты, устройства). Смоки переписаны осознанно, с пометкой #600: binding_ui (#marker-binding, .hpf-panel, #marker-show-entities), tap_run (панель набора, #marker-tap- confirm), value_face_source (.hpf-cand.sel), static_icon (callout'ы), climate_temp (#marker-use-climate-temp), help_affordance (#marker-name, .hpf-block.hpf-disabled вместо fieldset[disabled]; второй Escape в грязном диалоге спрашивает — К10), size_angle_parity (сетка Appearance), color_picker_consumers (подписи полей), device_preview_parity (сегмент стороны), ha_controls (тумблеры набора — нативные чекбоксы по К7, ha-slider прежний). Golden-harness device — .hpf-field/.hpf-block, скроллер .content. Новый фокусный свидетель demo/smoke_device_settings_form.mjs: пять карточек без старой разметки, оболочка, привязка/панель/Show entities, Q8, сегменты и блок свечения, плашки, иконка, бейдж, превью на тинте, размеры, Details, цели 44, Save/dirty/вопрос при закрытии, create-режим, семь полос экранов. Мутанты: marker-save-enabled-without-changes, marker-discard-without-asking, marker-confirm-row-shown-for-do-nothing, marker-never-keeps-glow-block-live, marker-badge-position-forgets-touch; якоря same-binding-click-resets-source, state-callout-hidden-under-help, form-kit-segment-drops-radio-semantics переставлены на новую разметку. Контрактные тесты: color-picker (два поля набора вместо flex-строк), native-select-contract (сторона бейджа — сегмент), i18n dead-keys (34 → 38 подсказок), German=English (marker.name_label). Словарь i18n/settings: 21 ключ marker.* на четыре локали; в основном каталоге marker.name_label «Name», marker.size_label «Icon size and rotation», удалены мёртвые marker.room_override, marker.light_role_always/never. Фикстуры листа диалогов набора пересняты намеренно (панель — байт в байт, AC11). Бюджет: стартовый граф 290 489 (потолок 291 600 без изменений); ленивый редакторский 243 173 — потолок 240 100 → 243 900 с разбором в bundle-budget.mjs; старые стили диалога устройства снимаются в финале. Гейты: typecheck, build, bundle:sync (три копии байт в байт), npm test 2790+ pass по четырём частям, mutation-gate --check, 39 смоков диалога устройства + новый + три формы серий 1–3. Issue: #600 User-Visible: no
2026-10-06 22:49:16 +00:00 · 2026-09-20 12:18:12 +03:00
parent e2c0c7ed81
commit bf6602bbd8
84 changed files with 2912 additions and 2610 deletions
@@ -422,8 +422,15 @@ export const LAZY_FURNITURE_ART_GZIP_CEILING = 17_900;
 *   диалогом устройства в серии 4), поля границ комфорта с легендой, dirty и
 *   валидация комнаты, девять строк словаря на четыре локали.
 *   `room-temperature-controls.ts` удалён.
+ * - серия 4: 240 100 → 243 900 (замер 243 173). «Устройство на плане» по §7:
+ *   пять карточек контролами набора — сегменты привязки/роли/свечения/стороны
+ *   бейджа, кнопка выбора с панелью и чекбоксом внутри, чипы, плашки цвета,
+ *   поля с единицами, блок превью; 21 строка словаря на четыре локали
+ *   (четыре «?» с aria); в листе набора — блок с отключением, панель, поле
+ *   иконки, заметки. Старые стили диалога (`.markerlightgroup`, `.srcrow`,
+ *   `.colorrow`, `.ctrlchip*`) пока в eager-листе — снимаются в финале серии.
 */
-export const LAZY_EDITOR_GZIP_CEILING = 240_100;
+export const LAZY_EDITOR_GZIP_CEILING = 243_900;
 export const LAZY_GRAPH_CEILING_BAND = 2_000;

 /**
@@ -2957,11 +2957,11 @@ const MUTANT_DEFINITIONS = [
    because: 'a same-binding click silently wiping the configured value source is exactly '
      + 'the #385(a) bug — only the dialog smoke drives the real handler',
    patches: [{
-      // #592: диалог устройства переехал в свой модуль целиком и побайтово —
-      // изменился только путь, текст якоря прежний.
+      // #592: диалог устройства переехал в свой модуль; #600: выбор кандидата —
+      // обработчик onPick кнопки выбора набора, охрана та же.
      file: 'src/editors/marker-dialog.ts',
-      find: "                                if (c.value === d.binding) {\n                                  this.host._markerDialog = { ...d, bindingOpen: false };\n                                  return;\n                                }",
-      replace: "                                if (false) { return; }",
+      find: "            if (value === d.binding) {\n              this.host._markerDialog = { ...d, bindingOpen: false };\n              return;\n            }",
+      replace: "            if (value === d.binding && d.bindingOpen === false) {\n              this.host._markerDialog = { ...d, bindingOpen: false };\n              return;\n            }",
    }],
  },
  {
@@ -10263,9 +10263,10 @@ const MUTANT_DEFINITIONS = [
      + 'узнает о сломанном действии, только если сам откроет подсказку — а подставить туда '
      + 'идентификатор пропавшей цели вообще нечем',
    patches: [{
+      // #600: сообщение — callout набора в карточке Tap action; спрятать его в title так же легко.
      file: 'src/editors/marker-dialog.ts',
-      find: "                    ? html`<div class=\"rhint\">${this.host._t('marker.run_target_gone', { id: d.tapTarget })}</div>`",
-      replace: "                    ? html`<span title=${this.host._t('marker.run_target_gone', { id: d.tapTarget })}></span>`",
+      find: "                ? callout({ kind: 'warning', role: 'status', text: t('marker.run_target_gone', { id: d.tapTarget }) })",
+      replace: "                ? html`<span title=${t('marker.run_target_gone', { id: d.tapTarget })}></span>`",
    }, {
      // Оба сообщения о состоянии — один контракт (К5), поэтому и мутант один:
      // спрятать под «?» можно каждое, и «поймано» обязано означать оба.
@@ -10438,6 +10439,63 @@ const MUTANT_DEFINITIONS = [
      replace: "                        onInput: (raw) => { host._roomTempMax = raw; host.requestUpdate(); },",
    }],
  },
+  {
+    id: 'marker-save-enabled-without-changes',
+    guard: 'node demo/smoke_device_settings_form.mjs',
+    because: '#600 К10/AC6 (Q1) для «Устройства на плане»: в edit Save активен только при '
+      + 'изменениях; без dirty футер снова врёт о том, есть ли что сохранять',
+    patches: [{
+      file: 'src/editors/marker-dialog.ts',
+      find: "      && (edit ? dirty : true);",
+      replace: "      && (edit ? true : true);",
+    }],
+  },
+  {
+    id: 'marker-discard-without-asking',
+    guard: 'node demo/smoke_device_settings_form.mjs',
+    because: '#600 К10 (Q1): закрытие диалога устройства с изменениями обязано спросить. '
+      + 'Перевёрнутое условие закрывает «грязный» диалог молча и спрашивает у чистого',
+    patches: [{
+      file: 'src/editors/marker-dialog.ts',
+      find: "      if (!edit || !dirty) { forgetMarkerBaseline(this.host); this._closeMarkerDialog(); return; }",
+      replace: "      if (!edit || dirty) { forgetMarkerBaseline(this.host); this._closeMarkerDialog(); return; }",
+    }],
+  },
+  {
+    id: 'marker-confirm-row-shown-for-do-nothing',
+    guard: 'node demo/smoke_device_settings_form.mjs',
+    because: '#600 Q8 (решение владельца): «Ask for confirmation» есть только у действий '
+      + '(toggle/run); при «Do nothing» строке подтверждать нечего, и её показ — ложь формы',
+    patches: [{
+      file: 'src/editors/marker-dialog.ts',
+      find: "        ${effectiveTapAction === 'run' || effectiveTapAction === 'toggle'\n          ? toggleRow({",
+      replace: "        ${effectiveTapAction !== 'info'\n          ? toggleRow({",
+    }],
+  },
+  {
+    id: 'marker-never-keeps-glow-block-live',
+    guard: 'node demo/smoke_device_settings_form.mjs',
+    because: '#600 §7.1: при «Never» блок свечения неактивен целиком и объясняет почему. '
+      + 'Перепутать признак отключения с признаком пассивного источника — блок остаётся '
+      + 'живым на вид, а его контролы уже отключены',
+    patches: [{
+      file: 'src/editors/marker-dialog.ts',
+      find: "        <div class=\"hpf-block markerglowblock ${glowSourceDisabled ? 'hpf-disabled' : ''}\" aria-disabled=${glowSourceDisabled ? 'true' : 'false'}>",
+      replace: "        <div class=\"hpf-block markerglowblock ${passiveSource ? 'hpf-disabled' : ''}\" aria-disabled=${passiveSource ? 'true' : 'false'}>",
+    }],
+  },
+  {
+    id: 'marker-badge-position-forgets-touch',
+    guard: 'node demo/smoke_device_settings_form.mjs',
+    because: '#600 К1: сегмент стороны бейджа заменил select и обязан, как и он, ставить '
+      + 'valueBadgeTouched — иначе выбор человека проигрывает проекции превью при следующем '
+      + 'рендере и в конфиг не попадает',
+    patches: [{
+      file: 'src/editors/marker-dialog.ts',
+      find: "                  valueBadgePosition: position,\n                  valueBadgeTouched: true,",
+      replace: "                  valueBadgePosition: position,\n                  valueBadgeTouched: false,",
+    }],
+  },
  {
    id: 'form-kit-segment-drops-radio-semantics',
    guard: 'node --test test/form-kit.test.mjs',
@@ -10446,8 +10504,9 @@ const MUTANT_DEFINITIONS = [
      + 'чекбоксы выглядит идентично и молча ломает стрелки и объявление скринридером',
    patches: [{
      file: 'src/editors/form-kit.ts',
-      find: '      <input type="radio" name=${name} .checked=${option.value === value}',
-      replace: '      <input type="checkbox" name=${name} .checked=${option.value === value}',
+      // #600: у радиокнопки сегмента появился value — якорь дописан.
+      find: '      <input type="radio" name=${name} value=${option.value} .checked=${option.value === value}',
+      replace: '      <input type="checkbox" name=${name} value=${option.value} .checked=${option.value === value}',
    }],
  },
 ];