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

textDocument/documentSymbol failed keeps showing #508

Open
jribbink opened this issue Jan 11, 2024 · 1 comment
Open

textDocument/documentSymbol failed keeps showing #508

jribbink opened this issue Jan 11, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@jribbink
Copy link
Contributor

Problem

textDocument/documentSymbol failed error displays any time the LS tries to check this code (i.e. just type in the window and each time it refreshes the LS it will show).

Screenshot 2024-01-10 at 4 18 50 PM

[Error - 4:18:09 PM] Request textDocument/documentSymbol failed.
Error: selectionRange must be contained in fullRange
    at Function.validate (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:137:17337)
    at new d (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:137:17551)
    at kn (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:77:72602)
    at kn (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:77:72781)
    at a (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:77:46794)
    at Object.FMe (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:77:46880)
    at Object.to [as asDocumentSymbols] (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:77:72569)
    at l (/Users/jordan/.vscode/extensions/onflow.cadence-2.2.0/out/extension/src/main.js:81:56108)
    at d.provideDocumentSymbols (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:143:39616)

Steps to Reproduce

Write an import similar to what I have and make LS refresh.

Acceptance Criteria

Doesn't show error.

Context

Bad UX, useless error

@jribbink jribbink added the bug Something isn't working label Jan 11, 2024
@jribbink jribbink self-assigned this Jan 11, 2024
@jribbink jribbink changed the title `textDocument/ textDocument/documentSymbol failed keeps showing Jan 11, 2024
@turbolent
Copy link
Member

Probably a LS issue, maybe move to the cadence-tools repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🧊 Backlog
Development

No branches or pull requests

2 participants