alet with interop forms produces invalid JavaScript

[Peeja]

With

[org.clojure/clojurescript "1.9.562"]
[funcool/promesa "1.8.1"]

The CLJS

(ns promesabug.core
  (:require [promesa.core :as p :include-macros true]))

(p/alet [a-binding (.aJSMethod (js-obj))]
  nil)

produces

// Compiled by ClojureScript 1.9.562 {:target :nodejs}
goog.provide('promesabug.core');
goog.require('cljs.core');
goog.require('promesa.core');
if(cljs.core._EQ_.call(null,promesa.core.await$,.aJSMethod)){
promesa.core.bind.call(null,{},(function (a_binding){
return promesa.core.promise.call(null,null);
}));
} else {
var a_binding_8847 = {}.aJSMethod();
promesa.core.promise.call(null,null);
}

//# sourceMappingURL=core.js.map

which includes cljs.core._EQ_.call(null,promesa.core.await$,.aJSMethod). .aJSMethod is nonsense JS here.

This appears to be what happens when you take the value of an interop symbol in ClojureScript; for instance, (println .aJSMethod) becomes cljs.core.println.call(null,.aJSMethod);. I'd call that a bug in ClojureScript, but until and unless that's changed, it would be nice to avoid causing the problem.

(A decent workaround for the current version of Promesa, incidentally, is to wrap the right-hand-side of the binding in (p/await …).)

[Peeja]

(Bug filed against ClojureScript, but the correct compiler behavior here would be to throw a compiler error and stop building, so Promesa still needs to do something differently to make this work.)

[niwinz]

Yeah, agree. The alet macro should be fixed.

[niwinz]

Fixed ;)