fix(browser-sdk): lower bulk queue failure logging

Author: roncohen

packages/browser-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reflag/browser-sdk",
-  "version": "1.4.6",
+  "version": "1.4.7",
   "packageManager": "yarn@4.1.1",
   "license": "MIT",
   "repository": {

packages/browser-sdk/src/bulkQueue.ts

@@ -151,7 +151,7 @@ export class BulkQueue {
     try {
       res = await this.sendBulk(batch);
     } catch (error) {
-      this.logger?.error("bulk request failed; dropping batch", {
+      this.logger?.debug("bulk request failed; dropping batch", {
         error,
         batchSize: batch.length,
       });
@@ -190,7 +190,7 @@ export class BulkQueue {
       !this.lastDropErrorAt ||
       now - this.lastDropErrorAt >= DROP_ERROR_THROTTLE_MS
     ) {
-      this.logger?.error("bulk queue dropped events due to max size", {
+      this.logger?.debug("bulk queue dropped events due to max size", {
         droppedEvents: this.droppedSinceLastError,
         totalDroppedEvents: this.totalDroppedEvents,
         queueSize: this.queue.length + this.getInFlightBatchSize(),

packages/browser-sdk/test/bulkQueue.test.ts

@@ -82,13 +82,14 @@ describe("BulkQueue", () => {
     await vi.advanceTimersByTimeAsync(10);
     expect(sendBulk).toHaveBeenCalledTimes(1);
     expect(await queue.size()).toBe(0);
-    expect(logger.error).toHaveBeenCalledWith(
+    expect(logger.debug).toHaveBeenCalledWith(
       "bulk request failed; dropping batch",
       expect.objectContaining({
         error: expect.any(Error),
         batchSize: 1,
       }),
     );
+    expect(logger.error).not.toHaveBeenCalled();
   });
 
   it("drops 4xx responses, logs error, and does not retry", async () => {
@@ -237,9 +238,16 @@ describe("BulkQueue", () => {
             resolveSend = resolve;
           }),
       );
+    const logger = {
+      debug: vi.fn(),
+      info: vi.fn(),
+      warn: vi.fn(),
+      error: vi.fn(),
+    };
     const queue = new BulkQueue(sendBulk, {
       flushDelayMs: 10_000,
       maxSize: 2,
+      logger,
     });
 
     await queue.enqueue(userEvent);
@@ -249,6 +257,16 @@ describe("BulkQueue", () => {
     expect(await queue.size()).toBe(2);
     expect(sendBulk).toHaveBeenCalledTimes(1);
     expect(sendBulk).toHaveBeenCalledWith([userEvent, companyEvent]);
+    expect(logger.debug).toHaveBeenCalledWith(
+      "bulk queue dropped events due to max size",
+      expect.objectContaining({
+        droppedEvents: 1,
+        totalDroppedEvents: 1,
+        queueSize: 2,
+        maxSize: 2,
+      }),
+    );
+    expect(logger.error).not.toHaveBeenCalled();
 
     resolveSend?.(new Response("", { status: 200 }));
   });

packages/openfeature-browser-provider/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reflag/openfeature-browser-provider",
-  "version": "1.3.4",
+  "version": "1.3.5",
   "packageManager": "yarn@4.1.1",
   "license": "MIT",
   "repository": {
@@ -35,7 +35,7 @@
     }
   },
   "dependencies": {
-    "@reflag/browser-sdk": "1.4.6"
+    "@reflag/browser-sdk": "1.4.7"
   },
   "devDependencies": {
     "@openfeature/core": "1.5.0",

packages/react-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reflag/react-sdk",
-  "version": "1.4.6",
+  "version": "1.4.7",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -37,7 +37,7 @@
     }
   },
   "dependencies": {
-    "@reflag/browser-sdk": "1.4.6"
+    "@reflag/browser-sdk": "1.4.7"
   },
   "peerDependencies": {
     "react": "*",

packages/vue-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reflag/vue-sdk",
-  "version": "1.3.4",
+  "version": "1.3.5",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -35,7 +35,7 @@
     }
   },
   "dependencies": {
-    "@reflag/browser-sdk": "1.4.6"
+    "@reflag/browser-sdk": "1.4.7"
   },
   "peerDependencies": {
     "vue": "^3.0.0"