Skip to content

feat(treesitter): add go AST rules and standardized testdata - #61

Merged
HarshK97 merged 3 commits into
mainfrom
feat/go-language-support
Aug 6, 2026
Merged

feat(treesitter): add go AST rules and standardized testdata#61
HarshK97 merged 3 commits into
mainfrom
feat/go-language-support

Conversation

@HarshK97

@HarshK97 HarshK97 commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Problem

  1. Go AST rules were sitting inside internal/treesitter/rules.yml with global defaults instead of having their own config folder.
  2. The test fixtures under tests/testdata/ had messy, unstandardized folder names and old test files that clogged test runner output.
  3. Adding new test fixtures was triggering the CI benchmark workflow even when no benchmarked code changed.

What Changed

  • Extracted Go AST normalization rules into internal/treesitter/go/rules.yml and cleaned up rule loading in internal/treesitter/rules.go.
  • Renamed and organized Go test cases under tests/testdata/go_gin_*, regenerated golden expectations, and removed outdated fixture folders(AI is quite good at s̶t̶e̶a̶l̶i̶n̶g̶ borrowing Open Source Codebase lol).
  • Added path filtering to .github/workflows/ci.yml so benchmark runs don't trigger on testdata changes.

@HarshK97 HarshK97 changed the title feat(go): standardize go testdata, extract go rules, and update ci workflow feat(treesitter): add go AST rules and standardized testdata Aug 6, 2026
@HarshK97
HarshK97 merged commit fe54136 into main Aug 6, 2026
13 checks passed
@HarshK97
HarshK97 deleted the feat/go-language-support branch August 6, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant