Skip to content

Commit 79200d3

Browse files
committed
Update TUI icons for reasoning and tool calls.
1 parent 0c4e89c commit 79200d3

2 files changed

Lines changed: 10 additions & 10 deletions

File tree

python_agent_harness/tui.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,7 @@ def _build_history_rows(self, full: bool = False) -> list[Any]:
612612
# the reasoning streamed live while it was being
613613
# produced; once it is done it collapses to a marker
614614
# so it doesn't eat the visible-row budget
615-
rows.append(Text("💭 ...", style="dim"))
615+
rows.append(Text("🤔 ...", style="dim"))
616616
if m.tool_calls:
617617
for tc in m.tool_calls:
618618
args = tc.arguments
@@ -629,7 +629,7 @@ def _build_history_rows(self, full: bool = False) -> list[Any]:
629629
)
630630
else:
631631
params = ""
632-
label = f"🤖 {tc.name}({params})" if params else f"🤖 {tc.name}"
632+
label = f"⚙️ {tc.name}({params})" if params else f"⚙️ {tc.name}"
633633
rows.append(Text(label, style="magenta"))
634634
if body.strip():
635635
rows.append(Markdown(f"**assistant:** {body}", style=ASSISTANT_STYLE))

tests/test_tui.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ def test_reasoning_collapsed_in_history(self):
287287
]
288288
tui.console.print(tui._render_conversation())
289289
out = buf.getvalue()
290-
self.assertIn("💭 ...", out)
290+
self.assertIn("🤔 ...", out)
291291
self.assertIn("Rayleigh scattering.", out)
292292
self.assertNotIn("Let me think", out)
293293
# the stored message keeps its reasoning — only the display hides it
@@ -308,7 +308,7 @@ def test_reasoning_collapsed_marker_shows_even_without_answer(self):
308308
]
309309
tui.console.print(tui._render_conversation())
310310
out = buf.getvalue()
311-
self.assertIn("💭 ...", out)
311+
self.assertIn("🤔 ...", out)
312312
self.assertNotIn("pensive thoughts", out)
313313

314314
def test_reasoning_marker_before_tool_call_label(self):
@@ -328,9 +328,9 @@ def test_reasoning_marker_before_tool_call_label(self):
328328
]
329329
tui.console.print(tui._render_conversation())
330330
out = buf.getvalue()
331-
self.assertIn("💭 ...", out)
332-
self.assertIn("🤖 Read", out)
333-
self.assertLess(out.index("💭 ..."), out.index("🤖 Read"))
331+
self.assertIn("🤔 ...", out)
332+
self.assertIn("⚙️ Read", out)
333+
self.assertLess(out.index("🤔 ..."), out.index("⚙️ Read"))
334334
self.assertNotIn("let me check", out)
335335

336336
def test_non_string_reasoning_does_not_crash(self):
@@ -348,7 +348,7 @@ def test_non_string_reasoning_does_not_crash(self):
348348
tui.console.print(tui._render_conversation())
349349
out = buf.getvalue()
350350
self.assertIn("full text here", out)
351-
self.assertNotIn("💭 ...", out)
351+
self.assertNotIn("🤔 ...", out)
352352

353353
def test_strip_reasoning(self):
354354
"""_strip_reasoning removes the leading reasoning prefix and
@@ -1642,7 +1642,7 @@ def test_tool_call_bad_json_arguments(self):
16421642
]
16431643
tui.console.print(tui._render_conversation())
16441644
out = buf.getvalue()
1645-
self.assertIn("🤖 Read", out)
1645+
self.assertIn("⚙️ Read", out)
16461646

16471647
def test_tool_call_non_dict_arguments(self):
16481648
"""A JSON array of arguments renders as a bare tool label."""
@@ -1658,7 +1658,7 @@ def test_tool_call_non_dict_arguments(self):
16581658
]
16591659
tui.console.print(tui._render_conversation())
16601660
out = buf.getvalue()
1661-
self.assertIn("🤖 Bash", out)
1661+
self.assertIn("⚙️ Bash", out)
16621662
self.assertNotIn("ls", out)
16631663

16641664
# ------------------------------------------------------------------

0 commit comments

Comments
 (0)