Skip to content

better premature close error, resubmit button on error dialog #6013

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

Merged
merged 2 commits into from
Jun 9, 2025

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Jun 6, 2025

Description

Fleshed out error message for "Premature close" error, and a new resubmit button that resubmits at the most recent user/tool message

Screenshots

image

@RomneyDa RomneyDa requested a review from a team as a code owner June 6, 2025 00:16
@RomneyDa RomneyDa requested review from sestinj and removed request for a team June 6, 2025 00:16
Copy link

cubic-dev-ai bot commented Jun 6, 2025

Your cubic subscription is currently inactive. Please reactivate your subscription to receive AI reviews and use cubic.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 6, 2025
Copy link

netlify bot commented Jun 6, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 6236529
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/6843292a95028a000839777b

if (chunks === 0) {
return "Connection was closed before any data was received (Premature close). Try again.";
} else {
return (
Copy link
Contributor

Choose a reason for hiding this comment

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

I really don't think the end user wants to see all of this. If they start trying to solve "Keep alive" on their own they will never succeed. Could it just be "The response was cancelled mid-stream"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Agreed, updated

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jun 6, 2025
@RomneyDa RomneyDa requested a review from sestinj June 6, 2025 17:45
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 7, 2025
@RomneyDa RomneyDa merged commit 183df6a into main Jun 9, 2025
151 of 160 checks passed
@RomneyDa RomneyDa deleted the dallin/premature-error-message branch June 9, 2025 18:31
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jun 9, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 9, 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:M This PR changes 30-99 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants