Fork me on GitHub
#chlorine-clover
<
2020-05-20
>
fabrao01:05:35

hello all, sorry my dumb question but is possible to start more than one repl session?

fabrao01:05:56

with Chlorine?

fabrao01:05:31

or connect to other repl session if already connected?

seancorfield01:05:03

You can only be connected to one REPL at a time but it is easy enough to switch between REPLs. I do that all the time.

fabrao01:05:35

you disconnect and connect to the other?

seancorfield01:05:20

Yes. The REPL state remains the same so it's easy to just jump between REPLs as you need.

seancorfield01:05:47

I often switch between a locally started REPL, a live app running locally, and a live app running on another server.

seancorfield02:05:51

Just upgraded to 0.7.0 and started rewriting my init.coffee file in Clojure using the new extension facility -- very nice!

seancorfield02:05:32

Although I don't think editor/get-selection works? I've tried a couple of functions based on it and when I execute them, nothing happens at all (well, Atom crashed one time).

seancorfield03:05:43

Looks like get-selection is returning {:text nil :range nil} regardless of whether I have text selected or not.

seancorfield04:05:31

I have all of my init.coffee file converted over to chlorine-config.cljs and everything works, modulo get-selection not working.

mauricio.szabo13:05:35

Oh, I'll look at get-selection. Let me see if there's anything I missed

mauricio.szabo13:05:23

Oh.... get-selection is using a completely wrong destructuring... will fix it.

dvingo16:05:38

Hello all, I have a similar question to fabrao above - when working on a fullstack app there will be at least 2 repls I'll want open at the same time - clj and cljs, and often more - another cljs for a node.js server for example. Is there a recommended approach to a setup like this with chlorine?

seancorfield16:05:14

@mauricio.szabo That works perfectly now! Thank you!

seancorfield16:05:35

(go click my Chlorine config link again to see the updated version)

seancorfield16:05:35

I've removed all the init.coffee code now and I'm doing all of it via cljs -- is there a way to show Atom info/error messages? I switched those to plain println calls for now.

mauricio.szabo17:05:21

@danvingo when I have to work with Clojure and ClojureScript, I use shadow-cljs together with lein or cli, and start the compilation process on the Clojure REPL

mauricio.szabo17:05:37

That way, I can migrate between Clojure and ClojureScript with ease

mauricio.szabo17:05:01

But... two different Clojure REPLs the thing starts to become complicated. I never had to use something like this, but I'm open to suggestions on how to implement it

mauricio.szabo17:05:01

(or, to be more specific, how to present it - like, how do you select which files are evaluated by which REPL, which REPL is the "active" one, and so on... things are quite complicated today on how things are implemented, specially when you consider the semantics of ClojureScript REPLs)

dvingo17:05:32

the obvious UX idea that comes to mind is having UI tabs for each repl, the active tab is the active repl - the nice to have is a keyboard shortcut to switch between them.

dvingo17:05:21

(i'm just throwing out ideas, I have no idea how complicated, or not, this would be to implement 🙂 )

seancorfield17:05:25

Yeah. That's why I'm happy to explicitly switch between REPLS as needed. Especially since I'm switching between live processes.

mauricio.szabo17:05:39

Well, you can have a situation when there's no REPL tab active at all 😄

dvingo17:05:03

maybe i just need some pointers on how to switch between repls

seancorfield17:05:11

And I have my REPL tab mostly minimized

seancorfield17:05:58

Having Chlorine remember recent REPLs and offer a drop down to switch would be less work than disconnecting and then connecting each time (and having to enter the port each time -- since all of my REPLs are over ssh tunnels or else local on different ports)

seancorfield17:05:15

For me switching is ctrl-; e ctrl-; y (then type the port and press return)

mauricio.szabo17:05:00

Yes, this approach can work fine - probably it'll be what I end up doing. Maybe add as a second command, so you don't accidentally disconnect your REPL 😄

dvingo17:05:20

yea the drop down of recent connections would work for me too

seancorfield18:05:44

@mauricio.szabo Not sure if you noticed my additional question among all of that: with the new cljs extension stuff, is there a way to do Atom info/error popups?

mauricio.szabo18:05:05

Oh, no, I didn't. Yes, there's a way, it's not yet documented, so let me see if I can reproduce here...

mauricio.szabo18:05:19

Yes, @seancorfield, it's not yet documented (some APIs can change on the future) but there's already a way to call any callbacks that Chlorine registers when it connects to a REPL. For example, the code below will notify on Atom the current selected var:

(defn info []
  (let [{:keys [text]} (editor/get-var)]
    (editor/run-callback :notify {:type :info :title text})))

seancorfield18:05:15

Oh this is so much nicer than writing CoffeeScript or JS to extend the editor! Thank you @mauricio.szabo -- this is truly awesome work!

mauricio.szabo19:05:42

Glad that you like it 😄. I'm also using it on some experiments, and as soon as I remove those strange internal state like Browsable and friends, I believe it'll be easier to hack more complex things!

fabrao20:05:48

Hello all, when I´m in deps.edn and I save the file, in Chlorine console shows

java.lang.IllegalArgumentException: "No value supplied for key: true"
in clojure.lang.PersistentHashMap.create (PersistentHashMap.java:77)
in clojure.core/hash-map (core.clj:387)
in clojure.core/hash-map (core.clj:379)
in clojure.lang.RestFn.applyTo (RestFn.java:137)
in clojure.core/apply (core.clj:665)
in clojure.core/load-lib (core.clj:5934)
in clojure.core/load-lib (core.clj:5928)
in clojure.lang.RestFn.applyTo (RestFn.java:142)
in clojure.core/apply (core.clj:667)
in clojure.core/load-libs (core.clj:5985)
why? The "Should we refresh namespaces when save a file (Clojure Only)?" is on

fabrao20:05:34

is Chlorine trying to evaluate deps.edn file?

mauricio.szabo20:05:24

With simple refresh, and evaluate on save, probably... I don't remember making a code to disable it for .edn files, will open an issue 🙂

seancorfield21:05:48

Seriously, just turn refresh off and get used to a tighter REPL-driven workflow instead. Auto-reload/-refresh is a can of worms.

4
jaihindhreddy21:05:17

I've been a fan of the reloaded workflow ever since Stuart Sierra's talk, but have increasingly been simplifying the tools I use, as per advice from you and Stuart Halloway. Badly written component impls may create dangling resources on reload, and reloads might get slow in large enough systems. I'm sure there are many more worms from that can. Can you please point out a couple more?

seancorfield22:05:59

The main thing I've seen is refresh-based workflows which sometimes seem unable to load code after removing definitions that it depends on. When beginners report these problems, it's almost impossible to tell what they've done "wrong" and a restart of their REPL nearly always fixes it.

seancorfield22:05:58

When I have tried various reload/refresh workflows in the past, they nearly always break in hard-to-debug ways -- and tooling should make life easier not create additional problems.

💯 4
seancorfield22:05:01

Since I've moved to a simpler REPL workflow, where I control all of the evaluation/removal/etc, I hardly ever get into a bad state even with a REPL running for weeks at a time.

😲 4
seancorfield22:05:55

Although funnily enough I did break my REPL state last week -- for the first time in many months. I think I got sloppy about eval'ing changes and then I removed a namespace to force some global state there to be recreated (`defonce`, protocol, record, something like that), and then a (require ... :reload-all) broke and it was just easier to restart the REPL than spend time figuring out the incantation needed to get things back to a sane state...

😄 4
seancorfield22:05:24

...if it happens again any time soon, I'll probably spend a bit more time figuring out a simple automation for the solution 🙂