cider-connect-cljs: make server, port and build configurable

[manuel-uberti]

Request

When I use cider-connect-cljs to connect to a shadow-cljs nREPL server, I always need to manually select the server, the port and the build, which for a project have always the same values every time.

Possible solution

It'd be nice if I can set the default server, port and build in .dir-locals.el, just as I do for the CLJS REPL type with cider-default-cljs-repl for instance.

Current alternative

As of now, I just do it manually. It's not a blocking issue, of course, it's just something to speed up the initial setup of the environment.

[bbatsov]

I guess we can have just some default server and port settings for all connect commands, not just the cljs connect.

[manuel-uberti]

That would be even better, of course. 👍

[bbatsov]

Thinking a bit more about this it should probably be a default endpoint or something like this, as we currently do. Have you seen cider-known-endpoints? I guess adding there your host is a decent workaround for now.

[manuel-uberti]

Thanks, didn't know about cider-known-endpoints.

It works fine, although I still have to select the endpoint and the build before having a working CLJS REPL. But it's a good workaround, thank you!

[vspinu]

although I still have to select the endpoint and the build

If you set it in .dir-locals to one option it basically comes to pressing RET, RET, which is not a big deal I would guess.

One solution without adding an extra config would be to allow cider-known-endpoints to be one endpoint (not a list of endpoints). In such a case it could be interpreted as the endpoint and no query performed.

[manuel-uberti]

I am already setting cider-known-endpoints in .dir-locals.el, but still Host: is a list of candidates, and I have to input the name of endpoint before hitting RET.

[vspinu]

The default should be most recently used endpoint. I guess it depends on the completion framework that you use. It works correctly for me with IDO.

[manuel-uberti]

I am on ivy, I'll check my settings.

[bbatsov]

I still think that adding a default endpoint is kind of orthogonal to the known endpoints (and in line with other config options with have). Even if you have one candidate but you have to select it all the time it can get somewhat annoying.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution and understanding!