Mongoose v7 not actually supported by @opentelemetry/instrumentation-mongoose

[mkohlmyr]

opentelemetry-js-contrib/plugins/node/instrumentation-mongoose/src/mongoose.ts

Lines 137 to 141 in 0fc1ed3

	this._wrap(
	moduleExports.Aggregate.prototype,
	'exec',
	this.patchAggregateExec(moduleVersion)
	);

Mongoose v7 does not appear to export .Aggregate which results in Cannot read properties of undefined (reading 'prototype').

The documentation states:

Supported Versions
mongoose versions >=5.9.7 <9

If this is intended to be the case, then wrapping of Aggregate will need to be optional or version dependent similar to instrumentRemove.

Would be happy to provide a patch if you agree this is the case, but might need to wait a few days as I first have to patch this internally somehow for our Sentry integration.

[mkohlmyr]

It actually looks like this is an issue either with ESM imports, or specifically with how @sentry/node loads the plugin, as I cannot reproduce it right now when using common js directly to load the SDK. Meanwhile loading via @sentry/node using ESM encounters issues patching Mongoose v5 & v7, presumably also v6.

[excalq]

I'm running into this same issue, in an ESM node application. I noticed that upgrading to the latest 7.8.6 does not work, but Mongoose v8 (8.9.5) works fine. Unfortunately a shared internal NPM package of DB models makes upgrading less feasible, so we're affected by this for the time being.