<@U090ER7JPHA> when displaying in terminal, an act...
# slackbot-chatter
c
@nutritious-balloon-65098 when displaying in terminal, an action that has no reasoning from GPT-5.5 is displayed like: Reasoning: the-actual-command-here-for-terminal-tool In other words, it looks like the command is reasoning, but it’s not. Open a PR to NOT display the string “Reasoning:” when the reasoning content is empty.
n
I'm on it! can track my progress here.
OpenHands encountered an error: **Request timeout after 30 seconds to https://clzrgjelzmrphoyr.prod-runtime.all-hands.dev/api/conversations/1b0448b3-45e4-40e7-90c1-2ef9bccb04a4/ask_agent** See the conversation for more information.
It’s just formatting control.
prefix
lets the shared helper decide whether to insert spacing before
Reasoning:
without hardcoding the same layout for every caller. In this case: •
ActionEvent
calls it with no prefix because the action view already manages its own blank lines before that section. •
MessageEvent
calls it with
"\n\n"
because the reasoning block should appear separated from the message text. So the point is not semantics, just avoiding duplicated reasoning-rendering logic while preserving each caller’s existing spacing. If you want the more opinionated take: it’s optional convenience, not fundamental. We could also remove
prefix
and have each caller add its own newline before calling the helper.
c
@nutritious-balloon-65098 Yeah simplify this. Have each caller add its own newline.
n
OpenHands encountered an error: **Request timeout after 30 seconds to https://clzrgjelzmrphoyr.prod-runtime.all-hands.dev/api/conversations/1b0448b3-45e4-40e7-90c1-2ef9bccb04a4/ask_agent** See the conversation for more information.
OpenHands encountered an error: **Request timeout after 30 seconds to https://clzrgjelzmrphoyr.prod-runtime.all-hands.dev/api/conversations/1b0448b3-45e4-40e7-90c1-2ef9bccb04a4/ask_agent** See the conversation for more information.