Instrument modules loaded via process.getBuiltinModule?

[timfish]

Node has added process.getBuiltinModule(id) to allow synchronous loading of built-in modules:
nodejs/node#52762

The idea is that library authors will now more easily be able to create cross runtime libraries that can conditionally load Node APIs only when running in Node:

if (globalThis.process?.getBuiltinModule) {
  // Running in Node.js, use the Node.js fs module.
  const fs = globalThis.process.getBuiltinModule('fs');
}

OpenTelemetry instruments modules via require-in-the-middle and import-in-the-middle but built-in modules loaded via this new API will not be instrumented as it doesn't go through require.

[github-actions]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[timfish]

Looks like this was added to require-in-the-middle a few weeks ago:
elastic/require-in-the-middle#92

So we just need to update to v7.4.0 and this can be closed?

[timfish]

Ok so we're using a caret for the require-in-the-middle version so if users update their dependencies they will get the latest version.

opentelemetry-js/experimental/packages/opentelemetry-instrumentation/package.json

Line 78 in f047db9

"require-in-the-middle": "^7.1.1",