fix: Prevent type errors when optional `@opentelemetry/api` dependency isn't installe...

* fix: Prevent type errors when optional `@opentelemetry/api` dependency isn't installed

Fixes #14774
Better alternative to #14848

* Apply suggestion from @teemingc

---------

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

diff --git a/.changeset/flat-steaks-yawn.md b/.changeset/flat-steaks-yawn.md
new file mode 100644 (file)
index 0000000..0a2e323
--- /dev/null
+++ b/.changeset/flat-steaks-yawn.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+fix: prevent type errors when optional `@opentelemetry/api` dependency isn't installed

diff --git a/packages/kit/src/exports/public.d.ts b/packages/kit/src/exports/public.d.ts
index 39174438f805b887580e5ed4d25093486b23531c..21ed3e6cda0006c08a5c150a40afb94ed1921d69 100644 (file)
--- a/packages/kit/src/exports/public.d.ts
+++ b/packages/kit/src/exports/public.d.ts
@@ -25,10 +25,12 @@ import {
        LayoutParams as AppLayoutParams,
        ResolvedPathname
 } from '$app/types';
-import { Span } from '@opentelemetry/api';
 
 export { PrerenderOption } from '../types/private.js';
 
+// @ts-ignore this is an optional peer dependency so could be missing. Written like this so dts-buddy preserves the ts-ignore
+type Span = import('@opentelemetry/api').Span;
+
 /**
  * [Adapters](https://svelte.dev/docs/kit/adapters) are responsible for taking the production build and turning it into something that can be deployed to a platform of your choosing.
  */

diff --git a/packages/kit/types/index.d.ts b/packages/kit/types/index.d.ts
index 548bc7a78d83addfb37cf145243c95425549b35d..35b387405f53fcf56aaf9e865e780f4971075fda 100644 (file)
--- a/packages/kit/types/index.d.ts
+++ b/packages/kit/types/index.d.ts
@@ -5,7 +5,9 @@ declare module '@sveltejs/kit' {
        import type { SvelteConfig } from '@sveltejs/vite-plugin-svelte';
        import type { StandardSchemaV1 } from '@standard-schema/spec';
        import type { RouteId as AppRouteId, LayoutParams as AppLayoutParams, ResolvedPathname } from '$app/types';
-       import type { Span } from '@opentelemetry/api';
+       // @ts-ignore this is an optional peer dependency so could be missing. Written like this so dts-buddy preserves the ts-ignore
+       type Span = import('@opentelemetry/api').Span;
+
        /**
         * [Adapters](https://svelte.dev/docs/kit/adapters) are responsible for taking the production build and turning it into something that can be deployed to a platform of your choosing.
         */