Remove method name prefix from warnings and errors (#28432)

This pattern is a petpeeve of mine. I don't consider this best practice
and so most don't have these prefixes. Very inconsistent.

At best this is useless and noisey that you have to parse because the
information is also in the stack trace.

At worse these are misleading because they're highlighting something
internal (like validateDOMNesting) which even suggests an internal bug.
Even the ones public to React aren't necessarily what you called because
you might be calling a wrapper around it.

That would be properly reflected in a stack trace - which can also
properly ignore list so that the first stack you see is your callsite,

Which might be like `render()` in react-testing-library rather than
`createRoot()` for example.

DiffTrain build for [d579e77](d579e77)

Author: sebmarkbage

compiled/facebook-www/REVISION

@@ -1 +1 @@
-8fb0233a845974b4b1049e54b6c25dc54d6dd173
+d579e7748218920331252b0528850943d5e2dd31

compiled/facebook-www/React-dev.classic.js

@@ -24,7 +24,7 @@ if (__DEV__) {
     ) {
       __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStart(new Error());
     }
-    var ReactVersion = "18.3.0-www-classic-b8eb15b2";
+    var ReactVersion = "18.3.0-www-classic-7910dce1";
 
     // ATTENTION
     // When adding new symbols to this file,
@@ -289,7 +289,7 @@ if (__DEV__) {
         partialState != null
       ) {
         throw new Error(
-          "setState(...): takes an object of state variables to update or a " +
+          "takes an object of state variables to update or a " +
             "function which returns an object of state variables."
         );
       }
@@ -1932,7 +1932,7 @@ if (__DEV__) {
     function cloneElement(element, config, children) {
       if (element === null || element === undefined) {
         throw new Error(
-          "React.cloneElement(...): The argument must be a React element, but you passed " +
+          "The argument must be a React element, but you passed " +
             element +
             "."
         );
@@ -2859,7 +2859,7 @@ if (__DEV__) {
             // $FlowFixMe[missing-local-annot]
             set: function (newDefaultProps) {
               error(
-                "React.lazy(...): It is not supported to assign `defaultProps` to " +
+                "It is not supported to assign `defaultProps` to " +
                   "a lazy component import. Either specify them where the component " +
                   "is defined, or create a wrapping component around it."
               );
@@ -2880,7 +2880,7 @@ if (__DEV__) {
             // $FlowFixMe[missing-local-annot]
             set: function (newPropTypes) {
               error(
-                "React.lazy(...): It is not supported to assign `propTypes` to " +
+                "It is not supported to assign `propTypes` to " +
                   "a lazy component import. Either specify them where the component " +
                   "is defined, or create a wrapping component around it."
               );

compiled/facebook-www/React-dev.modern.js

@@ -24,7 +24,7 @@ if (__DEV__) {
     ) {
       __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStart(new Error());
     }
-    var ReactVersion = "18.3.0-www-modern-5ca55b0c";
+    var ReactVersion = "18.3.0-www-modern-a1866c69";
 
     // ATTENTION
     // When adding new symbols to this file,
@@ -289,7 +289,7 @@ if (__DEV__) {
         partialState != null
       ) {
         throw new Error(
-          "setState(...): takes an object of state variables to update or a " +
+          "takes an object of state variables to update or a " +
             "function which returns an object of state variables."
         );
       }
@@ -1888,7 +1888,7 @@ if (__DEV__) {
     function cloneElement(element, config, children) {
       if (element === null || element === undefined) {
         throw new Error(
-          "React.cloneElement(...): The argument must be a React element, but you passed " +
+          "The argument must be a React element, but you passed " +
             element +
             "."
         );
@@ -2815,7 +2815,7 @@ if (__DEV__) {
             // $FlowFixMe[missing-local-annot]
             set: function (newDefaultProps) {
               error(
-                "React.lazy(...): It is not supported to assign `defaultProps` to " +
+                "It is not supported to assign `defaultProps` to " +
                   "a lazy component import. Either specify them where the component " +
                   "is defined, or create a wrapping component around it."
               );
@@ -2836,7 +2836,7 @@ if (__DEV__) {
             // $FlowFixMe[missing-local-annot]
             set: function (newPropTypes) {
               error(
-                "React.lazy(...): It is not supported to assign `propTypes` to " +
+                "It is not supported to assign `propTypes` to " +
                   "a lazy component import. Either specify them where the component " +
                   "is defined, or create a wrapping component around it."
               );

compiled/facebook-www/React-prod.classic.js

@@ -62,7 +62,7 @@ Component.prototype.setState = function (partialState, callback) {
     null != partialState
   )
     throw Error(
-      "setState(...): takes an object of state variables to update or a function which returns an object of state variables."
+      "takes an object of state variables to update or a function which returns an object of state variables."
     );
   this.updater.enqueueSetState(this, partialState, callback, "setState");
 };
@@ -433,9 +433,7 @@ exports.cache = function (fn) {
 exports.cloneElement = function (element, config, children) {
   if (null === element || void 0 === element)
     throw Error(
-      "React.cloneElement(...): The argument must be a React element, but you passed " +
-        element +
-        "."
+      "The argument must be a React element, but you passed " + element + "."
     );
   var props = assign({}, element.props),
     key = element.key,
@@ -635,4 +633,4 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
   return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-www-classic-59a8d1a0";
+exports.version = "18.3.0-www-classic-16574b61";

compiled/facebook-www/React-prod.modern.js

@@ -61,7 +61,7 @@ Component.prototype.setState = function (partialState, callback) {
     null != partialState
   )
     throw Error(
-      "setState(...): takes an object of state variables to update or a function which returns an object of state variables."
+      "takes an object of state variables to update or a function which returns an object of state variables."
     );
   this.updater.enqueueSetState(this, partialState, callback, "setState");
 };
@@ -394,9 +394,7 @@ exports.cache = function (fn) {
 exports.cloneElement = function (element, config, children) {
   if (null === element || void 0 === element)
     throw Error(
-      "React.cloneElement(...): The argument must be a React element, but you passed " +
-        element +
-        "."
+      "The argument must be a React element, but you passed " + element + "."
     );
   var props = assign({}, element.props),
     key = element.key,
@@ -627,4 +625,4 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
   return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-www-modern-16a3d0dd";
+exports.version = "18.3.0-www-modern-9a917e2a";

compiled/facebook-www/React-profiling.classic.js

@@ -66,7 +66,7 @@ Component.prototype.setState = function (partialState, callback) {
     null != partialState
   )
     throw Error(
-      "setState(...): takes an object of state variables to update or a function which returns an object of state variables."
+      "takes an object of state variables to update or a function which returns an object of state variables."
     );
   this.updater.enqueueSetState(this, partialState, callback, "setState");
 };
@@ -437,9 +437,7 @@ exports.cache = function (fn) {
 exports.cloneElement = function (element, config, children) {
   if (null === element || void 0 === element)
     throw Error(
-      "React.cloneElement(...): The argument must be a React element, but you passed " +
-        element +
-        "."
+      "The argument must be a React element, but you passed " + element + "."
     );
   var props = assign({}, element.props),
     key = element.key,
@@ -639,7 +637,7 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
   return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-www-classic-d95f23f2";
+exports.version = "18.3.0-www-classic-3a8f9fbb";
 "undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
   "function" ===
     typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&

compiled/facebook-www/React-profiling.modern.js

@@ -65,7 +65,7 @@ Component.prototype.setState = function (partialState, callback) {
     null != partialState
   )
     throw Error(
-      "setState(...): takes an object of state variables to update or a function which returns an object of state variables."
+      "takes an object of state variables to update or a function which returns an object of state variables."
     );
   this.updater.enqueueSetState(this, partialState, callback, "setState");
 };
@@ -398,9 +398,7 @@ exports.cache = function (fn) {
 exports.cloneElement = function (element, config, children) {
   if (null === element || void 0 === element)
     throw Error(
-      "React.cloneElement(...): The argument must be a React element, but you passed " +
-        element +
-        "."
+      "The argument must be a React element, but you passed " + element + "."
     );
   var props = assign({}, element.props),
     key = element.key,
@@ -631,7 +629,7 @@ exports.useSyncExternalStore = function (
 exports.useTransition = function () {
   return ReactCurrentDispatcher.current.useTransition();
 };
-exports.version = "18.3.0-www-modern-12626cf3";
+exports.version = "18.3.0-www-modern-1944f24a";
 "undefined" !== typeof __REACT_DEVTOOLS_GLOBAL_HOOK__ &&
   "function" ===
     typeof __REACT_DEVTOOLS_GLOBAL_HOOK__.registerInternalModuleStop &&

compiled/facebook-www/ReactART-dev.classic.js

@@ -66,7 +66,7 @@ if (__DEV__) {
       return self;
     }
 
-    var ReactVersion = "18.3.0-www-classic-f7f3772e";
+    var ReactVersion = "18.3.0-www-classic-b340e372";
 
     var LegacyRoot = 0;
     var ConcurrentRoot = 1;
@@ -12626,21 +12626,20 @@ if (__DEV__) {
       Object.freeze(fakeInternalInstance);
     }
 
-    function warnOnInvalidCallback(callback, callerName) {
+    function warnOnInvalidCallback(callback) {
       {
         if (callback === null || typeof callback === "function") {
           return;
-        }
+        } // eslint-disable-next-line react-internal/safe-string-coercion
 
-        var key = callerName + "_" + callback;
+        var key = String(callback);
 
         if (!didWarnOnInvalidCallback.has(key)) {
           didWarnOnInvalidCallback.add(key);
 
           error(
-            "%s(...): Expected the last optional `callback` argument to be a " +
+            "Expected the last optional `callback` argument to be a " +
               "function. Instead received: %s.",
-            callerName,
             callback
           );
         }
@@ -12714,7 +12713,7 @@ if (__DEV__) {
 
         if (callback !== undefined && callback !== null) {
           {
-            warnOnInvalidCallback(callback, "setState");
+            warnOnInvalidCallback(callback);
           }
 
           update.callback = callback;
@@ -12749,7 +12748,7 @@ if (__DEV__) {
 
         if (callback !== undefined && callback !== null) {
           {
-            warnOnInvalidCallback(callback, "replaceState");
+            warnOnInvalidCallback(callback);
           }
 
           update.callback = callback;
@@ -12784,7 +12783,7 @@ if (__DEV__) {
 
         if (callback !== undefined && callback !== null) {
           {
-            warnOnInvalidCallback(callback, "forceUpdate");
+            warnOnInvalidCallback(callback);
           }
 
           update.callback = callback;
@@ -12877,13 +12876,13 @@ if (__DEV__) {
         if (!renderPresent) {
           if (ctor.prototype && typeof ctor.prototype.render === "function") {
             error(
-              "%s(...): No `render` method found on the returned component " +
+              "No `render` method found on the %s " +
                 "instance: did you accidentally return an object from the constructor?",
               name
             );
           } else {
             error(
-              "%s(...): No `render` method found on the returned component " +
+              "No `render` method found on the %s " +
                 "instance: you may have forgotten to define `render`.",
               name
             );
@@ -13018,9 +13017,8 @@ if (__DEV__) {
 
         if (instance.props !== undefined && hasMutatedProps) {
           error(
-            "%s(...): When calling super() in `%s`, make sure to pass " +
+            "When calling super() in `%s`, make sure to pass " +
               "up the same props that your component's constructor was passed.",
-            name,
             name
           );
         }
@@ -16065,7 +16063,8 @@ if (__DEV__) {
         if (Component) {
           if (Component.childContextTypes) {
             error(
-              "%s(...): childContextTypes cannot be defined on a function component.",
+              "childContextTypes cannot be defined on a function component.\n" +
+                "  %s.childContextTypes = ...",
               Component.displayName || Component.name || "Component"
             );
           }
@@ -30184,7 +30183,7 @@ if (__DEV__) {
         {
           if (typeof callback !== "function") {
             error(
-              "render(...): Expected the last optional `callback` argument to be a " +
+              "Expected the last optional `callback` argument to be a " +
                 "function. Instead received: %s.",
               callback
             );