fix: fixed review comments

- fix breadcrumbStore field TSDoc in catcher
- destroy BrowserBreadcrumbStore instance before each tests

Author: Reversean

packages/javascript/src/catcher.ts

@@ -122,7 +122,7 @@ export default class Catcher {
   private readonly consoleCatcher: ConsoleCatcher | null = null;
 
   /**
-   * Breadcrumb manager instance
+   * Breadcrumb store instance
    */
   private readonly breadcrumbStore: BrowserBreadcrumbStore | null;
 

packages/javascript/tests/breadcrumbs.test.ts

@@ -5,7 +5,7 @@ import * as core from '@hawk.so/core';
 
 function resetManager(): void {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  (BrowserBreadcrumbStore as any).instance = null;
+  (BrowserBreadcrumbStore as any).instance?.destroy();
 }
 
 describe('BrowserBreadcrumbStore', () => {

packages/javascript/tests/catcher.addons.test.ts

@@ -11,7 +11,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── Environment addons ────────────────────────────────────────────────────

packages/javascript/tests/catcher.breadcrumbs.test.ts

@@ -11,7 +11,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── Breadcrumbs trail ─────────────────────────────────────────────────────

packages/javascript/tests/catcher.context.test.ts

@@ -11,7 +11,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── Context enrichment ────────────────────────────────────────────────────

packages/javascript/tests/catcher.global-handlers.test.ts

@@ -12,7 +12,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── Global error handlers ─────────────────────────────────────────────────

packages/javascript/tests/catcher.test.ts

@@ -17,7 +17,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── Constructor variants ──────────────────────────────────────────────────

packages/javascript/tests/catcher.transport.test.ts

@@ -12,7 +12,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   describe('transport failure', () => {

packages/javascript/tests/catcher.user.test.ts

@@ -11,7 +11,7 @@ describe('Catcher', () => {
   beforeEach(() => {
     localStorage.clear();
     mockParse.mockResolvedValue([]);
-    (BrowserBreadcrumbStore as any).instance = null;
+    (BrowserBreadcrumbStore as any).instance?.destroy();
   });
 
   // ── User identity ─────────────────────────────────────────────────────────