Backup: заменить JSON-клонирование снимка восстановления на типизированную копию

Open
  • Is something wrong?
#393 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
48/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Active
Tech stack
csharp
Domain
backend

Research direction

Start at ProfileStore.RestoreBackupAsync and BackupArchive.Clone(data), then inspect the backup models and the existing Merge/Replace, save failure, cancellation, and rollback tests. Implement an explicit copy that preserves every model field while giving mutable collections and nested models independent containers. Done means the listed isolation and restore tests pass, the full backup clone path avoids JSON serialization, and a repeatable benchmark shows lower allocations on both datasets.

Written by the indexing model from the issue text.

Description

Проблема

ProfileStore.RestoreBackupAsync вызывает BackupArchive.Clone(data). Клонирование сериализует весь BackupData в JSON и сразу десериализует обратно. Это создаёт дополнительный plaintext-буфер с секретами и повторные строки, включая исходные конфигурации. В Clone временный JSON-массив явно не обнуляется.

Измерения

Изолированный синтетический бенчмарк, Windows x64, .NET 10.0.7 Release, tiered compilation выключена, два прогрева и медиана семи повторов:

  • 1000 профилей: JSON-клон — 5,20 мс и 4,03 МиБ аллокаций; типизированный C#-прототип — 0,34 мс и 0,76 МиБ.
  • 200 профилей с отдельными строками конфигураций по 128 КиБ; JSON архива около 25,1 МиБ: JSON-клон — 21,92 мс и 138,95 МиБ; прототип — 0,15 мс и 0,17 МиБ.

Это суммарные управляемые аллокации за операцию, не пик RAM и не время полного восстановления. В прототипе создаются независимые изменяемые контейнеры, но переиспользуются неизменяемые строки. Проверены равенство сериализованного результата и отсутствие общей ссылки на словарь секретов; это не заменяет полноценные тесты всех моделей.

Решение

Заменить JSON round-trip для снимка восстановления на явное копирование в C#. Копировать все изменяемые коллекции и вложенные модели; строки разрешено переиспользовать. Проверку данных, Merge/Replace, откат секретов и порядок сохранения оставить семантически прежними. Остальные вызовы общего Clone<T> менять только при необходимости для этого пути.

Готово, когда

  • Изменение исходных или скопированных словарей, списков, профилей, подписок, цепочек и настроек не меняет другой снимок.
  • Сохраняются все поля моделей, включая идентификаторы, ссылки на подписки, параметры маршрутизации и секреты инструментов.
  • На пути клонирования полного backup отсутствует промежуточная JSON-сериализация секретов.
  • Изолированные тесты покрывают Merge/Replace, ошибки сохранения, отмену и rollback; существующие гарантии восстановления сохраняются.
  • Повторяемый бенчмарк подтверждает снижение аллокаций на обоих описанных наборах без требования конкретного времени на другом оборудовании.

Связано с #18 и #181. Смена языка, формата .bydlobackup и криптографических параметров не требуется. Все проверки — с BYDLO_SAFE_DEV=1, фейковым хранилищем, без запуска движков и системного сетевого воздействия.

Dominant language
C#
Stars
5
Forks
1
Avg merge
30m
Merged PRs (30d)
171

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from endercodezz/BYDLO

All issues in endercodezz/BYDLO

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.