[Bug] Remove peer dependencies from @temporalio/interceptors-opentelemetry

[TastyPi]

What are you really trying to do?

Depend on @temporalio/interceptors-opentelemetry in my app that uses @temporalio/client.

Describe the bug

When I run yarn install I get the following messages:

➤ YN0002: │ my-package doesn't provide @temporalio/activity (p98051a), requested by @temporalio/interceptors-opentelemetry.
➤ YN0002: │ my-package doesn't provide @temporalio/common (pd7eebd), requested by @temporalio/interceptors-opentelemetry.
➤ YN0002: │ my-package doesn't provide @temporalio/worker (p41c8fb), requested by @temporalio/interceptors-opentelemetry.
➤ YN0002: │ my-package doesn't provide @temporalio/workflow (pd62775), requested by @temporalio/interceptors-opentelemetry.

YN0002 is caused by missing peer dependencies. To clear these messages, I would have to add dependencies on those other libraries. Except it does not make sense for my app that only uses @temporalio/client to depend on those other libraries (except maybe @temporalio/common).

The proper fix would be to not have these peer dependencies, or split @temporalio/interceptors-opentelemetry into separate packages for clients/workflows/activities.

Minimal Reproduction

Create an app that depends on @temporalio/interceptors-opentelemetry with Yarn.

Environment/Versions

• OS and processor: Linux
• Temporal SDK Version: 1.13.0
• Yarn version: 4.9.4

Additional context

[mjameswh]

No, those dependencies should not get removed, or users would get other kind of errors. And splitting each of these to different packages would make proper configuration of this much more difficult on users.

The correct thing to do would be to add a peerDependenciesMeta.*.optional: true entry for all @temporalio/* dependencies. That should be pretty easy. The tricky thing, however, is that we'll need to make sure that the code is prepared to detect missing packages at run time, and handle these gracefully.