Skip to content

feat: add a notification publisher seam - #139

Open
springmin wants to merge 1 commit into
flectar:mainfrom
springmin:notification-publisher
Open

springmin wants to merge 1 commit into
flectar:mainfrom
springmin:notification-publisher

Conversation

@springmin

Copy link
Copy Markdown
Contributor

Adds the host seam for new-mail system notifications, extracted from #85.

  • Drains the existing notification outbox instead of formatting user-visible text in shared code.
  • Applies the Important / All / tabs setting: pending AI routing is deferred, out-of-scope rows are suppressed, and rows are claimed before dispatch and marked delivered afterwards.
  • Hands the host raw sender, subject and thread/message ids, so each platform translates the notification and attaches its own action (slot type, wantAgent, etc.). The default publisher is a no-op.
  • The scope logic has unit tests, which run on the desktop CI target.

No host implementation is included here. The OpenHarmony publisher (translated text, slot type, wantAgent) will follow in #85 once this lands; notifications are currently removed from the OHOS host as requested.

The dispatcher runs on the existing core event listener: once at startup and after each MailNew event.

Sync already records chime-worthy arrivals in the notification outbox. This adds the shared dispatcher that applies the Important/All/tabs scope, waits for pending AI routing, and hands raw sender/subject/thread ids to a host publisher. Hosts format and translate the notification and attach the platform action (slot type, wantAgent); the default publisher keeps the whole path a no-op.
@springmin springmin mentioned this pull request Oct 25, 2026
4 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant