diff --git a/docs/reviews/CODE-REVIEW-789-r1.md b/docs/reviews/CODE-REVIEW-789-r1.md index e558bbd2..dc8953ac 100644 --- a/docs/reviews/CODE-REVIEW-789-r1.md +++ b/docs/reviews/CODE-REVIEW-789-r1.md @@ -98,6 +98,12 @@ AC3 выполнен: warm p95 1242≤1350, старый camera p95 109≤135, * Снято с обоснованием: этот внешний fixed-camera toggle не воспроизводит поддержанный actual-scale пользовательский переход; строгий AC2 относится к заданной all-pixel LED composition матрице, которая исполнена с нулём отличий, а частный план требует локальной визуальной проверки. Полная детализация LED, цвет и форма восстановлены; видимая деградация соседнего проёма на просмотренных кадрах не установлена. Это не разрешение AA-tolerance для synthetic oracle и не доказательство byte-identical всего viewport на частном плане. Координатор согласовал именно такую узкую фиксацию ограничения. Приватные экспорт/изображения не публикуются. Новых бессодержательных прогонов или product fix для неустановленного supported-flow дефекта не требуется. +## Пост-гейт: Windows portability failure перед публикацией + +После review-only коммита `4de49e0db265e3fae8f6ffa0b80b8d5c770fb4d1` pre-push выбрал полный исполняемый diff ветки и остановил push: Windows `gate:small` **RED**. По точному диагностическому повтору координатора с Node 22 и тем же Git Bash PATH: 3641 tests, 3581 pass, 59 skip, **1 fail** — `#732 AC2: тип фикстуры…`, `test/iso-overlay-fixture-types.test.mjs:106`, `Cannot read properties of undefined (reading 'text')`; он совпадает с единственным отказом исходного hook. Это не новый canonical performance run и не успешный Windows gate. + +Независимое чтение и минимальный read-only TS-host probe подтвердили причину: TypeScript передаёт имя virtual source с `/`, тогда как Windows `join()` создаёт `PROBE` с `\`; строгие сравнения на строках 73/77 не создают SourceFile, после чего строка 110 читает `probe.text`. Та же несогласованность есть в сравнении dirname/TEST_DIR на строке 83. In-memory probe на Windows/TypeScript 5.9.3 даёт sourceFound=false при исходном сравнении и true при нормализации; он исполнен системным Node 24 только для механизма, не как pinned acceptance. Тест, обе его fixtures, tsconfig.json и package/lock между b07 и 80cba неизменны: классификация — прежний Windows-дефект тестовой обвязки #732/#754, не LED-регрессия. Свежие Linux Full Validate/Full Performance и отдельный WSL gate на 80cba остаются доказательствами кандидата; Windows RED не скрывается. Координатор выбрал документированный в DEVELOPMENT/TESTING явный `HP_PREPUSH_GATE=0` после отдельного WSL gate, с сохранением process-gate; это не `--no-verify`. Исходники/skip/assertions/бюджеты ради этого отказа не меняются, дефект portability требует отдельного исправления вне #789. + ## Чего не проверял и ограничения - Нет нового реального wall-tablet/телефона/WebKit/Firefox/GPU acceptance: browser evidence — закреплённый Chromium; touch задаётся настоящими browser input events. Общее правило touch editors best effort не расширяется.