[bug] Ошибка сохранения снимает dirty и отключает повторную попытку #6

New Issue

2026-10-29T11:44:03Z

chexit commented

2026-10-29 11:44:03 +00:00

Проверено на 90379f6.

Статический разбор: если storage::set() возвращает ошибку, save() пишет предупреждение, но затем безусловно выполняет self.dirty = false. При следующем тике контрольная сумма уже совпадает с health_mark, поэтому без новых изменений повторной попытки не будет. Изменения могут потеряться при выходе/переходе в другой мир.

Воспроизведение для будущего host-теста: хранилище возвращает Err при первой записи и Ok при второй; после первой ошибки dirty должен остаться true, а следующий запланированный autosave должен повторить запись.

Предложение: снимать dirty только после успешной записи. Для повторов добавить ограничение частоты/backoff, чтобы не забить лог и не вызывать failing storage на каждом тике.

Код:

		airdef/airdef-mod/src/lib.rs
		Line 520 in 90379f6
	
				        if let Err(e) = storage::set(&self.state_key(), text.as_bytes()) {

Проверено на `90379f6`. Статический разбор: если `storage::set()` возвращает ошибку, `save()` пишет предупреждение, но затем безусловно выполняет `self.dirty = false`. При следующем тике контрольная сумма уже совпадает с `health_mark`, поэтому без новых изменений повторной попытки не будет. Изменения могут потеряться при выходе/переходе в другой мир. Воспроизведение для будущего host-теста: хранилище возвращает Err при первой записи и Ok при второй; после первой ошибки `dirty` должен остаться true, а следующий запланированный autosave должен повторить запись. Предложение: снимать `dirty` только после успешной записи. Для повторов добавить ограничение частоты/backoff, чтобы не забить лог и не вызывать failing storage на каждом тике. Код: https://gitea.maoniao.team/chexit/airdef/src/commit/90379f6/airdef-mod/src/lib.rs#L520

chexit added the area:persistence priority:P1 type:bug labels 2026-10-29 12:56:53 +00:00

chexit referenced this issue from a commit

2026-10-29 13:28:10 +00:00

P1 issues: ground queries keep room for unknown land (#1), a destroyed truck frees its job (#2), trucks follow a system that moved (#3), repair cancels cook-offs (#5), a failed save is retried and autosaves are paced (#6, #7), a combined system reloads when either weapon is spent (#20), munitions go down to the land left under a destroyed target (#34); regression tests from the issues

chexit commented

2026-10-29 13:29:21 +00:00