use parseStringToURLObject for url handling and update test

Author: betegon

packages/core/src/integrations/mcp-server/utils.ts

@@ -236,11 +236,9 @@ function getNotificationAttributes(
       if (params?.uri) {
         attributes['mcp.resource.uri'] = String(params.uri);
         // Extract protocol from URI
-        try {
-          const url = new URL(String(params.uri));
-          attributes['mcp.resource.protocol'] = url.protocol;
-        } catch {
-          // Ignore invalid URIs
+        const urlObject = parseStringToURLObject(String(params.uri));
+        if (urlObject && !isURLObjectRelative(urlObject)) {
+          attributes['mcp.resource.protocol'] = urlObject.protocol.replace(':', '');
         }
       }
       break;

packages/core/test/lib/mcp-server.test.ts

@@ -1,5 +1,5 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { wrapMcpServerWithSentry } from '../../src/mcp-server';
+import { wrapMcpServerWithSentry } from '../../src/integrations/mcp-server';
 import * as tracingModule from '../../src/tracing';
 
 describe('wrapMcpServerWithSentry', () => {
@@ -497,7 +497,7 @@ describe('wrapMcpServerWithSentry', () => {
           attributes: expect.objectContaining({
             'mcp.method.name': 'notifications/resources/updated',
             'mcp.resource.uri': 'file:///tmp/data.json',
-            'mcp.resource.protocol': 'file:',
+            'mcp.resource.protocol': 'file',
             'sentry.op': 'mcp.notification.client_to_server',
             'sentry.origin': 'auto.mcp.notification',
             'sentry.source': 'route',
@@ -671,7 +671,7 @@ function createMockTransport() {
     send = vi.fn().mockResolvedValue(undefined);
     sessionId = 'test-session-123';
   }
-  
+
   return new StreamableHTTPServerTransport();
 }
 
@@ -684,7 +684,7 @@ function createMockStdioTransport() {
     send = vi.fn().mockResolvedValue(undefined);
     sessionId = 'stdio-session-456';
   }
-  
+
   return new StdioServerTransport();
 }
 
@@ -697,6 +697,6 @@ function createMockSseTransport() {
     send = vi.fn().mockResolvedValue(undefined);
     sessionId = 'sse-session-789';
   }
-  
+
   return new SSEServerTransport();
 }