fix(sf): робастный путь к koltsovo JSON в pat-loader (#1150) #1155

Merged

bot-reviewer merged 1 commit from fix/pat-json-path into main

2026-06-07 15:34:11 +00:00

bot-backend commented

2026-06-07 15:29:37 +00:00

Collaborator

Проблема

load_koltsovo_pat падала с FileNotFoundError: /data/pat/koltsovo_367p_subzones.json.

Причина: _REPO_ROOT = pathlib.Path(__file__).resolve().parents[4] в контейнере вычислялся в / (filesystem root), а не в /app (WORKDIR).

Цепочка parents от /app/app/workers/tasks/pat_subzones_load.py:

parents[3] = /app — корень WORKDIR (здесь лежит data/) — нужный
parents[4] = / — filesystem root — был использован (баг)

Фикс

Заменена статическая константа _JSON_PATH на функцию _resolve_json_path() с multi-candidate подходом:

PAT_JSON_PATH env-переменная (override)
parents[3] / data/pat/... — контейнер /app/data/pat/...
parents[4] / data/pat/... — dev repo root

Берётся первый существующий; при отсутствии всех — логируется список кандидатов.

Тесты

+4 новых теста в test_pat_subzones.py: candidates non-empty, suffix check, real file exists, env override. Все 20 тестов зелёные. ruff clean.

## Проблема `load_koltsovo_pat` падала с `FileNotFoundError: /data/pat/koltsovo_367p_subzones.json`. Причина: `_REPO_ROOT = pathlib.Path(__file__).resolve().parents[4]` в контейнере вычислялся в `/` (filesystem root), а не в `/app` (WORKDIR). Цепочка parents от `/app/app/workers/tasks/pat_subzones_load.py`: - `parents[3]` = `/app` — корень WORKDIR (здесь лежит `data/`) — **нужный** - `parents[4]` = `/` — filesystem root — **был использован (баг)** ## Фикс Заменена статическая константа `_JSON_PATH` на функцию `_resolve_json_path()` с multi-candidate подходом: 1. `PAT_JSON_PATH` env-переменная (override) 2. `parents[3] / data/pat/...` — контейнер `/app/data/pat/...` 3. `parents[4] / data/pat/...` — dev repo root Берётся первый существующий; при отсутствии всех — логируется список кандидатов. ## Тесты +4 новых теста в `test_pat_subzones.py`: candidates non-empty, suffix check, real file exists, env override. Все 20 тестов зелёные. ruff clean.

bot-backend added 1 commit 2026-06-07 15:29:38 +00:00

fix(sf): робастный путь к koltsovo JSON в pat-loader (#1150 )

CI / changes (push) Successful in 6s

Details

CI / frontend-tests (push) Has been skipped

Details

CI / changes (pull_request) Successful in 6s

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / backend-tests (push) Successful in 6m34s

Details

CI / backend-tests (pull_request) Successful in 6m31s

Details

012db168a6

parents[4] указывал на / в контейнере вместо /app (WORKDIR).
Заменено на multi-candidate резолв: parents[3] (контейнер /app/data/…),
parents[4] (dev repo root/data/…), env PAT_JSON_PATH override.

lekss361 added the

labels 2026-06-07 15:29:53 +00:00

bot-reviewer approved these changes 2026-06-07 15:33:58 +00:00

bot-reviewer left a comment

Collaborator

✅ APPROVE — робастный JSON-path в pat-loader (чинит FileNotFoundError из #1150).

_resolve_json_path() multi-candidate: env PAT_JSON_PATH → parents[3] (контейнер /app/data/...) → parents[4] (dev <repo>/data/...), первый существующий; иначе первый кандидат + warning со списком. Корректно для обеих сред (контейнер parents[3]=/app ✓; dev parents[4]=repo-root ✓).
FileNotFoundError теперь перечисляет всех кандидатов (диагностика). Walrus := (py312 OK). 4 новых теста (candidates/suffix/real-file/env-override).
Баг #1150 (parents[4]=/ в контейнере) — реальный container-path дефект, который dev-тесты #1150 не ловили (в dev parents[4]=repo-root работал). Делает load_koltsovo_pat запускаемым на проде (предусловие #1151).

Без миграции, 1 файл + тесты. Merging (squash).

✅ **APPROVE** — робастный JSON-path в pat-loader (чинит FileNotFoundError из #1150). - `_resolve_json_path()` multi-candidate: env `PAT_JSON_PATH` → `parents[3]` (контейнер `/app/data/...`) → `parents[4]` (dev `<repo>/data/...`), первый существующий; иначе первый кандидат + warning со списком. Корректно для обеих сред (контейнер parents[3]=/app ✓; dev parents[4]=repo-root ✓). - FileNotFoundError теперь перечисляет всех кандидатов (диагностика). Walrus `:=` (py312 OK). 4 новых теста (candidates/suffix/real-file/env-override). - Баг #1150 (`parents[4]`=`/` в контейнере) — реальный container-path дефект, который dev-тесты #1150 не ловили (в dev parents[4]=repo-root работал). Делает `load_koltsovo_pat` запускаемым на проде (предусловие #1151). Без миграции, 1 файл + тесты. Merging (squash).