test: clear local storage before each test (#15053)

## Summary

Child of #15052. Give every test a clean localStorage baseline through
the shared Vitest setup.

```ts
beforeEach(() => {
  globalThis.localStorage?.clear()
})
```

The optional access keeps the shared setup compatible with the website
and object-parser projects, which use Vitest's Node environment and do
not expose localStorage.

## Audit results

| Classification | Calls | Outcome |
|---|---:|---|
| Test-boundary setup/teardown | 50 | Removed as globally redundant |
| Intra-test fast-check run isolation | 1 | Retained |
| **Total audited** | **51** | **30 test files** |

The retained call separates generated command sequences and shrink
attempts inside a single Vitest test. A global `beforeEach` cannot
replace it.

The cleanup also removes empty hooks, stale comments, and unused
lifecycle imports. Net diff: 10 additions and 106 deletions across 31
files.

## Validation

| Project | Test files | Passed | Skipped | Failed |
|---|---:|---:|---:|---:|
| Root | 1,159 | 15,896 | 8 | 0 |
| Desktop UI | 16 | 125 | 0 | 0 |
| Website | 41 | 387 | 0 | 0 |
| Object info parser | 4 | 33 | 0 | 0 |

Also passed:

- `pnpm typecheck`
- `pnpm lint` (six pre-existing warnings, zero errors)
- oxfmt and type-aware Oxlint on all changed files
- pre-push `pnpm knip --cache`

Co-authored-by: Amp <amp@ampcode.com>

Author: DrJKL

src/components/TopMenuSection.test.ts

@@ -196,7 +196,6 @@ function createComfyActionbarStub(actionbarTarget: HTMLElement) {
 
 describe('TopMenuSection', () => {
   beforeEach(() => {
-    localStorage.clear()
     mockData.isLoggedIn = false
     mockData.setShowConflictRedDot(false)
   })

src/components/actionbar/ComfyActionbar.test.ts

@@ -63,7 +63,6 @@ const renderActionbar = (showRunProgressBar: boolean) => {
 describe('ComfyActionbar', () => {
   beforeEach(() => {
     i18n.global.locale.value = 'en'
-    localStorage.clear()
   })
 
   it('teleports inline progress when run progress bar is enabled', async () => {

src/composables/maskeditor/useBrushPersistence.test.ts

@@ -28,7 +28,6 @@ const STORAGE_KEY = 'maskeditor_brush_settings'
 
 beforeEach(() => {
   setActivePinia(createTestingPinia({ stubActions: false }))
-  localStorage.clear()
 })
 
 describe('loadAndApply', () => {

src/composables/useFeatureFlags.test.ts

@@ -235,10 +235,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('dev override via localStorage', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('resolveFlag returns localStorage override over remoteConfig and server value', () => {
       vi.mocked(api.getServerFeature).mockReturnValue(false)
       localStorage.setItem('ff:model_upload_button_enabled', 'true')
@@ -301,7 +297,6 @@ describe('useFeatureFlags', () => {
       remoteConfig.value = {}
       cachedBillingControlEnabled.value = undefined
       cachedV1PaymentRecovery.value = undefined
-      localStorage.clear()
     })
 
     afterEach(() => {
@@ -310,7 +305,6 @@ describe('useFeatureFlags', () => {
       remoteConfig.value = {}
       cachedBillingControlEnabled.value = undefined
       cachedV1PaymentRecovery.value = undefined
-      localStorage.clear()
     })
 
     it('returns the cached session value during the auth window', () => {
@@ -359,10 +353,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('signupTurnstileMode', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('falls back to the server feature flag with default off', () => {
       vi.mocked(api.getServerFeature).mockImplementation(
         (path, defaultValue) => {
@@ -436,7 +426,6 @@ describe('useFeatureFlags', () => {
     afterEach(() => {
       vi.mocked(distributionTypes).isCloud = false
       remoteConfig.value = {}
-      localStorage.clear()
     })
 
     it('is disabled outside the cloud distribution', () => {
@@ -466,10 +455,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('unifiedCloudAuthEnabled', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('reads the unified_cloud_auth server feature when set', () => {
       vi.mocked(api.getServerFeature).mockImplementation(
         (path, defaultValue) => {
@@ -497,7 +482,6 @@ describe('useFeatureFlags', () => {
       vi.mocked(distributionTypes).isCloud = false
       remoteConfigState.value = 'unloaded'
       cachedBillingControlEnabled.value = undefined
-      localStorage.clear()
       remoteConfig.value = {}
     })
 

src/composables/useRunButtonTelemetry.test.ts

@@ -45,7 +45,6 @@ import {
 
 describe('useRunButtonTelemetry', () => {
   beforeEach(() => {
-    localStorage.clear()
     state.telemetry.trackRunButton.mockClear()
     state.mode.value = 'graph'
     state.isAppMode.value = false

src/platform/cloud/subscription/components/CreditsTile.test.ts

@@ -209,7 +209,6 @@ describe('CreditsTile', () => {
     state.type = 'workspace'
     state.customerEventsError = null
     mockIsCloud.value = true
-    localStorage.clear()
   })
 
   it('renders the total balance (cents converted to credits) with the remaining suffix', () => {

src/platform/surveys/ErrorPanelSurveyCta.test.ts

@@ -1,6 +1,6 @@
 import { render, screen, within } from '@testing-library/vue'
 import userEvent from '@testing-library/user-event'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { nextTick, ref } from 'vue'
 
 const FEATURE_USAGE_KEY = 'Comfy.FeatureUsage'
@@ -82,7 +82,6 @@ describe('ErrorPanelSurveyCta', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
     vi.resetModules()
     mockOpen.mockReset()
 
@@ -97,10 +96,6 @@ describe('ErrorPanelSurveyCta', () => {
     }
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   async function renderComponent() {
     const { default: ErrorPanelSurveyCta } =
       await import('./ErrorPanelSurveyCta.vue')

src/platform/surveys/NightlySurveyPopover.test.ts

@@ -1,6 +1,6 @@
 import { render, screen } from '@testing-library/vue'
 import userEvent from '@testing-library/user-event'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { nextTick } from 'vue'
 import { createI18n } from 'vue-i18n'
 
@@ -44,18 +44,13 @@ describe('NightlySurveyPopover', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
     vi.resetModules()
 
     mockIsNightly.value = true
     mockIsCloud.value = false
     mockIsDesktop.value = false
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   async function renderComponent(
     config = defaultConfig,
     eventHandlers: Record<string, ReturnType<typeof vi.fn>> = {}

src/platform/surveys/useFeatureUsageTracker.test.ts

@@ -1,18 +1,10 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { useFeatureUsageTracker } from './useFeatureUsageTracker'
 
 const STORAGE_KEY = 'Comfy.FeatureUsage'
 
 describe('useFeatureUsageTracker', () => {
-  beforeEach(() => {
-    localStorage.clear()
-  })
-
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   it('initializes with zero count for new feature', () => {
     const { useCount } = useFeatureUsageTracker('test-feature-1')
 

src/platform/surveys/useSurveyEligibility.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { useSurveyEligibility } from './useSurveyEligibility'
 
@@ -30,17 +30,11 @@ describe('useSurveyEligibility', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
-
     mockDistribution.isNightly = true
     mockDistribution.isCloud = false
     mockDistribution.isDesktop = false
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   function setFeatureUsage(featureId: string, useCount: number) {
     const existing = JSON.parse(localStorage.getItem(FEATURE_USAGE_KEY) ?? '{}')
     existing[featureId] = {