Tried to add Clojure support. about astminer

[egor-bogomolov]

@FiV0 thanks for reporting the issue! Could you please share the link to the repository with Clojure integration and the failing example? I'll look into it tonight.

[FiV0]

@egor-bogomolov
This branch has the files: https://github.com/FiV0/astminer/tree/clojure-support
You just need to run /.gradlew test and one test will fail.
To get the stacktrace you need to comment out the try statement in the ClojureParser.kt

[egor-bogomolov]

@FiV0
I've reproduced the failure yesterday but didn't find out the fundamental reason. I will give it one more shot today.

[egor-bogomolov]

Sorry for the late reply, but I fixed the problem. For Clojure, some characters like braces do not have a special type in grammar and we need a special workaround for this case.
To fix the issue, update TreeUtil:convertTerminal as follows:

val typeName = vocabulary.getSymbolicName(terminalNode.symbol.type) ?: "ESCAPED"
return SimpleNode(typeName, parent, terminalNode.symbol.text)

And add a null-check in TreeUtil:convertRuleContext: ruleContext.children.forEach -> ruleContext.children?.forEach

I will update our implementation to avoid problems of this type.

[SpirinEgor]

We completely refactor our pipeline and make it easier to add new languages. If you are still interested in adding Clojure support, navigate to documentation and see examples of existing parsers.

If you still have questions, don't hesitate to reopen this issue.