german/houseplan-card
1
0
Fork 0
mirror of https://github.com/Matysh/houseplan-card synced 2026-09-29 03:09:36 +00:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

fix: keep a stored zero draft thickness and guard both thickness writers (#313)

Browse Source 
По находкам CODE-REVIEW-313-r1:

High — резолвер больше не превращает сохранённый 0 сегмента драфта в 15:
ноль — легитимное значение (docs/WALL-THICKNESS.md §6), к дефолту 15 падает
только ОТСУТСТВУЮЩАЯ запись. Смок дополнен: hit нулевого сегмента несёт 0,
диалог показывает пустое поле, Apply без правки отказывает и не портит
данные.

Medium — гвард #278 усилен: паттерн допускает перенос строки после скобки,
и счётчик требует РОВНО ДВЕ точки коммита wall_thickness — мутация любой из
них (включая новую независимую) красит юнит; краснота второго патча мутанта
проверена изолированным исполнением.

Issue: #313
User-Visible: no
This commit is contained in:
Codex
2026-08-26 08:35:58 +03:00
parent d34e56ba78
commit ed126d6a43
6 changed files with 42 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/wall-union-isolation.test.mjs
+10 -1
View File
@@ -104,8 +104,17 @@ test('#278 production source routes physical writers through one barrier and dec
'physical_move', 'resize_room', 'close_boundary', 'open_boundary', 'wall_thickness',
'move_opening', 'delete_opening', 'merge_rooms', 'contour_to_partitions',
]) {
assert.match(source, new RegExp(`_commitPhysicalGeometry\\(this\\._t\\('history\\.${historyKey}'`), historyKey);
// The pattern tolerates a line break after the opening parenthesis: a
// wrapped call must not slip past the barrier check (CODE-REVIEW-313-r1).
assert.match(source,
new RegExp(`_commitPhysicalGeometry\\(\\s*this\\._t\\('history\\.${historyKey}'`), historyKey);
}
// #313 introduced a second thickness commit point (independent masonry).
// BOTH must go through the barrier: replacing either with _recordGeometry
// reduces the count and reddens this line.
assert.equal(
(source.match(/_commitPhysicalGeometry\(\s*this\._t\('history\.wall_thickness'/g) || []).length,
2, 'both thickness writers route through the common barrier');
assert.match(source, /_commitPhysicalGeometry\([\s\S]{0,160}history\.edit_opening/);
assert.match(source, /_commitPhysicalGeometry\([\s\S]{0,160}history\.split_room/);
assert.match(source, /_recordGeometry\(this\._t\('history\.decor_edit'/);