refactor[RendererInterface]: expose onErrorOrWarning and getComponentStack

Author: hoxyq

packages/react-devtools-shared/src/backend/console.js

@@ -7,7 +7,11 @@
  * @flow
  */
 
-import type {ConsolePatchSettings} from './types';
+import type {
+  ConsolePatchSettings,
+  OnErrorOrWarning,
+  GetComponentStack,
+} from './types';
 
 import {
   formatConsoleArguments,
@@ -75,11 +79,6 @@ function restorePotentiallyModifiedArgs(args: Array<any>): Array<any> {
   }
 }
 
-type OnErrorOrWarning = (type: 'error' | 'warn', args: Array<any>) => void;
-type GetComponentStack = (
-  topFrame: Error,
-) => null | {enableOwnerStacks: boolean, componentStack: string};
-
 const injectedRenderers: Array<{
   onErrorOrWarning: ?OnErrorOrWarning,
   getComponentStack: ?GetComponentStack,

packages/react-devtools-shared/src/backend/fiber/renderer.js

@@ -5789,8 +5789,10 @@ export function attach(
     inspectElement,
     logElementToConsole,
     patchConsoleForStrictMode,
+    getComponentStack,
     getElementAttributeByPath,
     getElementSourceFunctionById,
+    onErrorOrWarning,
     overrideError,
     overrideSuspense,
     overrideValueAtPath,

packages/react-devtools-shared/src/backend/flight/renderer.js

@@ -162,6 +162,7 @@ export function attach(
     getBestMatchForTrackedPath() {
       return null;
     },
+    getComponentStack,
     getDisplayNameForElementID() {
       return null;
     },
@@ -207,6 +208,7 @@ export function attach(
     patchConsoleForStrictMode() {},
     getElementAttributeByPath() {},
     getElementSourceFunctionById() {},
+    onErrorOrWarning,
     overrideError() {},
     overrideSuspense() {},
     overrideValueAtPath() {},

packages/react-devtools-shared/src/backend/types.js

@@ -350,6 +350,14 @@ export type InstanceAndStyle = {
 
 type Type = 'props' | 'hooks' | 'state' | 'context';
 
+export type OnErrorOrWarning = (
+  type: 'error' | 'warn',
+  args: Array<any>,
+) => void;
+export type GetComponentStack = (
+  topFrame: Error,
+) => null | {enableOwnerStacks: boolean, componentStack: string};
+
 export type RendererInterface = {
   cleanup: () => void,
   clearErrorsAndWarnings: () => void,
@@ -364,6 +372,7 @@ export type RendererInterface = {
   findHostInstancesForElementID: FindHostInstancesForElementID,
   flushInitialOperations: () => void,
   getBestMatchForTrackedPath: () => PathMatch | null,
+  getComponentStack?: GetComponentStack,
   getNearestMountedDOMNode: (component: Element) => Element | null,
   getElementIDForHostInstance: GetElementIDForHostInstance,
   getDisplayNameForElementID: GetDisplayNameForElementID,
@@ -386,6 +395,7 @@ export type RendererInterface = {
     forceFullData: boolean,
   ) => InspectedElementPayload,
   logElementToConsole: (id: number) => void,
+  onErrorOrWarning?: OnErrorOrWarning,
   overrideError: (id: number, forceError: boolean) => void,
   overrideSuspense: (id: number, forceFallback: boolean) => void,
   overrideValueAtPath: (