fix: prevent Sentry initialization during test execution

Resolves CI test failures caused by 'module is already linked' errors
when running tests with the updated Sentry v10.5.0 dependency.

* Add test environment detection in sentry.ts and server.ts
* Skip Sentry.init() and tag setting during test execution
* Return base MCP server without Sentry wrapping in test mode
* Maintain full Sentry functionality in production environments

This ensures test isolation while preserving MCP instrumentation
capabilities for production deployments.

Author: cameroncooke

src/server/server.ts

@@ -43,13 +43,22 @@ export function createServer(): McpServer {
     },
   );
 
-  // Wrap server with Sentry for MCP instrumentation
-  const server = Sentry.wrapMcpServerWithSentry(baseServer);
+  // Check if we're in a test environment to avoid Sentry module conflicts
+  const isTestEnvironment =
+    process.env.NODE_ENV === 'test' ||
+    process.env.VITEST === 'true' ||
+    (typeof global !== 'undefined' && 'vitest' in global);
 
-  // Log server initialization
-  log('info', `Server initialized with Sentry MCP instrumentation (version ${version})`);
-
-  return server;
+  if (isTestEnvironment) {
+    // In test mode, return the base server without Sentry wrapping
+    log('info', `Server initialized without Sentry (test mode) (version ${version})`);
+    return baseServer;
+  } else {
+    // In production mode, wrap server with Sentry for MCP instrumentation
+    const server = Sentry.wrapMcpServerWithSentry(baseServer);
+    log('info', `Server initialized with Sentry MCP instrumentation (version ${version})`);
+    return server;
+  }
 }
 
 /**

src/utils/sentry.ts

@@ -77,51 +77,61 @@ function checkBinaryAvailability(binary: string): { available: boolean; version?
   return { available: true, version };
 }
 
-Sentry.init({
-  dsn: 'https://798607831167c7b9fe2f2912f5d3c665@o4509258288332800.ingest.de.sentry.io/4509258293837904',
-
-  // Setting this option to true will send default PII data to Sentry
-  // For example, automatic IP address collection on events
-  sendDefaultPii: true,
-
-  // Set release version to match application version
-  release: `xcodebuildmcp@${version}`,
-
-  // Set environment based on NODE_ENV
-  environment: process.env.NODE_ENV ?? 'development',
-
-  // Set tracesSampleRate to 1.0 to capture 100% of transactions for performance monitoring
-  // We recommend adjusting this value in production
-  tracesSampleRate: 1.0,
-});
-
-// Add additional context that might be helpful for debugging
-const tags: Record<string, string> = {
-  nodeVersion: process.version,
-  platform: process.platform,
-  arch: process.arch,
-};
-
-// Only add Xcode Info if it's available
-const xcodeInfo = getXcodeInfo();
-if (!xcodeInfo.error) {
-  tags.xcodeVersion = xcodeInfo.version;
-  tags.xcodePath = xcodeInfo.path;
-} else {
-  tags.xcodeVersion = 'Unknown';
-  tags.xcodePath = 'Unknown';
+// Skip Sentry initialization during tests to prevent module loading conflicts
+const isTestEnvironment =
+  process.env.NODE_ENV === 'test' ||
+  process.env.VITEST === 'true' ||
+  (typeof global !== 'undefined' && 'vitest' in global);
+
+if (!isTestEnvironment) {
+  Sentry.init({
+    dsn: 'https://798607831167c7b9fe2f2912f5d3c665@o4509258288332800.ingest.de.sentry.io/4509258293837904',
+
+    // Setting this option to true will send default PII data to Sentry
+    // For example, automatic IP address collection on events
+    sendDefaultPii: true,
+
+    // Set release version to match application version
+    release: `xcodebuildmcp@${version}`,
+
+    // Set environment based on NODE_ENV
+    environment: process.env.NODE_ENV ?? 'development',
+
+    // Set tracesSampleRate to 1.0 to capture 100% of transactions for performance monitoring
+    // We recommend adjusting this value in production
+    tracesSampleRate: 1.0,
+  });
 }
 
-const envVars = getEnvironmentVariables();
-tags.env_XCODEBUILDMCP_DEBUG = envVars['XCODEBUILDMCP_DEBUG'] ?? 'false';
-tags.env_XCODEMAKE_ENABLED = envVars['INCREMENTAL_BUILDS_ENABLED'] ?? 'false';
+// Add additional context that might be helpful for debugging (only in production)
+if (!isTestEnvironment) {
+  const tags: Record<string, string> = {
+    nodeVersion: process.version,
+    platform: process.platform,
+    arch: process.arch,
+  };
+
+  // Only add Xcode Info if it's available
+  const xcodeInfo = getXcodeInfo();
+  if (!xcodeInfo.error) {
+    tags.xcodeVersion = xcodeInfo.version;
+    tags.xcodePath = xcodeInfo.path;
+  } else {
+    tags.xcodeVersion = 'Unknown';
+    tags.xcodePath = 'Unknown';
+  }
 
-const miseAvailable = checkBinaryAvailability('mise');
-tags.miseAvailable = miseAvailable.available ? 'true' : 'false';
-tags.miseVersion = miseAvailable.version ?? 'Unknown';
+  const envVars = getEnvironmentVariables();
+  tags.env_XCODEBUILDMCP_DEBUG = envVars['XCODEBUILDMCP_DEBUG'] ?? 'false';
+  tags.env_XCODEMAKE_ENABLED = envVars['INCREMENTAL_BUILDS_ENABLED'] ?? 'false';
 
-const axeAvailable = checkBinaryAvailability('axe');
-tags.axeAvailable = axeAvailable.available ? 'true' : 'false';
-tags.axeVersion = axeAvailable.version ?? 'Unknown';
+  const miseAvailable = checkBinaryAvailability('mise');
+  tags.miseAvailable = miseAvailable.available ? 'true' : 'false';
+  tags.miseVersion = miseAvailable.version ?? 'Unknown';
 
-Sentry.setTags(tags);
+  const axeAvailable = checkBinaryAvailability('axe');
+  tags.axeAvailable = axeAvailable.available ? 'true' : 'false';
+  tags.axeVersion = axeAvailable.version ?? 'Unknown';
+
+  Sentry.setTags(tags);
+}