update:coderabbit_suggestions

Author: Utkarsh-0304

frontend/__tests__/unit/components/ScrollToTop.test.tsx

@@ -3,7 +3,7 @@ import ScrollToTop from 'components/ScrollToTop'
 
 describe('ScrollToTop component test', () => {
   beforeEach(() => {
-    window.scrollTo = jest.fn()
+    globalThis.scrollTo = jest.fn()
     Object.defineProperty(globalThis, 'scrollY', { value: 0, writable: true })
     Object.defineProperty(globalThis, 'innerHeight', { value: 1000, writable: true })
   })
@@ -45,6 +45,6 @@ describe('ScrollToTop component test', () => {
     })
 
     fireEvent.click(button)
-    expect(window.scrollTo).toHaveBeenCalledWith({ top: 0, behavior: 'smooth' })
+    expect(globalThis.scrollTo).toHaveBeenCalledWith({ top: 0, behavior: 'smooth' })
   })
 })

frontend/src/app/my/mentorship/page.tsx

@@ -118,7 +118,7 @@ const MyMentorshipPage: React.FC = () => {
         currentPage={page}
         onPageChange={(p) => {
           setPage(p)
-          window.scrollTo({ top: 0, behavior: 'smooth' })
+          globalThis.scrollTo({ top: 0, behavior: 'smooth' })
         }}
         onSearch={(q) => {
           setSearchQuery(q)

frontend/src/components/Header.tsx

@@ -26,7 +26,7 @@ export default function Header({ isGitHubAuthEnabled }: { readonly isGitHubAuthE
 
   useEffect(() => {
     const handleResize = () => {
-      if (window.innerWidth >= desktopViewMinWidth) {
+      if (globalThis.innerWidth >= desktopViewMinWidth) {
         setMobileMenuOpen(false)
       }
     }
@@ -45,11 +45,11 @@ export default function Header({ isGitHubAuthEnabled }: { readonly isGitHubAuthE
       }
     }
 
-    window.addEventListener('resize', handleResize)
+    globalThis.addEventListener('resize', handleResize)
     globalThis.addEventListener('click', handleOutsideClick)
 
     return () => {
-      window.removeEventListener('resize', handleResize)
+      globalThis.removeEventListener('resize', handleResize)
       globalThis.removeEventListener('click', handleOutsideClick)
     }
   }, [mobileMenuOpen])

frontend/src/components/MultiSearch.tsx

@@ -100,7 +100,7 @@ const MultiSearchBar: React.FC<MultiSearchBarProps> = ({
           router.push(`/chapters/${suggestion.key}`)
           break
         case 'events':
-          window.open((suggestion as Event).url, '_blank')
+          globalThis.open((suggestion as Event).url, '_blank')
           break
         case 'organizations':
           // Use type guard to safely access login property

frontend/src/components/SingleModuleCard.tsx

@@ -47,7 +47,7 @@ const SingleModuleCard: React.FC<SingleModuleCardProps> = ({
 
   const handleCreate = () => {
     setDropdownOpen(false)
-    router.push(`${window.location.pathname}/modules/create`)
+    router.push(`${globalThis.location.pathname}/modules/create`)
   }
 
   const moduleDetails = [

frontend/src/utils/helpers/githubHeatmap.ts

@@ -171,7 +171,7 @@ function getPixelRatio() {
   if (typeof globalThis === 'undefined') {
     return 1
   }
-  return window.devicePixelRatio || 1
+  return globalThis.devicePixelRatio || 1
 }
 
 const DATE_FORMAT = 'yyyy-MM-dd'

frontend/src/utils/scrollToAnchor.ts

@@ -10,8 +10,8 @@ export function scrollToAnchor(targetId: string, additionalOffset = 80): void {
     const yOffset = -headingHeight - additionalOffset
 
     // Use modern window.scrollY instead of deprecated pageYOffset
-    const y = element.getBoundingClientRect().top + window.scrollY + yOffset
-    window.scrollTo({
+    const y = element.getBoundingClientRect().top + globalThis.scrollY + yOffset
+    globalThis.scrollTo({
       top: y,
       behavior: 'smooth',
     })