Tim Fish

One nextjs-14 e2e test did need a change due to the differing attributes

ah, I need to add some integration tests that ensure that this feature is in fact working and limiting the iitm wrapping

Added some mode context to the description! > I was a bit torn on the option because I kinda thought that it should just be the default behaviour and we...

Shall I mark it as experimental? I'm now thinking the entire `registerEsmLoaderHooks` should have been marked as experimental from the start!

If we're unsure, I can mark this new option as experimental!

The tests that are failing in CI are passing locally 🤦🏻‍♂️. It looks like in CI there are no variables attached to frames at all.

- https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2356

Once this is released I'll open a PR to migrate to the official instrumentation. - https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2375

https://github.com/open-telemetry/opentelemetry-js/issues/4174 https://github.com/DataDog/dd-trace-js/blob/master/packages/datadog-esbuild/index.js

This error isn't unhandled as it has a `.catch()`. You're likely hitting this in the debugger because you have `Pause on caught exceptions` enabled: