search

cursorless-dev / cursorless

Don't let the cursor slow you down
https://www.cursorless.org/
MIT License
1.13k stars 79 forks source link

New .talon scopes fail it has a header #1776

Open rokubop opened 1 year ago

rokubop commented 1 year ago

Summary

New .talon scopes don't do anything if it has a header -

Reproducing

try visualize state or take state or visualize value or take value in a .talon file with an empty - at the top of the file, or with a proper header with mode and -.

Errors

image image

pokey commented 1 year ago

I thought the bug only occurred with an empty header, no?

rokubop commented 10 months ago

Continuing to see this issue. It seems that some of the scopes fail when you have a header in your talon files. Multiple reports by users. image

nriley commented 10 months ago

I ran into this issue (no scope visualizer/sidebar when .talon files have a header) but only when I had the Todo Tree VSCode extension enabled. Log of the exceptions generated is attached below.

vscode-app-1699139524389.log

pokey commented 10 months ago

And it doesn't matter whether the line ending is LF or CRLF?

pokey commented 10 months ago

@rokubop if you could drop a code snippet in a comment that would be great so I can copy-paste

nriley commented 10 months ago

It doesn't matter for me as long as I have that extension enabled. (If I need to file an issue on the other extension, happy to do so; just not sure who to blame at this point!)

pokey commented 10 months ago

@nriley mind trying this again after updating the VSCode parse tree extension to 0.28.1?

nriley commented 10 months ago

@pokey Same error. vscode-app-1699575433380.log