Hi. I am reading this https://shadow-cljs.github.io/docs/UsersGuide.html#_build_tool_integration and there is this caution which I don't understand. It says use standalone version. Standalone what? Is it telling me to NOT use deps.edn to manage dependencies?

standalone as in dependencies managed by shadow-cljs.edn

you can use deps.edn but you have been warned when you run into dependency conflict issues and so on

deps.edn is a bit better dealing with those than project.clj but they can still happen

I don't know enough to understand these tradeoffs, so I am inclined to take this as gospel : ). But it feels weird, I thought deps.edn is good.

it is ...

so, what kind of dependency conflict issues can there be? I mean, if I would use only deps.edn for it? Is this something inherent to clojure/clojurescript ecosystem or just shadowcljs vs the other tools? Sorry if it's the wrong kind of question to ask.

shadow-cljs tries to ensure that you get clojurescript, closure-compiler, closure-library versions that are actually compatible

if you use deps.edn you have to manage that yourself and keep it matched

usually that works out .. sometimes it doesnt ...

I see, I didn't realize there could be incompatibilities like that. Thanks for the explanation, I think I will use the built in for now : )

I am testing version 78 and my WebWorkers got broken:

;; for /browser-repl in case the page is reloaded
  ;; otherwise the browser seems to still have the websocket open
  ;; when doing the reload
  (js/window.addEventListener "beforeunload"
    (fn []
      (when-let [s @socket-ref]
        (.close s))))

browser.cljs:406 Uncaught ReferenceError: window is not defined
    at browser.cljs:406
    at layout.js:2110
    at Array.forEach (<anonymous>)
    at Object.env.load (layout.js:2106)
    at layout.js:2142

It should probably be js/self instead of js/window in WebWorker.

@thheller Is it possible to make figwheel work with shadow-cljs

?

Not really needed, shadow-cljs has something like figwheel build in, everything works for free.

Is there a way to get the build size report as data, instead of as HTML? Would like to keep a history of build sizes etc somehow.

@pavel.klavik sorry I forgot to document since the feature is kinda unfinished. you can set :devtools {:browser-inject :name-of-the-main-browser-module}

otherwise the browser devtools get injected into the shared module but they don't work in the workers

workers now have their own devtools inject so hotreload and REPL work

@orestis there is a .shadow-cljs/release-snapshot/the-build/latest/bundle-info.edn

the build-report stuff sort has its own versioning already built-in its just not accessible from the command line

How does the repl for web workers work?

just like any other REPL ... but your editor might not support selecting it

which editor do you use?

you can disable the worker inject (to disable hot-reload/REPL) via :worker-inject false

Appears shadow-cljs does not support require outside ns, right?

yes. curious why you'd want to use it? as it doesn't allow you to do anything you can't already do in the ns

Got a conditional macro (depending on :compiler-options {:custom-stuff {...}}) choosing what goes into a build.

(prefs/if-cljs-kernel
 (require '[pinkgorilla.kernel.shadowcljs :as cljs-kernel])
 (require '[pinkgorilla.kernel.mock :as cljs-kernel]))

that doesn't work. I mean it doesn't work in CLJS either. you can't make require conditional. CLJS stops parsing at the prefs/if-cljs-kernel

Right in the middle of "migrate to shadow-cljs" work (new to it) so I might be missing something.

It worked before.

https://anmonteiro.com/2016/10/clojurescript-require-outside-ns/

hmm worked where? its not supposed to work?

Worked with figwheel and lein-cljsbuild.

hmm strange. I thought it didn't run macros. guess I was wrong

I get an error complaining that requirecannot be used anywhere other than in the ns form which is generally not true.

FWIW you can set :build-options {:ns-aliases {pinkgorilla.kernel.shadow-cljs pinkgorilla.kernel.mock}}

that would make it use the mock ns whenever the shadow-cljs one was required

Never heard of :ns-aliases, but this looks like it could do the trick.

its shadow-cljs only and not documented but meant to solve exactly this problem 😛

I'll see about require. my assumption always was that it also didn't run macros but I guess it does?

it definitely doesn't work if if-cljs-kernel was a function

require outside ns was really meant for scripts that didn't have an ns (which shadow-cljs doesn't support either)

if-cljs-kernel is a macro which acts depending custom :compiler-options

yeah I figured as much

🙏 ...

it analyzes indeed ...

I swear this function is my arch nemesis https://github.com/clojure/clojurescript/blob/6ed949278ba61dceeafb709583415578b6f7649b/src/main/clojure/cljs/analyzer.cljc#L4326 😛

Appears there is an issue with 2.8.78 😉

[:app] Build failure:
IllegalArgumentException: Wrong number of args passed to keyword: :ns-aliases
        clojure.lang.Keyword.throwArity (Keyword.java:98)
        clojure.lang.Keyword.invoke (Keyword.java:149)
        clojure.core/update (core.clj:6200)
        clojure.core/update (core.clj:6188)
        shadow.build/copy-ns-aliases (build.clj:251)

what the heck is that? 😛

oh haha ... I'm dumb 😛

@thheller I am running Cursive.

fixed in 2.8.80

@pavel.klavik do you use nrepl?

Yes, nrepl

unfortunately there is no easy way to select which runtime to talk to in nrepl

What would be another option?

you can run (shadow.cljs.devtools.api/repl-runtimes :todomvc-split) in the clojure REPL

that'd give you something like

[{:runtime-info {:runtime-type :browser,
                 :lang :cljs,
                 :build-id :todomvc-split,
                 :user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.56 Safari/537.36",
                 :remote-addr "0:0:0:0:0:0:0:1"},
  :last-ping 1575462862870,
  :last-pong 1575462862872,
  :last-msg-received 1575462862872,
  :last-pong-runtime 1575462862871,
  :connected-since 1575462831938,
  :runtime-id "69b97643-1982-4922-b7cf-375bfa7496a8"}
 {:runtime-info {:runtime-type :worker,
                 :lang :cljs,
                 :build-id :todomvc-split,
                 :user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.56 Safari/537.36",
                 :remote-addr "0:0:0:0:0:0:0:1"},
  :last-ping 1575462863873,
  :last-pong 1575462863874,
  :last-msg-received 1575462863874,
  :last-pong-runtime 1575462863874,
  :connected-since 1575462832785,
  :runtime-id "ebd1298e-0201-430b-889b-30bc97eb0103"}]

hmm nevermind

it doesn't work in nrepl 😛

in theory you should be able to use (shadow.cljs.devtools.api/repl :the-build {:runtime-id "that-runtime-id-from-above"})

I will test it later. But i wouldnt mind to switch to some other repl, Cursive now supports SocketREPL as well.

Not very familar with repls, yet

I think it doesn't really work anywhere right now

picking a runtime-id by hand is a bit annoying

would be neat if the editor just supported selecting runtimes. I did some experiments with that in the shadow-cljs UI but its all unfinished

I see, it is ok to do this when you dont do it that frequently.

Is it possible to run multiple instances with different runtimes?

"instances"? of what?

Shadow is running nrepl on a port, and i open a window in Cursive which connect there. So the question is whether i can have multiple windows, one for each runtime?

you can open as many nrepl connections as you want yes. and each can talk to a different runtime yes.

I see. Than one could probably bind some keys to do the switching, I will take a look later.

thats the idea behind runtimes as there can always be multiple in a browser env. ie. opening the page in chrome/firefox. or just using workers since they run in their own little world

I see. Make sense, running a page in multiple tabs.

Cornerning hot reloading, I can basically remove restarting the web workers now?

maybe? I'm still testing this myself. in theory hot-reload should be fine now

Ok, I will test it later and will let you know.

@thheller Would you mind rolling to 2.8.80? 😂

hmm? I release 2.8.80 45 minutes ago or so?

Just pulled and saw 2.8.79 came out today. Did not expect that I was already lagging. 🙂

yeah I just pushed the fix to the problem you ran into with :ns-aliases

Started working on a PR alfready. Did not expect you to react at that pace. Thanks Thomas!

If I pass two target arguments to the shadow-cljs watch function, shadow-cljs shuts down once the targets are compiled. e.g.:

20:52 $ shadow-cljs watch dev test
shadow-cljs - config: /Users/scott/code/xxxx/shadow-cljs.edn  cli version: 2.8.52  node: v12.11.1
...
shadow-cljs - watching build :dev
shadow-cljs - watching build :test
[:dev] Configuring build.
[:test] Configuring build.
[:test] Compiling ...
[:dev] Compiling ...
[:dev] Build completed. (240 files, 1 compiled, 0 warnings, 3.65s)
[:test] Build completed. (240 files, 1 compiled, 0 warnings, 3.67s)
shutting down ...
Worker shutdown.
Worker shutdown.

thats not normal. you may be getting killed by the system?

do you use deps.edn/project.clj? you cut that bit of information out

also try upgrading since I did a bunch of fixes which reduced resource use a bunch. 2.8.80 is latest

it looks like you use macOS which seems to kill the shadow-cljs process sometimes for consuming too much memory

you can set :jvm-opts ["-Xmx1G"] in shadow-cljs.edn to constrain that

but upgrading will most likely take care of that

Thank you so much. That’s a good point about memory consumption. I will upgrade and if that doesn’t work, try tweaking the memory settings. Cheers for the incredibly fast response. shadow-cljs really is a fantastic project and a pleasure to use.

confirmed: Upgrading to 2.8.80 worked. No memory constraints required. Thanks again 🙂

man this had me thinking i was going insane https://github.com/thheller/shadow-cljs/issues/612

no idea whats up with that but it caused upstream in CLJS

fun

i guess i can file a jira

I have a stupid question on how to use "deps.cljs" I have a project https://github.com/pink-gorilla/leaflet-shadow/blob/master/src/deps.cljs

In the project I use a few npm dependencies and then I publish the project to clojars

If now other projects require leaflet-shadow when I define a deps.cljs file

will then shadow cljs automatically get the transient npm dependencies ?

Provided the projects that use my project use shadow-cljs

Essentially my library is a slim wrapper around react-leaflet. So I think the right place to publish this lib to is clojars...

I want to avoid having to remember to require transient npm dependencies

Is this possible?

Hi there! I'm trying to rewrite a project that uses webpack+react to ClojureScript with Shadow-CLJS. The thing is, I don't want to do a "big rewrite", and instead I would like to start migrating existing react components to CLJS little by little.

I though that using :node-library's target would allow me to integrate with webpack and all other tools on the React side, but its not working (because it needs some of the Node code, like readFileSync and others). I could use :npm-module but then I loose the ClojureScript REPL. Is there any alternative?

We're doing something similar with :npm-module as a target, going outside-in (i.e. from the leaves). We still use the REPL

These had to be added on the js entrypoint: (`cljs/` ` is a webpack alias pointing to the cljs build folder)

window.CLOSURE_BASE_PATH = '/javascripts' // Should match your assets location
require('cljs/shadow.cljs.devtools.client.browser') // REPL
require('cljs/shadow.remote.runtime.cljs.browser') // Shadow's Inspect

Great, I'm gonna test right now 🙂

Just for the record, if I add [shadow.cljs.devtools.client.browser] on one of the required namespaces, it works too 🙂

@hoertlehner yes that should work fine?

@mauricio.szabo :node-library presumes node not webpack + browser.

@thheller just a file in the src path named deps.cljs ?

yes

but it only works in libraries

ok

but I dont add a reference to that file anywhere?

no

just has to be in the classpath root in a library

Is :npm-module able to run before/after load code?

I'm not seeing it working on my setup

(in fact, I'm not sure the hot reload is working either)

@mauricio.szabo in general hot-reload and REPL are a bit constrained when running through webpack. it sometimes work if you require the REPL ns separately from JS

something like if (process.env.NODE_ENV === "development") { require("./path-to-output/shadow.cljs.devtools.client.browser") }

@thheller cool - got transient dependencies going with deps.cljs -> very cool feature!!