Skip to content

Conversation

@uinstinct
Copy link
Contributor

@uinstinct uinstinct commented Dec 5, 2025

Description

For pending tools, the count of tools not yet completed. This PR fixes that.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

before.mp4
after.mp4

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Fixes the pending tools count in ToolCallDiv so the grouped header shows the number of tools not yet completed. Uses a new pendingCalls list (status !== "done") and passes it to GroupedToolCallHeader, falling back to activeCalls when none are pending.

Written for commit 3d7be8a. Summary will update automatically on new commits.

@uinstinct uinstinct requested a review from a team as a code owner December 5, 2025 04:12
@uinstinct uinstinct requested review from sestinj and removed request for a team December 5, 2025 04:12
@continue
Copy link
Contributor

continue bot commented Dec 5, 2025

Keep this PR in a mergeable state →

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts

@continue-development-app
Copy link

Keep this PR in a mergeable state →

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 5, 2025
@continue
Copy link
Contributor

continue bot commented Dec 5, 2025

Reviewed the changes - no documentation updates needed. This is a UI bug fix that corrects the pending tool count display in the grouped tool call header. The change is internal to the GUI component and doesn't affect any documented user-facing features, APIs, or configuration options.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Dec 5, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 5, 2025
@RomneyDa RomneyDa merged commit 04b672b into continuedev:main Dec 5, 2025
59 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Dec 5, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants