Skip to content

feat(telegram): add outbound notification policy#37

Merged
tuchg merged 2 commits into
mainfrom
codex/issue-35-telegram-notification-policy
Jun 1, 2026
Merged

feat(telegram): add outbound notification policy#37
tuchg merged 2 commits into
mainfrom
codex/issue-35-telegram-notification-policy

Conversation

@tuchg

@tuchg tuchg commented Jun 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a shared channel NotificationPolicy for text, fallback files, and agent attachments.
  • Map silent text/file/photo/video/document Telegram sends to disable_notification(true).
  • Mark live progress/drafts/history replay silent, then send final assistant output as a new notify-eligible message and delete the silent preview after the final send succeeds.
  • Keep user-action prompts and final attachments notify-by-default.
  • Record the UX/boundary decision under docs/decisions/.

Closes #35.

Verification

  • cargo +nightly test -Zbuild-dir-new-layout -p lucarne-channel
  • cargo +nightly test -Zbuild-dir-new-layout -p lucarne-telegram --lib turn::tests::
  • cargo +nightly test -Zbuild-dir-new-layout -p lucarne-telegram --lib
  • cargo +nightly check -Zbuild-dir-new-layout --workspace --all-features

@tuchg tuchg merged commit 8f81e7d into main Jun 1, 2026
6 checks passed
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.

enhancement: make Telegram push notifications opt-in for final/user-visible messages

1 participant