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
tsx files are typically typescript files #1126
base: preview
Are you sure you want to change the base?
Conversation
These files were not showing it by that code search. This way might be why.
@jaresty by code search do you mean the @ codebase context provider, cmd+enter, @ files, or something else? The mapping here is for tree-sitter wasm files, and there happens to be a specific one or .tsx files that is separate from typescript, so there's likely something else causing the problem Edit: Just saw your issue. Has this change proven to solve the problem in your testing? The main reason I can think of that this would happen is that the tree-sitter-tsx.wasm file is broken, but that seems less likely. I'm noticing in testing that it's not just tsx files that have missing functions/classes in @ code |
I mean that when I do Code search in my project, I do see symbols but only symbols from files that end in ".ts", so my lsp is working. I noticed that in the file I linked, these things are true:
Since one of these look inconsistent, I'm theorizing that it could be a mistake. I'm not sure if it's the whole picture. |
I'm not sure how to test. I was hoping that someone here could help me with that. |
A method from that file is used in the context provider as well here |
@jaresty Just a heads up TSX is another valid grammar within tree-sitter separate from typescript.. https://github.com/tree-sitter/tree-sitter-typescript |
Isn't it also true of jsx files that they are a valid grammar too? Why is the mapping different there? |
I'm not sure if this fixes my issue, but if you can point me in the right direction I'll experiment and try to fix it. I'm not sure how to run an extension from source code. |
@jaresty true that .jsx should probably have it's own grammar, I think this is probably just because there wasn't a readily available jsx grammar found and jsx is slightly less common. Here is a guide to getting set up and running from source: https://github.com/continuedev/continue/blob/main/CONTRIBUTING.md#environment-setup And it seems like this function is probably where you'll want to set a conditional breakpoint to find out what's going on |
These files were not showing it by that code search. This way might be why.
Description
[ What changed? Feel free to be brief. ]
Checklist
preview
, rather thanmain