Skip to content

Allow tool call interruptions #6476

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 1 commit into from
Jul 7, 2025
Merged

Conversation

RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Jul 6, 2025

Description

#6333 added the robust message history to allow tool call interuptions, this PR actually allows users to interrupt, rejecting diffs if they are present.


Summary by cubic

Users can now interrupt tool calls in chat, and any pending diffs will be rejected when interrupted. This makes it easier to stop or cancel tool actions as needed.

@RomneyDa RomneyDa requested a review from a team as a code owner July 6, 2025 05:34
@RomneyDa RomneyDa requested review from Patrick-Erichsen and removed request for a team July 6, 2025 05:34
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 6, 2025
Copy link

netlify bot commented Jul 6, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 7abc031
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/686a0ae3bf125b0008b33056

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.

cubic reviewed 2 files and found no issues. Review PR in cubic.dev.

Copy link
Collaborator

@Patrick-Erichsen Patrick-Erichsen left a comment

Choose a reason for hiding this comment

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

👌

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jul 7, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit f36c6c2 into main Jul 7, 2025
42 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 7, 2025
@Patrick-Erichsen Patrick-Erichsen deleted the dallin/allow-interruptions branch July 7, 2025 18:28
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jul 7, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 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:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants