From 9cf1085fbb79dc2f132cb295423dbe4fe6ea831e Mon Sep 17 00:00:00 2001
From: Domagoj Kriskovic <dom.kriskovic@overleaf.com>
Date: Mon, 25 May 2026 12:51:38 +0200
Subject: [PATCH] [web] use updateProject for saving trackChangesState

GitOrigin-RevId: eecb2b78ff18547e8b3653fdff2d380d295c367f
---
 .../context/track-changes-state-context.tsx   | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/services/web/frontend/js/features/review-panel/context/track-changes-state-context.tsx b/services/web/frontend/js/features/review-panel/context/track-changes-state-context.tsx
index e68ef1fec1..f2fb4c7ebe 100644
--- a/services/web/frontend/js/features/review-panel/context/track-changes-state-context.tsx
+++ b/services/web/frontend/js/features/review-panel/context/track-changes-state-context.tsx
@@ -6,7 +6,6 @@ import {
   useContext,
   useEffect,
   useMemo,
-  useState,
 } from 'react'
 import useSocketListener from '@/features/ide-react/hooks/use-socket-listener'
 import { useConnectionContext } from '@/features/ide-react/context/connection-context'
@@ -48,16 +47,23 @@ export const TrackChangesStateProvider: FC<React.PropsWithChildren> = ({
 }) => {
   const permissions = usePermissionsContext()
   const { socket } = useConnectionContext()
-  const { projectId, project, features } = useProjectContext()
+  const { projectId, project, updateProject, features } = useProjectContext()
   const user = useUserContext()
   const { setWantTrackChanges } = useEditorPropertiesContext()
 
-  // TODO: update project.trackChangesState instead?
-  const [trackChangesValue, setTrackChangesValue] = useState<
-    ProjectMetadata['trackChangesState']
-  >(project?.trackChangesState ?? false)
+  const trackChangesValue: ProjectMetadata['trackChangesState'] =
+    project?.trackChangesState ?? false
 
-  useSocketListener(socket, 'toggle-track-changes', setTrackChangesValue)
+  useSocketListener(
+    socket,
+    'toggle-track-changes',
+    useCallback(
+      (newValue: ProjectMetadata['trackChangesState']) => {
+        updateProject({ trackChangesState: newValue })
+      },
+      [updateProject]
+    )
+  )
 
   useEffect(() => {
     setWantTrackChanges(