chore: simplify mail bridge life cycle - #1505
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This PR changes 737 lines. If it contains multiple concerns, consider splitting it. Large PRs are fine when the changes are mechanical, intentionally grouped, or belong together as part of the same feature. Please make sure the PR description gives reviewers enough context about what matters most to review. |
What is Changed / Added
Replaces the previous Mail Bridge manager/state lifecycle with focused, functional services.
Desktop now separates:
readyresponse, with a 30-second timeout and child exit/error handling.The new lifecycle service also coalesces concurrent start requests: a second activation waits for the first startup operation rather than starting another Bridge process.
Behaviour
start_sessiononly after the Bridge connects.ready.