Merge pull request #18 from ovistoica/main

fix: preserve per-build :js-options in merged-shadow-config

Author: plexus

CHANGELOG.md

@@ -4,6 +4,13 @@
 
 ## Fixed
 
+- Preserve per-build `:js-options` in `merged-shadow-config`. Previously the
+  root project's builds had their `:js-options` unconditionally replaced with
+  `{}`, which clobbered settings like `:js-provider :external` and
+  `:external-index`. Now the root project's `:js-options` are left untouched,
+  and submodule builds get `:js-package-dirs` merged into their existing
+  `:js-options` rather than replacing them.
+
 ## Changed
 
 # 0.47.210-alpha (2026-03-02 / 54388ed)

src/lambdaisland/launchpad/shadow.clj

@@ -96,11 +96,11 @@
                                      k
                                      (keyword module-name (name k)))]
                              [build-id
-                              (assoc (update-build-keys process-root module-root v)
-                                     :build-id build-id
-                                     :js-options (if (= "" module-path)
-                                                   {}
-                                                   {:js-package-dirs [(str module-path "/node_modules")]}))])))
+                              (cond-> (update-build-keys process-root module-root v)
+                                true
+                                (assoc :build-id build-id)
+                                (not= "" module-path)
+                                (update :js-options merge {:js-package-dirs [(str module-path "/node_modules")]}))])))
 
                     builds)))))
         (assoc :deps {})