refactor(cleanup): remove unused variables and improve naming consistency

Author: devsebastian44

src/app/dashboard/datasets/page.tsx

@@ -16,7 +16,7 @@ import {
 import { useRouter } from 'next/navigation';
 import { UserNav } from '@/components/layout/user-nav';
 
-const DatasetItem = ({ file, onDelete }: { file: DatasetFile, onDelete: (fileName: string) => void }) => {
+const DatasetItem = ({ file, onDelete }: { file: DatasetFile, onDelete: (_name: string) => void }) => {
     const router = useRouter();
 
     const handleViewAnalysis = () => {

src/components/pages/dashboard/project-card.tsx

@@ -45,10 +45,10 @@ import { formatDistanceToNow } from 'date-fns';
 
 type ProjectCardProps = {
   project: Project;
-  onArchive: (_projectId: string) => void;
-  onDelete: (_projectId: string) => void;
-  onRestore: (_projectId: string) => void;
-  onRename: (_projectId: string, _newName: string) => void;
+  onArchive: (_id: string) => void;
+  onDelete: (_id: string) => void;
+  onRestore: (_id: string) => void;
+  onRename: (_id: string, _newName: string) => void;
 };
 
 const statusInfo: { [key: string]: { style: string; icon: React.ReactNode } } = {

src/components/pages/dashboard/project-list-item.tsx

@@ -42,10 +42,10 @@ import { formatDistanceToNow } from 'date-fns';
 
 type ProjectListItemProps = {
   project: Project;
-  onArchive: (_projectId: string) => void;
-  onDelete: (_projectId: string) => void;
-  onRestore: (_projectId: string) => void;
-  onRename: (_projectId: string, _newName: string) => void;
+  onArchive: (_id: string) => void;
+  onDelete: (_id: string) => void;
+  onRestore: (_id: string) => void;
+  onRename: (_id: string, _newName: string) => void;
 };
 
 const statusInfo: { [key: string]: { style: string; icon: React.ReactNode } } = {
@@ -72,7 +72,7 @@ const statusInfo: { [key: string]: { style: string; icon: React.ReactNode } } =
 };
 
 
-export function ProjectListItem({ project, onArchive, onDelete, onRestore, onRename }: ProjectListItemProps) {
+export function ProjectListItem({ project, onArchive, onDelete, onRestore: _onRestore, onRename }: ProjectListItemProps) {
   const router = useRouter();
   const [isRenameDialogOpen, setIsRenameDialogOpen] = useState(false);
   const [newName, setNewName] = useState(project.name);

src/components/ui/carousel.tsx

@@ -18,7 +18,7 @@ type CarouselProps = {
   opts?: CarouselOptions
   plugins?: CarouselPlugin
   orientation?: "horizontal" | "vertical"
-  setApi?: (api: CarouselApi) => void
+  setApi?: (_api: any) => void
 }
 
 type CarouselContextProps = {

src/components/ui/sidebar.tsx

@@ -26,16 +26,16 @@ const SIDEBAR_WIDTH_MOBILE = "18rem"
 const SIDEBAR_WIDTH_ICON = "3rem"
 const SIDEBAR_KEYBOARD_SHORTCUT = "b"
 
-type SidebarContext = {
+type SidebarContextType = {
   state: "expanded" | "collapsed"
-  setOpen: (open: boolean) => void
+  setOpen: (_open: boolean) => void
   openMobile: boolean
-  setOpenMobile: (open: boolean) => void
+  setOpenMobile: (_open: boolean) => void
   isMobile: boolean
   toggleSidebar: () => void
 }
 
-const SidebarContext = React.createContext<SidebarContext | null>(null)
+const SidebarContext = React.createContext<SidebarContextType | null>(null)
 
 function useSidebar() {
   const context = React.useContext(SidebarContext)
@@ -51,7 +51,7 @@ const SidebarProvider = React.forwardRef<
   React.ComponentProps<"div"> & {
     defaultOpen?: boolean
     open?: boolean
-    onOpenChange?: (open: boolean) => void
+    onOpenChange?: (_open: boolean) => void
   }
 >(
   (
@@ -74,7 +74,7 @@ const SidebarProvider = React.forwardRef<
     const [_open, _setOpen] = React.useState(defaultOpen)
     const open = openProp ?? _open
     const setOpen = React.useCallback(
-      (value: boolean | ((value: boolean) => boolean)) => {
+      (value: boolean | ((_value: boolean) => boolean)) => {
         const openState = typeof value === "function" ? value(open) : value
         if (setOpenProp) {
           setOpenProp(openState)
@@ -115,7 +115,7 @@ const SidebarProvider = React.forwardRef<
     // This makes it easier to style the sidebar with Tailwind classes.
     const state = open ? "expanded" : "collapsed"
 
-    const contextValue = React.useMemo<SidebarContext>(
+    const contextValue = React.useMemo<SidebarContextType>(
       () => ({
         state,
         setOpen,

src/hooks/use-toast.ts

@@ -66,7 +66,7 @@ const addToRemoveQueue = (toastId: string) => {
   const timeout = setTimeout(() => {
     toastTimeouts.delete(toastId)
     dispatch({
-      type: "REMOVE_TOAST",
+      type: actionTypes.REMOVE_TOAST,
       toastId: toastId,
     })
   }, TOAST_REMOVE_DELAY)
@@ -76,21 +76,21 @@ const addToRemoveQueue = (toastId: string) => {
 
 export const reducer = (state: State, action: Action): State => {
   switch (action.type) {
-    case "ADD_TOAST":
+    case actionTypes.ADD_TOAST:
       return {
         ...state,
         toasts: [action.toast, ...state.toasts].slice(0, TOAST_LIMIT),
       }
 
-    case "UPDATE_TOAST":
+    case actionTypes.UPDATE_TOAST:
       return {
         ...state,
         toasts: state.toasts.map((t) =>
           t.id === action.toast.id ? { ...t, ...action.toast } : t
         ),
       }
 
-    case "DISMISS_TOAST": {
+    case actionTypes.DISMISS_TOAST: {
       const { toastId } = action
 
       // ! Side effects ! - This could be extracted into a dismissToast() action,
@@ -115,7 +115,7 @@ export const reducer = (state: State, action: Action): State => {
         ),
       }
     }
-    case "REMOVE_TOAST":
+    case actionTypes.REMOVE_TOAST:
       if (action.toastId === undefined) {
         return {
           ...state,
@@ -129,7 +129,7 @@ export const reducer = (state: State, action: Action): State => {
   }
 }
 
-const listeners: Array<(state: State) => void> = []
+const listeners: Array<(_state: State) => void> = []
 
 let memoryState: State = { toasts: [] }
 
@@ -147,13 +147,13 @@ function toast({ ...props }: Toast) {
 
   const update = (props: ToasterToast) =>
     dispatch({
-      type: "UPDATE_TOAST",
+      type: actionTypes.UPDATE_TOAST,
       toast: { ...props, id },
     })
-  const dismiss = () => dispatch({ type: "DISMISS_TOAST", toastId: id })
+  const dismiss = () => dispatch({ type: actionTypes.DISMISS_TOAST, toastId: id })
 
   dispatch({
-    type: "ADD_TOAST",
+    type: actionTypes.ADD_TOAST,
     toast: {
       ...props,
       id,
@@ -187,7 +187,7 @@ function useToast() {
   return {
     ...state,
     toast,
-    dismiss: (toastId?: string) => dispatch({ type: "DISMISS_TOAST", toastId }),
+    dismiss: (toastId?: string) => dispatch({ type: actionTypes.DISMISS_TOAST, toastId }),
   }
 }