fix: default storage setItem (#4472)

.changeset/nine-carrots-reply.md

@@ -0,0 +1,5 @@
+---
+"@wagmi/core": patch
+---
+
+Added handling to default storage for `setItem` errors, like `QuotaExceededError`, `SecurityError`, etc.

packages/core/src/createConfig.ts

@@ -21,7 +21,11 @@ import type {
 } from './connectors/createConnector.js'
 import { injected } from './connectors/injected.js'
 import { type Emitter, type EventData, createEmitter } from './createEmitter.js'
-import { type Storage, createStorage, noopStorage } from './createStorage.js'
+import {
+  type Storage,
+  createStorage,
+  getDefaultStorage,
+} from './createStorage.js'
 import { ChainNotConfiguredError } from './errors/config.js'
 import type {
   Compute,
@@ -43,10 +47,7 @@ export function createConfig<
   const {
     multiInjectedProviderDiscovery = true,
     storage = createStorage({
-      storage:
-        typeof window !== 'undefined' && window.localStorage
-          ? window.localStorage
-          : noopStorage,
+      storage: getDefaultStorage(),
     }),
     syncConnectedChain = true,
     ssr = false,

packages/core/src/createStorage.ts

@@ -88,3 +88,20 @@ export const noopStorage = {
   setItem: () => {},
   removeItem: () => {},
 } satisfies BaseStorage
+
+export function getDefaultStorage() {
+  const storage = (() => {
+    if (typeof window !== 'undefined' && window.localStorage)
+      return window.localStorage
+    return noopStorage
+  })()
+  return {
+    ...storage,
+    setItem(key, value) {
+      try {
+        storage.setItem(key, value)
+        // silence errors by default (QuotaExceededError, SecurityError, etc.)
+      } catch {}
+    },
+  } satisfies BaseStorage
+}