@google/gemini-cli-core 0.10.0-nightly.20251009.cd354aeb → 0.10.0-nightly.20251013.cfb71b9d

package/dist/src/telemetry/metrics.test.js

@@ -43,6 +43,7 @@ function originalOtelMockFactory() {
     };
 }
 vi.mock('@opentelemetry/api');
+vi.mock('./telemetryAttributes.js');
 describe('Telemetry Metrics', () => {
     let initializeMetricsModule;
     let recordTokenUsageMetricsModule;
@@ -61,6 +62,7 @@ describe('Telemetry Metrics', () => {
     let recordBaselineComparisonModule;
     let recordGenAiClientTokenUsageModule;
     let recordGenAiClientOperationDurationModule;
+    let recordFlickerFrameModule;
     let recordAgentRunMetricsModule;
     beforeEach(async () => {
         vi.resetModules();
@@ -69,6 +71,12 @@ describe('Telemetry Metrics', () => {
             actualApi.metrics.getMeter.mockReturnValue(mockMeterInstance);
             return actualApi;
         });
+        const { getCommonAttributes } = await import('./telemetryAttributes.js');
+        getCommonAttributes.mockReturnValue({
+            'session.id': 'test-session-id',
+            'installation.id': 'test-installation-id',
+            'user.email': 'test@example.com',
+        });
         const metricsJsModule = await import('./metrics.js');
         initializeMetricsModule = metricsJsModule.initializeMetrics;
         recordTokenUsageMetricsModule = metricsJsModule.recordTokenUsageMetrics;
@@ -92,6 +100,7 @@ describe('Telemetry Metrics', () => {
             metricsJsModule.recordGenAiClientTokenUsage;
         recordGenAiClientOperationDurationModule =
             metricsJsModule.recordGenAiClientOperationDuration;
+        recordFlickerFrameModule = metricsJsModule.recordFlickerFrame;
         recordAgentRunMetricsModule = metricsJsModule.recordAgentRunMetrics;
         const otelApiModule = await import('@opentelemetry/api');
         mockCounterAddFn.mockClear();
@@ -103,6 +112,39 @@ describe('Telemetry Metrics', () => {
         mockCreateCounterFn.mockReturnValue(mockCounterInstance);
         mockCreateHistogramFn.mockReturnValue(mockHistogramInstance);
     });
+    describe('recordFlickerFrame', () => {
+        it('does not record metrics if not initialized', () => {
+            const config = makeFakeConfig({});
+            recordFlickerFrameModule(config);
+            expect(mockCounterAddFn).not.toHaveBeenCalled();
+        });
+        it('records a flicker frame event when initialized', () => {
+            const config = makeFakeConfig({});
+            initializeMetricsModule(config);
+            recordFlickerFrameModule(config);
+            // Called for session, then for flicker
+            expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
+            expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 1, {
+                'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
+            });
+        });
+    });
+    describe('initializeMetrics', () => {
+        const mockConfig = {
+            getSessionId: () => 'test-session-id',
+            getTelemetryEnabled: () => true,
+        };
+        it('should apply common attributes including email', () => {
+            initializeMetricsModule(mockConfig);
+            expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
+                'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
+            });
+        });
+    });
     describe('recordChatCompressionMetrics', () => {
         it('does not record metrics if not initialized', () => {
             const lol = makeFakeConfig({});
@@ -121,6 +163,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 tokens_after: 100,
                 tokens_before: 200,
             });
@@ -147,9 +191,13 @@ describe('Telemetry Metrics', () => {
             expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
             });
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 100, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-pro',
                 type: 'input',
             });
@@ -163,6 +211,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(50, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-pro',
                 type: 'output',
             });
@@ -172,6 +222,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(25, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-pro',
                 type: 'thought',
             });
@@ -181,6 +233,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(75, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-pro',
                 type: 'cache',
             });
@@ -190,6 +244,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(125, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-pro',
                 type: 'tool',
             });
@@ -203,6 +259,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(200, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 model: 'gemini-ultra',
                 type: 'input',
             });
@@ -233,9 +291,13 @@ describe('Telemetry Metrics', () => {
             expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
             });
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.CREATE,
                 lines: 10,
                 mimetype: 'text/plain',
@@ -250,6 +312,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.READ,
             });
         });
@@ -262,6 +326,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.UPDATE,
                 mimetype: 'application/javascript',
             });
@@ -274,6 +340,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.UPDATE,
             });
         });
@@ -288,6 +356,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.UPDATE,
                 lines: 10,
                 mimetype: 'text/plain',
@@ -302,6 +372,8 @@ describe('Telemetry Metrics', () => {
             });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 operation: FileOperation.UPDATE,
             });
         });
@@ -323,6 +395,8 @@ describe('Telemetry Metrics', () => {
             recordModelRoutingMetricsModule(mockConfig, event);
             expect(mockHistogramRecordFn).toHaveBeenCalledWith(150, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 'routing.decision_model': 'gemini-pro',
                 'routing.decision_source': 'default',
             });
@@ -335,12 +409,16 @@ describe('Telemetry Metrics', () => {
             recordModelRoutingMetricsModule(mockConfig, event);
             expect(mockHistogramRecordFn).toHaveBeenCalledWith(200, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 'routing.decision_model': 'gemini-pro',
                 'routing.decision_source': 'classifier',
             });
             expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 'routing.decision_source': 'classifier',
                 'routing.error_message': 'test-error',
             });
@@ -366,17 +444,23 @@ describe('Telemetry Metrics', () => {
             // Verify agent run counter
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 agent_name: 'TestAgent',
                 terminate_reason: 'GOAL',
             });
             // Verify agent duration histogram
             expect(mockHistogramRecordFn).toHaveBeenCalledWith(1000, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 agent_name: 'TestAgent',
             });
             // Verify agent turns histogram
             expect(mockHistogramRecordFn).toHaveBeenCalledWith(5, {
                 'session.id': 'test-session-id',
+                'installation.id': 'test-installation-id',
+                'user.email': 'test@example.com',
                 agent_name: 'TestAgent',
             });
         });
@@ -407,6 +491,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(150, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.gen_ai',
                     'gen_ai.token.type': 'input',
@@ -425,6 +511,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(75, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.vertex_ai',
                     'gen_ai.token.type': 'output',
@@ -444,6 +532,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(200, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.vertex_ai',
                     'gen_ai.token.type': 'input',
@@ -472,6 +562,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(1.25, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.gen_ai',
                     'gen_ai.request.model': 'gemini-2.0-flash',
@@ -489,6 +581,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(3.75, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.vertex_ai',
                     'gen_ai.request.model': 'gemini-pro',
@@ -508,6 +602,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(0.95, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.vertex_ai',
                     'gen_ai.request.model': 'gemini-1.5-pro',
@@ -525,6 +621,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(2.1, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     'gen_ai.operation.name': 'generate_content',
                     'gen_ai.provider.name': 'gcp.gen_ai',
                 });
@@ -566,6 +664,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(150, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     phase: 'settings_loading',
                     auth_type: 'gemini',
                     telemetry_enabled: true,
@@ -578,6 +678,8 @@ describe('Telemetry Metrics', () => {
                 recordStartupPerformanceModule(mockConfig, 50, { phase: 'cleanup' });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(50, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     phase: 'cleanup',
                 });
             });
@@ -595,6 +697,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(floatingPointDuration, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     phase: 'total_startup',
                     is_tty: true,
                     has_question: false,
@@ -611,6 +715,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(15728640, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     memory_type: 'heap_used',
                     component: 'startup',
                 });
@@ -633,16 +739,22 @@ describe('Telemetry Metrics', () => {
                 expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 31457280, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     memory_type: 'heap_total',
                     component: 'api_call',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 2097152, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     memory_type: 'external',
                     component: 'tool_execution',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 41943040, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     memory_type: 'rss',
                     component: 'memory_monitor',
                 });
@@ -655,6 +767,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(15728640, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     memory_type: 'heap_used',
                 });
             });
@@ -668,6 +782,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(85.5, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     component: 'tool_execution',
                 });
             });
@@ -677,6 +793,8 @@ describe('Telemetry Metrics', () => {
                 recordCpuUsageModule(mockConfig, 42.3, {});
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(42.3, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                 });
             });
         });
@@ -687,6 +805,8 @@ describe('Telemetry Metrics', () => {
                 recordToolQueueDepthModule(mockConfig, 3);
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(3, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                 });
             });
             it('should record zero queue depth', () => {
@@ -695,6 +815,8 @@ describe('Telemetry Metrics', () => {
                 recordToolQueueDepthModule(mockConfig, 0);
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(0, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                 });
             });
         });
@@ -708,6 +830,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(25, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     function_name: 'Read',
                     phase: 'validation',
                 });
@@ -730,16 +854,22 @@ describe('Telemetry Metrics', () => {
                 expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 50, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     function_name: 'Bash',
                     phase: 'preparation',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 1500, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     function_name: 'Bash',
                     phase: 'execution',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 75, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     function_name: 'Bash',
                     phase: 'result_processing',
                 });
@@ -756,6 +886,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(0.85, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     metric: 'cache_hit_rate',
                     context: 'api_request',
@@ -770,6 +902,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(125.5, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     metric: 'tokens_per_operation',
                 });
@@ -785,6 +919,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(15, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     phase: 'request_preparation',
                 });
@@ -807,16 +943,22 @@ describe('Telemetry Metrics', () => {
                 expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 250, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     phase: 'network_latency',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 100, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     phase: 'response_processing',
                 });
                 expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 50, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     model: 'gemini-pro',
                     phase: 'token_processing',
                 });
@@ -832,6 +974,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(85.5, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     category: 'memory_efficiency',
                     baseline: 80.0,
                 });
@@ -844,6 +988,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(92.3, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     category: 'overall_performance',
                 });
             });
@@ -862,6 +1008,8 @@ describe('Telemetry Metrics', () => {
                 // Verify regression counter
                 expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'startup_time',
                     severity: 'medium',
                     current_value: 1200,
@@ -870,6 +1018,8 @@ describe('Telemetry Metrics', () => {
                 // Verify baseline comparison histogram (20% increase)
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(20, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'startup_time',
                     severity: 'medium',
                     current_value: 1200,
@@ -889,6 +1039,8 @@ describe('Telemetry Metrics', () => {
                 // Verify regression counter still recorded
                 expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'memory_usage',
                     severity: 'high',
                     current_value: 100,
@@ -914,6 +1066,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'api_latency',
                     severity: 'low',
                     current_value: 500,
@@ -921,6 +1075,8 @@ describe('Telemetry Metrics', () => {
                 });
                 expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 1, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'cpu_usage',
                     severity: 'high',
                     current_value: 90,
@@ -941,6 +1097,8 @@ describe('Telemetry Metrics', () => {
                 // 20% increase: (120 - 100) / 100 * 100 = 20%
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(20, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'memory_usage',
                     category: 'performance_tracking',
                     current_value: 120,
@@ -959,6 +1117,8 @@ describe('Telemetry Metrics', () => {
                 // 20% decrease: (800 - 1000) / 1000 * 100 = -20%
                 expect(mockHistogramRecordFn).toHaveBeenCalledWith(-20, {
                     'session.id': 'test-session-id',
+                    'installation.id': 'test-installation-id',
+                    'user.email': 'test@example.com',
                     metric: 'startup_time',
                     category: 'optimization',
                     current_value: 800,