Hi, I’m trying to require cljs.analyzer.api to use resolve but am getting errors, is this not possible?

@mudphone: Are you going to require it in clojurescript?

cljs.analyzer.api is a clojure namespace. You have to use it in clojure.

@crocket: ah, I see, okay

I wanted to use it in cljs

compiler API belongs to clojure.

got it

Inception

If you really want to use compiler API in clojurescript, you have to compile it to clojurescript, first.

Compile clojurescript to clojurescript.

@mudphone: https://github.com/clojure/clojurescript/wiki/Bootstrapping-the-Compiler

Bootstrapped clojurescript is not a full substitute for clojurescript compiler on JVM.

@crocket: got it, actually was just hoping to turn a key or string into a function and call it at runtime

@mudphone: Evaluating a string into a function creates security vulnerabilities in practice.

It allows crackers to execute arbitrary code in remote programs.

Crackers will find security vulnerabilities, given enough time.

When doing lein figwheel it seems I get

Caused by: java.io.FileNotFoundException: Could not locate cljs/analyzer__init.c
lass or cljs/analyzer.clj on classpath:

I think it is to do with there being devcards defined.

@cjmurphy: this nearly always means you haven’t specified a Clojure 1.7.0 dependency explicitly

Okay thanks, was just going straight off some other project.clj. Will check...

[org.clojure/clojure "1.7.0"]
                 [org.clojure/clojurescript "1.7.170"]

I just cloned devcards and then did lein figwheel.

git clone 

cd devcards

lein figwheel

I had the same issue with the devcards template, so put up this issue:

https://github.com/bhauman/devcards-template/issues/10

@cjmurphy: but you also need to be sure nothing is pulling in the wrong dependency, lein deps :tree

I'll do it manually. Wanting to get kanban demo going - seems the most comprehensive Om/Next demo out there...

Okay will do thanks..

I'm on Windows so boot does not work.

@cjmurphy: sure, but in general I recommend taking the slower path

Have to convert it to lein.

do the tutorials - then figure out why the template isn’t working based on that

The boot problem is a serious one too: https://github.com/Jannis/om-next-kanban-demo/issues/1

I've done all the tutorials. Jannis's code looks easy enough to follow - good to see lots of querys at the root level.

@cjmurphy: ok cool, just double checking

lots of people just jump into the deep end

Can anyone explain https://github.com/clojure-emacs/cider/issues/1489#issuecomment-167415309 ?

@dnolen: Can you?

@crocket: understood… actually, I’m not taking the string from the user, just want to use a keyword, turn into a string, and resolve into an already existing function

@mudphone you can use cljs.analyzer/resolve instead, have a look at https://github.com/ScalaConsultants/replumb/blob/master/src/cljs/replumb/repl.cljs#L83

Is there a good library for turning a callback hell into something better?

core.async :)

core.async doesn't handle errors by default. Promise does, though.

crocket: in go they use values as errors. Also you can define your own error protocol. Maybe it could be helpful

values as errors? Can you show me an example?

https://github.com/jamesmacaulay/cljs-promises or there is some promises library over core.async @crocket

crocket: https://blog.golang.org/errors-are-values

Since errors are emitted from the same channels as other values, it is inconvenient. Exceptions can't be used, either.

hmm... what about Failure record? I wrote a blog-post about it, but I'm not very experienced with core.async: https://s-mage.github.io/2015/10/16/better-errors.html

Typed Clojure may be a better alternative. Errors are of different types from other values. Something like Either<Error,HttpResponse> is not bad.

yes, it is

I guess cats is the most advanced library for it now

Using the regexp from http://stackoverflow.com/a/8317014/385505 in Clojurescript throws "Invalid regular expression flags". Why is that? EDIT: I see, forward slashes don't need to be escaped in Clojurescript.

hi guys, i have this simple function to catch keyboard shortcut, it doesn't seem to work, does anyone spot something wrong ?

(ns something
   (:require [goog.events :as events])
   (:import [goog.ui KeyboardShortcutHandler]))
;; ...
   (let [handler (doto
                     (KeyboardShortcutHandler. js/document)
                   (.registerShortcut 'A' 'a'))
         triggered-fn (fn [e]
                        (js/alert (str "Shortcut triggered" (.identifier e))))]
     (events/listen handler
                    js/goog.ui.KeyboardShortcutHandler.EventType.SHORTCUT_TRIGGERED
                    triggered-fn)
     )

@mudphone: why not keep the functions in a map and resolve it that way? Something like:

(let [functions {:keyword (fn [arg] (println "wheee" arg))}
      function  (functions :keyword)]
  (function "test'))

@crocket: as someone mentioned using Maybe and Either from cats - http://funcool.github.io/cats/latest/#maybe - could be a good solution.

yeah

@crocket: Use the go-catching and <? macros from the glossop libraries. They are slight variations of the go and <! macros from core.async that catches exceptions.

*they build upon the macros from core.async

thanks

Their implementation is actually fairly straight forward: https://github.com/nervous-systems/glossop/blob/master/src/glossop/core.cljc

How do people go about testing cljc at the moment?

Is there an 'easy' way of testing cljs and clj in the same project?

@phil: we are already doing it on our projects at funcool github organization

maybe https://github.com/funcool/cats or https://github.com/funcool/octet can serve as example

Thanks.

How do I run the cljs tests?

node out/tests.js

Awesome - I'll base it on that.

If you have small number of tests octet approach is much simplier, but if you are expecting have a bunch of namespaces, cats approach is more appropriate

I'll look at octet too - thanks again.

you are welcome!

Hi

I can't make macroexpand nor macroexpand-1 work in a macro called from CLJS. Does anyone have this problem?

@phil: or you could take a look at this: https://github.com/magomimmo/modern-cljs/blob/master/doc/second-edition/tutorial-15.md

@vibl: https://github.com/clojure/clojurescript/wiki/Differences-from-Clojure#macros

(defmacro test-macroexpand [] `(quote ~(macroexpand-1 '(-> 1 inc)))) returns (-> 1 inc) instead of (inc 1)

@vibl: sorry, I did not read quite well you question...

how to test that function didn’t throw in cljs.test? opposite of (is (thrown? …))

@magomimmo: no problem

Is defmacro supposed to work on a cljs repl?

@vibl: no. You can only macroexpand it. defmacro is still clojure, not clojurescript

A macro declared from the cljs repl with defmacro seems to work (which surprised me) but with an error message: "WARNING: Wrong number of args (0) passed to cljs.user/bla at line 1 <cljs repl>"

@magomimmo try it, you'll be surprised.

@vibl: surprised as well

@vibl: someone from the dev team will give us an answer

@magomimmo: maybe it's a feature preview

@vibl: just an unintended side effect from bootstrapped

@vibl: people are already abusing defmacro….imagine what could happen if they make it available in cljs as well

as stated in a few places already defmacro will never work in .cljs source files

@dnolen: thanks

@dnolen That's why I was surprised that defmacro works in my figwheel repl

it’s a minor issue that it doesn’t just throw an error, someone can submit a patch for that if they like

@dnolen On an unrelated subject: I can't make macroexpand and macroexpand-1 work within a clj macro called from cljs.

@vibl can’t help you with that

neither of those things are well supported

you can examine the analyzer if you want to sort out how to make it work

@dnolen: clj macros are not well supported in cljs?

@vibl now I don’t understand your question at all

in anycase in general there just no need to direct questions at me

there’s a #C07UQ678E channel with knowledgeable people

@dnolen: in a clj file, there is (defmacro test-macroexpand [] (quote ~(macroexpand-1 '(-> 1 inc))))`

I’m trying to hint I don’t have time to answer your question

ask in #C07UQ678E

@dnolen: ok, understand

@dnolen: thanks

is there any documentation or anything about preferred document organization in a SPA?

models/controllers/views etc

I think that there is no single solution for that...

the best documentation maybe is the source code organization of the existing open source applications

that comes down to personal taste.

hah a'ight

imma use my own opinions

👍

@richiardiandrea: Thanks, that gets me closer, I’m still seeing some odd errors, mostly because I probably have to dig into this source. Here’s my first attempt in case you’re interested:

(ana/resolve-var (empty-analyzer-env) (symbol "+") ana/confirm-var-exist-warning)
#object[Error Error: No protocol method IDeref.-deref defined for type null: ]

@jaen: Thanks that would work. I’m using a multimethod now, I think someone here recommended. I was really just trying to see if I could get clever and only have to specify the fn names twice, instead of three times.

Multimethod seems legit as well, I suppose.

But basically you can't select a function by string at runtime in Clojurescript, so you have to get creative.

@jaen, I guess I could pull all the fns from an ns and put them into a map, where I turn the fn names into keys?

Well, but not at the runtime.

You could do either write it out manually or write a macro using the analyser I suppose.

http://pastie.org/10657931 How could I resolve this keyword in cljs? It looks like clojure’s #resolve would work, but that’s not available

@heeton: make a map from keywords to functions.

HI, should (extend-protocol IDoThing number (do-thing [this] ...)) work in clojurescript?

I get the error: #object[TypeError TypeError: Cannot read property 'do_thing' of undefined]

The problem only arises with numbers.

@jaen: I’ll have hundreds of these, is there no automated way?

write a macro, I suppose.

You can use resolve in Clojurescript at macro expansion time.

Just not at runtime.

IIRC

@heeton: What are you trying to do exactly?

@spelufo: Don’t you want capital N number?

no, that throws another error, and I've seen lowercase number used in the clojurescript and garden's source code.

@christopherbui: trying to create a router. I have a bunch of pages defined and want to be able to transition, the result of my function being a reference, not the value

And stepping through the extend-protocol form in the chrome devtools it skips that type when going through the list.

I'm using figwheel with cider, don't know if that has anything to do with it.

@jaen I'm not sure about resolve, one month ago I have tried to something that requires resolve and I found that cljs does not provide it :S

I have asked some questions in the mailing list... but I don't found any solution at this moment

I'm having trouble with goog.string.format being recognised as a function. Could there be something obvious I'm doing wrong?

@heeton, @niwinz: I'm talking about https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/analyzer/api.clj#L141-L150

IIRC it should be available during macroexpansions in Clojurescript

@jaen this is the question that I have asked some time ago https://groups.google.com/d/msg/clojure/DTjqWKQx3Sc/xBWX3WBBCAAJ

@spelufo: Are you sure? I tried this out in the cljs repl and it works

cljs.user=> (defprotocol IDoThing (do-thing [this]))
nil
cljs.user=> (extend-protocol IDoThing js/Number (do-thing [this] 1))
#object[Function "function (this$){
var this$__$1 = this;
return 1;
}"]
cljs.user=> (do-thing 23)
1
cljs.user=> 

I'll try to revisit it again, now knowing about resolve.

Yes I just tried it and it works in the repl, but not in my project.

Very weird. It's the same code. Maybe figwheel is responsible?

@niwinz: hopefully it will be helpful; I didn't have need to try that out yet, but from what I understand it should work.

I don't think format is a macro. I just require it like: [goog.string :refer [format]], and call it the way you might call any function. But Cursive and then running figwheel/devcards - so both at code writing and code running times - do not think it is a function. I don't believe there is anything special I need to do in the project.clj file.

@jaen: thanks for pointing me on it 😉

hello If you have a minute, I've posted a question about following the QuickStart guide in getting a node REPL in the google group: https://groups.google.com/forum/#!topic/clojure/pf2GuFVtiAw Basically when I try to build with the prescribed node.clj, it can't find a dependency

but lein cljsbuild once works

ls

I've stopped on the exeption in the devtoold and the line that is making trouble is my-ns.core.my-ns.core.do_thing[goog.typeOf(3)]. Don't know how it got to that, but clearly it should be my-ns.core.do_thing[goog.typeOf(3)], the problem is that the namespace gets duplicated somehow.

I had a (def sth ...) in a namespace '(ns sth.core)', and when I changed it to (def sth-else ...) the duplicate sth.core.sth.core turned into sth.core.

This is clearly a bug.

Is there a way of giving a namespace you use a different name within the require, for example (which is bad syntax): [goog.string :refer [format :as format-str]]?

I want to use a name other than format in case there is some kind of shadowing problem...