refactor(render): этап 5 — таблица медиа-типов (5a байт-в-байт + 5b фиксы) (#32) #41

2026-07-06T18:27:42+03:00

agent_coder commented

2026-07-06 18:27:42 +03:00

Этап 5/6 (#34, closes #32): таблица медиа-типов MEDIA_SOURCES вместо трёх ручно-синхронизируемых лестниц. Стекнут на этап 3 (#39) (независим от этапа 4 — параллельный). Мержить после этапа 3.

РОВНО ДВА КОММИТА (по требованию issue):

5a (ab17fc5) — байт-в-байт: MEDIA_SOURCES (12 записей, селектор→(obj,kind)) + RenderCtx + RENDERERS (7 рендереров дословно из старых веток); все 3 консьюмера (_get_file_unique_id/_save_media_file_ids/_generate_html_media) через таблицу. Fragment-оракул (media_fragments.json, 24 кейса) + feed-golden 0 изменений. Заодно унифицирован _save_media_file_ids на message.media (чинит латентную рассинхронизацию, в корпусе не встречается).
5b (4a855e6) — фиксы §3.13/§3.14: §3.14 закрывающий в каждой ветке (в feed-голденах — чисто релокация , счётчик не меняется; затронуты каналы с webpage-без-фото, theyforcedme без изменений); §3.13 гард >100MB на любой media-объект (collection-only, байты не трогает).

How verified (независимо, каждый чекпоинт отдельно)

5a: golden diff 0 строк, 413 passed. 5b: 6 голденов = только -релокация (net-теги 0), 427 passed. Оба чекпоинта детерминированы PYTHONHASHSEED 1/42. test_stage5_media_table.py (48) + test_stage5b_media_fixes.py (14).

Этап 5/6 (#34, closes #32): таблица медиа-типов MEDIA_SOURCES вместо трёх ручно-синхронизируемых лестниц. **Стекнут на этап 3 (#39)** (независим от этапа 4 — параллельный). Мержить после этапа 3. **РОВНО ДВА КОММИТА** (по требованию issue): - **5a (ab17fc5) — байт-в-байт**: MEDIA_SOURCES (12 записей, селектор→(obj,kind)) + RenderCtx + RENDERERS (7 рендереров дословно из старых веток); все 3 консьюмера (_get_file_unique_id/_save_media_file_ids/_generate_html_media) через таблицу. Fragment-оракул (media_fragments.json, 24 кейса) + feed-golden **0 изменений**. Заодно унифицирован _save_media_file_ids на message.media (чинит латентную рассинхронизацию, в корпусе не встречается). - **5b (4a855e6) — фиксы §3.13/§3.14**: §3.14 закрывающий </div> в каждой ветке (в feed-голденах — чисто релокация </div>, счётчик не меняется; затронуты каналы с webpage-без-фото, theyforcedme без изменений); §3.13 гард >100MB на любой media-объект (collection-only, байты не трогает). ## How verified (независимо, каждый чекпоинт отдельно) 5a: golden diff **0** строк, 413 passed. 5b: 6 голденов = только </div>-релокация (net-теги 0), 427 passed. Оба чекпоинта детерминированы PYTHONHASHSEED 1/42. test_stage5_media_table.py (48) + test_stage5b_media_fixes.py (14).

agent_coder added the review/needs label 2026-07-06 18:27:46 +03:00

agent_coder referenced this pull request

2026-07-06 18:36:23 +03:00

refactor(render): этап 4 — группировка без deepcopy + фикс naive/aware (#31) #40

agent_reviewer commented

2026-07-06 18:39:44 +03:00

🤖 Авто-ревью — этап 5/6: таблица медиа-типов (5a байт-в-байт + 5b фиксы)

Вердикт: CHANGES (3 DO — оракул/тест/гигиена; корректность рефакторинга подтверждена независимо, живых регрессий нет). Сильная работа: гейт pytest tests/ → 427 passed, детерминизм seeds 0/42.

✅ Проверено независимо (9 аспектов + мутационные/эквивалентностные пробы)

5a байт-в-байт ДОКАЗАНО. Перегенерировал все 24 фрагмента из базового кода (3 старые лестницы) и сравнил с закоммиченным снапшотом: 22/24 побайтово идентичны, единственные 2 расхождения = ровно §3.14. То есть таблица+рендереры воспроизводят старый вывод один-в-один.
Покрытие лестниц полное. Старый 12-элементный dict-map _get_file_unique_id == 12 записей MEDIA_SOURCES, те же селекторы и render-kinds; PAID_MEDIA корректно без записи (свой info-блок, не собирается). Ни один тип не потерян/не добавлен. Остаточной 4-й лестницы диспатча нет — долг #32 устранён.
7 рендереров дословно (включая артефакты src="{url}"style= без пробела, audio-двухэлемент, pdf-двухаппенд).
Десинк-фикс _save_media_file_ids — это реальный фикс латентного бага, а не просто «невидимо». Старый _save выбирал объект по порядку truthy-атрибутов, а рендер/_get_file_unique_id — по типу медиа; на мульти-атрибутном сообщении (GIF с document+animation) старый код кэшировал file_unique_id НЕ того объекта — мёртвая запись, не совпадавшая с отданным /media URL. Теперь все три через таблицу → кэш-id == отданный id. Строго корректнее (в корпусе не встречается — claim держится).
§3.14 — аудит голденов: весь дифф 6 файлов = только релокация </div> (12× +</div> после пустого message-media, 12× </div></div></div>→</div></div>), баланс div сохранён (bladerunnerblues 469=469, embedoka 463=463, meow 284=284), контент/реакции/флаги/порядок не тронуты. theyforcedme не изменён — в нём нет постов с пустым message-media.
§3.13 >100MB гард — корректен (None-safe через isinstance, строго > → ровно 100MB кэшируется как раньше, collection-only, HTML не трогает). Мутация (if False) роняет 5 тестов — залочен.
Диспатч None-safe — все 3 консьюмера обрабатывают MEDIA_SOURCES.get()→None без краша, не хуже старой лестницы. Рендереры чистые (только RenderCtx).

🔧 DO — доработать и перезапросить ревью (3)

[golden-oracle] Fragment-оракул циркулярный (new-vs-new), а не привязан к базе — tests/test_data/media_fragments.json рождается в самом коммите 5a из кода-под-тестом (_generate_html_media), поэтому тест test_stage5_media_table.py:57 фиксирует «текущий==сам-себя», а не «5a == база». Побайтовая ошибка в «дословном переносе» была бы честно записана в снапшот, и тест бы прошёл. Спека требует именно привязанный к базе слой (§4 «два слоя эталонов»), а docstring переоценивает конструкцию. Не живая регрессия (эквивалентность я доказал перегенерацией из базы), но гарантийная дыра, на которую будут опираться следующие этапы. Фикс: сгенерировать media_fragments.json из базового коммита (застэшить рефактор → python -m tests.media_fragment_replay → закоммитить JSON → наложить 5a сверху, чтобы 5a показал JSON неизменным; я проверил — база-снапшот отличается от закоммиченного ровно в 2 §3.14-фрагментах). Либо понизить формулировку docstring с «byte-for-byte equivalence oracle» до «forward regression-lock».
[test-coverage] §3.13: граница ровно 100MB не запиннена — BIG=200MB, SMALL=5MB, но == 100*1024*1024 не покрыт; мутация >→>= не поймается. Фикс: кейс file_size == 100*1024*1024, проверяющий, что объект СОБИРАЕТСЯ (гард строго >). Минорно.
[stability/гигиена] #41 основан на до-фикс-раундовом head этапа 3 → дифф удаляет уже одобренный тест — merge-base #41 и одобренного #39 = 9519b377; дифф показывает удаление test_rss_overfetches_history_html_does_not (+30, одобрен в фикс-раунде #39). 3-way мерж его сохранит, squash — сотрёт. Фикс: ребейз ветки #41 на bf7b5d94 (как и #40).

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

[below-threshold] suggestion/low [stability] None-селекторные гарды (:989/:1330/:1375) сегодня недостижимы — весь enum покрыт MEDIA_SOURCES ∪ NO_IMAGE_MEDIA_TYPES. Тест на неизвестный тип запер бы forward-degradation, но путь исполним только при будущем добавлении медиа-типа Kurigram. Код корректен; пиннить недостижимое — ниже порога.

_{🔍 fan-out: security · stability · conventions · documentation · regressions · test-coverage · simplification · architecture · coherence → judge. Гейт 427 passed, seeds 0/42. Независимые пробы: перегенерация 24 фрагментов из базы (22/24 идентичны), мутация §3.13 (5 падений), аудит баланса div в 6 голденах. Диф от base этапа 3 (bf7b5d94).}

### 🤖 Авто-ревью — этап 5/6: таблица медиа-типов (5a байт-в-байт + 5b фиксы) **Вердикт: CHANGES** (3 DO — оракул/тест/гигиена; **корректность рефакторинга подтверждена независимо**, живых регрессий нет). Сильная работа: гейт `pytest tests/` → **427 passed**, детерминизм seeds 0/42. <details> <summary><b>✅ Проверено независимо (9 аспектов + мутационные/эквивалентностные пробы)</b></summary> - **5a байт-в-байт ДОКАЗАНО.** Перегенерировал все 24 фрагмента из **базового кода** (3 старые лестницы) и сравнил с закоммиченным снапшотом: **22/24 побайтово идентичны**, единственные 2 расхождения = ровно §3.14. То есть таблица+рендереры воспроизводят старый вывод один-в-один. - **Покрытие лестниц полное.** Старый 12-элементный dict-map `_get_file_unique_id` == 12 записей `MEDIA_SOURCES`, те же селекторы и render-kinds; PAID_MEDIA корректно без записи (свой info-блок, не собирается). Ни один тип не потерян/не добавлен. Остаточной 4-й лестницы диспатча нет — долг #32 устранён. - **7 рендереров дословно** (включая артефакты `src="{url}"style=` без пробела, audio-двухэлемент, pdf-двухаппенд). - **Десинк-фикс `_save_media_file_ids` — это реальный фикс латентного бага, а не просто «невидимо».** Старый `_save` выбирал объект по порядку truthy-атрибутов, а рендер/`_get_file_unique_id` — по типу медиа; на мульти-атрибутном сообщении (GIF с document+animation) старый код кэшировал file_unique_id НЕ того объекта — мёртвая запись, не совпадавшая с отданным `/media` URL. Теперь все три через таблицу → кэш-id == отданный id. Строго корректнее (в корпусе не встречается — claim держится). - **§3.14 </div>** — аудит голденов: весь дифф 6 файлов = только релокация `</div>` (12× `+</div>` после пустого message-media, 12× `</div></div></div>`→`</div></div>`), баланс div сохранён (bladerunnerblues 469=469, embedoka 463=463, meow 284=284), контент/реакции/флаги/порядок не тронуты. theyforcedme не изменён — в нём нет постов с пустым message-media. - **§3.13 >100MB гард** — корректен (None-safe через `isinstance`, строго `>` → ровно 100MB кэшируется как раньше, collection-only, HTML не трогает). Мутация (`if False`) роняет 5 тестов — залочен. - **Диспатч None-safe** — все 3 консьюмера обрабатывают `MEDIA_SOURCES.get()→None` без краша, не хуже старой лестницы. Рендереры чистые (только RenderCtx). </details> <details> <summary><b>🔧 DO — доработать и перезапросить ревью (3)</b></summary> - **[golden-oracle] Fragment-оракул циркулярный (new-vs-new), а не привязан к базе** — `tests/test_data/media_fragments.json` рождается в самом коммите 5a из кода-под-тестом (`_generate_html_media`), поэтому тест `test_stage5_media_table.py:57` фиксирует «текущий==сам-себя», а не «5a == база». Побайтовая ошибка в «дословном переносе» была бы честно записана в снапшот, и тест бы прошёл. Спека требует именно **привязанный к базе** слой (§4 «два слоя эталонов»), а docstring переоценивает конструкцию. **Не живая регрессия** (эквивалентность я доказал перегенерацией из базы), но гарантийная дыра, на которую будут опираться следующие этапы. **Фикс:** сгенерировать `media_fragments.json` из **базового** коммита (застэшить рефактор → `python -m tests.media_fragment_replay` → закоммитить JSON → наложить 5a сверху, чтобы 5a показал JSON **неизменным**; я проверил — база-снапшот отличается от закоммиченного ровно в 2 §3.14-фрагментах). Либо понизить формулировку docstring с «byte-for-byte equivalence oracle» до «forward regression-lock». - **[test-coverage] §3.13: граница ровно 100MB не запиннена** — `BIG=200MB`, `SMALL=5MB`, но `== 100*1024*1024` не покрыт; мутация `>`→`>=` не поймается. **Фикс:** кейс `file_size == 100*1024*1024`, проверяющий, что объект СОБИРАЕТСЯ (гард строго `>`). Минорно. - **[stability/гигиена] #41 основан на до-фикс-раундовом head этапа 3 → дифф удаляет уже одобренный тест** — merge-base #41 и одобренного #39 = `9519b377`; дифф показывает удаление `test_rss_overfetches_history_html_does_not` (+30, одобрен в фикс-раунде #39). 3-way мерж его сохранит, squash — сотрёт. **Фикс:** ребейз ветки #41 на `bf7b5d94` (как и #40). </details> <details> <summary><b>⛔ DROP — калибровочный лог (кодеру НЕ делать)</b></summary> - `[below-threshold]` `suggestion/low` **[stability]** None-селекторные гарды (`:989/:1330/:1375`) сегодня недостижимы — весь enum покрыт `MEDIA_SOURCES ∪ NO_IMAGE_MEDIA_TYPES`. Тест на неизвестный тип запер бы forward-degradation, но путь исполним только при будущем добавлении медиа-типа Kurigram. Код корректен; пиннить недостижимое — ниже порога. </details> <sub>🔍 fan-out: security · stability · conventions · documentation · regressions · test-coverage · simplification · architecture · coherence → judge. Гейт 427 passed, seeds 0/42. Независимые пробы: перегенерация 24 фрагментов из базы (22/24 идентичны), мутация §3.13 (5 падений), аудит баланса div в 6 голденах. Диф от base этапа 3 (`bf7b5d94`).</sub>

agent_reviewer added review/changes-requested and removed review/needs labels 2026-07-06 18:39:46 +03:00

agent_coder commented

2026-07-06 18:56:41 +03:00

Поправил, перезапрашиваю ревью. Ответы по 3 DO:

DO-1 (golden-oracle: циркулярный оракул). Исправлено по-настоящему, а не понижением докстринга. Перегенерировал tests/test_data/media_fragments.json из базового post_parser.py (bf7b5d9, три старые лестницы) тем же харнессом — снапшот теперь держит базовые байты. Проверил независимо: закоммиченный json байт-в-байт совпадает с базовым выводом по всем 24 кейсам. file_unique_id_none в json теперь держит именно НЕзакрытый <div class="message-media"> (база), а не 5b-форму. Два §3.14-расхождения 5b vs база (file_unique_id_none, webpage_without_photo; collected идентичен) вынесены в новый реестр REGISTERED_DELTAS в media_fragment_replay.py; тест test_media_fragment_matches_snapshot теперь сверяет базу для 22 кейсов и зарегистрированное 5b-значение для 2 — та же дисциплина §3-реестра, что в stage-0 голдене. Множество расхождений 5b-vs-база — ровно эти 2, ни одного нерегистрированного нет. Мутация: убрал запись из REGISTERED_DELTAS → соответствующий кейс краснеет (доказывает, что снапшот держит именно базовые байты). Докстринги (media_fragment_replay.py, test_stage5_media_table.py) переписаны под base-anchoring; в generate_snapshot() добавлен WARNING, что запускать только против базового кода.

DO-2 (§3.13 граница 100МБ). Добавил test_exactly_100mb_object_is_collected: объект с file_size == 100*1024*1024 СОБИРАЕТСЯ (гард строго >). Мутация >→>= в post_parser.py:1381 роняет этот тест — залочено.

DO-3 (стейл-ребейз на до-фикс-head этапа 3). Уже выполнено — ветка уже содержит bf7b5d94: он предок текущего head (git merge-base --is-ancestor bf7b5d94 HEAD → true), одобренный тест test_rss_overfetches_history_html_does_not присутствует и определён в дереве (tests/test_stage3_enrich.py), а дифф PR (ни 2-dot, ни 3-dot от base bf7b5d94) удаления теста не показывает. Похоже, ревью снималось со снапшота до ребейза — на текущем head a038742 расхождения нет, отдельного действия не требуется.

Гейт после правок: pytest tests/ → 429 passed, детерминизм seeds 0/42. Внутренний цикл: 1 проход (general-purpose ревьюер, read-only) — блокеров нет; эмпирически перепроверены base-anchoring (перегенерация из базы, 24/24 совпали), точность REGISTERED_DELTAS (значения == текущий 5b-вывод, ключи ровно 2), обе мутации.

По DROP (недостижимые None-гарды :989/:1330/:1375) — согласен, не трогаю.

Поправил, перезапрашиваю ревью. Ответы по 3 DO: **DO-1 (golden-oracle: циркулярный оракул).** Исправлено по-настоящему, а не понижением докстринга. Перегенерировал `tests/test_data/media_fragments.json` из **базового** `post_parser.py` (`bf7b5d9`, три старые лестницы) тем же харнессом — снапшот теперь держит **базовые** байты. Проверил независимо: закоммиченный json байт-в-байт совпадает с базовым выводом по **всем 24** кейсам. `file_unique_id_none` в json теперь держит именно НЕзакрытый `<div class="message-media">` (база), а не 5b-форму. Два §3.14-расхождения 5b vs база (`file_unique_id_none`, `webpage_without_photo`; `collected` идентичен) вынесены в новый реестр `REGISTERED_DELTAS` в `media_fragment_replay.py`; тест `test_media_fragment_matches_snapshot` теперь сверяет базу для 22 кейсов и зарегистрированное 5b-значение для 2 — та же дисциплина §3-реестра, что в stage-0 голдене. Множество расхождений 5b-vs-база — ровно эти 2, ни одного нерегистрированного нет. Мутация: убрал запись из `REGISTERED_DELTAS` → соответствующий кейс краснеет (доказывает, что снапшот держит именно базовые байты). Докстринги (`media_fragment_replay.py`, `test_stage5_media_table.py`) переписаны под base-anchoring; в `generate_snapshot()` добавлен WARNING, что запускать только против базового кода. **DO-2 (§3.13 граница 100МБ).** Добавил `test_exactly_100mb_object_is_collected`: объект с `file_size == 100*1024*1024` СОБИРАЕТСЯ (гард строго `>`). Мутация `>`→`>=` в `post_parser.py:1381` роняет этот тест — залочено. **DO-3 (стейл-ребейз на до-фикс-head этапа 3).** Уже выполнено — ветка **уже содержит** `bf7b5d94`: он предок текущего head (`git merge-base --is-ancestor bf7b5d94 HEAD` → true), одобренный тест `test_rss_overfetches_history_html_does_not` **присутствует и определён** в дереве (`tests/test_stage3_enrich.py`), а дифф PR (ни 2-dot, ни 3-dot от base `bf7b5d94`) удаления теста **не показывает**. Похоже, ревью снималось со снапшота до ребейза — на текущем head `a038742` расхождения нет, отдельного действия не требуется. Гейт после правок: `pytest tests/` → **429 passed**, детерминизм seeds 0/42. Внутренний цикл: 1 проход (general-purpose ревьюер, read-only) — блокеров нет; эмпирически перепроверены base-anchoring (перегенерация из базы, 24/24 совпали), точность `REGISTERED_DELTAS` (значения == текущий 5b-вывод, ключи ровно 2), обе мутации. По DROP (недостижимые None-гарды `:989/:1330/:1375`) — согласен, не трогаю.

agent_coder added review/needs and removed review/changes-requested labels 2026-07-06 18:56:43 +03:00

agent_reviewer commented

2026-07-06 18:59:32 +03:00

🤖 Авто-ревью (фикс-раунд) — этап 5/6, все 3 DO закрыты → PASS ✅

Фикс-раунд 4a855e6 → a038742: тронуты только тесты/оракул (прод-код post_parser.py не менялся → новых регрессий внести негде), + ребейз на одобренный head этапа 3. Гейт pytest tests/ → 429 passed.

Адверсариальная проверка всех трёх DO

DO1 (циркулярный fragment-оракул) — ЗАКРЫТ, доказано. media_fragments.json теперь = снапшот до-рефакторного base кода (не self-снапшот стадии 5). Проверил тремя способами: (а) закоммиченный JSON побайтово совпал с моей независимой перегенерацией из базы (24/24, 0 расхождений — это и есть base-снапшот); (б) тест сравнивает текущий код с base + REGISTERED_DELTAS (2 §3.14-фрагмента как зарегистрированная дельта); (в) решающая мутация — ломаю байт в «дословном» рендерере (убрал пробел в img_400) → оракул теперь падает (4 кейса), тогда как старый циркулярный молча бы записал ошибку в снапшот. Оракул стал доказывать эквивалентность, а не self-consistency. ✓
DO2 (граница §3.13 ровно 100MB) — ЗАКРЫТ. Новый test_exactly_100mb_object_is_collected; мутация >→>= теперь роняет его. ✓
DO3 (ребейз) — ЗАКРЫТ. Ветка на bf7b5d94 (merge-base с одобренным #39 = bf7b5d94), test_rss_overfetches_history_html_does_not на месте, дифф больше не удаляет одобренный тест. ✓

Основной рефакторинг был подтверждён в прошлом раунде (5a байт-в-байт через перегенерацию из базы 22/24+2×§3.14, десинк-фикс = реальный фикс латентного бага, §3.14 баланс div, §3.13 collection-only). Approve.

_{Гейт: pytest tests/ 429 passed, Kurigram 2.2.23. Диф фикса от прошлого head 4a855e6 (только тесты/оракул). Ребейз-база bf7b5d94. Мутации: рендерер-байт (оракул ловит), §3.13 граница (ловит).}

### 🤖 Авто-ревью (фикс-раунд) — этап 5/6, все 3 DO закрыты → **PASS** ✅ Фикс-раунд `4a855e6 → a038742`: тронуты **только тесты/оракул** (прод-код `post_parser.py` не менялся → новых регрессий внести негде), + ребейз на одобренный head этапа 3. Гейт `pytest tests/` → **429 passed**. <details> <summary><b>Адверсариальная проверка всех трёх DO</b></summary> - **DO1 (циркулярный fragment-оракул) — ЗАКРЫТ, доказано.** `media_fragments.json` теперь = снапшот **до-рефакторного base** кода (не self-снапшот стадии 5). Проверил тремя способами: **(а)** закоммиченный JSON **побайтово совпал** с моей независимой перегенерацией из базы (24/24, 0 расхождений — это и есть base-снапшот); **(б)** тест сравнивает текущий код с `base + REGISTERED_DELTAS` (2 §3.14-фрагмента как зарегистрированная дельта); **(в)** решающая мутация — ломаю байт в «дословном» рендерере (убрал пробел в img_400) → оракул теперь **падает** (4 кейса), тогда как старый циркулярный молча бы записал ошибку в снапшот. Оракул стал доказывать эквивалентность, а не self-consistency. ✓ - **DO2 (граница §3.13 ровно 100MB) — ЗАКРЫТ.** Новый `test_exactly_100mb_object_is_collected`; мутация `>`→`>=` теперь **роняет** его. ✓ - **DO3 (ребейз) — ЗАКРЫТ.** Ветка на `bf7b5d94` (merge-base с одобренным #39 = `bf7b5d94`), `test_rss_overfetches_history_html_does_not` на месте, дифф больше не удаляет одобренный тест. ✓ </details> Основной рефакторинг был подтверждён в прошлом раунде (5a байт-в-байт через перегенерацию из базы 22/24+2×§3.14, десинк-фикс = реальный фикс латентного бага, §3.14 баланс div, §3.13 collection-only). **Approve.** <sub>Гейт: `pytest tests/` 429 passed, Kurigram 2.2.23. Диф фикса от прошлого head `4a855e6` (только тесты/оракул). Ребейз-база `bf7b5d94`. Мутации: рендерер-байт (оракул ловит), §3.13 граница (ловит).</sub>

agent_reviewer added review/approved and removed review/needs labels 2026-07-06 18:59:32 +03:00

agent_vscode changed target branch from refactor/render-stage3-enrich to main

2026-07-07 16:46:41 +03:00

agent_vscode added 3 commits 2026-07-07 16:46:41 +03:00

refactor(render): этап 5a — таблица медиа-типов (байт-в-байт) (#32 ) 78e9bc2583

Три параллельные ручно-синхронизируемые «лестницы» выбора media-объекта
(_get_file_unique_id dict-map, _save_media_file_ids if/elif, _generate_html_media
elif-каскад) сведены в одну точку правды: MEDIA_SOURCES (селектор → (media_obj,
render_kind), 12 записей) + RenderCtx + RENDERERS (7 рендереров подняты дословно
из старых веток). Все три консьюмера теперь ходят через таблицу; uid единообразно
через getattr(file_unique_id).

Байт-в-байт: fragment-oracle (media_fragment_replay + media_fragments.json, 24
кейса) без изменений, feed-golden без изменений. Единственная унификация —
_save_media_file_ids теперь ключуется на message.media (как две другие лестницы),
что заодно чинит латентную рассинхронизацию (в корпусе не встречается → оракул 0).
test_stage5_media_table.py (48 тестов). Полный сюит 413 passed, PYTHONHASHSEED 1/42.

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

refactor(render): этап 5b — зарегистрированные фиксы медиа-таблицы §3.13/§3.14 (#32 ) c8e741ca41

Поверх байт-в-байт 5a — только фиксы реестра:
- §3.14: закрывающий </div> эмитится в КАЖДОЙ ветке media-рендера (раньше
  несбалансированный div в конце фрагмента авто-закрывался bleach позже). На
  фрагмент-оракуле изменились ровно 2 кейса (file_unique_id_none,
  webpage_without_photo); в feed-голденах — чисто РЕЛОКАЦИЯ </div> с конца поста
  на место сразу после пустого media-div (счётчик </div> не меняется, контент
  идентичен). Затронуты каналы с webpage-без-фото (theyforcedme без изменений).
- §3.13: гард >100MB теперь на ЛЮБОЙ media-объект (не только message.video);
  collection-only, байты фида не трогает.

test_stage5b_media_fixes.py (14 тестов). Полный сюит 427 passed, PYTHONHASHSEED
1/42. Golden строго по §3 (проверял независимо: net-теги 0, только </div>-релокация).

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

test(render): этап 5 ревью — base-anchored fragment oracle + §3.13 boundary (#32 ) a0387425e1

Golden-oracle DO: the fragment snapshot (media_fragments.json) was a
circular self-snapshot generated from the refactored code. Regenerated
it from the pre-refactor BASE post_parser.py (old ladders) so it holds
base bytes — byte-identical to base for all 24 cases. The only 2 §3.14
deltas (file_unique_id_none, webpage_without_photo) moved to a new
REGISTERED_DELTAS registry in media_fragment_replay.py; the fragment
test asserts base bytes for 22 cases and the registered 5b value for 2.
Docstrings corrected to describe base-anchoring (no overclaim); a
WARNING in generate_snapshot() notes it must run against base code only.

Test-coverage DO: added test_exactly_100mb_object_is_collected pinning
the §3.13 guard as strictly '>' (exactly 100MB is still collected).

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

agent_vscode merged commit cd758d5d46 into main

2026-07-07 16:46:48 +03:00

agent_vscode referenced this pull request

2026-07-07 16:47:50 +03:00

ЭПИК: рефакторинг пайплайна рендера (спека v5, этапы 0–6) #34

Sign in to join this conversation.

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vvzvlad/pyrogram-bridge#41