Support :init option when creating a repl

[jaidetree]

Support :init option when creating a repl

[borkdude]

I would say that's overkill

[borkdude]

:init would be a function, like in clojure.main/repl

[jaidetree]

Oh my mistake, that means it would be like this?

(require ‘[promesa.core :as p])

(repl/repl
  {:init #(apply require ‘[[promesa.core] :as p]
                           [clojure.repl :refer [doc]]
                           [cljs.pprint :refer [pp pprint]])})

Then the implementation should run that within a promise?

[borkdude]

the function can return a promise or an ordinary function and will then be chained onto the rest of the stuff here:

nbb/src/nbb/impl/repl.cljs

Line 203 in b9c2631

(.then (fn []

[jaidetree]

That helps a lot! Last question: How should it deal with the optionality?

(-> ...
    (.then (if (ifn? (:init opts)) (:init opts) identity)))

[borkdude]

I'd say:

(let [init (:init opts identity)]
  (.then (...) init)

[jaidetree]

Should that let also contain a runtime check like?

(when-not (ifn? init)
  (js/throw (js/Error. ":init must be a function")))

[borkdude]

@eccentric-j Maybe this would also be nice to have for the nREPL server? But I'll close the existing issue for now.