chore: fix eslint errors (#594)

* fix: improve accessibility by adding htmlFor attributes to labels; fix: eslint errors and improve code

* fix: add alt attribute to img component for improved accessibility

* fix: enhance accessibility by adding dependencies to hooks and improving effect dependencies

* blog post drafts, future date filtering, and ai blog posts

* ci: apply automated fixes

* ci: apply automated fixes

* fix: standardize icon sizes by removing size attributes for consistency; fix: hover sidebar bg

* ci: apply automated fixes

* fix: enhance accessibility by adding aria-label to menu button and improving background color for better contrast

* checkpoint

# Conflicts:
#	src/auth/oauth.server.ts

* fix: enhance accessibility by adding dependencies to hooks and improving effect dependencies

* ci: apply automated fixes

---------

Co-authored-by: Tanner Linsley <[email protected]>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: 3 people

scripts/test-npm-cache.ts

@@ -3,11 +3,10 @@
  * Tests that chunks are properly cached and retrieved
  */
 
-import { db } from '../src/db/client'
 import {
   getCachedNpmDownloadChunk,
   setCachedNpmDownloadChunk,
-} from '../src/utils/stats-db.server'
+} from '~/utils/stats-db.server'
 
 async function testCache() {
   console.log('\n' + '='.repeat(80))

src/auth/context.server.ts

@@ -11,7 +11,6 @@ import { OAuthService } from './oauth.server'
 import { SessionService } from './session.server'
 import { createAuthGuards } from './guards.server'
 import {
-  createRepositories,
   DrizzleUserRepository,
   DrizzleOAuthAccountRepository,
   DrizzleCapabilitiesRepository,

src/components/AILibraryHero.tsx

@@ -94,7 +94,7 @@ const MESSAGES = [
   },
 ]
 
-export function AILibraryHero({ project, cta, actions }: AILibraryHeroProps) {
+export function AILibraryHero({}: AILibraryHeroProps) {
   const isDark = useIsDark()
   const strokeColor = isDark ? 'rgba(255, 255, 255, 0.8)' : 'rgba(0, 0, 0, 0.6)'
   const textColor = isDark ? '#ffffff' : '#000000'

src/components/BlockWithFeedback.tsx

@@ -41,7 +41,6 @@ export function BlockWithFeedback({
     return feedback.userNotes.find((n) => n.blockSelector === blockSelector)
   }, [feedback.userNotes, blockSelector])
 
-  const isCollapsed = note ? feedback.collapsedNotes.has(note.id) : false
   const hasNote = !!note
 
   const handleAddNote = React.useCallback(() => {

src/components/CookieConsent.tsx

@@ -1,5 +1,5 @@
 import { Link } from '@tanstack/react-router'
-import { useCallback, useEffect, useState } from 'react'
+import { useEffect, useState } from 'react'
 
 declare global {
   interface Window {
@@ -134,6 +134,7 @@ export default function CookieConsent() {
     }
 
     checkLocationAndSetConsent()
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])
 
   const acceptAllCookies = () => {

src/components/Doc.tsx

@@ -182,7 +182,7 @@ function DocContent({
               href={`https://github.com/${repo}/edit/${branch}/${filePath}`}
               className="flex items-center gap-2"
             >
-              <SquarePen size={16} /> Edit on GitHub
+              <SquarePen /> Edit on GitHub
             </a>
           </div>
           <div className="h-24" />

src/components/DocFeedbackFloatingButton.tsx

@@ -59,6 +59,7 @@ export function DocFeedbackFloatingButton({
       document.removeEventListener('mousedown', handleClickOutside)
       document.removeEventListener('keydown', handleEscape)
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [isMenuOpen])
 
   const handleButtonClick = (e: React.MouseEvent) => {
@@ -86,6 +87,7 @@ export function DocFeedbackFloatingButton({
   }
 
   return (
+    // eslint-disable-next-line jsx-a11y/click-events-have-key-events,jsx-a11y/no-static-element-interactions
     <div
       ref={buttonRef}
       className="doc-feedback-floating-btn absolute top-0 right-0 -translate-y-full z-[100]"

src/components/DocFeedbackNote.tsx

@@ -298,6 +298,7 @@ export function DocFeedbackNote({
   }
 
   return (
+    // eslint-disable-next-line jsx-a11y/click-events-have-key-events,jsx-a11y/no-static-element-interactions
     <div
       className={twMerge(
         inline ? 'w-full' : 'fixed z-40 w-80 max-w-[calc(100vw-2rem)]',
@@ -334,6 +335,7 @@ export function DocFeedbackNote({
         )}
 
         {/* Header - always visible */}
+        {/* eslint-disable-next-line jsx-a11y/click-events-have-key-events,jsx-a11y/no-static-element-interactions */}
         <div
           className={twMerge(
             `flex flex-col gap-1 p-2 ${colors.header}`,

src/components/DocsLayout.tsx

@@ -288,8 +288,8 @@ export function DocsLayout({
       >
         <summary className="py-2 px-4 flex gap-2 items-center justify-between">
           <div className="flex-1 flex gap-4 items-center">
-            <TextAlignStart className="icon-open cursor-pointer" size={16} />
-            <X className="icon-close cursor-pointer" size={16} />
+            <TextAlignStart className="icon-open cursor-pointer" />
+            <X className="icon-close cursor-pointer" />
             Documentation
           </div>
         </summary>
@@ -359,7 +359,7 @@ export function DocsLayout({
                   className="py-1 px-2 bg-white/70 text-black dark:bg-gray-500/40 dark:text-white shadow-lg shadow-black/20 flex items-center justify-center backdrop-blur-sm z-20 rounded-lg overflow-hidden"
                 >
                   <div className="flex gap-2 items-center font-bold">
-                    <ArrowLeft size={16} />
+                    <ArrowLeft />
                     {prevItem.label}
                   </div>
                 </Link>
@@ -379,7 +379,7 @@ export function DocsLayout({
                     >
                       {nextItem.label}
                     </span>{' '}
-                    <ArrowRight className={textColor} size={16} />
+                    <ArrowRight className={textColor} />
                   </div>
                 </Link>
               ) : null}

src/components/FeedFilters.tsx

@@ -136,7 +136,7 @@ export function FeedFilters({
   // Update filters when debounced search changes
   React.useEffect(() => {
     onFiltersChange({ search: debouncedSearch || undefined })
-  }, [debouncedSearch])
+  }, [debouncedSearch, onFiltersChange])
 
   // Sync local state with prop when search prop changes externally
   React.useEffect(() => {
@@ -176,7 +176,7 @@ export function FeedFilters({
     featured !== undefined ||
     search ||
     (selectedReleaseLevels && selectedReleaseLevels.length > 0) ||
-    (includePrerelease !== undefined && includePrerelease !== true),
+    (includePrerelease !== undefined && !includePrerelease),
   )
 
   // Render filter content (shared between mobile and desktop)
@@ -383,6 +383,7 @@ export function FeedFilters({
           compact
         />
       )}
+      {/* eslint-disable-next-line jsx-a11y/click-events-have-key-events,jsx-a11y/no-static-element-interactions */}
       <div onClick={(e) => e.stopPropagation()} className="flex-shrink-0">
         <FilterSearch
           value={searchInput}