@builder.io/react 2.0.8-4 → 2.0.9

package/dist/lib/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@builder.io/react",
-    "version": "2.0.8-3",
+    "version": "2.0.8",
     "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

@@ -3,27 +3,30 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.safeDynamicRequire = void 0;
 var sdk_1 = require("@builder.io/sdk");
 var noop = function () { return null; };
-// Allow us to require things dynamically safe from webpack bundling
-var localSafeDynamicRequire;
 /*
- * 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 we have change it to
+ * That said, if change it to
  *
  * if (typeof require === 'function') {
  *   localSafeDynamicRequire = eval('require');
  * }
  *
- * Then the TSC compiler over-optimizes and replaces the if condition with true always
+ * 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 globalThis.require === 'function') {
-    localSafeDynamicRequire = eval('globalThis.require');
-}
-if (localSafeDynamicRequire === undefined && sdk_1.Builder.isServer) {
-    localSafeDynamicRequire = eval('require');
+if (typeof globalThis.require === 'function' || sdk_1.Builder.isServer) {
+    exports.safeDynamicRequire = eval('require');
 }
-localSafeDynamicRequire !== null && localSafeDynamicRequire !== void 0 ? localSafeDynamicRequire : (localSafeDynamicRequire = noop);
-exports.safeDynamicRequire = localSafeDynamicRequire;
+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;;;AAEb,uCAA0C;AAE1C,IAAM,IAAI,GAAG,cAAM,OAAA,IAAI,EAAJ,CAAI,CAAC;AACxB,oEAAoE;AAEpE,IAAI,uBAAmD,CAAC;AAExD;;;;;;;;;;;;GAYG;AACH,IAAI,OAAO,UAAU,CAAC,OAAO,KAAK,UAAU,EAAE;IAC5C,uBAAuB,GAAG,IAAI,CAAC,oBAAoB,CAAC,CAAC;CACtD;AAED,IAAI,uBAAuB,KAAK,SAAS,IAAI,aAAO,CAAC,QAAQ,EAAE;IAC7D,uBAAuB,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC;CAC3C;AAED,uBAAuB,aAAvB,uBAAuB,cAAvB,uBAAuB,IAAvB,uBAAuB,GAAK,IAAW,EAAC;AAE7B,QAAA,kBAAkB,GAAoB,uBAAyB,CAAC"}
+{"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-4",
+  "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/src/functions/safe-dynamic-require.ts

@@ -5,30 +5,32 @@ import { Builder } from '@builder.io/sdk';
 const noop = () => null;
 // Allow us to require things dynamically safe from webpack bundling
 
-let localSafeDynamicRequire: typeof require | undefined;
+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 we have change it to 
- * 
+ * That said, if change it to
+ *
  * if (typeof require === 'function') {
  *   localSafeDynamicRequire = eval('require');
  * }
- * 
- * Then the TSC compiler over-optimizes and replaces the if condition with true always
+ *
+ * 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 globalThis.require === 'function') {
-  localSafeDynamicRequire = eval('globalThis.require');
+if (typeof globalThis.require === 'function' || Builder.isServer) {
+  safeDynamicRequire = eval('require');
 }
 
-if (localSafeDynamicRequire === undefined && Builder.isServer) {
-  localSafeDynamicRequire = eval('require');
-}
-
-localSafeDynamicRequire ??= noop as any;
-
-export let safeDynamicRequire : typeof require = localSafeDynamicRequire!!;
-
+safeDynamicRequire ??= noop as any;