@reproapp/node-sdk 0.0.26 → 0.0.28

package/dist/ingest/client.js

@@ -40,6 +40,18 @@ const kickIngestQueueDrain = () => {
     }
 };
 exports.kickIngestQueueDrain = kickIngestQueueDrain;
+const isCriticalIngestEntry = (entry) => (entry.requestCaptureManifest !== undefined ||
+    entry.sessionCaptureManifest !== undefined ||
+    entry.request !== undefined ||
+    entry.db !== undefined ||
+    entry.trace !== undefined ||
+    entry.email !== undefined);
+const resolveBatchPriority = (entries, options) => {
+    if (options.priority === 'high') {
+        return 'high';
+    }
+    return entries.some(isCriticalIngestEntry) ? 'high' : 'normal';
+};
 const resolveIngestUrl = (config) => {
     const envBase = typeof process !== 'undefined' ? process?.env?.REPRO_INGEST_BASE : undefined;
     const ingestBase = String(envBase || config.ingestBase || DEFAULT_INGEST_BASE).replace(/\/+$/, '');
@@ -154,7 +166,7 @@ const drainIngestQueue = async () => {
                 continue;
             }
             batch.attempts += 1;
-            if (batch.attempts >= DEFAULT_QUEUE_MAX_ATTEMPTS) {
+            if (batch.attempts >= DEFAULT_QUEUE_MAX_ATTEMPTS && batch.priority !== 'high') {
                 ingestQueue.shift();
                 backoffMs = 0;
                 continue;
@@ -175,7 +187,7 @@ const enqueueIngestEntries = (config, sessionId, entries, options = {}) => {
     if (!sessionId || !Array.isArray(entries) || entries.length === 0) {
         return false;
     }
-    const priority = options.priority === 'high' ? 'high' : 'normal';
+    const priority = resolveBatchPriority(entries, options);
     const batch = {
         config: { ...config },
         sessionId,
@@ -197,13 +209,16 @@ const enqueueIngestEntries = (config, sessionId, entries, options = {}) => {
     }
     if (DEFAULT_QUEUE_MAX_ITEMS > 0 && ingestQueue.length > DEFAULT_QUEUE_MAX_ITEMS) {
         while (ingestQueue.length > DEFAULT_QUEUE_MAX_ITEMS) {
-            let removeIndex = ingestQueue.length - 1;
+            let removeIndex = -1;
             for (let index = ingestQueue.length - 1; index >= 0; index -= 1) {
                 if (ingestQueue[index]?.priority !== 'high') {
                     removeIndex = index;
                     break;
                 }
             }
+            if (removeIndex === -1) {
+                break;
+            }
             ingestQueue.splice(removeIndex, 1);
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reproapp/node-sdk",
-  "version": "0.0.26",
+  "version": "0.0.28",
   "description": "Repro Nest SDK",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -12,7 +12,7 @@
     "build": "tsc -p tsconfig.json",
     "dev": "tsc -p tsconfig.json --watch --preserveWatchOutput",
     "prepublishOnly": "npm run build",
-    "test": "npm run build && node test/unawaited.test.js && node test/integration-unawaited.js && node test/request-flush-timing.test.js && node test/request-capture-background.test.js && node test/client-performance-regression.test.js && node test/request-manifest-priority.test.js && node test/runtime-no-double-invoke.test.js && node test/express-trace-http-args.test.js && node test/trace-batch-size.test.js && node test/sdk-background-priority.test.js && node test/ingest-queue-retention.test.js && node test/ingest-timeout.test.js && node -r ./tracer/register test/promise-map.test.js && node test/disable-subtree.test.js && node test/circular-capture.test.js && node test/wrap-plugin-arrow-args.test.js && node test/wrap-plugin-super-call.test.js && node test/function-arg-identity.test.js && node test/privacy-runtime-policy.test.js && node test/runtime-privacy-materialization.test.js && node test/kafka-runtime-privacy-policy.test.js && node test/amqplib-integration.test.js",
+    "test": "npm run build && node test/unawaited.test.js && node test/integration-unawaited.js && node test/request-flush-timing.test.js && node test/request-capture-background.test.js && node test/client-performance-regression.test.js && node test/request-manifest-priority.test.js && node test/runtime-no-double-invoke.test.js && node test/express-trace-http-args.test.js && node test/trace-batch-size.test.js && node test/sdk-background-priority.test.js && node test/ingest-queue-retention.test.js && node test/ingest-high-priority-retention.test.js && node test/ingest-critical-retention.test.js && node test/ingest-critical-retry-retention.test.js && node test/ingest-timeout.test.js && node -r ./tracer/register test/promise-map.test.js && node test/disable-subtree.test.js && node test/circular-capture.test.js && node test/wrap-plugin-arrow-args.test.js && node test/wrap-plugin-super-call.test.js && node test/function-arg-identity.test.js && node test/privacy-runtime-policy.test.js && node test/runtime-privacy-materialization.test.js && node test/kafka-runtime-privacy-policy.test.js && node test/amqplib-integration.test.js && node test/mongodb-integration.test.js",
     "test:containers": "npm run build && node test/runtime-no-double-invoke.testcontainers.test.js && node test/wrap-plugin-super-call.testcontainers.test.js && node test/function-arg-identity.testcontainers.test.js"
   },
   "peerDependencies": {