Async eval results in wrong namespace

[panterarocks49]

version

{:git/url "https://github.com/babashka/sci"
  :sha "ff4db9056392f052f4809c06c68e2bef9e26da09"}

platform
Javascript, browser

problem
Evaluating two strings in parallel (with sci.async) with the same context results in evaluation in the wrong namespace and could not resolve symbol errors.

The output of the repro below

repro

(require '[sci.core :as sci])
(require '[sci.async :as scia])
(require '[promesa.core :as p])

(def ctx (sci/init {:namespaces    {'clojure.core {'require scia/require}}
                    :async-load-fn (fn [{:keys [libname ns]}]
                                     (prn libname ns)
                                     (p/resolved
                                      (case libname
                                        acme.foo {:source "(ns acme.foo) (defn the-fn [] :hello)"}
                                        acme.bar {:source "(ns acme.bar) (defn the-fn [] :bye)"})))}))

(scia/eval-string* ctx "(ns test (:require [acme.foo :as foo])) (foo/the-fn)")
(scia/eval-string* ctx "(require '[acme.bar :as bar]) (bar/the-fn)")

expected behavior
I would expect it to behave the same as executing one at a time
Like this

(p/let [res (scia/eval-string* ctx "(ns test (:require [acme.foo :as foo])) (foo/the-fn)")
        res (scia/eval-string* ctx "(require '[acme.bar :as bar]) (bar/the-fn)")])

The first eval should be in the test namespace and the second should be in the user namespace

Sponser

[borkdude]

Fixed with d315e98 but I'll leave this open until we have written tests for this.