refactor: clean up imports and remove unused components

Varun Raj · Varun Raj · commit ea51071538fe · 2025-03-25T22:31:40.000+05:30
- Removed unused imports and components across various files to streamline the codebase.
- Simplified the structure of components such as AlbumImages, AlbumPeople, and others by eliminating unnecessary dependencies.
- Enhanced overall readability and maintainability of the code.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -6,5 +6,9 @@
       "files": ["remotion/*.{ts,tsx}"],
       "extends": ["plugin:@remotion/recommended"]
     }
-  ]
+  ],
+  "rules": {
+    "react-hooks/exhaustive-deps": "off",
+    "@next/next/no-img-element": "off"
+  }
 }
diff --git a/src/components/albums/info/AlbumImages.tsx b/src/components/albums/info/AlbumImages.tsx
@@ -6,11 +6,9 @@ import { IAsset } from '@/types/asset'
 import React, { useEffect, useMemo, useRef, useState } from 'react'
 import { ExternalLink, Hourglass } from 'lucide-react';
 import { useRouter } from 'next/router';
-import { CalendarArrowUp } from 'lucide-react';
 import Lightbox from 'yet-another-react-lightbox';
 import Captions from 'yet-another-react-lightbox/plugins/captions';
 import LazyImage from '@/components/ui/lazy-image';
-import Image from 'next/image';
 import { Button } from "@/components/ui/button";
 import Link from "next/link";
 
diff --git a/src/components/albums/info/AlbumPeople.tsx b/src/components/albums/info/AlbumPeople.tsx
@@ -1,16 +1,12 @@
 import React, { useEffect, useMemo, useState } from 'react'
 import { IAlbum } from '@/types/album'
-import { IPerson } from '@/types/person'
 import { getAlbumPeople } from '@/handlers/api/album.handler'
 import Loader from '@/components/ui/loader'
 import LazyImage from '@/components/ui/lazy-image'
 import { PERSON_THUBNAIL_PATH } from '@/config/routes'
 import Link from 'next/link'
 import { useConfig } from '@/contexts/ConfigContext'
-import { Tooltip } from '@/components/ui/tooltip'
 import { cn } from '@/lib/utils'
-import { Accordion, AccordionContent, AccordionItem, AccordionTrigger } from '@/components/ui/accordion'
-import { Users, X } from 'lucide-react'
 import { useRouter } from 'next/router'
 import Image from 'next/image'
 import { PersonMergeDropdown } from '@/components/people/PersonMergeDropdown'
diff --git a/src/components/albums/potential-albums/PotentialAlbumsAssets.tsx b/src/components/albums/potential-albums/PotentialAlbumsAssets.tsx
@@ -4,9 +4,9 @@ import { listPotentialAlbumsAssets } from "@/handlers/api/album.handler";
 import type { IAsset } from "@/types/asset";
 import React, { type MouseEvent, useEffect, useMemo, useState } from "react";
 import { Gallery } from "react-grid-gallery";
-import Lightbox, { SlideImage, SlideTypes } from "yet-another-react-lightbox";
+import Lightbox from "yet-another-react-lightbox";
 import Captions from "yet-another-react-lightbox/plugins/captions";
-import { CalendarArrowDown, CalendarArrowUp, Hourglass } from "lucide-react";
+import { CalendarArrowUp, Hourglass } from "lucide-react";
 import Video from "yet-another-react-lightbox/plugins/video";
 import { useConfig } from "@/contexts/ConfigContext";
 import LazyGridImage from "@/components/ui/lazy-grid-image";
diff --git a/src/components/albums/potential-albums/PotentialAlbumsDates.tsx b/src/components/albums/potential-albums/PotentialAlbumsDates.tsx
@@ -2,13 +2,13 @@ import {
   IPotentialAlbumsDatesResponse,
   listPotentialAlbumsDates,
 } from "@/handlers/api/album.handler";
-import React, { use, useEffect, useState } from "react";
+import React, { useEffect, useState } from "react";
 import PotentialDateItem from "./PotentialDateItem";
 import { usePotentialAlbumContext } from "@/contexts/PotentialAlbumContext";
 import { useRouter } from "next/router";
 import { Select, SelectContent, SelectGroup, SelectItem, SelectLabel, SelectTrigger, SelectValue } from "@/components/ui/select";
 import { Button } from "@/components/ui/button";
-import { ArrowDown, ArrowUp, ArrowUpDownIcon, SortAsc, SortDesc } from "lucide-react";
+import { SortAsc, SortDesc } from "lucide-react";
 
 export default function PotentialAlbumsDates() {
   const router = useRouter();
diff --git a/src/components/assets/missing-location/MissingLocationDates.tsx b/src/components/assets/missing-location/MissingLocationDates.tsx
@@ -1,6 +1,5 @@
-import React, { use, useEffect, useState } from "react";
-import { usePotentialAlbumContext } from "@/contexts/PotentialAlbumContext";
-import { IMissingLocationDatesResponse, listMissingLocationAlbums, listMissingLocationDates } from "@/handlers/api/asset.handler";
+import React, { useEffect, useState } from "react";
+import { listMissingLocationAlbums, listMissingLocationDates } from "@/handlers/api/asset.handler";
 import MissingLocationDateItem from "./MissingLocationDateItem";
 import { useMissingLocationContext } from "@/contexts/MissingLocationContext";
 import { useRouter } from "next/router";
diff --git a/src/components/assets/missing-location/TagMissingLocationDialog/Map.tsx b/src/components/assets/missing-location/TagMissingLocationDialog/Map.tsx
@@ -7,8 +7,8 @@ import L, { LatLngExpression } from 'leaflet';
 import 'leaflet-defaulticon-compatibility/dist/leaflet-defaulticon-compatibility.webpack.css'; // Re-uses images from ~leaflet package
 import 'leaflet-defaulticon-compatibility';
 
-import { MapContainer, Marker, Popup, TileLayer, useMap, useMapEvents } from "react-leaflet";
-import { useEffect, useRef, useState } from "react";
+import { MapContainer, TileLayer, useMapEvents } from "react-leaflet";
+import { useState } from "react";
 import { IPlace } from "@/types/common";
 import CustomMarker from "./CustomMarker";
 
diff --git a/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationOSMSearchAndAdd.tsx b/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationOSMSearchAndAdd.tsx
@@ -2,7 +2,6 @@ import ErrorBlock from "@/components/shared/ErrorBlock";
 import { Button } from "@/components/ui/button";
 
 import { Input } from "@/components/ui/input";
-import { Label } from "@/components/ui/label";
 import Loader from "@/components/ui/loader";
 import { useToast } from "@/components/ui/use-toast";
 import { cn } from "@/lib/utils";
diff --git a/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationSearchAndAdd.tsx b/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationSearchAndAdd.tsx
@@ -2,13 +2,11 @@ import ErrorBlock from "@/components/shared/ErrorBlock";
 import { Button } from "@/components/ui/button";
 
 import { Input } from "@/components/ui/input";
-import { Label } from "@/components/ui/label";
 import Loader from "@/components/ui/loader";
 import { useToast } from "@/components/ui/use-toast";
 import { searchPlaces } from "@/handlers/api/common.handler";
 import { cn } from "@/lib/utils";
 import { IPlace } from "@/types/common";
-import clsx from "clsx";
 import { Check } from "lucide-react";
 import React, { useRef, useState } from "react";
 
diff --git a/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationSearchLatLong.tsx b/src/components/assets/missing-location/TagMissingLocationDialog/TagMissingLocationSearchLatLong.tsx
@@ -1,16 +1,11 @@
 
-import ErrorBlock from "@/components/shared/ErrorBlock";
 import { Button } from "@/components/ui/button";
 
 import { Input } from "@/components/ui/input";
 import { Label } from "@/components/ui/label";
-import Loader from "@/components/ui/loader";
 import { useToast } from "@/components/ui/use-toast";
-import { searchPlaces } from "@/handlers/api/common.handler";
-import { cn } from "@/lib/utils";
 import { IPlace } from "@/types/common";
-import { Check } from "lucide-react";
-import React, { useMemo, useRef, useState } from "react";
+import React, { useMemo, useState } from "react";
 
 interface TagMissingLocationSearchLatLongProps {
   onSubmit: (place: IPlace) => Promise<any>;
diff --git a/src/components/layouts/PageLayout.tsx b/src/components/layouts/PageLayout.tsx
@@ -1,8 +1,6 @@
 import React from "react";
 import Header from "../shared/Header";
 import { cn } from "@/lib/utils";
-import Head from "next/head";
-
 
 interface IProps {
   children: React.ReactNode[] | React.ReactNode;
diff --git a/src/components/layouts/RootLayout.tsx b/src/components/layouts/RootLayout.tsx
@@ -11,7 +11,6 @@ import { LoginForm } from "../auth/LoginForm";
 import { useConfig } from "@/contexts/ConfigContext";
 import { Toaster } from "react-hot-toast";
 import { useRouter } from "next/router";
-import PageLayout from "./PageLayout";
 
 type RootLayoutProps = {
   children: ReactNode;
diff --git a/src/components/people/PeopleFilters.tsx b/src/components/people/PeopleFilters.tsx
@@ -16,7 +16,6 @@ import { Input } from "../ui/input";
 import { IPersonListFilters } from "@/handlers/api/people.handler";
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from "../ui/select";
 import { removeNullOrUndefinedProperties } from "@/helpers/data.helper";
-import { ParsedUrlQueryInput } from "querystring";
 
 export function PeopleFilters() {
   const router = useRouter();
diff --git a/src/components/people/PersonHideCell.tsx b/src/components/people/PersonHideCell.tsx
@@ -1,11 +1,6 @@
 import { IPerson } from "@/types/person";
 import React, { useState } from "react";
-import { DatePicker } from "../ui/datepicker";
 import { updatePerson } from "@/handlers/api/people.handler";
-import { formatDate } from "@/helpers/date.helper";
-import { Button } from "../ui/button";
-import { Switch } from "../ui/switch";
-import { Label } from "../ui/label";
 
 interface IProps {
   person: IPerson;
diff --git a/src/components/people/PersonItem.tsx b/src/components/people/PersonItem.tsx
@@ -6,7 +6,7 @@ import { PersonMergeDropdown } from "./PersonMergeDropdown";
 import PersonBirthdayCell from "./PersonBirthdayCell";
 import clsx from "clsx";
 import Link from "next/link";
-import { ArrowUpRight, Info, Search } from "lucide-react";
+import { ArrowUpRight, Info } from "lucide-react";
 import { useConfig } from "@/contexts/ConfigContext";
 import { useToast } from "../ui/use-toast";
 import { Badge } from "../ui/badge";
diff --git a/src/components/people/info/PersonAlbumList.tsx b/src/components/people/info/PersonAlbumList.tsx
@@ -1,6 +1,5 @@
 import { ASSET_THUMBNAIL_PATH } from '@/config/routes';
 import { useConfig } from '@/contexts/ConfigContext';
-import { ExternalLink } from 'lucide-react';
 import Image from 'next/image';
 import Link from 'next/link';
 import React, { useMemo } from 'react'
diff --git a/src/components/shared/AlbumDropdown.tsx b/src/components/shared/AlbumDropdown.tsx
@@ -2,7 +2,6 @@ import { listAlbums } from '@/handlers/api/album.handler';
 import { IAlbum } from '@/types/album';
 import React, { useEffect, useState } from 'react'
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from '../ui/select';
-import { X } from 'lucide-react';
 
 interface IAlbumDropdownProps {
   albumIds?: string[];
diff --git a/src/components/shared/ErrorBlock.tsx b/src/components/shared/ErrorBlock.tsx
@@ -1,5 +1,3 @@
-import { useConfig } from '@/contexts/ConfigContext';
-import { Link } from 'lucide-react';
 import React from 'react'
 
 interface ErrorBlockProps {
diff --git a/src/components/shared/FloatingBar.tsx b/src/components/shared/FloatingBar.tsx
@@ -1,4 +1,3 @@
-import { Button } from '@/components/ui/button'
 import React from 'react'
 
 interface FloatingBarProps {  
diff --git a/src/components/shared/Header.tsx b/src/components/shared/Header.tsx
@@ -1,23 +1,7 @@
 import React, { useMemo } from "react";
 import Link from "next/link";
-import {
-  Home,
-  LineChart,
-  Menu,
-  Package,
-  Package2,
-  ShoppingCart,
-  Users,
-} from "lucide-react";
-import { Badge } from "@/components/ui/badge";
+import { Menu } from "lucide-react";
 import { Button } from "@/components/ui/button";
-import {
-  Card,
-  CardContent,
-  CardDescription,
-  CardHeader,
-  CardTitle,
-} from "@/components/ui/card";
 import { Sheet, SheetContent, SheetTrigger } from "@/components/ui/sheet";
 import Head from "next/head";
 import { sidebarNavs } from "@/config/constants/sidebarNavs";
diff --git a/src/components/shared/PeopleList.tsx b/src/components/shared/PeopleList.tsx
@@ -2,7 +2,6 @@ import React, { useCallback, useMemo, useState } from 'react'
 import { IPerson } from '@/types/person';
 import { cn } from '@/lib/utils';
 import LazyImage from '../ui/lazy-image';
-import { PERSON_THUBNAIL_PATH } from '@/config/routes';
 import { Input } from '../ui/input';
 
 interface PeopleListProps {
diff --git a/src/components/shared/ShareAssetsTrigger.tsx b/src/components/shared/ShareAssetsTrigger.tsx
@@ -9,7 +9,6 @@ import { Label } from '@radix-ui/react-label';
 import { Switch } from '../ui/switch';
 import { Select, SelectValue, SelectContent, SelectItem, SelectTrigger } from '../ui/select';
 
-
 interface ShareAssetsTriggerProps {
   filters: ShareLinkFilters
   buttonProps?: ButtonProps
diff --git a/src/components/shared/ThemeSwitcher.tsx b/src/components/shared/ThemeSwitcher.tsx
@@ -1,5 +1,4 @@
 import React from "react";
-import { Button } from "../ui/button";
 import { useTheme } from "next-themes";
 import { Moon, Sun } from "lucide-react";
 import { cn } from "@/lib/utils";
@@ -15,7 +14,7 @@ export default function ThemeSwitcher() {
     <button
       onClick={toggleTheme}
       className={cn(
-        "p-1 rounded-md rounded-full transition-all duration-300 border border-primary text-primary",
+        "p-1 rounded-full transition-all duration-300 border border-primary text-primary",
       )}
     >
       {theme === "light" ? <Moon size={10} /> : <Sun size={10} />}
diff --git a/src/pages/api/albums/[id]/assets.ts b/src/pages/api/albums/[id]/assets.ts
@@ -3,13 +3,10 @@ import { NextApiRequest } from "next";
 import { db } from "@/config/db";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { NextApiResponse } from "next";
-import { albums } from "@/schema/albums.schema";
-import { count, desc, eq, min, max, sql, and, asc, isNotNull } from "drizzle-orm";
+import { and, eq, isNotNull } from "drizzle-orm";
 import { assets } from "@/schema/assets.schema";
 import { albumsAssetsAssets } from "@/schema/albumAssetsAssets.schema";
-import { users } from "@/schema/users.schema";
 import { assetFaces, exif, person } from "@/schema";
-import { IAlbum } from "@/types/album";
 
 export default async function handler(
   req: NextApiRequest,
diff --git a/src/pages/api/albums/[id]/people.ts b/src/pages/api/albums/[id]/people.ts
@@ -4,12 +4,10 @@ import { db } from "@/config/db";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { NextApiResponse } from "next";
 import { albums } from "@/schema/albums.schema";
-import { count, desc, eq, min, max, sql, and, asc, isNotNull } from "drizzle-orm";
+import { count, desc, eq, and, isNotNull } from "drizzle-orm";
 import { assets } from "@/schema/assets.schema";
 import { albumsAssetsAssets } from "@/schema/albumAssetsAssets.schema";
-import { users } from "@/schema/users.schema";
-import { assetFaces, exif, person } from "@/schema";
-import { IAlbum } from "@/types/album";
+import { assetFaces, person } from "@/schema";
 
 export default async function handler(
   req: NextApiRequest,
diff --git a/src/pages/api/albums/[id]/public-info.ts b/src/pages/api/albums/[id]/public-info.ts
@@ -1,7 +1,6 @@
 import { NextApiRequest } from "next";
 
 import { db } from "@/config/db";
-import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { NextApiResponse } from "next";
 import { albums } from "@/schema/albums.schema";
 import { count, eq, min, max, sql, and } from "drizzle-orm";
diff --git a/src/pages/api/albums/list.ts b/src/pages/api/albums/list.ts
@@ -7,7 +7,6 @@ import { albums } from "@/schema/albums.schema";
 import { count, desc, eq, min, max, sql, and, sum, isNotNull } from "drizzle-orm";
 import { assets } from "@/schema/assets.schema";
 import { albumsAssetsAssets } from "@/schema/albumAssetsAssets.schema";
-import { users } from "@/schema/users.schema";
 import { assetFaces, exif, person } from "@/schema";
 import { IAlbum } from "@/types/album";
 
diff --git a/src/pages/api/albums/potential-albums-assets.ts b/src/pages/api/albums/potential-albums-assets.ts
@@ -1,10 +1,7 @@
-// Next.js API route support: https://nextjs.org/docs/api-routes/introduction
-import { CHART_COLORS } from "@/config/constants/chart.constant";
 import { db } from "@/config/db";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { isFlipped } from "@/helpers/asset.helper";
-import { exif } from "@/schema";
-import { count, desc, isNotNull, ne, sql } from "drizzle-orm";
+import { sql } from "drizzle-orm";
 import type { NextApiRequest, NextApiResponse } from "next";
 
 const SELECT_ORPHAN_PHOTOS = (date: string, ownerId:  string) =>
diff --git a/src/pages/api/albums/potential-albums-dates.ts b/src/pages/api/albums/potential-albums-dates.ts
@@ -4,12 +4,10 @@ import { IPotentialAlbumsDatesResponse } from "@/handlers/api/album.handler";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { parseDate } from "@/helpers/date.helper";
 import { albumsAssetsAssets } from "@/schema/albumAssetsAssets.schema";
-import { albums } from "@/schema/albums.schema";
 import { assets } from "@/schema/assets.schema";
 import { exif } from "@/schema/exif.schema";
-import { and, count, desc, eq, gte, isNotNull, isNull, sql } from "drizzle-orm";
+import { and, count, desc, eq, isNotNull, isNull, sql } from "drizzle-orm";
 import type { NextApiRequest, NextApiResponse } from "next";
-import { number } from "zod";
 
 
 export default async function handler(
diff --git a/src/pages/api/analytics/exif/[property].ts b/src/pages/api/analytics/exif/[property].ts
@@ -1,4 +1,3 @@
-// Next.js API route support: https://nextjs.org/docs/api-routes/introduction
 import { db } from "@/config/db";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { assets, exif } from "@/schema";
diff --git a/src/pages/api/analytics/exif/storage.ts b/src/pages/api/analytics/exif/storage.ts
@@ -2,7 +2,6 @@ import { db } from "@/config/db";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
 import { humanizeBytes } from "@/helpers/string.helper";
 import { assets, exif, users } from "@/schema";
-import { Value } from "@radix-ui/react-select";
 import { desc, eq } from "drizzle-orm";
 import { sum } from "drizzle-orm";
 import { NextApiResponse } from "next";
diff --git a/src/pages/api/assets/missing-location-albums.ts b/src/pages/api/assets/missing-location-albums.ts
@@ -1,7 +1,6 @@
 import { db } from "@/config/db";
 import { IMissingLocationDatesResponse } from "@/handlers/api/asset.handler";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
-import { parseDate } from "@/helpers/date.helper";
 import { assets, exif } from "@/schema";
 import { albumsAssetsAssets } from "@/schema/albumAssetsAssets.schema";
 import { albums } from "@/schema/albums.schema";
diff --git a/src/pages/api/assets/missing-location-dates.ts b/src/pages/api/assets/missing-location-dates.ts
@@ -1,5 +1,3 @@
-// Next.js API route support: https://nextjs.org/docs/api-routes/introduction
-import { CHART_COLORS } from "@/config/constants/chart.constant";
 import { db } from "@/config/db";
 import { IMissingLocationDatesResponse } from "@/handlers/api/asset.handler";
 import { getCurrentUser } from "@/handlers/serverUtils/user.utils";
diff --git a/src/pages/api/find/search.ts b/src/pages/api/find/search.ts
@@ -1,11 +1,11 @@
 import { db } from '@/config/db';
 import { ENV } from '@/config/environment';
 import { getCurrentUser } from '@/handlers/serverUtils/user.utils';
-import { cleanUpAsset, cleanUpAssets, isFlipped } from '@/helpers/asset.helper';
+import { cleanUpAsset, isFlipped } from '@/helpers/asset.helper';
 import { parseFindQuery } from '@/helpers/gemini.helper';
-import { assetFaces, assets, person } from '@/schema';
+import { person } from '@/schema';
 import { Person } from '@/schema/person.schema';
-import { and, eq, inArray } from 'drizzle-orm';
+import { inArray } from 'drizzle-orm';
 import { NextApiRequest, NextApiResponse } from 'next';
 
 export default async function search(req: NextApiRequest, res: NextApiResponse) {
@@ -58,11 +58,11 @@ export default async function search(req: NextApiRequest, res: NextApiResponse)
       },
     });
   }).catch(err => {
-    res.status(500).json({ error: err.message });
+    res.status(500).json({ message: err.message });
   });
   } catch (err: any) {
     console.error(err);
-    res.status(500).json({ error: "Failed to parse query" });
+    res.status(500).json({ message: err.message || "Failed to parse query" });
   }
 
 }
diff --git a/src/pages/api/immich-proxy/[...path].ts b/src/pages/api/immich-proxy/[...path].ts
@@ -1,5 +1,3 @@
-// pages/api/proxy/[...path].ts
-
 import { ENV } from '@/config/environment';
 import { getCurrentUser } from '@/handlers/serverUtils/user.utils';
 import { getUserHeaders } from '@/helpers/user.helper';
diff --git a/src/pages/api/people/[id]/similar-faces.ts b/src/pages/api/people/[id]/similar-faces.ts
diff --git a/src/pages/api/share-link/[token]/assets.ts b/src/pages/api/share-link/[token]/assets.ts
diff --git a/src/pages/api/share-link/[token]/download.ts b/src/pages/api/share-link/[token]/download.ts
diff --git a/src/pages/api/users/login.ts b/src/pages/api/users/login.ts
diff --git a/src/pages/assets/missing-locations.tsx b/src/pages/assets/missing-locations.tsx
diff --git a/src/pages/find/index.tsx b/src/pages/find/index.tsx
diff --git a/src/pages/people/[personId].tsx b/src/pages/people/[personId].tsx
diff --git a/src/pages/s/[token].tsx b/src/pages/s/[token].tsx

Original file line number	Diff line number	Diff line change
`@@ -6,5 +6,9 @@`
`6`	`6`	`"files": ["remotion/*.{ts,tsx}"],`
`7`	`7`	`"extends": ["plugin:@remotion/recommended"]`
`8`	`8`	`}`
`9`		`- ]`
	`9`	`+ ],`
	`10`	`+ "rules": {`
	`11`	`+ "react-hooks/exhaustive-deps": "off",`
	`12`	`+ "@next/next/no-img-element": "off"`
	`13`	`+ }`
`10`	`14`	`}`