Skip to content

feat(ast-splitter): Add support for Ruby language#188

Closed
matthewevans wants to merge 1 commit intozilliztech:masterfrom
everwise:feat/me-add-ruby-ast-support
Closed

feat(ast-splitter): Add support for Ruby language#188
matthewevans wants to merge 1 commit intozilliztech:masterfrom
everwise:feat/me-add-ruby-ast-support

Conversation

@matthewevans
Copy link

This commit adds support for the Ruby programming language in the AST
splitter. It includes the following changes:

  • Add the tree-sitter-ruby dependency to the package.json file.
  • Add the Ruby language and its corresponding node types to the
    SPLITTABLE_NODE_TYPES object in the ast-splitter.ts file.
  • Add the Ruby language and its file extensions to the language mapping
    in the ast-splitter.ts file.
  • Update the getSupportedLanguages() method to include the Ruby
    language and its file extensions.

This commit adds support for the Ruby programming language in the AST
splitter. It includes the following changes:

- Add the `tree-sitter-ruby` dependency to the `package.json` file.
- Add the Ruby language and its corresponding node types to the
  `SPLITTABLE_NODE_TYPES` object in the `ast-splitter.ts` file.
- Add the Ruby language and its file extensions to the language mapping
  in the `ast-splitter.ts` file.
- Update the `getSupportedLanguages()` method to include the Ruby
  language and its file extensions.
@matthewevans matthewevans deleted the feat/me-add-ruby-ast-support branch August 31, 2025 14:06
@kierr
Copy link

kierr commented Sep 12, 2025

Hi @matthewevans, any reason why you closed/deleted this?

I ask because I'm seeing Language ruby not supported by AST, using LangChain splitter for: example.rb despite the README saying Ruby is a supported language.

@matthewevans
Copy link
Author

Hi @kierr — because a little more was required to get it to work. I do have it working in my forked copy @everwise/claude-context. It took a couple commits because I initially tried upgrading the tree-sitter packages. 🙃

be5e2d1
fc34302
1e7f48a

@vardhanv-ship-it
Copy link

@matthewevans any chance you are going to merge this back in soon?

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