fix(state): address code quality review comments on PR #3286

- remove unused ProfileManager import in app-react-state-replication/App.tsx
- remove unused AppModulesInitializedEvent import in AppConfigurator.ts
- remove unused 'type' from event destructuring in StateProvider.ts
- export PouchDbSyncStorage class
- add revs_limit and size options to PouchDbStorageOptions

Author: alftore

cookbooks/app-react-state-replication/src/App.tsx

@@ -1,4 +1,3 @@
-import { ProfileManager } from './components/ProfileManager';
 import Router from './Router';
 
 /**

packages/app/src/AppConfigurator.ts

@@ -16,8 +16,8 @@ import http, {
 
 import auth from '@equinor/fusion-framework-module-msal';
 
-import type { AppEnv, AppModules, AppModulesInstance } from './types';
-import { AppModulesConfiguredEvent, AppModulesInitializedEvent } from './events';
+import type { AppEnv, AppModules } from './types';
+import { AppModulesConfiguredEvent } from './events';
 import { AppConfiguratorError } from './error';
 import { deepClone, deepFreeze, type DeepImmutable } from './utils';
 

packages/modules/state/src/StateProvider.ts

@@ -334,7 +334,6 @@ function applyStateChangeEvents<T extends AllowedValue = AllowedValue>(
       // Accumulate changes using scan operator, starting with the initial array
       scan((acc, event) => {
         const {
-          type,
           detail: { key, item },
         } = event;
 

packages/modules/state/src/storage/PouchDbStorage.ts

@@ -29,6 +29,17 @@ const STORAGE_KEY_SEPARATOR = '::';
  */
 export type PouchDbStorageOptions = PouchDB.Configuration.DatabaseConfiguration & {
   key_prefix?: string;
+  /**
+   * Number of old revisions to keep for each document.
+   * Lower values reduce storage usage at the cost of revision history.
+   */
+  revs_limit?: number;
+  /**
+   * Maximum local database size in megabytes.
+   * Important for Safari (IndexedDB quota enforcement).
+   * Valid values: 10, 50, 100, 500, 1000.
+   */
+  size?: number;
 };
 
 /**
@@ -554,7 +565,7 @@ export class PouchDbStorage implements IStorage, Disposable {
 
       // Try to get existing document for update, fallback to creation template
       // This pattern handles both create and update operations seamlessly
-      const existingDoc = await this.#db.get(key).catch((e) => {
+      const existingDoc = await this.#db.get(key).catch((_e) => {
         // Document doesn't exist - create template for new document
         // _rev: undefined tells PouchDB this is a new document
         return { _id: key, _rev: undefined };

packages/modules/state/src/storage/PouchDbSyncStorage.ts

@@ -15,6 +15,7 @@ type PouchDbSyncStorageOptions = {
   syncOptions: PouchDB.Replication.SyncOptions;
 };
 
+// biome-ignore lint/correctness/noUnusedVariables: intentionally internal — not yet tested or integrated into the public API
 class PouchDbSyncStorage extends PouchDbStorage {
   #remoteDb: PouchDB.Database;
   #syncOptions: PouchDB.Replication.SyncOptions;