leave configuration meta uncompressed

Author: leoromanovsky

local-install.sh

@@ -0,0 +1,46 @@
+#!/usr/bin/env bash
+set -eo pipefail
+
+## Example:
+##
+## ./scripts/local-install.sh ./js-client-sdk-common ./react-native-sdk && ./scripts/local-install.sh ./react-native-sdk ../TestReactNative0720
+##
+
+if [[ "$1" = "" || "$2" = "" ]]; then
+  echo 'USAGE: ./local-install.sh <package-to-install-path> <consuming-app-path>'
+  ehco ''
+  echo ''
+  exit 1
+fi
+
+if [ ! -d "$1" ]; then
+  echo "$1 is not a directory"
+  exit 1
+fi
+
+if [ ! -f "$1/package.json" ]; then
+  echo "$1 is not an npm package"
+  exit 1
+fi
+
+if [ ! -d "$2" ]; then
+  echo "$2 is not a directory"
+    exit 1
+fi
+
+### Create local package via "npm pack"
+pushd "$1" > /dev/null
+rm -rf package pack.out pack.err && npm pack 2> pack.err > pack.out
+COMPRESSED_PACKAGE="$(tail -1 pack.out)"
+tar -xzf "${COMPRESSED_PACKAGE}"
+rm "${COMPRESSED_PACKAGE}"
+PACKAGE_DIR="$(pwd)/package"
+popd > /dev/null
+
+### Install local package to target package
+pushd "$2" > /dev/null
+TARGET_DIR="$(pwd)"
+yarn add "${PACKAGE_DIR}" --exact
+popd > /dev/null
+
+echo "$PACKAGE_DIR installed in $TARGET_DIR"

src/local-storage-engine.spec.ts

@@ -178,21 +178,21 @@ describe('LocalStorageEngine Compression Migration', () => {
       consoleSpy.mockRestore();
     });
 
-    it('should compress and decompress meta data', async () => {
+    it('should store and retrieve meta data without compression', async () => {
       const metaData = JSON.stringify({ lastUpdated: Date.now() });
 
       await engine.setMetaJsonString(metaData);
 
+      // Meta data should be stored uncompressed
       expect(mockLocalStorage.setItem).toHaveBeenCalledWith(
         'eppo-configuration-meta-test',
-        LZString.compress(metaData),
+        metaData,
       );
 
       // Test reading back
-      const compressedMeta = LZString.compress(metaData);
       (mockLocalStorage.getItem as jest.Mock).mockImplementation((key) => {
         if (key === 'eppo-meta') return JSON.stringify({ version: 1, migratedAt: Date.now() });
-        if (key === 'eppo-configuration-meta-test') return compressedMeta;
+        if (key === 'eppo-configuration-meta-test') return metaData;
         return null;
       });
 

src/local-storage-engine.ts

@@ -49,16 +49,7 @@ export class LocalStorageEngine implements IStringStorageEngine {
   };
 
   public getMetaJsonString = async (): Promise<string | null> => {
-    const stored = this.localStorage.getItem(this.metaKey);
-    if (!stored) return null;
-
-    try {
-      return LZString.decompress(stored) || null;
-    } catch (e) {
-      console.warn('Failed to decompress meta, removing corrupted data');
-      this.localStorage.removeItem(this.metaKey);
-      return null;
-    }
+    return this.localStorage.getItem(this.metaKey);
   };
 
   public setContentsJsonString = async (configurationJsonString: string): Promise<void> => {
@@ -67,8 +58,7 @@ export class LocalStorageEngine implements IStringStorageEngine {
   };
 
   public setMetaJsonString = async (metaJsonString: string): Promise<void> => {
-    const compressed = LZString.compress(metaJsonString);
-    this.localStorage.setItem(this.metaKey, compressed);
+    this.localStorage.setItem(this.metaKey, metaJsonString);
   };
 
   private ensureCompressionMigration(): void {
@@ -78,10 +68,6 @@ export class LocalStorageEngine implements IStringStorageEngine {
       return; // Already migrated
     }
 
-    console.log(
-      `Running storage migration from v${globalMeta.version} to v${LocalStorageEngine.MIGRATION_VERSION}`,
-    );
-
     try {
       this.deleteAllConfigurations();
 
@@ -90,9 +76,8 @@ export class LocalStorageEngine implements IStringStorageEngine {
         version: LocalStorageEngine.MIGRATION_VERSION,
       });
 
-      console.log('Configuration cleanup completed - fresh configs will be compressed');
     } catch (e) {
-      console.warn('Migration failed:', e);
+      // Migration failed, continue silently
     }
   }
 
@@ -112,7 +97,7 @@ export class LocalStorageEngine implements IStringStorageEngine {
       this.localStorage.removeItem(key);
     });
 
-    console.log(`Deleted ${keysToDelete.length} old configuration keys.`);
+    // Deleted old configuration keys for compression migration
   }
 
   private getGlobalMeta(): EppoGlobalMeta {
@@ -122,7 +107,7 @@ export class LocalStorageEngine implements IStringStorageEngine {
         return JSON.parse(stored);
       }
     } catch (e) {
-      console.warn('Failed to parse global meta:', e);
+      // Failed to parse global meta, will use default
     }
 
     return { version: 0 }; // Default to version 0