Skip to content

Basic proof of concept for custom commands#2102

Open
frankitox wants to merge 2 commits intoclojure-lsp:masterfrom
frankitox:master
Open

Basic proof of concept for custom commands#2102
frankitox wants to merge 2 commits intoclojure-lsp:masterfrom
frankitox:master

Conversation

@frankitox
Copy link

Description

There's only two commits for now. First one has the implementation. Second one a basic custom command to illustrate the use.

TODO

  • I created an issue (Project custom command support #2101) to discuss the problem I am trying to solve or an open issue already exists.
  • Custom linters are searched in folder clojure-lsp.exports/linters. Use /commands for custom commands?

TODO further

  • I added a new entry to CHANGELOG.md
  • I updated documentation if applicable (docs folder)

@NoahTheDuke
Copy link
Contributor

i know this is a WIP but thought I'd mention that the clj-kondo hook api would be worthwhile to copy. node creation fns as well as node type predicates: api/map-node?, etc.

@frankitox
Copy link
Author

Right, I'll add that info to the issue. I rather wait until everything is a little clearer to modify the code

@ericdallo
Copy link
Member

ericdallo commented Oct 21, 2025

@frankitox I intend to take a look soon at this, is there anything missing from your part?

@frankitox
Copy link
Author

Hi! Not really. There's two commits. First has the implementation, the second one can be discarded and contains custom commands I've been toying with.

@frankitox
Copy link
Author

@ericdallo let me know if you start working on this! I'd like to keep an eye on what namespaces make it to the API

@ericdallo
Copy link
Member

@frankitox sorry, I'm focusing on Conj's talk this weekend, I intend to check that after get back from Conj!

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