cleanup/fix linting errors

Author: Lms24

packages/sveltekit/src/client/index.ts

@@ -1,11 +1,4 @@
 export * from '@sentry/svelte';
 
-// The `withSentryConfig` is exported from the `@sentry/svelte` package, but it has
-// nothing to do with the SvelteKit withSentryConfig. (Bad naming on our part)
-// const { withSentryConfig, ...restOfTheSDK } = SvelteSDK;
-
-// export { withSentryConfig as whatever };
-
-// export  {
-//   ...restOfTheSDK,
-// };
+// Just here so that eslint is happy until we export more stuff here
+export const PLACEHOLDER_SERVER = 'PLACEHOLDER';

packages/sveltekit/src/config/vitePlugins.ts

@@ -1,9 +1,8 @@
-import type { Plugin, TransformResult } from 'vite';
-
+import { logger } from '@sentry/utils';
 import * as fs from 'fs';
-import * as path from 'path';
 import MagicString from 'magic-string';
-import { logger } from '@sentry/utils';
+import * as path from 'path';
+import type { Plugin, TransformResult } from 'vite';
 
 /**
  * This plugin injects the `Sentry.init` calls from `sentry.(client|server).config.(ts|js)`

packages/sveltekit/src/config/withSentryViteConfig.ts

@@ -1,4 +1,5 @@
-import { UserConfig, UserConfigExport } from 'vite';
+import type { UserConfig, UserConfigExport } from 'vite';
+
 import { injectSentryInitPlugin } from './vitePlugins';
 
 /**

packages/sveltekit/src/index.types.ts

@@ -8,7 +8,6 @@ export * from './config';
 export * from './server';
 
 import type { Integration, Options, StackParser } from '@sentry/types';
-import { UserConfig, UserConfigExport } from 'vite';
 
 import type * as clientSdk from './client';
 import type * as serverSdk from './server';

packages/sveltekit/test/config/vitePlugins.test.ts

@@ -1,7 +1,7 @@
-import { injectSentryInitPlugin } from '../../src/config/vitePlugins';
-
 import * as fs from 'fs';
 
+import { injectSentryInitPlugin } from '../../src/config/vitePlugins';
+
 describe('injectSentryInitPlugin', () => {
   it('has its basic properties set', () => {
     expect(injectSentryInitPlugin.name).toBe('sentry-init-injection-plugin');
@@ -19,7 +19,7 @@ describe('injectSentryInitPlugin', () => {
       // @ts-ignore -- transform is definitely defined and callable. Seems like TS doesn't know that.
       const result = injectSentryInitPlugin.transform(code, id);
 
-      expect(result.code).toMatch(/foo\(\);\n.*import \".*sentry\.server\.config\.ts\";/gm);
+      expect(result.code).toMatch(/foo\(\);\n.*import ".*sentry\.server\.config\.ts";/gm);
       expect(result.map).toBeDefined();
     });
 
@@ -30,7 +30,7 @@ describe('injectSentryInitPlugin', () => {
       // @ts-ignore -- transform is definitely defined and callable. Seems like TS doesn't know that.
       const result = injectSentryInitPlugin.transform(code, id);
 
-      expect(result.code).toMatch(/foo\(\);\n.*import \".*sentry\.client\.config\.ts\";/gm);
+      expect(result.code).toMatch(/foo\(\);\n.*import ".*sentry\.client\.config\.ts";/gm);
       expect(result.map).toBeDefined();
     });
 
@@ -41,7 +41,7 @@ describe('injectSentryInitPlugin', () => {
       // @ts-ignore -- transform is definitely defined and callable. Seems like TS doesn't know that.
       const result = injectSentryInitPlugin.transform(code, id);
 
-      expect(result.code).toMatch(/foo\(\);\n.*import \".*sentry\.client\.config\.ts\";/gm);
+      expect(result.code).toMatch(/foo\(\);\n.*import ".*sentry\.client\.config\.ts";/gm);
       expect(result.map).toBeDefined();
     });
 

packages/sveltekit/test/config/withSentryViteConfig.test.ts

@@ -1,4 +1,5 @@
-import { UserConfig, Plugin } from 'vite';
+import type { Plugin, UserConfig } from 'vite';
+
 import { withSentryViteConfig } from '../../src/config/withSentryViteConfig';
 
 describe('withSentryViteConfig', () => {