Skip to content

fix(daemonclient): register IMAP/POP3#1668

Open
2005czq wants to merge 1 commit into
floatpane:release/v1from
2005czq:fix/daemonclient-direct-backends
Open

fix(daemonclient): register IMAP/POP3#1668
2005czq wants to merge 1 commit into
floatpane:release/v1from
2005czq:fix/daemonclient-direct-backends

Conversation

@2005czq

@2005czq 2005czq commented Jul 9, 2026

Copy link
Copy Markdown

What?

Register IMAP and POP3 backends in daemonclient's directService path.

Why?

daemonclient already imports JMAP and Maildir for directService backend registration. IMAP and POP3 are supported by the app as well, but are not registered locally in daemonclient.

This keeps the direct fallback backend set consistent with the supported protocols.

Copilot AI review requested due to automatic review settings July 9, 2026 15:41
@2005czq 2005czq requested a review from a team as a code owner July 9, 2026 15:41
@floatpanebot floatpanebot added area/daemon Daemon / RPC bug Something isn't working target/v1 PR targets the release/v1 branch directly size/XS Diff: 0–10 lines labels Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Registers additional backend implementations (IMAP and POP3) for daemonclient’s in-process directService fallback mode so that local/direct operation supports the same protocols as the rest of the app.

Changes:

  • Add blank imports for backend/imap and backend/pop3 so their init() registration runs in daemonclient.
  • Keep the direct-mode backend registry consistent with supported protocols (IMAP/JMAP/Maildir/POP3).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Daemon / RPC bug Something isn't working size/XS Diff: 0–10 lines target/v1 PR targets the release/v1 branch directly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants