lzh/picoclaw
1
0
Fork 0
mirror of https://github.com/sipeed/picoclaw.git synced 2026-06-12 18:08:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Address Copilot review feedback for Telegram message chunking

Browse Source 
- Add early return for empty content to avoid silent no-op
- Split raw markdown before HTML conversion so SplitMessage's
  code-fence-aware logic works correctly and HTML tags/entities
  are never broken by mid-tag splitting

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
I Putu Eddy Irawan
2026-03-01 09:40:09 +07:00
parent a6f4274870
commit 2dccee5044
1 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/channels/telegram/telegram.go
+10 -6
View File
@@ -173,14 +173,18 @@ func (c *TelegramChannel) Send(ctx context.Context, msg bus.OutboundMessage) err
return fmt.Errorf("invalid chat ID %s: %w", msg.ChatID, channels.ErrSendFailed)
}
htmlContent := markdownToTelegramHTML(msg.Content)
if msg.Content == "" {
return nil
}
// Split HTML content into chunks that fit within Telegram's message limit.
// Use 4000 to leave headroom for HTML tag overhead beyond the 4096 limit.
chunks := channels.SplitMessage(htmlContent, 4000)
// Split the raw markdown before converting to HTML so that
// SplitMessage's code-fence-aware logic works correctly and
// we never break HTML tags/entities by splitting converted output.
mdChunks := channels.SplitMessage(msg.Content, 4000)
for _, chunk := range chunks {
tgMsg := tu.Message(tu.ID(chatID), chunk)
for _, chunk := range mdChunks {
htmlContent := markdownToTelegramHTML(chunk)
tgMsg := tu.Message(tu.ID(chatID), htmlContent)
tgMsg.ParseMode = telego.ModeHTML
if _, err = c.bot.SendMessage(ctx, tgMsg); err != nil {