@reproapp/node-sdk 0.0.36 → 0.0.37

package/dist/index.js

@@ -330,7 +330,7 @@ let disabledFunctionTraceRules = computeDisabledFunctionTraceRules();
 let disabledFunctionTypePatterns = [];
 let disabledTraceFilePatterns = [];
 let __TRACE_LOG_PREF = null;
-let __FULL_VALUE_CAPTURE_ENABLED = false;
+let __FULL_VALUE_CAPTURE_ENABLED = true;
 function setInterceptorTracingEnabled(enabled) {
     const next = !!enabled;
     if (interceptorTracingEnabled === next)
@@ -581,9 +581,9 @@ function defaultTracerInitOpts() {
 /** Call this from the client app to enable tracing. Safe to call multiple times. */
 function initReproTracing(opts) {
     const options = opts ?? {};
-    if (hasOwn(options, 'fullValueCapture')) {
-        __FULL_VALUE_CAPTURE_ENABLED = options.fullValueCapture === true;
-    }
+    __FULL_VALUE_CAPTURE_ENABLED = hasOwn(options, 'fullValueCapture')
+        ? options.fullValueCapture === true
+        : true;
     if (hasOwn(options, 'traceInterceptors')) {
         setInterceptorTracingEnabled(!!options.traceInterceptors);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reproapp/node-sdk",
-  "version": "0.0.36",
+  "version": "0.0.37",
   "description": "Repro Nest SDK",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/index.ts

@@ -604,7 +604,7 @@ let disabledFunctionTraceRules: DisableFunctionTraceConfig[] = computeDisabledFu
 let disabledFunctionTypePatterns: Array<string | number | RegExp> = [];
 let disabledTraceFilePatterns: Array<string | number | RegExp> = [];
 let __TRACE_LOG_PREF: boolean | null = null;
-let __FULL_VALUE_CAPTURE_ENABLED = false;
+let __FULL_VALUE_CAPTURE_ENABLED = true;
 
 function setInterceptorTracingEnabled(enabled: boolean) {
     const next = !!enabled;
@@ -929,9 +929,9 @@ function defaultTracerInitOpts(): TracerInitOpts {
 export function initReproTracing(opts?: ReproTracingInitOptions) {
     const options = opts ?? {};
 
-    if (hasOwn(options, 'fullValueCapture')) {
-        __FULL_VALUE_CAPTURE_ENABLED = options.fullValueCapture === true;
-    }
+    __FULL_VALUE_CAPTURE_ENABLED = hasOwn(options, 'fullValueCapture')
+        ? options.fullValueCapture === true
+        : true;
 
     if (hasOwn(options, 'traceInterceptors')) {
         setInterceptorTracingEnabled(!!options.traceInterceptors);

package/test/client-performance-regression.test.js

@@ -170,10 +170,28 @@ async function main() {
             event?.returnValue,
         );
         assert(largeReturnExit, JSON.stringify(traceEvents));
-        assert.deepEqual(
-            largeReturnExit.returnValue,
-            { __kind: 'omitted', reason: 'inline-size-limit' },
+        assert(
+            Buffer.byteLength(JSON.stringify(largeReturnExit.returnValue), 'utf8') <= 2048,
+            JSON.stringify(largeReturnExit.returnValue),
         );
+        assert.equal(
+            largeReturnExit.returnValueCapture?.status,
+            'stored',
+            JSON.stringify(largeReturnExit),
+        );
+        assert(
+            typeof largeReturnExit.returnValueCapture?.valueId === 'string' &&
+                largeReturnExit.returnValueCapture.valueId.length > 0,
+            JSON.stringify(largeReturnExit),
+        );
+        const traceValues = eventsOf(capturedBodies, 'trace_batch')
+            .flatMap((event) => Array.isArray(event?.payload?.traceValues) ? event.payload.traceValues : []);
+        const capturedReturnValue = traceValues.find((entry) =>
+            entry?.id === largeReturnExit.returnValueCapture.valueId &&
+            entry?.target === 'trace.returnValue'
+        );
+        assert(capturedReturnValue, JSON.stringify(traceValues));
+        assert.equal(capturedReturnValue.value?.rows?.length, 400, JSON.stringify(capturedReturnValue));
 
         console.log('client performance regression guards OK');
     } finally {