From 90878a6d7f4098df69e518cf0001a8fb443121b6 Mon Sep 17 00:00:00 2001
From: Nico <Nico>
Date: Thu, 4 Jun 2026 06:11:14 -0700
Subject: [PATCH] fix: silence store selector debug logs

---
 frontend/src/context/StoreContext.jsx | 6 ------
 frontend/tests/store-selector.spec.ts | 8 ++++++++
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/frontend/src/context/StoreContext.jsx b/frontend/src/context/StoreContext.jsx
index 7a7f22f..d7b92d5 100644
--- a/frontend/src/context/StoreContext.jsx
+++ b/frontend/src/context/StoreContext.jsx
@@ -84,7 +84,6 @@ export const StoreProvider = ({ children }) => {
   useEffect(() => {
     if (!activeHousehold || stores.length === 0) return;
 
-    console.log('[StoreContext] Setting active store from:', stores);
     const householdStoreStorageKey = `activeHouseholdStoreId_${activeHousehold.id}`;
     const legacyLocationStorageKey = `activeStoreId_${activeHousehold.id}`;
     const savedHouseholdStoreId = localStorage.getItem(householdStoreStorageKey);
@@ -94,7 +93,6 @@ export const StoreProvider = ({ children }) => {
         chooseLocationByStoredIds(stores, activeHousehold.id, savedHouseholdStoreId) ||
         chooseStoreByHouseholdStoreId(stores, savedHouseholdStoreId);
       if (store) {
-        console.log('[StoreContext] Found saved household store:', store);
         setActiveStoreState(store);
         return;
       }
@@ -105,7 +103,6 @@ export const StoreProvider = ({ children }) => {
       const savedLocation = stores.find(s => String(s.id) === String(savedLocationId));
       const store = savedLocation || chooseStoreByHouseholdStoreId(stores, getHouseholdStoreKey(savedLocation));
       if (store) {
-        console.log('[StoreContext] Migrated saved location to store:', store);
         const householdStoreId = getHouseholdStoreKey(store);
         setActiveStoreState(store);
         localStorage.setItem(householdStoreStorageKey, householdStoreId);
@@ -122,7 +119,6 @@ export const StoreProvider = ({ children }) => {
       stores,
       getHouseholdStoreKey(stores.find(s => s.is_default) || stores[0])
     );
-    console.log('[StoreContext] Using store:', defaultStore);
     setActiveStoreState(defaultStore);
     const householdStoreId = getHouseholdStoreKey(defaultStore);
     localStorage.setItem(householdStoreStorageKey, householdStoreId);
@@ -146,9 +142,7 @@ export const StoreProvider = ({ children }) => {
     setLoading(true);
     setError(null);
     try {
-      console.log('[StoreContext] Loading stores for household:', activeHousehold.id);
       const response = await getHouseholdStores(activeHousehold.id);
-      console.log('[StoreContext] Loaded stores:', response.data);
       setStores(response.data);
     } catch (err) {
       console.error('[StoreContext] Failed to load stores:', err);
diff --git a/frontend/tests/store-selector.spec.ts b/frontend/tests/store-selector.spec.ts
index b020d40..9802db0 100644
--- a/frontend/tests/store-selector.spec.ts
+++ b/frontend/tests/store-selector.spec.ts
@@ -95,6 +95,13 @@ async function mockGroceryShell(page: Page, stores: Array<Record<string, unknown
 
 test("grocery store selector shows one selected store and picks from a modal", async ({ page }) => {
   await page.setViewportSize({ width: 473, height: 1000 });
+  const storeContextMessages: string[] = [];
+  page.on("console", (message) => {
+    const text = message.text();
+    if (text.includes("[StoreContext]")) {
+      storeContextMessages.push(text);
+    }
+  });
   await mockGroceryShell(page, [
     {
       id: 10,
@@ -191,6 +198,7 @@ test("grocery store selector shows one selected store and picks from a modal", a
   await expect(storeSelector).toContainText("Costco");
   await expect(locationSelector).toContainText("Fontana");
   await expect(page.getByRole("button", { name: "Manage Map" })).toBeVisible();
+  expect(storeContextMessages).toEqual([]);
 });
 
 test("single grocery store selector click does not open picker modal", async ({ page }) => {