fix(browser): add cache for system info getter

HuolalaTech · Dec 4, 2024 · a841840 · a841840
1 parent 0003107
commit a841840
Showing 1 changed file with 16 additions and 9 deletions.
diff --git a/packages/page-spy-browser/src/plugins/system/index.ts b/packages/page-spy-browser/src/plugins/system/index.ts
@@ -72,17 +72,24 @@ export default class SystemPlugin implements PageSpyPlugin {
     SystemPlugin.hasInitd = false;
   }
 
+  private _cache: SpySystem.DataItem | null = null;
+
   public async getSystemInfo() {
-    const features = await computeResult();
-    const info = {
-      system: {
-        ua: navigator.userAgent,
-      },
-      features,
-    } as SpySystem.DataItem;
-    const processedByUser = this.$pageSpyConfig?.dataProcessor?.system?.(info);
+    if (!this._cache) {
+      const features = await computeResult();
+      this._cache = {
+        system: {
+          ua: navigator.userAgent,
+        },
+        features,
+      } as SpySystem.DataItem;
+    }
+
+    const processedByUser = this.$pageSpyConfig?.dataProcessor?.system?.(
+      this._cache,
+    );
 
     if (processedByUser === false) return null;
-    return makeMessage('system', info);
+    return makeMessage('system', this._cache);
   }
 }