Types from `fastify` `devDependency` should not be imported in `index.d.ts`

[onurtemizkan]

Prerequisites

• I have written a descriptive issue title
• I have searched existing issues to ensure the bug has not already been reported

Fastify version

5

Plugin version

0.5.0

Node.js version

20.18.2

Operating system

macOS

Operating system version (i.e. 20.04, 11.3, 10)

15.3.2

Description

We bumped into this while trying to migrate Sentry's Node SDK to use this plugin. getsentry/sentry-javascript#15542

As fastify is declared as a devDependency of this plugin, it's not guaranteed to be available every time this package @fastify/otel is depended on.

This should be ok when @fastify/otel is used directly inside a fastify application, as it's available regardless.

But, to be able to ship @fastify/otel as a part of Sentry's NodeJS SDK, we need to depend on @fastify/otel like the other OpenTelemetry plugins. This breaks the usage of the SDK, where fastify is unavailable.

Link to code that reproduces the bug

CI-run: https://github.com/getsentry/sentry-javascript/actions/runs/14088564105/job/39459459303

Expected Behavior

No response

[metcoder95]

Thanks for the report, nice catch!

Would you like to send a PR for it?

[onurtemizkan]

Sure, my suggestion is to shallowly inline the required types and remove the dependency altogether, does that make sense?

[metcoder95]

Sure, that should be all right; let's just add tests to the index.test-d.ts file to ensure it works as expected 👍