reviewer(ET): auto-commit from reviewer run_id=234

docs/work-items/ET-013/12-review.md

@@ -2,7 +2,7 @@
 type: review
 work_item_id: ET-013
 verdict: APPROVED
-version: 13
+version: 14
 created_at: 2026-06-04
 updated_at: 2026-06-06
 authors:
@@ -14,7 +14,36 @@ related:
   - "ET-013:test-report"
 ---
 
-# Review ET-013 — Перепады высот на z9-z11 (re-run #13)
+# Review ET-013 — Перепады высот на z9-z11 (re-run #14)
+
+> **Re-run #14 (independent).** Перечитаны TRZ, AC, ADR-017, CLAUDE.md.
+> Построчно сверена реализация в рабочем дереве со всеми REQ-F-01..F-19
+> и решениями ADR-017 (P-A, O-B, C-A, R-A, U-A, A-A, M-A):
+> - `HILLSHADE_PAINT` (app.js:2734-2752) и `TRI_PAINT` (app.js:2755-2768)
+>   — stops побайтово совпадают с REQ-F-05/F-08 и решениями 4/5 ADR-017
+>   (hillshade opacity 9:0.65→14:0.40, contrast 9:0.40→14:0.00, nearest;
+>   TRI opacity z8=0.70, пик z9-z11=0.85, спад до z15=0.70, nearest).
+> - `applyTerrainLayer` (app.js:3371-3414) — нормализация number→legacy
+>   (`raster-opacity`+`linear`) / object as-is; `paint: paint` в `addLayer`.
+> - `updateHillshadeAvailability` (app.js:3425) — `zoom < 9`, `zoom < 10`
+>   отсутствует; вызовы (app.js:2825-2826) — hillshade minzoom=9/HILLSHADE_PAINT,
+>   TRI minzoom=5/maxzoom=15/TRI_PAINT.
+> - `index.html:60` — hint «Зум 9+».
+> - `terrain_tile` (main.py:1240-1266) — whitelist `("hypso","hillshade","tri")`,
+>   `Cache-Control: ...immutable` сохранён → REQ-F-18 соблюдён.
+> - Unit-тесты перезапущены локально — **17/17 PASS (0.03 s)**.
+> - Integration (`test_terrain_z9_tiles.py`) присутствует, корректно
+>   спроектирован: whitelist/404-регрессии работают без данных,
+>   тайло-зависимые кейсы `skipif` при отсутствии PH-6 тайлов.
+>
+> **Наблюдение по ветке (не finding):** фича ET-013 уже смержена в `main`
+> (PR #26, `be7a052`) и задеплоена (`v0.0.5`); `main` ушёл вперёд на
+> ET-014/ET-015. `git diff main..HEAD` показывает «удаления» только потому,
+> что ветка отстала от `main` — содержательный код ET-013 в рабочем дереве
+> корректен и идентичен merged-состоянию. Это вопрос мержа/процесса, а не
+> качества кода ET-013.
+>
+> Вердикт подтверждён повторно: **APPROVED**.
 
 > **Re-run #13 (independent).** Перечитаны TRZ (`02-trz.md`),
 > AC (`03-acceptance-criteria.md`), ADR-017, CLAUDE.md. Построчно сверена