metro-runtime 0.71.1 → 0.72.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metro-runtime",
-  "version": "0.71.1",
+  "version": "0.72.0",
   "description": "🚇 Module required for evaluating Metro bundles.",
   "main": "src",
   "repository": {

package/src/polyfills/require.js

@@ -23,7 +23,7 @@ const CYCLE_DETECTED = {};
 const { hasOwnProperty } = {};
 
 if (__DEV__) {
-  global.$RefreshReg$ = () => {};
+  global.$RefreshReg$ = () => {}; // $FlowFixMe[missing-local-annot]
 
   global.$RefreshSig$ = () => (type) => type;
 }
@@ -111,14 +111,17 @@ function metroRequire(moduleId) {
     if (initializingIndex !== -1) {
       const cycle = initializingModuleIds
         .slice(initializingIndex)
-        .map((id) => (modules[id] ? modules[id].verboseName : "[unknown]")); // We want to show A -> B -> A:
+        .map((id) => (modules[id] ? modules[id].verboseName : "[unknown]"));
 
-      cycle.push(cycle[0]);
-      console.warn(
-        `Require cycle: ${cycle.join(" -> ")}\n\n` +
-          "Require cycles are allowed, but can result in uninitialized values. " +
-          "Consider refactoring to remove the need for a cycle."
-      );
+      if (shouldPrintRequireCycle(cycle)) {
+        cycle.push(cycle[0]); // We want to print A -> B -> A:
+
+        console.warn(
+          `Require cycle: ${cycle.join(" -> ")}\n\n` +
+            "Require cycles are allowed, but can result in uninitialized values. " +
+            "Consider refactoring to remove the need for a cycle."
+        );
+      }
     }
   }
 
@@ -126,6 +129,21 @@ function metroRequire(moduleId) {
   return module && module.isInitialized
     ? module.publicModule.exports
     : guardedLoadModule(moduleIdReallyIsNumber, module);
+} // We print require cycles unless they match a pattern in the
+// `requireCycleIgnorePatterns` configuration.
+
+function shouldPrintRequireCycle(modules) {
+  const regExps =
+    global[__METRO_GLOBAL_PREFIX__ + "__requireCycleIgnorePatterns"];
+
+  if (!Array.isArray(regExps)) {
+    return true;
+  }
+
+  const isIgnored = (module) =>
+    module != null && regExps.some((regExp) => regExp.test(module)); // Print the cycle unless any part of it is ignored
+
+  return modules.every((module) => !isIgnored(module));
 }
 
 function metroImportDefault(moduleId) {

package/src/polyfills/require.js.flow

@@ -79,6 +79,7 @@ const {hasOwnProperty} = {};
 
 if (__DEV__) {
   global.$RefreshReg$ = () => {};
+  // $FlowFixMe[missing-local-annot]
   global.$RefreshSig$ = () => type => type;
 }
 
@@ -177,13 +178,15 @@ function metroRequire(moduleId: ModuleID | VerboseModuleNameForDev): Exports {
         .map((id: number) =>
           modules[id] ? modules[id].verboseName : '[unknown]',
         );
-      // We want to show A -> B -> A:
-      cycle.push(cycle[0]);
-      console.warn(
-        `Require cycle: ${cycle.join(' -> ')}\n\n` +
-          'Require cycles are allowed, but can result in uninitialized values. ' +
-          'Consider refactoring to remove the need for a cycle.',
-      );
+
+      if (shouldPrintRequireCycle(cycle)) {
+        cycle.push(cycle[0]); // We want to print A -> B -> A:
+        console.warn(
+          `Require cycle: ${cycle.join(' -> ')}\n\n` +
+            'Require cycles are allowed, but can result in uninitialized values. ' +
+            'Consider refactoring to remove the need for a cycle.',
+        );
+      }
     }
   }
 
@@ -194,6 +197,22 @@ function metroRequire(moduleId: ModuleID | VerboseModuleNameForDev): Exports {
     : guardedLoadModule(moduleIdReallyIsNumber, module);
 }
 
+// We print require cycles unless they match a pattern in the
+// `requireCycleIgnorePatterns` configuration.
+function shouldPrintRequireCycle(modules: $ReadOnlyArray<?string>): boolean {
+  const regExps =
+    global[__METRO_GLOBAL_PREFIX__ + '__requireCycleIgnorePatterns'];
+  if (!Array.isArray(regExps)) {
+    return true;
+  }
+
+  const isIgnored = (module: ?string) =>
+    module != null && regExps.some(regExp => regExp.test(module));
+
+  // Print the cycle unless any part of it is ignored
+  return modules.every(module => !isIgnored(module));
+}
+
 function metroImportDefault(moduleId: ModuleID | VerboseModuleNameForDev) {
   if (__DEV__ && typeof moduleId === 'string') {
     const verboseName = moduleId;
@@ -849,7 +868,10 @@ if (__DEV__) {
   };
 
   // Modules that only export components become React Refresh boundaries.
-  var isReactRefreshBoundary = function (Refresh, moduleExports): boolean {
+  var isReactRefreshBoundary = function (
+    Refresh: any,
+    moduleExports: Exports,
+  ): boolean {
     if (Refresh.isLikelyComponentType(moduleExports)) {
       return true;
     }
@@ -878,9 +900,9 @@ if (__DEV__) {
   };
 
   var shouldInvalidateReactRefreshBoundary = (
-    Refresh,
-    prevExports,
-    nextExports,
+    Refresh: any,
+    prevExports: Exports,
+    nextExports: Exports,
   ) => {
     const prevSignature = getRefreshBoundarySignature(Refresh, prevExports);
     const nextSignature = getRefreshBoundarySignature(Refresh, nextExports);
@@ -896,7 +918,10 @@ if (__DEV__) {
   };
 
   // When this signature changes, it's unsafe to stop at this refresh boundary.
-  var getRefreshBoundarySignature = (Refresh, moduleExports): Array<mixed> => {
+  var getRefreshBoundarySignature = (
+    Refresh: any,
+    moduleExports: Exports,
+  ): Array<mixed> => {
     const signature = [];
     signature.push(Refresh.getFamilyByType(moduleExports));
     if (moduleExports == null || typeof moduleExports !== 'object') {
@@ -919,7 +944,11 @@ if (__DEV__) {
     return signature;
   };
 
-  var registerExportsForReactRefresh = (Refresh, moduleExports, moduleID) => {
+  var registerExportsForReactRefresh = (
+    Refresh: any,
+    moduleExports: Exports,
+    moduleID: ModuleID,
+  ) => {
     Refresh.register(moduleExports, moduleID + ' %exports%');
     if (moduleExports == null || typeof moduleExports !== 'object') {
       // Exit if we can't iterate over exports.