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.

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

Codebase inference triggers when cody is not being used #51349

Closed
danielmarquespt opened this issue May 2, 2023 · 4 comments · Fixed by #51714
Closed

Codebase inference triggers when cody is not being used #51349

danielmarquespt opened this issue May 2, 2023 · 4 comments · Fixed by #51714
Assignees
Labels
bug An error, flaw or fault that produces an incorrect or unexpected result, or behavior. cody/vscode cody team/cody Cody team

Comments

@danielmarquespt
Copy link
Contributor

Image

It seems that Cody is constantly looking for the git clone url, but there are scenarios (like the example this user shared) that Cody shouldn't be trying to infer the codebase until it is focused.

Doing so it will display errors not relevant to the user.

@danielmarquespt danielmarquespt self-assigned this May 2, 2023
@mrnugget mrnugget added the bug An error, flaw or fault that produces an incorrect or unexpected result, or behavior. label May 4, 2023
@mrnugget
Copy link
Contributor

mrnugget commented May 4, 2023

@beyang can you take a look at this if you have the time?

@mrnugget mrnugget assigned abeatrix and unassigned danielmarquespt May 9, 2023
@abeatrix
Copy link
Contributor

abeatrix commented May 9, 2023

This should be taken care by #51274 and include in release v0.0.10
We are now logging the error in console instead of using popup during the auto detection step as we should limit system pop up to errors relevant to the system.
https://sourcegraph.com/github.com/sourcegraph/sourcegraph@306cce682150e32d0a2e8d1e3ad3a96b12d9110f/-/blob/client/cody/src/chat/ChatViewProvider.ts?L685

@mrnugget
Copy link
Contributor

@abeatrix does that mean that you're currently trying to confirm whether it's fixed or not, or that it is fixed?

@abeatrix
Copy link
Contributor

@mrnugget It was fixed only for that specific error, but this new PR should now fix the root issue: #51714

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error, flaw or fault that produces an incorrect or unexpected result, or behavior. cody/vscode cody team/cody Cody team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants