Skip to content

Conversation

@Antman261
Copy link
Contributor

Adding tree sitter for zig. Followed the instructions in the readme, not sure if there is anything else to do. It installed successfully

@Antman261 Antman261 marked this pull request as draft November 20, 2025 15:06
@Antman261
Copy link
Contributor Author

Having trouble getting this to build correctly, not sure if it's my environment, it's not even cloning the GitHub dependencies at all

package.json Outdated
"tree-sitter-typescript": "github:tree-sitter/tree-sitter-typescript#4ad3010c91d700026d036b5230e2d99ba94ae8a4",
"tree-sitter-xml": "github:tree-sitter-grammars/tree-sitter-xml#648183d86f6f8ffb240ea11b4c6873f6f45d8b67",
"tree-sitter-yaml": "github:tree-sitter-grammars/tree-sitter-yaml#7b03feefd36b5f155465ca736c6304aca983b267",
"tree-sitter-zig": "github:tree-sitter/zig-tree-sitter#a53dac49d66562006e84dd11211b7ae40db68263",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you picked up the wrong library. That's the tree-sitter bindings if you want to use tree-sitter in zig, eg if you want to parse typescript from your zig program. I think the correct library is https://github.com/tree-sitter-grammars/tree-sitter-zig

Copy link
Contributor Author

@Antman261 Antman261 Nov 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohhh! Thank you 😅 I've updated package.json to use the correct library and reran pnpm install --ignore-scripts

I have also removed my change to the readme based on Andreas's comment on slack that you don't recommend building it locally and instead relying on GitHub Actions. I can't trigger the GitHub action — let me know if there is anything else I should do?

@Antman261 Antman261 marked this pull request as ready for review November 24, 2025 09:28
@AndreasArvidsson AndreasArvidsson merged commit aa56a23 into cursorless-dev:main Nov 24, 2025
1 check 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.

3 participants