I think I got it working...

@thheller re: your comment early about advanced compilation. I managed to get the project to compile in advanced mode. After fiddling with it for a while, it seems to be working. I'm passing messages between workers. It just seems way, way faster. So I could be wrong. But I have plenty of (when (= "this-workers-name" (get-id)) ..., where computations are only supposed to take place in that workers context... All printing from workers appears to have broken, but I routed messages back to the ui thread, showing some computation done on the workers... I think it's working.

granted, I think you're right that certain things might get clobbered here and there. Need to explore.

It's just, like, 10 or 20 times faster.. So I'm not sure if it's actually not working or it's just way more performant with a smaller code base on workers.

What's the suggested way to pass url parameters to reagent components?

I found an answer http://stackoverflow.com/questions/38469327/what-is-correct-way-to-broadcast-secretary-url-parameters-to-reagent-component) and just wondering if it's still valid, i.e. putting the param into a db which the component can refer to. It seems counter intuitive to store in shared location rather than simply passing it directly to the component.

@mikeb you might want to ask your questions in the #reagent channel, or perhaps #re-frame

But in general, if you are using re-frame, then there's no problem putting the id into app-db. It is part of the application's state, afterall.

But is that the preferred way? Somehow it just doesn't feel right putting a parameter like that in a shared global var, rather than just passing it directly to the component. Do other cljs ui libs generally work the same way?

In a re-frame app ALL application state is held in app-db. We actively resist "distributing state" around the components.

If you have an OO mindset, which encourages partitioning, this can seem very wrong

But that discomfort will pass, I assure you. You'll later be quite delighted by the simplicity it brings. https://github.com/Day8/re-frame/blob/master/docs/ApplicationState.md#the-benefits-of-data-in-the-one-place

@mikeb ^^^

Thanks!

Hello everyone, I'm trying to use a js regex into my clojurescript code (the regex in question: http://regexr.com/355hb). I tried to escape some characters, but I cannot for the life of me get the right regex in the generated javascript code. Is there a way I can use the raw js regex (wihtout adding it as a foreign lib if possible)?

It's a bit infuriating when I know exactly the regex I want to get generated (and yes, I tried using (js* ...))

i don't think that is a valid js regexp:

new RegExp("\(?(?:(http|https|ftp):\/\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\/]?[^\s\?]*[\/]{1})*(?:\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?")

Uncaught SyntaxError: Invalid regular expression: /(?(?:(http|https|f....

oh, i see the tool is executing that in the browser so it must be - seems the problem is in copying it out of the tool

if you can get the regex to execute in JS then you should be able to just do a (js/RegExp. "regex-here")

If you use string, you need to escape \ and such

Use regex literal instead #"...", no escaping needed

You are right, it does not work when you do a new RegExp(...)

@danvingo but it works if you use the js literal regex syntax /.../

like this /\(?(?:(http|https|ftp):\/\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\/]?[^\s\?]*[\/]{1})*(?:\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?/

yea @juhoteperi is right

did that work @chpill ?

replacing / / with #""

@danvingo @juhoteperi sadly no. The thing generated is not even a valid js regex

(re-seq #"\(?(?:(http|https|ftp):\/\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\/]?[^\s\?]*[\/]{1})*(?:\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?" "URLs within text:
* Create Share Links to send your expressions to co-workers or link to them on Twitter or your blog [ex. ].")

that's working for me

verified in klipse: http://app.klipse.tech/

lol, I think this is evidnece someone needs ot invent a DSL that replaces regexes 🙂

@danvingo can you try that in a non self hosted cljs repl? I really need to check my sanity 😞

sure

crazy idea 🙂. But would be really nice to see a series on clojurescript on http://egghead.io. Maybe it would boost the adoption from javascript developers. Purescript & Elm seem to be there.

I'm getting Uncaught SyntaxError: Invalid regular expression flags

@juhoteperi yes exactly!

@juhoteperi if you look at the generated js regex, you will see there is a closing / that appears in the middle

this might be related to / needing escaping in JS regex literal but not in Cljs

verified, also getting that error in Clojure cljs:

(re-seq #"\(?(?:(http|https|ftp):\/\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\/]?[^\s\?]*[\/]{1})*(?:\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?" "URLs within text:
* Create Share Links to send your expressions to co-workers or link to them on Twitter or your blog [ex. ].")
#object[SyntaxError SyntaxError: Invalid regular expression flags]
   figwheel$client$utils$eval_helper (jar:file:/Users/dvingo/.m2/repository/figwheel/figwheel/0.5.9/figwheel-0.5.9.jar!/figwheel/client/utils.cljs:69:5)

So is it a bug in clojurescript itself?

Yeah, / is escaped in JS literal format because that uses / for start and end, but Cljs doesn't

Not sure if this is bug or not

output by cljs compiler:
/\(?(?:(http|https|ftp):\\/\\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\\/]?[^\s\?]*[\\/]{1})*(?:\\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?/

JS regex:
\(?(?:(http|https|ftp):\/\/)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=\/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([\/]?[^\s\?]*[\/]{1})*(?:\/?([^\s\n\?\[\]\{\}\#]*(?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#]*)?([\.]{1}[^\s\?\#]*)?)?(?:\?{1}([^\s\n\#\[\]]*))?([\#][^\s\n]*)?\)?

looks like all \ are getting escaped again

@juhoteperi note that in JS you do not always need to escape a / character in a regex (for example, /[/ab]/ will work)

or most

Cljs escapes / even if they are already escaped

https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/compiler.cljc#L249

very curious

allright this is way out of my league, I'll try to use my nasty js regex as a foreign lib

looks like you could just undo what the compiler is doing by removing all \\ with \

or ... use a context free grammar 🙂

looks like it may be a bug

perhaps just a single escape : (.replaceAll (re-matcher #"/" pattern) "\\/") to deal with the java pattern

@chpill you don’t need foreign libs for this btw

ClojureScript fully supports Google Closure JS namespaces that follow classpath conventions for some time now

just add a JS file to your project, done

seems like the core of the issue is that in a JS regex literal you need to escape internal / but in CLJS you don't, so you can't just copy and paste all JS regex literals

@dnolen thanks! But then, will I not get issues with advanced compilation?

goog.provide("my.utils"); my.utils.regex = ....;

and just require my.utils in your ClojureScript

@chpill Google Closure JS means no issues with advanced compilation

Okay, i'll try to do it that way

and you don’t need any additional compiler config for this

@chpill got it to work by just not escaping any /

(re-seq #"\(?(?:(http|https|ftp)://)?(?:((?:[^\W\s]|\.|-|[:]{1})+)@{1})?((?:www.)?(?:[^\W\s]|\.|-)+[\.][^\W\s]{2,4}|localhost(?=/)|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})(?::(\d*))?([/]?[^\s\?][/]{1})(?:/?([^\s\n\?\[\]\{\}\#](?:(?=\.)){1}|[^\s\n\?\[\]\{\}\.\#])?([\.]{1}[^\s\?\#])?)?(?:\?{1}([^\s\n\#\[\]]))?([\#][^\s\n]*)?\)?" "URLs within text: * Create Share Links to send your expressions to co-workers or link to them on Twitter or your blog [ex. http://RegExr.com/?2rjl6].")

I ran into the double backquote problem recently trying to round trip some structures between pr-str and read-string

I’m having a problem with my build as soon as I enable the :static-fns true compiler option together with :optimizations :simple. Getting ReferenceError: self__ is not defined errors which is triggered by this code: https://github.com/reagent-project/reagent/blob/master/src/reagent/impl/util.cljs#L61-L70

can this be a ClojureScript compiler bug?

@danvingo that's amazing! I tried it against a more meaty string "plop plop" and every matching group seems to be doing its job perfectly. Many thanks!

sweeet! no prob - i wasn't aware of that escaping behaviour before - learned something new 🙂

@kommen hard to tell without actual code. that snippet looks ok although I can't tell its purpose

anyone know if namespaces support "wrapping" other required namespaces - something like export * from …; in js

use case would be to provide one namespace that exposes implementation namespaces

@danvingo not supported

cool, thanks

@thheller thanks, I will write up a more thorough description with hopefully an example to compile

is it possible to extract the last update to a persistent structure from its internal tree? I'd like to implement distributed objects efficiently by pushing out only the diff of the shared structure in persistent collections.

or is there another way to efficiently get the diff using an add-watch?

ah looks like there's a lead here: https://gist.github.com/danjohansson/add5515b2067b3036044d450cbec08f3

anyone know why I always get these warnings? code still seems to work but shouldn't the clojure.string namespace be native?

WARNING: Use of undeclared Var clojure.string/replace at line 300 src-cljc/xxx.cljc

@kanwei well did you require clojure.string?

well, in clojure, the require isn't necessary, and the warnings only seem to be showing the first time (recompiling gives no errors)

clojure.string is not "native" but sometimes happens to be required by other code

does anyone have a good clojurescript pretty printer that will print html or ideally hiccup?

Hello. Can anyone suggest where this package is coming from? https://clojars.org/cljsjs/react-with-addons Can't find it's source anywhere in https://github.com/cljsjs/packages

@andrewboltachev It seems to be pulling the library from http://unpkg.com https://github.com/cljsjs/packages/blob/master/react/build.boot#L58-L61

gotcha! thanks a lot @rgdelato

Aside (just in case this is why you were asking): http://unpkg.com was having some issues yesterday, but it seems to be resolved today: https://twitter.com/unpkg/status/852655106562564098

@rgdelato thanks for info, and I'm rather looking now how to install this thing via NPM

(having bare JS file I can include it anyway, but it's not quite exciting)

I think you might have to install react, react-dom, and whatever addons you're using separately: https://facebook.github.io/react/docs/addons.html

yep, seems much like that, especially 'cause of deprecation

Now I realize: "Version changes and you're screwed" as Rich Hickey has said 😄

> "We're discontinuing active maintenance of React Addons packages. In truth, most of these packages haven't been actively maintained in a long time. They will continue to work indefinitely, but we recommend migrating away as soon as you can to prevent future breakages." > - https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html#discontinuing-support-for-react-addons

I was just about to ask whether :npm-deps is supposed to pull in transitive dependencies like react and react-dom … now I see one has to include those manually?

react and react-dom should be fine I think in many cases

I'm using {:antd "2.9.1"} and it barfed during build, wanting react and react-dom

@hagmonk btw also a little aside here: do you plan to use JS React components from NPM in your project?

I'm hoping so, I just started with a team that uses React and I've not done CLJS before, although I've done a ton of JVM Clojure … I figured now is a good time to try!

did it ask for cljsjs.react and cljsjs.react-dom?

I'm using boot-cljs, and the error it threw was during build of main.js

And I'm just passing npm-opts through as a compiler option for the cljs task

aha, well just feel free to add these there

@hagmonk are you using Reagent, Rum, or Om.Next btw?

I have another colleague who recommended Om.Next, so was going to start there

aha so I'm not quite sure, but at least in case of reagent you'll need also (as long as you're installing react via NPM) to add empty react.cljs and dom.cljs files: http://blob.tomerweller.com/reagent-import-react-components-from-npm

oh very interesting

feels like an opportunity for a template to get people going - I'm surprised there aren't more of these, I thought that central to the appeal of using react would be pulling in all these component libraries

yes, the template it's most wanted thing there I'd say

hmm now I do hit an error

clojure.lang.ExceptionInfo: Cannot build without root node being specified data: {:from :boot-cljs}

root node... may be you may check boot cljs's docs?

yeah, that's where I'm off to now. All the github repos using npm-deps are building with leiningen, it seems

make sure you have all leiningen plugins in place as well

'cause might be that you need lein-npm or sth

seems it could be an ES6 / Closure incompatibility: https://clojurians-log.clojureverse.org/cljs-dev/2017-02-17.html

humph, so what's the alternative? use the cljsjs versions? I recall you still have to npm install the libraries anyway to get the rest of the library resources?

@hagmonk cljsjs versions of which libraries do you mean?

antd in this case

aha

I think cljsjs itself isn't farsighted solution. You'll anyway reach the limit and need some library from NPM

I see

so, the solution: 1. Add to src/ (or src/cljs, what you have) files react.cljs and react/dom.cljs with only ns declarations to make your CLJS libraries happy (thinking that React actually exists) 2. Init NPM package and install via npm (or I might recommend via yarn) your required packages 3. Add Webpack (config file and entry file) and to entry file add:

window.React = require('react');
window.ReactDOM = require('react-dom');
window.antd = require('antd'); 
// and so on

that's what I'm using

cool, that's very helpful!

@hagmonk glad to help, especially if this would actually work for you also, this might be not advanced optimizations-fiendly, which you might not like, but IMO 1st comes the functionality, then perofrmance and so on

I completely agree there