fix(app): dismiss the desktop tip when session storage writes fail

apps/app/src/components/desktop-tip-modal.test.tsx

@@ -0,0 +1,29 @@
+import { afterEach, expect, it, vi } from "vitest";
+import { cleanup, fireEvent, render, screen } from "@testing-library/react";
+import "@testing-library/jest-dom/vitest";
+
+afterEach(() => { cleanup(); vi.restoreAllMocks(); sessionStorage.clear(); });
+
+it("dismisses for the rest of the tab when storage rejects writes", async () => {
+  vi.resetModules();
+  const { DesktopTipModal } = await import("./desktop-tip-modal");
+  vi.spyOn(Storage.prototype, "setItem").mockImplementation(() => {
+    throw new DOMException("Storage unavailable", "QuotaExceededError");
+  });
+  const view = render(<DesktopTipModal />);
+  expect(screen.getByText("Best viewed on desktop")).toBeInTheDocument();
+  fireEvent.click(screen.getByRole("button", { name: "Continue anyway" }));
+  expect(screen.queryByText("Best viewed on desktop")).not.toBeInTheDocument();
+  view.unmount();
+  render(<DesktopTipModal />);
+  expect(screen.queryByText("Best viewed on desktop")).not.toBeInTheDocument();
+});
+
+it("supports Escape when storage rejects writes", async () => {
+  vi.resetModules();
+  const { DesktopTipModal } = await import("./desktop-tip-modal");
+  vi.spyOn(Storage.prototype, "setItem").mockImplementation(() => { throw new Error("Denied"); });
+  render(<DesktopTipModal />);
+  fireEvent.keyDown(document, { key: "Escape" });
+  expect(screen.queryByText("Best viewed on desktop")).not.toBeInTheDocument();
+});

apps/app/src/components/desktop-tip-modal.tsx

@@ -3,6 +3,7 @@
 import { useEffect, useSyncExternalStore } from "react";
 
 const DISMISSED_KEY = "desktop-tip-dismissed";
+let dismissedInMemory = false;
 
 let listeners: Array<() => void> = [];
 function emitChange() {
@@ -18,13 +19,15 @@ export function DesktopTipModal() {
       };
     },
     () => {
+      if (dismissedInMemory) return false;
       try { return !sessionStorage.getItem(DISMISSED_KEY); }
       catch { return false; }
     },
     () => false,
   );
 
   const dismiss = () => {
+    dismissedInMemory = true;
     try { sessionStorage.setItem(DISMISSED_KEY, "1"); }
     catch { /* privacy-restricted context */ }
     emitChange();