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 · bcbf98f1
This commit adds social media logo links to the contact page and implements a ScrollToTop component that scrolls to the top on route changes. It also updates the site header to scroll smoothly to top on homepage clicks, and adds some Excel files to .gitignore. The changes increase user navigation convenience and social media presence but have minor issues with security and UX consistency.
Quality ?
75%
Security ?
60%
Business Value ?
70%
Maintainability ?
78%
Issues & Suggested Fixes ?
1AI detects the issue
2Shows file, line, or evidence
3Suggests the fix to apply
Href Is Set To '#' For Youtube And Threads
Where app/contact/page.tsx:396
Issue / Evidence href is set to '#' for YouTube and Threads
Suggested Fix use actual URLs or remove dead links to improve business value and user experience
Use Of 'Instant' Cast As Scrollbehavior Is...
Where components/scroll-to-top.tsx:10
Issue / Evidence use of 'instant' cast as ScrollBehavior is uncommon and may lack browser support
Suggested Fix consider changing behavior to 'auto' or 'smooth' to improve cross-browser UX and quality
Direct Use Of Window.scrollto Inside Oncli...
Where components/site-header.tsx:134
Issue / Evidence direct use of window.scrollTo inside onClick
Suggested Fix consider extracting scroll behavior to a reusable handler and add error handling to improve code quality
Similar Direct Window.scrollto Usage
Where components/site-header.tsx:168
Issue / Evidence similar direct window.scrollTo usage
Suggested Fix unify scroll behavior with ScrollToTop component for consistency
Adding Multiple Excel Files To Ignore With...
Where .gitignore:42-47
Issue / Evidence adding multiple Excel files to ignore without explanation
Suggested Fix clarify in commit message or documentation to reduce perceived fake work risk
Code Change Preview · .gitignore ?
Removed / Before Commit
- # typescript
- *.tsbuildinfo
- next-env.d.ts
- \ No newline at end of file
Added / After Commit
+ # typescript
+ *.tsbuildinfo
+ next-env.d.ts
+ /Avriti-Master-Products.xlsx
+ /braclet products.xlsx
+ /salt product (bracelet format).xlsx
+ /salt product.xlsx
+ 
+ .salt product.xlsx/
+ \ No newline at end of file
Removed / Before Commit
- <p className="mt-[10px] text-sm leading-7 text-[#6f6f6f] sm:text-base text-center">Stay connected for Vedic tips, new launches, and honest spiritual guidance.</p>
- <ul className="flex justify-center items-center gap-4 mt-[20px]">
- {[
-               { src: "/instagram.png", alt: "Instagram" },
-               { src: "/facebook.png", alt: "Facebook" },
-               { src: "/pinterest.png", alt: "Pinterest" },
-               { src: "/linkedin.png", alt: "LinkedIn" },
-               { src: "/youtube.png", alt: "YouTube" },
-               { src: "/threads.png", alt: "Threads" },
- ].map((social) => (
- <li key={social.alt}>
-                 <a href=""><img src={social.src} alt={social.alt} className="w-[25px] h-[25px]" /></a>
- </li>
- ))}
- </ul>
Added / After Commit
+ <p className="mt-[10px] text-sm leading-7 text-[#6f6f6f] sm:text-base text-center">Stay connected for Vedic tips, new launches, and honest spiritual guidance.</p>
+ <ul className="flex justify-center items-center gap-4 mt-[20px]">
+ {[
+               { src: "/instagram.png", alt: "Instagram", href: "https://www.instagram.com/_avriti__/" },
+               { src: "/facebook.png", alt: "Facebook", href: "https://www.facebook.com/profile.php?id=61589667186265" },
+               { src: "/pinterest.png", alt: "Pinterest", href: "https://in.pinterest.com/avriti_i/" },
+               { src: "/linkedin.png", alt: "LinkedIn", href: "https://www.linkedin.com/company/avriti/?viewAsMember=true" },
+               { src: "/youtube.png", alt: "YouTube", href: "#" },
+               { src: "/threads.png", alt: "Threads", href: "#" },
+ ].map((social) => (
+ <li key={social.alt}>
+                 <a href={social.href} target="_blank" rel="noopener noreferrer"><img src={social.src} alt={social.alt} className="w-[25px] h-[25px]" /></a>
+ </li>
+ ))}
+ </ul>
Removed / Before Commit
- import { SiteFooter } from "@/components/site-footer";
- import { SiteHeader } from "@/components/site-header";
- import { WhatsAppCta } from "@/components/whatsapp-cta";
- import "./globals.css";
- 
- export const metadata: Metadata = {
- <html lang="en" className="h-full antialiased">
- 
- <body className="flex min-h-full flex-col bg-white text-[var(--color-text-primary)] font-sans">
- <SiteHeader />
- 
- <main className="flex-1">{children}</main>
Added / After Commit
+ import { SiteFooter } from "@/components/site-footer";
+ import { SiteHeader } from "@/components/site-header";
+ import { WhatsAppCta } from "@/components/whatsapp-cta";
+ import { ScrollToTop } from "@/components/scroll-to-top";
+ import "./globals.css";
+ 
+ export const metadata: Metadata = {
+ <html lang="en" className="h-full antialiased">
+ 
+ <body className="flex min-h-full flex-col bg-white text-[var(--color-text-primary)] font-sans">
+         <ScrollToTop />
+ <SiteHeader />
+ 
+ <main className="flex-1">{children}</main>
Removed / Before Commit

                                                

                                                

                                                    
Added / After Commit

                                                    
+ "use client";
+ 
+ import { useEffect } from "react";
+ import { usePathname } from "next/navigation";
+ 
+ export function ScrollToTop() {
+   const pathname = usePathname();
+ 
+   useEffect(() => {
+     window.scrollTo({ top: 0, left: 0, behavior: "instant" as ScrollBehavior });
+   }, [pathname]);
+ 
+   return null;
+ }


                                                

                                            

                                        

                                                                            

                                            

                                                

                                                    
Removed / Before Commit

                                                    
- return (
- <header className="bg-[#fff] sticky top-0 z-40">
- <div className="relative bg-[#fff] mx-auto flex max-w-7xl items-center justify-between px-4 sm:px-6 lg:px-8 py-4 gap-[10px] md:gap-[20px] lg:gap-[50px]">
-         <Link href="/" className="w-[200px] flex items-center gap-3" aria-label="Avriti home">
- <span className="relative h-12 w-28 overflow-hidden">
- <Image
- src="/logo1.png"
- {/* Center Logo */}
- {/* Center Logo */}
- <div className="flex justify-center flex-1">
-             <Link href="/">
- <span className="relative h-12 w-20 block overflow-hidden">
- <Image
- src="/HeadIcon.png"


                                                

                                                

                                                    
Added / After Commit

                                                    
+ return (
+ <header className="bg-[#fff] sticky top-0 z-40">
+ <div className="relative bg-[#fff] mx-auto flex max-w-7xl items-center justify-between px-4 sm:px-6 lg:px-8 py-4 gap-[10px] md:gap-[20px] lg:gap-[50px]">
+         <Link href="/" onClick={() => window.scrollTo({ top: 0, behavior: "smooth" })} className="w-[200px] flex items-center gap-3" aria-label="Avriti home">
+ <span className="relative h-12 w-28 overflow-hidden">
+ <Image
+ src="/logo1.png"
+ {/* Center Logo */}
+ {/* Center Logo */}
+ <div className="flex justify-center flex-1">
+             <Link href="/" onClick={() => window.scrollTo({ top: 0, behavior: "smooth" })}>
+ <span className="relative h-12 w-20 block overflow-hidden">
+ <Image
+ src="/HeadIcon.png"