Skip to content

Fix: tool status saved as "undefined"#2785

Merged
kzsb03 merged 2 commits intodevfrom
tool-fix
Mar 2, 2026
Merged

Fix: tool status saved as "undefined"#2785
kzsb03 merged 2 commits intodevfrom
tool-fix

Conversation

@tevanburen
Copy link
Copy Markdown
Contributor

@tevanburen tevanburen commented Mar 2, 2026

Description

Calling tools in ask mode with custom UI was bricking the chat

Changes Made

Need to pass "success" to the BE instead of "undefined"

How to Test

Before pulling, on dev:

  1. Call a tool with a custom UI in ask mode
  2. Run the tool
  3. Notice that you can no longer send messages

Pull this branch, then repeat the above. Verify that it's ok now

@tevanburen tevanburen self-assigned this Mar 2, 2026
@tevanburen tevanburen marked this pull request as ready for review March 2, 2026 19:16
@tevanburen tevanburen requested a review from a team as a code owner March 2, 2026 19:16
@tevanburen tevanburen requested a review from spatel776 March 2, 2026 19:27
@spatel776
Copy link
Copy Markdown
Contributor

I have verified that this test works.
image

@kzsb03 kzsb03 merged commit 646c3a5 into dev Mar 2, 2026
3 checks passed
@kzsb03 kzsb03 deleted the tool-fix branch March 2, 2026 20:46
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 2, 2026

@CodiumAI-Agent /update_changelog

SunTheDev pushed a commit that referenced this pull request Mar 6, 2026
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