nrepl

https://github.com/nrepl/nrepl || https://nrepl.org
bozhidar 2019-04-24T11:22:42.047300Z

@pez Did you manage to solve your problem? I saw the convo just now.

pez 2019-04-24T14:53:36.049400Z

@bozhidar I am writing an issue on Leiningen right now. And will volunteer to prepare the PR is they think that removing the hard dependency on nrepl is a good idea.

pez 2019-04-24T14:53:41.049600Z

But that is for the long run. In the shorter perspective I must get Calva to work with latest piggieback.

bozhidar 2019-04-24T14:54:36.051Z

So, what exactly is the problem? The latest piggieback didn’t really change anything (it was just updated to work with the current printing middleware) and seems to work fine for me.

pez 2019-04-24T14:55:27.052100Z

There is a known (if I understoof @dominicm correctly) bug with out and err messages, that I think I can live with a while. But there is also an exception thrown. Probably because I do something stupid.

pez 2019-04-24T14:56:49.053700Z

OK. I don't know for sure it stems from piggieback. But using nrepl 0.6.0, cider-nrepl 0.21.1 and piggieback 0.4.0 I get those two problems, and I don't get them fith 0.5.2, 0.20.0, and 0.3.10, respectively.

bozhidar 2019-04-24T14:57:03.054100Z

The thing that @dpsutton discussed is a middleware in cider-nrepl, but I assume you’re referring to this.

bozhidar 2019-04-24T14:58:00.054300Z

https://github.com/nrepl/nrepl/issues/119

bozhidar 2019-04-24T14:59:37.054700Z

> OK. I don’t know for sure it stems from piggieback. But using nrepl 0.6.0, cider-nrepl 0.21.1 and piggieback 0.4.0 I get those two problems, and I don’t get them fith 0.5.2, 0.20.0, and 0.3.10, respectively.

pez 2019-04-24T14:59:44.055100Z

Let me read that issue...

bozhidar 2019-04-24T14:59:56.055300Z

Looking at the stacktraces you’ve posted they certainly seem like some version mismatch.

bozhidar 2019-04-24T15:00:22.055900Z

E.g. it’s not possible to get an error about pr_values missing with cider-nrepl 0.21.

pez 2019-04-24T15:00:33.056200Z

It doesn't look exactly like what I am experiencing.

bozhidar 2019-04-24T15:01:44.056700Z

> Exception in thread “nRepl-session-8067c4a9-e320-4f8e-b84f-890c8cbf2097” java.lang.IllegalArgumentException: No implementation of method: :send of protocol: #’nrepl.transport/Transport found for class: nil

bozhidar 2019-04-24T15:01:54.057200Z

Is that the stopper problem?

pez 2019-04-24T15:02:51.058300Z

I am not sure if it is. I think I should be able to keep my repl client standing, and if so it is not a stopper.

pez 2019-04-24T15:03:30.059Z

But I haven't really tried fixing it on my end yet, Just very rececently realized the problem was there.

pez 2019-04-24T15:08:43.061100Z

I have a pretty reproducable problem with those version combos I mentioned above. With the older set of versions, out and err messages arrive as I expect them to, but with the newer set, those messages are gone. What is pr_values, btw?

bozhidar 2019-04-24T15:10:58.061800Z

It used to be the middleware that converted EDN values into strings that can go in the :value slot.

bozhidar 2019-04-24T15:11:38.062300Z

It was replaced by a similar, but more flexible print middleware.

bozhidar 2019-04-24T15:13:21.062500Z

> With the older set of versions, out and err messages arrive as I expect them to, but with the newer set, those messages are gone

bozhidar 2019-04-24T15:13:31.062800Z

They behave now the same way they always did.

bozhidar 2019-04-24T15:14:16.063400Z

That’s what happened to pr_values - https://nrepl.org/nrepl/0.6.0/design/middleware.html#_pretty_printing

bozhidar 2019-04-24T15:14:29.063800Z

I think you’ll find it very useful.

bozhidar 2019-04-24T15:15:40.064700Z

Your error indicates that somehow the print transport is nil for you, but I can’t imagine how this can happen. Maybe you’re passing some weird params in the eval request?

pez 2019-04-24T15:28:27.066200Z

Maybe I am. Will investigate deeper tonight, and let you know if I still think the problem is with the new versions. But thing is that even with the new versions the clj repl behaves as I expect, it is only the cljs repl that gives me grief.

bozhidar 2019-04-24T15:37:13.066800Z

That’s pretty weird. This has been out for 3 months now and no one has reported similar issues.

pez 2019-04-24T15:42:24.068200Z

I'll try to produce a repro. What will probably happen is that I find out what i am doing wrong.

bozhidar 2019-04-24T15:43:21.068400Z

:thumbsup: