Skip to content

fix(composer): real background download, window-independent downloads, window geometry log - #4

Merged
Hongwei-MB merged 1 commit into
mainfrom
fix/background-download-and-window-log
Nov 3, 2026
Merged

Hongwei-MB merged 1 commit into
mainfrom
fix/background-download-and-window-log

Conversation

@Hongwei-MB

Copy link
Copy Markdown
Collaborator

Background download banner no longer returns; footer shows progress. Downloads run independent of the composer window (sequential chain in Core, joined by later summons, closed composers do nothing at the end). Adds composer window geometry logging (PROBLEM= lines) and a page-side clipped diagnostic. Local gate: ./build.sh Test green, 0 warnings; independent review findings fixed.

…, window geometry log

Background download
- "Download in background" is its own banner action: the banner no longer comes
  back with the next progress update, and the composer footer shows the overall
  progress with a bar until the models are installed or the download fails.

Downloads belong to the application
- ModelDownloadCoordinatorExtensions.StartAllAsync downloads models one after
  the other with no window involved. DownloadModelsAsync works off the missing
  models until none is left (what is needed can grow while downloading), ends a
  round that installed nothing as a failure, and does nothing for a composer
  that was closed meanwhile (it used to be able to start the microphone).
- A summon that finds a download running joins it in the background.
- The coordinator logs the start and end of every model download.

Window geometry diagnostics
- ComposerGeometryLog writes every size/position decision with a summon number
  and the whole geometry (requested and system size, position, cap, working
  area, scale); bursts are throttled. Problems are logged as PROBLEM=...:
  content taller than the cap, size changed by the system, window outside the
  working area.
- The page reports when its viewport stays smaller than its content (clipped
  intent), the symptom of a window that shows only part of the composer.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Hongwei-MB
Hongwei-MB merged commit 62e7d9c into main Nov 3, 2026
1 check passed
@Hongwei-MB
Hongwei-MB deleted the fix/background-download-and-window-log branch November 3, 2026 14:10
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.

1 participant