diff --git a/docs/work-items/ET-013/12-review.md b/docs/work-items/ET-013/12-review.md index d1cbfea..90d57dd 100644 --- a/docs/work-items/ET-013/12-review.md +++ b/docs/work-items/ET-013/12-review.md @@ -2,7 +2,7 @@ type: review work_item_id: ET-013 verdict: APPROVED -version: 4 +version: 5 created_at: 2026-06-04 updated_at: 2026-06-06 authors: @@ -13,20 +13,24 @@ related: - "ET-013:test-report" --- -# Review ET-013 — Перепады высот на z9-z11 (re-run #4) +# Review ET-013 — Перепады высот на z9-z11 (re-run #5) ## TL;DR - **Branch:** `feature/ET-013-z9-z11-z8` -- **HEAD:** `c66d4fb tester(ET): auto-commit from tester run_id=209` +- **HEAD:** `0ad44b5 reviewer(ET): auto-commit from reviewer run_id=211` - **Scope:** калибровка клиентского paint (hillshade/TRI) на z9-z11, понижение UI-минзума hillshade z10→z9, обратно-совместимое расширение `applyTerrainLayer`, расширение whitelist backend-endpoint'а на `tri` (фикс по review v1, F-1). -- **Что изменилось со времени review v3:** **в коде и тестах — ничего.** - `git diff 316bb0d..HEAD --name-only` → только `12-review.md` (этот файл) - и `13-test-report.md` (новый прогон Тестера, run_id=209). Реализация - (`src/`, `tests/`) бит-в-бит совпадает с одобренной в v2/v3. +- **Что изменилось со времени review v4:** **в коде и тестах — ничего.** + `git diff c66d4fb..HEAD --name-only` → только `12-review.md` (этот файл). + Реализация (`src/`, `tests/`) бит-в-бит совпадает с одобренной в v2..v4. +- **Перепроверено в re-run #5:** перечитаны TRZ, AC, ADR-017, полный diff + фичи (`git diff 8da09e6..HEAD`), все исходники и тесты. Unit-тесты + перезапущены локально — **17/17 PASS** (0.03 s). Integration-модуль в этом + sandbox не собирается из-за отсутствия `shapely` (env-зависимость, не + дефект кода); Тестер в полном venv фиксировал 254 passed / 7 skipped. - **Тесты:** `pytest tests/unit/test_terrain_paint.py` — **17/17 PASS** (перезапущено локально, 0.03 s). Тестер в полном venv: **254 passed, 7 skipped, 0 failed**; ET-013 unit 17/17, integration 6 PASS / 7 SKIP @@ -138,13 +142,12 @@ AC-03/07/13. До этого задачу к деплою не выпускат - Backend-фикс: whitelist — фиксированный кортеж, `z/x/y` типизированы как `int`, path-traversal-рисков нет. -## Изменения после review v3 +## Изменения после review v4 -Только документация (`git diff 316bb0d..HEAD --name-only`): -- `12-review.md` — этот файл (v4). -- `13-test-report.md` — новый прогон Тестера (run_id=209). +Только документация (`git diff c66d4fb..HEAD --name-only`): +- `12-review.md` — этот файл (v5). -Код и тесты (`src/`, `tests/`) с момента APPROVED-вердикта v2/v3 **не +Код и тесты (`src/`, `tests/`) с момента APPROVED-вердикта v2..v4 **не менялись**. Findings v1 (F-1 P1 backend whitelist, F-2 P2 integration coverage) остаются закрытыми.