autotel-tanstack 1.13.35 → 1.13.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autotel-tanstack",
-  "version": "1.13.35",
+  "version": "1.13.36",
   "description": "OpenTelemetry instrumentation for TanStack Start - automatic tracing for server functions, middleware, and route loaders",
   "type": "module",
   "main": "./dist/index.js",
@@ -96,7 +96,6 @@
   },
   "files": [
     "dist",
-    "src",
     "README.md",
     "skills"
   ],
@@ -120,9 +119,9 @@
   "license": "MIT",
   "dependencies": {
     "@opentelemetry/api": "^1.9.1",
-    "autotel": "4.1.0",
-    "autotel-adapters": "0.3.12",
-    "autotel-edge": "3.17.0"
+    "autotel": "4.2.0",
+    "autotel-adapters": "0.3.13",
+    "autotel-edge": "3.17.1"
   },
   "peerDependencies": {
     "@tanstack/react-start": "^1.168.25",

package/src/auto.test.ts

@@ -1,114 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-// Must mock before importing auto.ts (which runs at module level)
-const mockInit = vi.fn();
-const mockReset = vi.fn();
-const mockGetFinishedSpans = vi.fn(() => []);
-
-vi.mock('autotel', () => ({
-  init: mockInit,
-  // instrument() guards on this; false so every test proceeds to init().
-  isInitialized: vi.fn(() => false),
-}));
-
-const mockExporterInstance = {
-  reset: mockReset,
-  getFinishedSpans: mockGetFinishedSpans,
-};
-// Use regular function (not arrow) so vi.fn() can be called with `new` in vitest 4.x
-const MockInMemorySpanExporter = vi.fn(function () {
-  return mockExporterInstance;
-});
-const MockSimpleSpanProcessor = vi.fn(function (exp: unknown) {
-  return { exporter: exp };
-});
-
-vi.mock('autotel/exporters', () => ({
-  InMemorySpanExporter: MockInMemorySpanExporter,
-}));
-
-vi.mock('autotel/processors', () => ({
-  SimpleSpanProcessor: MockSimpleSpanProcessor,
-}));
-
-describe('auto.ts E2E mode', () => {
-  const originalEnv = { ...process.env };
-
-  beforeEach(() => {
-    vi.resetModules();
-    mockInit.mockReset();
-    MockInMemorySpanExporter.mockClear();
-    MockSimpleSpanProcessor.mockClear();
-    delete (globalThis as Record<string, unknown>).__testSpanExporter;
-  });
-
-  afterEach(() => {
-    process.env = { ...originalEnv };
-  });
-
-  it('uses InMemorySpanExporter when E2E=1', async () => {
-    process.env.E2E = '1';
-    delete process.env.OTEL_EXPORTER_OTLP_ENDPOINT;
-    await import('./auto');
-    expect(MockInMemorySpanExporter).toHaveBeenCalledOnce();
-    expect(MockSimpleSpanProcessor).toHaveBeenCalledOnce();
-    expect(mockInit).toHaveBeenCalledWith(
-      expect.objectContaining({ spanProcessors: expect.any(Array) }),
-    );
-  });
-
-  it('sets globalThis.__testSpanExporter when E2E=1', async () => {
-    process.env.E2E = '1';
-    delete process.env.OTEL_EXPORTER_OTLP_ENDPOINT;
-    await import('./auto');
-    expect((globalThis as Record<string, unknown>).__testSpanExporter).toBe(
-      mockExporterInstance,
-    );
-  });
-
-  it('does not set __testSpanExporter in production mode', async () => {
-    delete process.env.E2E;
-    delete process.env.OTEL_EXPORTER_OTLP_ENDPOINT;
-    await import('./auto');
-    expect(
-      (globalThis as Record<string, unknown>).__testSpanExporter,
-    ).toBeUndefined();
-  });
-
-  // Skipped: constructing a second OTLP BatchSpanProcessor requires
-  // @opentelemetry/exporter-trace-otlp-http which is not available in
-  // autotel-tanstack's direct dependencies. The combined E2E+OTLP path
-  // is tested at the integration level instead.
-  it.skip('adds both InMemory and OTLP processors when E2E=1 and endpoint set', async () => {
-    process.env.E2E = '1';
-    process.env.OTEL_EXPORTER_OTLP_ENDPOINT = 'http://localhost:4318';
-    await import('./auto');
-    const call = mockInit.mock.calls[0][0] as { spanProcessors?: unknown[] };
-    expect(call.spanProcessors).toHaveLength(2);
-  });
-
-  it('initializes in production mode without injecting an in-memory processor', async () => {
-    // The OTLP endpoint/headers are resolved by autotel core from env, so the
-    // zero-config path no longer passes them explicitly — it just inits.
-    delete process.env.E2E;
-    process.env.OTEL_EXPORTER_OTLP_ENDPOINT = 'http://localhost:4318';
-    await import('./auto');
-    expect(mockInit).toHaveBeenCalledOnce();
-    const call = mockInit.mock.calls[0][0] as {
-      spanProcessors?: unknown[];
-      service?: string;
-    };
-    expect(call.spanProcessors).toBeUndefined();
-    expect(call.service).toBe('tanstack-start');
-    expect(MockInMemorySpanExporter).not.toHaveBeenCalled();
-  });
-
-  it('uses OTEL_SERVICE_NAME env var', async () => {
-    process.env.E2E = '1';
-    process.env.OTEL_SERVICE_NAME = 'my-e2e-app';
-    await import('./auto');
-    expect(mockInit).toHaveBeenCalledWith(
-      expect.objectContaining({ service: 'my-e2e-app' }),
-    );
-  });
-});

package/src/auto.ts

@@ -1,60 +0,0 @@
-/**
- * Zero-config auto-instrumentation for TanStack Start.
- *
- * Importing this module configures OpenTelemetry tracing from environment
- * variables — it is simply `instrument()` with no options. For subscribers,
- * structured logs, canonical log lines, or an explicit endpoint, call
- * `instrument(options)` from `autotel-tanstack` directly (see ./instrument).
- *
- * Environment Variables (resolved by autotel core):
- * - OTEL_SERVICE_NAME: Service name (default: 'tanstack-start')
- * - OTEL_EXPORTER_OTLP_ENDPOINT: OTLP collector URL
- * - OTEL_EXPORTER_OTLP_HEADERS: Auth headers (key=value,key=value)
- * - AUTOTEL_DEBUG: 'true' | 'pretty' to log spans to the server console
- *
- * @example
- * ```typescript
- * // app/start.ts
- * import 'autotel-tanstack/auto';
- * import { createStart } from '@tanstack/react-start';
- * export const startInstance = createStart(() => ({}));
- * ```
- *
- * @module
- */
-
-import { instrument } from './instrument';
-
-instrument();
-
-// Surface what happened, in development only.
-if (process.env.NODE_ENV === 'development' || process.env.AUTOTEL_DEBUG) {
-  console.log('[autotel-tanstack] Auto-initialized', {
-    service: process.env.OTEL_SERVICE_NAME || 'tanstack-start',
-    endpoint:
-      process.env.E2E === '1'
-        ? '(E2E: in-memory)'
-        : process.env.OTEL_EXPORTER_OTLP_ENDPOINT || '(not configured)',
-  });
-}
-
-// Re-export the configurable form + middleware/helpers for convenience.
-export { instrument } from './instrument';
-export { tracingMiddleware, functionTracingMiddleware } from './middleware';
-export { traceServerFn } from './server-functions';
-export { traceLoader, traceBeforeLoad } from './loaders';
-
-/** Check if auto-instrumentation is active. */
-export function isAutoInstrumentationActive(): boolean {
-  return true;
-}
-
-/** The configured service name. */
-export function getServiceName(): string {
-  return process.env.OTEL_SERVICE_NAME || 'tanstack-start';
-}
-
-/** The configured OTLP endpoint, if any. */
-export function getEndpoint(): string | undefined {
-  return process.env.OTEL_EXPORTER_OTLP_ENDPOINT;
-}

package/src/browser/context.ts

@@ -1,88 +0,0 @@
-/**
- * Browser stub for context module
- *
- * In browser environments, context propagation is not needed.
- * These functions return empty/default values.
- */
-
-/**
- * Type representing values that can be used to initialize Headers
- */
-export type HeadersInitType =
-  | Headers
-  | Record<string, string>
-  | [string, string][];
-
-/**
- * Browser stub: Returns root context (no parent)
- */
-export function extractContextFromRequest(_request: Request): unknown {
-  void _request;
-  // Return an empty context-like object
-  return {};
-}
-
-/**
- * Browser stub: Returns headers unchanged
- */
-export function injectContextToHeaders(
-  headers: Headers,
-  ctx?: unknown,
-): Headers {
-  void ctx;
-  return headers;
-}
-
-/**
- * Browser stub: Create headers with optional initial values
- */
-export function createTracedHeaders(
-  existingHeaders?: HeadersInitType,
-  ctx?: unknown,
-): Headers {
-  void ctx;
-  return new Headers(existingHeaders);
-}
-
-/**
- * Browser stub: Run function and return result
- */
-export function runInContext<T>(parentContext: unknown, fn: () => T): T {
-  void parentContext;
-  return fn();
-}
-
-/**
- * Browser stub: Returns empty context object
- */
-export function getActiveContext(): unknown {
-  return {};
-}
-
-/**
- * Browser stub: Returns empty string
- */
-export function getTraceParent(): string {
-  return '';
-}
-
-/**
- * Browser stub: Returns empty string
- */
-export function getTraceState(): string {
-  return '';
-}
-
-/**
- * Browser stub: Returns undefined
- */
-export function getCurrentTraceId(): string | undefined {
-  return undefined;
-}
-
-/**
- * Browser stub: Returns undefined
- */
-export function getCurrentSpanId(): string | undefined {
-  return undefined;
-}

package/src/browser/debug-headers.ts

@@ -1,19 +0,0 @@
-/**
- * Browser stub for debug-headers module
- *
- * Debug headers are server-side only.
- * In browser, this returns pass-through middleware.
- */
-
-import type { MiddlewareHandler } from './middleware';
-
-/**
- * Browser stub: Returns pass-through middleware
- */
-export function debugHeadersMiddleware<
-  TContext = unknown,
->(): MiddlewareHandler<TContext> {
-  return async function noopMiddleware(opts) {
-    return opts.next();
-  };
-}

package/src/browser/error-reporting.ts

@@ -1,64 +0,0 @@
-/**
- * Browser stub for error-reporting module
- *
- * Error reporting/collection only happens on the server.
- * In browser, these are no-op functions.
- */
-
-/**
- * Error entry structure (stub)
- */
-export interface ErrorEntry {
-  timestamp: string;
-  message: string;
-  stack?: string;
-  context?: Record<string, unknown>;
-}
-
-/**
- * Browser stub: No-op
- */
-export function reportError(
-  error: Error,
-  context?: Record<string, unknown>,
-): void {
-  void error;
-  void context;
-  // No-op in browser
-}
-
-/**
- * Browser stub: Returns empty array
- */
-export function getRecentErrors(limit?: number): ErrorEntry[] {
-  void limit;
-  return [];
-}
-
-/**
- * Browser stub: No-op
- */
-export function clearErrors(): void {
-  // No-op in browser
-}
-
-/**
- * Browser stub: Returns JSON Response with empty errors
- */
-export function createErrorReportingHandler(): () => Response {
-  return () => {
-    return Response.json({ errors: [], count: 0 });
-  };
-}
-
-/**
- * Browser stub: Returns function unchanged
- */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Intentional for type passthrough
-export function withErrorReporting<T extends (...args: any[]) => any>(
-  fn: T,
-  context?: Record<string, unknown>,
-): T {
-  void context;
-  return fn;
-}

package/src/browser/handlers.ts

@@ -1,23 +0,0 @@
-/**
- * Browser stub for handlers module
- *
- * Handler wrapping only applies on the server side.
- * In browser, we just return the handler unchanged.
- */
-
-import type { WrapStartHandlerConfig } from './types';
-
-/**
- * Handler type
- */
-export type StartHandler<T = unknown> = (request: Request) => Promise<T>;
-
-/**
- * Browser stub: Returns the handler unchanged
- */
-export function wrapStartHandler<T>(
-  config?: WrapStartHandlerConfig,
-): (handler: StartHandler<T>) => StartHandler<T> {
-  void config;
-  return (handler) => handler;
-}

package/src/browser/index.ts

@@ -1,66 +0,0 @@
-/**
- * Browser entry point for autotel-tanstack
- *
- * This module provides no-op stubs for all exports to prevent
- * @opentelemetry/api and autotel from being bundled in client code.
- */
-
-// Export types from types.ts (single source of truth)
-export type {
-  Attributes,
-  TanStackInstrumentationConfig,
-  TracingMiddlewareConfig,
-  TraceServerFnConfig,
-  TraceLoaderConfig,
-  WrapStartHandlerConfig,
-} from './types';
-export { DEFAULT_CONFIG, SPAN_ATTRIBUTES } from './types';
-
-// Export functions from each module (excluding their duplicate type exports)
-export { traceLoader, traceBeforeLoad, createTracedRoute } from './loaders';
-export { traceServerFn, createTracedServerFnFactory } from './server-functions';
-export {
-  createTracingMiddleware,
-  tracingMiddleware,
-  functionTracingMiddleware,
-  createTracingServerHandler,
-  type MiddlewareHandler,
-} from './middleware';
-export {
-  extractContextFromRequest,
-  injectContextToHeaders,
-  createTracedHeaders,
-  runInContext,
-  getActiveContext,
-  getTraceParent,
-  getTraceState,
-  getCurrentTraceId,
-  getCurrentSpanId,
-  type HeadersInitType,
-} from './context';
-export { wrapStartHandler, type StartHandler } from './handlers';
-export {
-  getMetrics,
-  recordTiming,
-  recordError,
-  resetMetrics,
-  metricsCollector,
-  createMetricsHandler,
-  type MetricsData,
-} from './metrics';
-export {
-  reportError,
-  getRecentErrors,
-  clearErrors,
-  createErrorReportingHandler,
-  withErrorReporting,
-  type ErrorEntry,
-} from './error-reporting';
-export { debugHeadersMiddleware } from './debug-headers';
-export {
-  createTestCollector,
-  assertSpanCreated,
-  assertSpanHasAttribute,
-  type TestSpan,
-  type TestCollector,
-} from './testing';

package/src/browser/loaders.ts

@@ -1,62 +0,0 @@
-/**
- * Browser stub for loaders module
- *
- * In browser environments, these functions are no-ops that just call the
- * original functions without any tracing overhead.
- */
-
-import type { TraceLoaderConfig } from './types';
-
-/**
- * Loader context type (compatible with TanStack router loader context)
- */
-interface LoaderContext {
-  params?: Record<string, string>;
-  route?: {
-    id?: string;
-  };
-  [key: string]: unknown;
-}
-
-/**
- * Browser stub: Returns the loader function unchanged
- */
-export function traceLoader<
-  T extends (context: LoaderContext) => Promise<unknown>,
->(loaderFn: T, config?: TraceLoaderConfig): T {
-  void config;
-  return loaderFn;
-}
-
-/**
- * Browser stub: Returns the beforeLoad function unchanged
- */
-export function traceBeforeLoad<
-  T extends (context: LoaderContext) => Promise<unknown>,
->(beforeLoadFn: T, config?: TraceLoaderConfig): T {
-  void config;
-  return beforeLoadFn;
-}
-
-/**
- * Browser stub: Returns object with pass-through wrappers
- */
-export function createTracedRoute(
-  routeId: string,
-  config?: Omit<TraceLoaderConfig, 'name'>,
-) {
-  void routeId;
-  void config;
-  return {
-    loader<T extends (context: LoaderContext) => Promise<unknown>>(
-      loaderFn: T,
-    ): T {
-      return loaderFn;
-    },
-    beforeLoad<T extends (context: LoaderContext) => Promise<unknown>>(
-      beforeLoadFn: T,
-    ): T {
-      return beforeLoadFn;
-    },
-  };
-}

package/src/browser/metrics.ts

@@ -1,86 +0,0 @@
-/**
- * Browser stub for metrics module
- *
- * Metrics collection only happens on the server.
- * In browser, these are no-op functions.
- */
-
-/**
- * Metrics data structure (stub)
- */
-export interface MetricsData {
-  requestCount: number;
-  errorCount: number;
-  avgLatency: number;
-  p50Latency: number;
-  p95Latency: number;
-  p99Latency: number;
-  requestsPerSecond: number;
-  endpoints: Record<
-    string,
-    {
-      count: number;
-      errors: number;
-      avgLatency: number;
-    }
-  >;
-}
-
-/**
- * Browser stub: Returns empty metrics
- */
-export function getMetrics(): MetricsData {
-  return {
-    requestCount: 0,
-    errorCount: 0,
-    avgLatency: 0,
-    p50Latency: 0,
-    p95Latency: 0,
-    p99Latency: 0,
-    requestsPerSecond: 0,
-    endpoints: {},
-  };
-}
-
-/**
- * Browser stub: No-op
- */
-export function recordTiming(name: string, durationMs: number): void {
-  void name;
-  void durationMs;
-  // No-op in browser
-}
-
-/**
- * Browser stub: No-op
- */
-export function recordError(name: string): void {
-  void name;
-  // No-op in browser
-}
-
-/**
- * Browser stub: No-op
- */
-export function resetMetrics(): void {
-  // No-op in browser
-}
-
-/**
- * Browser stub: No-op metrics collector
- */
-export const metricsCollector = {
-  recordTiming: recordTiming,
-  recordError: recordError,
-  getMetrics: getMetrics,
-  reset: resetMetrics,
-};
-
-/**
- * Browser stub: Returns JSON Response with empty metrics
- */
-export function createMetricsHandler(): () => Response {
-  return () => {
-    return Response.json(getMetrics());
-  };
-}

package/src/browser/middleware.ts

@@ -1,77 +0,0 @@
-/**
- * Browser stub for middleware module
- *
- * In browser environments, these functions return pass-through middleware
- * that just calls next() without any tracing overhead.
- */
-
-import type { TracingMiddlewareConfig } from './types';
-
-/**
- * Generic middleware handler type
- */
-export interface MiddlewareHandler<TContext = unknown> {
-  (opts: {
-    next: (ctx?: Partial<TContext>) => Promise<TContext>;
-    context: TContext;
-    request?: Request;
-    pathname?: string;
-    data?: unknown;
-    method?: string;
-    filename?: string;
-    functionId?: string;
-    signal?: AbortSignal;
-  }): Promise<TContext>;
-}
-
-/**
- * Browser stub: Returns pass-through middleware
- */
-export function createTracingMiddleware<TContext = unknown>(
-  config?: TracingMiddlewareConfig,
-): MiddlewareHandler<TContext> {
-  void config;
-  return async function noopMiddleware(opts) {
-    return opts.next();
-  };
-}
-
-/**
- * Browser stub: Returns pass-through middleware
- */
-export function tracingMiddleware<TContext = unknown>(
-  config?: TracingMiddlewareConfig,
-): MiddlewareHandler<TContext> {
-  void config;
-  return async function noopMiddleware(opts) {
-    return opts.next();
-  };
-}
-
-/**
- * Browser stub: Returns pass-through middleware
- */
-export function functionTracingMiddleware<TContext = unknown>(
-  config?: Omit<TracingMiddlewareConfig, 'type'>,
-): MiddlewareHandler<TContext> {
-  void config;
-  return async function noopMiddleware(opts) {
-    return opts.next();
-  };
-}
-
-/**
- * Browser stub: Returns pass-through handler for createMiddleware().server()
- */
-export function createTracingServerHandler<TContext = unknown>(
-  config?: TracingMiddlewareConfig,
-): (opts: {
-  next: (ctx?: Partial<TContext>) => Promise<TContext>;
-  context: TContext;
-  request?: Request;
-}) => Promise<TContext> {
-  void config;
-  return async function noopHandler(opts) {
-    return opts.next();
-  };
-}