Skip to content

fix: allow users to cancel mlx model download#7502

Merged
louis-jan merged 1 commit into
release/v0.7.7from
fix/cancel-mlx-model-download
Feb 10, 2026
Merged

fix: allow users to cancel mlx model download#7502
louis-jan merged 1 commit into
release/v0.7.7from
fix/cancel-mlx-model-download

Conversation

@louis-jan

Copy link
Copy Markdown
Contributor

Describe Your Changes

  • This PR fixes download cancel issue with new backend-supported model

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Copilot AI review requested due to automatic review settings February 10, 2026 12:49
@github-actions

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to fix cancellation of MLX model downloads by wiring MLX downloads into the same cancellation flows as other providers and ensuring the MLX extension can cancel active download tasks.

Changes:

  • Update model download cancellation to attempt aborting both llamacpp and mlx engines.
  • Refactor MLX model download initiation to use EngineManager directly and adjust which HF repo files are queued.
  • Add MLX download task IDs + implement abortImport() cancellation and cleanup in the MLX extension.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
web-app/src/services/models/default.ts Attempts to abort downloads across both llamacpp and mlx engines and emits a stopped event.
web-app/src/containers/MlxModelDownloadAction.tsx Adjusts MLX download flow (engine lookup, file selection, import invocation).
web-app/src/containers/DownloadManegement.tsx Extends direct task cancellation condition to include mlx-prefixed IDs.
extensions/mlx-extension/src/index.ts Adds MLX download task ID creation, implements abort/cancel behavior, and cleans up model folder on abort.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread web-app/src/services/models/default.ts
Comment thread web-app/src/services/models/default.ts
Comment thread web-app/src/containers/MlxModelDownloadAction.tsx
Comment thread web-app/src/containers/MlxModelDownloadAction.tsx
Comment thread extensions/mlx-extension/src/index.ts
@louis-jan louis-jan merged commit ea3f876 into release/v0.7.7 Feb 10, 2026
20 checks passed
@louis-jan louis-jan deleted the fix/cancel-mlx-model-download branch February 10, 2026 13:39
@github-project-automation github-project-automation Bot moved this to QA in Jan Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

3 participants