AI Review Center

Commit Review Workspace ?

Select a commit on the left to inspect quality, security, business value, findings, and suggested code changes.
Showing AI reviews for website
Project AI Score ?
71%
Quality Avg ?
72%
Security Avg ?
74%
Reviews ?
123

Review Result ?

solutionbowl/website · c0e832af
The commit improves wallet balance handling with updates to user context and real-time updates via sockets. The added fallback logic and balance update functions add robustness. However, commented-out debug logs remain, and reliance on localStorage without explicit validation may pose some risk. Logging sensitive wallet balances should be handled carefully. Minor formatting and cleanup could enhance maintainability and clarity.
Quality ?
80%
Security ?
60%
Business Value ?
70%
Maintainability ?
80%
Issues & Suggested Fixes ?
1AI detects the issue
2Shows file, line, or evidence
3Suggests the fix to apply
Commented Out Console Logs
Where src/components/racing/HorseRow.tsx:263-271
Issue / Evidence commented-out console logs
Suggested Fix remove or convert to proper debug logging to reduce noise and improve quality
Logging Wallet Balances
Where src/contexts/AuthContext.tsx:223,234,478-480
Issue / Evidence logging wallet balances
Suggested Fix review logs for sensitive data exposure and consider limiting or redacting wallet balance info to improve security
Missing Validation
Where src/contexts/AuthContext.tsx:225-227
Issue / Evidence localStorage usage without validation
Suggested Fix add validation and error handling when parsing stored user data to reduce bug risk
Console Logs And Warnings
Where src/hooks/useLoginSoket.ts:45,216-231
Issue / Evidence console logs and warnings
Suggested Fix consider using a structured logging method or toggling debug logs to improve quality and reduce noise
Very Generic
Where commit message
Issue / Evidence very generic
Suggested Fix improve commit message clarity by explicitly describing the change in wallet balance updates and real-time integration for better business value and reviewability
Code Change Preview · src/components/layout/Header.tsx ?
Removed / Before Commit
- diff --git a/src/components/layout/Header.tsx b/src/components/layout/Header.tsx
- index 51e7303..bcc6eb1 100644
- const colors = useThemeColors();
- const { user, isAuthenticated } = useAuth();
- 
-   const balance = Number(user?.walletBalance ?? 0);
- const balanceDisplay = balance.toLocaleString("en-IN");
- const auth = useAuthModal();
- 
- >
- <i className="fa-solid fa-wallet"></i>
- </div>
-                 </div>
- 
- <button
- onClick={onProfileClick}
Added / After Commit
+ diff --git a/src/components/layout/Header.tsx b/src/components/layout/Header.tsx
+ index 51e7303..bcc6eb1 100644
+ const colors = useThemeColors();
+ const { user, isAuthenticated } = useAuth();
+ 
+   const balance = Number(user?.walletBalance ?? user?.wallet?.balance ?? user?.balance ?? 0);
+ const balanceDisplay = balance.toLocaleString("en-IN");
+ const auth = useAuthModal();
+ 
+ >
+ <i className="fa-solid fa-wallet"></i>
+ </div>
+                 </div>        
+ 
+ <button
+ onClick={onProfileClick}
Removed / Before Commit
- diff --git a/src/components/racing/HorseRow.tsx b/src/components/racing/HorseRow.tsx
- index 99a8d5c..54d199d 100644
- 
- 
- 
-   if (runner.exposure !== undefined || runner.exposureProfit !== undefined) {
-     console.log(`🐴 Runner ${horseData.name}:`, {
-       exposure: runner.exposure,
-       exposureProfit: runner.exposureProfit,
-       hasExposure,
-       profitLoss,
-       marketName
-     })
-   }
- 
- 
- const isWithdrawn = runner.withdrawn === true
Added / After Commit
+ diff --git a/src/components/racing/HorseRow.tsx b/src/components/racing/HorseRow.tsx
+ index 99a8d5c..54d199d 100644
+ 
+ 
+ 
+   // if (runner.exposure !== undefined || runner.exposureProfit !== undefined) {
+   //   console.log(`🐴 Runner ${horseData.name}:`, {
+   //     exposure: runner.exposure,
+   //     exposureProfit: runner.exposureProfit,
+   //     hasExposure,
+   //     profitLoss,
+   //     marketName
+   //   })
+   // }
+ 
+ 
+ const isWithdrawn = runner.withdrawn === true
Removed / Before Commit
- diff --git a/src/contexts/AuthContext.tsx b/src/contexts/AuthContext.tsx
- index 0cc0159..7e1da08 100644
- };
- 
- const getBalanceFromSocketPayload = (data: any): number | null => {
- const rawBalance =
- data?.walletBalance ??
- data?.data?.walletBalance ??
- data?.user?.walletBalance ??
- data?.data?.user?.walletBalance ??
- data?.balance ??
-     data?.data?.balance;
- 
- const walletBalance = Number(rawBalance);
- 
- return Number.isFinite(walletBalance) ? walletBalance : null;
- };
-
Added / After Commit
+ diff --git a/src/contexts/AuthContext.tsx b/src/contexts/AuthContext.tsx
+ index 0cc0159..7e1da08 100644
+ };
+ 
+ const getBalanceFromSocketPayload = (data: any): number | null => {
+   // Direct format: { userId: "...", walletBalance: 9500 }
+ const rawBalance =
+ data?.walletBalance ??
+ data?.data?.walletBalance ??
+ data?.user?.walletBalance ??
+ data?.data?.user?.walletBalance ??
+ data?.balance ??
+     data?.data?.balance ??
+     data?.wallet?.balance ??
+     data?.data?.wallet?.balance;
+ 
+ const walletBalance = Number(rawBalance);
+ return Number.isFinite(walletBalance) ? walletBalance : null;
Removed / Before Commit
- diff --git a/src/hooks/useLoginSoket.ts b/src/hooks/useLoginSoket.ts
- index 4a4e8ec..83c9c99 100644
- onForceLogout?: (data?: unknown) => void;
- 
- onBalanceUpdate?: (data: {
- walletBalance: number;
- }) => void;
- 
- return Number.isFinite(walletBalance) ? walletBalance : null;
- };
- 
- export const useLoginSocket = ({
- userId,
- token,
- "Login socket already connected"
- );
- 
- return;
Added / After Commit
+ diff --git a/src/hooks/useLoginSoket.ts b/src/hooks/useLoginSoket.ts
+ index 4a4e8ec..83c9c99 100644
+ onForceLogout?: (data?: unknown) => void;
+ 
+ onBalanceUpdate?: (data: {
+     userId?: string;
+ walletBalance: number;
+ }) => void;
+ 
+ return Number.isFinite(walletBalance) ? walletBalance : null;
+ };
+ 
+ const joinBalanceRoom = (socket: Socket, userId: string) => {
+   socket.emit("join", { userId });
+   console.log("Joined balance socket room:", { userId });
+ };
+ 
+ export const useLoginSocket = ({
Removed / Before Commit
- diff --git a/src/pages/HorseRacingDetails.tsx b/src/pages/HorseRacingDetails.tsx
- index 02c4db0..1a7a731 100644
- const STANDALONE_FANCY_TYPES = new Set(["OBJECTION", "ODDS & EVENS", "PHOTO"])
- const MARKET_TYPE_DISPLAY_ORDER = [
- "WIN",
-   "2PLACE",
-   "3PLACE",
-   "4PLACE",
- "FANCY",
- "OBJECTION",
- "ODDS & EVENS",
Added / After Commit
+ diff --git a/src/pages/HorseRacingDetails.tsx b/src/pages/HorseRacingDetails.tsx
+ index 02c4db0..1a7a731 100644
+ const STANDALONE_FANCY_TYPES = new Set(["OBJECTION", "ODDS & EVENS", "PHOTO"])
+ const MARKET_TYPE_DISPLAY_ORDER = [
+ "WIN",
+   "2PLACES",
+   "3PLACES",
+   "4PLACES",
+ "FANCY",
+ "OBJECTION",
+ "ODDS & EVENS",