Clojurians Log v2

Thomas. Your docs are amazing. Thank you! It has helped me make sense of ClojureScript macros. 🙂

Is it possible to write a macro that uses an npm package as a dependency? I’ve run into Gotcha #1: Namespace Aliases and Dependencies from https://code.thheller.com/blog/shadow-cljs/2019/10/12/clojurescript-macros.html. My goal was to write a quick utility for faking timers in tests with @sinonjs/fake-timers but it seems like that’s not possible with deps from npm. :/ my.util.clj

(ns test.fixtures)

(defmacro with-fake-timers
  [symbol & body]
  `(let [~symbol (fake-timers/install)]
    (try ~@body
      (finally (.uninstall ~symbol)))))

(ns test.fixtures
  (:require
   ["@sinonjs/fake-timers" :as fake-timers])
  (:require-macros
   [test.fixtures :refer [with-fake-timers]]))

(ns test.fixtures-test
  (:require
    [cljs.test :refer [deftest testing is]]
    [test.fixtures :refer [with-fake-timers]]))

(deftest macro-test
  (is (= '_ (macroexpand-1 '(with-fake-timers clock (prn "Hello, world"))))))

(deftest with-fake-timers-test
  (testing "Given a symbol to bind to and a body to execute, "
           "it should replace the global timer APIs and execute the body"
    (let [previous-value js/window.setTimeout]
      (with-fake-timers clock
        (let [actual (= js/window.setTimeout previous-value)
              expected false]
          (is (= actual expected)))))))

Could this be related to it being a :karma build target? The macro expands as expected but uses the missing ns alias, then throws: > Chrome Headless 88.0.4324.150 (Mac OS 10.14.6) test.fixtures-test with-fake-timers-test FAILED > FAIL in (with-fake-timers-test) (ReferenceError:NaN:NaN) > failed with ReferenceError: fake_timers is not defined > message: Uncaught exception, not in assertion. >

I’m guessing I should stop fighting and give up on the macro. 😕

hihi, I'm trying to use shadow-cljs where I need to consume some javascript generated by the typescript compiler. I've tried different ways to transpile and I just don't seem to be getting it right. If I transpile in ES2015+, shadow-cljs complains about references to import * as cdktf from 'cdktf';. If I transpile to CommonJS, it complains as well:

; Execution error (ReferenceError) at (<cljs repl>:1).
module$docker$index is not defined

> tree ./.gen/providers-out/
./.gen/providers-out/
└── docker
    ├── config.d.ts
    ├── config.js
    ├── container.d.ts
    ├── container.js
    ├── data-docker-network.d.ts
    ├── data-docker-network.js
    ├── data-docker-registry-image.d.ts
    ├── data-docker-registry-image.js
    ├── docker-provider.d.ts
    ├── docker-provider.js
    ├── image.d.ts
    ├── image.js
    ├── index.d.ts
    ├── index.js
    ├── network.d.ts
    ├── network.js
    ├── secret.d.ts
    ├── secret.js
    ├── service.d.ts
    ├── service.js
    ├── volume.d.ts
    └── volume.js

1 directory, 22 files

all the exports are within index.js and the modules it refers to call out to cdktf (expected from node_modules)

any help would be much appreciated 🙂

for CommonJS:

> (require '["/docker"])
FileNotFoundException: ~/hello-world/.gen/providers-out/docker (Is a directory)

> (require '["/docker/index" :as docker])
nil

> docker

Execution error (ReferenceError) at (<cljs repl>:1).
module$docker$index is not defined
nil

for ES2015:

ExceptionInfo closure errors {:tag :shadow.build.closure/errors, :errors [{:resource-name "docker/config.js", :source-name "docker/config.js", :line 3, :column 0, :msg "Namespace imports (goog:some.Namespace) cannot use import * as. Did you mean to import cdktf from 'goog:shadow.js.shim.module$cdktf';?"}]

and here's how the module looks like in the node REPL after transpiling to CommonJS:

> var docker = require('./.gen/providers-out/docker')
undefined
> docker
{
  Config: [Getter],
  ContainerNetworkData: [Getter],
  Container: [Getter],
  Image: [Getter],
  Network: [Getter],
  Secret: [Getter],
  Service: [Getter],
  Volume: [Getter],
  DataDockerNetworkIpamConfig: [Getter],
  DataDockerNetwork: [Getter],
  DataDockerRegistryImage: [Getter],
  DockerProvider: [Getter]
}

@sean.poulter the easiest path is just creating a function in CLJS that delegates to the npm dep (defn install-timer [] (fake-timers/install)) and call that from the macro instead.

Thanks for keeping it simple. 🙂

That works perfectly. 🙏

Do you have a preference for Patreon or PayPal?

Wouldn't it be useful to also take into account file deletions when it comes to live reloading? (Eg. UI reflects that a CSS file has been deleted (browser will reload non-existing file))

how would it reload a non-existing file? doesn't make much sense to me

all fine 🙂

Do what it does for modifications (ie. modify the r query argument -> browser tries to reload file and fails)

yes but why would you willingly force a 404?

For instance in the context of live reloading CSS, it's a bit confusing that nothing changes when a CSS file gets deleted (because the browser knows nothing about that)

yeah still no clue what you are doing but so far I'm not interested in adding that

Haha, all right, it's just I would expect that deleting any live-reloaded asset that is still being used by my code would induce a change to help me notice it, but all right

yeah I can only imagine getting into weird situations. the only sensible things to do would be deleting the link tag used to import the style as thats the only way to remove the css

but then when you add it again the HTML doesn'lt have the link anymore so it won't be loaded

in general you should only have one css file anyways. at least thats what I have always done. take the many files and generate one that is loaded in the html