Let me check but likely yes, I've been happy with the refactored feature over clj/s :)

Awesome, glad to hear it. Let me know if we have more steps there.

I'm running emacs via ssh on a pretty dinky server 1G RAM and 1VCPU.

It's possible that I also have some large jars on the classpath from including binaries resources.

versions:

;; CIDER 1.10.0 (Sant Cugat), nREPL 1.0.0                                                                                                                       
;; Clojure 1.11.1, Java 18.0.2-ea   
Ubuntu linux

I've had this issue for a while, but I thought it was because I was running old versions. However, I just ran into issue again with the same project on a fresh setup.

Yikes, certainly shouldn't happen. Are all of that dependencies public, or is there some private stuff?

Most of it's public

If you just paste the public part of the dependencies, I'll take a look what's going on

https://github.com/phronmophobic/slackcommands

everything there should be public except com.phronemophobic.discord {:local/root "../discord"}

Cool, I'm in the middle of refactoring Compliment, so this is very apposite

Actually, let me try and reproduce without that dep.

Truth be told, a large category of Clojure programs are subject to fail with just 1GB RAM knowing Alex, I'd bet that this will be fixed, but in general limited official support can be expected from CIDER under constrained resources

Everything else works fine. I also added 4G of swap which isn't great for performance, but it doesn't need to do anything quickly.

...A good example being clj-kondo, if you want to analyze the whole classpath it can take 4GB with no possible workaround. I'm fine with that.

Oooh, I overlooked the 1Gb part. Still, this should certainly be enough for Compliment, I bet there is a bug somewhere and this is not a natural limit

Yea, completely turning it off is an acceptable workaround for me.

It's just my muscle memory hoses the server 😛.

Anyway, just thought I'd make a report since it seems reproducable.

Ok, so I was able to reproduce without the private dependency.

Any particular prefix that you try to trigger the completion with?

there's an example in the error log, but it's usually when I'm in slackcommands.util and trying to call save-large-png

so something like save-l

Hm, could not reproduce right away with my default heap sizing, let me try downsize it

Hm, works fine on my machine with Xmx1g. Could you check if it still reproduces if you e.g. clear the target directory? I see it isnt' listed in paths so it shouldn't matter but who knows 🤷

the target directory?

I don't think there is one.

Alright

There is indeed a good amount of classes on the classpath (100k+) but it goes through without a hitch on my laptop

Could you please try this instead of hitting Tab?

(require '[cider.nrepl.inlined.deps.compliment.v0v4v4.compliment.utils :as cutils])

(count (cutils/namespaces&files-on-classpath))

1168

But it finished, that's interesting

What if you trigger completion now?

ok, now it's working

let me double check it

I wonder if it's a classic concat / mem problem

I thought of concat as well. Gonna remove each and every of those for sure

However, concat was more about stackoverflow I think

oh, ok. just calling that code doesn't fix it. I think I just realized the problem.

When I want a completion, I mash a combination of tab and ESC+TAB several times. 😆

Which I think overloads it.

oh weird. at some point it caches the response and it works fine again?

It does cache the list of all files/classes/namespaces

Still weird that you run into this because it flies almost instantly. But that's on a decent machine

In a setup like yours, I can imagine it being slower so that you can squeeze in multiple invocations of that stuff

Guess I gonna protect those fat one-time calls with a mutex just in case.

I was able to reproduce the OOM with the following steps: • open slackcommands.util • cider-jack-in • C-c M-n M-n (cider-repl-set-ns NS) • C-c C-k cider-load-buffer • switch to cider terminal • type save- • tap ESC+TAB 3 times, then immediately TAB 3 times

mashing the different forms of tab is just muscle memory :man-facepalming:

Would you try this for me? 🙂

(require '[cider.nrepl.inlined.deps.compliment.v0v4v4.compliment.utils :as cutils])

(dotimes [_ 100]
  (.start (Thread. #(println (count (cutils/classes-on-classpath))))))

I didn't include it in the error log, but I also end up with several messages in the cider log:

ERROR: Unhandled REPL handler exception processing message {:op complete, :ns slackcommands.util, :prefix save-la, :context :same, :enhanced-cljs-completion? t, :session 3f3f37d6-2df9-4b11-868b-98810d8eda29, :id 85}

Great, it hung up on me as well, finally!

Ok, running the above snippet. The CPU is pegged and OOM exceptions are being printed sporadically.

Now I have something to work with. Thanks for the help!

Thank you!

Thanks for the quick response! Glad my impatient mashing is good for something.

Just curious if this issue was ever addressed. It's not critical, but it would be helpful.

From memory, @U06PNK4HG fixed this and it made it into cider-nrepl latest as bundled with cider.el latest stable

cool. I'll try it out!

Indeed, this should no longer happen with the latest CIDER.

awesome! looking forward to trying it out!

Seems to be working! Thanks again! 🎉