refactor: return types for hooks

src/app/deals/hooks/useDealsAndStatusesDnd.ts

@@ -8,7 +8,16 @@ import { getStatusId, isStatusId } from "@/app/deals/utils/statusId";
 import { DealSchema, StatusSchema } from "@/lib/client";
 import { sortByLexorank } from "@/utils/lexorank";
 
-const useDealsAndStatusesDnd = () => {
+type ReturnType = {
+    sortedStatuses: StatusSchema[];
+    handleDragStart: ({ active }: DragStartEvent) => void;
+    handleDragOver: ({ active, over }: DragOverEvent) => void;
+    handleDragEnd: ({ active, over }: DragOverEvent) => void;
+    activeStatus: StatusSchema | null;
+    activeDeal: DealSchema | null;
+};
+
+const useDealsAndStatusesDnd = (): ReturnType => {
     const [activeDeal, setActiveDeal] = useState<DealSchema | null>(null);
     const [activeStatus, setActiveStatus] = useState<StatusSchema | null>(null);
     const { statuses, setStatuses, updateStatus } = useStatusesContext();

src/app/deals/hooks/useGetNewRank.ts

@@ -3,14 +3,30 @@ import { useStatusesContext } from "@/app/deals/contexts/StatusesContext";
 import { DealSchema } from "@/lib/client";
 import { getNewLexorank, sortByLexorank } from "@/utils/lexorank";
 
-const useGetNewRank = () => {
+type NewRankGetters = {
+    getNewRankForSameStatus: (
+        statusDeals: DealSchema[],
+        overDealIndex: number,
+        activeDealId: number
+    ) => string;
+    getNewRankForAnotherStatus: (
+        statusDeals: DealSchema[],
+        overDealIndex: number
+    ) => string;
+    getNewStatusRank: (
+        activeStatusId: number,
+        overStatusId: number
+    ) => string | null;
+};
+
+const useGetNewRank = (): NewRankGetters => {
     const { statuses } = useStatusesContext();
 
     const getNewRankForSameStatus = (
         statusDeals: DealSchema[],
         overDealIndex: number,
         activeDealId: number
-    ) => {
+    ): string => {
         const activeDealIndex = statusDeals.findIndex(
             deal => deal.id === activeDealId
         );
@@ -34,7 +50,7 @@ const useGetNewRank = () => {
     const getNewRankForAnotherStatus = (
         statusDeals: DealSchema[],
         overDealIndex: number
-    ) => {
+    ): string => {
         const leftLexorank =
             overDealIndex > 0
                 ? LexoRank.parse(statusDeals[overDealIndex - 1].lexorank)
@@ -46,7 +62,10 @@ const useGetNewRank = () => {
         return getNewLexorank(leftLexorank, rightLexorank).toString();
     };
 
-    const getNewStatusRank = (activeStatusId: number, overStatusId: number) => {
+    const getNewStatusRank = (
+        activeStatusId: number,
+        overStatusId: number
+    ): string | null => {
         const sortedStatusList = sortByLexorank(statuses);
         const overIndex = sortedStatusList.findIndex(
             s => s.id === overStatusId

src/hooks/useBoardsOperations.tsx

@@ -24,12 +24,18 @@ type UseBoardsOperationsProps = {
     projectId?: number;
 };
 
+type BoardsOperations = {
+    onCreateBoard: (name: string) => void;
+    onUpdateBoard: (boardId: number, board: UpdateBoardSchema) => void;
+    onDeleteBoard: (board: BoardSchema) => void;
+};
+
 export const useBoardsOperations = ({
     boards,
     setBoards,
     refetchBoards,
     projectId,
-}: UseBoardsOperationsProps) => {
+}: UseBoardsOperationsProps): BoardsOperations => {
     const onError = (error: AxiosError<HttpValidationError>) => {
         console.error(error);
         notifications.error({

src/hooks/useIsMobile.ts

@@ -1,6 +1,6 @@
 import { useMediaQuery } from "@mantine/hooks";
 
-const useIsMobile = () => {
+const useIsMobile = (): boolean => {
     return useMediaQuery("(max-width: 48em)");
 };
 

src/hooks/useProjectsOperations.tsx

@@ -21,11 +21,17 @@ type Props = {
     refetchProjects: () => void;
 };
 
+type ProjectsOperations = {
+    onCreateProject: (name: string) => void;
+    onUpdateProject: (projectId: number, project: UpdateProjectSchema) => void;
+    onDeleteProject: (project: ProjectSchema) => void;
+};
+
 export const useProjectsOperations = ({
     projects,
     setProjects,
     refetchProjects,
-}: Props) => {
+}: Props): ProjectsOperations => {
     const onError = (error: AxiosError<HttpValidationError>) => {
         console.error(error);
         notifications.error({

src/hooks/useStatusesOperations.tsx

@@ -24,12 +24,18 @@ type Props = {
     boardId?: number;
 };
 
+type StatusesOperations = {
+    onCreateStatus: (name: string) => void;
+    onUpdateStatus: (statusId: number, status: UpdateStatusSchema) => void;
+    onDeleteStatus: (status: StatusSchema) => void;
+};
+
 export const useStatusesOperations = ({
     statuses,
     setStatuses,
     refetchStatuses,
     boardId,
-}: Props) => {
+}: Props): StatusesOperations => {
     const onError = (error: AxiosError<HttpValidationError>) => {
         console.error(error);
         notifications.error({

src/modals/EnterNameModal/EnterNameModal.tsx

@@ -10,7 +10,7 @@ type Props = {
 };
 
 type FormType = {
-    name?: string;
+    name: string;
 };
 
 const EnterNameModal = ({
@@ -28,7 +28,7 @@ const EnterNameModal = ({
     });
 
     const onSubmit = (values: FormType) => {
-        innerProps.onComplete(values.name!);
+        innerProps.onComplete(values.name);
         context.closeModal(id);
     };