From bb00c99678ce3ca0493599c730a357f9daf2346b Mon Sep 17 00:00:00 2001
From: Mohamed Bassem <me@mbassem.com>
Date: Sun, 2 Nov 2025 19:38:17 +0000
Subject: refactor: improve the userLocalSetting server functions

---
 .../web/lib/userLocalSettings/userLocalSettings.ts | 41 ++++++++--------------
 1 file changed, 14 insertions(+), 27 deletions(-)

(limited to 'apps/web/lib/userLocalSettings')

diff --git a/apps/web/lib/userLocalSettings/userLocalSettings.ts b/apps/web/lib/userLocalSettings/userLocalSettings.ts
index 25c10e1b..aff3645c 100644
--- a/apps/web/lib/userLocalSettings/userLocalSettings.ts
+++ b/apps/web/lib/userLocalSettings/userLocalSettings.ts
@@ -16,46 +16,33 @@ export async function getUserLocalSettings(): Promise<UserLocalSettings> {
   );
 }
 
-export async function updateBookmarksLayout(layout: BookmarksLayoutTypes) {
+async function readModifyWrite(
+  modifier: (settings: UserLocalSettings) => Partial<UserLocalSettings>,
+) {
   const userSettings = (await cookies()).get(USER_LOCAL_SETTINGS_COOKIE_NAME);
-  const parsed = parseUserLocalSettings(userSettings?.value);
+  const parsed =
+    parseUserLocalSettings(userSettings?.value) ?? defaultUserLocalSettings();
+  const updated = { ...parsed, ...modifier(parsed) };
   (await cookies()).set({
     name: USER_LOCAL_SETTINGS_COOKIE_NAME,
-    value: JSON.stringify({ ...parsed, bookmarkGridLayout: layout }),
+    value: JSON.stringify(updated),
     maxAge: 34560000, // Chrome caps max age to 400 days
     sameSite: "lax",
   });
 }
 
+export async function updateBookmarksLayout(layout: BookmarksLayoutTypes) {
+  await readModifyWrite(() => ({ bookmarkGridLayout: layout }));
+}
+
 export async function updateInterfaceLang(lang: string) {
-  const userSettings = (await cookies()).get(USER_LOCAL_SETTINGS_COOKIE_NAME);
-  const parsed = parseUserLocalSettings(userSettings?.value);
-  (await cookies()).set({
-    name: USER_LOCAL_SETTINGS_COOKIE_NAME,
-    value: JSON.stringify({ ...parsed, lang }),
-    maxAge: 34560000, // Chrome caps max age to 400 days
-    sameSite: "lax",
-  });
+  await readModifyWrite(() => ({ lang }));
 }
 
 export async function updateGridColumns(gridColumns: number) {
-  const userSettings = (await cookies()).get(USER_LOCAL_SETTINGS_COOKIE_NAME);
-  const parsed = parseUserLocalSettings(userSettings?.value);
-  (await cookies()).set({
-    name: USER_LOCAL_SETTINGS_COOKIE_NAME,
-    value: JSON.stringify({ ...parsed, gridColumns }),
-    maxAge: 34560000, // Chrome caps max age to 400 days
-    sameSite: "lax",
-  });
+  await readModifyWrite(() => ({ gridColumns }));
 }
 
 export async function updateShowNotes(showNotes: boolean) {
-  const userSettings = (await cookies()).get(USER_LOCAL_SETTINGS_COOKIE_NAME);
-  const parsed = parseUserLocalSettings(userSettings?.value);
-  (await cookies()).set({
-    name: USER_LOCAL_SETTINGS_COOKIE_NAME,
-    value: JSON.stringify({ ...parsed, showNotes }),
-    maxAge: 34560000, // Chrome caps max age to 400 days
-    sameSite: "lax",
-  });
+  await readModifyWrite(() => ({ showNotes }));
 }
-- 
cgit v1.2.3-70-g09d2