cleanup and e2e test

Author: Lms24

dev-packages/e2e-tests/test-applications/sveltekit-2/package.json

@@ -23,7 +23,7 @@
     "@sentry/core": "latest || *",
     "@sveltejs/adapter-auto": "^3.0.0",
     "@sveltejs/adapter-node": "^2.0.0",
-    "@sveltejs/kit": "^2.5.0",
+    "@sveltejs/kit": "^2.16.0",
     "@sveltejs/vite-plugin-svelte": "^3.0.0",
     "svelte": "^4.2.8",
     "svelte-check": "^3.6.0",

dev-packages/e2e-tests/test-applications/sveltekit-2/tests/sdk.test.ts

@@ -0,0 +1,10 @@
+import { expect, test } from '@playwright/test';
+import { waitForInitialPageload } from './utils';
+
+test.describe('SDK-internal behavior', () => {
+  test("Doesn't inject fetch proxy script for SvelteKit>=2.16.0", async ({ page }) => {
+    await waitForInitialPageload(page, { route: '/client-error' });
+    const proxyHandle = await page.waitForFunction(() => globalThis._sentryFetchProxy);
+    expect(proxyHandle).toBeUndefined();
+  });
+});

packages/sveltekit/package.json

@@ -7,7 +7,7 @@
   "author": "Sentry",
   "license": "MIT",
   "engines": {
-    "node": ">=16"
+    "node": ">=18"
   },
   "files": ["/build"],
   "main": "build/cjs/index.server.js",

packages/sveltekit/src/server/handle.ts

@@ -14,7 +14,7 @@ import {
   winterCGRequestToRequestData,
   withIsolationScope,
 } from '@sentry/core';
-import { type Handle, type ResolveOptions } from '@sveltejs/kit';
+import type { Handle, ResolveOptions } from '@sveltejs/kit';
 
 import { DEBUG_BUILD } from '../common/debug-build';
 import { flushIfServerless, getTracePropagationData, sendErrorToSentry } from './utils';

packages/sveltekit/src/vite/sentryVitePlugins.ts

@@ -26,11 +26,6 @@ export async function sentrySvelteKit(options: SentrySvelteKitPluginOptions = {}
     adapter: options.adapter || (await detectAdapter(options.debug)),
   };
 
-  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-  // @ts-ignore - just testing for now
-  const kit = await import('@sveltejs/kit');
-  console.log('XX kit version', kit.VERSION);
-
   const sentryPlugins: Plugin[] = [];
 
   if (mergedOptions.autoInstrument) {