@reproapp/node-sdk 0.0.7 → 0.0.8

package/dist/index.js

@@ -1929,6 +1929,50 @@ function sanitizeTraceValue(value, depth = 0, seen = new WeakMap(), options = {}
     const mongoId = coerceMongoId(value);
     if (mongoId !== null)
         return mongoId;
+    if (isHttpRequestLike(value)) {
+        const projected = {
+            __kind: 'http-request',
+        };
+        if (typeof value.method === 'string') {
+            projected.method = value.method;
+        }
+        const url = typeof value.originalUrl === 'string'
+            ? value.originalUrl
+            : typeof value.url === 'string'
+                ? value.url
+                : undefined;
+        if (url) {
+            projected.url = url;
+        }
+        const headers = sanitizeHeaders(value.headers, true);
+        if (headers !== undefined) {
+            projected.headers = sanitizeTraceValue(headers, depth + 1, seen, options, childCapturePath(valuePath, 'headers'));
+        }
+        if (value.params !== undefined) {
+            projected.params = sanitizeTraceValue(value.params, depth + 1, seen, options, childCapturePath(valuePath, 'params'));
+        }
+        if (value.query !== undefined) {
+            projected.query = sanitizeTraceValue(value.query, depth + 1, seen, options, childCapturePath(valuePath, 'query'));
+        }
+        if (value.body !== undefined) {
+            projected.body = sanitizeTraceValue(value.body, depth + 1, seen, options, childCapturePath(valuePath, 'body'));
+        }
+        return projected;
+    }
+    if (isHttpResponseLike(value)) {
+        const projected = {
+            __kind: 'http-response',
+            statusCode: Number(value.statusCode) || 0,
+        };
+        const rawHeaders = typeof value.getHeaders === 'function'
+            ? value.getHeaders()
+            : value._headers;
+        const headers = sanitizeHeaders(rawHeaders, true);
+        if (headers !== undefined) {
+            projected.headers = sanitizeTraceValue(headers, depth + 1, seen, options, childCapturePath(valuePath, 'headers'));
+        }
+        return projected;
+    }
     if (isMongooseQueryLike(value)) {
         const captured = value.__repro_result;
         if (captured !== undefined) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reproapp/node-sdk",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "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 -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/privacy-runtime-policy.test.js && node test/runtime-privacy-materialization.test.js && node test/kafka-runtime-privacy-policy.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/express-trace-http-args.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/privacy-runtime-policy.test.js && node test/runtime-privacy-materialization.test.js && node test/kafka-runtime-privacy-policy.test.js"
   },
   "peerDependencies": {
     "express": "^5.1.0",

package/src/index.ts

@@ -2318,6 +2318,55 @@ function sanitizeTraceValue(
     const mongoId = coerceMongoId(value);
     if (mongoId !== null) return mongoId;
 
+    if (isHttpRequestLike(value)) {
+        const projected: Record<string, any> = {
+            __kind: 'http-request',
+        };
+        if (typeof (value as any).method === 'string') {
+            projected.method = (value as any).method;
+        }
+        const url =
+            typeof (value as any).originalUrl === 'string'
+                ? (value as any).originalUrl
+                : typeof (value as any).url === 'string'
+                    ? (value as any).url
+                    : undefined;
+        if (url) {
+            projected.url = url;
+        }
+
+        const headers = sanitizeHeaders((value as any).headers, true);
+        if (headers !== undefined) {
+            projected.headers = sanitizeTraceValue(headers, depth + 1, seen, options, childCapturePath(valuePath, 'headers'));
+        }
+        if ((value as any).params !== undefined) {
+            projected.params = sanitizeTraceValue((value as any).params, depth + 1, seen, options, childCapturePath(valuePath, 'params'));
+        }
+        if ((value as any).query !== undefined) {
+            projected.query = sanitizeTraceValue((value as any).query, depth + 1, seen, options, childCapturePath(valuePath, 'query'));
+        }
+        if ((value as any).body !== undefined) {
+            projected.body = sanitizeTraceValue((value as any).body, depth + 1, seen, options, childCapturePath(valuePath, 'body'));
+        }
+        return projected;
+    }
+
+    if (isHttpResponseLike(value)) {
+        const projected: Record<string, any> = {
+            __kind: 'http-response',
+            statusCode: Number((value as any).statusCode) || 0,
+        };
+        const rawHeaders =
+            typeof (value as any).getHeaders === 'function'
+                ? (value as any).getHeaders()
+                : (value as any)._headers;
+        const headers = sanitizeHeaders(rawHeaders, true);
+        if (headers !== undefined) {
+            projected.headers = sanitizeTraceValue(headers, depth + 1, seen, options, childCapturePath(valuePath, 'headers'));
+        }
+        return projected;
+    }
+
     if (isMongooseQueryLike(value)) {
         const captured = (value as any).__repro_result;
         if (captured !== undefined) {

package/test/express-trace-http-args.test.js

@@ -0,0 +1,126 @@
+const assert = require('assert');
+const http = require('http');
+
+const originalFetch = global.fetch;
+const capturedBodies = [];
+
+global.fetch = async (url, init = {}) => {
+    const target = String(url || '');
+    if (!target.includes('/v1/ingest/events')) {
+        throw new Error(`unexpected fetch target: ${target}`);
+    }
+    capturedBodies.push({
+        at: Date.now(),
+        url: target,
+        body: JSON.parse(String(init.body || '{}')),
+    });
+    return {
+        ok: true,
+        status: 200,
+        json: async () => ({ ok: true }),
+        text: async () => '{"ok":true}',
+    };
+};
+
+const { initRepro } = require('../dist');
+const { flushIngestQueue } = require('../dist/ingest/client');
+
+function findEvents(eventType) {
+    return capturedBodies.flatMap((entry) => Array.isArray(entry.body?.events) ? entry.body.events : [])
+        .filter((event) => event?.event_type === eventType);
+}
+
+function sendGet(url) {
+    return new Promise((resolve, reject) => {
+        const request = http.get(url, (response) => {
+            let text = '';
+            response.setEncoding('utf8');
+            response.on('data', (chunk) => {
+                text += chunk;
+            });
+            response.on('end', () => resolve({
+                statusCode: response.statusCode,
+                body: text,
+            }));
+        });
+        request.on('error', reject);
+    });
+}
+
+async function main() {
+    await initRepro({
+        tenantId: 'TENANT_express_trace_http_args',
+        appId: 'APP_express_trace_http_args',
+        appSecret: 'secret',
+        appName: 'express-trace-http-args',
+        serviceName: 'express-trace-http-args',
+        ingestBase: 'http://127.0.0.1:65535',
+        tracing: {
+            disableFunctionTypes: ['constructor'],
+            logFunctionCalls: false,
+        },
+    });
+
+    const { startServer } = require('./fixtures/express-trace-http-args-server');
+    const server = await startServer({
+        tenantId: 'TENANT_express_trace_http_args',
+        appId: 'APP_express_trace_http_args',
+        appSecret: 'secret',
+        appName: 'express-trace-http-args',
+        serviceName: 'express-trace-http-args',
+        ingestBase: 'http://127.0.0.1:65535',
+    });
+
+    try {
+        const port = server.address().port;
+        const requestUrl = `http://127.0.0.1:${port}/ping?name=Avery&__repro_sid=S_express_trace_http_args&__repro_aid=A_express_trace_http_args&__repro_start=${Date.now()}`;
+        const response = await sendGet(requestUrl);
+        assert.equal(response.statusCode, 200);
+
+        await new Promise((resolve) => setTimeout(resolve, 2500));
+        await flushIngestQueue();
+
+        const requestEvents = findEvents('backend_request');
+        const traceEvents = findEvents('trace_batch');
+
+        assert.equal(requestEvents.length, 1, JSON.stringify(capturedBodies));
+        assert.equal(traceEvents.length, 1, JSON.stringify(capturedBodies));
+
+        const batch = traceEvents[0]?.payload?.trace;
+        assert(Array.isArray(batch) && batch.length > 0, JSON.stringify(traceEvents[0]));
+
+        const handlePingEnter = batch.find((event) => event?.type === 'enter' && event?.fn === 'handlePing');
+        assert(handlePingEnter, JSON.stringify(batch));
+        assert(Array.isArray(handlePingEnter.args), JSON.stringify(handlePingEnter));
+        assert.equal(handlePingEnter.args[0]?.__kind, 'http-request', JSON.stringify(handlePingEnter.args[0]));
+        assert.equal(handlePingEnter.args[1]?.__kind, 'http-response', JSON.stringify(handlePingEnter.args[1]));
+        assert.equal(handlePingEnter.args[0]?.url, '/ping?name=Avery');
+        assert.equal(handlePingEnter.args[0]?.query?.name, 'Avery');
+
+        const serializedBatch = JSON.stringify(traceEvents[0]);
+        assert(serializedBatch.length < 250000, `trace batch still too large: ${serializedBatch.length}`);
+
+        // eslint-disable-next-line no-console
+        console.log('express trace http arg projection OK');
+    } finally {
+        await new Promise((resolve, reject) => {
+            server.close((error) => {
+                if (!error || error.code === 'ERR_SERVER_NOT_RUNNING') {
+                    resolve();
+                    return;
+                }
+                reject(error);
+            });
+        });
+    }
+}
+
+main()
+    .catch((error) => {
+        // eslint-disable-next-line no-console
+        console.error(error);
+        process.exitCode = 1;
+    })
+    .finally(() => {
+        global.fetch = originalFetch;
+    });

package/test/fixtures/express-trace-http-args-controller.js

@@ -0,0 +1,29 @@
+async function buildPayload(name) {
+    const normalized = normalizeName(name);
+    const emphasized = emphasize(normalized);
+    return {
+        original: name,
+        normalized,
+        emphasized,
+    };
+}
+
+function normalizeName(name) {
+    return String(name || 'anonymous').trim().toLowerCase();
+}
+
+function emphasize(name) {
+    return `${name.toUpperCase()}!`;
+}
+
+async function handlePing(req, res) {
+    const payload = await buildPayload(req.query?.name || 'Avery Debugson');
+    res.json({
+        ok: true,
+        payload,
+    });
+}
+
+module.exports = {
+    handlePing,
+};

package/test/fixtures/express-trace-http-args-server.js

@@ -0,0 +1,21 @@
+const express = require('express');
+const { reproMiddleware } = require('../../dist');
+const { handlePing } = require('./express-trace-http-args-controller');
+
+async function startServer(cfg) {
+    const app = express();
+    app.use(reproMiddleware(cfg));
+    app.get('/ping', handlePing);
+
+    const server = await new Promise((resolve, reject) => {
+        const instance = app.listen(0);
+        instance.once('listening', () => resolve(instance));
+        instance.once('error', reject);
+    });
+
+    return server;
+}
+
+module.exports = {
+    startServer,
+};