Skip to content

Preserve update logs in precheck update info - #535

Draft
JusterZhu with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-update-log-retrieval
Draft

Preserve update logs in precheck update info#535
JusterZhu with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-update-log-retrieval

Conversation

Copilot AI commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

AddListenerUpdatePrecheck received update version metadata without UpdateLog, even when the server response included updateLog. The field was lost while converting VersionEntry into DownloadAsset, then reconstructing VersionEntry for event dispatch.

  • Carry release notes through the download model

    • Added DownloadAsset.UpdateLog as an init-only property.
    • Preserves existing DownloadAsset constructor compatibility.
  • Preserve mappings across the update pipeline

    • HttpDownloadSource.MapVersionEntry now copies VersionEntry.UpdateLog.
    • ClientStrategy now copies DownloadAsset.UpdateLog when building UpdateInfoEventArgs.
  • Regression coverage

    • Covers VersionEntry → DownloadAsset mapping.
    • Covers ClientStrategy.ExecuteAsync precheck callback receiving UpdateLog.
var versionInfos = downloadPlan.Assets.Select(a => new VersionEntry
{
    Version = a.Version,
    UpdateLog = a.UpdateLog,
    // existing fields...
}).ToList();

Co-authored-by: JusterZhu <11714536+JusterZhu@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue with updating logs in AddListenerUpdatePrecheck Preserve update logs in precheck update info Sep 6, 2026
Copilot AI requested a review from JusterZhu September 6, 2026 14:22
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.

[ERROR] AddListenerUpdatePrecheck中updateInfo拿不到UpdateLog,UpdateLog为空

2 participants