@reproapp/node-sdk 0.0.32 → 0.0.33

package/dist/index.js

@@ -869,6 +869,18 @@ function releaseNativeMongoSuppression(target) {
         catch { }
     }
 }
+function isMongooseInternalNativeMongoCall() {
+    let stack = '';
+    try {
+        stack = new Error().stack ?? '';
+    }
+    catch {
+        stack = '';
+    }
+    return stack.includes('/node_modules/mongoose/') ||
+        stack.includes('\\node_modules\\mongoose\\') ||
+        stack.includes('mongoose/lib/');
+}
 function isNativeMongoSuppressed(collection, op) {
     const suppressions = getCtx().nativeMongoSuppressions;
     if (!Array.isArray(suppressions) || suppressions.length === 0)
@@ -876,6 +888,7 @@ function isNativeMongoSuppressed(collection, op) {
     const normalizedCollection = collection == null ? null : String(collection);
     const normalizedOp = normalizeNativeMongoSuppressionOp(op);
     const now = Date.now();
+    const matchingTokens = [];
     for (let i = suppressions.length - 1; i >= 0; i -= 1) {
         const token = suppressions[i];
         if (!token || now - token.startedAt > 60000) {
@@ -886,9 +899,17 @@ function isNativeMongoSuppressed(collection, op) {
             continue;
         if (token.ops && token.ops.size > 0 && !token.ops.has(normalizedOp))
             continue;
-        return true;
+        matchingTokens.push(token);
     }
-    return false;
+    if (matchingTokens.length === 0)
+        return false;
+    if (!isMongooseInternalNativeMongoCall()) {
+        return false;
+    }
+    for (const token of matchingTokens) {
+        popNativeMongoSuppression(token);
+    }
+    return true;
 }
 function currentClockSkewMs() {
     const store = als.getStore();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reproapp/node-sdk",
-  "version": "0.0.32",
+  "version": "0.0.33",
   "description": "Repro Nest SDK",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -13,7 +13,7 @@
     "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/init-auto-http-server.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 && node test/mongodb-esm-import-hook.test.js && node test/mongodb-suppression-cleanup.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"
+    "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 && node test/mongodb-mongoose-native-suppression.testcontainers.test.js"
   },
   "peerDependencies": {
     "express": "^5.1.0",

package/src/index.ts

@@ -1267,12 +1267,25 @@ function releaseNativeMongoSuppression(target: any): void {
     }
 }
 
+function isMongooseInternalNativeMongoCall(): boolean {
+    let stack = '';
+    try {
+        stack = new Error().stack ?? '';
+    } catch {
+        stack = '';
+    }
+    return stack.includes('/node_modules/mongoose/') ||
+        stack.includes('\\node_modules\\mongoose\\') ||
+        stack.includes('mongoose/lib/');
+}
+
 function isNativeMongoSuppressed(collection: any, op: string): boolean {
     const suppressions = (getCtx() as Ctx).nativeMongoSuppressions;
     if (!Array.isArray(suppressions) || suppressions.length === 0) return false;
     const normalizedCollection = collection == null ? null : String(collection);
     const normalizedOp = normalizeNativeMongoSuppressionOp(op);
     const now = Date.now();
+    const matchingTokens: NativeMongoSuppression[] = [];
     for (let i = suppressions.length - 1; i >= 0; i -= 1) {
         const token = suppressions[i];
         if (!token || now - token.startedAt > 60000) {
@@ -1281,9 +1294,18 @@ function isNativeMongoSuppressed(collection: any, op: string): boolean {
         }
         if (token.collection && normalizedCollection && token.collection !== normalizedCollection) continue;
         if (token.ops && token.ops.size > 0 && !token.ops.has(normalizedOp)) continue;
-        return true;
+        matchingTokens.push(token);
     }
-    return false;
+    if (matchingTokens.length === 0) return false;
+
+    if (!isMongooseInternalNativeMongoCall()) {
+        return false;
+    }
+
+    for (const token of matchingTokens) {
+        popNativeMongoSuppression(token);
+    }
+    return true;
 }
 
 function currentClockSkewMs(): number {

package/test/mongodb-integration.test.js

@@ -8,6 +8,7 @@ process.env.TRACE_LINGER_AFTER_FINISH_MS = '0';
 const assert = require('assert');
 const { EventEmitter } = require('events');
 const mongodb = require('mongodb');
+const mongoose = require('mongoose');
 
 const originalNativeMongoInsertOne = mongodb.Collection.prototype.insertOne;
 
@@ -18,7 +19,7 @@ mongodb.Collection.prototype.insertOne = async function reproFakeNativeMongoInse
 };
 
 const { flushIngestQueue } = require('../dist/ingest/client');
-const { reproMiddleware, __reproTestHooks } = require('../dist');
+const { reproMiddleware, reproMongoosePlugin, __reproTestHooks } = require('../dist');
 
 const sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
 
@@ -126,12 +127,80 @@ async function runCapturedRequest(cfg, sessionId, actionId, fn) {
     res.end();
 }
 
+function execPre(hooks, name, context) {
+    return new Promise((resolve, reject) => {
+        hooks.execPre(name, context, [], (error) => (error ? reject(error) : resolve()));
+    });
+}
+
 function dbChanges(events) {
     return events
         .filter((event) => event.event_type === 'db_change')
         .flatMap((event) => Array.isArray(event.payload?.db) ? event.payload.db : []);
 }
 
+async function testMongooseSuppressionDoesNotHideExplicitNativeInsertOne() {
+    await withCapturedFetch(async (capturedBodies) => {
+        __reproTestHooks.resetSdkBackgroundQueuesForTest();
+        const sessionId = 'S_mongo_mixed_driver_insert';
+        const actionId = 'A_mongo_mixed_driver_insert';
+        const cfg = makeConfig(sessionId, actionId);
+
+        const schema = new mongoose.Schema({ subjectNumber: String, status: String });
+        reproMongoosePlugin(cfg)(schema);
+        const Model = mongoose.model(`ReproMixedNativeMongo${Date.now()}`, schema, 'subjects');
+        const pendingMongooseDoc = new Model({ subjectNumber: 'pre-save', status: 'pending' });
+
+        class FakeCollection {
+            constructor() {
+                this.collectionName = 'subjects';
+                this.dbName = 'clinical';
+            }
+
+            async insertOne(doc, options) {
+                assert.strictEqual(options.session.id, 'native-explicit-session-1');
+                doc._id = doc._id ?? 'subject-explicit-1';
+                return { acknowledged: true, insertedId: doc._id };
+            }
+        }
+
+        __reproTestHooks.patchMongoCollectionPrototypeForTest(FakeCollection.prototype, cfg);
+        const subjectsCollection = new FakeCollection();
+
+        await runCapturedRequest(cfg, sessionId, actionId, async () => {
+            await execPre(schema.s.hooks, 'save', pendingMongooseDoc);
+            assert.strictEqual(__reproTestHooks.getNativeMongoSuppressionCountForTest(), 1);
+
+            const result = await subjectsCollection.insertOne(
+                { subjectNumber: '10013038', status: 'screened' },
+                {
+                    session: {
+                        id: 'native-explicit-session-1',
+                        hasEnded: false,
+                        inTransaction: () => true,
+                    },
+                },
+            );
+            assert.deepStrictEqual(result, { acknowledged: true, insertedId: 'subject-explicit-1' });
+        });
+
+        await waitFor(() => dbChanges(flattenEvents(capturedBodies)).length === 1);
+        await drainSdk();
+
+        const [change] = dbChanges(flattenEvents(capturedBodies));
+        assert.strictEqual(change.type, 'mongo');
+        assert.strictEqual(change.collection, 'subjects');
+        assert.strictEqual(change.op, 'insertOne');
+        assert.strictEqual(change.pk._id, 'subject-explicit-1');
+        assert.strictEqual(change.query.driver, 'mongodb');
+        assert.strictEqual(change.query.db, 'clinical');
+        assert.strictEqual(change.query.doc.subjectNumber, '10013038');
+        assert.strictEqual(change.query.options.session.__kind, 'mongodb_client_session');
+        assert.strictEqual(change.query.options.session.id, 'native-explicit-session-1');
+        assert.strictEqual(change.after.subjectNumber, '10013038');
+    });
+}
+
 async function testReproMiddlewarePatchesPreloadedNativeMongoClientInsertOne() {
     await withCapturedFetch(async (capturedBodies) => {
         __reproTestHooks.resetSdkBackgroundQueuesForTest();
@@ -452,6 +521,7 @@ async function testNativeOperationFailureIsCapturedAndRethrown() {
 }
 
 (async () => {
+    await testMongooseSuppressionDoesNotHideExplicitNativeInsertOne();
     await testReproMiddlewarePatchesPreloadedNativeMongoClientInsertOne();
     await testMongoClientDbCollectionChainIsPatchedWithoutCollectionExport();
     await testNativeInsertOneIsCaptured();

package/test/mongodb-mongoose-native-suppression.testcontainers.test.js

@@ -0,0 +1,158 @@
+process.env.REPRO_INGEST_REQUEST_TIMEOUT_MS = '50';
+process.env.REPRO_SDK_BACKGROUND_MAX_DEFER_MS = '25';
+process.env.REPRO_SDK_BACKGROUND_IDLE_DELAY_MS = '0';
+process.env.REPRO_SDK_BACKGROUND_BUSY_IDLE_DELAY_MS = '0';
+process.env.TRACE_IDLE_FLUSH_MS = '10';
+process.env.TRACE_LINGER_AFTER_FINISH_MS = '0';
+
+const assert = require('assert');
+const { EventEmitter } = require('events');
+const mongoose = require('mongoose');
+const { GenericContainer, Wait } = require('testcontainers');
+
+const { flushIngestQueue } = require('../dist/ingest/client');
+const { reproMiddleware, reproMongoosePlugin, __reproTestHooks } = require('../dist');
+
+const sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+
+async function waitFor(predicate, timeoutMs = 5000) {
+    const deadline = Date.now() + timeoutMs;
+    while (Date.now() < deadline) {
+        if (predicate()) return;
+        await sleep(25);
+    }
+    assert(predicate(), 'timed out waiting for expected ingest payload');
+}
+
+function flattenEvents(posts) {
+    return posts.flatMap((body) => Array.isArray(body?.events) ? body.events : []);
+}
+
+function dbChanges(events) {
+    return events
+        .filter((event) => event.event_type === 'db_change')
+        .flatMap((event) => Array.isArray(event.payload?.db) ? event.payload.db : []);
+}
+
+function makeReqRes(sessionId, actionId) {
+    const req = new EventEmitter();
+    req.method = 'POST';
+    req.url = '/subjects';
+    req.originalUrl = '/subjects';
+    req.headers = {
+        'x-bug-session-id': sessionId,
+        'x-bug-action-id': actionId,
+    };
+    req.query = {};
+    req.params = {};
+
+    const res = new EventEmitter();
+    res.statusCode = 200;
+    res.getHeader = () => undefined;
+    res.getHeaders = () => ({});
+    res.write = () => true;
+    res.end = function end() {
+        this.emit('finish');
+        return this;
+    };
+
+    return { req, res };
+}
+
+async function runCapturedRequest(cfg, sessionId, actionId, fn) {
+    const middleware = reproMiddleware(cfg);
+    const { req, res } = makeReqRes(sessionId, actionId);
+    await new Promise((resolve, reject) => {
+        middleware(req, res, (err) => {
+            if (err) {
+                reject(err);
+                return;
+            }
+            Promise.resolve().then(fn).then(resolve, reject);
+        });
+    });
+    res.end();
+}
+
+async function startMongo() {
+    if (process.env.REPRO_MONGO_TEST_URI) {
+        return {
+            uri: process.env.REPRO_MONGO_TEST_URI,
+            stop: async () => {},
+        };
+    }
+
+    const container = await new GenericContainer('mongo:7')
+        .withExposedPorts(27017)
+        .withTmpFs({ '/data/db': 'rw,noexec,nosuid,size=1g' })
+        .withCommand(['mongod', '--bind_ip_all', '--setParameter', 'diagnosticDataCollectionEnabled=false'])
+        .withWaitStrategy(Wait.forListeningPorts())
+        .start();
+    return {
+        uri: `mongodb://${container.getHost()}:${container.getMappedPort(27017)}/repro_mongoose_native_suppression`,
+        stop: () => container.stop(),
+    };
+}
+
+async function main() {
+    const capturedBodies = [];
+    const originalFetch = global.fetch;
+    global.fetch = async (_url, init) => {
+        capturedBodies.push(JSON.parse(String(init?.body ?? '{}')));
+        return {
+            ok: true,
+            status: 202,
+            json: async () => ({ ok: true }),
+            text: async () => '{"ok":true}',
+        };
+    };
+
+    let container;
+    let connection;
+    try {
+        __reproTestHooks.resetSdkBackgroundQueuesForTest();
+        container = await startMongo();
+        connection = await mongoose.createConnection(container.uri).asPromise();
+
+        const cfg = {
+            tenantId: 'TENANT_test',
+            appId: 'APP_test',
+            appSecret: 'secret',
+            captureHeaders: false,
+            privacy: { environment: 'dev', captureRaw: true },
+            ingestBase: 'http://127.0.0.1:65535',
+        };
+
+        const schema = new mongoose.Schema({ subjectNumber: String, status: String });
+        reproMongoosePlugin(cfg)(schema);
+        const Subject = connection.model('Subject', schema, 'subjects');
+
+        await runCapturedRequest(cfg, 'S_mongoose_native_suppression', 'A_mongoose_native_suppression', async () => {
+            await Subject.create({ subjectNumber: '10013038', status: 'screened' });
+        });
+
+        await waitFor(() => dbChanges(flattenEvents(capturedBodies)).length === 1);
+        await __reproTestHooks.drainSdkBackgroundQueueForTest();
+        await flushIngestQueue();
+
+        const changes = dbChanges(flattenEvents(capturedBodies));
+        assert.strictEqual(changes.length, 1);
+        assert.strictEqual(changes[0].type, 'mongo');
+        assert.strictEqual(changes[0].collection, 'subjects');
+        assert.strictEqual(changes[0].op, 'insert');
+        assert.strictEqual(changes[0].after.subjectNumber, '10013038');
+
+        console.log('mongodb mongoose native suppression testcontainers OK');
+    } finally {
+        await __reproTestHooks.drainSdkBackgroundQueueForTest().catch(() => {});
+        await flushIngestQueue().catch(() => {});
+        global.fetch = originalFetch;
+        if (connection) await connection.close().catch(() => {});
+        if (container) await container.stop().catch(() => {});
+    }
+}
+
+main().catch((error) => {
+    console.error(error);
+    process.exit(1);
+});