@builder.io/react 2.0.8 → 2.0.10-0

package/dist/lib/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@builder.io/react",
-    "version": "2.0.8",
+    "version": "2.0.9",
     "description": "",
     "keywords": [],
     "main": "dist/builder-react.cjs.js",
@@ -84,7 +84,7 @@
         "jest": "^28.1.3",
         "jest-environment-jsdom": "^28.1.3",
         "lint-staged": "^7.0.0",
-        "prettier": "^1.4.4",
+        "prettier": "^2.6.2",
         "prompt": "^1.0.0",
         "react": ">=16.8.0",
         "react-dom": ">=16.8.0",

package/dist/lib/src/functions/safe-dynamic-require.js

@@ -1,13 +1,32 @@
 'use strict';
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.safeDynamicRequire = void 0;
+var sdk_1 = require("@builder.io/sdk");
 var noop = function () { return null; };
 /*
- * The if condition below used to be if (typeof globalThis.require === "function"
+ * The if condition below used to be
+ *
+ *     if (typeof globalThis.require === "function")
+ *
  * That broke in case Builder was running on the server (Next, SSR use-cases) where
  * globalThis.require was undefined. Avoiding use of Builder.isServer for Cloudflare worker cases
+ * That said, if change it to
+ *
+ * if (typeof require === 'function') {
+ *   localSafeDynamicRequire = eval('require');
+ * }
+ *
+ * Then the TSC / rollup compiler over-optimizes and replaces the if condition with true always
+ * causing it to blow up on the client side. Hence this convoluted way of doing it.
+ *
+ * In Summary:
+ *
+ * 1. Node -> globalThis.require does not work
+ * 2. Client -> typeof require === 'function' does not work because of overoptimization by the compiler
+ * 3. Cloudflare edge -> only globalThis.require works
  */
-if (typeof require === 'function')
+if (typeof globalThis.require === 'function' || sdk_1.Builder.isServer) {
     exports.safeDynamicRequire = eval('require');
+}
 exports.safeDynamicRequire !== null && exports.safeDynamicRequire !== void 0 ? exports.safeDynamicRequire : (exports.safeDynamicRequire = noop);
 //# sourceMappingURL=safe-dynamic-require.js.map

package/dist/lib/src/functions/safe-dynamic-require.js.map

@@ -1 +1 @@
-{"version":3,"file":"safe-dynamic-require.js","sourceRoot":"","sources":["../../../../src/functions/safe-dynamic-require.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;;;AAIb,IAAM,IAAI,GAAG,cAAM,OAAA,IAAI,EAAJ,CAAI,CAAC;AAKxB;;;;GAIG;AACH,IAAI,OAAO,OAAO,KAAK,UAAU;IAAE,0BAAkB,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC;AACxE,0BAAkB,aAAlB,0BAAkB,cAAlB,0BAAkB,IAAlB,0BAAkB,GAAK,IAAW,EAAC"}
+{"version":3,"file":"safe-dynamic-require.js","sourceRoot":"","sources":["../../../../src/functions/safe-dynamic-require.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;;;AAEb,uCAA0C;AAE1C,IAAM,IAAI,GAAG,cAAM,OAAA,IAAI,EAAJ,CAAI,CAAC;AAKxB;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,IAAI,OAAO,UAAU,CAAC,OAAO,KAAK,UAAU,IAAI,aAAO,CAAC,QAAQ,EAAE;IAChE,0BAAkB,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC;CACtC;AAED,0BAAkB,aAAlB,0BAAkB,cAAlB,0BAAkB,IAAlB,0BAAkB,GAAK,IAAW,EAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@builder.io/react",
-  "version": "2.0.8",
+  "version": "2.0.10-0",
   "description": "",
   "keywords": [],
   "main": "dist/builder-react.cjs.js",
@@ -84,7 +84,7 @@
     "jest": "^28.1.3",
     "jest-environment-jsdom": "^28.1.3",
     "lint-staged": "^7.0.0",
-    "prettier": "^1.4.4",
+    "prettier": "^2.6.2",
     "prompt": "^1.0.0",
     "react": ">=16.8.0",
     "react-dom": ">=16.8.0",

package/src/functions/safe-dynamic-require.ts

@@ -8,9 +8,29 @@ const noop = () => null;
 export let safeDynamicRequire: typeof require;
 
 /*
- * The if condition below used to be if (typeof globalThis.require === "function"
+ * The if condition below used to be
+ *
+ *     if (typeof globalThis.require === "function")
+ *
  * That broke in case Builder was running on the server (Next, SSR use-cases) where
  * globalThis.require was undefined. Avoiding use of Builder.isServer for Cloudflare worker cases
+ * That said, if change it to
+ *
+ * if (typeof require === 'function') {
+ *   localSafeDynamicRequire = eval('require');
+ * }
+ *
+ * Then the TSC / rollup compiler over-optimizes and replaces the if condition with true always
+ * causing it to blow up on the client side. Hence this convoluted way of doing it.
+ *
+ * In Summary:
+ *
+ * 1. Node -> globalThis.require does not work
+ * 2. Client -> typeof require === 'function' does not work because of overoptimization by the compiler
+ * 3. Cloudflare edge -> only globalThis.require works
  */
-if (typeof require === 'function') safeDynamicRequire = eval('require');
+if (typeof globalThis.require === 'function' || Builder.isServer) {
+  safeDynamicRequire = eval('require');
+}
+
 safeDynamicRequire ??= noop as any;