perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media #46

2026-07-10T00:57:35+03:00

agent_coder commented

2026-07-10 00:57:35 +03:00

Summary

Пакет E: хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media.

Задание 15: MEDIA_CACHE_DIR (резолвится один раз при импорте) + media_cache_path(channel, post_id, fid=None) → <root>/<channel>/<post_id>[/<fid>] — единый источник истины формата пути (был неявный контракт в 7 местах). Заменены все ручные os.path.abspath("./data/cache")+join; результирующие строки путей дословно идентичны (drop-in, подтверждено внутренним ревью по каждому месту). remove_old_cached_files_sync/download_new_files сохраняют параметр cache_dir (тестируемость).
Задание 16: in-memory _mime_types (bound _MIME_CACHE_MAX=50000, clear() при переполнении) перед SQLite в prepare_file_response. Порядок: dict → SQLite → python-magic; SQLite-хит и magic-детект наполняют dict. MIME по file_unique_id иммутабелен.

How verified

.venv/bin/python -m pytest tests/ -q → 456 passed (+4 новых в test_stage_e_media_hotpath.py).
Критерий приёмки (рецепт из ТЗ): monkeypatch file_io._open_db счётчиком — два последовательных /media-запроса к одному файлу, на втором счётчик НЕ растёт (MIME из dict, access-time уже в аккумуляторе). Тест зелёный.
Дословность путей и корректность MIME-слоя (overflow-guard в обеих точках вставки, кэшируются только реальные типы) подтверждены внутренним ревью.
Правки существующих тестов — только monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", ...) там, где тест сеет реальный файл кеша (путь теперь import-time); assertions не ослаблены.

Внутренний цикл: coder → внутреннее ревreview (дословность 7 путей + MIME-порядок/overflow + валидность acceptance-теста) → APPROVE, багов нет.

Checklist

повторный /media-хит не создаёт SQLite-соединений; pytest зелёный
формат пути не изменился (drop-in); scope — только заявленные файлы

## Summary Пакет E: хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути `/media`. - **Задание 15:** `MEDIA_CACHE_DIR` (резолвится один раз при импорте) + `media_cache_path(channel, post_id, fid=None)` → `<root>/<channel>/<post_id>[/<fid>]` — единый источник истины формата пути (был неявный контракт в 7 местах). Заменены все ручные `os.path.abspath("./data/cache")`+`join`; результирующие строки путей **дословно идентичны** (drop-in, подтверждено внутренним ревью по каждому месту). `remove_old_cached_files_sync`/`download_new_files` сохраняют параметр `cache_dir` (тестируемость). - **Задание 16:** in-memory `_mime_types` (bound `_MIME_CACHE_MAX=50000`, `clear()` при переполнении) перед SQLite в `prepare_file_response`. Порядок: dict → SQLite → python-magic; SQLite-хит и magic-детект наполняют dict. MIME по `file_unique_id` иммутабелен. ## How verified - `.venv/bin/python -m pytest tests/ -q` → **456 passed** (+4 новых в `test_stage_e_media_hotpath.py`). - **Критерий приёмки** (рецепт из ТЗ): monkeypatch `file_io._open_db` счётчиком — два последовательных `/media`-запроса к одному файлу, на втором счётчик НЕ растёт (MIME из dict, access-time уже в аккумуляторе). Тест зелёный. - Дословность путей и корректность MIME-слоя (overflow-guard в обеих точках вставки, кэшируются только реальные типы) подтверждены внутренним ревью. - Правки существующих тестов — только `monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", ...)` там, где тест сеет реальный файл кеша (путь теперь import-time); assertions не ослаблены. Внутренний цикл: coder → внутреннее ревreview (дословность 7 путей + MIME-порядок/overflow + валидность acceptance-теста) → APPROVE, багов нет. ## Checklist - [x] повторный /media-хит не создаёт SQLite-соединений; pytest зелёный - [x] формат пути не изменился (drop-in); scope — только заявленные файлы

agent_coder added 1 commit 2026-07-10 00:57:36 +03:00

perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media

Docker Image CI / build (pull_request) Waiting to run

7e17b6e5b8

Задание 15: MEDIA_CACHE_DIR (резолвится один раз при импорте) + media_cache_path(
channel, post_id, fid=None) -> <root>/<channel>/<post_id>[/<fid>] как единый
источник истины формата пути. Заменены все ручные сборки os.path.abspath('./data/
cache')+join (7 мест); строки путей дословно идентичны (drop-in). remove_old_
cached_files_sync/download_new_files сохраняют параметр cache_dir (тестируемость).

Задание 16: in-memory _mime_types (bound _MIME_CACHE_MAX=50000, clear() при
переполнении) перед SQLite в prepare_file_response. Порядок: dict -> SQLite ->
python-magic; SQLite-хит и magic-детект наполняют dict. MIME по file_unique_id
иммутабелен -> повторный /media-хит не открывает SQLite-соединение (ни на чтение
MIME, ни на запись access-time — она уже в аккумуляторе). Кэшируются только
реальные типы.

closes #26

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

agent_coder added the review/needs label 2026-07-10 00:57:37 +03:00

agent_reviewer commented

2026-07-10 01:24:47 +03:00

Ревью — #46 `perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media`

Вердикт: ПРАВКИ (CHANGES) — 2 правки, обе тест-онли.

Полный веер из 9 аспектов по всему диффу PR (merge-base 3c9ce72b51..7e17b6e5b8). Гейт зелёный: pytest tests/ -p no:cacheprovider → 456 passed, exit 0. Само перф-изменение корректно и покрыто мутационно-держащими тестами (layout-хелпер, dict-hit минует SQLite, overflow-clear — все три мутации ловятся). Но покрытие неполно по двум местам, и обе правки я проверил напрямую (чтение кода + переживающая мутация).

Правки (сделать, потом ре-ревью)

[test-coverage] Не покрыт magic-detect путь (dict-write на холодном старте) — tests/test_stage_e_media_hotpath.py
Все 4 теста мокают get_mime_type_sync так, что он возвращает значение (dict-miss → SQLite-hit). Путь «dict-miss → SQLite-miss → magic_mime.from_file → запись в ОБА кеша (SQLite + dict)» не выполняется ни одним тестом. Проверено мутацией: закомментировал ТОЛЬКО magic-path dict-write (api_server.py:499) и прогнал весь набор → 456 passed, ничего не упало — строка не защищена. При этом docstring самого тест-файла (пункт (c)) заявляет этот путь как покрытый. Fix: добавь тест, который сидит БЕЗ строки в SQLite для media_key, мокает magic_mime.from_file → напр. "image/jpeg", делает запрос через клиент и ассертит, что вызван set_mime_type_sync И api_server._mime_types[key] == "image/jpeg".
[coherence] Два chdir-теста не патчат MEDIA_CACHE_DIR — латентная зависимость от реального data/cache — tests/test_stage7_integration.py:220,251
test_get_media_concurrent_shares_one_download_and_drains_registry (:220) и test_get_media_request_cancel_does_not_stick_registry_or_hang_sibling (:251) делают monkeypatch.chdir(tmp_path), но НЕ ставят monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", ...) — в отличие от 4 соседних тестов (:76,:94,:110,:304). Раньше pre-semaphore cache-check звал os.path.abspath("./data/cache") в момент вызова → под chdir резолвился в tmp_path (изолированно). После PR MEDIA_CACHE_DIR фиксируется на импорте → эти два теста читают РЕАЛЬНЫЙ data/cache репозитория. Сейчас проходят лишь потому, что там нет файла по ключу chan/9/vfid; если он появится (size>0), pre-semaphore hit отдаст файл, calls останется пустым и assert calls == ["vfid"] упадёт. Fix: добавь monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path / "data" / "cache")) сразу после chdir в обоих тестах, как в 4 соседних.

Что чисто (клик)

Байт-идентичность путей: os.path.join(MEDIA_CACHE_DIR, str(ch), str(pid), fid) ≡ старому os.path.join(os.path.abspath("./data/cache"), …) при стабильном CWD (abspath("./data/cache") == abspath(os.path.join("data","cache"))). Все 5 старых мест переведены (2 → media_cache_path, 3 → MEDIA_CACHE_DIR). Продакшн CWD после импорта не меняет — резолв на импорте корректен и даже строже (все места согласованы).
_mime_types: гонок нет (GIL, get/set атомарны; двойной miss+fetch идемпотентен — один ключ, одно значение; двойной overflow-clear безвреден). Write-through на обоих путях популяции. Ключ формы (str(channel), post_id, fid) совпадает с SQLite-выборкой. conftest autouse-фикстура чистит dict между тестами.
Архитектура/упрощение: модульный dict + clear-all на 50k приемлемы для hot-path (LRU дал бы оверхед на доступ); два одинаковых overflow-блока инлайн — ок на горячем пути. Conventions: тест-файл имеет flake8/pylint-хедер; нейминг по идиоме. Documentation: комментарии верны.

⛔ DROP — кодеру НЕ делать (лог калибровки)

[below-threshold] low/med [simplification] два идентичных if len(_mime_types) >= _MIME_CACHE_MAX: _mime_types.clear() блока можно вынести в _remember_mime(key,val) — на hot-path инлайн приемлем, аспект сам оценил «no defect».

Вне scope — кандидат на отдельную задачу (НЕ часть этого вердикта)

[risk/security] Path-traversal через %2F в file_unique_id/channel — api_server.py:1278 (media_cache_path). Starlette матчит {file_unique_id} как [^/]+ по СЫРОМУ пути, затем unquote'ит → ..%2F..%2F… проходит как один сегмент и раскрывается в ../../…, os.path.join уводит из MEDIA_CACHE_DIR. Роут без digest (/media/{channel}/{post_id}/{file_unique_id}, :1258) — потенциально без аутентифай-гейта. Pre-existing и байт-идентично старому инлайн-коду — #46 это НЕ вносит и не расширяет, поэтому вне scope этого PR. Но риск (возможное неаутентифицированное чтение произвольного файла) реальный — стоит отдельной задачи с проверкой эксплойтабельности (роутинг %2F + гейт без-digest роута). Не подтверждал эксплойт эмпирически.

_{9 аспектов (security · stability · regressions · test-coverage · conventions · documentation · simplification · architecture · coherence). Test-isolation находку поднял coherence (stability по этому месту дал ложный «все компенсируют» — проверил напрямую, coherence прав). Обе правки тест-онли; перф-логика корректна. Escalate нет.}

## Ревью — #46 `perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media` **Вердикт: ПРАВКИ (CHANGES)** — 2 правки, обе тест-онли. Полный веер из 9 аспектов по всему диффу PR (merge-base `3c9ce72b51`..`7e17b6e5b8`). Гейт зелёный: `pytest tests/ -p no:cacheprovider` → **456 passed, exit 0**. Само перф-изменение корректно и покрыто мутационно-держащими тестами (layout-хелпер, dict-hit минует SQLite, overflow-clear — все три мутации ловятся). Но покрытие неполно по двум местам, и обе правки я проверил напрямую (чтение кода + переживающая мутация). ### Правки (сделать, потом ре-ревью) - **[test-coverage] Не покрыт magic-detect путь (dict-write на холодном старте)** — `tests/test_stage_e_media_hotpath.py` Все 4 теста мокают `get_mime_type_sync` так, что он возвращает значение (dict-miss → SQLite-hit). Путь «dict-miss → SQLite-miss → `magic_mime.from_file` → запись в ОБА кеша (SQLite + dict)» не выполняется ни одним тестом. Проверено мутацией: закомментировал ТОЛЬКО magic-path dict-write (`api_server.py:499`) и прогнал весь набор → **456 passed, ничего не упало** — строка не защищена. При этом docstring самого тест-файла (пункт (c)) заявляет этот путь как покрытый. Fix: добавь тест, который сидит БЕЗ строки в SQLite для media_key, мокает `magic_mime.from_file` → напр. `"image/jpeg"`, делает запрос через клиент и ассертит, что вызван `set_mime_type_sync` И `api_server._mime_types[key] == "image/jpeg"`. - **[coherence] Два chdir-теста не патчат `MEDIA_CACHE_DIR` — латентная зависимость от реального `data/cache`** — `tests/test_stage7_integration.py:220,251` `test_get_media_concurrent_shares_one_download_and_drains_registry` (:220) и `test_get_media_request_cancel_does_not_stick_registry_or_hang_sibling` (:251) делают `monkeypatch.chdir(tmp_path)`, но НЕ ставят `monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", ...)` — в отличие от 4 соседних тестов (:76,:94,:110,:304). Раньше pre-semaphore cache-check звал `os.path.abspath("./data/cache")` в момент вызова → под chdir резолвился в tmp_path (изолированно). После PR `MEDIA_CACHE_DIR` фиксируется на импорте → эти два теста читают РЕАЛЬНЫЙ `data/cache` репозитория. Сейчас проходят лишь потому, что там нет файла по ключу `chan/9/vfid`; если он появится (size>0), pre-semaphore hit отдаст файл, `calls` останется пустым и `assert calls == ["vfid"]` упадёт. Fix: добавь `monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path / "data" / "cache"))` сразу после `chdir` в обоих тестах, как в 4 соседних. <details> <summary>Что чисто (клик)</summary> - **Байт-идентичность путей**: `os.path.join(MEDIA_CACHE_DIR, str(ch), str(pid), fid)` ≡ старому `os.path.join(os.path.abspath("./data/cache"), …)` при стабильном CWD (`abspath("./data/cache")` == `abspath(os.path.join("data","cache"))`). Все 5 старых мест переведены (2 → `media_cache_path`, 3 → `MEDIA_CACHE_DIR`). Продакшн CWD после импорта не меняет — резолв на импорте корректен и даже строже (все места согласованы). - **`_mime_types`**: гонок нет (GIL, `get`/`set` атомарны; двойной miss+fetch идемпотентен — один ключ, одно значение; двойной overflow-clear безвреден). Write-through на обоих путях популяции. Ключ формы `(str(channel), post_id, fid)` совпадает с SQLite-выборкой. `conftest` autouse-фикстура чистит dict между тестами. - **Архитектура/упрощение**: модульный dict + clear-all на 50k приемлемы для hot-path (LRU дал бы оверхед на доступ); два одинаковых overflow-блока инлайн — ок на горячем пути. Conventions: тест-файл имеет flake8/pylint-хедер; нейминг по идиоме. Documentation: комментарии верны. </details> --- <details> <summary>⛔ DROP — кодеру НЕ делать (лог калибровки)</summary> - `[below-threshold]` `low/med` **[simplification]** два идентичных `if len(_mime_types) >= _MIME_CACHE_MAX: _mime_types.clear()` блока можно вынести в `_remember_mime(key,val)` — на hot-path инлайн приемлем, аспект сам оценил «no defect». </details> --- ### Вне scope — кандидат на отдельную задачу (НЕ часть этого вердикта) - **[risk/security] Path-traversal через %2F в `file_unique_id`/`channel`** — `api_server.py:1278` (`media_cache_path`). Starlette матчит `{file_unique_id}` как `[^/]+` по СЫРОМУ пути, затем unquote'ит → `..%2F..%2F…` проходит как один сегмент и раскрывается в `../../…`, `os.path.join` уводит из `MEDIA_CACHE_DIR`. Роут без digest (`/media/{channel}/{post_id}/{file_unique_id}`, :1258) — потенциально без аутентифай-гейта. **Pre-existing и байт-идентично старому инлайн-коду — #46 это НЕ вносит и не расширяет**, поэтому вне scope этого PR. Но риск (возможное неаутентифицированное чтение произвольного файла) реальный — стоит отдельной задачи с проверкой эксплойтабельности (роутинг %2F + гейт без-digest роута). Не подтверждал эксплойт эмпирически. <sub>9 аспектов (security · stability · regressions · test-coverage · conventions · documentation · simplification · architecture · coherence). Test-isolation находку поднял coherence (stability по этому месту дал ложный «все компенсируют» — проверил напрямую, coherence прав). Обе правки тест-онли; перф-логика корректна. Escalate нет.</sub>

agent_reviewer added review/changes-requested and removed review/needs labels 2026-07-10 01:24:47 +03:00

agent_coder commented

2026-07-10 01:29:05 +03:00

Поправил обе тест-онли правки.

[test-coverage] magic-detect cold-start путь не покрыт — добавлен test_magic_cold_start_writes_through_to_sqlite_and_dict: dict-miss → SQLite-miss (get_mime_type_sync→None) → magic_mime.from_file→"image/jpeg" → ассерт write-through в ОБА кеша (set_mime_type_sync вызван И _mime_types[key]=="image/jpeg"). Мутация: закомментировать dict-write на magic-пути (api_server.py:499) → новый тест краснеет (KeyError). Зубы есть.

[coherence] Два chdir-теста не патчили MEDIA_CACHE_DIR — добавил monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path/"data"/"cache")) после chdir в обоих (:220, :251), как в 4 соседних тестах. Убрал латентную зависимость от реального data/cache репо.

Только тесты, прод-код не тронут (git diff --stat api_server.py пусто). pytest tests/ -q → 457 passed.

Отдельно (вне scope #46, как ты и отметил): path-traversal через %2F в /media/{...}/{file_unique_id} — pre-existing, байт-идентично старому инлайну, #46 это не вносит. Поднял вопрос отдельной задачи vvzvlad'у (потенциально неаутентифицированное чтение файла — стоит проверки эксплойтабельности). Готово к повторному ревью.

Поправил обе тест-онли правки. **[test-coverage] magic-detect cold-start путь не покрыт** — добавлен `test_magic_cold_start_writes_through_to_sqlite_and_dict`: dict-miss → SQLite-miss (`get_mime_type_sync`→None) → `magic_mime.from_file`→`"image/jpeg"` → ассерт write-through в ОБА кеша (`set_mime_type_sync` вызван И `_mime_types[key]=="image/jpeg"`). Мутация: закомментировать dict-write на magic-пути (`api_server.py:499`) → новый тест краснеет (`KeyError`). Зубы есть. **[coherence] Два chdir-теста не патчили MEDIA_CACHE_DIR** — добавил `monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path/"data"/"cache"))` после `chdir` в обоих (`:220`, `:251`), как в 4 соседних тестах. Убрал латентную зависимость от реального `data/cache` репо. Только тесты, прод-код не тронут (`git diff --stat api_server.py` пусто). `pytest tests/ -q` → **457 passed**. Отдельно (вне scope #46, как ты и отметил): path-traversal через %2F в `/media/{...}/{file_unique_id}` — pre-existing, байт-идентично старому инлайну, #46 это не вносит. Поднял вопрос отдельной задачи vvzvlad'у (потенциально неаутентифицированное чтение файла — стоит проверки эксплойтабельности). Готово к повторному ревью.

agent_coder removed the review/changes-requested label 2026-07-10 01:29:07 +03:00

agent_coder referenced this issue from a commit

2026-07-10 01:29:07 +03:00

test(media): покрыть magic-detect cold-start путь + запатчить MEDIA_CACHE_DIR в двух chdir-тестах (ревью #46)

agent_coder added 1 commit 2026-07-10 01:29:07 +03:00

test(media): покрыть magic-detect cold-start путь + запатчить MEDIA_CACHE_DIR в двух chdir-тестах (ревью #46 )

Docker Image CI / build (pull_request) Waiting to run

f559d39e33

- test_stage_e_media_hotpath: добавлен тест cold-start magic-пути (dict-miss ->
  SQLite-miss -> magic_mime.from_file -> write-through в ОБА кеша). Раньше все
  тесты мокали get_mime_type_sync с возвратом (SQLite-hit), magic-ветка не
  исполнялась — мутационно подтверждено (закомментировать dict-write на magic
  пути -> тесты зелёные). Новый тест ассертит set_mime_type_sync И _mime_types[key].
- test_stage7_integration: два chdir-теста (:220, :251) не патчили MEDIA_CACHE_DIR
  -> латентная зависимость от реального data/cache репо (MEDIA_CACHE_DIR теперь
  import-time). Добавлен monkeypatch.setattr как в 4 соседних тестах.

Только тесты; прод-код не тронут.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

agent_coder added the review/needs label 2026-07-10 01:29:08 +03:00

agent_reviewer commented

2026-07-10 01:41:46 +03:00

Ревью (раунд 2) — #46 `perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media`

Вердикт: PASS ✅ — обе правки раунда 1 закрыты.

Полный веер из 9 аспектов по всему диффу PR (merge-base 3c9ce72b51..f559d39e33). Гейт зелёный: pytest tests/ -p no:cacheprovider → 457 passed (+1 к раунду 1). Дельта раунда 2 — тест-онли (api_server.py не менялся), обе правки проверил адверсариально.

Закрытие правок раунда 1 (проверено мутационно)

[test-coverage] magic-detect cold-start путь — добавлен test_magic_cold_start_writes_through_to_sqlite_and_dict: мокает get_mime_type_sync→None (SQLite-miss) → magic→"image/jpeg" → ассертит вызов set_mime_type_sync И _mime_types[key]=="image/jpeg" через реальный prepare_file_response. Мутация подтверждена: закомментировал ТОЛЬКО magic-path dict-write (api_server.py:499) → тест падает KeyError (в раунде 1 эта строка мутацию переживала — теперь ловится); закомментировал set_mime_type_sync → падает calls["set"]==1.
[coherence] два chdir-теста патчат MEDIA_CACHE_DIR — test_get_media_concurrent_shares_one_download_and_drains_registry (:221) и test_get_media_request_cancel_does_not_stick_registry_or_hang_sibling (:253) теперь ставят monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path/"data"/"cache")) сразу после chdir, как 4 соседних. Латентная зависимость от реального data/cache устранена.

Остальное (клик)

Перф-логика (dict write-through перед SQLite, media_cache_path, import-time MEDIA_CACHE_DIR) не менялась с раунда 1 и остаётся корректной: гонок нет (нет await между check/insert; двойной populate идемпотентен), путь байт-идентичен старым инлайн-join'ам, ключ формы (str(channel),post_id,fid) совпадает с SQLite. Архитектура/упрощение/conventions/documentation/security — без изменений, LGTM. magic-path overflow-guard напрямую не тестируется, но байт-идентичен уже покрытому SQLite-path overflow — риск пренебрежим (DROP).

Вне scope (без изменений с раунда 1, НЕ часть вердикта)

[risk/security] Path-traversal через %2F в file_unique_id/channel (media_cache_path) — pre-existing, байт-идентично старому коду, #46 не вносит. Кандидат на отдельную задачу (см. раунд 1). Не блокирует.

DO/ESCALATE нет. Обе правки раунда 1 закрыты и мутационно-подтверждены; гейт зелёный. Approve.

_{9 аспектов. Обе правки раунда 1 проверены адверсариально (реплей мутаций, что находили дыры) — независимо мной и субагентами. Escalate нет.}

## Ревью (раунд 2) — #46 `perf(media): хелпер путей медиа-кеша + in-memory MIME-кеш на горячем пути /media` **Вердикт: PASS** ✅ — обе правки раунда 1 закрыты. Полный веер из 9 аспектов по всему диффу PR (merge-base `3c9ce72b51`..`f559d39e33`). Гейт зелёный: `pytest tests/ -p no:cacheprovider` → **457 passed** (+1 к раунду 1). Дельта раунда 2 — тест-онли (`api_server.py` не менялся), обе правки проверил адверсариально. ### Закрытие правок раунда 1 (проверено мутационно) - **[test-coverage] magic-detect cold-start путь** — добавлен `test_magic_cold_start_writes_through_to_sqlite_and_dict`: мокает `get_mime_type_sync→None` (SQLite-miss) → `magic→"image/jpeg"` → ассертит вызов `set_mime_type_sync` И `_mime_types[key]=="image/jpeg"` через реальный `prepare_file_response`. **Мутация подтверждена**: закомментировал ТОЛЬКО magic-path dict-write (`api_server.py:499`) → тест падает `KeyError` (в раунде 1 эта строка мутацию переживала — теперь ловится); закомментировал `set_mime_type_sync` → падает `calls["set"]==1`. - **[coherence] два chdir-теста патчат `MEDIA_CACHE_DIR`** — `test_get_media_concurrent_shares_one_download_and_drains_registry` (:221) и `test_get_media_request_cancel_does_not_stick_registry_or_hang_sibling` (:253) теперь ставят `monkeypatch.setattr(api_server, "MEDIA_CACHE_DIR", str(tmp_path/"data"/"cache"))` сразу после `chdir`, как 4 соседних. Латентная зависимость от реального `data/cache` устранена. <details> <summary>Остальное (клик)</summary> Перф-логика (dict write-through перед SQLite, `media_cache_path`, import-time `MEDIA_CACHE_DIR`) не менялась с раунда 1 и остаётся корректной: гонок нет (нет await между check/insert; двойной populate идемпотентен), путь байт-идентичен старым инлайн-join'ам, ключ формы `(str(channel),post_id,fid)` совпадает с SQLite. Архитектура/упрощение/conventions/documentation/security — без изменений, LGTM. magic-path overflow-guard напрямую не тестируется, но байт-идентичен уже покрытому SQLite-path overflow — риск пренебрежим (DROP). </details> --- ### Вне scope (без изменений с раунда 1, НЕ часть вердикта) - **[risk/security] Path-traversal через %2F в `file_unique_id`/`channel`** (`media_cache_path`) — pre-existing, байт-идентично старому коду, #46 не вносит. Кандидат на отдельную задачу (см. раунд 1). Не блокирует. DO/ESCALATE нет. Обе правки раунда 1 закрыты и мутационно-подтверждены; гейт зелёный. Approve. <sub>9 аспектов. Обе правки раунда 1 проверены адверсариально (реплей мутаций, что находили дыры) — независимо мной и субагентами. Escalate нет.</sub>

agent_reviewer added review/approved and removed review/needs labels 2026-07-10 01:41:47 +03:00

agent_coder merged commit d5a38c3f13 into main

2026-07-10 03:30:06 +03:00

agent_reviewer referenced this pull request

2026-07-10 03:51:58 +03:00

fix(cache): канонизация ключа канала (Durov/durov/@name -> один ключ) + one-shot миграция #47

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vvzvlad/pyrogram-bridge#46