Skip to content

Conversation

@Antman261
Copy link
Contributor

@Antman261 Antman261 commented Nov 20, 2025

@Antman261 Antman261 reopened this Nov 25, 2025
@Antman261 Antman261 marked this pull request as ready for review November 25, 2025 09:52
@Antman261 Antman261 requested a review from a team as a code owner November 25, 2025 09:52
@Antman261
Copy link
Contributor Author

The scopes are showing up as unsupported in the language, which I believe means that the tree parsing is working correctly:
Screenshot 2025-11-25 at 5 56 16 pm

@AndreasArvidsson
Copy link
Member

Not necessarily. The best way is to actually test the parse tree. Open a zig file and run the parse tree action. You should get a markdown file with the source code and parse tree in it.

https://github.com/cursorless-dev/cursorless/blob/46851008df2ba99f3e1051e2a47c90604e99d0cf/cursorless-talon-dev/src/cursorless_dev.talon#L38-L39

@Antman261
Copy link
Contributor Author

Does this look correct?:

Cursorless parse tree

test.zig [3:10-3:12]

le
(source_file
  (function_declaration
    body: (block
      (variable_declaration
        (identifier)
      )
    )
  )
)
source_file [0:0-29:0]
  function_declaration [2:0-28:1]
    body: block [2:19-28:1]
      variable_declaration [3:4-3:29]
        identifier [3:10-3:12]

@AndreasArvidsson
Copy link
Member

I don't use zig personally, but you definitely got a valid parse tree back. :)

@AndreasArvidsson AndreasArvidsson merged commit c72adb3 into cursorless-dev:main Nov 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants