fix(mail): prefer special-use folder when resolving move-by-role destination #92

Merged

JMR-dev merged 2 commits from fix-move-by-role-specialuse into main

2026-07-02 03:20:47 +00:00

Author	SHA1	Message	Date
Jason Ross	bb3dcd8217	Merge branch 'main' into fix-move-by-role-specialuse	2026-07-01 22:07:18 -05:00
JMR-devandClaude Fable 5	a6436719b1	fix(mail): prefer special-use folder when resolving move-by-role destination resolveRoleFolder().pick() chose the destination for archive/reportSpam/ trash as the first selectable folder with the matching role, in server LIST order. A provider's built-in folder (role via an RFC 6154 attribute, e.g. [Gmail]/Spam via \Junk) and a same-named user folder (role via roleFromDisplayName) can share a role, so the winner depended on which one the server happened to LIST first — silently misrouting mail past the provider's junk training, retention, and auto-purge. Prefer the server-advertised special-use folder among same-role matches: maxByOrNull { it.specialUse } picks a specialUse=true folder over name-derived ones, and, because maxByOrNull returns the first max, keeps the existing LIST-order behavior when no special-use folder exists. Closes #58 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-01 21:48:46 -05:00

Author

SHA1

Message

Date

Jason Ross

bb3dcd8217

Merge branch 'main' into fix-move-by-role-specialuse

2026-07-01 22:07:18 -05:00

JMR-devandClaude Fable 5

a6436719b1

fix(mail): prefer special-use folder when resolving move-by-role destination

resolveRoleFolder().pick() chose the destination for archive/reportSpam/
trash as the first selectable folder with the matching role, in server
LIST order. A provider's built-in folder (role via an RFC 6154 attribute,
e.g. [Gmail]/Spam via \Junk) and a same-named user folder (role via
roleFromDisplayName) can share a role, so the winner depended on which
one the server happened to LIST first — silently misrouting mail past
the provider's junk training, retention, and auto-purge.

Prefer the server-advertised special-use folder among same-role matches:
maxByOrNull { it.specialUse } picks a specialUse=true folder over
name-derived ones, and, because maxByOrNull returns the first max, keeps
the existing LIST-order behavior when no special-use folder exists.

Closes #58

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

2026-07-01 21:48:46 -05:00