Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Backport 7.0.4 fixes to 7.0.2 #2322

Closed
wants to merge 3 commits into from
Closed

Backport 7.0.4 fixes to 7.0.2 #2322

wants to merge 3 commits into from

Conversation

pkukielka
Copy link
Contributor

Changes

This PR backports JetBrains 7.0.4 fixes to 7.0.2, as we need to rollback due to few cody agent issues we found in the meantime.

Cherry-picked bugfixes:

Test plan

Full QA

@pkukielka
Switch back to cody main (#2270)
f610529 
## Test plan

Full QA
@pkukielka
Fix chat history download (#2282)
3c38578 
Fixes #2274

## Test plan

1. Open Cody chat history
2. Click "Export History"
3. Select a file and path of your choice to save it
4. Verify that file content contains proper json
@pkukielka
Do not run EDT code in CodyFixHighlightPass:doCollectInformation (#2289)
d1068e2 
Fixes #2287

## Changes 

Looks like we cannot do EDT actions in the
`CodyFixHighlightPass:doCollectInformation` or we are risking a
deadlock.

## Test plan

It's a bit hard to test deterministically but for me adding and removing
few special characters on an empty file was triggering IDE crash pretty
quickly (10-20 sec).
I used combination of:

`Option + \` (`«`)
`Option + P` (`Ļ`)
`Backspace`

In a random order.
Timing matters, keep changing the tempo of the keystrokes.
@pkukielka pkukielka closed this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pkukielka