Skip to content

feat: Make API error messages expandable to show full details #6143

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

roomote[bot]
Copy link

@roomote roomote bot commented Jul 23, 2025

Fixes #6139

Summary

This PR makes API error messages expandable in the chat feed, allowing users to view full error details similar to how successful API requests can be expanded.

Changes

  • Added errorDetails field to ClineApiReqInfo interface to store full error information
  • Modified ChatRow component to make failed API requests clickable/expandable
  • Updated Task.ts to capture and store complete error details when API requests fail
  • Added comprehensive tests for the new functionality

Testing

  • Added unit tests for all new functionality
  • All existing tests pass
  • Manually tested expanding/collapsing error messages in the UI

Screenshots

Before: API errors show only a brief message with no way to see details
After: API errors can be clicked to expand and show full error details including status codes, error types, and complete error messages


Important

This PR makes API error messages expandable in the chat feed, capturing full error details in Task.ts and updating ChatRow to display them, with comprehensive tests added.

  • Behavior:
    • API error messages in ChatRow are now expandable to show full details, including status codes and error types.
    • Task.ts captures and stores complete error details when API requests fail.
  • Components:
    • ChatRow.tsx modified to make failed API requests clickable/expandable.
    • errorDetails field added to ClineApiReqInfo in ExtensionMessage.ts.
  • Testing:
    • Added unit tests in ChatRow.spec.tsx for expanding/collapsing error messages.
    • Tests ensure error details are displayed correctly when expanded.

This description was created by Ellipsis for 023398d. You can customize this summary. It will automatically update as commits are pushed.

- Added errorDetails field to ClineApiReqInfo interface to store full error information
- Modified ChatRow component to make failed API requests clickable/expandable
- Updated Task.ts to capture and store complete error details when API requests fail
- Added comprehensive tests for the new functionality

Fixes #6139
@roomote roomote bot requested review from mrubens, cte and jr as code owners July 23, 2025 22:47
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request UI/UX UI/UX related or focused labels Jul 23, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jul 23, 2025
@daniel-lxs
Copy link
Collaborator

Closing for now while the issue author works on his implementation, but this should be checked again before merging.

@daniel-lxs daniel-lxs closed this Jul 24, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jul 24, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:L This PR changes 100-499 lines, ignoring generated files. UI/UX UI/UX related or focused
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Improve API Failure Message Handling & Display
3 participants