KrustyPlanet/hermes-agent
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(telegram): guard _post_connect_task access for object.__new__ test pattern

Browse source 
disconnect() reads self._post_connect_task, but several tests build a bare
TelegramAdapter via object.__new__() without calling __init__ (which sets the
attr). Use getattr(..., None) so disconnect() works on those instances too
(pitfall #17).
This commit is contained in:
teknium1 2026-07-01 03:06:16 -07:00 • committed by Teknium
parent 3362bdb4e5
commit 69f08c2eb5
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
plugins/platforms/telegram/adapter.py
View file

@ -3056,8 +3056,9 @@ class TelegramAdapter(BasePlatformAdapter):
# Cancel deferred post-connect housekeeping (command-menu / DM-topic /
# status-indicator Bot API calls) so it cannot fire into a half-torn-down
# bot client (#46298).
post_connect_task = self._post_connect_task
# bot client (#46298). getattr guards the object.__new__ test pattern
# where __init__ (which sets this attr) is never called.
post_connect_task = getattr(self, "_post_connect_task", None)
if post_connect_task and not post_connect_task.done():
post_connect_task.cancel()
await asyncio.gather(post_connect_task, return_exceptions=True)