Skip to content

fix(ui): batch completed read-only tools into ReadOnlyBatch in compact/minimal mode#4031

Merged
esengine merged 1 commit into
esengine:main-v2from
CVEngineer66:fix/readonly-tool-batching-compact
Jun 11, 2026
Merged

fix(ui): batch completed read-only tools into ReadOnlyBatch in compact/minimal mode#4031
esengine merged 1 commit into
esengine:main-v2from
CVEngineer66:fix/readonly-tool-batching-compact

Conversation

@CVEngineer66

Copy link
Copy Markdown
Contributor

Three render paths were rendering read-only tools (read_file, grep, glob, web_fetch, ls) as individual tool cards even after they completed:

  1. Hot zone final step (compact/minimal) — running tools render individually, completed tools batch into ReadOnlyBatch.
  2. TurnCollapse body (compact/minimal folded steps) — same batching logic added to the pre-computed body array.
  3. WarmTurnItems (expanded history turns) — new roBatch/flushRO loop added so warm read-only tools are also grouped.

This keeps running read-only tools visible (with shimmer animation) and hides completed ones behind a compact "Read N files · Search N files" fold line, consistent across all compact/minimal display modes.

…t/minimal mode

Three render paths were rendering read-only tools (read_file, grep, glob,
web_fetch, ls) as individual tool cards even after they completed:

1. Hot zone final step (compact/minimal) — running tools render
   individually, completed tools batch into ReadOnlyBatch.
2. TurnCollapse body (compact/minimal folded steps) — same batching
   logic added to the pre-computed body array.
3. WarmTurnItems (expanded history turns) — new roBatch/flushRO loop
   added so warm read-only tools are also grouped.

This keeps running read-only tools visible (with shimmer animation) and
hides completed ones behind a compact "Read N files · Search N files"
fold line, consistent across all compact/minimal display modes.
@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) labels Jun 11, 2026
@esengine esengine merged commit 2656669 into esengine:main-v2 Jun 11, 2026
14 checks passed
@esengine

Copy link
Copy Markdown
Owner

Thanks @CVEngineer66 — verified all three render paths: hot-zone running tools stay individual (shimmer preserved), TurnCollapse mirrors the existing displayItems filter (no items dropped), and WarmTurnItems batches without losing turn-action state. Merged.

esengine added a commit that referenced this pull request Jun 11, 2026
…#4032)

WarmTurnItems batched read-only tools without the status guard the hot-zone and TurnCollapse paths use. Add it for consistency so an interrupted running tool lingering in history renders individually instead of folding into the batch. Drop a stray blank line. Follow-up to #4031.

Co-authored-by: reasonix <reasonix@deepseek.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants