refactor: extract draft form handling into separate hook

This change extracts the form-related logic from useDraftUtil into a new useDraftForm hook, improving separation of concerns and making the draft utility more modular. The new hook encapsulates form open/close behavior and form props management.

Author: tyler-dane

packages/web/src/views/Calendar/hooks/draft/form/useDraftForm.ts

@@ -0,0 +1,33 @@
+import { OpenChangeReason } from "@floating-ui/react";
+import { useEventForm } from "@web/views/Forms/hooks/useEventForm";
+
+export const useDraftForm = (
+  isFormOpen: boolean,
+  reset: () => void,
+  discard: () => void,
+  setIsFormOpen: (isOpen: boolean) => void
+) => {
+  const onIsFormOpenChange = (isOpen: boolean, reason?: OpenChangeReason) => {
+    const formAlreadyOpen = isFormOpen === true;
+
+    if (formAlreadyOpen) {
+      reset();
+
+      // Not including click or outside press reasons
+      // to avoid conflicting with custom mouse
+      // handlers (useMouseHandlers.ts)
+      if (reason === "escape-key") {
+        discard();
+      }
+      return;
+    }
+    setIsFormOpen(isOpen);
+
+    if (isOpen === false) {
+      reset();
+      discard();
+    }
+  };
+  const formProps = useEventForm("grid", isFormOpen, onIsFormOpenChange);
+  return { formProps };
+};

packages/web/src/views/Calendar/hooks/draft/useDraftUtil.ts

@@ -1,6 +1,5 @@
 import { MouseEvent, useCallback, useEffect, useState } from "react";
 import dayjs, { Dayjs } from "dayjs";
-import { OpenChangeReason } from "@floating-ui/react";
 import {
   Priorities,
   SOMEDAY_WEEK_LIMIT_MSG,
@@ -26,7 +25,6 @@ import {
   selectDraftStatus,
 } from "@web/ducks/events/selectors/draft.selectors";
 import { GRID_TIME_STEP } from "@web/views/Calendar/layout.constants";
-import { useEventForm } from "@web/views/Forms/hooks/useEventForm";
 import {
   selectIsAtWeeklyLimit,
   selectSomedayWeekCount,
@@ -35,6 +33,7 @@ import { getUserId } from "@web/auth/auth.util";
 
 import { DateCalcs } from "../grid/useDateCalcs";
 import { WeekProps } from "../useWeek";
+import { useDraftForm } from "./form/useDraftForm";
 export interface Status_Drag {
   durationMin: number;
   hasMoved?: boolean;
@@ -71,38 +70,6 @@ export const useDraftUtil = (
     "startDate" | "endDate" | null
   >("endDate");
   const [isFormOpen, setIsFormOpen] = useState(false);
-  const onIsFormOpenChange = (isOpen: boolean, reason?: OpenChangeReason) => {
-    const formAlreadyOpen = isFormOpen === true;
-
-    if (formAlreadyOpen) {
-      reset();
-
-      // Not including click or outside press reasons
-      // to avoid conflicting with custom mouse
-      // handlers (useMouseHandlers.ts)
-      if (reason === "escape-key") {
-        discard();
-      }
-      return;
-    }
-    setIsFormOpen(isOpen);
-
-    if (isOpen === false) {
-      reset();
-      discard();
-    }
-  };
-  const formProps = useEventForm("grid", isFormOpen, onIsFormOpenChange);
-
-  const reset = () => {
-    setDraft(null);
-    setIsDragging(false);
-    setIsFormOpen(false);
-    setIsResizing(false);
-    setDragStatus(null);
-    setResizeStatus(null);
-    setDateBeingChanged(null);
-  };
 
   useEffect(() => {
     reset();
@@ -124,6 +91,25 @@ export const useDraftUtil = (
     }
   }, [isDrafting, isFormOpen]);
 
+  const reset = () => {
+    setDraft(null);
+    setIsDragging(false);
+    setIsFormOpen(false);
+    setIsResizing(false);
+    setDragStatus(null);
+    setResizeStatus(null);
+    setDateBeingChanged(null);
+  };
+  const discard = useCallback(() => {
+    if (draft) {
+      setDraft(null);
+    }
+
+    if (reduxDraft || reduxDraftType) {
+      dispatch(draftSlice.actions.discard());
+    }
+  }, [dispatch, draft, reduxDraft, reduxDraftType]);
+
   const handleResizing = useCallback(() => {
     console.log("-setting local draft to:", reduxDraft);
     setDraft(reduxDraft);
@@ -225,16 +211,6 @@ export const useDraftUtil = (
     discard();
   };
 
-  const discard = useCallback(() => {
-    if (draft) {
-      setDraft(null);
-    }
-
-    if (reduxDraft || reduxDraftType) {
-      dispatch(draftSlice.actions.discard());
-    }
-  }, [dispatch, draft, reduxDraft, reduxDraftType]);
-
   const drag = useCallback(
     (e: MouseEvent) => {
       const updateTimesDuringDrag = (e: MouseEvent) => {
@@ -445,6 +421,8 @@ export const useDraftUtil = (
     discard();
   };
 
+  const { formProps } = useDraftForm(isFormOpen, reset, discard, setIsFormOpen);
+
   return {
     draftState: {
       draft,