sdk core: load debugIds on DebugIdProvider construction

Author: perf2711

packages/sdk-core/src/modules/data/BacktraceDataBuilder.ts

@@ -24,7 +24,6 @@ export class BacktraceDataBuilder {
 
         const stackTrace = this._stackTraceConverter.convert(report.stackTrace, report.message);
 
-        this._debugIdProvider.loadDebugIds();
         for (const frame of stackTrace) {
             frame.debug_identifier = this._debugIdProvider.getDebugId(frame.library);
         }

packages/sdk-core/src/sourcemaps/DebugIdProvider.ts

@@ -4,23 +4,23 @@ import { DebugIdMapProvider } from './interfaces/DebugIdMapProvider';
 export const SOURCE_DEBUG_ID_VARIABLE = '_btDebugIds';
 
 export class DebugIdProvider {
-    private _fileDebugIds?: Record<string, string>;
+    private readonly _fileDebugIds: Record<string, string>;
 
     constructor(
         private readonly _stackTraceConverter: BacktraceStackTraceConverter,
         private readonly _debugIdMapProvider?: DebugIdMapProvider,
-    ) {}
+    ) {
+        this._fileDebugIds = this.loadDebugIds();
+    }
 
-    public loadDebugIds(debugIdMap?: Record<string, string>) {
-        if (this._fileDebugIds) {
-            return this._fileDebugIds;
-        }
+    public getDebugId(file: string): string | undefined {
+        return this._fileDebugIds[file];
+    }
 
+    private loadDebugIds() {
+        const debugIdMap = this._debugIdMapProvider?.getDebugIdMap();
         if (!debugIdMap) {
-            debugIdMap = this._debugIdMapProvider?.getDebugIdMap();
-            if (!debugIdMap) {
-                return;
-            }
+            return {};
         }
 
         const message = new Error().message;
@@ -36,10 +36,6 @@ export class DebugIdProvider {
             result[frame.library] = debugId;
         }
 
-        return (this._fileDebugIds = result);
-    }
-
-    public getDebugId(file: string): string | undefined {
-        return this._fileDebugIds?.[file];
+        return result;
     }
 }