[bug] Загрузка состояния принимает NaN/inf, а сценарий очищает сцену до проверки тела #86

New Issue

2026-10-30T10:29:41Z

chexit commented

2026-10-30 10:29:41 +00:00

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

Архитектурный проход по 76f680eba64b64eb28435e71ce9f22a84b0e5da1.

Что есть сейчас

lib.rs::apply_state разбирает позиционные строки и возвращает (): parse::<f64>() допускает NaN/inf, ошибочные строки молча пропускаются. scenario.rs::apply_scenario проверяет заголовок и мир, затем очищает текущую расстановку и только после этого вызывает такой разбор тела, после чего сообщает успех.

Native characterization test на отдельной копии загрузил:

target NaN inf 0 -100
sam s400 NaN 70 0 0

Результат: объект (NaN, inf, 0) с HP -100 и SAM с NaN-координатой реально добавлены. Это дефект устойчивости к повреждённому/редактированному состоянию; удалённый exploit не заявляется. Для частично повреждённого сценария риск потерять текущую расстановку следует из порядка операций, отдельно host-загрузка не запускалась.

Предложение

Разбирать сначала в LayoutSnapshot/DTO: версия, конечность координат/углов, допустимые HP/роли/боезапас, лимиты сущностей и уникальность устойчивых ID/ссылок.
Проверить весь документ и только затем атомарно заменить сцену. На ошибке сохранить старую сцену и указать строку/поле. Не оставлять половину загрузки.
Развести расстановку, стартовый сценарий и возможный сейв боя. Сейчас seed восстанавливает только sim.rng, тогда как отдельный Sim.seed, tick, next ID и часть истории сессии продолжают жить. Явно определить, что сбрасывается при повторном старте; не обещать полное восстановление боя из layout.
Сохранить чтение старых корректных форматов через явные миграции. Не обязательно вводить тяжёлый сериализатор: важны схема, валидация и commit после parse.

Приёмка

Тесты: NaN/±inf, обрезанная строка, неверный kind, невозможные HP, невалидная ссылка/версия; ни один отказ не меняет активную сцену. Корректный старый сейв читается. Перезапуск сценария с тем же seed имеет определённую и проверяемую семантику независимо от ранее сыгранного боя. Индекс сценариев и payload не должны молча расходиться при неудачной записи.

Общий порядок и зависимости: архитектурная дорожная карта.

🤖 Filed by Codex, AI-агент Владимира (chexit). Архитектурный проход по `76f680eba64b64eb28435e71ce9f22a84b0e5da1`. ### Что есть сейчас `lib.rs::apply_state` разбирает позиционные строки и возвращает `()`: `parse::<f64>()` допускает NaN/inf, ошибочные строки молча пропускаются. `scenario.rs::apply_scenario` проверяет заголовок и мир, затем очищает текущую расстановку и только после этого вызывает такой разбор тела, после чего сообщает успех. Native characterization test на отдельной копии загрузил: ``` target NaN inf 0 -100 sam s400 NaN 70 0 0 ``` Результат: объект `(NaN, inf, 0)` с HP `-100` и SAM с NaN-координатой реально добавлены. Это дефект устойчивости к повреждённому/редактированному состоянию; удалённый exploit не заявляется. Для частично повреждённого сценария риск потерять текущую расстановку следует из порядка операций, отдельно host-загрузка не запускалась. ### Предложение - Разбирать сначала в `LayoutSnapshot`/DTO: версия, конечность координат/углов, допустимые HP/роли/боезапас, лимиты сущностей и уникальность устойчивых ID/ссылок. - Проверить весь документ и только затем атомарно заменить сцену. На ошибке сохранить старую сцену и указать строку/поле. Не оставлять половину загрузки. - Развести **расстановку**, **стартовый сценарий** и возможный **сейв боя**. Сейчас seed восстанавливает только `sim.rng`, тогда как отдельный `Sim.seed`, tick, next ID и часть истории сессии продолжают жить. Явно определить, что сбрасывается при повторном старте; не обещать полное восстановление боя из layout. - Сохранить чтение старых корректных форматов через явные миграции. Не обязательно вводить тяжёлый сериализатор: важны схема, валидация и commit после parse. ### Приёмка Тесты: NaN/±inf, обрезанная строка, неверный kind, невозможные HP, невалидная ссылка/версия; ни один отказ не меняет активную сцену. Корректный старый сейв читается. Перезапуск сценария с тем же seed имеет определённую и проверяемую семантику независимо от ранее сыгранного боя. Индекс сценариев и payload не должны молча расходиться при неудачной записи. Общий порядок и зависимости: [архитектурная дорожная карта](https://gitea.maoniao.team/chexit/airdef/issues/103).

chexit added the area:architecture type:bug priority:P2 area:persistence labels 2026-10-30 10:29:41 +00:00

chexit referenced this issue

2026-10-30 10:31:31 +00:00

[architecture] Дорожная карта AirDef: границы ядра, единая сцена, акустика и развитие механик #103

chexit commented

2026-10-30 11:57:03 +00:00

🤖 Agent: сделано в b43d471706 (запушено в main). Проверено нативными тестами, в игре не запускалось.

🤖 Agent: сделано в https://gitea.maoniao.team/chexit/airdef/commit/b43d47170626e5a403d4c210246cce8fd4b763bc (запушено в main). Проверено нативными тестами, в игре не запускалось.

chexit closed this issue

2026-10-30 11:57:03 +00:00

chexit referenced this issue from a commit

2026-10-30 11:57:03 +00:00

Combat fixes: exact-ray answers tied to their segment and block (#73), no fuse kill through a wall while waiting (#74), seeker needs confirmed sight and stops reading true velocity (#75), combat RNG and ids independent of graphics (#85), saved layouts validated whole before the scene is touched, refused autosave kept (#86)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: chexit/airdef#86