reviewer(ET): auto-commit from reviewer run_id=222
This commit is contained in:
@@ -2,7 +2,7 @@
|
||||
type: review
|
||||
work_item_id: ET-013
|
||||
verdict: APPROVED
|
||||
version: 7
|
||||
version: 8
|
||||
created_at: 2026-06-04
|
||||
updated_at: 2026-06-06
|
||||
authors:
|
||||
@@ -13,26 +13,27 @@ related:
|
||||
- "ET-013:test-report"
|
||||
---
|
||||
|
||||
# Review ET-013 — Перепады высот на z9-z11 (re-run #7)
|
||||
# Review ET-013 — Перепады высот на z9-z11 (re-run #8)
|
||||
|
||||
## TL;DR
|
||||
|
||||
- **Branch:** `feature/ET-013-z9-z11-z8`
|
||||
- **HEAD:** `23cd31d tester(ET): auto-commit from tester run_id=217`
|
||||
- **HEAD:** `498c7e1 tester(ET): auto-commit from tester run_id=221`
|
||||
- **Scope:** zoom-aware paint (hillshade/TRI) на z9-z11, понижение
|
||||
UI-минзума hillshade z10→z9, обратно-совместимое расширение
|
||||
`applyTerrainLayer`, расширение backend-whitelist на `tri` (фикс F-1 из v1).
|
||||
- **Что изменилось со времени review v6:** **в коде и тестах — ничего.**
|
||||
`git diff 316bb0d..HEAD --stat` даёт ровно 2 файла — оба docs
|
||||
(`12-review.md`, `13-test-report.md`). Feature-diff (`8da09e6..HEAD`
|
||||
по `src/`+`tests/`) идентичен одобренному в v2..v6: 5 файлов.
|
||||
- **Перепроверено в re-run #7 независимо:** перечитаны TRZ, AC, ADR-017,
|
||||
полный diff фичи (`git diff 8da09e6..HEAD`), исходники и оба тест-файла.
|
||||
Unit-тесты перезапущены локально — **17/17 PASS** (0.03 s). `grep
|
||||
applyTerrainLayer` → 1 декларация + ровно 2 call-site.
|
||||
- **Что изменилось со времени review v7:** **в коде и тестах — ничего.**
|
||||
Содержательные коммиты фичи — `5be81f9` (feat) и `099669d` (fix whitelist);
|
||||
всё после них — docs-коммиты ревью/тестера. `src/` и `tests/` бит-в-бит
|
||||
совпадают с одобренным в v2..v7.
|
||||
- **Перепроверено в re-run #8 независимо:** перечитаны TRZ (REQ-F-01..F-21),
|
||||
AC (AC-01..AC-22), ADR-017, feature-diff по `src/web/app.js`,
|
||||
`src/web/index.html`, `src/api/main.py` и обоим тест-файлам. Unit-тесты
|
||||
перезапущены локально — **17/17 PASS** (0.03 s). `grep applyTerrainLayer`
|
||||
→ 1 декларация + ровно 2 call-site (`onTerrainCheckbox`).
|
||||
- **Verdict кода: APPROVED.** P0/P1 в коде/ADR/тестах не найдено. Код
|
||||
соответствует TRZ REQ-F-01..F-19, F-21 и всем семи решениям ADR-017.
|
||||
- **⚠️ Деплой не разблокирован ревью.** Тестер (run_id=217, test-report v4)
|
||||
- **⚠️ Деплой не разблокирован ревью.** Тестер (run_id=221, test-report v5)
|
||||
держит `verdict: BLOCKED` из-за **AC-19 (P1): на test-среде
|
||||
`/terrain/hillshade/9/{x}/{y}.png` → 404** (тайлы z9 не нарезаны).
|
||||
Это предусловие данных PH-6, **не дефект PR** (см. §AC-19). Шиппинг
|
||||
@@ -43,7 +44,7 @@ related:
|
||||
- `docs/work-items/ET-013/02-trz.md`
|
||||
- `docs/work-items/ET-013/03-acceptance-criteria.md`
|
||||
- `docs/work-items/ET-013/06-adr/ADR-017-zoom-aware-terrain-paint.md`
|
||||
- `docs/work-items/ET-013/12-review.md` (v6), `13-test-report.md` (v4, BLOCKED)
|
||||
- `docs/work-items/ET-013/12-review.md` (v7), `13-test-report.md` (v5, BLOCKED)
|
||||
- `CLAUDE.md`
|
||||
- `git diff 8da09e6..HEAD` (полный diff фичи), `--stat` по `src/`+`tests/`
|
||||
- `src/web/app.js` (2725-2826, 3356-3434)
|
||||
@@ -101,7 +102,7 @@ T-A (единый paint на все темы), M-A (константы в `app.j
|
||||
|
||||
## AC-19 / деплой-блокер (анализ от ревью)
|
||||
|
||||
Тестер (run_id=217, v4) воспроизвёл: на выкаченной test-среде
|
||||
Тестер (run_id=221, v5) воспроизвёл: на выкаченной test-среде
|
||||
`GET …/terrain/hillshade/9/{x}/{y}.png` → **404** (grid 5×5 вокруг
|
||||
`(309,348)` + 6 точек ЦФО → 0/25, 0/6 → 200; hillshade-стек начинается
|
||||
с z10). Тайлы `hillshade/z10`, `z11`, `tri/z5..z12` присутствуют.
|
||||
@@ -134,13 +135,13 @@ PH-6 follow-up «нарезать hillshade z9-z11 для региона test-с
|
||||
монотонности stops, back-compat и whitelist-регрессии. Integration —
|
||||
корректный `skipif` при отсутствии PNG-данных (AC-16).
|
||||
|
||||
## Изменения после review v6
|
||||
## Изменения после review v7
|
||||
|
||||
Только документация (`git diff 316bb0d..HEAD --name-only` — два docs-файла,
|
||||
по `src/`/`tests/` пусто): `12-review.md` (этот файл, v7), `13-test-report.md`
|
||||
(v4 Тестера, run 217). Код и тесты с момента APPROVED-вердикта v2..v6
|
||||
**не менялись**. Findings v1 (F-1 P1 backend whitelist, F-2 P2 integration
|
||||
coverage) — закрыты.
|
||||
Только документация (по `src/`/`tests/` пусто): `12-review.md` (этот файл,
|
||||
v8), `13-test-report.md` (v5 Тестера, run 221). Код и тесты с момента
|
||||
APPROVED-вердикта v2..v7 **не менялись** (содержательные коммиты —
|
||||
`5be81f9` feat, `099669d` fix-whitelist). Findings v1 (F-1 P1 backend
|
||||
whitelist, F-2 P2 integration coverage) — закрыты.
|
||||
|
||||
## Findings
|
||||
|
||||
|
||||
Reference in New Issue
Block a user