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 · caf43375
The commit removes the INR symbol and replaces amounts with fixed decimal formatting across multiple UI components. There is also an addition of logic to render jersey avatars with fallback error handling. While the changes improve UI consistency and handle image loading errors, the commit message is vague and does not convey the scope of changes, making it harder to understand the business motivation. The numerous UI changes scattered across many files raise the chance of regression bugs if not carefully tested.
Quality ?
60%
Security ?
100%
Business Value ?
40%
Maintainability ?
70%
Issues & Suggested Fixes ?
1AI detects the issue
2Shows file, line, or evidence
3Suggests the fix to apply
Vague
Where commit message
Issue / Evidence vague
Suggested Fix improve message to summarize scope and intent of UI formatting and jersey avatar additions
Missing Comment
Where src/components/racing/HorseRow.tsx:584
Issue / Evidence missing comment
Suggested Fix add explanatory comments for jersey image error state and rendering logic
Inconsistent Ui Updates
Where src/components/bets/BetItem.tsx:112
Issue / Evidence inconsistent UI updates
Suggested Fix review that removing INR symbols aligns with business requirements for all amounts
Numeric Formatting
Where src/components/racing/HorseRow.tsx:301
Issue / Evidence numeric formatting
Suggested Fix consider centralizing numeric formatting logic to prevent duplication and ensure consistency
Onerror Handler
Where src/components/racing/HorseRow.tsx:594
Issue / Evidence onError handler
Suggested Fix ensure error fallback properly tested to avoid broken UI
Code Change Preview · src/components/bets/BetItem.tsx ?
Removed / Before Commit
- diff --git a/src/components/bets/BetItem.tsx b/src/components/bets/BetItem.tsx
- index 0dc7f58..b9d0c59 100644
- >
- <span>Stake</span>
- <span className="shrink-0 text-right text-[13px] font-bold" style={{ color: colors.primary }}>
-             &#8377;{potentialWin}
- </span>
- </label>
Added / After Commit
+ diff --git a/src/components/bets/BetItem.tsx b/src/components/bets/BetItem.tsx
+ index 0dc7f58..b9d0c59 100644
+ >
+ <span>Stake</span>
+ <span className="shrink-0 text-right text-[13px] font-bold" style={{ color: colors.primary }}>
+             {potentialWin}
+ </span>
+ </label>
Removed / Before Commit
- diff --git a/src/components/bets/BetSlipDrawer.tsx b/src/components/bets/BetSlipDrawer.tsx
- index 62bfba3..72be023 100644
- </span>
- </div>
- <div className="flex justify-between text-[12px]">
-                     <span style={{ color: colors.textSecondary }}>Potential Win</span>
- <span style={{ color: colors.primary }} className="font-semibold">
- {potentialWinnings.toFixed(2)}
- </span>
- </span>
- </div>
- <div className="flex justify-between text-[12px]">
-                   <span style={{ color: colors.textSecondary }}>Potential Win</span>
- <span style={{ color: colors.primary }} className="font-semibold">
- {potentialWinnings.toFixed(2)}
- </span>
Added / After Commit
+ diff --git a/src/components/bets/BetSlipDrawer.tsx b/src/components/bets/BetSlipDrawer.tsx
+ index 62bfba3..72be023 100644
+ </span>
+ </div>
+ <div className="flex justify-between text-[12px]">
+                     <span style={{ color: colors.textSecondary }}>Potential Win </span>
+ <span style={{ color: colors.primary }} className="font-semibold">
+ {potentialWinnings.toFixed(2)}
+ </span>
+ </span>
+ </div>
+ <div className="flex justify-between text-[12px]">
+                   <span style={{ color: colors.textSecondary }}>Potential Win </span>
+ <span style={{ color: colors.primary }} className="font-semibold">
+ {potentialWinnings.toFixed(2)}
+ </span>
Removed / Before Commit
- diff --git a/src/components/bets/DesktopBetSidebar.tsx b/src/components/bets/DesktopBetSidebar.tsx
- index fd99a49..e9a38d5 100644
- <div className="flex items-center justify-between text-[12px] font-semibold">
- <span style={{ color: colors.textSecondary }}>Stake</span>
- <span className="text-[14px] font-bold" style={{ color: colors.textPrimary }}>
-                 &#8377;{totalStake.toFixed(2)}
- </span>
- </div>
- <div className="flex items-center justify-between text-[12px] font-semibold">
-               <span style={{ color: colors.textSecondary }}>Potential Win</span>
- <span className="text-[14px] font-bold" style={{ color: colors.primary }}>
-                 &#8377;{potentialWinnings.toFixed(2)}
- </span>
- </div>
- </div>
Added / After Commit
+ diff --git a/src/components/bets/DesktopBetSidebar.tsx b/src/components/bets/DesktopBetSidebar.tsx
+ index fd99a49..e9a38d5 100644
+ <div className="flex items-center justify-between text-[12px] font-semibold">
+ <span style={{ color: colors.textSecondary }}>Stake</span>
+ <span className="text-[14px] font-bold" style={{ color: colors.textPrimary }}>
+                 {totalStake.toFixed(2)}
+ </span>
+ </div>
+ <div className="flex items-center justify-between text-[12px] font-semibold">
+               <span style={{ color: colors.textSecondary }}>Potential Win </span>
+ <span className="text-[14px] font-bold" style={{ color: colors.primary }}>
+                {potentialWinnings.toFixed(2)}
+ </span>
+ </div>
+ </div>
Removed / Before Commit
- diff --git a/src/components/betslip/BetslipItem.tsx b/src/components/betslip/BetslipItem.tsx
- index b6cf4f2..6f988ac 100644
- </div>
- <div style={{ backgroundColor: colors.background }} className="flex-1 flex items-center justify-end border border-[#ffffff0d] rounded-[8px] px-[10px] py-[7px] focus-within:border-[#ffd70050] transition-colors duration-200">
- <div className="flex items-center gap-1 text-xs text-gray-400">
-           <i className="fa-solid fa-indian-rupee-sign"></i>
- <input
- type="number"
- value={bet.stake}
- </div>
- 
- <div className="flex justify-between mt-[6px] px-[2px]">
-         <span className="text-[11px] text-gray-600">Potential win</span>
- <span className="text-[11px] text-green-400 font-semibold">{potentialWin}</span>
- </div>
- </div>
- );
- }
Added / After Commit
+ diff --git a/src/components/betslip/BetslipItem.tsx b/src/components/betslip/BetslipItem.tsx
+ index b6cf4f2..6f988ac 100644
+ </div>
+ <div style={{ backgroundColor: colors.background }} className="flex-1 flex items-center justify-end border border-[#ffffff0d] rounded-[8px] px-[10px] py-[7px] focus-within:border-[#ffd70050] transition-colors duration-200">
+ <div className="flex items-center gap-1 text-xs text-gray-400">
+ <input
+ type="number"
+ value={bet.stake}
+ </div>
+ 
+ <div className="flex justify-between mt-[6px] px-[2px]">
+         <span className="text-[11px] text-gray-600">Potential win </span>
+ <span className="text-[11px] text-green-400 font-semibold">{potentialWin}</span>
+ </div>
+ </div>
+ );
+ \ No newline at end of file
+ }
Removed / Before Commit
- diff --git a/src/components/layout/Header.tsx b/src/components/layout/Header.tsx
- index bcc6eb1..78efdb0 100644
- className="border border-[#ffd70017] flex items-center gap-[6px] px-[8px] py-[4px] rounded-[5px] text-xs whitespace-nowrap shrink-0"
- >
- <i className="fa-solid fa-wallet shrink-0"></i>
-                   <span>{`Rs ${balanceDisplay}`}</span>
- </div>
- 
- <button
- className="border border-[#ffd70017] min-w-max flex items-center justify-end gap-[8px] px-[8px] py-[5px] rounded-[5px] text-[14px] shrink-0"
- >
- <span className="whitespace-nowrap text-right font-medium">
-                     {`Rs ${balanceDisplay}`}
- </span>
- <div
- style={{ backgroundColor: colors.primary }}
Added / After Commit
+ diff --git a/src/components/layout/Header.tsx b/src/components/layout/Header.tsx
+ index bcc6eb1..78efdb0 100644
+ className="border border-[#ffd70017] flex items-center gap-[6px] px-[8px] py-[4px] rounded-[5px] text-xs whitespace-nowrap shrink-0"
+ >
+ <i className="fa-solid fa-wallet shrink-0"></i>
+                   <span>{balanceDisplay}</span>
+ </div>
+ 
+ <button
+ className="border border-[#ffd70017] min-w-max flex items-center justify-end gap-[8px] px-[8px] py-[5px] rounded-[5px] text-[14px] shrink-0"
+ >
+ <span className="whitespace-nowrap text-right font-medium">
+                     {balanceDisplay}
+ </span>
+ <div
+ style={{ backgroundColor: colors.primary }}
Removed / Before Commit
- diff --git a/src/components/racing/HorseRow.tsx b/src/components/racing/HorseRow.tsx
- index afeac9f..9ebc805 100644
- import { useState, useCallback, memo } from "react"
- import type { RunnerOdds } from "../../types/raceMarkets"
- import { DEFAULT_RACING_BET_AMOUNT, useBetStore } from "../../store/useBetStore"
- import { BetChipAnimation } from "../bets/BetChipAnimation"
- className="font-semibold"
- style={{ color: getAmountColor(amount) }}
- >
-         {isNegative ? '- ' : '+ '}&#8377;{Math.abs(amount).toFixed(2)}
- </span>
- </span>
- );
- return (
- <span className="inline-flex flex-wrap items-center gap-x-1 gap-y-[2px] leading-[14px]">
- <span className="font-semibold" style={{ color: getAmountColor(apiExposureDisplayValue.exposure) }}>
-               {apiExposureDisplayValue.exposure >= 0 ? '+' : ''}&#8377;{apiExposureDisplayValue.exposure.toFixed(2)}
- </span>
Added / After Commit
+ diff --git a/src/components/racing/HorseRow.tsx b/src/components/racing/HorseRow.tsx
+ index afeac9f..9ebc805 100644
+ import { useState, useCallback, memo, useEffect } from "react"
+ import type { RunnerOdds } from "../../types/raceMarkets"
+ import { DEFAULT_RACING_BET_AMOUNT, useBetStore } from "../../store/useBetStore"
+ import { BetChipAnimation } from "../bets/BetChipAnimation"
+ className="font-semibold"
+ style={{ color: getAmountColor(amount) }}
+ >
+         {isNegative ? '- ' : '+ '}{Math.abs(amount).toFixed(2)}
+ </span>
+ </span>
+ );
+ return (
+ <span className="inline-flex flex-wrap items-center gap-x-1 gap-y-[2px] leading-[14px]">
+ <span className="font-semibold" style={{ color: getAmountColor(apiExposureDisplayValue.exposure) }}>
+               {apiExposureDisplayValue.exposure >= 0 ? '+' : ''}{apiExposureDisplayValue.exposure.toFixed(2)}
+ </span>
Removed / Before Commit
- diff --git a/src/components/racing/MarketCard.tsx b/src/components/racing/MarketCard.tsx
- index c41e7d2..cde5adf 100644
- const showBlockBetting = isWinnerBlockMarket(market)
- const isMobile = typeof window !== "undefined" && window.innerWidth < 768
- 
-   // Check if any runner has jersey image URL (API sends 'jersey' field)
-   const hasAnyJerseyImage = market.odds.some(runner => 
-     runner.jersey || runner.runner?.jersey
-   )
- 
- const mobileBlockBetGridColumns = isToteMarket
- ? "minmax(0, 1fr) 60px"
- className="grid px-[14px] py-[8px]"
- style={{
- gridTemplateColumns: showBlockBetting
-                     ? hasAnyJerseyImage ? `auto ${blockBetGridColumns}` : blockBetGridColumns
- : window.innerWidth >= 768
- ? isToteMarket
Added / After Commit
+ diff --git a/src/components/racing/MarketCard.tsx b/src/components/racing/MarketCard.tsx
+ index c41e7d2..cde5adf 100644
+ const showBlockBetting = isWinnerBlockMarket(market)
+ const isMobile = typeof window !== "undefined" && window.innerWidth < 768
+ 
+   const showJerseySlot = true
+ 
+ const mobileBlockBetGridColumns = isToteMarket
+ ? "minmax(0, 1fr) 60px"
+ className="grid px-[14px] py-[8px]"
+ style={{
+ gridTemplateColumns: showBlockBetting
+                     ? `auto ${blockBetGridColumns}`
+ : window.innerWidth >= 768
+ ? isToteMarket 
+                         ? "auto 1fr 130px"
+                         : "auto 1fr 130px 130px"
+ : isToteMarket
Removed / Before Commit
- diff --git a/src/components/racing/WinnerBlockBetSummary.tsx b/src/components/racing/WinnerBlockBetSummary.tsx
- index 9fd1741..b1ffc31 100644
- <span className="hidden md:inline">Total Stake</span>
- </p>
- <p className="text-white text-[14px] md:text-[16px] font-semibold leading-[18px] truncate">
-                 &#8377; {totalStake.toFixed(2)}
- </p>
- </div>
- </div>
- </p>
- 
- <p className="text-[#18c99b] text-[14px] md:text-[16px] font-semibold leading-[18px] truncate">
-                 &#8377; {potentialReturn.toFixed(2)}
- </p>
- </div>
- </div>
Added / After Commit
+ diff --git a/src/components/racing/WinnerBlockBetSummary.tsx b/src/components/racing/WinnerBlockBetSummary.tsx
+ index 9fd1741..b1ffc31 100644
+ <span className="hidden md:inline">Total Stake</span>
+ </p>
+ <p className="text-white text-[14px] md:text-[16px] font-semibold leading-[18px] truncate">
+                  {totalStake.toFixed(2)}
+ </p>
+ </div>
+ </div>
+ </p>
+ 
+ <p className="text-[#18c99b] text-[14px] md:text-[16px] font-semibold leading-[18px] truncate">
+                 {potentialReturn.toFixed(2)}
+ </p>
+ </div>
+ </div>
Removed / Before Commit
- diff --git a/src/components/transactions/TransactionRow.tsx b/src/components/transactions/TransactionRow.tsx
- index 2a986ae..0d6fdc5 100644
- },
- {
- label: "Stake",
-         value: `Rs.${Number(row.stake || 0).toFixed(2)}`,
- icon: CircleDollarSign,
- color: "#16a34a",
- },
- },
- {
- label: "Potential Payout",
-         value: `Rs.${Number(row.potential_payout || 0).toFixed(2)}`,
- icon: CircleDollarSign,
- color: "#16a34a",
- },
- },
- {
Added / After Commit
+ diff --git a/src/components/transactions/TransactionRow.tsx b/src/components/transactions/TransactionRow.tsx
+ index 2a986ae..0d6fdc5 100644
+ },
+ {
+ label: "Stake",
+         value: Number(row.stake || 0).toFixed(2),
+ icon: CircleDollarSign,
+ color: "#16a34a",
+ },
+ },
+ {
+ label: "Potential Payout",
+         value: Number(row.potential_payout || 0).toFixed(2),
+ icon: CircleDollarSign,
+ color: "#16a34a",
+ },
+ },
+ {