Skip to content

fix(sync): keep status responsive during indexing - #236

Merged
veryCrunchy merged 5 commits into
mainfrom
fix/234-media-indexing-responsiveness
Jul 27, 2026
Merged

fix(sync): keep status responsive during indexing#236
veryCrunchy merged 5 commits into
mainfrom
fix/234-media-indexing-responsiveness

Conversation

@veryCrunchy

Copy link
Copy Markdown
Member

Closes #234

Outcome

  • Keep Sync & offline usable while local media discovery runs.
  • Return the latest durable Transfer Center snapshot without waiting for an active upload or indexing pass.
  • Reconcile interrupted transfers only when the sync engine is idle.
  • Prevent Refresh from starting overlapping MediaStore scans.
  • Avoid rescheduling a removed sync pair from a stale presentation snapshot.

Validation

  • Focused Android engine and atomic-store tests pass.
  • Focused desktop FileOfflineCenter tests pass.
  • Repository hygiene and changelog validation pass.
  • An independent concurrency review was completed and its scheduling-race and scan-coalescing findings were fixed before opening this PR.

@veryCrunchy veryCrunchy added the priority:P0 Required for a trustworthy mobile replacement or prevents data loss label Jul 27, 2026
@veryCrunchy veryCrunchy added this to the 0.2 Safe sync and media milestone Jul 27, 2026
@veryCrunchy veryCrunchy added area:media Media backup, MediaStore, storage reclaim, and sharing area:files-sync Files, offline storage, transfer, and synchronization area:ux Navigation, accessibility, responsive design, and interaction platform:android Android-specific implementation or validation type:performance Startup, rendering, network, memory, or cache performance type:bug labels Jul 27, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 54f245de48

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread changes/unreleased/234-media-indexing-responsiveness.md

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 545aa3ead9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread androidApp/src/main/kotlin/dev/obiente/nextcloudnative/AndroidFileSyncEngine.kt Outdated
Comment thread androidApp/src/main/kotlin/dev/obiente/nextcloudnative/AndroidFileSyncEngine.kt Outdated
@veryCrunchy
veryCrunchy merged commit 0298d77 into main Jul 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:files-sync Files, offline storage, transfer, and synchronization area:media Media backup, MediaStore, storage reclaim, and sharing area:ux Navigation, accessibility, responsive design, and interaction platform:android Android-specific implementation or validation priority:P0 Required for a trustworthy mobile replacement or prevents data loss type:bug type:performance Startup, rendering, network, memory, or cache performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MEDIA-023] Keep sync settings and Transfer Center responsive during media indexing

1 participant