CoPilot right click context -> Review and Comment causes a 400

[lukehinds]

Describe the issue

Performing CoPilot right click context -> Review and Comment causes a 400

CoPilot Log:

2024-12-31 18:11:53.617 [error] Error: net::ERR_PROXY_CONNECTION_FAILED
    at SimpleURLLoaderWrapper.<anonymous> (node:electron/js2c/utility_init:2:10511)
    at SimpleURLLoaderWrapper.emit (node:events:518:28): Error on conversation request
2024-12-31 18:12:04.833 [info] Request ID for failed request: d8e22004-e848-4df0-992f-ec3c7bb0fbae
2024-12-31 18:12:04.837 [error] Request Failed: 400 Bad Request

2024-12-31 18:12:54.263 [info] Request ID for failed request: 82bfc0af-0cea-47bd-b94c-35f855c71116
2024-12-31 18:12:54.265 [error] Request Failed: 400 Bad Request

CoPilot Log

2024-12-31T18:12:04.3dZ [error ] Pipeline processing error: Unterminated string starting at: line 1 column 3199 (char 3198) module=pipeline pathname=/Users/lhinds/repos/stacklok/codegate-repos/codegate/src/codegate/providers/copilot/pipeline.py

Steps to Reproduce

Select code -> Right Click -> CoPilot -> Review and Comment

Operating System

MacOS (Arm)

IDE and Version

Version: 1.96.2 (Universal) Commit: fabdb6a30b49f79a7aba0f2ad9df9b399473380f

Extension and Version

1.254.0

Provider

GitHub Copilot

Model

GPT 4o

Logs

No response

Additional Context

No response

[yrobla]

@lukehinds can you share the piece of code you used? i tried several and got:

it returns normally, telling that there is no comment

[yrobla]

@lukehinds are you still seeing this problem?

[lukehinds]

@lukehinds to recheck

[lukehinds]

This issue still persists

[blkt]

@lukehinds I tried reproducing the issue with the latest version from main using Copilot free without success (i.e. works on my machine).
We should probably look into it live.

[blkt]

I dug into this issue and I believe @jhrozek solved in this commit 3e4790d, so versions v0.1.6 and above have the fix.
@lukehinds could you please ensure you're running the latest version?

[lukehinds]

I just checked again and interestingly I get the same thing with no codegate proxy inline (using CoPilot direct), so I figure lets close this for now