chexit/airdef
1
0
Fork 0
Code Issues 22 Pull Requests Actions Packages Projects 1 Releases 54 Wiki Activity

[bug] Переключение replay mode во время экспорта оставляет захват писать бесконечно #83

New Issue
Closed
opened 2026-10-30 09:56:59 +00:00 by chexit · 1 comment
chexit commented 2026-10-30 09:56:59 +00:00
Owner
Copy Link
Copy Source

🤖 Filed by Codex, AI-агент Владимира (chexit).

Ревью 76f680eba64b64eb28435e71ce9f22a84b0e5da1 (после 6f3fc64). Новые entry points повтора заменяют Player, не согласуя это с запущенным export state machine.

Воспроизведение по путям команд

  1. /airdef replay export ..., дождаться Stage::Recording.
  2. /airdef replay map (также full/reel или кнопка другого момента).
  3. start_replay записывает новый Player в self.replay, не останавливая self.export/capture. Новый Player имеет ended=false.
  4. export_step продолжает обрабатывать старый Export. В Stage::Recording единственный нормальный stop — r.ended, который устанавливает только tick_film. Map/Full/Clips этого поля никогда не выставляют: они останавливаются через paused.

Поэтому файл продолжает расти/захват остаётся активным, хотя запрошенный фильм уже заменён. Также start_film можно вызвать во время текущего export и подменить материал. Переход Film→обычный replay оставляет film_prev/set_film_quality до общего stop, то есть режимы наследуют временные настройки.

Нужно

Единая функция перехода между replay/film/export: запретить конфликтующую замену либо корректно завершить/отменить текущую сессию перед сменой Player. Привязать exporter к поколению конкретного фильма/записи; случайный новый Player не должен им управлять. Восстанавливать качество/effects/clean HUD и обработать команды из UI так же, как из чата. Ошибки/cancel не оставляют writer и audio offline mode висеть.

Приёмка

Из Planning/Arming/Recording/Finishing переключиться в map/full/reel/другой film и stop, сменить мир. Нет бесконечного захвата, содержимое не подменяется, статус правдивый. После завершения возвращаются прежние настройки. Статически подтверждённый дефект состояний, реальный MP4-прогон здесь не запускался.

🤖 Filed by Codex, AI-агент Владимира (chexit). Ревью `76f680eba64b64eb28435e71ce9f22a84b0e5da1` (после `6f3fc64`). Новые entry points повтора заменяют Player, не согласуя это с запущенным export state machine. ### Воспроизведение по путям команд 1. `/airdef replay export ...`, дождаться Stage::Recording. 2. `/airdef replay map` (также full/reel или кнопка другого момента). 3. `start_replay` записывает новый Player в `self.replay`, **не останавливая self.export/capture**. Новый Player имеет `ended=false`. 4. `export_step` продолжает обрабатывать старый Export. В Stage::Recording единственный нормальный stop — `r.ended`, который устанавливает только tick_film. Map/Full/Clips этого поля никогда не выставляют: они останавливаются через paused. Поэтому файл продолжает расти/захват остаётся активным, хотя запрошенный фильм уже заменён. Также `start_film` можно вызвать во время текущего export и подменить материал. Переход Film→обычный replay оставляет film_prev/set_film_quality до общего stop, то есть режимы наследуют временные настройки. ### Нужно Единая функция перехода между replay/film/export: запретить конфликтующую замену либо корректно завершить/отменить текущую сессию перед сменой Player. Привязать exporter к поколению конкретного фильма/записи; случайный новый Player не должен им управлять. Восстанавливать качество/effects/clean HUD и обработать команды из UI так же, как из чата. Ошибки/cancel не оставляют writer и audio offline mode висеть. ### Приёмка Из Planning/Arming/Recording/Finishing переключиться в map/full/reel/другой film и stop, сменить мир. Нет бесконечного захвата, содержимое не подменяется, статус правдивый. После завершения возвращаются прежние настройки. Статически подтверждённый дефект состояний, реальный MP4-прогон здесь не запускался.
chexit added the area:uiarea:toolingtype:bugarea:renderingpriority:P2 labels 2026-10-30 09:56:59 +00:00
chexit commented 2026-10-30 12:19:17 +00:00
Author
Owner
Copy Link
Copy Source

🤖 Agent: Сделано в 0f5c4136f3 (запушено в main). Смена режима во время экспорта запрещена, сессии завершаются в одном месте. Проверено тестом, отказ в игре не видели.

🤖 Agent: Сделано в https://gitea.maoniao.team/chexit/airdef/commit/0f5c4136f3bbd2ae3ca06e0b4985f86882a76eb0 (запушено в main). Смена режима во время экспорта запрещена, сессии завершаются в одном месте. Проверено тестом, отказ в игре не видели.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area:architecture
Границы модулей, контракты, состояние сессии и развитие общей архитектуры
 area:audio
Звуки, громкость, задержка распространения и аудиоассеты.
 area:logistics
Боезапас, склады, ТЗМ, обслуживание и заявки снабжения.
 area:persistence
Сохранение, загрузка, сценарии и изоляция состояния между мирами.
 area:rendering
Шейдеры, GPU-ресурсы, освещение, геометрия, эффекты и рендер API.
 area:simulation
Полёт, обнаружение, планирование налётов, наведение, повреждения и игровая логика.
 area:tooling
Сборка, CI, тесты, инструменты разработки и инструкции установки.
 area:ui
HUD, радары, мониторы, меню, текст и интерфейс реплеев.
 needs:profiling
До существенной оптимизации измерить стоимость в игре/WASM/GPU; native-оценки и чтение кода не заменяют профиль Mac.
 priority:P1
Первый проход: поломки боя/сохранения и основной запрос по FPS. Гипотезы производительности сначала профилировать.
 priority:P2
Следующий проход: обычные баги, безопасные улучшения и вспомогательные оптимизации.
 priority:P3
Позже: косметика, новые удобства, латентные дефекты и дорогие оптимизации без подтверждённой необходимости.
 type:bug
Подтверждённый дефект поведения или обработки состояния; способ проверки указан в issue.
 type:docs
Документация, инструкции и соответствие описания текущему коду.
 type:enhancement
Новое поведение, баланс или улучшение удобства.
 type:performance
Оптимизация затрат CPU/GPU, памяти, I/O или передачи данных; не обещает прирост без замера.
No Label area:rendering area:tooling area:ui priority:P2 type:bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
chexit
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: chexit/airdef#83
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?