refactor: use isWrapped from instrumentation package

[pichlermarc]

Which problem is this PR solving?

For @opentelemetry/[email protected] we'll remove isWrapped() since it's duplicated in @opentelemetry/instrumentation. It makes more sense to have it in @opentelemetry/instrumentation since that's actually an instrumentation-related utility function.

Refs open-telemetry/opentelemetry-js#5444

Short description of the changes

Uses isWrapped() from @opentelemetry/instrumentation instead. It is a drop-in replacement as the implementations are identical:

• from core: https://github.com/open-telemetry/opentelemetry-js/blob/4553b29d4a04b5b7e4bf87cad64dc2fc8c740d8e/packages/opentelemetry-core/src/utils/wrap.ts#L19-L30
• from instrumentation: https://github.com/open-telemetry/opentelemetry-js/blob/fdab6422ae238b04beb2e92ac38153e35e5800ac/experimental/packages/opentelemetry-instrumentation/src/utils.ts#L72-L83

ShimWrapped is also fully compatible:

• from core: https://github.com/open-telemetry/opentelemetry-js/blob/4a6021af7a8fd8a912d221cab3d96b40a0b0433c/packages/opentelemetry-core/src/common/types.ts#L30-L40
• from instrumentation: https://github.com/open-telemetry/opentelemetry-js/blob/d2812d2810a812f069f7d40fb9c6c870e30fb8e0/experimental/packages/opentelemetry-instrumentation/src/types.ts#L66-L76

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.40%. Comparing base (2989b94) to head (9af54bb).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2704   +/-   ##
=======================================
  Coverage   92.40%   92.40%           
=======================================
  Files         171      171           
  Lines        8142     8142           
  Branches     1647     1647           
=======================================
  Hits         7524     7524           
  Misses        618      618           

Files with missing lines	Coverage Δ
...lugin-react-load/src/BaseOpenTelemetryComponent.ts	97.04% <100.00%> (ø)

[trentm]

Hrm, this should have failed npm run lint:deps. It does on main now:

[10:36:37 trentm@peach:~/src/opentelemetry-js-contrib (git:main)]
% npm run lint:deps

> [email protected] lint:deps
> npx --yes [email protected] --dependencies --production --tags=-knipignore

Unused dependencies (2)
@opentelemetry/otlp-transformer  packages/opentelemetry-test-utils/package.json
@opentelemetry/core              plugins/web/opentelemetry-plugin-react-load/package.json
Unlisted dependencies (1)
@opentelemetry/instrumentation  plugins/web/opentelemetry-plugin-react-load/src/BaseOpenTelemetryComponent.ts

[10:36:46 trentm@peach:~/src/opentelemetry-js-contrib (git:main rv:1)]
% echo $?
1

plugins/web/opentelemetry-plugin-react-load needs to change its dep to be on instrumentation instead of core.

[trentm]

Ah, lint:deps was accidentally removed from linting in #2493

https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2493/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L26-R24

Likely from a merge: that PR and adding lint:deps were both within days of each other.

[trentm]

#2709 added to fix that up.