autotel-tanstack 1.13.34 → 1.13.36

package/src/metrics.ts

@@ -1,184 +0,0 @@
-/**
- * Performance metrics collection for TanStack Start
- *
- * Provides utilities to collect and expose performance metrics
- * following the patterns from TanStack Start observability guide.
- */
-
-/**
- * Performance timing data
- */
-export interface TimingStats {
-  count: number;
-  avg: number;
-  p50: number;
-  p95: number;
-  min: number;
-  max: number;
-}
-
-/**
- * Metrics collector for performance data
- *
- * Collects timing metrics and provides statistical analysis.
- * Thread-safe for concurrent access.
- *
- * @example
- * ```typescript
- * import { metricsCollector } from 'autotel-tanstack/metrics';
- *
- * // Record a timing
- * metricsCollector.recordTiming('serverFn.getUser', 150);
- *
- * // Get stats
- * const stats = metricsCollector.getStats('serverFn.getUser');
- * console.log(`Average: ${stats.avg}ms, P95: ${stats.p95}ms`);
- * ```
- */
-class MetricsCollector {
-  private metrics = new Map<string, number[]>();
-  private readonly maxSamples = 1000; // Limit memory usage
-
-  /**
-   * Record a timing measurement
-   */
-  recordTiming(name: string, duration: number): void {
-    if (!this.metrics.has(name)) {
-      this.metrics.set(name, []);
-    }
-
-    const timings = this.metrics.get(name)!;
-    timings.push(duration);
-
-    // Limit samples to prevent memory issues
-    if (timings.length > this.maxSamples) {
-      timings.shift(); // Remove oldest
-    }
-  }
-
-  /**
-   * Get statistics for a metric
-   */
-  getStats(name: string): TimingStats | null {
-    const timings = this.metrics.get(name);
-    if (!timings || timings.length === 0) {
-      return null;
-    }
-
-    const sorted = [...timings].toSorted((a, b) => a - b);
-    const sum = timings.reduce((a, b) => a + b, 0);
-
-    return {
-      count: timings.length,
-      avg: sum / timings.length,
-      p50: sorted.at(Math.floor(sorted.length * 0.5)) ?? 0,
-      p95: sorted.at(Math.floor(sorted.length * 0.95)) ?? 0,
-      min: sorted[0] ?? 0,
-      max: sorted.at(-1) ?? 0,
-    };
-  }
-
-  /**
-   * Get all collected metrics
-   */
-  getAllStats(): Record<string, TimingStats> {
-    const stats: Record<string, TimingStats> = {};
-    for (const [name] of this.metrics) {
-      const stat = this.getStats(name);
-      if (stat) {
-        stats[name] = stat;
-      }
-    }
-    return stats;
-  }
-
-  /**
-   * Reset all metrics
-   */
-  reset(): void {
-    this.metrics.clear();
-  }
-
-  /**
-   * Reset a specific metric
-   */
-  resetMetric(name: string): void {
-    this.metrics.delete(name);
-  }
-}
-
-/**
- * Global metrics collector instance
- */
-export const metricsCollector = new MetricsCollector();
-
-/**
- * Helper to create a metrics endpoint handler
- *
- * Returns a handler that exposes metrics in JSON format.
- * Use this to create a `/metrics` endpoint.
- *
- * @example
- * ```typescript
- * // routes/metrics.ts
- * import { createFileRoute } from '@tanstack/react-router';
- * import { json } from '@tanstack/react-start';
- * import { createMetricsHandler } from 'autotel-tanstack/metrics';
- *
- * export const Route = createFileRoute('/metrics')({
- *   server: {
- *     handlers: {
- *       GET: createMetricsHandler(),
- *     },
- *   },
- * });
- * ```
- */
-export function createMetricsHandler() {
-  return async () => {
-    const { json } = await import('@tanstack/react-start');
-
-    return json({
-      system: {
-        uptime: process.uptime(),
-        memory: process.memoryUsage(),
-        timestamp: new Date().toISOString(),
-      },
-      application: metricsCollector.getAllStats(),
-    });
-  };
-}
-
-/**
- * Auto-record timing from a function execution
- *
- * Wraps a function to automatically record its execution time.
- *
- * @example
- * ```typescript
- * import { recordTiming } from 'autotel-tanstack/metrics';
- *
- * const getUser = createServerFn({ method: 'GET' })
- *   .handler(recordTiming('serverFn.getUser', async ({ data: id }) => {
- *     return await db.users.findUnique({ where: { id } });
- *   }));
- * ```
- */
-export function recordTiming<T extends (...args: any[]) => any>(
-  metricName: string,
-  fn: T,
-): T {
-  return (async (...args: Parameters<T>) => {
-    const startTime = Date.now();
-    try {
-      const result = await fn(...args);
-      const duration = Date.now() - startTime;
-      metricsCollector.recordTiming(metricName, duration);
-      return result as ReturnType<T>;
-    } catch (error) {
-      const duration = Date.now() - startTime;
-      metricsCollector.recordTiming(`${metricName}.error`, duration);
-      throw error;
-    }
-  }) as T;
-}

package/src/middleware.test.ts

@@ -1,198 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-  createTracingMiddleware,
-  tracingMiddleware,
-  functionTracingMiddleware,
-} from './middleware';
-
-// Mock autotel
-vi.mock('autotel', () => ({
-  trace: vi.fn((name, fn) =>
-    fn({
-      setAttributes: vi.fn(),
-      setAttribute: vi.fn(),
-      setStatus: vi.fn(),
-      recordException: vi.fn(),
-    }),
-  ),
-}));
-
-// Mock context module
-vi.mock('./context.js', () => ({
-  extractContextFromRequest: vi.fn(() => ({})),
-}));
-
-describe('middleware', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('createTracingMiddleware', () => {
-    it('should create request middleware by default', async () => {
-      const middleware = createTracingMiddleware();
-      const request = new Request('http://localhost/api/users');
-      const next = vi.fn().mockResolvedValue({ status: 200 });
-
-      await middleware({
-        next,
-        request,
-        pathname: '/api/users',
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should create function middleware when type is "function"', async () => {
-      const middleware = createTracingMiddleware({ type: 'function' });
-      const next = vi.fn().mockResolvedValue({ data: 'test' });
-
-      await middleware({
-        next,
-        context: {},
-        data: { id: '123' },
-        functionId: 'getUser',
-        method: 'GET',
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should skip excluded paths', async () => {
-      const middleware = createTracingMiddleware({
-        excludePaths: ['/health', '/metrics'],
-      });
-      const request = new Request('http://localhost/health');
-      const next = vi.fn().mockResolvedValue({ status: 200 });
-
-      await middleware({
-        next,
-        request,
-        pathname: '/health',
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should handle glob patterns in excludePaths', async () => {
-      const middleware = createTracingMiddleware({
-        excludePaths: ['/api/internal/*'],
-      });
-      const request = new Request('http://localhost/api/internal/debug');
-      const next = vi.fn().mockResolvedValue({ status: 200 });
-
-      await middleware({
-        next,
-        request,
-        pathname: '/api/internal/debug',
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should handle regex patterns in excludePaths', async () => {
-      const middleware = createTracingMiddleware({
-        excludePaths: [/^\/api\/v\d+\/health$/],
-      });
-      const request = new Request('http://localhost/api/v1/health');
-      const next = vi.fn().mockResolvedValue({ status: 200 });
-
-      await middleware({
-        next,
-        request,
-        pathname: '/api/v1/health',
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should propagate errors from next()', async () => {
-      const middleware = createTracingMiddleware();
-      const error = new Error('Handler error');
-      const next = vi.fn().mockRejectedValue(error);
-      const request = new Request('http://localhost/api/users');
-      // Middleware reports the rejected error via console.error in
-      // error-reporting.ts — that's the contract under test (errors don't
-      // get silently swallowed). Silence the noise; the rejection assertion
-      // proves the propagation.
-      const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
-
-      await expect(
-        middleware({
-          next,
-          request,
-          pathname: '/api/users',
-          context: {},
-        }),
-      ).rejects.toThrow('Handler error');
-
-      errSpy.mockRestore();
-    });
-
-    it('should pass through when no request is available', async () => {
-      const middleware = createTracingMiddleware();
-      const next = vi.fn().mockResolvedValue({ data: 'test' });
-
-      const result = await middleware({
-        next,
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-      expect(result).toEqual({ data: 'test' });
-    });
-  });
-
-  describe('tracingMiddleware', () => {
-    it('should create middleware with sensible defaults', async () => {
-      const middleware = tracingMiddleware();
-      expect(middleware).toBeDefined();
-    });
-
-    it('should exclude common health check paths', async () => {
-      const middleware = tracingMiddleware();
-      const request = new Request('http://localhost/healthz');
-      const next = vi.fn().mockResolvedValue({ status: 200 });
-
-      await middleware({
-        next,
-        request,
-        pathname: '/healthz',
-        context: {},
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should allow config overrides', async () => {
-      const middleware = tracingMiddleware({
-        captureHeaders: ['x-custom-header'],
-      });
-      expect(middleware).toBeDefined();
-    });
-  });
-
-  describe('functionTracingMiddleware', () => {
-    it('should create function middleware', async () => {
-      const middleware = functionTracingMiddleware();
-      const next = vi.fn().mockResolvedValue({ data: 'test' });
-
-      await middleware({
-        next,
-        context: {},
-        data: { id: '123' },
-        functionId: 'testFn',
-      });
-
-      expect(next).toHaveBeenCalled();
-    });
-
-    it('should not require type in config', () => {
-      const middleware = functionTracingMiddleware({ captureArgs: false });
-      expect(middleware).toBeDefined();
-    });
-  });
-});