feat: File Duplicate Bulk Actions (fixes #92) #102

Merged

ValleyGeek merged 3 commits from 0.1.34/issue-92-bulk-dupe-delete into main 2026-06-19 07:29:12 +00:00

Conversation 0 Commits 3 Files changed 9 +269 -2

Claude commented 2026-06-19 07:22:11 +00:00

Member

Copy link

Summary

• Adds pick_keep_path_by_strategy / bulk_keep_delete_paths to core/dupe_detector.py, implementing the six keep rules from #92 (Newest Files default, Oldest Files, Shortest/Longest File Name, Shortest/Longest Path).
• Adds a File Duplicate Bulk Actions button to the bottom of the Duplicates toolbox that opens BulkDupeActionsDialog (radio list + Cancel/Delete), per the proposed UI in the issue.
• On Delete, applies the chosen strategy across every F-DUPE_# group at once, staging losers with the existing non-destructive DELETE tag (same mechanism as Keep Best) as a single undoable batch.
• Version bump to 0.1.35 (patch), CHANGELOG entry, and USER_GUIDE.md update for the new button.

Threading notes: all new code (dialog, handler, core helpers) runs on the main thread / is pure-data; no worker or signal changes.

Fixes #92.

Test plan

• ruff check . && ruff format --check .
• pytest — 354 passed, 1 skipped (one unrelated transient segfault reproduced and cleared on rerun — not tied to this branch's changes, confirmed by isolating tests/test_pr13_config.py)
• ./scripts/check-version-bump.sh — VERSION updated 0.1.34 → 0.1.35
• Fresh editable install + scripts/get-version.py --canonical confirms PEP 440 normalization
• Manual click-through of the new dialog in a running app (not done in this session)

## Summary - Adds `pick_keep_path_by_strategy` / `bulk_keep_delete_paths` to `core/dupe_detector.py`, implementing the six keep rules from #92 (Newest Files default, Oldest Files, Shortest/Longest File Name, Shortest/Longest Path). - Adds a **File Duplicate Bulk Actions** button to the bottom of the Duplicates toolbox that opens `BulkDupeActionsDialog` (radio list + Cancel/Delete), per the proposed UI in the issue. - On Delete, applies the chosen strategy across every `F-DUPE_#` group at once, staging losers with the existing non-destructive `DELETE` tag (same mechanism as Keep Best) as a single undoable batch. - Version bump to `0.1.35` (patch), CHANGELOG entry, and USER_GUIDE.md update for the new button. Threading notes: all new code (dialog, handler, core helpers) runs on the main thread / is pure-data; no worker or signal changes. Fixes #92. ## Test plan - [x] `ruff check . && ruff format --check .` - [x] `pytest` — 354 passed, 1 skipped (one unrelated transient segfault reproduced and cleared on rerun — not tied to this branch's changes, confirmed by isolating `tests/test_pr13_config.py`) - [x] `./scripts/check-version-bump.sh` — VERSION updated 0.1.34 → 0.1.35 - [x] Fresh editable install + `scripts/get-version.py --canonical` confirms PEP 440 normalization - [ ] Manual click-through of the new dialog in a running app (not done in this session)

Claude added 3 commits 2026-06-19 07:22:11 +00:00

feat(dupes): add bulk keep-strategy helpers for duplicate groups ... e33f71e3b1

Adds pick_keep_path_by_strategy/bulk_keep_delete_paths to core/dupe_detector.py
covering the six keep rules from issue #92 (newest/oldest file, shortest/longest
filename, shortest/longest path), with unit test coverage.

Threading notes: pure functions on plain dicts, no Qt imports, no worker changes.

feat(ui): add File Duplicate Bulk Actions dialog and button ... 47b69fe88b

Adds a "File Duplicate Bulk Actions" button to the bottom of the Duplicates
toolbox (DupeContainer) that opens a dialog to pick one of the six keep
strategies and apply it across every F-DUPE_# group at once, staging losers
with the existing non-destructive DELETE tag (same mechanism as Keep Best).

Threading notes: dialog and handler run entirely on the main thread; no new
worker code, no QWidget/model access from off-thread code.

docs: PR-prep for issue 92 — bump to 0.1.35 ... 0348a538c6

Releases File Duplicate Bulk Actions: bump VERSION to the next canonical
release, add the dated CHANGELOG entry, and document the new toolbox button
in the bundled USER_GUIDE.md.

Claude referenced this pull request 2026-06-19 07:22:19 +00:00

[Feature]: More Options for Duplicate File Deletion #92

ValleyGeek merged commit 094e248533 into main 2026-06-19 07:29:12 +00:00

ValleyGeek deleted branch 0.1.34/issue-92-bulk-dupe-delete 2026-06-19 07:29:13 +00:00

ValleyGeek referenced this pull request from a commit 2026-06-19 07:29:15 +00:00

feat: File Duplicate Bulk Actions (fixes #92) (#102)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Kind/Bug

Something isn't working as expected

  

Kind/Feature

New capability or user-requested feature

  

Priority/High

High priority

  

Priority/Medium

Medium priority

  

Reviewed/Confirmed

Bug or feature report confirmed valid

  

Compat/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No labels

Kind/Bug

Kind/Feature

Priority/High

Priority/Medium

Reviewed/Confirmed

Compat/Breaking

Kind/Bug

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Security

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

ai-collab/bulk-image-organizer!102

No description provided.