env setup

packages/web-util/src/index.build.ts

@@ -70,9 +70,13 @@ function copyFilesPlugin(files: Array<string>) {
   return {
     name: "copy-files",
     setup(build: PluginBuild) {
+      if (!files || !files.length) {
+        return;
+      }
       const outDir = build.initialOptions.outdir;
-      if (outDir === undefined)
+      if (outDir === undefined) {
         throw Error("esbuild build options does not specify outdir");
+      }
       build.onEnd(() => {
         for (const file of files) {
           const name = path.parse(file).base;
@@ -143,7 +147,6 @@ function linariaPlugin() {
 
 const defaultEsBuildConfig: esbuild.BuildOptions = {
   bundle: true,
-  minify: false,
   loader: {
     ".svg": "file",
     ".inline.svg": "text",
@@ -157,7 +160,6 @@ const defaultEsBuildConfig: esbuild.BuildOptions = {
   target: ["es6"],
   format: "esm",
   platform: "browser",
-  sourcemap: true,
   jsxFactory: "h",
   jsxFragment: "Fragment",
   alias: {
@@ -171,6 +173,7 @@ const defaultEsBuildConfig: esbuild.BuildOptions = {
 };
 
 export interface BuildParams {
+  type: "development" | "test" | "production";
   source: {
     assets: string[];
     js: string[];
@@ -186,6 +189,7 @@ export function computeConfig(params: BuildParams): esbuild.BuildOptions {
     copyFilesPlugin(params.source.assets),
   ];
 
+  if (params.css) {
     switch (params.css) {
       case "sass": {
         plugins.push(sassPlugin);
@@ -208,11 +212,31 @@ export function computeConfig(params: BuildParams): esbuild.BuildOptions {
         throw Error(`not supported: ${cssType}`);
       }
     }
+  }
+  if (!params.type) {
+    throw Error(
+      `missing build type, it should be "test", "development" or "production"`,
+    );
+  }
+
+  if (!params.source.js) {
+    throw Error(`no javascript entry points, nothing to compile?`);
+  }
+  if (!params.destination) {
+    throw Error(`missing destination folder`);
+  }
+
   return {
     ...defaultEsBuildConfig,
     entryPoints: params.source.js,
     publicPath: params.public,
     outdir: params.destination,
+    minify: false, //params.type === "production",
+    sourcemap: true, //params.type !== "production",
+    define: {
+      ...defaultEsBuildConfig.define,
+      "process.env.NODE_ENV": JSON.stringify(params.type),
+    },
     plugins,
   };
 }

packages/web-util/src/utils/http-impl.browser.ts

@@ -108,9 +108,9 @@ export class BrowserHttpLib implements HttpRequestLibrary {
         }, requestTimeout.d_ms);
       }
 
-      for (const headerName in Object.keys(requestHeadersMap)) {
+      Object.keys(requestHeadersMap).forEach((headerName) => {
         myRequest.setRequestHeader(headerName, requestHeadersMap[headerName]);
-      }
+      });
 
       myRequest.responseType = "arraybuffer";
       myRequest.send(myBody);