Clojurians Log v2

Here's the full code as a reference: https://github.com/BizMentorAI/nace-lookup/blob/master/src/workers/autocomplete/autocomplete.cljs#L37 And the compiled version: https://github.com/BizMentorAI/nace-lookup/blob/master/public/workers/autocomplete/autocomplete.js

please make a smaller repro with result being explicit?

Note that it typically works, just in this one case it doesn't

Yes I'll try.

@borkdude OK it's something to do with the transducer:

(defn empty-transducer [xf]
  (fn
    ([] (xf))
    ([acc input] acc)
    ([result] result)))

(js/console.log "Empty" (clj->js (into [] empty-transducer data)))

and does it work in regular CLJS?

or in #nbb?

No idea – yet.

$ npx nbb
Welcome to nbb v1.2.173!
user=> (defn empty-transducer [xf]
  (fn
    ([] (xf))
    ([acc input] acc)
    ([result] result)))

(js/console.log "Empty" (clj->js (into [] empty-transducer data)))#'user/empty-transducer
user=> (js/console.log "Empty" (clj->js (into [] empty-transducer data)))

please specify an example without any free variables

(defn empty-transducer [xf]
  (fn
    ([] (xf))
    ([acc input] acc)
    ([result] result)))

(js/console.log "Empty" (clj->js (into [] empty-transducer [])))

Just with an empty array.

works the same in nbb, a CLJS data structure is logged

Yep.

The type seems to be an Empty

ah it's a TransientVector actually

I think your reducing function should still call persistent!?

or there is a bug in CLJS, could be

OK that's the first time I'm hearing about persistent!.

(defn empty-persistent-transducer [xf]
  (fn
    ([] (xf))
    ([acc input] acc)
    ([result] (persistent! result))))

Can't see much documentation about what it actually does though.

it turns a transient object into a persistent one

OK not sure what that means, but hey, it works, I fixed the code where I had the original problem. Thanks!

👍

@jakub.stastny.pt_serv It's possible to use (:require-macros [my-macros :refer [inline-resources]) but this is very limited ATM. The macro namespace must live under the src directory as a .cljc file. When you're running cherry with Node.js the macro namespace will also be executed in node

That's a good news. Thanks.

At least this is how it worked in #squint but not sure if I ported this to cherry... I can check tomorrow

I'm going to have a go at it in a few mins, I'll shout if it's broken 😄

It looks like it's working, generally, but I'm getting Error: Could not resolve symbol: slurp. • https://github.com/BizMentorAI/nace-lookup/blob/master/src/macros.cljc • https://github.com/BizMentorAI/nace-lookup/blob/master/src/workers/autocomplete/autocomplete.cljs#L5 Is that a known issue (missing slurp)? Is there a workaround?

I know Cherry's experimental, but really so far it works great so I want to keep using it. It's much easier to reason about for me and the fact that it doesn't need nearly 1GB of MEM is pretty important to me.

yes, in CLJS/node slurp is not available, so you can use (:require ["fs" :as fs]) + fs/readFileSync

Right, got it.