Merge pull request #22 from spatie/feature/react-flare-error-boundary-callback-improvements

add error to onReset and pass componentStack

Author: sebastienhenau

.claude/docs/react-improvements.md

@@ -102,3 +102,5 @@ For the `@flareapp/react` package specifically:
 - [x] FlareErrorBoundary supports resetKeys property
 - [x] FlareErrorBoundary supports fallback with a reset method for resetting the Error Boundary
 - [x] FlareErrorBoundary supports beforeCapture callback
+- [x] FlareErrorBoundary onReset passes previous error
+- [x] FlareErrorBoundary fallback passes componentStack

packages/react/src/FlareErrorBoundary.ts

@@ -5,6 +5,7 @@ import { formatComponentStack } from './format-component-stack';
 
 export type FlareErrorBoundaryFallbackProps = {
     error: Error;
+    componentStack: string[];
     resetErrorBoundary: () => void;
 };
 
@@ -13,17 +14,18 @@ export type FlareErrorBoundaryProps = PropsWithChildren<{
     resetKeys?: unknown[];
     beforeCapture?: (params: { error: Error; errorInfo: ErrorInfo }) => void;
     onError?: (params: { error: Error; errorInfo: ErrorInfo }) => void;
-    onReset?: () => void;
+    onReset?: (error: Error | null) => void;
 }>;
 
 export type FlareErrorBoundaryState = {
     error: Error | null;
+    componentStack: string[];
 };
 
 export class FlareErrorBoundary extends Component<FlareErrorBoundaryProps, FlareErrorBoundaryState> {
-    state: FlareErrorBoundaryState = { error: null };
+    state: FlareErrorBoundaryState = { error: null, componentStack: [] };
 
-    static getDerivedStateFromError(error: Error): FlareErrorBoundaryState {
+    static getDerivedStateFromError(error: Error): Partial<FlareErrorBoundaryState> {
         return { error };
     }
 
@@ -39,6 +41,8 @@ export class FlareErrorBoundary extends Component<FlareErrorBoundaryProps, Flare
             },
         };
 
+        this.setState({ componentStack: context.react.componentStack });
+
         flare.report(error, context, { react: { errorInfo } });
 
         this.props.onError?.({
@@ -64,9 +68,11 @@ export class FlareErrorBoundary extends Component<FlareErrorBoundaryProps, Flare
     }
 
     reset = () => {
-        this.setState({ error: null });
+        const { error } = this.state;
+
+        this.props.onReset?.(error);
 
-        this.props.onReset?.();
+        this.setState({ error: null, componentStack: [] });
     };
 
     render() {
@@ -78,6 +84,7 @@ export class FlareErrorBoundary extends Component<FlareErrorBoundaryProps, Flare
             if (typeof fallback === 'function') {
                 return fallback({
                     error,
+                    componentStack: this.state.componentStack,
                     resetErrorBoundary: this.reset,
                 });
             }