Skip to content

Conversation

@humbertoyusta
Copy link
Contributor

@humbertoyusta humbertoyusta commented Dec 26, 2025

I did this once, then we reverted it because we weren't doing subs restart on purpose, but we can still have subscription restart without restart of all other things (there is even a bc.thread_tracker.clear() with a comment about it), and it was happening somehow in scenarios some times

There are other less likely to happen cases that could also send a toolcall more than once, like a toolcall appearing at the same time than the subscription, so it could be fetched from database, then the news from the database being processed and send to redis, then subs sends it again in redis, which could cause tool call to be sent twice, so it is probably better to deduplicate in bot still

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.

2 participants