Avoid passing custom stacks to console.error (#18685)

* Detect double stacks in the new format in tests

* Remove unnecessary uses of getStackByFiberInDevAndProd

These all execute in the right execution context already.

* Set the debug fiber around the cases that don't have an execution context

* Remove stack detection in our console log overrides

We never pass custom stacks as part of the args anymore.

* Bonus: Don't append getStackAddendum to invariants

We print component stacks for every error anyway so this is just duplicate
information.

scripts/error-codes/codes.json

@@ -30,7 +30,7 @@
   "28": "Transaction.closeAll(): Cannot close transaction when none are open.",
   "29": "accumulate(...): Accumulated items must be not be null or undefined.",
   "30": "accumulateInto(...): Accumulated items must not be null or undefined.",
-  "31": "Objects are not valid as a React child (found: %s).%s",
+  "31": "Objects are not valid as a React child (found: %s). If you meant to render a collection of children, use an array instead.",
   "32": "Unable to find element with ID %s.",
   "33": "getNodeFromInstance: Invalid argument.",
   "34": "React DOM tree root should always have a node reference.",
@@ -58,10 +58,10 @@
   "56": "dangerouslyReplaceNodeWithMarkup(...): Cannot render markup in a worker thread. Make sure `window` and `document` are available globally before requiring React when unit testing or use ReactDOMServer.renderToString() for server rendering.",
   "57": "dangerouslyReplaceNodeWithMarkup(...): Missing markup.",
   "58": "dangerouslyReplaceNodeWithMarkup(...): Cannot replace markup of the <html> node. This is because browser quirks make this unreliable and/or slow. If you want to render to the root you must use server rendering. See ReactDOMServer.renderToString().",
-  "59": "%s is a void element tag and must not have `children` or use `props.dangerouslySetInnerHTML`.%s",
+  "59": "%s is a void element tag and must not have `children` or use `props.dangerouslySetInnerHTML`.",
   "60": "Can only set one of `children` or `props.dangerouslySetInnerHTML`.",
   "61": "`props.dangerouslySetInnerHTML` must be in the form `{__html: ...}`. Please visit https://fb.me/react-invariant-dangerously-set-inner-html for more information.",
-  "62": "The `style` prop expects a mapping from style properties to values, not a string. For example, style={{marginRight: spacing + 'em'}} when using JSX.%s",
+  "62": "The `style` prop expects a mapping from style properties to values, not a string. For example, style={{marginRight: spacing + 'em'}} when using JSX.",
   "63": "Must be mounted to trap events",
   "64": "trapBubbledEvent(...): Requires node to be rendered.",
   "65": "Invalid tag: %s",
@@ -136,7 +136,7 @@
   "134": "Touch data should have been recorded on start",
   "135": "Cannot find single active touch",
   "136": "Attempted to update component `%s` that has already been unmounted (or failed to mount).",
-  "137": "%s is a void element tag and must neither have `children` nor use `dangerouslySetInnerHTML`.%s",
+  "137": "%s is a void element tag and must neither have `children` nor use `dangerouslySetInnerHTML`.",
   "138": "Touch object is missing identifier.",
   "139": "ReactTestRenderer: .update() can't be called after unmount.",
   "140": "Expected hook events to fire for the child before its parent includes it in onSetChildren().",
@@ -321,7 +321,7 @@
   "320": "Expected ReactFiberErrorDialog.showErrorDialog to be a function.",
   "321": "Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:\n1. You might have mismatching versions of React and the renderer (such as React DOM)\n2. You might be breaking the Rules of Hooks\n3. You might have more than one copy of React in the same app\nSee https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.",
   "322": "forwardRef requires a render function but was given %s.",
-  "323": "React has blocked a javascript: URL as a security precaution.%s",
+  "323": "React has blocked a javascript: URL as a security precaution.",
   "324": "An event responder context was used outside of an event cycle. Use context.setTimeout() to use asynchronous responder context outside of event cycle .",
   "325": "addRootEventTypes() found a duplicate root event type of \"%s\". This might be because the event type exists in the event responder \"rootEventTypes\" array or because of a previous addRootEventTypes() using this root event type.",
   "326": "Expected a valid priority level",