Skip to content

fix(telegram): stop typing for topic replies#2776

Open
bogdanovich wants to merge 1 commit intosipeed:mainfrom
bogdanovich:fix/telegram-topic-typing-cleanup
Open

fix(telegram): stop typing for topic replies#2776
bogdanovich wants to merge 1 commit intosipeed:mainfrom
bogdanovich:fix/telegram-topic-typing-cleanup

Conversation

@bogdanovich
Copy link
Copy Markdown

Summary

  • add a generic channel-level outbound chat resolver used by manager cleanup paths
  • stop typing/reaction state for both raw and resolved chat IDs before outbound delivery
  • use Telegram's resolved chat/topic key for forum-topic cleanup while preserving existing raw-chat behavior

Why

Telegram forum topic inbound handling starts typing under a composite chat ID such as -100123/42, but final outbound messages may carry the raw chat ID with TopicID in context. Manager cleanup previously only looked up the raw chat ID, so the typing refresh goroutine could keep sending sendChatAction until the fallback TTL expired after the answer was already delivered.

This keeps target resolution channel-specific and generic rather than tying typing cleanup to tool-feedback-only naming.

Tests

  • go test ./pkg/channels
  • go test ./pkg/agent ./pkg/bus

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