Skip to content

Bot: replace static "Thinking..." with rotating typing indicator#63

Closed
Dhereal1 wants to merge 25 commits intoHyperlinksSpace:mainfrom
Dhereal1:main
Closed

Bot: replace static "Thinking..." with rotating typing indicator#63
Dhereal1 wants to merge 25 commits intoHyperlinksSpace:mainfrom
Dhereal1:main

Conversation

@Dhereal1
Copy link
Contributor

Replace static "Thinking..." with rotating typing indicator

This PR implements the animated placeholder for the bot typing state.

Instead of a static "Thinking...", the bot now cycles:

Thinking.
Thinking..
Thinking...

The animation runs until the first AI response is ready, then immediately stops so the real response replaces the placeholder without delay.

Changes

  • Implemented animated typing indicator logic in bot/bot.py
  • Added helper utilities for frame generation and safe truncation
  • Added regression test bot/tests/test_typing_indicator.py

Verification

  • Bot runs successfully locally
  • python -m py_compile passes
  • Animation stops correctly once the AI response begins

@Dhereal1
Copy link
Contributor Author

Closes #62

@SevaAIgnatyev
Copy link
Member

SevaAIgnatyev commented Mar 15, 2026

Many unnessesary file's changes. Should be a tiny PR to app/bot/reponder.ts mainly. We already have handler.ts functionality in app/bot/responder.ts and app/ai/transmitter.ts. Webhook is already ok, fatal error on unsuccessful setup is normal. Bot.py was switched to TS implementation, strange you still commit to that file. I see a lack of understanding of what is happening.

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