@machina.ai/cell-cli-core 1.6.1-rc2 → 1.8.2-rc1

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

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { FileOperation } from './metrics.js';
+import { FileOperation, MemoryMetricType, ToolExecutionPhase, ApiRequestPhase, } from './metrics.js';
 import { makeFakeConfig } from '../test-utils/config.js';
 import { ModelRoutingEvent } from './types.js';
 const mockCounterAddFn = vi.fn();
@@ -28,9 +28,11 @@ function originalOtelMockFactory() {
         },
         ValueType: {
             INT: 1,
+            DOUBLE: 2,
         },
         diag: {
             setLogger: vi.fn(),
+            warn: vi.fn(),
         },
     };
 }
@@ -41,6 +43,16 @@ describe('Telemetry Metrics', () => {
     let recordFileOperationMetricModule;
     let recordChatCompressionMetricsModule;
     let recordModelRoutingMetricsModule;
+    let recordStartupPerformanceModule;
+    let recordMemoryUsageModule;
+    let recordCpuUsageModule;
+    let recordToolQueueDepthModule;
+    let recordToolExecutionBreakdownModule;
+    let recordTokenEfficiencyModule;
+    let recordApiRequestBreakdownModule;
+    let recordPerformanceScoreModule;
+    let recordPerformanceRegressionModule;
+    let recordBaselineComparisonModule;
     beforeEach(async () => {
         vi.resetModules();
         vi.doMock('@opentelemetry/api', () => {
@@ -55,6 +67,18 @@ describe('Telemetry Metrics', () => {
         recordChatCompressionMetricsModule =
             metricsJsModule.recordChatCompressionMetrics;
         recordModelRoutingMetricsModule = metricsJsModule.recordModelRoutingMetrics;
+        recordStartupPerformanceModule = metricsJsModule.recordStartupPerformance;
+        recordMemoryUsageModule = metricsJsModule.recordMemoryUsage;
+        recordCpuUsageModule = metricsJsModule.recordCpuUsage;
+        recordToolQueueDepthModule = metricsJsModule.recordToolQueueDepth;
+        recordToolExecutionBreakdownModule =
+            metricsJsModule.recordToolExecutionBreakdown;
+        recordTokenEfficiencyModule = metricsJsModule.recordTokenEfficiency;
+        recordApiRequestBreakdownModule = metricsJsModule.recordApiRequestBreakdown;
+        recordPerformanceScoreModule = metricsJsModule.recordPerformanceScore;
+        recordPerformanceRegressionModule =
+            metricsJsModule.recordPerformanceRegression;
+        recordBaselineComparisonModule = metricsJsModule.recordBaselineComparison;
         const otelApiModule = await import('@opentelemetry/api');
         mockCounterAddFn.mockClear();
         mockCreateCounterFn.mockClear();
@@ -91,14 +115,21 @@ describe('Telemetry Metrics', () => {
     describe('recordTokenUsageMetrics', () => {
         const mockConfig = {
             getSessionId: () => 'test-session-id',
+            getTelemetryEnabled: () => true,
         };
         it('should not record metrics if not initialized', () => {
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 100, 'input');
+            recordTokenUsageMetricsModule(mockConfig, 100, {
+                model: 'gemini-pro',
+                type: 'input',
+            });
             expect(mockCounterAddFn).not.toHaveBeenCalled();
         });
         it('should record token usage with the correct attributes', () => {
             initializeMetricsModule(mockConfig);
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 100, 'input');
+            recordTokenUsageMetricsModule(mockConfig, 100, {
+                model: 'gemini-pro',
+                type: 'input',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
                 'session.id': 'test-session-id',
@@ -112,25 +143,37 @@ describe('Telemetry Metrics', () => {
         it('should record token usage for different types', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 50, 'output');
+            recordTokenUsageMetricsModule(mockConfig, 50, {
+                model: 'gemini-pro',
+                type: 'output',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(50, {
                 'session.id': 'test-session-id',
                 model: 'gemini-pro',
                 type: 'output',
             });
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 25, 'thought');
+            recordTokenUsageMetricsModule(mockConfig, 25, {
+                model: 'gemini-pro',
+                type: 'thought',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(25, {
                 'session.id': 'test-session-id',
                 model: 'gemini-pro',
                 type: 'thought',
             });
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 75, 'cache');
+            recordTokenUsageMetricsModule(mockConfig, 75, {
+                model: 'gemini-pro',
+                type: 'cache',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(75, {
                 'session.id': 'test-session-id',
                 model: 'gemini-pro',
                 type: 'cache',
             });
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-pro', 125, 'tool');
+            recordTokenUsageMetricsModule(mockConfig, 125, {
+                model: 'gemini-pro',
+                type: 'tool',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(125, {
                 'session.id': 'test-session-id',
                 model: 'gemini-pro',
@@ -140,7 +183,10 @@ describe('Telemetry Metrics', () => {
         it('should handle different models', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordTokenUsageMetricsModule(mockConfig, 'gemini-ultra', 200, 'input');
+            recordTokenUsageMetricsModule(mockConfig, 200, {
+                model: 'gemini-ultra',
+                type: 'input',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(200, {
                 'session.id': 'test-session-id',
                 model: 'gemini-ultra',
@@ -151,14 +197,25 @@ describe('Telemetry Metrics', () => {
     describe('recordFileOperationMetric', () => {
         const mockConfig = {
             getSessionId: () => 'test-session-id',
+            getTelemetryEnabled: () => true,
         };
         it('should not record metrics if not initialized', () => {
-            recordFileOperationMetricModule(mockConfig, FileOperation.CREATE, 10, 'text/plain', 'txt');
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.CREATE,
+                lines: 10,
+                mimetype: 'text/plain',
+                extension: 'txt',
+            });
             expect(mockCounterAddFn).not.toHaveBeenCalled();
         });
         it('should record file creation with all attributes', () => {
             initializeMetricsModule(mockConfig);
-            recordFileOperationMetricModule(mockConfig, FileOperation.CREATE, 10, 'text/plain', 'txt');
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.CREATE,
+                lines: 10,
+                mimetype: 'text/plain',
+                extension: 'txt',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledTimes(2);
             expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
                 'session.id': 'test-session-id',
@@ -174,7 +231,9 @@ describe('Telemetry Metrics', () => {
         it('should record file read with minimal attributes', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordFileOperationMetricModule(mockConfig, FileOperation.READ);
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.READ,
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
                 operation: FileOperation.READ,
@@ -183,7 +242,10 @@ describe('Telemetry Metrics', () => {
         it('should record file update with some attributes', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordFileOperationMetricModule(mockConfig, FileOperation.UPDATE, undefined, 'application/javascript');
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.UPDATE,
+                mimetype: 'application/javascript',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
                 operation: FileOperation.UPDATE,
@@ -193,7 +255,9 @@ describe('Telemetry Metrics', () => {
         it('should record file operation without diffStat', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordFileOperationMetricModule(mockConfig, FileOperation.UPDATE, undefined, undefined, undefined);
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.UPDATE,
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
                 operation: FileOperation.UPDATE,
@@ -202,7 +266,12 @@ describe('Telemetry Metrics', () => {
         it('should record minimal file operation when optional parameters are undefined', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordFileOperationMetricModule(mockConfig, FileOperation.UPDATE, 10, 'text/plain', 'txt', undefined);
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.UPDATE,
+                lines: 10,
+                mimetype: 'text/plain',
+                extension: 'txt',
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
                 operation: FileOperation.UPDATE,
@@ -214,7 +283,9 @@ describe('Telemetry Metrics', () => {
         it('should not include diffStat attributes when diffStat is not provided', () => {
             initializeMetricsModule(mockConfig);
             mockCounterAddFn.mockClear();
-            recordFileOperationMetricModule(mockConfig, FileOperation.UPDATE, undefined, undefined, undefined);
+            recordFileOperationMetricModule(mockConfig, {
+                operation: FileOperation.UPDATE,
+            });
             expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
                 'session.id': 'test-session-id',
                 operation: FileOperation.UPDATE,
@@ -224,6 +295,7 @@ describe('Telemetry Metrics', () => {
     describe('recordModelRoutingMetrics', () => {
         const mockConfig = {
             getSessionId: () => 'test-session-id',
+            getTelemetryEnabled: () => true,
         };
         it('should not record metrics if not initialized', () => {
             const event = new ModelRoutingEvent('gemini-pro', 'default', 100, 'test-reason', false, undefined);
@@ -260,5 +332,456 @@ describe('Telemetry Metrics', () => {
             });
         });
     });
+    describe('Performance Monitoring Metrics', () => {
+        const mockConfig = {
+            getSessionId: () => 'test-session-id',
+            getTelemetryEnabled: () => true,
+        };
+        describe('recordStartupPerformance', () => {
+            it('should not record metrics when performance monitoring is disabled', async () => {
+                // Re-import with performance monitoring disabled by mocking the config
+                const mockConfigDisabled = {
+                    getSessionId: () => 'test-session-id',
+                    getTelemetryEnabled: () => false, // Disable telemetry to disable performance monitoring
+                };
+                initializeMetricsModule(mockConfigDisabled);
+                mockHistogramRecordFn.mockClear();
+                recordStartupPerformanceModule(mockConfigDisabled, 100, {
+                    phase: 'settings_loading',
+                    details: {
+                        auth_type: 'gemini',
+                    },
+                });
+                expect(mockHistogramRecordFn).not.toHaveBeenCalled();
+            });
+            it('should record startup performance with phase and details', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordStartupPerformanceModule(mockConfig, 150, {
+                    phase: 'settings_loading',
+                    details: {
+                        auth_type: 'gemini',
+                        telemetry_enabled: true,
+                        settings_sources: 2,
+                    },
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(150, {
+                    'session.id': 'test-session-id',
+                    phase: 'settings_loading',
+                    auth_type: 'gemini',
+                    telemetry_enabled: true,
+                    settings_sources: 2,
+                });
+            });
+            it('should record startup performance without details', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordStartupPerformanceModule(mockConfig, 50, { phase: 'cleanup' });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(50, {
+                    'session.id': 'test-session-id',
+                    phase: 'cleanup',
+                });
+            });
+            it('should handle floating-point duration values from performance.now()', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                // Test with realistic floating-point values that performance.now() would return
+                const floatingPointDuration = 123.45678;
+                recordStartupPerformanceModule(mockConfig, floatingPointDuration, {
+                    phase: 'total_startup',
+                    details: {
+                        is_tty: true,
+                        has_question: false,
+                    },
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(floatingPointDuration, {
+                    'session.id': 'test-session-id',
+                    phase: 'total_startup',
+                    is_tty: true,
+                    has_question: false,
+                });
+            });
+        });
+        describe('recordMemoryUsage', () => {
+            it('should record memory usage for different memory types', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordMemoryUsageModule(mockConfig, 15728640, {
+                    memory_type: MemoryMetricType.HEAP_USED,
+                    component: 'startup',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(15728640, {
+                    'session.id': 'test-session-id',
+                    memory_type: 'heap_used',
+                    component: 'startup',
+                });
+            });
+            it('should record memory usage for all memory metric types', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordMemoryUsageModule(mockConfig, 31457280, {
+                    memory_type: MemoryMetricType.HEAP_TOTAL,
+                    component: 'api_call',
+                });
+                recordMemoryUsageModule(mockConfig, 2097152, {
+                    memory_type: MemoryMetricType.EXTERNAL,
+                    component: 'tool_execution',
+                });
+                recordMemoryUsageModule(mockConfig, 41943040, {
+                    memory_type: MemoryMetricType.RSS,
+                    component: 'memory_monitor',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 31457280, {
+                    'session.id': 'test-session-id',
+                    memory_type: 'heap_total',
+                    component: 'api_call',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 2097152, {
+                    'session.id': 'test-session-id',
+                    memory_type: 'external',
+                    component: 'tool_execution',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 41943040, {
+                    'session.id': 'test-session-id',
+                    memory_type: 'rss',
+                    component: 'memory_monitor',
+                });
+            });
+            it('should record memory usage without component', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordMemoryUsageModule(mockConfig, 15728640, {
+                    memory_type: MemoryMetricType.HEAP_USED,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(15728640, {
+                    'session.id': 'test-session-id',
+                    memory_type: 'heap_used',
+                });
+            });
+        });
+        describe('recordCpuUsage', () => {
+            it('should record CPU usage percentage', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordCpuUsageModule(mockConfig, 85.5, {
+                    component: 'tool_execution',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(85.5, {
+                    'session.id': 'test-session-id',
+                    component: 'tool_execution',
+                });
+            });
+            it('should record CPU usage without component', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordCpuUsageModule(mockConfig, 42.3, {});
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(42.3, {
+                    'session.id': 'test-session-id',
+                });
+            });
+        });
+        describe('recordToolQueueDepth', () => {
+            it('should record tool queue depth', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordToolQueueDepthModule(mockConfig, 3);
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(3, {
+                    'session.id': 'test-session-id',
+                });
+            });
+            it('should record zero queue depth', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordToolQueueDepthModule(mockConfig, 0);
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(0, {
+                    'session.id': 'test-session-id',
+                });
+            });
+        });
+        describe('recordToolExecutionBreakdown', () => {
+            it('should record tool execution breakdown for all phases', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordToolExecutionBreakdownModule(mockConfig, 25, {
+                    function_name: 'Read',
+                    phase: ToolExecutionPhase.VALIDATION,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(25, {
+                    'session.id': 'test-session-id',
+                    function_name: 'Read',
+                    phase: 'validation',
+                });
+            });
+            it('should record execution breakdown for different phases', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordToolExecutionBreakdownModule(mockConfig, 50, {
+                    function_name: 'Bash',
+                    phase: ToolExecutionPhase.PREPARATION,
+                });
+                recordToolExecutionBreakdownModule(mockConfig, 1500, {
+                    function_name: 'Bash',
+                    phase: ToolExecutionPhase.EXECUTION,
+                });
+                recordToolExecutionBreakdownModule(mockConfig, 75, {
+                    function_name: 'Bash',
+                    phase: ToolExecutionPhase.RESULT_PROCESSING,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 50, {
+                    'session.id': 'test-session-id',
+                    function_name: 'Bash',
+                    phase: 'preparation',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 1500, {
+                    'session.id': 'test-session-id',
+                    function_name: 'Bash',
+                    phase: 'execution',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 75, {
+                    'session.id': 'test-session-id',
+                    function_name: 'Bash',
+                    phase: 'result_processing',
+                });
+            });
+        });
+        describe('recordTokenEfficiency', () => {
+            it('should record token efficiency metrics', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordTokenEfficiencyModule(mockConfig, 0.85, {
+                    model: 'gemini-pro',
+                    metric: 'cache_hit_rate',
+                    context: 'api_request',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(0.85, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    metric: 'cache_hit_rate',
+                    context: 'api_request',
+                });
+            });
+            it('should record token efficiency without context', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordTokenEfficiencyModule(mockConfig, 125.5, {
+                    model: 'gemini-pro',
+                    metric: 'tokens_per_operation',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(125.5, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    metric: 'tokens_per_operation',
+                });
+            });
+        });
+        describe('recordApiRequestBreakdown', () => {
+            it('should record API request breakdown for all phases', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordApiRequestBreakdownModule(mockConfig, 15, {
+                    model: 'gemini-pro',
+                    phase: ApiRequestPhase.REQUEST_PREPARATION,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(15, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    phase: 'request_preparation',
+                });
+            });
+            it('should record API request breakdown for different phases', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordApiRequestBreakdownModule(mockConfig, 250, {
+                    model: 'gemini-pro',
+                    phase: ApiRequestPhase.NETWORK_LATENCY,
+                });
+                recordApiRequestBreakdownModule(mockConfig, 100, {
+                    model: 'gemini-pro',
+                    phase: ApiRequestPhase.RESPONSE_PROCESSING,
+                });
+                recordApiRequestBreakdownModule(mockConfig, 50, {
+                    model: 'gemini-pro',
+                    phase: ApiRequestPhase.TOKEN_PROCESSING,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledTimes(3); // One for each call
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(1, 250, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    phase: 'network_latency',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(2, 100, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    phase: 'response_processing',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenNthCalledWith(3, 50, {
+                    'session.id': 'test-session-id',
+                    model: 'gemini-pro',
+                    phase: 'token_processing',
+                });
+            });
+        });
+        describe('recordPerformanceScore', () => {
+            it('should record performance score with category and baseline', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordPerformanceScoreModule(mockConfig, 85.5, {
+                    category: 'memory_efficiency',
+                    baseline: 80.0,
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(85.5, {
+                    'session.id': 'test-session-id',
+                    category: 'memory_efficiency',
+                    baseline: 80.0,
+                });
+            });
+            it('should record performance score without baseline', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordPerformanceScoreModule(mockConfig, 92.3, {
+                    category: 'overall_performance',
+                });
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(92.3, {
+                    'session.id': 'test-session-id',
+                    category: 'overall_performance',
+                });
+            });
+        });
+        describe('recordPerformanceRegression', () => {
+            it('should record performance regression with baseline comparison', () => {
+                initializeMetricsModule(mockConfig);
+                mockCounterAddFn.mockClear();
+                mockHistogramRecordFn.mockClear();
+                recordPerformanceRegressionModule(mockConfig, {
+                    metric: 'startup_time',
+                    current_value: 1200,
+                    baseline_value: 1000,
+                    severity: 'medium',
+                });
+                // Verify regression counter
+                expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
+                    'session.id': 'test-session-id',
+                    metric: 'startup_time',
+                    severity: 'medium',
+                    current_value: 1200,
+                    baseline_value: 1000,
+                });
+                // Verify baseline comparison histogram (20% increase)
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(20, {
+                    'session.id': 'test-session-id',
+                    metric: 'startup_time',
+                    severity: 'medium',
+                    current_value: 1200,
+                    baseline_value: 1000,
+                });
+            });
+            it('should handle zero baseline value gracefully', () => {
+                initializeMetricsModule(mockConfig);
+                mockCounterAddFn.mockClear();
+                mockHistogramRecordFn.mockClear();
+                recordPerformanceRegressionModule(mockConfig, {
+                    metric: 'memory_usage',
+                    current_value: 100,
+                    baseline_value: 0,
+                    severity: 'high',
+                });
+                // Verify regression counter still recorded
+                expect(mockCounterAddFn).toHaveBeenCalledWith(1, {
+                    'session.id': 'test-session-id',
+                    metric: 'memory_usage',
+                    severity: 'high',
+                    current_value: 100,
+                    baseline_value: 0,
+                });
+                // Verify no baseline comparison due to zero baseline
+                expect(mockHistogramRecordFn).not.toHaveBeenCalled();
+            });
+            it('should record different severity levels', () => {
+                initializeMetricsModule(mockConfig);
+                mockCounterAddFn.mockClear();
+                recordPerformanceRegressionModule(mockConfig, {
+                    metric: 'api_latency',
+                    current_value: 500,
+                    baseline_value: 400,
+                    severity: 'low',
+                });
+                recordPerformanceRegressionModule(mockConfig, {
+                    metric: 'cpu_usage',
+                    current_value: 90,
+                    baseline_value: 70,
+                    severity: 'high',
+                });
+                expect(mockCounterAddFn).toHaveBeenNthCalledWith(1, 1, {
+                    'session.id': 'test-session-id',
+                    metric: 'api_latency',
+                    severity: 'low',
+                    current_value: 500,
+                    baseline_value: 400,
+                });
+                expect(mockCounterAddFn).toHaveBeenNthCalledWith(2, 1, {
+                    'session.id': 'test-session-id',
+                    metric: 'cpu_usage',
+                    severity: 'high',
+                    current_value: 90,
+                    baseline_value: 70,
+                });
+            });
+        });
+        describe('recordBaselineComparison', () => {
+            it('should record baseline comparison with percentage change', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordBaselineComparisonModule(mockConfig, {
+                    metric: 'memory_usage',
+                    current_value: 120,
+                    baseline_value: 100,
+                    category: 'performance_tracking',
+                });
+                // 20% increase: (120 - 100) / 100 * 100 = 20%
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(20, {
+                    'session.id': 'test-session-id',
+                    metric: 'memory_usage',
+                    category: 'performance_tracking',
+                    current_value: 120,
+                    baseline_value: 100,
+                });
+            });
+            it('should handle negative percentage change (improvement)', () => {
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordBaselineComparisonModule(mockConfig, {
+                    metric: 'startup_time',
+                    current_value: 800,
+                    baseline_value: 1000,
+                    category: 'optimization',
+                });
+                // 20% decrease: (800 - 1000) / 1000 * 100 = -20%
+                expect(mockHistogramRecordFn).toHaveBeenCalledWith(-20, {
+                    'session.id': 'test-session-id',
+                    metric: 'startup_time',
+                    category: 'optimization',
+                    current_value: 800,
+                    baseline_value: 1000,
+                });
+            });
+            it('should skip recording when baseline is zero', async () => {
+                // Access the actual mocked module
+                const mockedModule = (await vi.importMock('@opentelemetry/api'));
+                const diagSpy = vi.spyOn(mockedModule.diag, 'warn');
+                initializeMetricsModule(mockConfig);
+                mockHistogramRecordFn.mockClear();
+                recordBaselineComparisonModule(mockConfig, {
+                    metric: 'new_metric',
+                    current_value: 50,
+                    baseline_value: 0,
+                    category: 'testing',
+                });
+                expect(diagSpy).toHaveBeenCalledWith('Baseline value is zero, skipping comparison.');
+                expect(mockHistogramRecordFn).not.toHaveBeenCalled();
+            });
+        });
+    });
 });
 //# sourceMappingURL=metrics.test.js.map