Skip to content

perf(tui): fix CPU lockup and O(N) scans during waiting/running states #494

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mattimustang wants to merge 7 commits into
base: main
Choose a base branch
from
Open

perf(tui): fix CPU lockup and O(N) scans during waiting/running states #494

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
97f577a
feat: add HTTP request smuggling skill (#405)
sandiyochristan May 21, 2026
7418a23
Merge branch 'usestrix:main' into main
mattimustang May 21, 2026
3508da4
perf(tui): fix O(N) scans and redundant 60fps renders during waiting …
mattimustang May 21, 2026
4b17b57
perf(tui): cache event renders and eliminate redundant work during ru…
mattimustang May 21, 2026
d0cbaec
perf(tui): index chat_messages by agent to eliminate O(N) scan per frame
mattimustang May 22, 2026
e68c267
Merge branch 'usestrix:main' into main
mattimustang May 30, 2026
849c70d
Merge main into fix/tui-cpu-lockup-clean and re-apply perf fixes
mattimustang May 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 21 additions & 3 deletions strix/interface/tui/app.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -709,6 +709,8 @@ def __init__(self, args: argparse.Namespace):
self._displayed_agents: set[str] = set()
self._displayed_events: list[str] = []

self._event_render_cache: dict[str, Any] = {}

self._scan_thread: threading.Thread | None = None
self._scan_loop: asyncio.AbstractEventLoop | None = None
self._scan_stop_event = threading.Event()
Expand Down Expand Up @@ -902,7 +904,8 @@ def _update_ui(self) -> None:

self._update_chat_view()

self._update_agent_status_display()
if self._dot_animation_timer is None:
self._update_agent_status_display()

self._update_stats_display()

Expand Down Expand Up @@ -1086,11 +1089,24 @@ def _get_rendered_events_content(self, events: list[dict[str, Any]]) -> Any:

for event in events:
content: Any = None
event_id = event["id"]

if event["type"] == "chat":
content = self._render_chat_content(event["data"])
if event_id in self._event_render_cache:
content = self._event_render_cache[event_id]
else:
content = self._render_chat_content(event["data"])
if content is not None:
self._event_render_cache[event_id] = content
elif event["type"] == "tool":
content = render_tool_widget(event["data"])
status = event["data"].get("status", "")
cache_key = f"{event_id}_{status}"
if cache_key in self._event_render_cache:
content = self._event_render_cache[cache_key]
else:
content = render_tool_widget(event["data"])
if content is not None and status in ("completed", "failed", "error"):
self._event_render_cache[cache_key] = content

if content:
if renderables:
Expand Down Expand Up @@ -1305,6 +1321,7 @@ def _animate_dots(self) -> None:
status = agent_data.get("status", "running")
if status in ["running", "waiting"]:
has_active_agents = True
if status == "running":
num_colors = len(self._sweep_colors)
offset = num_colors - 1
max_pos = (self._sweep_num_squares - 1) + offset
Expand Down Expand Up @@ -1346,6 +1363,7 @@ def watch_selected_agent_id(self, _agent_id: str | None) -> None:
return

self._displayed_events.clear()
self._event_render_cache.clear()

self.call_later(self._update_chat_view)
self._update_agent_status_display()
Expand Down