sourcegraph / jetbrains

Apache License 2.0
61 stars 19 forks source link

JetBrains: Continuous loading on clicking Apply button after sending request using same source code file #2103

Open RajatNagoria opened 3 weeks ago

RajatNagoria commented 3 weeks ago

IDE Information

IntellJ IDEA 2024.1.4 (Community Edition) Build #PC-233.13763.11, built on January 26, 2024 Runtime version: 17.0.9+7-b1087.11 aarch64 VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o. Windows 11 GC: G1 Young Generation, G1 Old Generation Memory: 2048M Cores: 8 Metal Rendering is ON Non-Bundled Plugins: com.sourcegraph.jetbrains (6.0.31)

Bug Description

Platform: Windows

Steps to reproduce:

  1. Have Cody plugin installed in your JB IntelliJ editor
  2. Login to Cody with Enterprise user
  3. Open Chat snippet and search for anything > Insert the search code to source code file
  4. Now start New chat > Make sure current repository and current source file is shown
  5. Send the request using that > Make sure same code is given as response
  6. Now click on 'Apply button'

Observed Behavior:

Continuous loading on clicking Apply button after sending request using same source code file

Expected behavior:

Apply button should work properly and no continuous loading should be shown

Additional context

Video link: https://drive.google.com/file/d/1vvMIEM-WWYVWqIWOZ-Dmlqxwh9DHGQ0R/view

mkondratek commented 3 weeks ago

I tried this scenario. It works for me. Possibly it has been fixed already. IMO we can close this one.

RajatNagoria commented 3 weeks ago

Looks Like issue is fixed, hence closing

RajatNagoria commented 3 weeks ago

Reopening as issue is again repro in 6.0.35 Nightly

mkondratek commented 3 weeks ago

I cannot reproduce it. Are the steps up to date? I admit that I am working on macOS while your example shows Windows. I am going to try it too.

mkondratek commented 3 weeks ago

I manage to reproduce some issue (not sure if that exactly what we see in this report). The problem seems to be related to the windows env and the space character in the file name. I would consider this one as a medium prio at most since the conditions are extraordinary. I am going to look into it.