Skip to content

Commit

Permalink
Disable infinite render loop detection (#31088)
Browse files Browse the repository at this point in the history
We're seeing issues with this feature internally including bugs with
sibling prerendering and errors that are difficult for developers to
action on. We'll turn off the feature for the time being until we can
improve the stability and ergonomics.

This PR does two things:
- Turn off `enableInfiniteLoopDetection` everywhere while leaving it as
a variant on www so we can do further experimentation.
- Revert #31061 which was a
temporary change for debugging. This brings the feature back to
baseline.

DiffTrain build for [d8c90fa](d8c90fa)
  • Loading branch information
jackpope committed Oct 1, 2024
1 parent e2de2ad commit 56bda35
Show file tree
Hide file tree
Showing 34 changed files with 116 additions and 116 deletions.
2 changes: 1 addition & 1 deletion compiled/facebook-www/REVISION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
67fee58b1f72754cc77488c40c44e786572ef954
d8c90fa48d3addefe4b805ec56a3c65e4ee39127
2 changes: 1 addition & 1 deletion compiled/facebook-www/REVISION_TRANSFORMS
Original file line number Diff line number Diff line change
@@ -1 +1 @@
67fee58b1f72754cc77488c40c44e786572ef954
d8c90fa48d3addefe4b805ec56a3c65e4ee39127
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -2000,7 +2000,7 @@ __DEV__ &&
exports.useTransition = function () {
return resolveDispatcher().useTransition();
};
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -1980,7 +1980,7 @@ __DEV__ &&
exports.useTransition = function () {
return resolveDispatcher().useTransition();
};
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-prod.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -665,4 +665,4 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-prod.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -665,4 +665,4 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-profiling.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -669,7 +669,7 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
2 changes: 1 addition & 1 deletion compiled/facebook-www/React-profiling.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -669,7 +669,7 @@ exports.useSyncExternalStore = function (
exports.useTransition = function () {
return ReactSharedInternals.H.useTransition();
};
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
10 changes: 5 additions & 5 deletions compiled/facebook-www/ReactART-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -16779,12 +16779,12 @@ __DEV__ &&
pendingPassiveEffectsLanes = 0,
pendingPassiveEffectsRemainingLanes = 0,
pendingPassiveTransitions = null,
NESTED_UPDATE_LIMIT = 100,
NESTED_UPDATE_LIMIT = 50,
nestedUpdateCount = 0,
rootWithNestedUpdates = null,
isFlushingPassiveEffects = !1,
didScheduleUpdateDuringPassiveEffects = !1,
NESTED_PASSIVE_UPDATE_LIMIT = 100,
NESTED_PASSIVE_UPDATE_LIMIT = 50,
nestedPassiveUpdateCount = 0,
rootWithPassiveNestedUpdates = null,
isRunningInsertionEffect = !1,
Expand Down Expand Up @@ -17014,11 +17014,11 @@ __DEV__ &&
(function () {
var internals = {
bundleType: 1,
version: "19.0.0-www-classic-67fee58b-20240926",
version: "19.0.0-www-classic-d8c90fa4-20241001",
rendererPackageName: "react-art",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getInstanceFromNode,
reconcilerVersion: "19.0.0-www-classic-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-classic-d8c90fa4-20241001"
};
internals.overrideHookState = overrideHookState;
internals.overrideHookStateDeletePath = overrideHookStateDeletePath;
Expand Down Expand Up @@ -17052,7 +17052,7 @@ __DEV__ &&
exports.Shape = Shape;
exports.Surface = Surface;
exports.Text = Text;
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
10 changes: 5 additions & 5 deletions compiled/facebook-www/ReactART-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -16223,12 +16223,12 @@ __DEV__ &&
pendingPassiveEffectsLanes = 0,
pendingPassiveEffectsRemainingLanes = 0,
pendingPassiveTransitions = null,
NESTED_UPDATE_LIMIT = 100,
NESTED_UPDATE_LIMIT = 50,
nestedUpdateCount = 0,
rootWithNestedUpdates = null,
isFlushingPassiveEffects = !1,
didScheduleUpdateDuringPassiveEffects = !1,
NESTED_PASSIVE_UPDATE_LIMIT = 100,
NESTED_PASSIVE_UPDATE_LIMIT = 50,
nestedPassiveUpdateCount = 0,
rootWithPassiveNestedUpdates = null,
isRunningInsertionEffect = !1,
Expand Down Expand Up @@ -16457,11 +16457,11 @@ __DEV__ &&
(function () {
var internals = {
bundleType: 1,
version: "19.0.0-www-modern-67fee58b-20240926",
version: "19.0.0-www-modern-d8c90fa4-20241001",
rendererPackageName: "react-art",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getInstanceFromNode,
reconcilerVersion: "19.0.0-www-modern-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-modern-d8c90fa4-20241001"
};
internals.overrideHookState = overrideHookState;
internals.overrideHookStateDeletePath = overrideHookStateDeletePath;
Expand Down Expand Up @@ -16495,7 +16495,7 @@ __DEV__ &&
exports.Shape = Shape;
exports.Surface = Surface;
exports.Text = Text;
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
8 changes: 4 additions & 4 deletions compiled/facebook-www/ReactART-prod.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -10251,7 +10251,7 @@ function resolveRetryWakeable(boundaryFiber, wakeable) {
retryTimedOutBoundary(boundaryFiber, retryLane);
}
function throwIfInfiniteUpdateLoopDetected() {
if (100 < nestedUpdateCount)
if (50 < nestedUpdateCount)
throw (
((nestedUpdateCount = 0),
(rootWithNestedUpdates = null),
Expand Down Expand Up @@ -10826,13 +10826,13 @@ var slice = Array.prototype.slice,
})(React.Component);
var internals$jscomp$inline_1467 = {
bundleType: 0,
version: "19.0.0-www-classic-67fee58b-20240926",
version: "19.0.0-www-classic-d8c90fa4-20241001",
rendererPackageName: "react-art",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: function () {
return null;
},
reconcilerVersion: "19.0.0-www-classic-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-classic-d8c90fa4-20241001"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_1468 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand All @@ -10858,4 +10858,4 @@ exports.RadialGradient = RadialGradient;
exports.Shape = TYPES.SHAPE;
exports.Surface = Surface;
exports.Text = Text;
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
8 changes: 4 additions & 4 deletions compiled/facebook-www/ReactART-prod.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -9805,7 +9805,7 @@ function resolveRetryWakeable(boundaryFiber, wakeable) {
retryTimedOutBoundary(boundaryFiber, retryLane);
}
function throwIfInfiniteUpdateLoopDetected() {
if (100 < nestedUpdateCount)
if (50 < nestedUpdateCount)
throw (
((nestedUpdateCount = 0),
(rootWithNestedUpdates = null),
Expand Down Expand Up @@ -10337,13 +10337,13 @@ var slice = Array.prototype.slice,
})(React.Component);
var internals$jscomp$inline_1459 = {
bundleType: 0,
version: "19.0.0-www-modern-67fee58b-20240926",
version: "19.0.0-www-modern-d8c90fa4-20241001",
rendererPackageName: "react-art",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: function () {
return null;
},
reconcilerVersion: "19.0.0-www-modern-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-modern-d8c90fa4-20241001"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_1460 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand All @@ -10369,4 +10369,4 @@ exports.RadialGradient = RadialGradient;
exports.Shape = TYPES.SHAPE;
exports.Surface = Surface;
exports.Text = Text;
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
14 changes: 7 additions & 7 deletions compiled/facebook-www/ReactDOM-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -26943,12 +26943,12 @@ __DEV__ &&
pendingPassiveEffectsLanes = 0,
pendingPassiveEffectsRemainingLanes = 0,
pendingPassiveTransitions = null,
NESTED_UPDATE_LIMIT = 100,
NESTED_UPDATE_LIMIT = 50,
nestedUpdateCount = 0,
rootWithNestedUpdates = null,
isFlushingPassiveEffects = !1,
didScheduleUpdateDuringPassiveEffects = !1,
NESTED_PASSIVE_UPDATE_LIMIT = 100,
NESTED_PASSIVE_UPDATE_LIMIT = 50,
nestedPassiveUpdateCount = 0,
rootWithPassiveNestedUpdates = null,
isRunningInsertionEffect = !1,
Expand Down Expand Up @@ -27771,11 +27771,11 @@ __DEV__ &&
: flushSyncErrorInBuildsThatSupportLegacyMode;
(function () {
var isomorphicReactPackageVersion = React.version;
if ("19.0.0-www-classic-67fee58b-20240926" !== isomorphicReactPackageVersion)
if ("19.0.0-www-classic-d8c90fa4-20241001" !== isomorphicReactPackageVersion)
throw Error(
'Incompatible React versions: The "react" and "react-dom" packages must have the exact same version. Instead got:\n - react: ' +
(isomorphicReactPackageVersion +
"\n - react-dom: 19.0.0-www-classic-67fee58b-20240926\nLearn more: https://react.dev/warnings/version-mismatch")
"\n - react-dom: 19.0.0-www-classic-d8c90fa4-20241001\nLearn more: https://react.dev/warnings/version-mismatch")
);
})();
("function" === typeof Map &&
Expand Down Expand Up @@ -27819,11 +27819,11 @@ __DEV__ &&
!(function () {
var internals = {
bundleType: 1,
version: "19.0.0-www-classic-67fee58b-20240926",
version: "19.0.0-www-classic-d8c90fa4-20241001",
rendererPackageName: "react-dom",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getClosestInstanceFromNode,
reconcilerVersion: "19.0.0-www-classic-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-classic-d8c90fa4-20241001"
};
internals.overrideHookState = overrideHookState;
internals.overrideHookStateDeletePath = overrideHookStateDeletePath;
Expand Down Expand Up @@ -28476,7 +28476,7 @@ __DEV__ &&
exports.useFormStatus = function () {
return resolveDispatcher().useHostTransitionStatus();
};
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
14 changes: 7 additions & 7 deletions compiled/facebook-www/ReactDOM-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -26057,12 +26057,12 @@ __DEV__ &&
pendingPassiveEffectsLanes = 0,
pendingPassiveEffectsRemainingLanes = 0,
pendingPassiveTransitions = null,
NESTED_UPDATE_LIMIT = 100,
NESTED_UPDATE_LIMIT = 50,
nestedUpdateCount = 0,
rootWithNestedUpdates = null,
isFlushingPassiveEffects = !1,
didScheduleUpdateDuringPassiveEffects = !1,
NESTED_PASSIVE_UPDATE_LIMIT = 100,
NESTED_PASSIVE_UPDATE_LIMIT = 50,
nestedPassiveUpdateCount = 0,
rootWithPassiveNestedUpdates = null,
isRunningInsertionEffect = !1,
Expand Down Expand Up @@ -26885,11 +26885,11 @@ __DEV__ &&
return_targetInst = null;
(function () {
var isomorphicReactPackageVersion = React.version;
if ("19.0.0-www-modern-67fee58b-20240926" !== isomorphicReactPackageVersion)
if ("19.0.0-www-modern-d8c90fa4-20241001" !== isomorphicReactPackageVersion)
throw Error(
'Incompatible React versions: The "react" and "react-dom" packages must have the exact same version. Instead got:\n - react: ' +
(isomorphicReactPackageVersion +
"\n - react-dom: 19.0.0-www-modern-67fee58b-20240926\nLearn more: https://react.dev/warnings/version-mismatch")
"\n - react-dom: 19.0.0-www-modern-d8c90fa4-20241001\nLearn more: https://react.dev/warnings/version-mismatch")
);
})();
("function" === typeof Map &&
Expand Down Expand Up @@ -26932,11 +26932,11 @@ __DEV__ &&
!(function () {
var internals = {
bundleType: 1,
version: "19.0.0-www-modern-67fee58b-20240926",
version: "19.0.0-www-modern-d8c90fa4-20241001",
rendererPackageName: "react-dom",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getClosestInstanceFromNode,
reconcilerVersion: "19.0.0-www-modern-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-modern-d8c90fa4-20241001"
};
internals.overrideHookState = overrideHookState;
internals.overrideHookStateDeletePath = overrideHookStateDeletePath;
Expand Down Expand Up @@ -27541,7 +27541,7 @@ __DEV__ &&
exports.useFormStatus = function () {
return resolveDispatcher().useHostTransitionStatus();
};
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
"undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
"function" ===
typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&
Expand Down
12 changes: 6 additions & 6 deletions compiled/facebook-www/ReactDOM-prod.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -12210,7 +12210,7 @@ function resolveRetryWakeable(boundaryFiber, wakeable) {
retryTimedOutBoundary(boundaryFiber, retryLane);
}
function throwIfInfiniteUpdateLoopDetected() {
if (100 < nestedUpdateCount)
if (50 < nestedUpdateCount)
throw (
((nestedUpdateCount = 0),
(rootWithNestedUpdates = null),
Expand Down Expand Up @@ -17412,14 +17412,14 @@ function getCrossOriginStringAs(as, input) {
}
var isomorphicReactPackageVersion$jscomp$inline_1774 = React.version;
if (
"19.0.0-www-classic-67fee58b-20240926" !==
"19.0.0-www-classic-d8c90fa4-20241001" !==
isomorphicReactPackageVersion$jscomp$inline_1774
)
throw Error(
formatProdErrorMessage(
527,
isomorphicReactPackageVersion$jscomp$inline_1774,
"19.0.0-www-classic-67fee58b-20240926"
"19.0.0-www-classic-d8c90fa4-20241001"
)
);
function flushSyncFromReconciler(fn) {
Expand Down Expand Up @@ -17464,11 +17464,11 @@ Internals.Events = [
];
var internals$jscomp$inline_2287 = {
bundleType: 0,
version: "19.0.0-www-classic-67fee58b-20240926",
version: "19.0.0-www-classic-d8c90fa4-20241001",
rendererPackageName: "react-dom",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getClosestInstanceFromNode,
reconcilerVersion: "19.0.0-www-classic-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-classic-d8c90fa4-20241001"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_2288 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand Down Expand Up @@ -17924,4 +17924,4 @@ exports.useFormState = function (action, initialState, permalink) {
exports.useFormStatus = function () {
return ReactSharedInternals.H.useHostTransitionStatus();
};
exports.version = "19.0.0-www-classic-67fee58b-20240926";
exports.version = "19.0.0-www-classic-d8c90fa4-20241001";
12 changes: 6 additions & 6 deletions compiled/facebook-www/ReactDOM-prod.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -11720,7 +11720,7 @@ function resolveRetryWakeable(boundaryFiber, wakeable) {
retryTimedOutBoundary(boundaryFiber, retryLane);
}
function throwIfInfiniteUpdateLoopDetected() {
if (100 < nestedUpdateCount)
if (50 < nestedUpdateCount)
throw (
((nestedUpdateCount = 0),
(rootWithNestedUpdates = null),
Expand Down Expand Up @@ -16780,14 +16780,14 @@ function getCrossOriginStringAs(as, input) {
}
var isomorphicReactPackageVersion$jscomp$inline_1745 = React.version;
if (
"19.0.0-www-modern-67fee58b-20240926" !==
"19.0.0-www-modern-d8c90fa4-20241001" !==
isomorphicReactPackageVersion$jscomp$inline_1745
)
throw Error(
formatProdErrorMessage(
527,
isomorphicReactPackageVersion$jscomp$inline_1745,
"19.0.0-www-modern-67fee58b-20240926"
"19.0.0-www-modern-d8c90fa4-20241001"
)
);
Internals.findDOMNode = function (componentOrElement) {
Expand All @@ -16805,11 +16805,11 @@ Internals.Events = [
];
var internals$jscomp$inline_2278 = {
bundleType: 0,
version: "19.0.0-www-modern-67fee58b-20240926",
version: "19.0.0-www-modern-d8c90fa4-20241001",
rendererPackageName: "react-dom",
currentDispatcherRef: ReactSharedInternals,
findFiberByHostInstance: getClosestInstanceFromNode,
reconcilerVersion: "19.0.0-www-modern-67fee58b-20240926"
reconcilerVersion: "19.0.0-www-modern-d8c90fa4-20241001"
};
if ("undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__) {
var hook$jscomp$inline_2279 = __REACT_DEVTOOLS_GLOBAL_HOOK__;
Expand Down Expand Up @@ -17172,4 +17172,4 @@ exports.useFormState = function (action, initialState, permalink) {
exports.useFormStatus = function () {
return ReactSharedInternals.H.useHostTransitionStatus();
};
exports.version = "19.0.0-www-modern-67fee58b-20240926";
exports.version = "19.0.0-www-modern-d8c90fa4-20241001";
Loading

0 comments on commit 56bda35

Please sign in to comment.