Support for Dart/Flutter?

[alreadyarrived]

Issue

Hi, looking at the supported languages on https://aider.chat/docs/languages.html, it does not list Dart / Flutter as one. Wondering if you could consider being able to generate a repository map for it? thank you!

Version and model info

No response

[paul-gauthier]

Thanks for trying aider and filing this issue.

Did you read the section of that web page about adding language support?

[paul-gauthier]

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

[britannio]

Hey @paul-gauthier, I found a tags.scm file for Dart at https://github.com/UserNobody14/tree-sitter-dart/blob/master/queries/tags.scm. Zed is using it for the Dart extension (https://zed.dev/docs/languages/dart).

Is it suitable for Aider and if not, is there anything I can do?

[paul-gauthier]

It has a non standard license. I'm not sure we can use it.

[britannio]

It has a non standard license. I'm not sure we can use it.

The tree-sitter-dart repo uses the MIT license which should be okay right?

[britannio]

Looks like there's already a PR: #2236

[paul-gauthier]

CC: @malkoG

It would be fine to use a tags.scm that was adapted from an MIT-licensed repo. But the one in the PR was sourced from a repo with a custom copyright license.

[malkoG]

I found tree-sitter-dart is MIT license, and in order to check whether if it is really custom license, I diffed those of them I found this:

left: https://opensource.org/license/mit
right: https://github.com/UserNobody14/tree-sitter-dart/blob/master/LICENSE

If removing line breaks, I got:

https://www.diffchecker.com/text-compare/

[britannio]

@paul-gauthier is there still a licensing issue? Should I create a separate PR using https://github.com/UserNobody14/tree-sitter-dart/blob/master/queries/tags.scm? Or even a PR from scratch?

[malkoG]

Actually I brought those queries from the repository. And I wrote those queries, too. As you can see the repository, I wrote tests for many cases. @britannio

So, you don't need to create separate PR

[albemala]

@malkoG thank you for working on this PR #2236 !
it is now available in the latest Aider v0.65.0
however, I now get the following error, when asking aider for the repo map:
dlsym(0x43268d10, tree_sitter_dart): symbol not found
this is when reading .dart files
cc @paul-gauthier

[malkoG]

This article describes how aider builds repo-map.
So I wanted to know reason why, I found here. https://github.com/grantjenks/py-tree-sitter-languages/blob/main/build.py

If I think is right,
we need to mention that
when adding additional language support, it will also have dependency to here

[paul-gauthier]

It appears that the PR does not work. I added test coverage after merging it, but there was a bug in my tests which made them pass incorrectly.

py-tree-sitter-languages does not support dart, so this seems unlikely to be salvageable in the short term. I'm looking into it though.