Use iife when bundling browser apps (#880)

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+Other improvements:
+- Use `iife` format when bundling browser apps (#880)
+
 ## [0.20.8] - 2022-04-27
 
 Features:

src/Spago/Build.hs

@@ -425,7 +425,10 @@ bundleWithEsbuild withMain srcMap (ModuleName moduleName) (TargetPath targetPath
     srcMapOpt = case srcMap of
       WithSrcMap -> ["--sourcemap"]
       WithoutSrcMap -> []
-    esbuildBase = platformOpt <> minifyOpt <> srcMapOpt <> ["--format=esm", "--bundle", "--outfile=" <> targetPath]
+    formatOpt = case (platform, withMain) of
+      (Browser, WithMain) -> ["--format=iife"]
+      (_,_) -> ["--format=esm"]
+    esbuildBase = platformOpt <> minifyOpt <> srcMapOpt <> formatOpt <> ["--bundle", "--outfile=" <> targetPath]
     (input, cmd) = case withMain of
       WithMain -> do
         let

test/fixtures/bundle-app-esm.js

@@ -1,12 +1,14 @@
-// output/Effect.Console/foreign.js
-var log = function(s) {
-  return function() {
-    console.log(s);
+(() => {
+  // output/Effect.Console/foreign.js
+  var log = function(s) {
+    return function() {
+      console.log(s);
+    };
   };
-};
 
-// output/Main/index.js
-var main = /* @__PURE__ */ log("\u{1F35D}");
+  // output/Main/index.js
+  var main = /* @__PURE__ */ log("\u{1F35D}");
 
-// <stdin>
-main();
+  // <stdin>
+  main();
+})();

test/fixtures/packages-prepare-0-15.dhall

@@ -1,4 +1,3 @@
 let upstream =
-      https://github.com/JordanMartinez/package-sets/releases/download/psc-0.15.0-20220329/packages.dhall
-
-in  upstream
+      https://github.com/purescript/package-sets/releases/download/psc-0.15.0-20220429/packages.dhall
+in  upstream