Fix babashka#219: import relative to script (babashka#237)

Co-authored-by: Brandon Olivier <[email protected]>

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 For a list of breaking changes, check [here](#breaking-changes).
 
+## 0.6.125
+
+- Fix [#219](https://github.com/babashka/nbb/issues/219): nbb doesn't resolve local node_modules when using the dynamic `js/import`
+
 ## 0.6.124
 
 - Fix [#136](https://github.com/babashka/nbb/issues/136): allow `set!` on any var

src/nbb/core.cljs

@@ -53,7 +53,11 @@
 
 (def sci-ctx (atom nil))
 
-(set! (.-import goog/global) esm/dynamic-import)
+(set! (.-import goog/global)
+      (fn [what]
+        ;; need to resolve based on the current file
+        (-> ((:resolve @ctx) what)
+            (.then #(esm/dynamic-import %)))))
 
 (def loaded-modules (atom {}))
 

test-scripts/esm-test/script.cljs

@@ -26,3 +26,9 @@
 (await (zx/$ #js ["ls"]))
 
 (prn (execa/execaSync "ls"))
+
+(def term-size' (await (js/import "term-size")))
+(prn (term-size'.default))
+
+(def js-file (await (js/import "./test-js-file.mjs")))
+(assert (= 10 (.-x js-file)))

test-scripts/esm-test/test-js-file.mjs

@@ -0,0 +1 @@
+export const x = 10;