Summary
BrowserRouter, ThemeProvider, and CssBaseline are instantiated in both main.tsx and App.tsx, creating nested routers and redundant contexts.
Impact
- Double routing can cause navigation bugs
- Extra renders from redundant context providers
- Potential performance issues
Files Affected
Recommendation
Consolidate them into a single top-level wrapper (typically in main.tsx) and render App inside once to avoid double routing and extra renders.
Source: Code Review (PR #5)
Summary
BrowserRouter,ThemeProvider, andCssBaselineare instantiated in bothmain.tsxandApp.tsx, creating nested routers and redundant contexts.Impact
Files Affected
src/main.tsxsrc/App.tsxRecommendation
Consolidate them into a single top-level wrapper (typically in
main.tsx) and renderAppinside once to avoid double routing and extra renders.Source: Code Review (PR #5)