Skip to content

Feature request: Add missing tags.scm file for R language #1729

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
bcgalvin opened this issue Sep 25, 2024 · 5 comments · May be fixed by #2915
Closed

Feature request: Add missing tags.scm file for R language #1729

bcgalvin opened this issue Sep 25, 2024 · 5 comments · May be fixed by #2915
Labels
enhancement New feature or request fixed

Comments

@bcgalvin
Copy link

Issue

The aider/queries/README.md lists https://github.com/r-lib/tree-sitter-r for R language support, but there's no tags.scm file for R in aider/queries/ (and is the only listed language missing this file).

I plan to submit a PR for this but am curious about this line in the queries/ README: Aider uses modified versions of the tags.scm files.. is there any guidance on what modifications are made from the language scm files to be compatible with aider?

Version and model info

No response

@fry69 fry69 added the enhancement New feature or request label Sep 25, 2024
@bcgalvin
Copy link
Author

think I know why its missing, tree-sitter-languages is kind of unmaintained and is using an old version of the r-lib tree sitter library:

https://github.com/grantjenks/py-tree-sitter-languages/blob/v1.10.2/repos.txt#L18C40-L18C80

and the tags are empty
https://github.com/r-lib/tree-sitter-r/blob/c55f8b4dfaa32c80ddef6c0ac0e79b05cb0cbf57/queries/tags.scm

are there plans to move off from tree-sitter-languages? it would be nice to not have it as a dependency since it seems semi-abandoned and stuck on the older tree-sitter api

@gdmcdonald
Copy link

Second/third this and just clarifying; the tags.scm file for R language support is here:
https://github.com/r-lib/tree-sitter-r/blob/main/queries/tags.scm

@paul-gauthier
Copy link
Collaborator

Thanks for trying aider and filing this issue.

Please see these two related issues for relevant updates:

@paul-gauthier
Copy link
Collaborator

Main branch has a big upgrade in programming languages supported by adopting tree-sitter-language-pack.

  • 130 new languages with linter support.
  • 20 new languages with repo-map support.

Including R.

The change is available in the main branch. You can get it by installing the latest version from github:

aider --install-main-branch

# or...

python -m pip install --upgrade --upgrade-strategy only-if-needed git+https://github.com/Aider-AI/aider.git

If you have a chance to try it, let me know if it works for you.

Copy link

github-actions bot commented Apr 3, 2025

I'm closing this enhancement request since it has been marked as 'fixed' for over 3 weeks. The requested feature should now be available in recent versions of aider.

If you find that this enhancement is still needed, please feel free to reopen this issue or create a new one.

Note: A bot script made these updates to the issue.

@github-actions github-actions bot closed this as completed Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants