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

bug: errors when using "ask cody to explain" an error #2970

Closed
vadi2 opened this issue Jan 31, 2024 · 2 comments · Fixed by #3015
Closed

bug: errors when using "ask cody to explain" an error #2970

vadi2 opened this issue Jan 31, 2024 · 2 comments · Fixed by #3015
Labels
bug Something isn't working clients/vscode

Comments

@vadi2
Copy link

vadi2 commented Jan 31, 2024

Version

1.2

Describe the bug

Using the "ask cody to explain" an error always gives this message:

Cannot read properties of undefined (reading 'match')

Expected behavior

An explanation of the error.

Additional context

No response

@vadi2 vadi2 added bug Something isn't working clients/vscode labels Jan 31, 2024
@cbjerg
Copy link

cbjerg commented Feb 2, 2024

Same error here. Also "Ask Cody to fix" gives me "retrieved codebase context before initialization"

@umpox
Copy link
Contributor

umpox commented Feb 2, 2024

Thanks for reporting!

PR to fix: #3015

@cbjerg That error should be fixed in the latest release, if not please LMK and I'll take another look

umpox added a commit that referenced this issue Feb 5, 2024
## Description

- Pass the correct arguments to the `cody.action.chat` command
- Add type safety to avoid this mistake again

closes #2970

## Test plan

1. Try "Ask Cody to Explain"
2. Check there is no error

<!-- Required. See
https://sourcegraph.com/docs/dev/background-information/testing_principles.
-->

---------

Co-authored-by: Beatrix <beatrix@sourcegraph.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working clients/vscode
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants