Skip to content

Commit 2c6e5a0

Browse files
committed
fix lint
1 parent 1ae03e5 commit 2c6e5a0

File tree

4 files changed

+11
-18
lines changed

4 files changed

+11
-18
lines changed

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

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,7 @@ import {
1212
MCP_RESOURCE_URI_ATTRIBUTE,
1313
} from './attributes';
1414
import { extractTargetInfo, getRequestArguments } from './methodConfig';
15-
import type {
16-
JsonRpcNotification,
17-
JsonRpcRequest,
18-
McpSpanType,
19-
} from './types';
20-
21-
15+
import type { JsonRpcNotification, JsonRpcRequest, McpSpanType } from './types';
2216

2317
/**
2418
* Extracts additional attributes for specific notification types
@@ -96,8 +90,6 @@ export function getNotificationAttributes(
9690
return attributes;
9791
}
9892

99-
100-
10193
/**
10294
* Build type-specific attributes based on message type
10395
* @param type - Span type (request or notification)

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

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,11 @@ import {
1616
NETWORK_PROTOCOL_VERSION_ATTRIBUTE,
1717
NETWORK_TRANSPORT_ATTRIBUTE,
1818
} from './attributes';
19-
import { getClientInfoForTransport, getProtocolVersionForTransport, getSessionDataForTransport } from './sessionManagement';
19+
import {
20+
getClientInfoForTransport,
21+
getProtocolVersionForTransport,
22+
getSessionDataForTransport,
23+
} from './sessionManagement';
2024
import type { ExtraHandlerData, JsonRpcRequest, MCPTransport, PartyInfo, SessionData } from './types';
2125

2226
/**

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,7 @@ import { startInactiveSpan, withActiveSpan } from '../../tracing';
1010
import { fill } from '../../utils/object';
1111
import { cleanupPendingSpansForTransport, completeSpanWithResults, storeSpanForRequest } from './correlation';
1212
import { captureError } from './errorCapture';
13-
import {
14-
extractSessionDataFromInitializeRequest,
15-
extractSessionDataFromInitializeResponse,
16-
} from './sessionExtraction';
13+
import { extractSessionDataFromInitializeRequest, extractSessionDataFromInitializeResponse } from './sessionExtraction';
1714
import {
1815
cleanupSessionDataForTransport,
1916
storeSessionDataForTransport,

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -206,15 +206,15 @@ describe('MCP Server PII Filtering', () => {
206206
// Client info should be filtered
207207
expect(result).not.toHaveProperty('client.address');
208208
expect(result).not.toHaveProperty('client.port');
209-
209+
210210
// Request arguments should be filtered
211211
expect(result).not.toHaveProperty('mcp.request.argument.location');
212212
expect(result).not.toHaveProperty('mcp.request.argument.units');
213-
213+
214214
// Specific PII content attributes should be filtered
215215
expect(result).not.toHaveProperty('mcp.tool.result.content');
216216
expect(result).not.toHaveProperty('mcp.prompt.result.description');
217-
217+
218218
// Indexed/dynamic result attributes (not in PII_ATTRIBUTES) should remain
219219
expect(result).toHaveProperty('mcp.tool.result.content_count', 1);
220220
expect(result).toHaveProperty('mcp.prompt.result.message_count', 2);
@@ -225,7 +225,7 @@ describe('MCP Server PII Filtering', () => {
225225
expect(result).toHaveProperty('mcp.resource.result.content_count', 1);
226226
expect(result).toHaveProperty('mcp.resource.result.uri', 'file:///private/file.txt');
227227
expect(result).toHaveProperty('mcp.resource.result.content', 'Sensitive resource content');
228-
228+
229229
// Other PII attributes should be filtered
230230
expect(result).not.toHaveProperty('mcp.logging.message');
231231
expect(result).not.toHaveProperty('mcp.resource.uri');

0 commit comments

Comments
 (0)