feat: resize tasks and consolidate storage (#1907)

* style: lighten header color

* feat: pov of adjustable stuff

* fix: timed task widths

* refactor: simplify reusable task list

* style: header positioning

* style: divider color

* refactor: storage interfaces

* refactor(web): simplify storage boundaries

Author: tyler-dane

docs/architecture/event-and-task-domain-model.md

@@ -115,18 +115,19 @@ Task fields:
 - `description`
 - `user`
 
-Tasks are currently local-storage centric and are stored with a `dateKey` in the adapter layer, not in the public `Task` shape.
+Tasks are stored with a `dateKey` in the offline data store, not in the public
+`Task` shape.
 
 ## Storage-Specific Task Shape
 
-The IndexedDB adapter wraps tasks as `StoredTask`:
+The IndexedDB offline data store wraps tasks as `StoredTask`:
 
 - public task data
 - plus `dateKey`
 
 Source:
 
-- `packages/web/src/common/storage/adapter/storage.adapter.ts`
+- `packages/web/src/common/storage/offline-data/offline-data.store.ts`
 
 ## Invariants To Preserve
 

docs/development/common-change-recipes.md

@@ -61,10 +61,10 @@ Do not edit recurring behavior from one layer only.
 
 ## Add Or Change Local Storage Data
 
-1. Update `packages/web/src/common/storage/adapter/storage.adapter.ts` if the public adapter contract changes.
-2. Update `packages/web/src/common/storage/adapter/indexeddb.adapter.ts`.
+1. Update `packages/web/src/common/storage/offline-data/offline-data.store.ts` if the public store contract changes.
+2. Update `packages/web/src/common/storage/offline-data/indexeddb-offline-data.store.ts`.
 3. Add a migration if existing user data could become invalid.
-4. Add adapter and migration tests.
+4. Add offline data store and migration tests.
 
 ### Common Mistakes
 

docs/development/feature-file-map.md

@@ -40,7 +40,7 @@ Use this document to find the first files to inspect for common Compass changes.
 - Day task hooks: `packages/web/src/views/Day/hooks/tasks`
 - Task UI components: `packages/web/src/views/Day/components/TaskList`
 - Drag handle floating placement guard: `packages/web/src/views/Day/components/Task/DraggableTask.tsx`
-- Local storage for tasks: `packages/web/src/common/storage/adapter`
+- Local storage for tasks: `packages/web/src/common/storage/offline-data`
 
 ## Day / Week Views
 
@@ -68,10 +68,11 @@ Use this document to find the first files to inspect for common Compass changes.
 
 ## Offline Storage
 
-- Adapter singleton and readiness: `packages/web/src/common/storage/adapter/adapter.ts`
-- IndexedDB implementation: `packages/web/src/common/storage/adapter/indexeddb.adapter.ts`
-- Legacy schema migration: `packages/web/src/common/storage/adapter/legacy-primary-key.migration.ts`
+- Offline data store singleton and readiness: `packages/web/src/common/storage/offline-data/offline-data.store.registry.ts`
+- IndexedDB implementation: `packages/web/src/common/storage/offline-data/indexeddb-offline-data.store.ts`
+- Legacy schema migration: `packages/web/src/common/storage/offline-data/legacy-primary-key.migration.ts`
 - Data/external migrations: `packages/web/src/common/storage/migrations`
+- Browser key-value state: `packages/web/src/common/storage/browser-key-value.store.ts`
 
 ## Sync And SSE
 

docs/features/offline-storage-and-migrations.md

@@ -2,29 +2,41 @@
 
 Compass supports a meaningful local-first path for unauthenticated users and resilient fallback behavior for authenticated users.
 
-## Storage Entry Points
+## Storage Boundaries
+
+Compass uses two intentionally separate storage abstractions:
+
+- `OfflineDataStore` owns asynchronous event, task, and migration-record data.
+  Its current implementation is `IndexedDbOfflineDataStore`.
+- `BrowserKeyValueStore` owns synchronous browser key-value state backed by
+  `localStorage` or `sessionStorage`.
 
 Primary files:
 
-- `packages/web/src/common/storage/adapter/adapter.ts`
-- `packages/web/src/common/storage/adapter/storage.adapter.ts`
-- `packages/web/src/common/storage/adapter/indexeddb.adapter.ts`
+- `packages/web/src/common/storage/offline-data/offline-data.store.registry.ts`
+- `packages/web/src/common/storage/offline-data/offline-data.store.ts`
+- `packages/web/src/common/storage/offline-data/indexeddb-offline-data.store.ts`
+- `packages/web/src/common/storage/browser-key-value.store.ts`
 
-The adapter layer exists so application code is not tightly bound to Dexie.
+Components and hooks must use feature-level storage utilities. Native
+`localStorage`, `sessionStorage`, and IndexedDB access belong only in these
+storage implementations, migrations, or black-box test setup.
 
-## Adapter Lifecycle
+## Offline Data Store Lifecycle
 
-`initializeStorage()` does this:
+`initializeOfflineDataStore()` does this:
 
-1. lazily create the storage adapter singleton
+1. lazily create the offline data store singleton
 2. open the underlying IndexedDB database
-3. run adapter-level schema upgrades
+3. run store-level schema upgrades
 4. run app-level data migrations
 5. run app-level external import migrations
 
 The initialization call is idempotent and memoized by `initPromise`.
 
-For tests that instantiate `IndexedDBAdapter` directly, call `adapter.close()` during teardown before deleting the test database. This closes the Dexie connection and resets adapter readiness state for the next test.
+For tests that instantiate `IndexedDbOfflineDataStore` directly, call
+`store.close()` during teardown before deleting the test database. This closes
+the Dexie connection and resets readiness state for the next test.
 
 ## IndexedDB Schema
 
@@ -38,7 +50,7 @@ Current table groups:
 - `tasks`
 - `_migrations`
 
-The IndexedDB adapter keeps:
+The IndexedDB store keeps:
 
 - events keyed by `_id`
 - tasks keyed by `_id` and associated to a `dateKey`
@@ -48,7 +60,7 @@ The IndexedDB adapter keeps:
 
 File:
 
-- `packages/web/src/common/storage/adapter/legacy-primary-key.migration.ts`
+- `packages/web/src/common/storage/offline-data/legacy-primary-key.migration.ts`
 
 There is explicit support for an older task schema that used `id` instead of `_id`.
 
@@ -80,7 +92,7 @@ Current example:
 
 External migrations:
 
-- import data from outside the adapter
+- import data from outside the offline data store
 - are tracked in localStorage, not IndexedDB
 - are non-blocking on failure
 
@@ -131,15 +143,15 @@ Add a migration when you change:
 
 Choose the right mechanism:
 
-- adapter schema/version change for storage structure
+- offline data store schema/version change for storage structure
 - data migration for data already in storage
 - external migration for imports from localStorage or other sources
 
 ## Safe Editing Checklist
 
 Before changing storage behavior:
 
-1. update adapter code
+1. update the relevant storage abstraction
 2. add or adjust migration if existing user data could break
 3. add tests for fresh database and migrated database paths
 4. confirm startup still degrades gracefully when storage fails

docs/frontend/frontend-runtime-flow.md

@@ -176,7 +176,7 @@ The web app uses multiple state layers:
 | Loading states, modal visibility, async status | Redux Toolkit slices | `packages/web/src/ducks/events/slices/` |
 | Async sequences and persistence orchestration | redux-saga | `packages/web/src/ducks/events/sagas/event.sagas.ts` |
 | Event entity CRUD, active event, and draft state | Elf store | `packages/web/src/store/events.ts` |
-| Offline persistence | IndexedDB adapter | `packages/web/src/common/storage/adapter/indexeddb.adapter.ts` |
+| Offline persistence | IndexedDB offline data store | `packages/web/src/common/storage/offline-data/indexeddb-offline-data.store.ts` |
 | Local vs remote persistence choice | Repository factory | `packages/web/src/common/repositories/event/event.repository.util.ts` |
 
 These layers are intentional. Do not collapse event entities into Redux slices
@@ -251,12 +251,12 @@ Revoked state details:
 
 Files:
 
-- `packages/web/src/common/storage/adapter/adapter.ts`
+- `packages/web/src/common/storage/offline-data/offline-data.store.registry.ts`
 - `packages/web/src/common/storage/migrations/migrations.ts`
 
 Startup storage flow:
 
-1. create or reuse the storage adapter singleton
+1. create or reuse the offline data store singleton
 2. open IndexedDB and run internal schema migrations
 3. run data migrations
 4. run external import migrations

packages/web/src/__tests__/utils/storage/indexeddb.test.util.ts

@@ -1,13 +1,13 @@
-import {
-  initializeStorage,
-  resetStorageAsync,
-} from "@web/common/storage/adapter/adapter";
 import { DEMO_DATA_SEED_FLAG_KEY } from "@web/common/storage/migrations/external/demo-data-seed";
+import {
+  initializeOfflineDataStore,
+  resetOfflineDataStoreAsync,
+} from "@web/common/storage/offline-data/offline-data.store.registry";
 
 const COMPASS_LOCAL_DB_NAME = "compass-local";
 
 export async function clearCompassLocalDb(): Promise<void> {
-  await resetStorageAsync();
+  await resetOfflineDataStoreAsync();
   await new Promise<void>((resolve) => {
     const request = indexedDB.deleteDatabase(COMPASS_LOCAL_DB_NAME);
     request.onsuccess = () => resolve();
@@ -25,5 +25,5 @@ export async function prepareEmptyStorageForTests(): Promise<void> {
   localStorage.clear();
   await clearCompassLocalDb();
   localStorage.setItem(DEMO_DATA_SEED_FLAG_KEY, "completed");
-  await initializeStorage();
+  await initializeOfflineDataStore();
 }

…ils/storage/mock-storage-adapter.util.ts …/storage/mock-offline-data-store.util.tspackages/web/src/__tests__/utils/storage/mock-storage-adapter.util.ts renamed to packages/web/src/__tests__/utils/storage/mock-offline-data-store.util.ts packages/web/src/__tests__/utils/storage/mock-storage-adapter.util.ts renamed to packages/web/src/__tests__/utils/storage/mock-offline-data-store.util.ts

@@ -1,21 +1,21 @@
-import { type StorageAdapter } from "@web/common/storage/adapter/storage.adapter";
+import { type OfflineDataStore } from "@web/common/storage/offline-data/offline-data.store";
 import { mock } from "bun:test";
 
 /**
- * Creates a mock StorageAdapter for tests.
+ * Creates a mock OfflineDataStore for tests.
  *
  * All methods default to empty results or no-op implementations.
  * Override specific methods (e.g., getAllEvents, getAllTasks) as needed per test.
  */
-type MockedStorageAdapter = {
-  [K in keyof StorageAdapter]: StorageAdapter[K] extends (
+type MockedOfflineDataStore = {
+  [K in keyof OfflineDataStore]: OfflineDataStore[K] extends (
     ...args: infer _Args
   ) => infer _Return
     ? ReturnType<typeof mock>
-    : StorageAdapter[K];
+    : OfflineDataStore[K];
 };
 
-export function createMockStorageAdapter(): MockedStorageAdapter {
+export function createMockOfflineDataStore(): MockedOfflineDataStore {
   return {
     initialize: mock().mockResolvedValue(undefined),
     isReady: mock().mockReturnValue(true),

packages/web/src/auth/compass/state/auth.state.util.ts

@@ -4,6 +4,7 @@ import {
   DEFAULT_AUTH_STATE,
 } from "@web/common/constants/auth.constants";
 import { STORAGE_KEYS } from "@web/common/constants/storage.constants";
+import { persistentBrowserStore } from "@web/common/storage/browser-key-value.store";
 import { clearGoogleRevokedState } from "../../google/state/google.auth.state";
 
 const authStateListeners = new Set<() => void>();
@@ -44,47 +45,38 @@ function normalizeStoredAuthState(parsed: unknown): AuthState {
 }
 
 /**
- * Get the current authentication state from localStorage.
+ * Get the current authentication state from persistent browser storage.
  * Returns default state if not found or invalid.
  */
 export function getAuthState(): AuthState {
   if (typeof window === "undefined") return DEFAULT_AUTH_STATE;
 
-  try {
-    const stored = localStorage.getItem(STORAGE_KEYS.AUTH);
-    if (stored) {
-      const parsed: unknown = JSON.parse(stored);
-      return normalizeStoredAuthState(parsed);
-    }
+  const stored = persistentBrowserStore.get(STORAGE_KEYS.AUTH);
+  if (!stored) return DEFAULT_AUTH_STATE;
 
-    return DEFAULT_AUTH_STATE;
+  try {
+    return normalizeStoredAuthState(JSON.parse(stored));
   } catch {
     return DEFAULT_AUTH_STATE;
   }
 }
 
 /**
- * Update authentication state in localStorage.
+ * Update authentication state in persistent browser storage.
  * Merges partial updates into existing state.
  */
 export function updateAuthState(updates: Partial<AuthState>): void {
   if (typeof window === "undefined") return;
 
-  try {
-    const current = getAuthState();
-    const updated: AuthState = {
-      ...current,
-      ...updates,
-    };
+  const result = AuthStateSchema.safeParse({ ...getAuthState(), ...updates });
+  if (!result.success) return;
 
-    // Validate with zod schema
-    const result = AuthStateSchema.safeParse(updated);
-    if (result.success) {
-      localStorage.setItem(STORAGE_KEYS.AUTH, JSON.stringify(result.data));
-      emitAuthStateChange();
-    }
-  } catch {
-    // Silently fail if localStorage is unavailable
+  const wasStored = persistentBrowserStore.set(
+    STORAGE_KEYS.AUTH,
+    JSON.stringify(result.data),
+  );
+  if (wasStored) {
+    emitAuthStateChange();
   }
 }
 
@@ -97,15 +89,11 @@ export function updateAuthState(updates: Partial<AuthState>): void {
 export function markUserAsAuthenticated(lastKnownEmail?: string): void {
   if (typeof window === "undefined") return;
 
-  try {
-    updateAuthState({
-      hasAuthenticated: true,
-      ...(lastKnownEmail ? { lastKnownEmail } : {}),
-    });
-    clearGoogleRevokedState();
-  } catch {
-    // Silently fail if localStorage is unavailable
-  }
+  updateAuthState({
+    hasAuthenticated: true,
+    ...(lastKnownEmail ? { lastKnownEmail } : {}),
+  });
+  clearGoogleRevokedState();
 }
 
 /**
@@ -115,19 +103,11 @@ export function markUserAsAuthenticated(lastKnownEmail?: string): void {
  * @returns true if user has previously authenticated
  */
 export function hasUserEverAuthenticated(): boolean {
-  try {
-    return getAuthState().hasAuthenticated;
-  } catch {
-    return false;
-  }
+  return getAuthState().hasAuthenticated;
 }
 
 export function getLastKnownEmail(): string | undefined {
-  try {
-    return getAuthState().lastKnownEmail;
-  } catch {
-    return undefined;
-  }
+  return getAuthState().lastKnownEmail;
 }
 
 /**
@@ -137,11 +117,8 @@ export function getLastKnownEmail(): string | undefined {
 export function clearAuthenticationState(): void {
   if (typeof window === "undefined") return;
 
-  try {
-    localStorage.removeItem(STORAGE_KEYS.AUTH);
+  if (persistentBrowserStore.remove(STORAGE_KEYS.AUTH)) {
     emitAuthStateChange();
-  } catch {
-    // Silently fail if localStorage is unavailable
   }
 }
 
@@ -154,13 +131,7 @@ export function markAnonymousCalendarChangeForSignUpPrompt(): void {
 }
 
 export function shouldShowAnonymousCalendarChangeSignUpPrompt(): boolean {
-  try {
-    return (
-      getAuthState().shouldPromptSignUpAfterAnonymousCalendarChange === true
-    );
-  } catch {
-    return false;
-  }
+  return getAuthState().shouldPromptSignUpAfterAnonymousCalendarChange === true;
 }
 
 export function subscribeToAuthState(listener: () => void): () => void {