Skip to content

Batch 12: StorageManager instance + CancellationToken贯穿#384

Merged
JusterZhu merged 1 commit into
masterfrom
batch-12-storage-ct
May 24, 2026
Merged

Batch 12: StorageManager instance + CancellationToken贯穿#384
JusterZhu merged 1 commit into
masterfrom
batch-12-storage-ct

Conversation

@JusterZhu
Copy link
Copy Markdown
Collaborator

Summary

Final batch — adds instance-based storage management and propagates cancellation tokens.

Changes

CancellationToken propagation

  • VersionService.Validate() / Report() now accept CancellationToken
  • HttpDownloadSource passes CancellationToken to VersionService calls

IStorageManager + DefaultStorageManager

  • Instance interface: BackupAsync, RestoreAsync, CleanBackupAsync, ListBackups
  • Default implementation wrapping static StorageManager
  • Ready for DI registration

Build

  • dotnet build src/c#/GeneralUpdate.slnx — 0 errors

Closes #383

- Add CancellationToken parameter to VersionService.Validate/Report static API
- Propagate CancellationToken through HttpDownloadSource
- Create IStorageManager interface with BackupAsync/RestoreAsync/CleanBackupAsync
- Create DefaultStorageManager implementation wrapping static StorageManager

Closes #383
Copilot AI review requested due to automatic review settings May 24, 2026 11:37
@JusterZhu JusterZhu merged commit 76b67b7 into master May 24, 2026
1 check failed
@JusterZhu JusterZhu deleted the batch-12-storage-ct branch May 24, 2026 11:38
@JusterZhu JusterZhu review requested due to automatic review settings May 24, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Batch 12: StorageManager instance + CancellationToken贯穿

1 participant