Skip to content

feat: Group consecutive bash tool cards#176

Merged
yongkangc merged 3 commits into
cfal:mainfrom
0xSolarPunk:codex/group-bash-tool-cards
May 30, 2026
Merged

feat: Group consecutive bash tool cards#176
yongkangc merged 3 commits into
cfal:mainfrom
0xSolarPunk:codex/group-bash-tool-cards

Conversation

@0xSolarPunk
Copy link
Copy Markdown
Contributor

@0xSolarPunk 0xSolarPunk commented May 29, 2026

Summary

  • Groups adjacent bash-tool-use messages into one compact Bash card in the chat feed.
  • Keeps single Bash tool-use rows on the existing renderer path.
  • Preserves hidden tool-result handling so Bash calls still group when their result events sit between tool-use events.
  • Adds a copy-all control for grouped commands and focused coverage for grouping behavior.

Screenshots

Desktop:
image

Mobile:

Grouped Bash cards on mobile

Validation

  • bun run --cwd web test src/lib/chat/__tests__/conversation-feed-items.test.ts src/lib/components/chat/tools/__tests__/ChatToolEventRenderer.test.ts
  • bun run check
  • bun run test
  • timeout 45s bun run start --port 0
  • Screenshot server: bun server/main.js --port 0 --disable-auth on http://127.0.0.1:16850

@yongkangc yongkangc closed this May 29, 2026
@yongkangc yongkangc reopened this May 30, 2026
@yongkangc yongkangc changed the title [codex] Group consecutive bash tool cards feat: Group consecutive bash tool cards May 30, 2026
@yongkangc yongkangc enabled auto-merge (squash) May 30, 2026 01:12
@yongkangc yongkangc requested a review from Copilot May 30, 2026 01:12
@yongkangc yongkangc merged commit e8516dc into cfal:main May 30, 2026
4 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants