Epic: Dedupe addon code review (2026-06-20) — B+ tracking

[jasincanada]

Overall grade: B+

Tracking issue for addon code review (dedupe integration + list filter/sort). Ship-worthy for small mailboxes; not A+ until scale, search integration, and remove confirmations are addressed.

Critical

• [Dedupe] Browser loads entire mailbox into memory on scan (149k+ scale risk) #77 — Browser loads entire mailbox on dedupe scan (149k+ scale)
• [Dedupe] Account-wide remove auto-starts without confirmation on /dedupe #78 — Account-wide remove auto-starts without confirmation
• [Dedupe] Body match criterion can trigger massive body fetch #79 — Body match criterion massive body fetch

High

• [List] searchEmails/advancedSearch ignore list filter and sort #80 — searchEmails/advancedSearch ignore list filter/sort
• [List] refreshCurrentMailbox bypasses list filter during active search #81 — refreshCurrentMailbox bypasses list filter during search
• [List] Subject/sender sort only applies to loaded page of threads #82 — Subject/sender sort only on loaded page
• [Dedupe] getEmailsForDedupe skips namespaceMailboxIds for shared folders #83 — getEmailsForDedupe missing shared-folder namespace
• [Dedupe] Folder scan creates dupes/ subfolder (prepareDupesFolder) #84 — Folder scan creates dupes/ subfolder
• [Dedupe] No cancellation for long-running browser scan/remove #85 — No cancellation for long dedupe operations

Medium

• [Dedupe] List header duplicate count can be stale after remove/scan #86 — Stale duplicate count in list header
• [Dedupe] GroupCard Copy icon has no copy-to-clipboard handler #87 — GroupCard Copy icon non-functional
• [Dedupe] Operations page may not re-run on same URL without reset #88 — Operations page same-URL re-entry
• [List] mergeJMAPFilters may produce duplicate inMailbox conditions #89 — mergeJMAPFilters duplicate inMailbox
• [Dedupe] Account scan processes folders sequentially (slow wall-clock) #90 — Account scan sequential folders
• [i18n] Pass 12 list filter/sort strings not translated (en-only hints) #91 — i18n pass 12 incomplete
• [Dedupe] Settings account Run skips scanned/unscanned confirmation #92 — Settings Run skips confirmation

Low / docs / testing

• [Testing] Add unit/integration tests for dedupe addon modules #93 — Addon test coverage gaps
• [a11y] List filter/sort dropdowns lack keyboard roving focus #94 — List dropdown a11y
• [Dedupe] sidebar dedupeBusy state appears unused after /dedupe redirect #95 — sidebar dedupeBusy dead state
• [Docs] Add human QA checklist to repo (TESTER_TASKS.md) #96 — Commit TESTER_TASKS.md to repo

What's solid (no issue needed)

• Addon does not write keywords/$seen
• JMAP ID resolution for dupes/ creation
• Batch moves with verification
• List controls UX (pass 12)

[jasincanada]

Implementation landed locally and on fork:

• PR: feat: mail dedupe addon + list filter/sort (passes 8–12) #98 (feat: mail dedupe addon + list filter/sort, passes 8–12)
• Commit: jasincanada@1b9453e
• QA doc: docs/TESTER_TASKS.md (closes [Docs] Add human QA checklist to repo (TESTER_TASKS.md) #96 once merged)

Child issues #77–#96 remain open for follow-up work from the B+ review.