Finish cleaning up digest from onRecoverableError

Don't need to track it separately on the captured value anymore.

Shouldn't be in the types.

I used a getter for the warning instead because Proxies are kind of heavy
weight options for this kind of warning. We typically use getters.

Author: sebmarkbage

packages/react-reconciler/src/ReactCapturedValue.js

@@ -17,7 +17,6 @@ export type CapturedValue<T> = {
   +value: T,
   source: Fiber | null,
   stack: string | null,
-  digest: string | null,
 };
 
 export function createCapturedValueAtFiber<T>(
@@ -43,22 +42,19 @@ export function createCapturedValueAtFiber<T>(
     value,
     source,
     stack,
-    digest: null,
   };
 }
 
 export function createCapturedValueFromError(
   value: Error,
-  digest: ?string,
-  stack: ?string,
+  stack: null | string,
 ): CapturedValue<Error> {
   if (typeof stack === 'string') {
     CapturedStacks.set(value, stack);
   }
   return {
     value,
     source: null,
-    stack: stack != null ? stack : null,
-    digest: digest != null ? digest : null,
+    stack: stack,
   };
 }

packages/react-reconciler/src/ReactFiberBeginWork.js

@@ -2647,7 +2647,8 @@ function updateDehydratedSuspenseComponent(
       // get an update and we'll never be able to hydrate the final content. Let's just try the
       // client side render instead.
       let digest: ?string;
-      let message, stack;
+      let message;
+      let stack = null;
       if (__DEV__) {
         ({digest, message, stack} =
           getSuspenseInstanceFallbackErrorDetails(suspenseInstance));
@@ -2670,7 +2671,10 @@ function updateDehydratedSuspenseComponent(
           );
         }
         (error: any).digest = digest;
-        capturedValue = createCapturedValueFromError(error, digest, stack);
+        capturedValue = createCapturedValueFromError(
+          error,
+          stack === undefined ? null : stack,
+        );
       }
       return retrySuspenseComponentWithoutHydrating(
         current,
@@ -2812,6 +2816,7 @@ function updateDehydratedSuspenseComponent(
           'There was an error while hydrating this Suspense boundary. ' +
             'Switched to client rendering.',
         ),
+        null,
       );
       return retrySuspenseComponentWithoutHydrating(
         current,

packages/react-reconciler/src/ReactFiberWorkLoop.js

@@ -3121,37 +3121,21 @@ function commitRootImpl(
 }
 
 function makeErrorInfo(componentStack: ?string) {
+  const errorInfo = {
+    componentStack,
+  };
   if (__DEV__) {
-    const errorInfo = {
-      componentStack,
-    };
-    return new Proxy(errorInfo, {
-      get(target, prop, receiver) {
-        if (prop === 'digest') {
-          console.error(
-            'You are accessing "digest" from the errorInfo object passed to onRecoverableError.' +
-              ' This property is no longer provided as part of errorInfo but can be accessed as a property' +
-              ' of the Error instance itself.',
-          );
-        }
-        return Reflect.get(target, prop, receiver);
-      },
-      has(target, prop) {
-        if (prop === 'digest') {
-          console.error(
-            'You are accessing "digest" from the errorInfo object passed to onRecoverableError.' +
-              ' This property is no longer provided as part of errorInfo but can be accessed as a property' +
-              ' of the Error instance itself.',
-          );
-        }
-        return Reflect.has(target, prop);
+    Object.defineProperty((errorInfo: any), 'digest', {
+      get() {
+        console.error(
+          'You are accessing "digest" from the errorInfo object passed to onRecoverableError.' +
+            ' This property is no longer provided as part of errorInfo but can be accessed as a property' +
+            ' of the Error instance itself.',
+        );
       },
     });
-  } else {
-    return {
-      componentStack,
-    };
   }
+  return errorInfo;
 }
 
 function releaseRootPooledCache(root: FiberRoot, remainingLanes: Lanes) {

packages/react-reconciler/src/ReactInternalTypes.js

@@ -273,7 +273,7 @@ type BaseFiberRootProperties = {
   ) => void,
   onRecoverableError: (
     error: mixed,
-    errorInfo: {+digest?: ?string, +componentStack?: ?string},
+    errorInfo: {+componentStack?: ?string},
   ) => void,
 
   formState: ReactFormState<any, any> | null,