Clojurians Log v2

Anyone got a solution to this:

Uncaught TypeError: goog.net.jsloader.load is not a function
    at figwheel$client$file_reloading$reload_file_in_html_env (file_reloading.cljs?rel=1516980062278:208)
    at figwheel$client$file_reloading$reload_file (file_reloading.cljs?rel=1516980062278:250)
    at figwheel$client$file_reloading$blocking_load (file_reloading.cljs?rel=1516980062278:269)
    at file_reloading.cljs?rel=1516980062278:279
    at file_reloading.cljs?rel=1516980062278:277
    at figwheel$client$file_reloading$state_machine__59709__auto____1 (file_reloading.cljs?rel=1516980062278:277)
    at figwheel$client$file_reloading$state_machine__59709__auto__ (file_reloading.cljs?rel=1516980062278:277)
    at cljs$core$async$impl$ioc_helpers$run_state_machine (ioc_helpers.cljs?rel=1516980058153:35)
    at cljs$core$async$impl$ioc_helpers$run_state_machine_wrapped (ioc_helpers.cljs?rel=1516980058153:39)
    at ioc_helpers.cljs?rel=1516980058153:48

@luke any ideas ⬆️ ? I'm using the latest versions of all Arache deps.

org.arachne-framework/arachne-figwheel {:mvn/version "0.2.0-master-0032-bb33280"}
org.arachne-framework/arachne-pedestal {:mvn/version "0.2.0-master-0064-fb2edbc"}
org.arachne-framework/arachne-sass {:mvn/version "0.2.0-master-0034-1769b93"}
org.arachne-framework/pedestal-assets {:mvn/version "0.2.0-master-0019-580de8f"}

I've also had to disable the ::webjars config because I get a spec failure with some ZIP object in place of a directory.

@jcf hmm, I haven’t ever seen that before. Any chance you could post your config script including what your cljs compiler settings are?

Of course.

@jcf uploaded a file: https://clojurians.slack.com/files/U06FTAZV3/F8YMVAPJL/config_clj.txt

That's my config, including the cljs-opts I pulled out of the enterprise SPA template.

and is this in dev or whitespace mode?

er, sorry, dev or advanced?

And here's the exception from webjars not being happy:

ExceptionInfo Call to #'arachne.fileset/add did not conform to spec:
In: [1] val: #object[com.sun.nio.zipfs.ZipPath 0x55b26a2b "/META-INF/resources/webjars"] fails spec: :arachne.fileset.specs/directory at: [:args :directory] predicate: (fn [f] (and (instance? java.io.File f) (.isDirectory f)))
 {:clojure.spec.alpha/problems [{:path [:args :directory], :pred (clojure.core/fn [f] (clojure.core/and (clojure.core/instance? java.io.File f) (.isDirectory f))), :val #object[com.sun.nio.zipfs.ZipPath 0x55b26a2b "/META-INF/resources/webjars"], :via [:arachne.fileset.specs/directory :arachne.fileset.specs/directory], :in [1]}], :clojure.spec.alpha/spec #object[clojure.spec.alpha$regex_spec_impl$reify__2436 0x6d2bdcdc "clojure.spec.alpha$regex_spec_impl$reify__2436@6d2bdcdc"], :clojure.spec.alpha/value (#arachne.fileset.impl.TmpFileSet{:tree {}, :blob #object[java.io.File 0x47780f5c "/tmp/arachne-fs1560548959238895091"], :scratch #object[java.io.File 0x31a747 "/tmp/arachne-fs3449467318418299623"], :cache #object[java.io.File 0x6b214f83 "/tmp/arachne-fs283219324983436992"]} #object[com.sun.nio.zipfs.ZipPath 0x55b26a2b "/META-INF/resources/webjars"]), :clojure.spec.alpha/args (#arachne.fileset.impl.TmpFileSet{:tree {}, :blob #object[java.io.File 0x47780f5c "/tmp/arachne-fs1560548959238895091"], :scratch #object[java.io.File 0x31a747 "/tmp/arachne-fs3449467318418299623"], :cache #object[java.io.File 0x6b214f83 "/tmp/arachne-fs283219324983436992"]} #object[com.sun.nio.zipfs.ZipPath 0x55b26a2b "/META-INF/resources/webjars"]), :clojure.spec.alpha/failure :instrument, :clojure.spec.test.alpha/caller {:file "pipeline.clj", :line 135, :var-scope arachne.assets.pipeline/classpath-fileset, :local-fn fn}}
	clojure.core/ex-info (core.clj:4739)
	clojure.core/ex-info (core.clj:4739)
	clojure.spec.test.alpha/spec-checking-fn/conform!--2938 (alpha.clj:131)
	clojure.spec.test.alpha/spec-checking-fn/fn--2943 (alpha.clj:139)
	arachne.assets.pipeline/classpath-fileset/fn--32336 (pipeline.clj:135)
	clojure.core.protocols/fn--7852 (protocols.clj:168)
	clojure.core.protocols/fn--7852 (protocols.clj:124)
	clojure.core.protocols/fn--7807/G--7802--7816 (protocols.clj:19)
	clojure.core.protocols/seq-reduce (protocols.clj:31)
	clojure.core.protocols/fn--7833 (protocols.clj:75)
	clojure.core.protocols/fn--7833 (protocols.clj:75)
	clojure.core.protocols/fn--7781/G--7776--7794 (protocols.clj:13)

Dev.

I have (def dev? (constantly true)), and my cljs-opts:

(def cljs-opts {:main 'usr.client
                :optimizations (if (dev?) :none :advanced)
                :asset-path "/js/out"
                :output-to "js/app.js"
                :output-dir "js/out"
                :source-map-timestamp true})

So optimisations will be :none.

Commenting out the ::webjars input-dir and related pipeline entries circumvents the spec error with a ZipPath.

I'm not using any webjars so don't need that for now. I can open an issue/PR that changes the spec so zipped ditties are accepted…

right, so that error is (fairly) straightforward, apparently a directory in a Zip is not considered a directory and therefore can’t be used in Arachne’s asset pipeline. That definitely is something that should be enhanced

although I could have sworn I had that working…

how private is this project? Is it the kind of thing you would be willing to clone and let me reproduce locally? I will continue to think about it but honestly with FactUI and some other client work my head has been out of the asset pipeline for a while.

Yeah, the Zip bit seems straightforward enough to fix if ZipPaths are supported.

I'll open up the repo.

@luke https://github.com/jcf/usr

Not production quality! 🙂

Oh, and I'm doing some gnarly stuff with deps.edn mixed with Leiningen because I want to try out something simpler (ultimately Arachne + tools.deps).

awesome

I need to go pick up my kid right now but I will to reproduce your problem this afternoon.

Cool! I'll keep an eye on Slack, and keep at trying to fix this myself.

Thanks, @luke!

May be a relevant issue: https://github.com/adzerk-oss/boot-reload/issues/121

@luke fixed it here: https://github.com/jcf/usr/commit/8281da793b18a95aa3bfcdb1b9a2c76f631b2576 Looks like something's been fixed in a recent release of Figwheel (I read that load was replaced with safeLoad but haven't dug deeper).

Upgrading Figwheel fixed the problem with jsloader.load not being around but now I get this:

@jcf uploaded a file: https://clojurians.slack.com/files/U06FTAZV3/F8ZCB9B50/2018-01-26-18_35_28-11aa9cc00a96.png

Too many open files (I guess something to do with the fs abstraction inspired by Boot?).

THen clicking the error spits out this:

heads_up.cljs?rel=1516991443932:39 Uncaught TypeError: Cannot read property 'figwheelEvent' of undefined
    at heads_up.cljs?rel=1516991443932:39
    at core.cljs:4353
    at cljs.core.keep.cljs$core$IFn$_invoke$arity$2 (core.cljs:4353)
    at cljs.core.LazySeq.sval (core.cljs:3380)
    at cljs.core.LazySeq.cljs$core$ISeqable$_seq$arity$1 (core.cljs:3438)
    at cljs.core.LazySeq.cljs$core$ISeq$_first$arity$1 (core.cljs:3404)
    at cljs$core$first (core.cljs:1220)
    at figwheel$client$heads_up$get_dataset (heads_up.cljs?rel=1516991443932:39)
    at HTMLDivElement.figwheel$client$heads_up$heads_up_onclick_handler (heads_up.cljs?rel=1516991443932:43)

If we've exhausted the number of files we're allowed to open, all bets are off. 😉

List of all the files opened by this Arachne-managed process:

I can up my ulimit etc.

If I change the string in my views/Layout a few times I can reliably max out the open file limit.

$ ulimit -Hn
4096

@jetmind has left the channel

@donaldball has left the channel

Is there any way to update a fact in the session?

in factui session

@luke

I have a rule that says if A is true then insert-logical B. And, at first, one event-handler inserts the fact that A is true, so B gets inserted. Now, in another event-handler, I want to either set A to false or retract A, so that B can automatically be retracted.

@jcf ok, let me get my head around the open files issue. that came up on another project a couple months ago and we found a workaround, I’ll refresh my memory on that end get back to you.

@amarjeet that should work already if the fact is cardinality one; it will be replaced and the logical rule effect will happen

if it’s cardinality many then the old fact is still true as well, so nothing will be retracted

oh okay, I wasn't setting the cardinality option. I just used (f/transact-logical! [{:db/ui true}])

yeah facts are cardinality many by default

ohho

unless you have a schema setting them to one

got it, thats helpful. thanks much luke

@jcf ok I actually fixed a similar issue a while back: https://github.com/arachne-framework/arachne-fileset/commit/8893b28ad1adfe2a4c9bf0a32a27b6c483004cd5#diff-81355e05491a6b7a3ef3d7f8e1bc3d5e

It’s possible I forgot/didn’t make a release of that for some reason

what version of arachne-fileset is your project using? (you can check this using lein deps :tree or something like that

@luke https://github.com/jcf/usr/blob/master/deps.edn gets slurped into project.clj. I’ll copy and paste the version now…

@jcf right: the arachne-fileset lib is a transitive dependency of the Arachne assets module

Oh, sorry. Didn’t see you wanted a transitive dependency.

yeah np

you want version 1.6.1, 1.6.0 has the “open files” bug

I’m AFK at the mo (it’s Friday night and 19:55 here).

ah, k 🙂

for now you can just manually include the newest version of org.arachne-framework/arachne-fileset

I’ll add an explicit dependency to my deps.edn just like Figewheel Sidecar.

I’ll cut a release of Arachne proper upgrading the dependency soon

Awesome. Thanks, @luke.

and include the sidecar version bump which it sounds like I’ll need as well

might be a few days, I will probably port Arachne to deps.edn at the same time.

since I now prefer that approach, since it’s out

https://github.com/jcf/usr/commit/8281da793b18a95aa3bfcdb1b9a2c76f631b2576

Added Figwheel Sidecar in master.

woot, thanks

I’ll roll that into the core project

tools.deps is a breath of fresh air. Loving the recent output from the Cognitect guys(I guess mainly Alex??)

Rich has the ideas, Alex does the work 😉

How do I ‘retweet’ a Slack comment? 🤣

you don’t, which is why I can say such things here