Skip to content

Conversation

@QianKuang8
Copy link
Contributor

@QianKuang8 QianKuang8 commented Aug 15, 2025

Description

Fixed an issue where the VS Code extension's chat window would get stuck in the "applying" state if an error occurred during the applying process

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-general-review or @continue-detailed-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 Fix

Screenity.video.-.Aug.14.2025.mp4

After Fix

Screenity.video.-.Aug.15.2025.mp4

Tests

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


Summary by cubic

Fixed an issue where the VS Code extension chat window would stay stuck in "applying" if an error occurred during the apply process.

  • Bug Fixes
  • Ensured the apply state resets to closed even if an error happens.

@QianKuang8 QianKuang8 requested a review from a team as a code owner August 15, 2025 05:34
@QianKuang8 QianKuang8 requested review from Patrick-Erichsen and removed request for a team August 15, 2025 05:34
@github-actions
Copy link

github-actions bot commented Aug 15, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@QianKuang8
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 15, 2025
@QianKuang8 QianKuang8 force-pushed the qian/fix-vscode-apply branch from 278ac10 to 48a54c6 Compare August 15, 2025 05:51
@QianKuang8
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

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.

Nice fix, much appreciated!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 15, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 15, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 74287e8 into continuedev:main Aug 15, 2025
34 of 38 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 15, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 15, 2025
@sestinj
Copy link
Contributor

sestinj commented Aug 18, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Aug 18, 2025

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Aug 19, 2025

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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 released size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants