fix(web): lint — set-state-in-effect disable for URL sync, remove unused get param in test mock

Made-with: Cursor

Author: haasonsaas

web/src/pages/ReviewView.tsx

@@ -421,13 +421,16 @@ export function ReviewView() {
         ? normalizeCommentFilePath(comment.file_path)
         : null
 
+    // Sync URL search params to local state (single batch per param change)
+    /* eslint-disable react-hooks/set-state-in-effect -- URL is source of truth for view state */
     setViewMode(nextViewMode ?? (comment ? 'list' : 'diff'))
     setSelectedFile(nextSelectedFile)
     setCategoryFilter(nextCategoryFilter)
     setRuleFilter(nextRuleFilter)
     setLifecycleFilter(nextLifecycleFilter)
     setShowOnlyBlockers(searchParams.get('blockers') === '1')
     setActiveCommentId(comment?.id ?? null)
+    /* eslint-enable react-hooks/set-state-in-effect */
   }, [review, searchParams])
 
   useEffect(() => {

web/src/pages/__tests__/ReviewView.test.tsx

@@ -12,7 +12,7 @@ const lifecycleMutate = vi.fn()
 vi.mock('react-router-dom', () => ({
   useParams: () => ({ id: 'review-1' }),
   useSearchParams: () => [
-    { get: (_key: string) => null },
+    { get: () => null },
     () => {},
   ],
 }))