Share general review and documentation rules

[onechat] / telegram.go

diff --git a/telegram.go b/telegram.go
index 85b0f44d1ed368cfcb340f0aa7f0edcd9fa3e9fc..78a1c7af9079c22982a02ec19bd2d377c0f71fde 100644 (file)
--- a/telegram.go
+++ b/telegram.go
@@ -313,11 +313,7 @@ poll:
                                        continue poll
                                }
 
-                               if err != nil {
-                                       return fmt.Errorf("handle Telegram update: %w", err)
-                               }
-
-                               return nil
+                               return fmt.Errorf("handle Telegram update: %w", err)
                        }
 
                        imageOutage.End()
@@ -589,13 +585,13 @@ func (b *TelegramBot) sendOutage() *TelegramOutage {
 }
 
 // Notify queues event for the chat. It never blocks, so a slow network does not
-// stop the agent. The WhatsApp batches and the runs of the schedules do not go
-// to the chat: the reply of the agent tells the user what needs the user.
+// stop the agent. External events, WhatsApp batches, and schedule runs do not
+// go to the chat: the reply of the agent tells the user what needs the user.
 func (b *TelegramBot) Notify(event Event) {
        switch event.Kind {
        case "busy", "idle", "reply", "error", "notice", "subagent":
        case "work":
-               if event.Source == whatsappSource || event.Source == scheduleSource {
+               if event.Source == whatsappSource || event.Source == scheduleSource || event.Source == notifySource {
                        return
                }
        case "user":