diff --git a/packages/core/src/telemetry/clearcut-logger/clearcut-logger.ts b/packages/core/src/telemetry/clearcut-logger/clearcut-logger.ts index 32dcf926..dd506bf0 100644 --- a/packages/core/src/telemetry/clearcut-logger/clearcut-logger.ts +++ b/packages/core/src/telemetry/clearcut-logger/clearcut-logger.ts @@ -385,12 +385,14 @@ export class ClearcutLogger { { gemini_cli_key: EventMetadataKey.GEMINI_CLI_START_SESSION_MCP_SERVERS_COUNT, - value: event.mcp_servers_count ? event.mcp_servers_count : '', + value: event.mcp_servers_count + ? event.mcp_servers_count.toString() + : '', }, { gemini_cli_key: EventMetadataKey.GEMINI_CLI_START_SESSION_MCP_TOOLS_COUNT, - value: event.mcp_tools_count ? event.mcp_tools_count : '', + value: event.mcp_tools_count?.toString() ?? '', }, { gemini_cli_key: EventMetadataKey.GEMINI_CLI_START_SESSION_MCP_TOOLS, diff --git a/packages/core/src/telemetry/loggers.test.ts b/packages/core/src/telemetry/loggers.test.ts index d52cb600..a3375d95 100644 --- a/packages/core/src/telemetry/loggers.test.ts +++ b/packages/core/src/telemetry/loggers.test.ts @@ -162,7 +162,7 @@ describe('loggers', () => { file_filtering_respect_git_ignore: true, debug_mode: true, mcp_servers: 'test-server', - mcp_servers_count: '1', + mcp_servers_count: 1, mcp_tools: undefined, mcp_tools_count: undefined, }, diff --git a/packages/core/src/telemetry/types.ts b/packages/core/src/telemetry/types.ts index 8b1b5d68..9310992c 100644 --- a/packages/core/src/telemetry/types.ts +++ b/packages/core/src/telemetry/types.ts @@ -39,8 +39,8 @@ export class StartSessionEvent implements BaseTelemetryEvent { telemetry_enabled: boolean; telemetry_log_user_prompts_enabled: boolean; file_filtering_respect_git_ignore: boolean; - mcp_servers_count?: string; - mcp_tools_count?: string; + mcp_servers_count: number; + mcp_tools_count?: number; mcp_tools?: string; constructor(config: Config, toolRegistry?: ToolRegistry) { @@ -70,14 +70,12 @@ export class StartSessionEvent implements BaseTelemetryEvent { config.getTelemetryLogPromptsEnabled(); this.file_filtering_respect_git_ignore = config.getFileFilteringRespectGitIgnore(); - this.mcp_servers_count = mcpServers - ? Object.keys(mcpServers).length.toString() - : ''; + this.mcp_servers_count = mcpServers ? Object.keys(mcpServers).length : 0; if (toolRegistry) { const mcpTools = toolRegistry .getAllTools() .filter((tool) => tool instanceof DiscoveredMCPTool); - this.mcp_tools_count = mcpTools.length.toString(); + this.mcp_tools_count = mcpTools.length; this.mcp_tools = mcpTools .map((tool) => (tool as DiscoveredMCPTool).name) .join(',');