Skip to content

fix(session-ui): show one count before used - #48038

Merged
Hona merged 1 commit into
v2from
tool-summary-count
Sep 8, 2026
Merged

Hona merged 1 commit into
v2from
tool-summary-count

Conversation

@iamdavidhill

Copy link
Copy Markdown
Contributor

Issue for this PR

Requested UI polish; no linked issue.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Changes grouped tool summaries from Used 1 Shell, 1 Read, 2 Agent to 4 used Shell, Read, Agent.

The overall count reflects calls, while tool names appear once in their original order. Uses lowercase used, keeps the copy localized, and updates the affected component and end-to-end expectations.

How did you verify your code works?

  • Package typechecks and the repository's pre-push typecheck passed (35 tasks).
  • 76 component tests passed, covering count updates, disclosure state, narrow/wide layouts, light/dark themes, and localized labels.
  • One existing test fails before rendering: groups every collapsed tool until visible text separates the stack throws textVisible(...).get(...) is not a function. Reproduced the same error in an unchanged worktree at base commit 5c30292daa.
  • Formatting and git diff --check passed.

Screenshots / recordings

Same production component and current-tool-group--mixed-tools Storybook fixture at 840 × 360, captured from the base commit and this branch.

Before After
Before: Used 1 Shell, 1 Read, 2 Agent After: 4 used Shell, Read, Agent

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@Hona
Hona merged commit 8c5eca5 into v2 Sep 8, 2026
10 checks passed
@Hona
Hona deleted the tool-summary-count branch September 8, 2026 23:15
jinhuang712 pushed a commit to jinhuang712/opencode that referenced this pull request Sep 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants