From 136e0cb77334b2c5ac9a7ee64295332373083796 Mon Sep 17 00:00:00 2001
From: Sergey Matyunin <s.matyunin@justbusiness.site>
Date: Mon, 24 Aug 2026 23:48:36 +0300
Subject: [PATCH] test: align Optimize smoke with role-aware compaction

The full pre-release smoke still expected #198's T-node fixture to compact into one record. #299 intentionally preserves the outer/shared ownership breakpoints, so the smoke now requires three canonical 22 cm role runs while retaining Preview, atomic Apply, Reload, and Undo coverage.

Issue: #299
User-Visible: no
---
 demo/smoke_optimize_micro_interval.mjs | 33 ++++++++++++++++----------
 1 file changed, 21 insertions(+), 12 deletions(-)

diff --git a/demo/smoke_optimize_micro_interval.mjs b/demo/smoke_optimize_micro_interval.mjs
index 02699e3d..f3b76e6a 100644
--- a/demo/smoke_optimize_micro_interval.mjs
+++ b/demo/smoke_optimize_micro_interval.mjs
@@ -1,5 +1,6 @@
-// #198: explicit Optimize removes only a proven isolated wall-thickness
-// micro-interval. Exercise the production bundle's Preview → Apply → Undo flow.
+// #198/#299: explicit Optimize removes only a proven isolated wall-thickness
+// micro-interval without compacting across a physical wall-owner role boundary.
+// Exercise the production bundle's Preview → Apply → Reload → Undo flow.
 import { launch, checkAll, finish } from './serve.mjs';
 
 const { page, browser } = await launch({ width: 900, height: 820 });
@@ -40,6 +41,19 @@ const out = await page.evaluate(async () => {
     }],
     markers: [], settings: {},
   };
+  const expectedRuns = [
+    [0.8, 203 / 240],
+    [203 / 240, 213 / 240],
+    [213 / 240, 0.95],
+  ];
+  const hasRoleAwareCanonicalRuns = (walls) => {
+    if (walls.length !== expectedRuns.length || walls.some((wall) => wall.cm !== 22)) return false;
+    const runs = walls.map((wall) => {
+      if (!wall.a || !wall.b || wall.a[1] !== canonicalY || wall.b[1] !== canonicalY) return null;
+      return [Math.min(wall.a[0], wall.b[0]), Math.max(wall.a[0], wall.b[0])];
+    }).filter(Boolean).sort((left, right) => left[0] - right[0]);
+    return JSON.stringify(runs) === JSON.stringify(expectedRuns);
+  };
   const clone = (value) => JSON.parse(JSON.stringify(value));
   let serverConfig = clone(original), serverLayout = {}, backup = null;
   const sent = [];
@@ -71,10 +85,9 @@ const out = await page.evaluate(async () => {
 
   card._openAlignDialog(); await card.updateComplete;
   const preview = card._alignDialog;
-  result.previewOffersChange = !!preview?.changed
-    && preview.report.canonicalized === 1 && preview.report.wallsMerged === 2
-    && preview.config.spaces[0].walls.length === 1
-    && preview.config.spaces[0].walls[0].cm === 22;
+  result.previewOffersRoleAwareChange = !!preview?.changed
+    && preview.report.canonicalized === 1 && preview.report.wallsMerged === 0
+    && hasRoleAwareCanonicalRuns(preview.config.spaces[0].walls);
   result.previewDoesNotWrite = sent.length === 0
     && JSON.stringify(card._serverCfg) === JSON.stringify(original);
   result.previewIsVisible = !!card.renderRoot.querySelector('hp-dialog .alignmsg');
@@ -87,16 +100,12 @@ const out = await page.evaluate(async () => {
   await card._runAlignToGrid(); await card.updateComplete;
   const appliedWalls = card._serverCfg.spaces[0].walls;
   result.applyUsesOneAtomicWrite = sent.filter((type) => type === 'houseplan/plan/optimize').length === 1;
-  result.applyStoresOneUniformRun = appliedWalls.length === 1 && appliedWalls[0].cm === 22
-    && JSON.stringify(appliedWalls[0].a) === JSON.stringify([0.8, canonicalY])
-    && JSON.stringify(appliedWalls[0].b) === JSON.stringify([0.95, canonicalY]);
+  result.applyStoresRoleAwareRuns = hasRoleAwareCanonicalRuns(appliedWalls);
   result.applyEnablesUndo = card._canOptimizeUndo === true;
 
   await card._loadFromServer(); await card.updateComplete;
   const reloadedWalls = card._serverCfg.spaces[0].walls;
-  result.reloadKeepsCanonicalRun = reloadedWalls.length === 1 && reloadedWalls[0].cm === 22
-    && JSON.stringify(reloadedWalls[0].a) === JSON.stringify([0.8, canonicalY])
-    && JSON.stringify(reloadedWalls[0].b) === JSON.stringify([0.95, canonicalY]);
+  result.reloadKeepsRoleAwareRuns = hasRoleAwareCanonicalRuns(reloadedWalls);
 
   await card._undoPlanOptimization(); await card.updateComplete;
   result.undoUsesServerSnapshot = sent.filter((type) => type === 'houseplan/plan/optimize_undo').length === 1;