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 avriti
Project AI Score ?
63%
Quality Avg ?
68%
Security Avg ?
45%
Reviews ?
56

Review Result ?

jattin01/avriti · 550bbd50
The commit adds a new product detail field 'caringForCrystal' to multiple components, including state management, UI display in product details, and product accordion content. The update improves user experience by providing more product information. However, the use of 'dangerouslySetInnerHTML' introduces potential XSS security risks, the commit message is vague, and there is no input sanitation visible. The code quality is good but could improve with more descriptive commits and security considerations.
Quality ?
75%
Security ?
40%
Business Value ?
70%
Maintainability ?
73%
Issues & Suggested Fixes ?
1AI detects the issue
2Shows file, line, or evidence
3Suggests the fix to apply
Vague Message
Where commit message
Issue / Evidence vague message
Suggested Fix provide a more descriptive commit message explaining what 'product import working' refers to in terms of features or bug fixes
Security Issue
Where components/product-accordion.tsx:174
Issue / Evidence security risk
Suggested Fix sanitize or escape 'caringForCrystal' content before using dangerouslySetInnerHTML to prevent XSS vulnerabilities
Security Issue
Where components/product-accordion.tsx:174
Issue / Evidence security risk
Suggested Fix consider using a safer method than dangerouslySetInnerHTML or validate HTML content
Consistency
Where app/shop/[slug]/page.tsx:76
Issue / Evidence consistency
Suggested Fix handle cases where 'caring_for_crystal' might be undefined or not a string to avoid potential bugs
Code Clarity
Where components/product-accordion.tsx:349-356
Issue / Evidence code clarity
Suggested Fix add comments explaining the effect hook logic for setting active tab to improve maintainability
Code Change Preview · app/shop/[slug]/page.tsx ?
Removed / Before Commit
- const [productDetails, setProductDetails] = useState("");
- const [closingTagline, setClosingTagline] = useState("");
- const [cleansing, setCleansing] = useState("");
- const [charging, setCharging] = useState("");
- const [wearing, setWearing] = useState("");
- const [dailyUse, setDailyUse] = useState("");
- setProductDetails(currentProduct?.product_details ?? "");
- setClosingTagline(currentProduct?.closing_tagline ?? "");
- setCleansing(currentProduct?.cleansing ?? "");
- setCharging(currentProduct?.charging ?? "");
- setWearing(currentProduct?.wearing ?? "");
- setDailyUse(currentProduct?.daily_use ?? "");
- productDetails={productDetails}
- closingTagline={closingTagline}
- cleansing={cleansing}
- charging={charging}
- wearing={wearing}
- dailyUse={dailyUse}
Added / After Commit
+ const [productDetails, setProductDetails] = useState("");
+ const [closingTagline, setClosingTagline] = useState("");
+ const [cleansing, setCleansing] = useState("");
+   const [caringForCrystal, setCaringForCrystal] = useState("");
+ const [charging, setCharging] = useState("");
+ const [wearing, setWearing] = useState("");
+ const [dailyUse, setDailyUse] = useState("");
+ setProductDetails(currentProduct?.product_details ?? "");
+ setClosingTagline(currentProduct?.closing_tagline ?? "");
+ setCleansing(currentProduct?.cleansing ?? "");
+           setCaringForCrystal(currentProduct?.caring_for_crystal ?? "");
+ setCharging(currentProduct?.charging ?? "");
+ setWearing(currentProduct?.wearing ?? "");
+ setDailyUse(currentProduct?.daily_use ?? "");
+ productDetails={productDetails}
+ closingTagline={closingTagline}
+ cleansing={cleansing}
+         caringForCrystal={caringForCrystal}
Removed / Before Commit
- "use client";
- 
- import { useState } from "react";
- 
- type AccordionItem = {
- title: string;
- productDetails?: string;
- closingTagline?: string;
- cleansing?: string;
- charging?: string;
- wearing?: string;
- dailyUse?: string;
- productDetails,
- closingTagline,
- cleansing,
- charging,
- wearing,
- dailyUse,
Added / After Commit
+ "use client";
+ 
+ import { useState, useEffect } from "react";
+ 
+ type AccordionItem = {
+ title: string;
+ productDetails?: string;
+ closingTagline?: string;
+ cleansing?: string;
+   caringForCrystal?: string;
+ charging?: string;
+ wearing?: string;
+ dailyUse?: string;
+ productDetails,
+ closingTagline,
+ cleansing,
+   caringForCrystal,
+ charging,
Removed / Before Commit
- )}
- {product.closing_tagline && (
- <div className="mt-1 px-1 py-1 text-[13px] italic text-[#9e8a7a]">
- {product.closing_tagline}
- </div>
- )}
Added / After Commit
+ )}
+ {product.closing_tagline && (
+ <div className="mt-1 px-1 py-1 text-[13px] italic text-[#9e8a7a]">
+                 <span className="font-semibold not-italic text-[#2c2420]">Tagline: </span>
+ {product.closing_tagline}
+ </div>
+ )}