docs: review document for #199

Issue: #199 User-Visible: no
feat: block unsafe Optimize geometry
2026-09-29 03:09:36 +00:00 · 2026-08-22 13:44:26 +00:00 · 2026-08-22 16:29:33 +03:00 · 2026-08-22 13:12:13 +00:00 · 2026-08-22 16:08:04 +03:00 · 2026-08-22 13:07:27 +00:00
483 changed files with 66370 additions and 3646 deletions
@@ -132,9 +132,17 @@ jobs:
      - name: Install candidate and baseline dependencies
        run: npm ci --prefix candidate && npm ci --prefix baseline

+      # То же, что в validate.yml: кэш браузеров, apt не трогаем (#206).
+      - name: Кэш браузеров Playwright
+        id: pw
+        uses: actions/cache@v6
+        with:
+          path: ~/.cache/ms-playwright
+          key: playwright-${{ runner.os }}-${{ hashFiles('candidate/package-lock.json') }}
      - name: Install pinned Chromium
+        if: steps.pw.outputs.cache-hit != 'true'
        working-directory: candidate
-        run: npx playwright install --with-deps chromium
+        run: npx playwright install chromium

      - name: Build both exact source trees
        run: |
@@ -39,6 +39,7 @@ jobs:
    outputs:
      stage: ${{ steps.decide.outputs.stage }}
      cycle: ${{ steps.decide.outputs.cycle }}
+      spent: ${{ steps.decide.outputs.spent }}
      limit: ${{ steps.decide.outputs.limit }}
    steps:
      - id: decide
@@ -63,19 +64,41 @@ jobs:
          limit=4
          if [ "$SMALL" = "true" ] || [ "$TRIVIAL" = "true" ]; then limit=2; fi

-          # Счётчик считает вердикты ТОЛЬКО своего этапа. Раньше он брал все
-          # подряд, и вердикт по ТЗ съедал цикл из бюджета код-ревью: на #89
-          # первое код-ревью получило r2/4. На задаче с двумя циклами ТЗ второе
-          # код-ревью упиралось бы в review-4 после одной правки.
+          # Считаются ДВЕ РАЗНЫЕ величины, и это не педантизм (#227).
          #
-          # Этап опознаётся по имени документа в теле комментария. Если документа
-          # нет, вердикт не посчитается — недосчёт даёт лишний цикл, а перерасчёт
-          # остановил бы работу досрочно; из двух ошибок выбрана обратимая.
-          done_cycles=0
+          # `attempt` — сколько раз ревью уже отработало на этом этапе. Он нужен
+          # только для имени документа и метки: два захода с одинаковым номером
+          # означают, что второй документ перезапишет первый и артефакт ревью
+          # исчезнет.
+          #
+          # `spent` — сколько циклов израсходовано из бюджета §4. Цикл — это
+          # «отправка на ревью → вердикт с блокирующими находками → возврат
+          # автору», поэтому бюджет тратят ТОЛЬКО жёлтые и красные вердикты.
+          # Зелёный ничего на правки не вернул и цикла не образует.
+          #
+          # Раньше обе роли исполнял один счётчик всех вердиктов, и конвейер
+          # наказывал за то, что предписывал сам: при неудавшемся слиянии он
+          # велит вернуть S7-code-review после ребейза, и этот заход добивал
+          # бюджет. На #225 (лёгкий трек, лимит 2) последовательность
+          # жёлтый → зелёный → ребейз дала review-4 на задаче с зелёным ревью и
+          # зелёным CI: работа встала, хотя после вердикта не было ни одной
+          # правки продуктового кода.
+          #
+          # Вердикты считаются ТОЛЬКО своего этапа: иначе вердикт по ТЗ съедал
+          # цикл из бюджета код-ревью (#89 получило r2/4). Этап опознаётся по
+          # имени документа в теле комментария; документа нет — вердикт не
+          # посчитается. Недосчёт даёт лишний заход, перерасчёт остановил бы
+          # работу досрочно: из двух ошибок выбрана обратимая.
+          attempt=1; spent=0; spent_list=""
          if [ -n "$stage" ]; then
-            done_cycles=$(gh issue view "$NUM" --repo "${{ github.repository }}" \
-              --json comments \
-              -q "[.comments[] | select(.body | test(\"Вердикт:\")) | select(.body | test(\"$marker\"))] | length")
+            comments=$(gh issue view "$NUM" --repo "${{ github.repository }}" --json comments)
+            of_stage="[.comments[] | select(.body | test(\"Вердикт:\")) | select(.body | test(\"$marker\"))]"
+            # Блокирующим считается вердикт, у которого в строке вердикта стоит
+            # «жёлтый» или «красный». Регистр и окружение слова не важны.
+            blocking="$of_stage | map(select(.body | test(\"Вердикт:[^\\n]*(жёлт|красн)\"; \"i\")))"
+            attempt=$(( $(printf '%s' "$comments" | jq -r "$of_stage | length") + 1 ))
+            spent=$(printf '%s' "$comments" | jq -r "$blocking | length")
+            spent_list=$(printf '%s' "$comments" | jq -r "$blocking | map(\"- \" + .url) | join(\"\\n\")")
          fi

          # Отказ обязан быть виден в issue, а не только в логе прогона.
@@ -109,19 +132,33 @@ jobs:
            refuse "стоит blocked — конвейер не запускается" \
              "на issue стоит \`blocked\` — задача ждёт внешнего решения. Снять метку, когда решение принято."
          elif [ "$EXHAUSTED" = "true" ]; then
+            # Метку снимает владелец, а не конвейер: автоматика, отменяющая
+            # остановку работы, дороже ручного снятия. Но пересчёт печатается —
+            # метка могла остаться от прежнего правила, когда бюджет тратил и
+            # зелёный вердикт (#227).
+            stale=""
+            if [ "$spent" -lt "$limit" ]; then
+              stale=" Пересчёт по действующему правилу: блокирующих циклов $spent из $limit — метка могла остаться от прежнего правила, когда бюджет тратил любой вердикт. Снять её может владелец."
+            fi
            refuse "стоит review-4 — решение за владельцем" \
-              "на issue стоит \`review-4\`: лимит циклов ревью исчерпан, дальше решает владелец — разделить задачу, отклонить или арбитраж (PROCESS.md §4)."
-          elif [ "$done_cycles" -ge "$limit" ]; then
-            echo "циклов этапа $stage пройдено $done_cycles из $limit — лимит исчерпан"
+              "на issue стоит \`review-4\`: лимит циклов ревью исчерпан, дальше решает владелец — разделить задачу, отклонить или арбитраж (PROCESS.md §4).$stale"
+          elif [ "$spent" -ge "$limit" ]; then
+            echo "блокирующих циклов этапа $stage: $spent из $limit — лимит исчерпан"
            gh issue edit "$NUM" --repo "${{ github.repository }}" --add-label review-4
+            # Перечень учтённого обязателен: иначе владельцу приходится читать
+            # всю ленту, чтобы понять, из чего сложился счёт.
            gh issue comment "$NUM" --repo "${{ github.repository }}" --body \
-              "Лимит циклов ревью исчерпан ($done_cycles из $limit на этапе \`$stage\`). Пятого захода нет: решение владельца — разделить задачу, отклонить или арбитраж (PROCESS.md §4)."
+              "Лимит циклов ревью исчерпан: блокирующих циклов $spent из $limit на этапе \`$stage\` (заход $attempt). Следующего захода нет: решение владельца — разделить задачу, отклонить или арбитраж (PROCESS.md §4).
+
+          Учтены вердикты с блокирующими находками — зелёные бюджет не тратят:
+          $spent_list"
            stage=""
          else
-            echo "этап $stage, цикл $((done_cycles + 1)) из $limit"
+            echo "этап $stage, заход $attempt, блокирующих циклов $spent из $limit"
          fi
          echo "stage=$stage" >> "$GITHUB_OUTPUT"
-          echo "cycle=$((done_cycles + 1))" >> "$GITHUB_OUTPUT"
+          echo "cycle=$attempt" >> "$GITHUB_OUTPUT"
+          echo "spent=$spent" >> "$GITHUB_OUTPUT"
          echo "limit=$limit" >> "$GITHUB_OUTPUT"

  review:
@@ -218,6 +255,10 @@ jobs:
      - name: Review
        id: review
        uses: anthropics/claude-code-action@v1
+        env:
+          # Вне рабочей копии: восстановление дерева ревьюером не должно
+          # уничтожать его собственный артефакт (#220).
+          REVIEW_DOC: ${{ runner.temp }}/review-document.md
        with:
          # Подписка, а не отдельный счёт API: токен выпускается через
          # `claude setup-token` (Pro/Max). Действуют лимиты подписки.
@@ -230,6 +271,42 @@ jobs:
            Этап: ${{ needs.guard.outputs.stage }}
              spec — ревью ТЗ (PROCESS.md §2.4)
              code — код-ревью (PROCESS.md §2.7)
+            Заход: r${{ needs.guard.outputs.cycle }} · блокирующих циклов израсходовано ${{ needs.guard.outputs.spent }} из ${{ needs.guard.outputs.limit }}
+              Бюджет §4 тратят только жёлтые и красные вердикты: зелёный
+              ничего не вернул на правки и цикла не образует (#227).
+              Номер захода нужен для имени документа — два документа с
+              одинаковым номером затёрли бы друг друга.
+
+            **Если цикл не первый — объём разбора по дельте, а не заново**
+            (PROCESS.md §2.9, issue #214). Раньше промпт был одинаковым для
+            всех раундов, и повторный цикл заново выводил продуктовую рамку и
+            перепроверял AC, которых правка не касалась: r2 по #150 стоил
+            полного прогона ради одной строки в тестовой фикстуре.
+
+            Порядок для r2 и дальше:
+              1. найди вердикт предыдущего раунда в комментариях issue и SHA,
+                 на котором он получен. SHA в вердикте не назван — это находка;
+              2. объяви дельту: `git diff <тот SHA>..HEAD` для кода, дифф файла
+                 ТЗ или тела issue для spec. Дельта — предмет этого раунда;
+              3. по каждой находке предыдущего раунда покажи, чем именно она
+                 закрыта: строка кода или текста, а не заявление автора;
+              4. заново проверяй только те AC, чьё доказательство дельта
+                 задевает. Остальные наследуй;
+              5. в документе обязателен раздел «Унаследовано из r<N-1>»: что
+                 принято без повторной проверки, со ссылкой на документ того
+                 раунда и SHA, на котором вывод получен. Без этого перечня
+                 сокращение — молчаливое доверие, а такой тихий успех уже
+                 дважды стоил дня (#171, #207).
+
+            Разбор остаётся ПОЛНЫМ, если дельта не локальна: ребейз на ушедший
+            вперёд dev (после ребейза это другой код, §7.2), смена контракта
+            поведения, задета новая подсистема, либо объём дельты сопоставим с
+            исходной задачей. Сомневаешься — разбирай полностью и скажи почему.
+
+            Сокращается объём РАЗБОРА, а не строгость: правка по замечанию
+            способна сломать AC, который предыдущий раунд признал выполненным —
+            так появилась регрессия #102. Поэтому граница не «только находки», а
+            «находки плюс всё, до чего дотягивается дельта».

            Прочитай в этом порядке, прежде чем судить:
              1. docs/SCOPE.md — зачем продукт существует и для кого. Он
@@ -273,14 +350,23 @@ jobs:
            правке — не тщательность, а потеря времени: полные наборы это
            предрелизный гейт (PROCESS.md §8), а не гейт ревью.

-            Всегда, они дешёвые:
+            Всегда, они дешёвые, и в повторном раунде тоже: код изменился,
+            а стоят они минуты:
              `npx tsc --noEmit`, `npm test`, `npm run build` со сверкой трёх
-              копий бандла.
+              копий бандла. Плюс `node scripts/check-docs.mjs`, если diff трогает
+              `src/**`: отпечаток скриншотов документации считается по всему
+              `src/**`, поэтому любая правка фронтенда делает его устаревшим —
+              выбирать тут нечего. Пропуск этого шага в #230 и #234 оставил `dev`
+              с красным job `docs` до следующей задачи (#237).

            По необходимости, и «необходимость» определяется diff'ом и AC:
              - браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс
-                относящиеся к тронутым поверхностям. Их 127; прогон всех уместен
-                только когда задача действительно задевает всё;
+                относящиеся к тронутым поверхностям. Их 163; прогон всех уместен
+                только когда задача действительно задевает всё. Выбирать по теме
+                недостаточно: регресс #234 поймал `smoke_wall_junctions`, который
+                по названию про стыки стен, а не про толщину отрезка. Надёжнее
+                grep — искать в `demo/smoke_*.mjs` имена изменённых функций и
+                полей, и прогонять найденное;
              - `npm run golden:verify` — если diff может изменить видимый
                результат: рендер, геометрия, стили, слои;
              - `python -m pytest tests_backend -q` — если тронут
@@ -299,27 +385,46 @@ jobs:

            Ты НЕ правишь ни ТЗ, ни продуктовый код. Только оцениваешь.

-            Серьёзность: High блокирует; Medium обязан стать отдельным issue;
-            Low либо правится, либо снимается с записью. Жёлтый вердикт
-            допустим при полностью выполненных AC, если изменение не решает
-            заявленный сценарий или ухудшает смежный. Продуктовое рассуждение
-            расширяет вопросы, но не отменяет AC и не даёт права менять скоуп.
+            Серьёзность: High блокирует; Medium В СКОУПЕ задачи чинится в ней
+            же — без High это жёлтый вердикт и возврат автору, отдельный issue
+            НЕ заводится (решение владельца 2026-08-19, #202: заведение и
+            обслуживание issue дороже правки на месте); Low либо правится,
+            либо снимается с записью. Жёлтый вердикт допустим и при полностью
+            выполненных AC, если изменение не решает заявленный сценарий или
+            ухудшает смежный. Продуктовое рассуждение расширяет вопросы, но не
+            отменяет AC и не даёт права менять скоуп.

-            Каждую Medium-находку заведи отдельным issue со ссылкой на
-            #${{ github.event.issue.number }} и метками: тип, приоритет,
-            S1-new. «Оставили в тексте ревью» закрытием не считается и прямо
-            запрещено §12.
+            Только Medium-находку ВНЕ скоупа задачи (попутный дефект соседнего
+            поведения, который в этой ветке чинить нельзя) заведи отдельным
+            issue со ссылкой на #${{ github.event.issue.number }} и метками:
+            тип, приоритет, S1-new. «Оставили в тексте ревью» закрытием не
+            считается и прямо запрещено §12.

-            Напиши полный документ ревью в файл
-            docs/reviews/<SPEC|CODE>-REVIEW-${{ github.event.issue.number }}-r${{ needs.guard.outputs.cycle }}.md
-            (SPEC для этапа spec, CODE для code): скоуп, как проверялось,
-            находки с воспроизведением, что проверено и корректно, чего не
-            проверял. Каталог docs/reviews/ создай, если его нет. Больше не
-            пиши ничего: любой файл вне docs/reviews/ опубликован не будет.
+            Напиши полный документ ревью в файл, путь которого лежит в
+            переменной окружения REVIEW_DOC (абсолютный, ВНЕ репозитория).
+
+            Почему не в docs/reviews: документ там был некоммитнутым файлом того
+            же дерева, которое ты мутируешь, проверяя «умеет ли тест падать». На
+            #220 три раунда подряд документ исчезал — восстановление дерева
+            (`git checkout -- .`, `git clean -fd`) сносит собственный артефакт
+            ревью, потому что он untracked. В репозиторий его положит шаг
+            публикации, взяв из REVIEW_DOC; тебе трогать docs/reviews не нужно.
+
+            В самом репозитории не создавай файлов вообще: любые изменения в
+            рабочей копии будут отброшены. Имя документа в docs/reviews шаг
+            публикации соберёт сам — SPEC-REVIEW для этапа spec, CODE-REVIEW для
+            code, с номером issue и заходом.
+
+            Содержание документа: скоуп, как проверялось, находки с
+            воспроизведением, что проверено и корректно, чего не проверял. Для
+            r2 и дальше добавь два раздела: «Закрытие раунда r<N-1>» — таблица
+            «находка | чем закрыта | где это видно», и «Унаследовано из r<N-1>» —
+            что принято без повторной проверки, с документом и SHA.

            Затем оставь в issue краткий комментарий: вердикт, ключевые находки
            и ссылка на документ. Первой строкой — вердикт в формате §7.2:
-            `Вердикт: зелёный/жёлтый/красный · цикл r${{ needs.guard.outputs.cycle }}/${{ needs.guard.outputs.limit }} · High: N · Medium: N → #…`
+            `Вердикт: зелёный/жёлтый/красный · заход r${{ needs.guard.outputs.cycle }} · блокирующих циклов ${{ needs.guard.outputs.spent }}/${{ needs.guard.outputs.limit }} · High: N · Medium: N → в задаче | #…`
+            («→ #…» — только у Medium вне скоупа; находки в скоупе возвращаются автору жёлтым)

            Затем верни JSON по схеме. Это последнее действие и оно обязательно:
            без него метка не переставится и конвейер встанет.
@@ -337,6 +442,7 @@ jobs:
          NUM: ${{ github.event.issue.number }}
          STAGE: ${{ needs.guard.outputs.stage }}
          CYCLE: ${{ needs.guard.outputs.cycle }}
+          SOURCE: ${{ runner.temp }}/review-document.md
        run: |
          # Ветки задачи может не быть: у задач, размеченных до появления
          # конвейера, ТЗ лежит прямо в dev. Раньше шаг в этом случае молча
@@ -350,8 +456,29 @@ jobs:
          marker=CODE-REVIEW
          if [ "$STAGE" = "spec" ]; then marker=SPEC-REVIEW; fi
          doc="docs/reviews/${marker}-${NUM}-r${CYCLE}.md"
+          # Рабочая копия отбрасывается ДО того, как документ попадёт в дерево:
+          # ревьюер правит код, проверяя «умеет ли тест падать», и его правки
+          # публиковаться не должны.
          git checkout -- . 2>/dev/null || true
+          # docs/reviews исключён из уборки: ревьюер мог написать документ по
+          # старому пути, и клин не должен его съесть до `git add` — ровно так
+          # оба пути остаются работоспособными.
          git clean -fd -e docs/reviews -e node_modules >/dev/null 2>&1 || true
+          # Документ приезжает извне репозитория (#220). Три раунда подряд он
+          # терялся, пока лежал некоммитнутым файлом в том же дереве, которое
+          # ревьюер мутирует и затем восстанавливает: `git checkout -- .` плюс
+          # `git clean -fd` сносят собственный артефакт ревью, потому что он
+          # untracked. Теперь его место — RUNNER_TEMP, и уборка дерева ему не
+          # страшна.
+          if [ -f "$SOURCE" ]; then
+            mkdir -p docs/reviews
+            cp "$SOURCE" "$doc"
+            echo "документ взят из $SOURCE ($(wc -c < "$doc") байт)"
+          else
+            # Совместимость: ревьюер мог написать по старому пути, если промпт
+            # ещё не обновился в этой ветке.
+            echo "::warning::$SOURCE не найден — ищу документ в рабочей копии"
+          fi
          git add docs/reviews 2>/dev/null || true
          if git diff --cached --quiet; then
            # Пустая рабочая копия — ещё не провал: ревьюер иногда коммитит
@@ -367,7 +494,7 @@ jobs:
            # остались только комментариями, метки переставились, и пропажу
            # заметило лишь следующее ревью — issue #171. Падение ДО шага с
            # меткой сохраняет инвариант «метка не сменилась = прогон упал».
-            echo "::error::вердикт есть, а документа $doc нет ни в рабочей копии, ни в $target — ревью без артефакта (#171)"
+            echo "::error::вердикт есть, а документа нет: ни $SOURCE, ни $doc в рабочей копии, ни $doc в $target — ревью без артефакта (#171, #220)"
            exit 1
          fi
          git -c user.name="claude[bot]" \
@@ -116,6 +116,98 @@ jobs:
            echo "integration=$(has '^(custom_components/houseplan/manifest\.json$|hacs\.json$|custom_components/.*\.py$|custom_components/.*/translations/)')"
          } >> "$GITHUB_OUTPUT"

+  # Переиспользование результата тяжёлой job (#208). Ключ = входы поведения
+  # (sourceFingerprint: src/**, demo/fixtures, demo/golden/*.mjs, манифесты
+  # сборки) ПЛЮС оснастка именно этой job. Маркер в кэше пишет только успешный
+  # прогон с тем же ключом, поэтому попадание доказывает: job с побайтово теми
+  # же входами уже завершилась успешно.
+  #
+  # Это НЕ фильтр путей из job `changes` (на dev они отключены намеренно): там
+  # объём прогона угадывается по путям и «зелёный» начинает значить разное,
+  # здесь эквивалентность входов доказана хешем.
+  #
+  # Свойство, снимающее главный риск: релизный кандидат бампает версию, а
+  # CARD_VERSION и package.json входят в фингерпринт, поэтому ключи кандидата
+  # заведомо новые и полный набор гейтов перед бетой и релизом идёт всегда.
+  reuse:
+    runs-on: ubuntu-latest
+    outputs:
+      smoke: ${{ steps.probe.outputs.smoke }}
+      golden: ${{ steps.probe.outputs.golden }}
+      performance_smoke: ${{ steps.probe.outputs.performance_smoke }}
+      backend: ${{ steps.probe.outputs.backend }}
+      smoke_key: ${{ steps.keys.outputs.smoke }}
+      golden_key: ${{ steps.keys.outputs.golden }}
+      performance_smoke_key: ${{ steps.keys.outputs.performance_smoke }}
+      backend_key: ${{ steps.keys.outputs.backend }}
+    steps:
+      - uses: actions/checkout@v7
+      - uses: actions/setup-node@v7
+        with: { node-version: 22 }
+      - name: Ключи переиспользования
+        id: keys
+        run: |
+          for job in smoke golden performance_smoke backend; do
+            key=$(node scripts/gate-reuse.mjs --job="$job")
+            echo "$job=$key" >> "$GITHUB_OUTPUT"
+            echo "$job: $key"
+          done
+      # lookup-only: маркер только проверяется, но не восстанавливается —
+      # сохранять его в этой job нечего, она ничего не прогоняла.
+      - name: Маркер smoke
+        id: m_smoke
+        uses: actions/cache/restore@v6
+        with:
+          path: .reuse-marker
+          key: reuse-smoke-${{ steps.keys.outputs.smoke }}
+          lookup-only: true
+      - name: Маркер golden
+        id: m_golden
+        uses: actions/cache/restore@v6
+        with:
+          path: .reuse-marker
+          key: reuse-golden-${{ steps.keys.outputs.golden }}
+          lookup-only: true
+      - name: Маркер performance_smoke
+        id: m_perf
+        uses: actions/cache/restore@v6
+        with:
+          path: .reuse-marker
+          key: reuse-performance_smoke-${{ steps.keys.outputs.performance_smoke }}
+          lookup-only: true
+      - name: Маркер backend
+        id: m_backend
+        uses: actions/cache/restore@v6
+        with:
+          path: .reuse-marker
+          key: reuse-backend-${{ steps.keys.outputs.backend }}
+          lookup-only: true
+      - name: Что переиспользуем
+        id: probe
+        env:
+          SMOKE: ${{ steps.m_smoke.outputs.cache-hit }}
+          GOLDEN: ${{ steps.m_golden.outputs.cache-hit }}
+          PERF: ${{ steps.m_perf.outputs.cache-hit }}
+          BACKEND: ${{ steps.m_backend.outputs.cache-hit }}
+        run: |
+          # Пропуск обязан быть громким: молчаливый skip — тот самый тихий
+          # успех, который уже дважды стоил нам дня (#171, #207).
+          waive() {
+            if [ "$2" = "true" ]; then
+              echo "$1=true" >> "$GITHUB_OUTPUT"
+              echo "::notice::$1 не прогоняется: входы побайтово те же, что в предыдущем успешном прогоне (#208)"
+              echo "- **$1** переиспользована: входы не менялись" >> "$GITHUB_STEP_SUMMARY"
+            else
+              echo "$1=false" >> "$GITHUB_OUTPUT"
+              echo "- $1: прогоняется" >> "$GITHUB_STEP_SUMMARY"
+            fi
+          }
+          echo "### Переиспользование гейтов (#208)" >> "$GITHUB_STEP_SUMMARY"
+          waive smoke "$SMOKE"
+          waive golden "$GOLDEN"
+          waive performance_smoke "$PERF"
+          waive backend "$BACKEND"
+
  hacs:
    needs: changes
    if: needs.changes.outputs.integration == 'true'
@@ -160,7 +252,8 @@ jobs:

  smoke:
    # Gated on `frontend` so a typecheck failure does not burn browser minutes.
-    needs: frontend
+    needs: [frontend, reuse]
+    if: needs.reuse.outputs.smoke != 'true'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v7
@@ -169,8 +262,20 @@ jobs:
          node-version: 22
          cache: npm
      - run: npm ci
-      - name: Install Chromium for Playwright
-        run: npx playwright install --with-deps chromium
+      # Браузеры кэшируются, а apt не запускается вовсе: на GitHub-раннере
+      # системные библиотеки Chromium уже в образе, а --with-deps тратил минуты
+      # и подолгу перебирал недоступное azure-зеркало (#175, #206). Если
+      # библиотека когда-нибудь исчезнет из образа, Chromium не запустится с
+      # внятной ошибкой — тогда флаг вернуть.
+      - name: Кэш браузеров Playwright
+        id: pw
+        uses: actions/cache@v6
+        with:
+          path: ~/.cache/ms-playwright
+          key: playwright-${{ runner.os }}-${{ hashFiles('package-lock.json') }}
+      - name: Install pinned Chromium
+        if: steps.pw.outputs.cache-hit != 'true'
+        run: npx playwright install chromium
      - name: Build a fresh bundle for the smokes
        run: npm run build && cp dist/houseplan-card.js demo/srv/assets/houseplan-card.js
      - name: Smoke suite
@@ -194,11 +299,28 @@ jobs:
        with:
          name: smoke-logs
          path: /tmp/smoke-logs
+      # Маркер пишется последним шагом: он существует только если всё выше
+      # прошло. Кэш сохраняется post-шагом, то есть тоже лишь при успехе job.
+      - name: Записать маркер успеха
+        run: |
+          printf '%s\n' "smoke прогнана успешно" \
+            "SHA: ${{ github.sha }}" \
+            "прогон: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
+            > .reuse-marker
+      - uses: actions/cache/save@v6
+        # Гонка двух прогонов с одинаковым ключом даёт «Cache already exists».
+        # Это не отказ гейта: работа выполнена, маркер уже записал сосед — job
+        # краснеть из-за этого не должна.
+        continue-on-error: true
+        with:
+          path: .reuse-marker
+          key: reuse-smoke-${{ needs.reuse.outputs.smoke_key }}

  golden:
    # Deterministic visual correctness stays in every prerelease gate: it is
    # inexpensive and catches a different class of regressions than timings.
-    needs: frontend
+    needs: [frontend, reuse]
+    if: needs.reuse.outputs.golden != 'true'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v7
@@ -207,8 +329,20 @@ jobs:
          node-version: 22
          cache: npm
      - run: npm ci
+      # Браузеры кэшируются, а apt не запускается вовсе: на GitHub-раннере
+      # системные библиотеки Chromium уже в образе, а --with-deps тратил минуты
+      # и подолгу перебирал недоступное azure-зеркало (#175, #206). Если
+      # библиотека когда-нибудь исчезнет из образа, Chromium не запустится с
+      # внятной ошибкой — тогда флаг вернуть.
+      - name: Кэш браузеров Playwright
+        id: pw
+        uses: actions/cache@v6
+        with:
+          path: ~/.cache/ms-playwright
+          key: playwright-${{ runner.os }}-${{ hashFiles('package-lock.json') }}
      - name: Install pinned Chromium
-        run: npx playwright install --with-deps chromium
+        if: steps.pw.outputs.cache-hit != 'true'
+        run: npx playwright install chromium
      - name: Build the exact source under review
        run: npm run build && cp dist/houseplan-card.js demo/srv/assets/houseplan-card.js
      - name: Capture or verify golden matrix
@@ -227,13 +361,33 @@ jobs:
        with:
          name: golden-images
          path: artifacts/golden
+      # Маркер пишется последним шагом: он существует только если всё выше
+      # прошло. Кэш сохраняется post-шагом, то есть тоже лишь при успехе job.
+      - name: Записать маркер успеха
+        run: |
+          printf '%s\n' "golden прогнана успешно" \
+            "SHA: ${{ github.sha }}" \
+            "прогон: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
+            > .reuse-marker
+      - uses: actions/cache/save@v6
+        # Гонка двух прогонов с одинаковым ключом даёт «Cache already exists».
+        # Это не отказ гейта: работа выполнена, маркер уже записал сосед — job
+        # краснеть из-за этого не должна.
+        continue-on-error: true
+        with:
+          path: .reuse-marker
+          key: reuse-golden-${{ needs.reuse.outputs.golden_key }}

  performance_smoke:
    # Candidate-only catastrophic-regression guard for ordinary pushes and
    # prereleases. The expensive same-runner comparison lives in performance.yml.
-    needs: frontend
+    needs: [frontend, reuse]
+    if: needs.reuse.outputs.performance_smoke != 'true'
    runs-on: ubuntu-latest
-    timeout-minutes: 15
+    # 15 минут не хватало, когда установка браузера шла через apt: замер
+    # начинался на исходе окна (#206). Запас на холодный кэш — при попадании
+    # job укладывается в те же минуты, что и раньше.
+    timeout-minutes: 20
    steps:
      - uses: actions/checkout@v7
      - uses: actions/setup-node@v7
@@ -241,8 +395,20 @@ jobs:
          node-version: 22
          cache: npm
      - run: npm ci
+      # Браузеры кэшируются, а apt не запускается вовсе: на GitHub-раннере
+      # системные библиотеки Chromium уже в образе, а --with-deps тратил минуты
+      # и подолгу перебирал недоступное azure-зеркало (#175, #206). Если
+      # библиотека когда-нибудь исчезнет из образа, Chromium не запустится с
+      # внятной ошибкой — тогда флаг вернуть.
+      - name: Кэш браузеров Playwright
+        id: pw
+        uses: actions/cache@v6
+        with:
+          path: ~/.cache/ms-playwright
+          key: playwright-${{ runner.os }}-${{ hashFiles('package-lock.json') }}
      - name: Install pinned Chromium
-        run: npx playwright install --with-deps chromium
+        if: steps.pw.outputs.cache-hit != 'true'
+        run: npx playwright install chromium
      - name: Build the exact candidate source
        run: npm run build && cp dist/houseplan-card.js demo/srv/assets/houseplan-card.js
      - name: Capture the heaviest Glow state
@@ -257,10 +423,26 @@ jobs:
        with:
          name: performance-smoke
          path: artifacts/performance-smoke
+      # Маркер пишется последним шагом: он существует только если всё выше
+      # прошло. Кэш сохраняется post-шагом, то есть тоже лишь при успехе job.
+      - name: Записать маркер успеха
+        run: |
+          printf '%s\n' "performance_smoke прогнана успешно" \
+            "SHA: ${{ github.sha }}" \
+            "прогон: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
+            > .reuse-marker
+      - uses: actions/cache/save@v6
+        # Гонка двух прогонов с одинаковым ключом даёт «Cache already exists».
+        # Это не отказ гейта: работа выполнена, маркер уже записал сосед — job
+        # краснеть из-за этого не должна.
+        continue-on-error: true
+        with:
+          path: .reuse-marker
+          key: reuse-performance_smoke-${{ needs.reuse.outputs.performance_smoke_key }}

  backend:
-    needs: changes
-    if: needs.changes.outputs.backend == 'true'
+    needs: [changes, reuse]
+    if: needs.changes.outputs.backend == 'true' && needs.reuse.outputs.backend != 'true'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v7
@@ -273,3 +455,19 @@ jobs:
      - run: pip install pytest voluptuous pytest-homeassistant-custom-component home-assistant-frontend
      - name: Backend unit tests (pure + HA harness)
        run: python -m pytest tests_backend/ -q
+      # Маркер пишется последним шагом: он существует только если всё выше
+      # прошло. Кэш сохраняется post-шагом, то есть тоже лишь при успехе job.
+      - name: Записать маркер успеха
+        run: |
+          printf '%s\n' "backend прогнана успешно" \
+            "SHA: ${{ github.sha }}" \
+            "прогон: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
+            > .reuse-marker
+      - uses: actions/cache/save@v6
+        # Гонка двух прогонов с одинаковым ключом даёт «Cache already exists».
+        # Это не отказ гейта: работа выполнена, маркер уже записал сосед — job
+        # краснеть из-за этого не должна.
+        continue-on-error: true
+        with:
+          path: .reuse-marker
+          key: reuse-backend-${{ needs.reuse.outputs.backend_key }}
@@ -255,10 +255,15 @@ The exchange happens in **issue comments** — there is no local message bus. Ve
 format:

 ```text
-Verdict: green/yellow/red · cycle r<N>/4 · High: N · Medium: N → #… · Document: …
+Verdict: green/yellow/red · cycle r<N>/4 · High: N · Medium: N → in-task | #… · Document: …
 ```

-High blocks. Medium must become its own issue. Low is fixed or waived with a note
+High blocks. A Medium finding INSIDE the task's scope is fixed within the task:
+with no High findings the verdict is yellow, the author fixes it and the fix
+passes another review cycle — no separate issue (owner's decision 2026-08-19,
+#202: filing and servicing an issue costs far more than fixing in place). Only
+a Medium finding OUTSIDE the scope becomes its own issue — foreign scope is
+never patched from this branch. Low is fixed or waived with a note
 in the review document. A yellow verdict is legitimate even when every acceptance
 criterion passes, if the change does not solve the stated scenario or degrades a
 neighbouring one.
@@ -353,14 +358,22 @@ complete Linux CI artifact; never accept a partial scenario or images merely to
 CI green. See `demo/golden/README.md`.

 **Freshness contract**: the embedded fingerprint covers `src/` plus Rollup,
-TypeScript and package-lock build inputs. Benchmark and golden tooling must call
-`assertFreshDemoBundle` before recording any result; a missing or mismatched
-fingerprint is a hard failure, not a warning.
+TypeScript and package-lock build inputs. Every browser check must verify it
+before trusting a result — benchmarks, golden runs and documentation captures
+call `assertFreshDemoBundle` themselves, and smokes get it from `launch()` in
+`demo/serve.mjs` (#236). A missing or mismatched fingerprint is a hard failure,
+not a warning; `HP_ALLOW_STALE_BUNDLE=1` skips the check for debugging and says
+so out loud. A smoke against a stale bundle does not fail cleanly: part of its
+assertions go red and part stay green, which reads as a logic defect.

 **CI is pinned to an exact SHA.** The release gate accepts only a `completed
 success` run for the candidate's SHA, not "the last green one"; a new push cancels
-an unfinished Validate for the same branch. Jobs: `provenance`, `hacs`, `hassfest`,
-`frontend`, `smoke`, `golden`, `performance_smoke`, `backend`.
+an unfinished Validate for the same branch. Gate jobs, matching the actual
+`validate.yml` (#191): `docs`, `provenance`, `process-gate`, `hacs`, `hassfest`,
+`frontend`, `smoke`, `golden`, `performance_smoke`, `backend`. The `changes` job
+is a service path-filter, not a gate. `docs` is a real blocker: it checks the
+screenshots `sourceFingerprint` against current `src/**`, which is exactly what
+went red after the #113 merge.

 **"Verified" without a named command and its result is not evidence.**

@@ -131,10 +131,14 @@ S1-new → S2-analysis → S3-spec → S4-spec-review ⟲ → S5-ready →
  Его задача — не согласиться, а найти, где ТЗ не выполнимо или не проверяемо.
 - **Артефакт:** `docs/reviews/SPEC-REVIEW-<NN>-r<N>.md`, вердикт
  зелёный / жёлтый / красный. Лёгкий трек — комментарий в issue.
- **High-находки блокируют.** Medium/Low — либо правятся, либо становятся
-  отдельными issue со ссылкой; «оставили в тексте ревью» не считается закрытием.
+- **High-находки блокируют.** Medium **в скоупе задачи** чинится в текущем
+  issue: без High это жёлтый вердикт, автор правит ТЗ, фикс проходит повторный
+  цикл. Medium **вне скоупа** — отдельный issue: чужой скоуп в этой задаче не
+  правится. «Оставили в тексте ревью» не считается закрытием ни для одной
+  (решение владельца 2026-08-19, #202: отдельный issue дороже правки на месте).
+  Low либо правится, либо снимается решением ревьюера с записью.
 - **Выход:** «Готово к разработке» либо возврат в «ТЗ в работе» — не более
-  4 циклов (§4).
+  4 циклов (§4). Второй и последующие циклы разбираются по дельте (§2.10).

 ### 2.5 Готово к разработке (DoR)

@@ -187,9 +191,11 @@ S1-new → S2-analysis → S3-spec → S4-spec-review ⟲ → S5-ready →
  кода отвечает на вопрос «оно вообще работает»: каждый AC либо доказан
  автотестом — и ревьюер убедился, что **тест умеет падать**, — либо разобран по
  коду с явной записью «проверено чтением, не исполнением».
- **High блокируют.** Medium **обязаны** превратиться в issue.
+- **High блокируют.** Medium **в скоупе задачи** чинится в текущем issue:
+  без High это жёлтый вердикт и возврат автору, фикс проходит повторный цикл.
+  Medium **вне скоупа** — отдельный issue (#202).
 - **Выход:** очередь на пре-релиз либо возврат в «В разработке», не более
-  4 циклов (§4).
+  4 циклов (§4). Второй и последующие циклы разбираются по дельте (§2.10).

 ### 2.8 Закрытие после выпуска беты

@@ -210,6 +216,42 @@ S1-new → S2-analysis → S3-spec → S4-spec-review ⟲ → S5-ready →
 - **Отклонено:** закрытие с записанной причиной (вне скоупа, дубликат, цена не
  оправдана). Тихое закрытие без причины запрещено.

+### 2.10 Повторный раунд ревью — объём по дельте
+
+Решение владельца 2026-08-19 (issue #214). Относится и к ревью ТЗ, и к
+код-ревью, начиная со второго цикла.
+
+**Предмет повторного раунда — дельта, а не задача целиком.** Раньше объём
+разбора не был оговорён, промпт ревьюера для всех раундов был одинаковым, и
+повторный цикл заново выводил продуктовую рамку и перепроверял AC, которых
+правка не касалась: r2 по #150 стоил полного прогона конвейера ради одной
+строки в тестовой фикстуре.
+
+Порядок:
+
+1. найти вердикт предыдущего раунда и **SHA, на котором он получен**; SHA в
+   вердикте не назван — это находка;
+2. объявить дельту: `git diff <тот SHA>..HEAD` для кода, дифф файла ТЗ либо тела
+   issue для этапа ТЗ;
+3. по каждой находке предыдущего раунда показать, **чем именно она закрыта** —
+   строкой кода или текста, а не заявлением автора;
+4. заново проверять только те AC, чьё доказательство дельта задевает;
+5. **раздел «Унаследовано из r<N−1>»** обязателен: что принято без повторной
+   проверки, со ссылкой на документ того раунда и SHA. Без перечня сокращение
+   превращается в молчаливое доверие.
+
+Дешёвые гейты (`typecheck`, `test`, `build` со сверкой копий бандла) гоняются в
+каждом раунде: код изменился, а стоят они минуты. Тяжёлые — по дельте (§10.2).
+
+**Разбор остаётся полным**, если дельта не локальна: ребейз на ушедший вперёд
+`dev` (после ребейза это другой код, §7.2), смена контракта поведения, задета
+новая подсистема, либо объём дельты сопоставим с исходной задачей.
+
+Сокращается объём **разбора, а не строгость**: правка по замечанию способна
+сломать AC, который предыдущий раунд признал выполненным — так появилась
+регрессия #102. Граница не «только находки», а «находки плюс всё, до чего
+дотягивается дельта».
+
 ---

 ## 3. Правила
@@ -231,8 +273,10 @@ S1-new → S2-analysis → S3-spec → S4-spec-review ⟲ → S5-ready →
   ревью-гейт.
 7. **Ревью возвращает не более 4 раз.** Пятый заход — решение владельца: разделить,
   отклонить или арбитраж (§4).
-8. **High блокирует. Medium становится issue.** Low либо правится, либо снимается
-   решением ревьюера с записью в документе.
+8. **High блокирует. Medium в скоупе чинится в текущем issue** (без High —
+   жёлтый вердикт и повторный цикл); Medium вне скоупа становится отдельным
+   issue (#202). Low либо правится, либо снимается решением ревьюера с записью
+   в документе.
 9. **Скоуп не расширяется.** Всё найденное вне ТЗ — новый issue, а не попутная
   правка. Блокирующая находка отправляет текущий issue в «Заблокировано».
 10. **Каждый коммит класса A и B несёт трейлер `Issue: #NN`**, ветка называется
@@ -263,11 +307,24 @@ S1-new → S2-analysis → S3-spec → S4-spec-review ⟲ → S5-ready →

 ## 4. Лимит циклов ревью: 4

-Оба ревью-гейта возвращают задачу на правки не более **4 раз**. Счётчик виден в
-имени документа: `-r1` … `-r4`; на четвёртом заходе ставится метка `review-4`.
+Оба ревью-гейта возвращают задачу на правки не более **4 раз**.

 - **Что считается циклом:** отправка на ревью → вердикт с блокирующими находками
  → возврат. Уточняющий вопрос без вердикта циклом не считается.
+- **Зелёный вердикт цикла не образует** и бюджет не тратит (решение владельца
+  2026-08-20, issue #227): он ничего не вернул на правки. Практический случай —
+  зелёное ревью, слияние которого не удалось: конвейер сам предписывает ребейз и
+  возврат метки, и этот заход не должен наказываться. Раньше счётчик считал все
+  вердикты подряд, и на #225 последовательность жёлтый → зелёный → ребейз дала
+  `review-4` на задаче с зелёным ревью и зелёным CI.
+- **Заход и цикл — разные величины.** Заход — сколько раз ревью отработало; он
+  виден в имени документа (`-r1`, `-r2`, …) и нужен, чтобы два документа не
+  затёрли друг друга. Цикл — единица бюджета §4. Заходов законно бывает больше,
+  чем циклов, поэтому порог проверки 7 в `scripts/process-gate.mjs` выше лимита
+  циклов (шесть документов = четыре цикла плюс два ребейза).
+- Метка `review-4` ставится, когда исчерпан **бюджет циклов**; конвейер снимать
+  её не вправе — это решение владельца. Если бюджет пересчитан и оказался ниже
+  лимита, конвейер сообщает пересчёт, но метку не трогает.
 - **Исчерпание лимита — не «пятая попытка», а разбор.** Задача уходит владельцу,
  решение одно из трёх:
  1. **разделить** — issue закрывается как «заменён», вместо него 2–3 меньших с
@@ -444,8 +501,12 @@ issue #NN
 - **Занятие:** `Взял: <роль> · сессия <id> · ветка issue/NN-slug`
 - **Хендофф:** `Сделано: … · Файлы: … · Гейты: <команда → результат> ·
  НЕ сделано: … · Риски: … · Следующий статус: … · Новые issue: #…`
- **Вердикт ревью:** `Вердикт: зелёный/жёлтый/красный · цикл r<N>/<лимит> ·
-  High: N · Medium: N → #… · Документ: docs/reviews/…`
+- **Вердикт ревью:** `Вердикт: зелёный/жёлтый/красный · заход r<N> ·
+  блокирующих циклов K/<лимит> · High: N · Medium: N → в задаче | #… ·
+  Документ: docs/reviews/…`
+  («→ #…» — только у Medium вне скоупа; находки в скоупе возвращаются автору.
+  Заход — номер прогона ревью, K — израсходованный бюджет §4: зелёные вердикты
+  его не тратят, поэтому заход и K расходятся, #227)
 - **Закрытие:** `Выпущено в <тег беты> · CI: <ссылка> · Changelog: <ссылка>`

 **Вперёд двигает только зелёный вердикт.** Жёлтый и красный возвращают автору;
@@ -479,16 +540,26 @@ npm run build && cmp dist/houseplan-card.js custom_components/houseplan/frontend
              && cmp dist/houseplan-card.js demo/srv/assets/houseplan-card.js
 node demo/smoke_<целевые>.mjs
 npm run golden:verify                 # если менялся визуал
+node scripts/check-docs.mjs           # если менялся src/**
 python -m pytest tests_backend -q     # py3.13, если менялся бэкенд
 ```

 **Объём гейтов на код-ревью соразмерен задаче** (issue #127). Всегда:
-`typecheck`, `npm test`, `npm run build` со сверкой трёх копий бандла. По
-необходимости, определяемой diff'ом и AC: браузерные смоки (их 127 — прогон всех
+`typecheck`, `npm test`, `npm run build` со сверкой трёх копий бандла, а при
+любом diff'е по `src/**` — ещё и `node scripts/check-docs.mjs`. По
+необходимости, определяемой diff'ом и AC: браузерные смоки (их 163 — прогон всех
 уместен только когда задача задевает всё), `golden:verify` при изменении видимого
 результата, `pytest tests_backend` при правках в Python, performance-профили при
 названном в AC влиянии. **Полные наборы — предрелизный гейт, а не гейт ревью.**

+`check-docs` стоит в обязательной части не по важности, а по механике: отпечаток
+скриншотов документации считается по всему `src/**`, поэтому **любая** правка
+фронтенда делает его устаревшим. Выборка «по diff и AC» здесь не работает — diff
+всегда попадает, и решать нечего. Цена пропуска измерена: скриншоты не
+пересняли в #230 и #234, и `dev` стоял с красным job `docs`, пока это не нашли
+при следующей задаче (#237). Пересъёмка — `npm run build && node
+demo/docs/capture.mjs`, коммит вместе с задачей.
+
 Условие честности такого сужения: ревьюер обязан перечислить, какие гейты прогнал,
 какие нет и почему. Непрогнанный гейт становится видимым решением, а не молчаливым
 пропуском.
@@ -686,8 +757,8 @@ S7-code-review → код-ревью → слияние в dev → S8-merged л
 ```

 Ревьюер — `anthropics/claude-code-action`. Он читает `docs/SCOPE.md`, `AGENTS.md`,
-этот документ и тело issue, публикует разбор комментарием, заводит issue на каждую
-Medium-находку, кладёт документ в `docs/reviews/` ветки задачи и возвращает вердикт
+этот документ и тело issue, публикует разбор комментарием, заводит issue на Medium-находки
+вне скоупа задачи (#202), кладёт документ в `docs/reviews/` ветки задачи и возвращает вердикт
 структурированным JSON. **Метку переставляет отдельный детерминированный шаг по
 вердикту, а не модель.**

@@ -811,7 +882,8 @@ Golden, браузерные смоки, performance и полный HA-харн
 - принятие golden-эталонов ради зелёного CI или по частичному артефакту;
 - закрытие issue до выпуска беты с зелёным CI;
 - переоткрытие закрытого issue вместо нового бага;
- Medium-находки, оставленные как TODO в документе ревью;
+- Medium-находки, оставленные как TODO в документе ревью: в скоупе — чинятся
+  в текущем issue, вне скоупа — становятся отдельным (#202);
 - **параллельные бэклоги** в файлах (`BACKLOG-*.md`, «планы» в docs);
 - ревью-документы вне репозитория;
 - попутные правки «раз уж я здесь»;
@@ -46,7 +46,7 @@ PLAN_ORPHAN_TTL_S = 3600
 SCHEDULED_GRACE_S = 30 * 24 * 3600
 FILES_DIR = "houseplan/files"
 CONF_ADMIN_ONLY = "admin_only"
-VERSION = "1.65.0-beta.1"
+VERSION = "1.66.0"

 # Portable backup format.  This is deliberately independent from the Home
 # Assistant Store version above: storage migrations and files exported by a
@@ -0,0 +1,149 @@
+"""Lossless, allow-listed canonicalisation of persisted geometry.
+
+The frontend mirrors this module in src/coordinate-canonicalization.ts.
+Keep the precision, scalar formula and field allow-list in lockstep; a shared
+fixture is exercised by both runtimes.
+"""
+from __future__ import annotations
+
+import copy
+import math
+from typing import Any
+
+
+COORDINATE_DECIMALS = 9
+COORDINATE_FACTOR = 10**COORDINATE_DECIMALS
+
+
+def canonicalize_number(value: Any) -> Any:
+    """Return one stable IEEE-754 representation for an allow-listed number."""
+    if isinstance(value, bool) or not isinstance(value, (int, float)):
+        return value
+    number = float(value)
+    if not math.isfinite(number):
+        return value
+    sign = -1.0 if math.copysign(1.0, number) < 0 else 1.0
+    result = sign * (
+        math.floor(abs(number) * COORDINATE_FACTOR + 0.5)
+        / COORDINATE_FACTOR
+    )
+    if result == 0:
+        return 0.0
+    return result
+
+
+def _record(value: Any) -> dict[str, Any] | None:
+    return value if isinstance(value, dict) else None
+
+
+def _records(value: Any) -> list[dict[str, Any]]:
+    if not isinstance(value, list):
+        return []
+    return [item for item in value if isinstance(item, dict)]
+
+
+def _fields(record: dict[str, Any], names: tuple[str, ...]) -> None:
+    for name in names:
+        if name in record:
+            record[name] = canonicalize_number(record[name])
+
+
+def _point(value: Any) -> None:
+    if not isinstance(value, list):
+        return
+    for index in range(min(2, len(value))):
+        value[index] = canonicalize_number(value[index])
+
+
+def _points(value: Any) -> None:
+    if not isinstance(value, list):
+        return
+    for point in value:
+        _point(point)
+
+
+def canonicalize_position(position: Any) -> Any:
+    """Canonicalise x/y in one layout record without touching its metadata."""
+    result = copy.deepcopy(position)
+    record = _record(result)
+    if record is not None:
+        _fields(record, ("x", "y"))
+    return result
+
+
+def canonicalize_layout_geometry(layout: Any) -> Any:
+    """Canonicalise x/y in every layout record, preserving future fields."""
+    result = copy.deepcopy(layout)
+    record = _record(result)
+    if record is None:
+        return result
+    for position in record.values():
+        item = _record(position)
+        if item is not None:
+            _fields(item, ("x", "y"))
+    return result
+
+
+def canonicalize_config_geometry(config: Any) -> Any:
+    """Canonicalise only the named persisted geometry fields."""
+    result = copy.deepcopy(config)
+    root = _record(result)
+    if root is None:
+        return result
+
+    for space in _records(root.get("spaces")):
+        _fields(
+            space,
+            (
+                "plan_x",
+                "plan_y",
+                "plan_scale",
+                "plan_scale_x",
+                "plan_scale_y",
+                "plan_angle",
+            ),
+        )
+
+        for room in _records(space.get("rooms")):
+            _fields(room, ("x", "y", "w", "h"))
+            _points(room.get("poly"))
+
+        for wall in _records(space.get("walls")):
+            _point(wall.get("a"))
+            _point(wall.get("b"))
+
+        for opening in _records(space.get("openings")):
+            _fields(opening, ("x", "y", "angle", "length"))
+            host = _record(opening.get("host"))
+            if host is not None:
+                _fields(host, ("t",))
+
+        for decor in _records(space.get("decor")):
+            kind = decor.get("kind")
+            if kind == "line":
+                _fields(decor, ("x1", "y1", "x2", "y2"))
+            elif kind in ("rect", "ellipse", "furniture"):
+                _fields(decor, ("x", "y", "w", "h", "angle"))
+            elif kind == "text":
+                _fields(decor, ("x", "y", "scale", "angle"))
+
+        for draft in _records(space.get("room_drafts")):
+            _points(draft.get("points"))
+
+        for partition in _records(space.get("partitions")):
+            _point(partition.get("a"))
+            _point(partition.get("b"))
+
+        for column in _records(space.get("wall_columns")):
+            _point(column.get("center"))
+            if column.get("shape") == "square":
+                _fields(column, ("angle",))
+
+        for span in _records(space.get("open_spans")):
+            _point(span.get("a"))
+            _point(span.get("b"))
+
+    for marker in _records(root.get("markers")):
+        _fields(marker, ("angle",))
+
+    return result
@@ -17,6 +17,7 @@ import unicodedata
 from datetime import UTC, datetime
 from pathlib import Path
 from typing import Any, Callable
+from urllib.parse import urlsplit

 import voluptuous as vol

@@ -48,9 +49,11 @@ from .validation import (
    validate_marker_controls,
    validate_marker_light_entities,
    validate_marker_value_badges,
-    validate_opening_passages,
+    validate_opening_passages, validate_partition_opening_hosts,
    MarkerControlError,
    OpeningPassageError,
+    PartitionOpeningHostError,
+    PartitionOpeningJambMarginError,
 )

 FORMAT = "houseplan-export"
@@ -264,7 +267,8 @@ def _project_plan_only_space(space: dict[str, Any]) -> dict[str, Any]:
            _pick_fields(
                opening,
                ("id", "type", "x", "y", "angle", "length")
-                + (() if opening.get("type") == "passage" else ("flip_h", "flip_v")),
+                + (() if opening.get("type") == "passage" else ("flip_h", "flip_v"))
+                + (("host",) if opening.get("host") else ()),
            )
            for opening in space.get("openings") or []
        ]
@@ -345,6 +349,24 @@ def placement_manifest(config: dict[str, Any], layout: dict[str, Any]) -> list[d


 def _internal_path(root: Path, url: str) -> tuple[str, Path] | None:
+    # A url is parsed as a url, not as a string: everything after "?" or "#"
+    # addresses the transfer, never the file. Legacy attachments carry a
+    # cache-buster (".../files/m1/doc.pdf?v=1783170649"), and while the string
+    # form fed "doc.pdf?v=1783170649" to sanitize_filename the name never
+    # matched itself — the reference read as internal-but-non-canonical and
+    # every backup holding one refused to import (issue #225). Path segments
+    # keep doing the guarding: dropping the query cannot widen what a segment
+    # is allowed to be.
+    #
+    # Only a same-document reference may be trusted this way: with a scheme or
+    # an authority the path belongs to another host, and taking it would let
+    # "https://evil.example/houseplan_files/files/m1/doc.pdf" resolve onto a
+    # local file (review CODE-REVIEW-225-r1, M1). Such a url stays external,
+    # which is also what _looks_internal says about it.
+    parsed = urlsplit(url)
+    if parsed.scheme or parsed.netloc:
+        return None
+    url = parsed.path
    content_plan = CONTENT_URL + "/plans/_/"
    if url.startswith(content_plan) or url.startswith(PLANS_URL + "/"):
        prefix = content_plan if url.startswith(content_plan) else PLANS_URL + "/"
@@ -840,6 +862,16 @@ def build_space_merge(
    for room in space.get("rooms") or []:
        if room.get("open_to"):
            room["open_to"] = [old_room_ids.get(str(value), str(value)) for value in room["open_to"]]
+    # Opening ownership is part of the same space-local id graph. Remap the
+    # nested reference together with the partition itself; otherwise the
+    # invariant validator correctly rejects a copied space whose host.id still
+    # names the source partition.
+    for opening in space.get("openings") or []:
+        host = opening.get("host") if isinstance(opening, dict) else None
+        if isinstance(host, dict) and host.get("kind") == "partition":
+            old_host_id = str(host.get("id"))
+            if old_host_id in id_map:
+                host["id"] = id_map[old_host_id]
    space["id"] = new_space_id
    space["title"] = _unique_title(
        str(space.get("title") or old_space_id), current_config.get("spaces") or []
@@ -980,7 +1012,11 @@ def build_space_merge(
        validate_marker_light_entities(merged_config, current_config)
        validate_marker_value_badges(merged_config, current_config)
        validate_opening_passages(merged_config, current_config)
-    except (MarkerControlError, OpeningPassageError) as err:
+        validate_partition_opening_hosts(merged_config, current_config)
+    except (
+        MarkerControlError, OpeningPassageError, PartitionOpeningHostError,
+        PartitionOpeningJambMarginError,
+    ) as err:
        raise ImportFailure(err.code, str(err)) from err
    try:
        merged_layout = LAYOUT_SCHEMA(merged_layout)
@@ -16,5 +16,5 @@
  "issue_tracker": "https://github.com/Matysh/houseplan-card/issues",
  "requirements": [],
  "single_config_entry": true,
-  "version": "1.65.0-beta.1"
+  "version": "1.66.0"
 }
@@ -20,6 +20,10 @@ from .const import (
    STORAGE_VERSION,
    STORAGE_VIRTUAL_LIGHTS_KEY,
 )
+from .coordinate_canonicalization import (
+    canonicalize_config_geometry,
+    canonicalize_layout_geometry,
+)


 _LOGGER = logging.getLogger(__name__)
@@ -157,7 +161,7 @@ def layout_store_payload(
    }
    if metadata:
        out.update(metadata)
-    out["layout"] = layout
+    out["layout"] = canonicalize_layout_geometry(layout)
    out["rev"] = rev
    return out

@@ -205,7 +209,8 @@ async def async_save_config_state(
        except (TypeError, ValueError):
            previous_rev = 0

-    payload = {"config": config, "rev": rev}
+    canonical_config = canonicalize_config_geometry(config)
+    payload = {"config": canonical_config, "rev": rev}
    await runtime.config_store.async_save(payload)

    # The config is already durable at this point.  Reconciliation remains a
@@ -216,7 +221,7 @@ async def async_save_config_state(
    try:
        await async_reconcile_virtual_lights(
            runtime.virtual_light_store,
-            config,
+            canonical_config,
            rev,
            previous_config_rev=previous_rev,
        )
@@ -10,6 +10,7 @@ want to see where the cleanup has already been).
 from __future__ import annotations

 import asyncio
+import math
 import time
 from typing import Any

@@ -24,11 +25,33 @@ import logging
 _LOGGER = logging.getLogger(__name__)

 TRAIL_CAP = 2000          # raw points per run before decimation
+TRAIL_RESUME_GRACE_S = 30 * 60  # same-map stop/pause belongs to one cleanup
 SAVE_DELAY_S = 10         # debounce store writes — flash wear over precision
 FIRE_THROTTLE_S = 2.0     # event-bus updates for live cards
 MOVING_STATES = {"cleaning", "returning", "on"}


+def can_resume_trail_run(run: Any, map_id: str, now: float) -> bool:
+    """Whether an ended current run may be reopened for this point.
+
+    Store timestamps are untrusted persisted data. Only finite JSON-number
+    timestamps and a non-negative inclusive grace interval are accepted;
+    malformed values and wall-clock rollback fail closed into a new run.
+    """
+    if not isinstance(run, dict) or run.get("map_id") != map_id:
+        return False
+    ended = run.get("ended")
+    if (
+        isinstance(ended, bool)
+        or not isinstance(ended, (int, float))
+        or isinstance(now, bool)
+        or not isinstance(now, (int, float))
+    ):
+        return False
+    elapsed = now - ended
+    return math.isfinite(elapsed) and 0 <= elapsed <= TRAIL_RESUME_GRACE_S
+
+
 def resolve_map_id(src_attrs: Any, vac_attrs: Any) -> str:
    """Map-id normalisation contract, shared with the frontend.

@@ -64,7 +87,10 @@ class TrailBook:
    def on_point(self, marker: str, map_id: str, x: float, y: float, now: float) -> bool:
        rec = self.data.setdefault(marker, {})
        cur = rec.get("current")
-        if not cur or cur.get("ended") or cur.get("map_id") != map_id:
+        resumed = bool(cur and can_resume_trail_run(cur, map_id, now))
+        if resumed:
+            cur["ended"] = None
+        if not cur or cur.get("ended") is not None or cur.get("map_id") != map_id:
            # a new run begins: the old one becomes "previous" (and the one
            # before it is forgotten — we keep exactly two, per the owner)
            if cur:
@@ -73,7 +99,9 @@ class TrailBook:
            rec["current"] = cur
        pts: list[list[float]] = cur["points"]
        if pts and pts[-1][0] == x and pts[-1][1] == y:
-            return False
+            # Clearing ended is observable state even if the source repeats
+            # the dock point: it must still reach Store and live cards.
+            return resumed
        pts.append([x, y])
        if len(pts) > TRAIL_CAP:
            # decimate by two but never lose the freshest point
@@ -85,7 +113,7 @@ class TrailBook:

    def end_run(self, marker: str, now: float) -> bool:
        cur = (self.data.get(marker) or {}).get("current")
-        if cur and not cur.get("ended"):
+        if cur and cur.get("ended") is None:
            cur["ended"] = now
            return True
        return False
@@ -9,6 +9,12 @@ import re

 import voluptuous as vol

+from custom_components.houseplan.coordinate_canonicalization import (
+    canonicalize_config_geometry,
+    canonicalize_layout_geometry,
+    canonicalize_position,
+)
+
 # ---------- limits and extension sets ----------
 PLAN_EXTENSIONS = {"svg": "image/svg+xml", "png": "image/png", "jpg": "image/jpeg", "webp": "image/webp"}
 MAX_PLAN_BYTES = 8 * 1024 * 1024
@@ -46,6 +52,112 @@ class OpeningPassageError(ValueError):
        )


+class PartitionOpeningHostError(ValueError):
+    """A write tried to strip explicit host identity from a surviving opening."""
+
+    code = "invalid_partition_opening_host"
+
+
+class PartitionOpeningJambMarginError(ValueError):
+    """A direct geometry write leaves no physical jamb at a wall endpoint."""
+
+    code = "invalid_partition_opening_jamb_margin"
+
+    def __init__(
+        self, space_id: str, opening_id: str, margin: float, margin_cm: float
+    ) -> None:
+        self.space_id = space_id
+        self.opening_id = opening_id
+        self.margin = margin
+        self.margin_cm = margin_cm
+        super().__init__(
+            f"space={space_id}; opening={opening_id}; "
+            f"margin={margin:.12g}; margin_cm={margin_cm:.12g}"
+        )
+
+
+# One normalized canvas width contains this many physical grid cells. Keep in
+# sync with GRID_STEP_N/NORM_W in the frontend; it is a geometry scale, not a
+# user setting.
+NORMALIZED_CANVAS_CELLS = 240.0
+
+
+def validate_partition_opening_hosts(
+    config: dict, previous: dict | None = None
+) -> None:
+    """Validate hosted-opening write deltas without rejecting legacy reads.
+
+    Deleting the opening together with its partition remains valid. Surviving
+    records keep their host, while new/direct geometry changes reserve half the
+    actual wall depth at both endpoints. Rigid host translation and unrelated
+    edits round-trip existing near-end records unchanged.
+    """
+    old_spaces = {
+        str(space.get("id")): space for space in (previous or {}).get("spaces") or []
+    }
+    for space in config.get("spaces") or []:
+        space_id = str(space.get("id", ""))
+        old_space = old_spaces.get(space_id)
+        old_openings = {
+            str(opening.get("id")): opening
+            for opening in (old_space or {}).get("openings") or []
+        }
+        partitions = {
+            str(partition.get("id")): partition
+            for partition in space.get("partitions") or []
+        }
+        old_partitions = {
+            str(partition.get("id")): partition
+            for partition in (old_space or {}).get("partitions") or []
+        }
+        for opening in space.get("openings") or []:
+            opening_id = str(opening.get("id", ""))
+            old = old_openings.get(opening_id)
+            if old and old.get("host") is not None and opening.get("host") is None:
+                raise PartitionOpeningHostError(
+                    f"space={space_id}; opening={opening_id}; host removed"
+                )
+            host = opening.get("host")
+            if host is None:
+                continue
+            partition = partitions.get(str(host.get("id", "")))
+            if partition is None:
+                # SPACE_SCHEMA owns missing-host diagnostics.
+                continue
+            old_host = (old or {}).get("host")
+            old_partition = old_partitions.get(str((old_host or {}).get("id", "")))
+            ax, ay = partition["a"]
+            bx, by = partition["b"]
+            span = ((bx - ax) ** 2 + (by - ay) ** 2) ** 0.5
+            old_span = None
+            if old_partition is not None:
+                old_ax, old_ay = old_partition["a"]
+                old_bx, old_by = old_partition["b"]
+                old_span = ((old_bx - old_ax) ** 2 + (old_by - old_ay) ** 2) ** 0.5
+            strict = (
+                old is None
+                or old_host is None
+                or old_host.get("id") != host.get("id")
+                or old_host.get("t") != host.get("t")
+                or old.get("length") != opening.get("length")
+                or old_partition is None
+                or old_partition.get("cm") != partition.get("cm")
+                or abs(old_span - span) > 1e-9
+            )
+            if not strict:
+                continue
+            cell_cm = float(space.get("cell_cm", 5))
+            margin_cm = float(partition["cm"]) / 2
+            margin = margin_cm / cell_cm / NORMALIZED_CANVAS_CELLS
+            along = float(host["t"]) * span
+            half = float(opening["length"]) / 2
+            if (along - half < margin - 1e-9
+                    or along + half > span - margin + 1e-9):
+                raise PartitionOpeningJambMarginError(
+                    space_id, opening_id, margin, margin_cm
+                )
+
+
 PASSAGE_FORBIDDEN_FIELDS = {"contact", "lock", "invert", "flip_h", "flip_v"}


@@ -416,11 +528,18 @@ _URL = vol.All(str, vol.Length(max=MAX_URL))
 CANVAS_LIMIT = 5000.0
 _COORD = vol.All(_finite, vol.Range(min=-CANVAS_LIMIT, max=CANVAS_LIMIT))

-POS_SCHEMA = vol.Schema(
-    {vol.Required("x"): _COORD, vol.Required("y"): _COORD},
-    extra=vol.ALLOW_EXTRA,  # v2 records carry the "s" key (space id)
+POS_SCHEMA = vol.All(
+    vol.Schema(
+        {vol.Required("x"): _COORD, vol.Required("y"): _COORD},
+        extra=vol.ALLOW_EXTRA,  # v2 records carry the "s" key (space id)
+    ),
+    canonicalize_position,
+)
+LAYOUT_SCHEMA = vol.All(
+    vol.Schema({str: POS_SCHEMA}),
+    vol.Length(max=MAX_LAYOUT),
+    canonicalize_layout_geometry,
 )
-LAYOUT_SCHEMA = vol.All(vol.Schema({str: POS_SCHEMA}), vol.Length(max=MAX_LAYOUT))

 # Room/opening geometry: same story, same range (docs/CANVAS.md). A vertex at
 # 2.5 is a plan that grew past the old square, not corruption; 1e100 is
@@ -767,6 +886,15 @@ WALL_COLUMN_SCHEMA = vol.All(
    _strict_wall_column,
 )

+PARTITION_OPENING_HOST_SCHEMA = vol.Schema(
+    {
+        vol.Required("kind"): vol.Equal("partition"),
+        vol.Required("id"): vol.All(str, vol.Length(min=1, max=64)),
+        vol.Required("t"): vol.All(_finite, vol.Range(min=0, max=1)),
+    },
+    extra=vol.PREVENT_EXTRA,
+)
+

 def _space_geometry_invariants(value: dict) -> dict:
    """All stored geometry shares ids; draft segments also have a space cap."""
@@ -784,6 +912,29 @@ def _space_geometry_invariants(value: dict) -> dict:
    )
    if draft_segments > MAX_DRAFT_SEGMENTS:
        raise vol.Invalid("too many saved room-draft segments")
+    partitions = {
+        item.get("id"): item for item in value.get("partitions", []) if item.get("id")
+    }
+    hosted_intervals: dict[str, list[tuple[float, float]]] = {}
+    for opening in value.get("openings", []):
+        host = opening.get("host")
+        if host is None:
+            continue
+        partition = partitions.get(host["id"])
+        if partition is None:
+            raise vol.Invalid("partition opening host must exist in the same space")
+        ax, ay = partition["a"]
+        bx, by = partition["b"]
+        span = ((bx - ax) ** 2 + (by - ay) ** 2) ** 0.5
+        length = float(opening["length"])
+        along = float(host["t"]) * span
+        if length > span or along - length / 2 < -1e-9 or along + length / 2 > span + 1e-9:
+            raise vol.Invalid("partition opening must fit inside its host")
+        lo, hi = along - length / 2, along + length / 2
+        occupied = hosted_intervals.setdefault(host["id"], [])
+        if any(max(lo, old_lo) < min(hi, old_hi) - 1e-9 for old_lo, old_hi in occupied):
+            raise vol.Invalid("partition openings must not overlap")
+        occupied.append((lo, hi))
    return value


@@ -849,6 +1000,7 @@ SPACE_SCHEMA = vol.All(vol.Schema(
                    vol.Optional("invert"): bool,
                    vol.Optional("flip_h"): bool,
                    vol.Optional("flip_v"): bool,
+                    vol.Optional("host"): PARTITION_OPENING_HOST_SCHEMA,
                },
                extra=vol.ALLOW_EXTRA,
            )
@@ -985,38 +1137,41 @@ MARKER_SCHEMA = vol.Schema(
    },
    extra=vol.ALLOW_EXTRA,
 )
-CONFIG_SCHEMA = vol.Schema(
-    {
-        vol.Required("spaces"): vol.All([SPACE_SCHEMA], vol.Length(max=MAX_SPACES)),
-        vol.Optional("markers", default=list): vol.All([MARKER_SCHEMA], vol.Length(max=MAX_MARKERS)),
-        vol.Optional("settings", default=dict): vol.Schema(
-            {
-                vol.Optional("glow_radius_cm"): vol.All(vol.Coerce(float), vol.Range(min=10, max=10000)),
-                # background around the plan, all spaces (a space may override)
-                vol.Optional("bg_color"): _COLOR,
-                # sun on the plan (docs/SUN.md): global defaults
-                vol.Optional("north_deg"): _north_deg,
-                vol.Optional("bg_mode"): _BG_MODE,
-                vol.Optional("sun_rays"): bool,
-                # Removed from the UI/runtime in 2026-08-08. Keep accepting the
-                # legacy field so an existing stored config can still load; the
-                # frontend ignores it and removes it on the next settings save.
-                vol.Optional("weather_entity"): vol.Any(None, _TEXT),
-                vol.Optional("known_devices"): vol.All([_TEXT], vol.Length(max=MAX_KNOWN_DEVICES)),
-                vol.Optional("new_device_ids"): vol.All([_TEXT], vol.Length(max=MAX_KNOWN_DEVICES)),
-                vol.Optional("fill_colors"): vol.Schema(
-                    {
-                        str: vol.Schema(
-                            {
-                                vol.Required("c"): _COLOR,
-                                vol.Required("a"): vol.All(vol.Coerce(float), vol.Range(min=0, max=1)),
-                            }
-                        )
-                    }
-                ),
-            },
-            extra=vol.ALLOW_EXTRA,
-        ),
-    },
-    extra=vol.ALLOW_EXTRA,  # unknown (legacy) keys do not break loading
+CONFIG_SCHEMA = vol.All(
+    vol.Schema(
+        {
+            vol.Required("spaces"): vol.All([SPACE_SCHEMA], vol.Length(max=MAX_SPACES)),
+            vol.Optional("markers", default=list): vol.All([MARKER_SCHEMA], vol.Length(max=MAX_MARKERS)),
+            vol.Optional("settings", default=dict): vol.Schema(
+                {
+                    vol.Optional("glow_radius_cm"): vol.All(vol.Coerce(float), vol.Range(min=10, max=10000)),
+                    # background around the plan, all spaces (a space may override)
+                    vol.Optional("bg_color"): _COLOR,
+                    # sun on the plan (docs/SUN.md): global defaults
+                    vol.Optional("north_deg"): _north_deg,
+                    vol.Optional("bg_mode"): _BG_MODE,
+                    vol.Optional("sun_rays"): bool,
+                    # Removed from the UI/runtime in 2026-08-08. Keep accepting the
+                    # legacy field so an existing stored config can still load; the
+                    # frontend ignores it and removes it on the next settings save.
+                    vol.Optional("weather_entity"): vol.Any(None, _TEXT),
+                    vol.Optional("known_devices"): vol.All([_TEXT], vol.Length(max=MAX_KNOWN_DEVICES)),
+                    vol.Optional("new_device_ids"): vol.All([_TEXT], vol.Length(max=MAX_KNOWN_DEVICES)),
+                    vol.Optional("fill_colors"): vol.Schema(
+                        {
+                            str: vol.Schema(
+                                {
+                                    vol.Required("c"): _COLOR,
+                                    vol.Required("a"): vol.All(vol.Coerce(float), vol.Range(min=0, max=1)),
+                                }
+                            )
+                        }
+                    ),
+                },
+                extra=vol.ALLOW_EXTRA,
+            ),
+        },
+        extra=vol.ALLOW_EXTRA,  # unknown (legacy) keys do not break loading
+    ),
+    canonicalize_config_geometry,
 )
@@ -25,6 +25,10 @@ from .const import (
    MAX_SIGN_PATHS,
    PLANS_DIR, PLANS_URL,
 )
+from .coordinate_canonicalization import (
+    canonicalize_config_geometry,
+    canonicalize_layout_geometry,
+)
 from .auth import may_write
 from .import_export import (
    ImportFailure,
@@ -57,8 +61,9 @@ from .virtual_lights import (
 from .registry_snapshot import import_registry_snapshot
 from .validation import (
    CONFIG_SCHEMA, LAYOUT_SCHEMA, MAX_CONFIG_BYTES, MAX_PLAN_BYTES,
-    PLAN_EXTENSIONS, POS_SCHEMA, MarkerControlError, OpeningPassageError, sanitize_filename,
-    validate_opening_passages,
+    PLAN_EXTENSIONS, POS_SCHEMA, MarkerControlError, OpeningPassageError,
+    PartitionOpeningHostError, PartitionOpeningJambMarginError, sanitize_filename,
+    validate_opening_passages, validate_partition_opening_hosts,
    validate_marker_controls, validate_marker_light_entities,
    validate_marker_value_badges, valid_space_id,
 )
@@ -425,8 +430,12 @@ async def ws_import_apply(hass: HomeAssistant, connection, msg: dict[str, Any])
            if kind == "full":
                backup = {
                    "kind": "import",
-                    "config": config_data.get("config") or DEFAULT_CONFIG,
-                    "layout": layout_data.get("layout") or {},
+                    "config": canonicalize_config_geometry(
+                        config_data.get("config") or DEFAULT_CONFIG
+                    ),
+                    "layout": canonicalize_layout_geometry(
+                        layout_data.get("layout") or {}
+                    ),
                    "created": int(time.time()),
                    "after_config_rev": new_config_rev,
                    "after_layout_rev": new_layout_rev,
@@ -443,16 +452,20 @@ async def ws_import_apply(hass: HomeAssistant, connection, msg: dict[str, Any])
                final_metadata[_OPTIMIZE_BACKUP] = backup
            pending = {
                "kind": "import",
-                "config": target_config,
-                "layout": target_layout,
+                "config": canonicalize_config_geometry(target_config),
+                "layout": canonicalize_layout_geometry(target_layout),
                "config_rev": new_config_rev,
                "layout_rev": new_layout_rev,
                "final_metadata": final_metadata,
            }
            rollback = {
                "kind": "import_rollback",
-                "config": config_data.get("config") or DEFAULT_CONFIG,
-                "layout": layout_data.get("layout") or {},
+                "config": canonicalize_config_geometry(
+                    config_data.get("config") or DEFAULT_CONFIG
+                ),
+                "layout": canonicalize_layout_geometry(
+                    layout_data.get("layout") or {}
+                ),
                "config_rev": config_rev,
                "layout_rev": layout_rev,
                "final_metadata": original_metadata,
@@ -565,6 +578,9 @@ async def ws_layout_set(hass: HomeAssistant, connection, msg: dict[str, Any]) ->
            )
            return
        layout = _live_layout(config_data.get("config") or {}, msg["layout"])
+        if layout == data.get("layout", {}):
+            connection.send_result(msg["id"], {"ok": True, "rev": current_rev})
+            return
        new_rev = current_rev + 1
        await async_save_layout_state(
            rt, data, layout, new_rev,
@@ -626,7 +642,10 @@ async def ws_layout_update(hass: HomeAssistant, connection, msg: dict[str, Any])
            return
        data = await rt.store.async_load() or {}
        layout = data.get("layout", {})
-        layout[msg["device_id"]] = msg["pos"]
+        if layout.get(msg["device_id"]) == msg["pos"]:
+            connection.send_result(msg["id"], {"ok": True, "rev": int(data.get("rev", 0))})
+            return
+        layout = {**layout, msg["device_id"]: msg["pos"]}
        # keep the revision: a point-wise write used to drop it, which made the
        # optimistic locking on layout/set meaningless — every drag reset the
        # counter to 0 (HP-1454-08)
@@ -734,7 +753,10 @@ async def ws_geometry_repair(hass: HomeAssistant, connection, msg: dict[str, Any
            rt, data, new_layout, new_rev,
            metadata={
                **_optimizer_backup_after_layout_maintenance(data, new_rev),
-                "repair_backup": {"space": space_id, "positions": touched},
+                "repair_backup": {
+                    "space": space_id,
+                    "positions": canonicalize_layout_geometry(touched),
+                },
            },
            remove=("repair_backup",),
        )
@@ -1256,7 +1278,11 @@ async def ws_config_set(hass: HomeAssistant, connection, msg: dict[str, Any]) ->
            validate_marker_light_entities(msg["config"], data.get("config"))
            validate_marker_value_badges(msg["config"], data.get("config"))
            validate_opening_passages(msg["config"], data.get("config"))
-        except (MarkerControlError, OpeningPassageError) as err:
+            validate_partition_opening_hosts(msg["config"], data.get("config"))
+        except (
+            MarkerControlError, OpeningPassageError, PartitionOpeningHostError,
+            PartitionOpeningJambMarginError,
+        ) as err:
            connection.send_error(msg["id"], err.code, str(err))
            return
        # An internal plan url must name a file that exists. The card can pick a
@@ -1276,6 +1302,17 @@ async def ws_config_set(hass: HomeAssistant, connection, msg: dict[str, Any]) ->
                "Plan file no longer exists: " + ", ".join(sorted(missing)),
            )
            return
+        if msg["config"] == data.get("config"):
+            # A semantic no-op still has to reconcile Repairs with external
+            # file-system changes. It must not create a revision, event, or
+            # discard the optimizer snapshot merely to refresh diagnostics.
+            entry = get_entry(hass)
+            if entry is not None:
+                from .repairs import async_check_plan_files
+
+                hass.async_create_task(async_check_plan_files(hass, entry))
+            connection.send_result(msg["id"], {"ok": True, "rev": int(current_rev)})
+            return
        new_rev = current_rev + 1
        await async_save_config_state(
            rt,
@@ -1367,7 +1404,11 @@ async def ws_plan_optimize(hass: HomeAssistant, connection, msg: dict[str, Any])
            validate_marker_light_entities(msg["config"], config_data.get("config"))
            validate_marker_value_badges(msg["config"], config_data.get("config"))
            validate_opening_passages(msg["config"], config_data.get("config"))
-        except (MarkerControlError, OpeningPassageError) as err:
+            validate_partition_opening_hosts(msg["config"], config_data.get("config"))
+        except (
+            MarkerControlError, OpeningPassageError, PartitionOpeningHostError,
+            PartitionOpeningJambMarginError,
+        ) as err:
            connection.send_error(msg["id"], err.code, str(err))
            return

@@ -1388,15 +1429,19 @@ async def ws_plan_optimize(hass: HomeAssistant, connection, msg: dict[str, Any])
        new_layout_rev = layout_rev + 1
        backup = {
            "kind": "optimize",
-            "config": config_data.get("config") or DEFAULT_CONFIG,
-            "layout": layout_data.get("layout", {}),
+            "config": canonicalize_config_geometry(
+                config_data.get("config") or DEFAULT_CONFIG
+            ),
+            "layout": canonicalize_layout_geometry(
+                layout_data.get("layout", {})
+            ),
            "created": int(time.time()),
            "after_config_rev": new_config_rev,
            "after_layout_rev": new_layout_rev,
        }
        pending = {
-            "config": msg["config"],
-            "layout": msg["layout"],
+            "config": canonicalize_config_geometry(msg["config"]),
+            "layout": canonicalize_layout_geometry(msg["layout"]),
            "config_rev": new_config_rev,
            "layout_rev": new_layout_rev,
            "clear_backup": False,
@@ -1466,8 +1511,12 @@ async def ws_plan_optimize_undo(hass: HomeAssistant, connection, msg: dict[str,

        backup = layout_data[_OPTIMIZE_BACKUP]
        restored_kind = str(backup.get("kind") or "optimize")
-        restored_config = backup.get("config") or DEFAULT_CONFIG
-        restored_layout = backup.get("layout") or {}
+        restored_config = canonicalize_config_geometry(
+            backup.get("config") or DEFAULT_CONFIG
+        )
+        restored_layout = canonicalize_layout_geometry(
+            backup.get("layout") or {}
+        )
        new_config_rev = config_rev + 1
        new_layout_rev = layout_rev + 1
        pending = {
@@ -0,0 +1,96 @@
+// #199: same-process production-builder baseline versus the complete Optimize
+// preflight wrapper on the deterministic 3-floor large-house fixture.
+import { performance } from 'node:perf_hooks';
+
+import { makeLargeHouseFixture, LARGE_HOUSE_COUNTS } from './fixtures/large-house.mjs';
+import {
+  checkOptimizeGeometry,
+  prepareSpacePhysicalGeometryInputs,
+} from '../test-build/plan-geometry-preflight.js';
+import { spaceModels } from '../test-build/space-geometry.js';
+import {
+  floorFootprintGeometry,
+  wallBodiesGeometry,
+} from '../test-build/wall-thickness.js';
+
+const WARMUPS = 3;
+const SAMPLES = 20;
+const ABSOLUTE_P95_MS = 250;
+const RELATIVE_RATIO = 1.2;
+const RELATIVE_NOISE_MS = 15;
+
+const fixture = makeLargeHouseFixture();
+const models = spaceModels(fixture.config);
+const prepared = fixture.config.spaces.map((space, index) =>
+  prepareSpacePhysicalGeometryInputs(space, models[index]));
+
+const directProductionPass = () => {
+  for (const input of prepared) {
+    const hasWalls = input.walls.length > 0 || input.physicalBodies.length > 0;
+    const united = hasWalls
+      ? wallBodiesGeometry(
+          input.space.rooms, input.walls, input.openCuts, input.roomOpenings,
+          input.wallKeyPitch, input.cellCm, input.gridPitch, input.coordScale,
+          input.physicalBodies,
+        )
+      : null;
+    if (hasWalls && united == null) throw new Error(`baseline wall failure: ${input.space.id}`);
+    if (input.space.rooms.length && united?.paperGeom == null) {
+      const floor = floorFootprintGeometry(
+        input.space.rooms, input.walls, input.openCuts,
+        input.wallKeyPitch, input.cellCm, input.gridPitch, input.coordScale,
+      );
+      if (floor == null) throw new Error(`baseline floor failure: ${input.space.id}`);
+    }
+  }
+};
+
+const completePreflight = () => {
+  const result = checkOptimizeGeometry(fixture.config);
+  if (!result.ok || result.spaces.length !== LARGE_HOUSE_COUNTS.floors
+      || result.spaces.some((space) => space.status !== 'ok')) {
+    throw new Error(`candidate preflight failure: ${JSON.stringify(result.spaces)}`);
+  }
+};
+
+const sample = (operation) => {
+  const start = performance.now();
+  operation();
+  return performance.now() - start;
+};
+const run = (operation) => {
+  for (let index = 0; index < WARMUPS; index++) operation();
+  return Array.from({ length: SAMPLES }, () => sample(operation));
+};
+const quantile = (values, ratio) => {
+  const sorted = [...values].sort((a, b) => a - b);
+  return sorted[Math.min(sorted.length - 1, Math.ceil(sorted.length * ratio) - 1)];
+};
+const summary = (values) => ({
+  min: Math.min(...values),
+  median: quantile(values, 0.5),
+  p95: quantile(values, 0.95),
+  max: Math.max(...values),
+});
+
+const baseline = summary(run(directProductionPass));
+const candidate = summary(run(completePreflight));
+const relativeLimit = baseline.p95 * RELATIVE_RATIO + RELATIVE_NOISE_MS;
+const pass = candidate.p95 <= ABSOLUTE_P95_MS && candidate.p95 <= relativeLimit;
+const report = {
+  issue: 199,
+  fixture: LARGE_HOUSE_COUNTS,
+  warmups: WARMUPS,
+  samples: SAMPLES,
+  baseline,
+  candidate,
+  budgets: {
+    absoluteP95Ms: ABSOLUTE_P95_MS,
+    relativeRatio: RELATIVE_RATIO,
+    relativeNoiseMs: RELATIVE_NOISE_MS,
+    relativeLimitP95Ms: relativeLimit,
+  },
+  pass,
+};
+console.log(JSON.stringify(report, null, 2));
+if (!pass) process.exitCode = 1;
@@ -29,3 +29,32 @@ export async function assertFreshDemoBundle(page, root = process.cwd()) {
  }
  return expected;
 }
+
+/** Env switch that lets a debugging session run against a stale bundle. */
+export const ALLOW_STALE_BUNDLE = 'HP_ALLOW_STALE_BUNDLE';
+
+/**
+ * The freshness gate for every browser check, escape hatch included (#236).
+ *
+ * The smoke launcher had no freshness check at all, while golden runs and
+ * benchmarks did. A smoke against a stale `demo/srv/assets/houseplan-card.js`
+ * does not fail cleanly: on #234 three assertions went red and a fourth went
+ * GREEN, because the old code was wrong in two places that agreed with each
+ * other. A partly-red partly-green result looks like a logic defect and sends
+ * the reader hunting in the wrong file.
+ *
+ * Skipping is allowed for debugging, but never silently: a skipped guard that
+ * says nothing is the same silent success this project keeps digging out.
+ */
+export async function assertFreshDemoBundleUnlessAllowed(
+  page, root = process.cwd(), env = process.env,
+) {
+  if (env[ALLOW_STALE_BUNDLE]) {
+    console.warn(
+      `[houseplan] ${ALLOW_STALE_BUNDLE} is set — bundle freshness NOT verified. `
+      + 'A red result may mean a stale bundle rather than a defect (#236).',
+    );
+    return null;
+  }
+  return assertFreshDemoBundle(page, root);
+}
@@ -0,0 +1,202 @@
+#!/usr/bin/env node
+// Issue #211: human-reviewable Reference SVG <-> Runtime matrix.
+import { mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { resolve } from 'node:path';
+import { mdiLightbulbSpot } from '@mdi/js';
+import { launch } from './serve.mjs';
+
+const artifactDir = resolve('artifacts/device-icon-reference');
+mkdirSync(artifactDir, { recursive: true });
+
+const referenceAsset = (theme, file, coreSize) => {
+  let source = readFileSync(resolve('demo/srv/reference/device-icons', theme, file), 'utf8');
+  if (file === 'Lock.svg') {
+    const old = theme === 'Dark' ? '#252525' : 'black';
+    source = source.replaceAll(old, '#66D17A');
+    if (theme === 'Dark') source = source.replaceAll('fill="white"', 'fill="#252525"');
+  }
+  if (file === 'Unlock.svg') {
+    source = source.replaceAll(theme === 'Dark' ? '#1DC21D' : '#F0A00C', '#F0410C');
+  }
+  const nativeWidth = Number(source.match(/<svg[^>]*width="([\d.]+)"/)?.[1] || 127);
+  return {
+    url: `data:image/svg+xml;base64,${Buffer.from(source).toString('base64')}`,
+    displayWidth: nativeWidth * coreSize / 80,
+  };
+};
+
+const { page, browser } = await launch(
+  { width: 1280, height: 960 }, 1, [], { colorScheme: 'dark' },
+);
+
+await page.evaluate((path) => { window.__ICONS['mdi:lightbulb-spot'] = path; }, mdiLightbulbSpot);
+
+await page.evaluate(async () => {
+  const c = window.__card;
+  const marker = (id, patch) => ({
+    ...(c._serverCfg.markers || []).find((item) => item.id === id),
+    id, binding: `device:${id}`, ...patch,
+  });
+  const replacements = new Map([
+    ['d_light1', marker('d_light1', { display: 'badge', icon: 'mdi:lightbulb-spot' })],
+    ['d_tv', marker('d_tv', { display: 'value' })],
+    ['d_temp', marker('d_temp', {
+      display: 'badge',
+      value_badge: {
+        enabled: true,
+        source: { kind: 'entity_state', entity_id: 'sensor.living_temp' },
+        position: 'right',
+      },
+    })],
+  ]);
+  c._serverCfg.markers = [
+    ...(c._serverCfg.markers || []).filter((item) => !replacements.has(item.id)),
+    ...replacements.values(),
+  ];
+  c.hass = {
+    ...c.hass,
+    states: {
+      ...c.hass.states,
+      'sensor.living_temp': {
+        ...c.hass.states['sensor.living_temp'],
+        state: '23',
+        attributes: { ...c.hass.states['sensor.living_temp']?.attributes, unit_of_measurement: '%' },
+      },
+      'media_player.tv': {
+        ...c.hass.states['media_player.tv'],
+        state: 'Working',
+      },
+    },
+  };
+  c._regSignature = '';
+  c._cfgEpoch++;
+  c._maybeRebuildDevices();
+  c._setMode('view');
+  c.requestUpdate();
+  await c.updateComplete;
+  const qaStyle = document.createElement('style');
+  qaStyle.textContent = '.devtip{display:none!important}';
+  (c.renderRoot || c.shadowRoot).append(qaStyle);
+  await new Promise((resolveFrame) => requestAnimationFrame(() => requestAnimationFrame(resolveFrame)));
+});
+
+const selector = (id) => `.dev[data-id="${id}"]`;
+
+async function runtimePng(theme, row, size) {
+  await page.mouse.move(1, 1);
+  await page.evaluate(({ id, themeName, classes, px, clearValues }) => {
+    const node = (window.__card.renderRoot || window.__card.shadowRoot)
+      .querySelector(`.dev[data-id="${id}"]`);
+    for (const marker of (window.__card.renderRoot || window.__card.shadowRoot).querySelectorAll('.dev'))
+      marker.style.visibility = marker === node ? 'visible' : 'hidden';
+    node.classList.remove(...[
+      'theme-light', 'theme-dark', 'on', 'open', 'alarm', 'unavail', 'virtual',
+      'sel', 'lock-locked', 'lock-unlocked',
+    ]);
+    node.classList.add(`theme-${themeName}`, ...classes);
+    node.style.setProperty('--device-base-size', `${px}px`);
+    node.style.setProperty('--dev-scale', '1');
+    node.querySelector('.device-core')?.style.setProperty('transition', 'none');
+    node.querySelector('.device-shell-frame')?.style.setProperty('transition', 'none');
+    if (clearValues) node.querySelectorAll('.value-badge').forEach((value) => value.remove());
+    node.blur();
+  }, {
+    id: row.id,
+    themeName: theme.toLowerCase(),
+    classes: row.classes || [],
+    px: size,
+    clearValues: row.clearValues || false,
+  });
+  if (row.hover) {
+    await page.hover(selector(row.id));
+    await page.waitForTimeout(180);
+  }
+  if (row.focus) {
+    await page.$eval(selector(row.id), (node) => node.focus());
+  }
+  await page.$eval(selector(row.id), (node) => {
+    for (const tooltip of (window.__card.renderRoot || window.__card.shadowRoot).querySelectorAll('.devtip'))
+      tooltip.style.setProperty('display', 'none', 'important');
+    node.querySelector('.lqi')?.style.setProperty('display', 'none');
+  });
+  const clip = await page.$eval(selector(row.id), (node) => {
+    const shell = node.querySelector('.device-shell-frame').getBoundingClientRect();
+    const pad = 22;
+    return {
+      x: Math.max(0, shell.left - pad),
+      y: Math.max(0, shell.top - pad),
+      width: shell.width + pad * 2,
+      height: shell.height + pad * 2,
+    };
+  });
+  return (await page.screenshot({ clip })).toString('base64');
+}
+
+const rows = [
+  { label: 'Default', file: 'Icon Default.svg', id: 'd_light1' },
+  { label: 'Hover', file: 'Icon Hover.svg', id: 'd_light1', hover: true },
+  { label: 'Active', file: 'Icon Active.svg', id: 'd_light1', classes: ['on'] },
+  { label: 'Lock', file: 'Lock.svg', id: 'd_lock', classes: ['lock-locked'] },
+  { label: 'Unlock', file: 'Unlock.svg', id: 'd_lock', classes: ['lock-unlocked'] },
+  { label: 'Selected', file: 'Selected.svg', id: 'd_light1', classes: ['sel'] },
+  { label: 'Focus', file: 'Focus Visible.svg', id: 'd_light1', focus: true },
+  { label: 'Alert', file: 'Alert Value.svg', id: 'd_temp', classes: ['alarm'] },
+  { label: 'Virtual', file: 'Virtual Device Default.svg', id: 'd_motion', classes: ['virtual'] },
+  { label: 'Unavailable', file: 'Unavailable.svg', id: 'd_light1', classes: ['unavail'] },
+  { label: 'Text', file: 'Text Default.svg', id: 'd_tv' },
+  { label: 'Double Right', file: 'Double Default Right.svg', id: 'd_temp' },
+];
+
+const matrix = [];
+for (const theme of ['Light', 'Dark']) {
+  for (const row of rows) {
+    matrix.push({
+      theme,
+      row,
+      size: 56,
+      runtime: await runtimePng(theme, row, 56),
+    });
+  }
+  for (const size of [32, 96]) {
+    const row = rows[0];
+    matrix.push({ theme, row, size, runtime: await runtimePng(theme, row, size) });
+  }
+  const textRow = rows.find((row) => row.label === 'Text');
+  matrix.push({ theme, row: textRow, size: 96, runtime: await runtimePng(theme, textRow, 96) });
+}
+
+const escapeHtml = (value) => String(value)
+  .replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;');
+const body = matrix.map(({ theme, row, size, runtime }) => {
+  const reference = referenceAsset(theme, row.file, size);
+  return `
+  <tr>
+    <td>${theme}</td><td>${escapeHtml(row.label)}</td><td>${size}px</td>
+    <td class="preview"><img style="width:${reference.displayWidth}px" src="${reference.url}" alt="Reference ${escapeHtml(row.label)}"></td>
+    <td class="preview runtime"><img src="data:image/png;base64,${runtime}" alt="Runtime ${escapeHtml(row.label)}"></td>
+  </tr>`;
+}).join('');
+const html = `<!doctype html>
+<html><head><meta charset="utf-8"><title>Device icon reference/runtime matrix</title>
+<style>
+  body{margin:24px;background:#777;color:#111;font:16px system-ui,sans-serif}
+  h1,p{max-width:1100px} table{border-collapse:collapse;width:100%;background:#aaa}
+  th,td{border:1px solid #555;padding:8px;text-align:left} th{position:sticky;top:0;background:#ddd;z-index:2}
+  .preview{width:38%;text-align:center;background:linear-gradient(135deg,#d5d5d5 50%,#666 50%)}
+  .preview img{display:block;margin:auto;max-width:300px;max-height:180px}.runtime img{image-rendering:auto}
+</style></head><body>
+<h1>House Plan device icons: package 1.1.1 vs runtime</h1>
+<p>Issues #211/#217. Reference SVG is loaded directly from the designer package; Runtime is a fresh browser capture. Default covers 32/56/96 px and Text has an additional large 96 px row so its outer stadium curvature is reviewable. Dark Unlock is evaluated using the owner's amber override from #179.</p>
+<table><thead><tr><th>Theme</th><th>State/layout</th><th>Core</th><th>Reference SVG</th><th>Runtime</th></tr></thead>
+<tbody>${body}</tbody></table></body></html>`;
+const htmlPath = resolve(artifactDir, 'device-icons-reference-runtime.html');
+writeFileSync(htmlPath, html);
+
+await page.setViewportSize({ width: 1600, height: 1000 });
+await page.setContent(html, { waitUntil: 'load' });
+await page.screenshot({
+  path: resolve(artifactDir, 'device-icons-reference-runtime.png'),
+  fullPage: true,
+});
+await browser.close();
+console.log(`OK device icon reference/runtime matrix: ${htmlPath}`);
@@ -1,72 +1,91 @@
 {
  "schema": 1,
-  "matrixVersion": 24,
-  "acceptedAt": "2026-08-17T10:51:28.998Z",
-  "sourceFingerprint": "b73e5020215967d2d13e2a08ce303916e2bdbcbd45c94c40c6e56336a8097656",
+  "matrixVersion": 33,
+  "acceptedAt": "2026-08-21T12:03:00.868Z",
+  "sourceFingerprint": "73154ad392659d101a708f2bce5f02f2851e810a615bda97d19172cdb576c6bd",
  "chromium": "151.0.7922.34",
  "scenarios": {
    "split-corner-wall-before-dark": "3176dc67f54d5309f87c94e1077b4f69eb1db9f660469fbf97953038323430f3",
    "split-corner-wall-thin-dark": "6da64905a3a4f8e4b4d457e5b20d2d55e0e7c2c601316a088c4bcc6557d35cc6",
    "split-corner-wall-thick-dark": "494d559aa71ee85f90b8cfa11c1d3087fa123e963dec2b0975e7ce6c1520852a",
-    "isometric-geometry-view-dark": "4816ec4b22211c73765f9fac81741df685202d21df6b0fbc1649b23357086da3",
-    "isometric-geometry-view-light": "8ec81fc4f254ae6ebab55bc5f0ef5325a4e5d327cd1fb53bf5f293f06fbec671",
-    "isometric-live-layers-dark": "c841269f672c7ad8b208a549cc87592bd26f2a9e226f793a2b79592b65ee54c6",
-    "isometric-no-borders-dark": "36f972f95704bff81ea1a59bdf3cd2cf7b636ec7871780460e98b23e3ebc3da2",
-    "isometric-touch-kiosk-dark": "fd2e74c5966b4adcf4e66021e2cef781873f9a2b58f16ad19d57fb451ba45dc9",
-    "isometric-large-warm-remount-dark": "0afc1069d334f10be2d0ed135ee0eb52e9272a1a1ad1224e00f05753b8789d0c",
-    "geometry-view-dark-fit": "3df272f6c3c3d20e9e375ea037f3dbb885657b94b0b29d0067505f4a73741237",
-    "geometry-view-light-fit": "a7f2c9667d9872dd84a37d5318fd238c9eabcc0f413017eb19e02204587b4e1a",
-    "washer-active-cycle-dark": "7597ffad91165f2f3cc9647e5081c03d4ffd722313bf74f8dcf5fe6dc52d1dcf",
-    "washer-idle-cycle-dark": "6b1de9786f4852659d2d453fa39340626a7944417f88c5a37355b62c458b6ac1",
-    "day-cycle-dawn-dark": "a573083c4ee21993cf2e482b60418a30e388a14117d056a2b0ea559a0c928039",
-    "day-cycle-day-dark": "6f24cd1d8669c9f3451c06ca5a5fd499d8f1ae2698a34f4a63ffee2e9e6b4330",
-    "day-cycle-dusk-dark": "80029577c25f8759090ee2550fe530a35c189806dd6fa26e887da4f5d1cf54a3",
-    "day-cycle-night-dark": "d855785914d3e11198d3e4671c1fbcad15104c95954bc1ed9b7366aa51aac65e",
-    "geometry-plan-editor-dark": "16364738754515e81e6d0ae13c0358db8c7f9d26c2f34dfde15f5855c6af13fa",
-    "plan-snap-endpoint-light": "c5f63ca2ca2706a062a2fc97e25670768662810bd7e0b251f0a9cfdbaf60c758",
-    "plan-snap-line-gaps-dark": "44808a816e62416b9c2c39a6e06cd4a8631860e178f246772ce8c7a46f98edd0",
-    "wall-junctions-plan-preview-light": "9e3a07da3e3ae1b2a95299f92b9500f347f0bfbd20d87429505b30769ee627c2",
-    "wall-junctions-plan-t-dark": "a4a958f20ed5f4b8d4e6bca9ebd1b289186c0cb48b48897a1624b49014f3ae5d",
-    "wall-junctions-view-dark": "73a64c65c8e77aa767bb7f401d68c61df5749e65e75273c85b0d6c72cb430766",
-    "isometric-wall-junctions-dark": "cb1e28f484b304ecda3e35f66e0c0429016d10a5a3022dcc9315a92eb537300f",
-    "opening-placement-door-thick-wall-dark": "395c03bbf5d968e83664fd6621f0ac25902e718022f2e92ffbcddb8ce629cf9c",
-    "geometry-devices-editor-dark": "a9e4846ce5453400b87e6ad3d575882bc23a07b59dc3eecb612ed872b0c871ec",
-    "geometry-decor-editor-dark": "435b36096bbb2996d56ff0af262ddebff4a727edd841b0fad9b8d4f507b987ac",
-    "tray-wide-selection-en": "06b0df980fd79e8bfc2957878966a11ae9d1a61e80ee870094dc60eae4de26bb",
-    "tray-wide-tool-ru": "e5a6b2057acd9af2c5417bf413778395112eb1ebecd784251c68c74a16b9dc5f",
-    "tray-medium-group-en": "5115910bc0f359ce91f361794a51ae1ae6a493203941d09166b412b696eda775",
-    "tray-medium-selection-ru": "4e5f235be8ed6296e136641d172e727a6a6b7a9d061f0928a96ccc1103c19f1f",
-    "tray-narrow-palette-en": "88b9846e4b451ed95b7ae7d2c3183a2ea191d7668768992a1364c6a7a53eb0c6",
-    "tray-narrow-tool-ru": "c4130715b3cb31c68619dfc706a3aa308e86edf272bfae6833b20666764ece2b",
-    "geometry-diagonal-45-opening-dark": "01206d25631c8fd09fa077932fbb5c1ee115b65ba76b38e6f7b09315dbbf6002",
-    "openings-thick-wall-dark": "5aa0b3d26894bef9ab9fca25c31bbef2f13f2c410f5f6d3f61c8d608ceb929f8",
-    "openings-filled-tunnel-dark": "167d92c11e6a8b3ff0f31177ac5905f8db4b5fb03ee78b4965c40bc45aeee50f",
-    "openings-hidden-view-dark": "c85cc04d1d8622b98215e2bb83f5bb233a7cfb0ac684c912475ef7bc44245897",
-    "lighting-glow-sun-dark": "a98eee332f25a43c8d9d126c118c8cfea4ee73752b1060227548efedaf0efcdd",
-    "device-value-badge-positions-dark": "1ad43f2bd866733aa75c34de38fb97d469661799b540a8ae22150067d788cec8",
+    "split-zero-divider-taper-dark": "3de38befb41f15ef4047da1390e061b1b5756340142dd9204d011287ff39be5b",
+    "isometric-geometry-view-dark": "9bc8eb0da8746bddc0b5245b337eef7477d4ce6fb39ac576cab1e49057fd837c",
+    "isometric-geometry-view-light": "144f1cc3107562bc252cdcb54165f2c92569f4cf3484554148a7fdb78de372ec",
+    "isometric-live-layers-dark": "01660f4cf3e7344a87c9acc9e80239757b0ef5f365a37257c4b147144399b7cc",
+    "isometric-no-borders-dark": "ac92e782e5f58e514c3731f742ead6939cde3344810fe2f2080876b3deb1335b",
+    "isometric-touch-kiosk-dark": "745482a797e236058e3ddd56cb5d85d700c9333a3aeb764333addeaba40cf6b0",
+    "isometric-large-warm-remount-dark": "37b47189e395d56e746501955a5b215298a3261a49c7f27e248b7957b2f31e46",
+    "geometry-view-dark-fit": "438817c56cd8f91ef63778a3d3d22a4ee1fb675064bc24bf613fb752d6c503d9",
+    "geometry-view-light-fit": "667d38fd55a946ea6930b8674c4036e75c261ba4f111f893367ae72ed19500fc",
+    "room-label-parity-view-dark": "3a636de579cd2d1de338d017ed6b53c3e2f70a901a0f5c81dfca85870d1b4bd5",
+    "room-label-parity-plan-dark": "e782f0386a6b1e75c7bb9f135e9ec01a9ff4b540f0384d518c7ea9f2198683ea",
+    "room-label-parity-view-light": "6eb2ba0023932c9b4e82a45b7ca9565a9ed1b89d14e543facbaf1f84aa5db9ff",
+    "room-label-parity-plan-light": "333139affbf8936554072aebcaf58b4640779adbd6a64686c0929ca071063733",
+    "washer-active-cycle-dark": "d74a646bc71b8d6d0d5b09e6862b4846b0e369da2520afa76d5b3d601c832a15",
+    "washer-idle-cycle-dark": "f7fae5c2f6856a3df5bd3f3028fcf829ef0039c7b43c91c27ac86d25626e8d66",
+    "day-cycle-dawn-dark": "289e6edd6c206308d3257775b2948085a24698398995bc6ce72201745705c58c",
+    "day-cycle-day-dark": "df27409e83f4492d3f69dc6cebf2a6e7799e154698425b17fb3ab8cfddb426a6",
+    "day-cycle-dusk-dark": "9798e9dac27e69727adbe9c9a782a3bd3dc5dd0850e7a49852c069d54b382f9a",
+    "day-cycle-night-dark": "1abf9528ac05dc3a963eb19a8b3ed4b7bb9ce647f7b83dab4fd2e736756cd770",
+    "geometry-plan-editor-dark": "a3828bdd877c23d283dcd30dc88287c2104c8d39e849aa826a76190d7daa30ae",
+    "plan-snap-endpoint-light": "2df0bd2abcb615d6bd1a4422b29747f95a1edd7b02d4ae3a6b333e24b27e9f78",
+    "plan-snap-line-gaps-dark": "a29d42ae49a0a513dc36e933bdd2b5f3cb97aa40b0e7ec897c602fe65a5a5fe8",
+    "wall-junctions-plan-preview-light": "cdb96edabd5b04a47e5026eed4255e06b802bd8ba7f0c7ba2cab66c4ae2f371a",
+    "wall-junctions-plan-t-dark": "ec2b29ca87bba9b41e4c22ce94d2c4fe06c6d9c0833152d3a6224371763ae6bc",
+    "wall-junctions-view-dark": "7b859c4f25f8a5dd4fca64d5b2f7aa64b65845fabbe83bc2388c2f99d29f71b1",
+    "junction-patch-resilience-plan-dark": "c42203eea92d2bc972eed829aa726ad51f36fc76d0ae049730d0f4155ef8efbf",
+    "junction-patch-resilience-view-dark": "d531f2a01def73fb074e55543c8543112af158df65f24dc549e0da5a23f1af40",
+    "isometric-wall-junctions-dark": "d1956cbbde9a6a02953ce80eb8a0f74ca06ea268a4c60bb09b5b3981c26d9731",
+    "opening-placement-door-thick-wall-dark": "c398f53391bc697e859ba389bac0ce586526d6f14c85f32d9a4b314bebe902ab",
+    "opening-placement-passage-thick-wall-dark": "98bb63f72886986d71d598fffcaa4fb8e7dce3e7f3a27b5afb6012f9eac4537c",
+    "opening-placement-passage-thick-wall-light": "2f19063a0e792c7bec92f3e4069739954dcdd79eb4933355a575aec5463b9ade",
+    "geometry-devices-editor-dark": "2ec2647797c71f6c6406f6e33fcbf70d5cb6d36a01a65c70ecabc7ad11e86183",
+    "geometry-decor-editor-dark": "46798a59d99982c4e73596861e20b6e466f6e7aa1d70a26bf917e1bd3bf73cb1",
+    "tray-wide-selection-en": "457b53c4e5ae5f691457f98f3c2fd255e5b7ed1c599af92c303c93c11f87b7d4",
+    "tray-wide-tool-ru": "1fcc25ee374c95f51c71b33720daa37f82a92ccd52119b8bdb48de5484591e49",
+    "tray-medium-group-en": "3bebd852929ff67cff11827d09c4859e32e17fa0be36fcabd4a591c4728a4827",
+    "tray-medium-selection-ru": "e40dcdadf3ebdbec200175dc6648382daa91a232d081c8add1f6646c7d1a0ee5",
+    "tray-narrow-palette-en": "861adc403fbf0aff1e45b27fc07c4856f3ca409b58ef87cc4a81d870a09dbb9a",
+    "tray-narrow-tool-ru": "ccf11295b104cecd4bf1e1bc95417003e2c40cfba29b5555ac9ba021b90d16a6",
+    "geometry-diagonal-45-opening-dark": "dd93866af62313806a4444b707943c693e2b7709367156ebdc3b80168233bbe8",
+    "openings-thick-wall-dark": "0c33cdb4637ee9788e75931ba142099e1d068b770e26ca24f4a59f8631f3373f",
+    "openings-filled-tunnel-dark": "bb2a4343a285e2fd867c6090108589549facf87b1a9471f3ae331d743af6653c",
+    "openings-hidden-view-dark": "19ab4fd2ffce0fea0fe609b007a3e60f7abd84b40935cbf3c5fa9dc1fda21046",
+    "lighting-glow-sun-dark": "3f02abbcb3379b2e224ba7dd3126ec4cb736c0f71edbcdd2f2961c33d7d1b12a",
+    "device-value-badge-positions-dark": "831f04e8b97e4cb473e2c9e53976a5422dd3718eb74086ed2503ffc6d3f5ad87",
+    "device-icon-state-table-light": "b956debb1e3845c03b1a69ba1a45f65b682ea6ee058f40003657bb6f5ad09ee8",
+    "device-icon-state-table-dark": "bf3caed74f7832a142f730481e50f9e2706aeeee009b39118a9ac4d011954e7c",
+    "device-text-shell-long-light": "4ad4f05ebd24d65b98bd4c32f71184088062ae75a52a5a7cc54300ea1961b924",
+    "device-text-shell-long-dark": "f35d426dccd0705465bfedc112776df06233aaa5ffc37765dd71222787cfb3d6",
    "lighting-sun-window-state-only-dark": "3bd581a23a2e0ebba58530db5182adea5cba6ec10bc032bee024415c19108a17",
-    "lighting-fill-light-axis-split-dark": "4f867528aeb9124229f81659876b03ff297a3a7a92bc57ffb32d5c24913c4938",
-    "lighting-fill-temp-axis-split-dark": "e0535b70701c9fe6753f74c943b9f288a0fb1a23a9cfc8590d4945e0a1724eb5",
-    "lighting-fill-lqi-axis-split-dark": "485ab183144913569ddc11154553ab4ac7db522ab188de74d652b717dc786d9c",
-    "lighting-temp-glow-dark": "ecaed039fb6aab4e1fdc9f1856c89e5f563219b8ff813877027737cecbeca41a",
-    "lighting-temp-glow-light": "5bc8a35aaa94c427d465d198f1cd5eecfdc704f5abeded9e407f32ce93aa7d32",
-    "lighting-custom-glow-dark": "899e334dfc0d3490ca291b7239bf7b88ca9d1ef3be199ddf347314bab1513f27",
-    "lighting-opaque-glow-two-doorways-dark": "413f5a8e39193ba941f72955a391ccac954c09f24322b7bf67494c7691275980",
-    "lighting-custom-glow-light": "266bba4ae1744a884b2cd224b37fdc36447d57405a1c5298ca30027e2957cc8a",
-    "lighting-temp-glow-no-sources-dark": "5100c81543fc30a7934a6db6f9e67e2c4fa185df0a974be5911879e43c0d3fc9",
-    "lighting-temp-glow-room-override-dark": "0a35d3508526187ea18e44456cfb8cd9e578a1864e896fad1c3eec2892c753e0",
-    "lighting-manual-auto-spill-overlap-dark": "6324dbe2079a255e7a194720c8c19f210549ac734e564270bc1373e6385b9cac",
-    "hover-over-glow-dark": "fc14ba6f6b670e61c0fb5be277e67551ea2da7a06b5c167a8c2c989f1de08910",
-    "hover-nested-room-dark": "6c09526ad885c4555063def5b43287b41124a81d90972c425084dba6e622d055",
-    "large-house-zoom-040-dark": "5f11c4b78318a64c2a7cf803716661eea506609d4f0a6bb3d64a709f8c49db1d",
-    "large-house-zoom-250-dark": "c906426f888ff4e306c5c334c6329b387fc5ca368e229f55acc33c202351a1ac",
-    "large-house-warm-remount-dark": "6baf4baed1c735c64dfe1e69d9864ca287ffc0e8452d00e801f0873e98b187ee",
-    "device-dialog-desktop-en": "d6fcc83aa1335df1041e2b1aa445b0019d1e3567f98ef8f47a889894051f2b62",
-    "device-dialog-mobile-ru": "8cb928853ddacb61882804c3d00ead31da31bc4559ee6a8e293ef6b55cd5a463",
-    "device-help-popover-light-ru": "f1bf21d62a5dd349aa57b746069c5aef58d7a26b0b9d9e0c233fde0c1d56d7eb",
-    "decor-color-popover-mobile-ru": "46d4c2e4dd20c3a38e90efe3db59b3e878bdbcf273fbc1aa23de4b230723fa6e",
-    "backup-full-preview-desktop-en": "cc42a621f55f043b272014b9c32127823ca3573520e502966c71642027f7fdaf",
-    "backup-plan-only-export-desktop-en": "2833ee45acac2936e76546e9ff5d0031932c02fbf66fe904c1be94dc3e33dc05",
+    "lighting-fill-light-axis-split-dark": "4cb3f7591f005768d1d8e42f651184a7317b439318302d6bc48fcca000266c82",
+    "lighting-fill-temp-axis-split-dark": "47383a2e81b34cd7cf385a0f33efe755906c1cb20de8345dc69f599597378801",
+    "lighting-fill-lqi-axis-split-dark": "480b518fb2da24fc7d70297a4456a92485f5f5629a6bc951a36eedcc7631e5dc",
+    "lighting-temp-glow-dark": "a5c8f7b981d4f310ca8cf6c19f72236dc29abdd88a1d60be6ef916dd9091f7b8",
+    "lighting-temp-glow-light": "86620d7b68e797469fbec36438bf2725bd913bc8b4641dbe75c03098210d17f7",
+    "lighting-custom-glow-dark": "86a1ba7ac8e6bc9515a4df550065c60fd600ce6898bdc5447122c9fc87fb3965",
+    "lighting-opaque-glow-two-doorways-dark": "0cff53678f935bb9cee4570a47bf34a05d71b1fd512d76158e846c0b1aa7a5c5",
+    "lighting-custom-glow-light": "2568f1ca7be270faabd2c97d955fd6e27dec00e921d8e6f21ab7425a86e7f958",
+    "lighting-temp-glow-no-sources-dark": "ee37a7dda19a0bfe02405d4e9cf61023cecb623896388a4197adc4a5afe6b584",
+    "lighting-temp-glow-room-override-dark": "fa20be4d2a29710143f437d5049c6e71959c6a56df2a12a9d2e9728e7833d9ed",
+    "lighting-manual-auto-spill-overlap-dark": "9bb31a72bfa073eae4630ea5edf01b5a7dbb1885eb25153c4bf3b2574fc6e5c5",
+    "hover-over-glow-dark": "d505d1514e492dd3f5e299ebace0392e14734f21626310a8efdc0d148a00ddf0",
+    "hover-nested-room-dark": "2db78e53a76fa9b7cdc4597f23fc2c8ae439e5a56d82895109ca968d73075640",
+    "large-house-zoom-040-dark": "a21faa9eafea706cf8d6366e93f526a8f83798b6b6cf058ee43010607182677d",
+    "large-house-zoom-250-dark": "48b13045d83ac83bc35aba7182057ec56023ac448ad57f6b7dc9981349cea563",
+    "large-house-warm-remount-dark": "99ef82407bd148cef968e99a593f25da06da2df7363af842f617113349c7ee85",
+    "device-dialog-desktop-en": "ea341f6ea9db5f61bbcb6fd7ca5078fd2f347023de4279cb70bf190c1379c364",
+    "device-dialog-mobile-ru": "c90be98e65c37963fd4443e413848ca68a412c9ee568d648aba76bf9232ee59d",
+    "toggle-entity-dialog-desktop-en": "f4924466de5e134c2d1f2456ff0c05f8aa8106f6b06398e87a263e3c536f16ea",
+    "toggle-entity-dialog-mobile-ru": "7d3aecd318c6c0774dd3ce1c21c0fafb1ae2be23cebed6f548fbf8bd11ab2f62",
+    "device-help-popover-light-ru": "c74c83ecfb4ee6cd693d87d4516e9ee4920ad453f5b7b9606cd98853b7273568",
+    "decor-color-popover-mobile-ru": "af002e4c3106c008bbdb58ed1f00d8f8415dc86a8dfab2552defb4d16d8bbe71",
+    "decor-color-popover-desktop-en": "11189416fd3752310979444aa92ce38764996655871009ff028ba9b4b31959a8",
+    "general-color-popover-desktop-en": "da85d0159802afce90fd87ecf940ef1b97ffd57c27fbdc79469cc11082b9a482",
+    "device-ripple-color-popover-mobile-ru": "3ab12c339106004c76fe3c7741d242269e157c40ddfa29afd8f13b445b0a7023",
+    "space-room-color-popover-desktop-ru": "a5d62d68097af1ff5d2556be0c889efc25768ffc61af5d302f0b5775e082f296",
+    "backup-full-preview-desktop-en": "6f0cfecf587b73f38088d414f489b67cc68ed71c37c96d4e307e66f6d97bda9f",
+    "backup-plan-only-export-desktop-en": "1e1c8a9cc5383394b91e24af8342d92103a5e85b3e5c9b53c45ab0e9b0d57da4",
    "backup-space-preview-mobile-ru": "a4719cbe29b378ef7baa63bb7ff23e201b2943025008d939f2c08cee63bb9038"
  }
 }
@@ -1,5 +1,10 @@
 import { makeLargeHouseFixture } from '../fixtures/large-house.mjs';
 import { fixtureWallKey, makeVisualMatrixFixture } from '../fixtures/visual-matrix.mjs';
+import { readFileSync } from 'node:fs';
+
+const junctionPatchFixture = JSON.parse(readFileSync(
+  new URL('../../test/fixtures/197-junction-patch.json', import.meta.url), 'utf8',
+));

 const fixtureFor = (scenario) => scenario.fixture === 'large'
  ? makeLargeHouseFixture()
@@ -139,11 +144,43 @@ export function prepareGoldenFixture(scenario) {
      wall_columns: [],
    });
  }
+  if (scenario.junctionPatchResilience) {
+    fixture.config.spaces.push({
+      ...structuredClone(junctionPatchFixture),
+      id: scenario.space,
+      title: 'Junction patch resilience',
+      view_box: [0, 0, 1, 1],
+      settings: {
+        ...(junctionPatchFixture.settings || {}),
+        fill_mode: 'none',
+        show_borders: true,
+        show_names: false,
+      },
+    });
+  }
  const requireSpace = () => {
    const space = fixture.config.spaces.find((item) => item.id === scenario.space);
    if (!space) throw new Error(`golden override references missing space: ${scenario.space}`);
    return space;
  };
+  if (scenario.roomLabelParity) {
+    const space = requireSpace();
+    if (space.id !== 'golden-lighting' || space.rooms.length !== 2)
+      throw new Error(`golden roomLabelParity requires the two-room lighting fixture: ${space.id}`);
+    space.settings = {
+      ...(space.settings || {}),
+      show_names: true,
+      label_temp: true,
+      label_hum: true,
+      label_lqi: true,
+      label_light: true,
+    };
+    fixture.layout = {
+      ...(fixture.layout || {}),
+      'rl_light-left': { s: space.id, x: 0.28, y: 0.24 },
+      'rl_light-right': { s: space.id, x: 0.72, y: 0.24 },
+    };
+  }
  if (scenario.deviceName) {
    if (!scenario.deviceId || !fixture.devices?.[scenario.deviceId])
      throw new Error(`golden deviceName references missing device: ${scenario.deviceId || '<empty>'}`);
@@ -151,6 +188,7 @@ export function prepareGoldenFixture(scenario) {
  }
  if (scenario.fillMode || scenario.bgMode || typeof scenario.glowEnabled === 'boolean'
      || typeof scenario.sunRays === 'boolean' || typeof scenario.showBorders === 'boolean'
+      || typeof scenario.showNames === 'boolean'
      || typeof scenario.northDeg === 'number') {
    const space = requireSpace();
    space.settings = {
@@ -160,6 +198,7 @@ export function prepareGoldenFixture(scenario) {
      ...(typeof scenario.glowEnabled === 'boolean' ? { glow_enabled: scenario.glowEnabled } : {}),
      ...(typeof scenario.sunRays === 'boolean' ? { sun_rays: scenario.sunRays } : {}),
      ...(typeof scenario.showBorders === 'boolean' ? { show_borders: scenario.showBorders } : {}),
+      ...(typeof scenario.showNames === 'boolean' ? { show_names: scenario.showNames } : {}),
      ...(typeof scenario.northDeg === 'number' ? { north_deg: scenario.northDeg } : {}),
      ...(scenario.customFill ? { custom_fill: scenario.customFill } : {}),
    };
@@ -276,9 +315,13 @@ export function prepareGoldenFixture(scenario) {

 export async function prepareGoldenScenario(page, scenario) {
  await stableEnvironment(page, scenario);
+  // Scenarios share one Playwright page, including its pointer position. A
+  // scenario that deliberately hovers a marker must not leave the next one
+  // capturing an unrelated room hover at the same viewport coordinates.
+  await page.mouse.move(0, 0);
  const fixture = prepareGoldenFixture(scenario);

-  return page.evaluate(async ({ fixture, scenario }) => {
+  const result = await page.evaluate(async ({ fixture, scenario }) => {
    const wait = (ms) => new Promise((done) => setTimeout(done, ms));
    const frame = () => new Promise((done) => requestAnimationFrame(() => requestAnimationFrame(done)));
    const until = async (predicate, timeout = 10000) => {
@@ -380,6 +423,14 @@ export async function prepareGoldenScenario(page, scenario) {
      await card.updateComplete;
      await settleMode(card);
    }
+    if (scenario.roomLabelParity) {
+      const labels = [...card.renderRoot.querySelectorAll('.roomlabel')];
+      if (labels.length !== 2
+          || labels.some((label) => !label.querySelector('.rlgo') || !label.querySelector('.rlmetrics'))
+          || labels.some((label) => !label.querySelector('.rlmetrics')?.textContent?.trim())) {
+        throw new Error(`golden room-label parity core is incomplete: ${scenario.id}`);
+      }
+    }
    if (scenario.projection === 'iso' && typeof card._setProjection === 'function') {
      card._setProjection('iso');
      await card.updateComplete;
@@ -416,7 +467,7 @@ export async function prepareGoldenScenario(page, scenario) {
      const { tool, anchor, pointer, expectedKind } = scenario.planSnap;
      const validPoint = (point) => Array.isArray(point) && point.length === 2
        && point.every(Number.isFinite);
-      if (!['draw', 'partition'].includes(tool) || !validPoint(pointer)
+      if (tool !== 'draw' || !validPoint(pointer)
          || (anchor != null && !validPoint(anchor))
          || !['endpoint', 'line'].includes(expectedKind)) {
        throw new Error(`invalid golden planSnap contract: ${scenario.id}`);
@@ -447,7 +498,7 @@ export async function prepareGoldenScenario(page, scenario) {
    }
    if (scenario.openingPreview) {
      const { type, pointer } = scenario.openingPreview;
-      if (!['window', 'door', 'gate'].includes(type)
+      if (!['window', 'door', 'passage', 'gate'].includes(type)
        || !Array.isArray(pointer) || pointer.length !== 2
        || !pointer.every(Number.isFinite)) {
        throw new Error(`invalid golden openingPreview: ${scenario.id}`);
@@ -471,7 +522,11 @@ export async function prepareGoldenScenario(page, scenario) {
      await card.updateComplete;
      await frame();
      const preview = card.renderRoot.querySelector(`.opening-preview[data-kind="${type}"]`);
-      if (!preview || !preview.querySelector('.op-leaf')) {
+      const expectedGeometry = type === 'passage'
+        ? !!preview?.querySelector('.passage-preview-cut')
+          && preview.querySelectorAll('.passage-preview-boundary').length === 2
+        : !!preview?.querySelector('.op-leaf');
+      if (!preview || !expectedGeometry) {
        const intervals = card._openingPlacementIntervalsCache?.value || [];
        const nearest = intervals.map((interval) => {
          const [px, py] = card._cursorPt || [0, 0];
@@ -544,7 +599,36 @@ export async function prepareGoldenScenario(page, scenario) {
      card.requestUpdate();
      await card.updateComplete;
    }
-    if (scenario.dialog === 'device') {
+    if (scenario.dialog === 'optimize-preflight') {
+      const names = [
+        'Ground floor', 'Garage', card._t('gs.align_preflight_space', { n: '3' }), 'Attic',
+      ];
+      const failures = names.map((displayName, index) => ({
+        spaceId: `golden-failure-${index + 1}`,
+        displayName,
+        status: 'failed',
+        reason: 'wall-null',
+      }));
+      card._alignDialog = {
+        report: {}, config: card._serverCfg, layout: card._layout,
+        preflight: {
+          fingerprint: 'golden-optimize-preflight',
+          spaces: failures,
+          failures,
+          ok: false,
+        },
+        cm: 0, where: '', changed: true, busy: false,
+      };
+      card.requestUpdate();
+      await card.updateComplete;
+      const dialog = card.renderRoot.querySelector('hp-dialog');
+      const body = dialog?.querySelector('.body');
+      if (!body?.textContent?.includes(names[0])
+          || !body.textContent.includes(card._t('gs.align_preflight_hint'))
+          || dialog.querySelector('.btn.on')) {
+        throw new Error('golden Optimize preflight failure dialog is incomplete');
+      }
+    } else if (scenario.dialog === 'device') {
      card._setMode('devices');
      await card.updateComplete;
      await settleMode(card);
@@ -663,6 +747,74 @@ export async function prepareGoldenScenario(page, scenario) {
      if (!trigger) throw new Error('golden decor color trigger missing');
      trigger.click();
      await picker.updateComplete;
+    } else if (scenario.dialog === 'general-color') {
+      card._openSettingsDialog();
+      await card.updateComplete;
+      const dialog = card.renderRoot.querySelector('hp-dialog');
+      const picker = [...(dialog?.querySelectorAll('hp-color-opacity') || [])]
+        .find((item) => item.label === card._t('gs.light_on'));
+      await picker?.updateComplete;
+      const trigger = picker?.renderRoot?.querySelector('.trigger');
+      if (!trigger) throw new Error('golden general-settings color trigger missing');
+      trigger.scrollIntoView({ block: 'center' });
+      await frame();
+      trigger.click();
+      await picker.updateComplete;
+    } else if (scenario.dialog === 'device-ripple-color') {
+      card._setMode('devices');
+      await card.updateComplete;
+      await settleMode(card);
+      const device = card._devices.find((item) => item.id === scenario.deviceId);
+      if (!device) throw new Error(`golden ripple device missing: ${scenario.deviceId}`);
+      card._openMarkerDialog(device);
+      card._markerDialog = { ...card._markerDialog, display: 'icon_ripple' };
+      card.requestUpdate();
+      await card.updateComplete;
+      const dialog = card.renderRoot.querySelector('hp-dialog');
+      const picker = [...(dialog?.querySelectorAll('hp-color-opacity') || [])]
+        .find((item) => item.label === card._t('marker.activity_color'));
+      await picker?.updateComplete;
+      const trigger = picker?.renderRoot?.querySelector('.trigger');
+      if (!trigger) throw new Error('golden ripple color trigger missing');
+      trigger.scrollIntoView({ block: 'center' });
+      await frame();
+      trigger.click();
+      await picker.updateComplete;
+    } else if (scenario.dialog === 'space-room-color') {
+      card._openSpaceDialog('edit', scenario.space);
+      await card.updateComplete;
+      const dialog = card.renderRoot.querySelector('hp-dialog');
+      const picker = [...(dialog?.querySelectorAll('hp-color-opacity') || [])]
+        .find((item) => item.label === card._t('space.room_color'));
+      await picker?.updateComplete;
+      const trigger = picker?.renderRoot?.querySelector('.trigger');
+      if (!trigger) throw new Error('golden space room-color trigger missing');
+      trigger.scrollIntoView({ block: 'center' });
+      await frame();
+      trigger.click();
+      await picker.updateComplete;
+    }
+    if (scenario.deviceClassOverrides) {
+      for (const [id, classes] of Object.entries(scenario.deviceClassOverrides)) {
+        const marker = card.renderRoot.querySelector(`[data-hp="device"][data-id="${CSS.escape(id)}"]`);
+        if (!marker || !Array.isArray(classes) || !classes.length) {
+          throw new Error(`invalid golden device class override: ${scenario.id}/${id}`);
+        }
+        marker.classList.add(...classes);
+      }
+    }
+    if (scenario.deviceOnly) {
+      const markers = [...card.renderRoot.querySelectorAll('[data-hp="device"]')];
+      const selected = markers.find((marker) => marker.dataset.id === scenario.deviceOnly);
+      if (!selected) throw new Error(`golden isolated device missing: ${scenario.deviceOnly}`);
+      for (const marker of markers) marker.style.visibility = marker === selected ? 'visible' : 'hidden';
+    }
+    if (scenario.focusDevice) {
+      const marker = card.renderRoot.querySelector(
+        `[data-hp="device"][data-id="${CSS.escape(scenario.focusDevice)}"]`,
+      );
+      if (!marker) throw new Error(`golden focus device missing: ${scenario.focusDevice}`);
+      marker.focus({ focusVisible: true });
    }
    if (scenario.dayCycle) {
      const environment = card.renderRoot.querySelector('.hp-day-cycle-env');
@@ -697,6 +849,27 @@ export async function prepareGoldenScenario(page, scenario) {
      } } : {}),
    };
  }, { fixture, scenario });
+  if (scenario.hoverDevice) {
+    const point = await page.evaluate((id) => {
+      const marker = window.__goldenCard?.renderRoot?.querySelector(
+        `[data-hp="device"][data-id="${CSS.escape(id)}"]`,
+      );
+      if (!marker) return null;
+      const rect = marker.getBoundingClientRect();
+      return { x: rect.left + rect.width / 2, y: rect.top + rect.height / 2 };
+    }, scenario.hoverDevice);
+    if (!point) throw new Error(`golden hover device missing: ${scenario.hoverDevice}`);
+    await page.mouse.move(point.x, point.y);
+    if (scenario.hideHoverTooltip) {
+      await page.evaluate(async () => {
+        const card = window.__goldenCard;
+        card._tip = null;
+        card.requestUpdate();
+        await card.updateComplete;
+      });
+    }
+  }
+  return result;
 }

 export async function goldenClip(page, capture) {
@@ -1,7 +1,7 @@
 import { fixtureWallKey } from '../fixtures/visual-matrix.mjs';

 /** Data-only HP-QA-01 capture matrix. Bump when framing or scenarios change. */
-export const GOLDEN_MATRIX_VERSION = 26;
+export const GOLDEN_MATRIX_VERSION = 34;

 const stage = { capture: 'stage', threshold: { maxChannelDelta: 10, maxDiffRatio: 0.0005 } };
 const page = { capture: 'page', threshold: { maxChannelDelta: 10, maxDiffRatio: 0.0008 } };
@@ -44,6 +44,14 @@ export const GOLDEN_SCENARIOS = Object.freeze([
    theme: 'dark', viewport: { width: 1000, height: 900 }, ...stage },
  { id: 'geometry-view-light-fit', fixture: 'visual', space: 'golden-geometry', mode: 'view',
    theme: 'light', viewport: { width: 1000, height: 900 }, ...stage },
+  { id: 'room-label-parity-view-dark', fixture: 'visual', space: 'golden-lighting', mode: 'view',
+    roomLabelParity: true, theme: 'dark', viewport: { width: 1000, height: 900 }, ...page },
+  { id: 'room-label-parity-plan-dark', fixture: 'visual', space: 'golden-lighting', mode: 'plan',
+    roomLabelParity: true, theme: 'dark', viewport: { width: 1000, height: 900 }, ...page },
+  { id: 'room-label-parity-view-light', fixture: 'visual', space: 'golden-lighting', mode: 'view',
+    roomLabelParity: true, theme: 'light', viewport: { width: 1000, height: 900 }, ...page },
+  { id: 'room-label-parity-plan-light', fixture: 'visual', space: 'golden-lighting', mode: 'plan',
+    roomLabelParity: true, theme: 'light', viewport: { width: 1000, height: 900 }, ...page },
  { id: 'washer-active-cycle-dark', fixture: 'visual', applianceLifecycle: true,
    space: 'golden-appliance', mode: 'view',
    stateOverrides: { 'sensor.golden_washer_status': { state: 'start' } },
@@ -93,6 +101,12 @@ export const GOLDEN_SCENARIOS = Object.freeze([
  { id: 'wall-junctions-view-dark', fixture: 'visual',
    space: 'golden-wall-junctions', wallJunctions: true, mode: 'view',
    theme: 'dark', viewport: { width: 1000, height: 900 }, ...stage },
+  { id: 'junction-patch-resilience-plan-dark', fixture: 'visual',
+    space: 'golden-junction-patch-resilience', junctionPatchResilience: true, mode: 'plan',
+    theme: 'dark', viewport: { width: 1180, height: 900 }, ...page },
+  { id: 'junction-patch-resilience-view-dark', fixture: 'visual',
+    space: 'golden-junction-patch-resilience', junctionPatchResilience: true, mode: 'view',
+    theme: 'dark', viewport: { width: 1000, height: 900 }, ...stage },
  { id: 'isometric-wall-junctions-dark', fixture: 'visual',
    space: 'golden-wall-junctions', wallJunctions: true, mode: 'view',
    labs: ['iso'], projection: 'iso', ...expiredIsoFixture,
@@ -104,6 +118,14 @@ export const GOLDEN_SCENARIOS = Object.freeze([
    mode: 'plan', openingPreview: { type: 'door', pointer: [0.48, 0.65] },
    openingPreviewPixels: { minPixels: 150, minInsideWallPixels: 8, minChannelDelta: 4 },
    theme: 'dark', viewport: { width: 1180, height: 900 }, ...page },
+  { id: 'opening-placement-passage-thick-wall-dark', fixture: 'visual', space: 'golden-geometry',
+    mode: 'plan', openingPreview: { type: 'passage', pointer: [0.48, 0.65] },
+    openingPreviewPixels: { minPixels: 150, minInsideWallPixels: 8, minChannelDelta: 4 },
+    theme: 'dark', viewport: { width: 1180, height: 900 }, ...page },
+  { id: 'opening-placement-passage-thick-wall-light', fixture: 'visual', space: 'golden-geometry',
+    mode: 'plan', openingPreview: { type: 'passage', pointer: [0.48, 0.65] },
+    openingPreviewPixels: { minPixels: 150, minInsideWallPixels: 8, minChannelDelta: 4 },
+    theme: 'light', viewport: { width: 1180, height: 900 }, ...page },
  { id: 'geometry-devices-editor-dark', fixture: 'visual', space: 'golden-geometry', mode: 'devices',
    theme: 'dark', viewport: { width: 1180, height: 900 }, ...page },
  { id: 'geometry-decor-editor-dark', fixture: 'visual', space: 'golden-geometry', mode: 'decor',
@@ -175,6 +197,76 @@ export const GOLDEN_SCENARIOS = Object.freeze([
      'golden-climate': { s: 'golden-lighting', x: 0.80, y: 0.28 },
    },
    theme: 'dark', viewport: { width: 1000, height: 900 }, ...stage },
+  ...['light', 'dark'].map((theme) => ({
+    id: `device-icon-state-table-${theme}`, fixture: 'visual', space: 'golden-lighting', mode: 'view',
+    glowEnabled: false, sunRays: false,
+    markerOverrides: [
+      { id: 'golden-light-one', binding: 'device:golden-light-one' },
+      { id: 'golden-light-two', binding: 'device:golden-light-two' },
+      { id: 'golden-light-three', binding: 'device:golden-light-three' },
+      { id: 'golden-presence', binding: 'device:golden-presence' },
+      { id: 'golden-climate', binding: 'device:golden-climate' },
+      { id: 'golden-left-temperature', binding: 'device:golden-left-temperature', display: 'value' },
+      { id: 'golden-right-temperature', binding: 'device:golden-right-temperature', display: 'static_icon' },
+      { id: 'golden-left-linkquality', binding: 'device:golden-left-linkquality', value_badge: {
+        enabled: true,
+        source: { kind: 'entity_state', entity_id: 'sensor.golden_left_linkquality' },
+        position: 'right',
+      } },
+      { id: 'golden-right-linkquality', binding: 'device:golden-right-linkquality', value_badge: {
+        enabled: true,
+        source: { kind: 'entity_state', entity_id: 'sensor.golden_right_linkquality' },
+        position: 'bottom',
+      } },
+    ],
+    stateOverrides: {
+      'light.golden_light_one': { attributes: { lqi: 40 } },
+      'light.golden_light_two': { attributes: { lqi: 41 } },
+      'light.golden_light_three': { attributes: { lqi: 180 } },
+      'sensor.golden_left_temperature': { state: 'Complete long localized state' },
+      'sensor.golden_right_temperature': { state: 'unavailable' },
+    },
+    layoutOverrides: {
+      'golden-light-one': { s: 'golden-lighting', x: 0.16, y: 0.25 },
+      'golden-light-two': { s: 'golden-lighting', x: 0.34, y: 0.25 },
+      'golden-light-three': { s: 'golden-lighting', x: 0.66, y: 0.25 },
+      'golden-presence': { s: 'golden-lighting', x: 0.84, y: 0.25 },
+      'golden-climate': { s: 'golden-lighting', x: 0.16, y: 0.72 },
+      'golden-left-temperature': { s: 'golden-lighting', x: 0.34, y: 0.72 },
+      'golden-right-temperature': { s: 'golden-lighting', x: 0.66, y: 0.72 },
+      'golden-left-linkquality': { s: 'golden-lighting', x: 0.77, y: 0.55 },
+      'golden-right-linkquality': { s: 'golden-lighting', x: 0.84, y: 0.72 },
+    },
+    deviceClassOverrides: {
+      'golden-light-two': ['lock-locked'],
+      'golden-light-three': ['lock-unlocked'],
+      'golden-presence': ['virtual'],
+      'golden-climate': ['alarm', 'sel'],
+      'golden-right-linkquality': ['open'],
+    },
+    focusDevice: 'golden-climate',
+    hoverDevice: 'golden-presence',
+    hideHoverTooltip: true,
+    theme, viewport: { width: 1000, height: 900 }, ...stage,
+  })),
+  ...['light', 'dark'].map((theme) => ({
+    id: `device-text-shell-long-${theme}`, fixture: 'visual', space: 'golden-lighting', mode: 'view',
+    glowEnabled: false, sunRays: false, showNames: false,
+    markerOverrides: [{
+      id: 'golden-left-linkquality', binding: 'device:golden-left-linkquality',
+      display: 'value', size: 3,
+    }],
+    stateOverrides: {
+      'sensor.golden_left_linkquality': {
+        state: '498', attributes: { unit_of_measurement: 'ppm' },
+      },
+    },
+    layoutOverrides: {
+      'golden-left-linkquality': { s: 'golden-lighting', x: 0.50, y: 0.50 },
+    },
+    deviceOnly: 'golden-left-linkquality',
+    theme, viewport: { width: 1000, height: 900 }, ...stage,
+  })),
  { id: 'lighting-sun-window-state-only-dark', fixture: 'visual', space: 'golden-lighting', mode: 'view',
    // The golden screenshot is backed by a second, sun-layer-hidden capture.
    // A real painted ray must account for enough changed pixels; DOM-only
@@ -269,6 +361,18 @@ export const GOLDEN_SCENARIOS = Object.freeze([
  { id: 'decor-color-popover-mobile-ru', fixture: 'visual', space: 'golden-geometry',
    dialog: 'decor-color', language: 'ru', theme: 'dark',
    viewport: { width: 390, height: 760 }, ...page },
+  { id: 'decor-color-popover-desktop-en', fixture: 'visual', space: 'golden-geometry',
+    dialog: 'decor-color', language: 'en', theme: 'light',
+    viewport: { width: 760, height: 820 }, ...page },
+  { id: 'general-color-popover-desktop-en', fixture: 'visual', space: 'golden-geometry',
+    dialog: 'general-color', language: 'en', theme: 'light',
+    viewport: { width: 900, height: 900 }, ...page },
+  { id: 'device-ripple-color-popover-mobile-ru', fixture: 'visual', space: 'golden-lighting',
+    dialog: 'device-ripple-color', deviceId: 'golden-light-two',
+    language: 'ru', theme: 'dark', viewport: { width: 390, height: 1000 }, ...page },
+  { id: 'space-room-color-popover-desktop-ru', fixture: 'visual', space: 'golden-geometry',
+    dialog: 'space-room-color', language: 'ru', theme: 'dark',
+    viewport: { width: 900, height: 1000 }, ...page },
  { id: 'backup-full-preview-desktop-en', fixture: 'visual', space: 'golden-geometry',
    dialog: 'backup-full', language: 'en', theme: 'dark',
    viewport: { width: 1000, height: 900 }, ...page },
@@ -278,4 +382,10 @@ export const GOLDEN_SCENARIOS = Object.freeze([
  { id: 'backup-space-preview-mobile-ru', fixture: 'visual', space: 'golden-geometry',
    dialog: 'backup-space', language: 'ru', theme: 'light',
    viewport: { width: 390, height: 820 }, ...page },
+  { id: 'optimize-preflight-dialog-dark-en', fixture: 'visual', space: 'golden-geometry',
+    dialog: 'optimize-preflight', language: 'en', theme: 'dark',
+    viewport: { width: 900, height: 760 }, ...page },
+  { id: 'optimize-preflight-dialog-light-ru', fixture: 'visual', space: 'golden-geometry',
+    dialog: 'optimize-preflight', language: 'ru', theme: 'light',
+    viewport: { width: 900, height: 760 }, ...page },
 ]);
@@ -1,10 +1,14 @@
 // Shared launcher for demo captures: starts headless Chromium serving demo/srv/
 // via request interception (no HTTP server needed). Usage: const {page,browser}=await launch();
 import { chromium } from 'playwright';
+import { assertFreshDemoBundleUnlessAllowed } from './bundle-freshness.mjs';
 import { readFileSync, existsSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
 import { dirname } from 'node:path';
 const ROOT = dirname(fileURLToPath(import.meta.url)) + '/srv';
+// Корень репозитория, а не каталог раздачи: фингерпринт считается по
+// src/** и demo/fixtures, которых внутри demo/srv нет (#236).
+const REPO_ROOT = dirname(dirname(fileURLToPath(import.meta.url)));
 const CT = { '.html': 'text/html', '.js': 'text/javascript', '.svg': 'image/svg+xml' };
 // ---- assertion harness (audit T1) --------------------------------------
 // Until 2026-07-27 the smokes printed booleans and always exited 0: a broken
@@ -64,6 +68,14 @@ export async function launch(
  });
  await page.goto('http://demo.local/demo.html', { waitUntil: 'domcontentloaded' });
  await page.waitForFunction(() => window.__card?._model?.length > 0, { timeout: 9000 });
+  // Свежесть бандла проверяется здесь, а не в каждом смоке (#236). Смок читает
+  // demo/srv/assets/houseplan-card.js; если туда не скопирован свежий dist,
+  // проверяется прежняя версия карточки — и результат выглядит осмысленным,
+  // потому что часть проверок краснеет, а часть зеленеет. На #234 это стоило
+  // круга разбора: три проверки упали, четвёртая ложно прошла, поскольку старый
+  // код одинаково врал в двух местах, которые сверялись друг с другом.
+  // golden и бенчмарки эту защиту имели с самого начала, смоки — нет.
+  await assertFreshDemoBundleUnlessAllowed(page, REPO_ROOT);
  // HP-1552: the first-open boot veil hides the plan (visibility:hidden) until
  // the stage height settles — real pointer interaction cannot hit a hidden
  // plan, so every smoke starts where the user does: with the plan revealed.
@@ -63,7 +63,11 @@ const res = await page.evaluate(async () => {
  };
  c._openSettingsDialog();
  await c.updateComplete;
-  out.dialogHasBgRow = !!sr().querySelector('hp-dialog .colorrow input[type=color]');
+  const settingsDialog = sr().querySelector('hp-dialog');
+  const bgPicker = [...settingsDialog.querySelectorAll('hp-color-opacity')]
+    .find((picker) => picker.label === c._t('gs.bg_color'));
+  out.dialogHasBgRow = !!bgPicker && bgPicker.showOpacity === false
+    && !settingsDialog.querySelector('input[type=color]');
  c._settingsDialog = { ...c._settingsDialog, bgColor: '#0a2a4a' };
  out.dialogPreviews = (await stageBg()) === rgb('#0a2a4a');
  await c._saveSettingsDialog();
@@ -38,22 +38,27 @@ const res = await page.evaluate(async () => {
    clientX: r.left + r.width * fx, clientY: r.top + r.height * fy }));
  await c.updateComplete;
  out.normalClickWorks = c._path.length === 1;
-  // 5) delroom: клик по карточке не зовёт confirm (переопределим)
-  let confirmCalled = false;
-  window.confirm = () => { confirmCalled = true; return false; };
-  c._tool = 'delroom'; c._path = []; await c.updateComplete;
+  // 5) delroom: клик по карточке устройства не заводит удаление комнаты.
+  // Раньше здесь подменялся window.confirm; #228 заменил его собственным
+  // диалогом карточки, и проверка стала невыполнимой в обе стороны — confirm
+  // теперь не звонит никто. Признак — открытый диалог.
+  c._tool = 'delroom'; c._path = []; c._roomDeleteDialog = null; await c.updateComplete;
  label.dispatchEvent(new MouseEvent('click', { bubbles: true, composed: true, clientX: 10, clientY: 10 }));
  await c.updateComplete;
-  out.delroomIgnored = !confirmCalled;
+  out.delroomIgnored = c._roomDeleteDialog === null;
  // 6) Delete has exactly one semantic result: the clicked room disappears;
  // it never closes a virtual boundary or merges a neighbour into it.
  const roomCount = c._curSpaceCfg.rooms.length;
  const deleteId = room.id;
-  window.confirm = () => true;
  c._deleteRoomClick(c._roomCenter(room));
  await c.updateComplete;
+  // Клик по комнате только спрашивает; удаляет ответ на вопрос (#228).
+  out.deleteAsksFirst = c._roomDeleteDialog?.roomId === deleteId;
+  c._confirmRoomDelete(false);
+  await c.updateComplete;
  out.deleteOnlyClickedRoom = c._curSpaceCfg.rooms.length === roomCount - 1
    && !c._curSpaceCfg.rooms.some((x) => x.id === deleteId);
+  out.deleteClosesDialog = c._roomDeleteDialog === null;
  c._undoGeometry();
  await c.updateComplete;
  out.deleteUndoRestoresRoom = c._curSpaceCfg.rooms.length === roomCount
--- a/Show More
+++ b/Show More