Remove redundant thinking updates

Resolved 💬 5 comments Opened Apr 17, 2025 by SamPatt Closed Apr 17, 2025

💡 Likely answer: A maintainer (tibo-openai, collaborator) responded on this thread — see the highlighted reply below.

    thinking for 201s

    thinking for 201s

    thinking for 202s

    thinking for 202s

    thinking for 202s

I don't know if this is intended or not, but I occasionally will get a group of these thinking responses within a second or two of each other, typically in groups of five before a command is displayed.

    thinking for 59s

    thinking for 59s

    thinking for 59s

    thinking for 59s

    thinking for 60s

    command

This takes up vertical space in terminal without giving useful information.

View original on GitHub ↗

5 Comments

tibo-openai collaborator · 1 year ago

Yes, would be good to fix, this isn't super nice.

aaaa-zhen · 1 year ago

<img width="559" alt="Image" src="https://github.com/user-attachments/assets/7bc46bb8-cb30-477b-91f4-3bfd535d1443" />keep thinking...

<img width="792" alt="Image" src="https://github.com/user-attachments/assets/0d2e7ca1-b7aa-4a55-b6d8-1b43f1e81090" /> the network works..

JacobMGEvans · 1 year ago

Definite in this section
https://github.com/openai/codex/blob/f5b02ed1e1f099997d95089b3364edca899d8ff9/codex-cli/src/components/chat/terminal-chat-response-item.tsx#L90-L109

I bet there is a more React way to iterate over the numbers for "state" changes since Ink is being utilized.

juzarantri · 1 year ago

@tibo-openai is this issue assigned to anyone? If not, I would love to fix this.

tibo-openai collaborator · 1 year ago

Fixed by https://github.com/openai/codex/pull/216