feat: rely on esbuild to handle process.env.NODE_ENV

README.md

@@ -167,7 +167,6 @@ Underlying classes the power the other features. You normally won't have to use
 
 - `bundle=true`
 - `platform=browser`
-- `--define:process.env.NODE_ENV=\\\"production\\\"` or actual value of `NODE_ENV` if set
 
 > 💡 See [Static Website with React](examples/website) for a complete working example of a how to deploy a React app to S3.
 

lib/source.ts

@@ -34,15 +34,6 @@ abstract class Source<
   constructor(entryPoints: string | string[], props: Props) {
     const defaultOptions: Partial<BuildOptions> = {
       platform: "browser",
-      ...(!props.buildOptions?.define
-        ? {
-            define: {
-              "process.env.NODE_ENV": `"${
-                process.env.NODE_ENV ?? "production"
-              }"`,
-            },
-          }
-        : {}),
     };
 
     this.props = {

test/source.test.ts

@@ -129,36 +129,5 @@ describe("source", () => {
 
       expect(source.props.buildOptions.platform).toBe("browser");
     });
-
-    describe("NODE_ENV is set", () => {
-      it("defines NODE_ENV as that value", () => {
-        const source = new JavaScriptSource(
-          "fixtures/handlers/js-handler.js"
-        ) as any;
-
-        expect(source.props.buildOptions.define?.["process.env.NODE_ENV"]).toBe(
-          '"test"'
-        );
-      });
-    });
-
-    describe("NODE_ENV not set", () => {
-      beforeEach(() => {
-        delete process.env.NODE_ENV;
-      });
-
-      afterEach(() => {
-        process.env.NODE_ENV = "test";
-      });
-      it("defines NODE_ENV as production", () => {
-        const source = new JavaScriptSource(
-          "fixtures/handlers/js-handler.js"
-        ) as any;
-
-        expect(source.props.buildOptions.define?.["process.env.NODE_ENV"]).toBe(
-          '"production"'
-        );
-      });
-    });
   });
 });