refactor(web): drop draft context setters for setLocalDraft

packages/web/src/views/Week/components/Draft/Draft.tsx

@@ -34,9 +34,9 @@ export const Draft: FC<Props> = ({ measurements, weekProps }) => {
     startOfView: weekProps.query.startOfView,
     endOfView: weekProps.query.endOfView,
   });
-  const { setters, state } = useDraftContext();
+  const { actions, state } = useDraftContext();
   const { draft, isDragging, isFormOpen, isResizing } = state;
-  const { setDraft } = setters;
+  const { setLocalDraft } = actions;
   const gridDraftFromStore = useDraftStore(selectGridDraft);
   const activity = useDraftStore(selectDraftActivity);
 
@@ -53,7 +53,7 @@ export const Draft: FC<Props> = ({ measurements, weekProps }) => {
     gridDraftFromStore &&
     (draft === null || activity === "eventRightClick")
   ) {
-    setDraft(gridDraftFromStore);
+    setLocalDraft(gridDraftFromStore);
   }
 
   // Sidebar edits write the shared draft store. Mirror into Week local draft
@@ -63,8 +63,8 @@ export const Draft: FC<Props> = ({ measurements, weekProps }) => {
       return;
     }
 
-    setDraft(gridDraftFromStore);
-  }, [gridDraftFromStore, isDragging, isFormOpen, isResizing, setDraft]);
+    setLocalDraft(gridDraftFromStore);
+  }, [gridDraftFromStore, isDragging, isFormOpen, isResizing, setLocalDraft]);
 
   // GridEvent-shaped projection of the canonical GridEventDraft, for
   // the still-unconverted grid-layout helpers below (deck layout, all-day

packages/web/src/views/Week/components/Draft/context/DraftContext.tsx

@@ -1,15 +1,11 @@
 import { createContext } from "react";
 import { type Actions_Draft } from "../hooks/actions/useDraftActions";
-import {
-  type Setters_Draft,
-  type State_Draft_Local,
-} from "../hooks/state/useDraftState";
+import { type State_Draft_Local } from "../hooks/state/useDraftState";
 
 export type State_Draft = State_Draft_Local;
 
 interface DraftContextValue {
   state: State_Draft;
-  setters: Setters_Draft;
   actions: Actions_Draft;
 }
 

packages/web/src/views/Week/components/Draft/context/DraftProvider.tsx

@@ -19,7 +19,7 @@ export const DraftProvider = ({
   const actions = useDraftActions(state, setters, dateCalcs, weekProps);
 
   return (
-    <DraftContext.Provider value={{ state, setters, actions }}>
+    <DraftContext.Provider value={{ state, actions }}>
       {children}
     </DraftContext.Provider>
   );

packages/web/src/views/Week/components/Draft/grid/GridDraft.test.tsx

@@ -95,9 +95,6 @@ const renderGridDraft = ({
       startDragging: mock(),
       startResizing: mock(),
     },
-    setters: {
-      setDraft: mock(),
-    },
     state: {
       draft,
       dragOffset: { x: 0, y: 0 },

packages/web/src/views/Week/components/Draft/hooks/actions/useDraftActions.ts

@@ -620,6 +620,7 @@ export const useDraftActions = (
     openForm,
     repositionDraftByKeyboard,
     resize,
+    setLocalDraft: setDraft,
     startDragging: (offset?: DragOffset) => {
       // Placing `setIsFormOpenBeforeDragging` here rather than inside `startDragging`
       // because `setIsFormOpenBeforeDragging` depends on `isFormOpen` and re-calculates

packages/web/src/views/Week/components/Grid/MainGrid/MainGrid.test.tsx

@@ -241,7 +241,6 @@ const renderMainGrid = () => {
         value={
           {
             actions,
-            setters: {},
             state: {},
           } as never
         }
@@ -276,7 +275,6 @@ const renderGridRegions = () => {
               stopDragging: mock(),
               stopResizing: mock(),
             },
-            setters: {},
             state: {},
           } as never
         }
@@ -317,7 +315,6 @@ const renderWeekGrid = (events: CompassEvent[] = []) => {
               stopDragging: mock(),
               stopResizing: mock(),
             },
-            setters: {},
             state: {},
           } as never
         }

packages/web/src/views/Week/components/Grid/MainGrid/eventReadOnlyInteraction.test.tsx

@@ -170,7 +170,6 @@ const renderMainGridEvents = () =>
         value={
           {
             actions: { stopDragging: mock(), stopResizing: mock() },
-            setters: {},
             state: {},
           } as never
         }
@@ -190,7 +189,6 @@ const renderAllDayEvents = () =>
         value={
           {
             actions: { stopDragging: mock(), stopResizing: mock() },
-            setters: {},
             state: {},
           } as never
         }

packages/web/src/views/Week/hooks/shortcuts/useWeekShortcuts.test.tsx

@@ -191,7 +191,6 @@ const renderShortcuts = (options?: {
             value={
               {
                 actions: { repositionDraftByKeyboard },
-                setters: {},
                 state: {},
               } as never
             }

packages/web/src/views/Week/interaction/WeekInteractionCoordinator.tsx

@@ -40,7 +40,7 @@ export const WeekInteractionCoordinator: FC<Props> = ({
     startOfView: weekProps.query.startOfView,
     endOfView: weekProps.query.endOfView,
   });
-  const { actions, setters, state } = useDraftContext();
+  const { actions, state } = useDraftContext();
   const updateEvent = useUpdateEvent();
   const activeInteractionEventRef = useRef<Event | null>(null);
   const layoutSourcesRef = useRef(getLayoutSources);
@@ -144,7 +144,7 @@ export const WeekInteractionCoordinator: FC<Props> = ({
     if (result.hadFormOpenBeforeInteraction) {
       const draft = gridEventDraftFromSavedResult(result.event);
       if (draft) {
-        setters.setDraft(draft);
+        actions.setLocalDraft(draft);
         draftActions.setGridDraft(draft);
       }