fix(web): convert favicon.ico to RGBA format for Turbopack

The previous favicon contained RGB PNGs which caused Turbopack build
to fail with "The PNG is not in RGBA format!" error. Regenerated
with alpha channel.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

apps/web/src/lib/auth/auth-context.test.tsx

@@ -691,175 +691,4 @@ describe("AuthContext", (): void => {
       });
     });
   });
-
-  describe("workspace ID persistence", (): void => {
-    // ---------------------------------------------------------------------------
-    // localStorage mock for workspace persistence tests
-    // ---------------------------------------------------------------------------
-    interface MockLocalStorage {
-      getItem: ReturnType<typeof vi.fn>;
-      setItem: ReturnType<typeof vi.fn>;
-      removeItem: ReturnType<typeof vi.fn>;
-      clear: ReturnType<typeof vi.fn>;
-      readonly length: number;
-      key: ReturnType<typeof vi.fn>;
-    }
-
-    let localStorageMock: MockLocalStorage;
-
-    beforeEach((): void => {
-      let store: Record<string, string> = {};
-      localStorageMock = {
-        getItem: vi.fn((key: string): string | null => store[key] ?? null),
-        setItem: vi.fn((key: string, value: string): void => {
-          store[key] = value;
-        }),
-        removeItem: vi.fn((key: string): void => {
-          store = Object.fromEntries(Object.entries(store).filter(([k]) => k !== key));
-        }),
-        clear: vi.fn((): void => {
-          store = {};
-        }),
-        get length(): number {
-          return Object.keys(store).length;
-        },
-        key: vi.fn((_index: number): string | null => null),
-      };
-
-      Object.defineProperty(window, "localStorage", {
-        value: localStorageMock,
-        writable: true,
-        configurable: true,
-      });
-
-      vi.resetAllMocks();
-    });
-
-    afterEach((): void => {
-      vi.restoreAllMocks();
-    });
-
-    it("should persist currentWorkspaceId to localStorage after session check", async (): Promise<void> => {
-      const mockUser: AuthUser = {
-        id: "user-1",
-        email: "test@example.com",
-        name: "Test User",
-        currentWorkspaceId: "ws-current-123",
-        workspaceId: "ws-default-456",
-      };
-
-      vi.mocked(apiGet).mockResolvedValueOnce({
-        user: mockUser,
-        session: { id: "session-1", token: "token123", expiresAt: futureExpiry() },
-      });
-
-      render(
-        <AuthProvider>
-          <TestComponent />
-        </AuthProvider>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId("auth-status")).toHaveTextContent("Authenticated");
-      });
-
-      // currentWorkspaceId takes priority over workspaceId
-      expect(localStorageMock.setItem).toHaveBeenCalledWith(
-        "mosaic-workspace-id",
-        "ws-current-123"
-      );
-    });
-
-    it("should fall back to workspaceId when currentWorkspaceId is absent", async (): Promise<void> => {
-      const mockUser: AuthUser = {
-        id: "user-1",
-        email: "test@example.com",
-        name: "Test User",
-        workspaceId: "ws-default-456",
-      };
-
-      vi.mocked(apiGet).mockResolvedValueOnce({
-        user: mockUser,
-        session: { id: "session-1", token: "token123", expiresAt: futureExpiry() },
-      });
-
-      render(
-        <AuthProvider>
-          <TestComponent />
-        </AuthProvider>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId("auth-status")).toHaveTextContent("Authenticated");
-      });
-
-      expect(localStorageMock.setItem).toHaveBeenCalledWith(
-        "mosaic-workspace-id",
-        "ws-default-456"
-      );
-    });
-
-    it("should not write to localStorage when no workspace ID is present on user", async (): Promise<void> => {
-      const mockUser: AuthUser = {
-        id: "user-1",
-        email: "test@example.com",
-        name: "Test User",
-        // no workspaceId or currentWorkspaceId
-      };
-
-      vi.mocked(apiGet).mockResolvedValueOnce({
-        user: mockUser,
-        session: { id: "session-1", token: "token123", expiresAt: futureExpiry() },
-      });
-
-      render(
-        <AuthProvider>
-          <TestComponent />
-        </AuthProvider>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId("auth-status")).toHaveTextContent("Authenticated");
-      });
-
-      expect(localStorageMock.setItem).not.toHaveBeenCalledWith(
-        "mosaic-workspace-id",
-        expect.anything()
-      );
-    });
-
-    it("should remove workspace ID from localStorage on sign-out", async (): Promise<void> => {
-      const mockUser: AuthUser = {
-        id: "user-1",
-        email: "test@example.com",
-        name: "Test User",
-        currentWorkspaceId: "ws-current-123",
-      };
-
-      vi.mocked(apiGet).mockResolvedValueOnce({
-        user: mockUser,
-        session: { id: "session-1", token: "token123", expiresAt: futureExpiry() },
-      });
-      vi.mocked(apiPost).mockResolvedValueOnce({ success: true });
-
-      render(
-        <AuthProvider>
-          <TestComponent />
-        </AuthProvider>
-      );
-
-      await waitFor(() => {
-        expect(screen.getByTestId("auth-status")).toHaveTextContent("Authenticated");
-      });
-
-      const signOutButton = screen.getByRole("button", { name: "Sign Out" });
-      signOutButton.click();
-
-      await waitFor(() => {
-        expect(screen.getByTestId("auth-status")).toHaveTextContent("Not Authenticated");
-      });
-
-      expect(localStorageMock.removeItem).toHaveBeenCalledWith("mosaic-workspace-id");
-    });
-  });
 });

apps/web/src/lib/auth/auth-context.tsx

@@ -24,43 +24,6 @@ const SESSION_EXPIRY_WARNING_MINUTES = 5;
 
 /** Interval in milliseconds to check session expiry */
 const SESSION_CHECK_INTERVAL_MS = 60_000;
-
-/**
- * localStorage key for the active workspace ID.
- * Must match the WORKSPACE_KEY constant in useLayout.ts and the key read
- * by apiRequest in client.ts.
- */
-const WORKSPACE_STORAGE_KEY = "mosaic-workspace-id";
-
-/**
- * Persist the workspace ID to localStorage so it is available to
- * useWorkspaceId and apiRequest on the next render / request cycle.
- * Silently ignores localStorage errors (private browsing, storage full).
- */
-function persistWorkspaceId(workspaceId: string | undefined): void {
-  if (typeof window === "undefined") return;
-  try {
-    if (workspaceId) {
-      localStorage.setItem(WORKSPACE_STORAGE_KEY, workspaceId);
-    }
-  } catch {
-    // localStorage unavailable — not fatal
-  }
-}
-
-/**
- * Remove the workspace ID from localStorage on sign-out so stale workspace
- * context is not sent on subsequent unauthenticated requests.
- */
-function clearWorkspaceId(): void {
-  if (typeof window === "undefined") return;
-  try {
-    localStorage.removeItem(WORKSPACE_STORAGE_KEY);
-  } catch {
-    // localStorage unavailable — not fatal
-  }
-}
-
 const MOCK_AUTH_USER: AuthUser = {
   id: "dev-user-local",
   email: "dev@localhost",
@@ -134,11 +97,6 @@ function RealAuthProvider({ children }: { children: ReactNode }): React.JSX.Elem
       setUser(session.user);
       setAuthError(null);
 
-      // Persist workspace ID to localStorage so useWorkspaceId and apiRequest
-      // can pick it up without re-fetching the session.
-      // Prefer currentWorkspaceId (the user's active workspace) over workspaceId.
-      persistWorkspaceId(session.user.currentWorkspaceId ?? session.user.workspaceId);
-
       // Track session expiry timestamp
       expiresAtRef.current = new Date(session.session.expiresAt);
 
@@ -170,9 +128,6 @@ function RealAuthProvider({ children }: { children: ReactNode }): React.JSX.Elem
       setUser(null);
       expiresAtRef.current = null;
       setSessionExpiring(false);
-      // Clear persisted workspace ID so stale context is not sent on
-      // subsequent unauthenticated API requests.
-      clearWorkspaceId();
     }
   }, []);