PR7: Metadata enrichment scan respects UI sort order (DR-014) #17

New issue

Closed

opened 2026-06-04 18:08:10 +00:00 by Grok · 1 comment

Grok commented

2026-06-04 18:08:10 +00:00

Member

Context

DR-014 (issue #15) specifies:

Discovery scan stays in filesystem/alphabetical order for fastest placeholder population.
Metadata enrichment scan processes files in the current UI sort order and re-orders the remaining queue if the user changes sort mid-scan.

Current behavior

Both discovery and metadata workers iterate in os.scandir presentation order. PR5 grid uses direct ImageListModel without sort UI yet.

Scope

Define default UI sort (likely filename) before proxy lands in PR7
Metadata ScanWorker / scanner queue ordered by active sort key
On sort change during scan: reorder pending metadata work only (do not restart discovery)
Emit dataChanged/proxy invalidate as needed when sort affects visible order

Dependencies

PR7 introduces QSortFilterProxyModel + filmstrip sync (DR-014)
Sort controls may expand in PR8 filters

References

docs/DECISIONS.md DR-014
docs/DESIGN.md PR7, decision gate #6

Blocks

Non-blocking for PR5 merge; should land with or before PR7 viewer/filmstrip.

## Context DR-014 (issue #15) specifies: - Discovery scan stays in filesystem/alphabetical order for fastest placeholder population. - **Metadata enrichment scan** processes files in the **current UI sort order** and re-orders the remaining queue if the user changes sort mid-scan. ## Current behavior Both discovery and metadata workers iterate in `os.scandir` presentation order. PR5 grid uses direct `ImageListModel` without sort UI yet. ## Scope - Define default UI sort (likely filename) before proxy lands in PR7 - Metadata `ScanWorker` / scanner queue ordered by active sort key - On sort change during scan: reorder pending metadata work only (do not restart discovery) - Emit `dataChanged`/proxy `invalidate` as needed when sort affects visible order ## Dependencies - PR7 introduces `QSortFilterProxyModel` + filmstrip sync (DR-014) - Sort controls may expand in PR8 filters ## References - `docs/DECISIONS.md` DR-014 - `docs/DESIGN.md` PR7, decision gate #6 ## Blocks Non-blocking for PR5 merge; should land with or before PR7 viewer/filmstrip.

Grok referenced this issue

2026-06-04 18:08:15 +00:00

PR5: Thumbnail grid foundation (IconMode + delegate) #16

Grok referenced this issue

2026-06-04 19:23:52 +00:00

PR5: Thumbnail grid foundation (IconMode + delegate) #16

Grok commented

2026-06-05 22:48:13 +00:00

Author

Member

PR7 complete — this issue stays open

PR7 (merged in commit on main) delivered the #19 UI shell, including:

Sort By / Sort Order chrome on the sub-toolbar (apply is still a stub)
ImageFilterProxy on the grid for Trash (DELETE-only) and tag-filter stub
Justified grid column spacing across the viewport width

Not in PR7 scope (actionable here): metadata enrichment scan queue ordered by active UI sort + mid-scan reorder per DR-014. Scan workers still use discovery/filesystem order for metadata batches.

Plan: implement sort-aware metadata queue when sort apply and/or full sort proxy land (PR8/PR9). Leaving #17 open until that work is done.

## PR7 complete — this issue stays open PR7 (merged in commit on `main`) delivered the #19 UI shell, including: - Sort By / Sort Order chrome on the sub-toolbar (apply is still a stub) - `ImageFilterProxy` on the grid for Trash (DELETE-only) and tag-filter stub - Justified grid column spacing across the viewport width **Not in PR7 scope (actionable here):** metadata enrichment scan queue ordered by active UI sort + mid-scan reorder per DR-014. Scan workers still use discovery/filesystem order for metadata batches. **Plan:** implement sort-aware metadata queue when sort apply and/or full sort proxy land (**PR8/PR9**). Leaving #17 open until that work is done.

Grok referenced this issue from a commit

2026-06-05 22:48:31 +00:00

feat(ui): complete PR7 #19 shell, workbench switching, and grid refactor

Grok referenced this issue from a commit

2026-06-06 02:37:20 +00:00

PR9 RC: UI sorting and issue #17 metadata scan queue ordering