guard for double import

Author: nicohrubec

packages/tanstackstart-react/src/vite/autoInstrumentMiddleware.ts

@@ -166,6 +166,11 @@ export function arrayToObjectShorthand(contents: string): string | null {
 export function addSentryImport(code: string): string {
   const sentryImport = "import { wrapMiddlewaresWithSentry } from '@sentry/tanstackstart-react';\n";
 
+  // Don't add the import if it already exists
+  if (code.includes(sentryImport.trimEnd())) {
+    return code;
+  }
+
   // Check for 'use server' or 'use client' directives, these need to be before any imports
   const directiveMatch = code.match(/^(['"])use (client|server)\1;?\s*\n?/);
 

packages/tanstackstart-react/test/vite/autoInstrumentMiddleware.test.ts

@@ -352,6 +352,16 @@ describe('addSentryImport', () => {
     expect(result).toMatch(/^"use client";\nimport \{ wrapMiddlewaresWithSentry \}/);
     expect(result).toContain('const foo = 1;');
   });
+
+  it('does not add import if it already exists', () => {
+    const code = "import { wrapMiddlewaresWithSentry } from '@sentry/tanstackstart-react';\nconst foo = 1;";
+    const result = addSentryImport(code);
+
+    expect(result).toBe(code);
+    // Verify the import appears exactly once
+    const importCount = (result.match(/import \{ wrapMiddlewaresWithSentry \}/g) || []).length;
+    expect(importCount).toBe(1);
+  });
 });
 
 describe('arrayToObjectShorthand', () => {