autotel 2.24.1 → 2.25.1

package/dist/workflow-distributed.cjs

@@ -1,12 +1,12 @@
 'use strict';
 
 var chunkINJD3G4K_cjs = require('./chunk-INJD3G4K.cjs');
-var chunkAXNPD6XW_cjs = require('./chunk-AXNPD6XW.cjs');
+var chunk7ZYUFMWU_cjs = require('./chunk-7ZYUFMWU.cjs');
 require('./chunk-W4EUTSB2.cjs');
 require('./chunk-GML3FBOT.cjs');
 require('./chunk-D5LMF53P.cjs');
 require('./chunk-XRKAL7WJ.cjs');
-require('./chunk-PDRIJURL.cjs');
+require('./chunk-GIND746I.cjs');
 require('./chunk-6YIDHH2S.cjs');
 require('./chunk-GVLK7YUU.cjs');
 require('./chunk-ZNMBW67B.cjs');
@@ -59,7 +59,7 @@ var WorkflowBaggage = chunkINJD3G4K_cjs.createSafeBaggageSchema(workflowBaggageF
 function traceDistributedWorkflow(config) {
   const spanName = `workflow.${config.name}`;
   return (fnFactory) => {
-    return chunkAXNPD6XW_cjs.trace(
+    return chunk7ZYUFMWU_cjs.trace(
       { name: spanName, spanKind: api.SpanKind.INTERNAL },
       (baseCtx) => {
         return async (...args) => {
@@ -158,7 +158,7 @@ function traceDistributedWorkflow(config) {
 function traceDistributedStep(config) {
   const spanName = `workflow.step.${config.name}`;
   return (fnFactory) => {
-    return chunkAXNPD6XW_cjs.trace(
+    return chunk7ZYUFMWU_cjs.trace(
       { name: spanName, spanKind: api.SpanKind.INTERNAL },
       (baseCtx) => {
         return async (...args) => {

package/dist/workflow-distributed.js

@@ -1,10 +1,10 @@
 import { createSafeBaggageSchema } from './chunk-4IFSYQVX.js';
-import { trace } from './chunk-2D74YR4Z.js';
+import { trace } from './chunk-JDV3LYOI.js';
 import './chunk-SR35DG5A.js';
 import './chunk-B3ZHLLMP.js';
 import './chunk-WD4RP6IV.js';
 import './chunk-USSL3D6L.js';
-import './chunk-47KW5CV7.js';
+import './chunk-7RV6L24E.js';
 import './chunk-YTGF4L2C.js';
 import './chunk-X4RMFFMR.js';
 import './chunk-WGWSHJ2N.js';

package/dist/workflow.cjs

@@ -1,12 +1,12 @@
 'use strict';
 
-var chunkCLR5RQW4_cjs = require('./chunk-CLR5RQW4.cjs');
-require('./chunk-AXNPD6XW.cjs');
+var chunkDBUNRUDB_cjs = require('./chunk-DBUNRUDB.cjs');
+require('./chunk-7ZYUFMWU.cjs');
 require('./chunk-W4EUTSB2.cjs');
 require('./chunk-GML3FBOT.cjs');
 require('./chunk-D5LMF53P.cjs');
 require('./chunk-XRKAL7WJ.cjs');
-require('./chunk-PDRIJURL.cjs');
+require('./chunk-GIND746I.cjs');
 require('./chunk-6YIDHH2S.cjs');
 require('./chunk-GVLK7YUU.cjs');
 require('./chunk-ZNMBW67B.cjs');
@@ -25,19 +25,19 @@ require('./chunk-JEQ2X3Z6.cjs');
 
 Object.defineProperty(exports, "getCurrentWorkflowContext", {
   enumerable: true,
-  get: function () { return chunkCLR5RQW4_cjs.getCurrentWorkflowContext; }
+  get: function () { return chunkDBUNRUDB_cjs.getCurrentWorkflowContext; }
 });
 Object.defineProperty(exports, "isInWorkflow", {
   enumerable: true,
-  get: function () { return chunkCLR5RQW4_cjs.isInWorkflow; }
+  get: function () { return chunkDBUNRUDB_cjs.isInWorkflow; }
 });
 Object.defineProperty(exports, "traceStep", {
   enumerable: true,
-  get: function () { return chunkCLR5RQW4_cjs.traceStep; }
+  get: function () { return chunkDBUNRUDB_cjs.traceStep; }
 });
 Object.defineProperty(exports, "traceWorkflow", {
   enumerable: true,
-  get: function () { return chunkCLR5RQW4_cjs.traceWorkflow; }
+  get: function () { return chunkDBUNRUDB_cjs.traceWorkflow; }
 });
 //# sourceMappingURL=workflow.cjs.map
 //# sourceMappingURL=workflow.cjs.map

package/dist/workflow.js

@@ -1,10 +1,10 @@
-export { getCurrentWorkflowContext, isInWorkflow, traceStep, traceWorkflow } from './chunk-IJMW6CTV.js';
-import './chunk-2D74YR4Z.js';
+export { getCurrentWorkflowContext, isInWorkflow, traceStep, traceWorkflow } from './chunk-P5YCN2Z3.js';
+import './chunk-JDV3LYOI.js';
 import './chunk-SR35DG5A.js';
 import './chunk-B3ZHLLMP.js';
 import './chunk-WD4RP6IV.js';
 import './chunk-USSL3D6L.js';
-import './chunk-47KW5CV7.js';
+import './chunk-7RV6L24E.js';
 import './chunk-YTGF4L2C.js';
 import './chunk-X4RMFFMR.js';
 import './chunk-WGWSHJ2N.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autotel",
-  "version": "2.24.1",
+  "version": "2.25.1",
   "description": "Write Once, Observe Anywhere",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -139,11 +139,13 @@
     },
     "./register": {
       "types": "./dist/register.d.ts",
-      "import": "./dist/register.js"
+      "import": "./dist/register.js",
+      "require": "./dist/register.cjs"
     },
     "./auto": {
       "types": "./dist/auto.d.ts",
-      "import": "./dist/auto.js"
+      "import": "./dist/auto.js",
+      "require": "./dist/auto.cjs"
     },
     "./hook.mjs": {
       "import": "./src/hook.mjs"
@@ -257,7 +259,7 @@
     "@opentelemetry/sdk-node": "^0.213.0",
     "@opentelemetry/sdk-trace-base": "^2.6.0",
     "@opentelemetry/semantic-conventions": "^1.40.0",
-    "@tanstack/intent": "^0.0.19",
+    "@tanstack/intent": "^0.0.23",
     "import-in-the-middle": "^3.0.0"
   },
   "peerDependencies": {
@@ -274,7 +276,7 @@
     "@traceloop/node-server-sdk": "^0.22.8",
     "pino": "^10.3.1",
     "pino-pretty": "^13.1.3",
-    "yaml": "^2.8.2"
+    "yaml": "^2.8.3"
   },
   "peerDependenciesMeta": {
     "@opentelemetry/auto-instrumentations-node": {
@@ -334,13 +336,13 @@
     "@opentelemetry/resource-detector-gcp": "^0.48.0",
     "@opentelemetry/sdk-logs": "^0.213.0",
     "@opentelemetry/sdk-trace-node": "^2.6.0",
-    "@swc/core": "^1.15.18",
+    "@swc/core": "^1.15.21",
     "@total-typescript/ts-reset": "^0.6.1",
     "@total-typescript/tsconfig": "^1.0.4",
     "@types/eslint-config-prettier": "^6.11.3",
     "@types/node": "^25.5.0",
-    "@typescript-eslint/eslint-plugin": "^8.57.0",
-    "@typescript-eslint/parser": "^8.57.0",
+    "@typescript-eslint/eslint-plugin": "^8.57.1",
+    "@typescript-eslint/parser": "^8.57.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-unicorn": "^63.0.0",
     "pino": "^10.3.1",
@@ -349,13 +351,13 @@
     "tsup": "^8.5.1",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",
-    "typescript-eslint": "^8.57.0",
+    "typescript-eslint": "^8.57.1",
     "unplugin-swc": "^1.5.9",
     "vite-tsconfig-paths": "^6.1.1",
     "vitest": "^4.1.0",
     "vitest-mock-extended": "^3.1.0",
     "winston": "^3.19.0",
-    "yaml": "^2.8.2"
+    "yaml": "^2.8.3"
   },
   "repository": {
     "type": "git",

package/src/init.integrations.test.ts

@@ -20,18 +20,24 @@ const mockedModules = [
   '@opentelemetry/sdk-metrics',
 ];
 
-// Mock instrumentation classes with exact names from OpenTelemetry
-class MongoDBInstrumentation {
+// Mock instrumentation classes with exact names from OpenTelemetry.
+// NodeSDK.start() calls lifecycle hooks on each instrumentation instance.
+class MockInstrumentationBase {
   constructor(public config?: Record<string, unknown>) {}
-}
 
-class MongooseInstrumentation {
-  constructor(public config?: Record<string, unknown>) {}
+  setConfig(_config: Record<string, unknown>) {}
+  setTracerProvider(_provider: unknown) {}
+  setMeterProvider(_provider: unknown) {}
+  setLoggerProvider(_provider: unknown) {}
+  enable() {}
+  disable() {}
 }
 
-class HttpInstrumentation {
-  constructor(public config?: Record<string, unknown>) {}
-}
+class MongoDBInstrumentation extends MockInstrumentationBase {}
+
+class MongooseInstrumentation extends MockInstrumentationBase {}
+
+class HttpInstrumentation extends MockInstrumentationBase {}
 
 async function loadInitWithMocks() {
   const sdkInstances: SdkRecord[] = [];

package/src/init.openllmetry.test.ts

@@ -1,138 +1,36 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import type { NodeSDK } from '@opentelemetry/sdk-node';
-import { mock, mockDeep, type DeepMockProxy } from 'vitest-mock-extended';
-
-type SdkRecord = {
-  options: Record<string, unknown>;
-  instance: DeepMockProxy<NodeSDK>;
-};
-
-const mockedModules = [
-  '@opentelemetry/sdk-node',
-  '@opentelemetry/exporter-trace-otlp-http',
-  '@opentelemetry/exporter-metrics-otlp-http',
-  '@opentelemetry/sdk-metrics',
-  './node-require',
-  './node-require.ts',
-];
-
-// Track traceloop initialize calls globally
-const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
-
-// Store mock initialize function globally
-let globalMockInitialize: ReturnType<typeof vi.fn> | null = null;
-
-async function loadInitWithMocks() {
-  const sdkInstances: SdkRecord[] = [];
-
-  class MockNodeSDK {
-    constructor(options: Record<string, unknown>) {
-      const instance = mockDeep<NodeSDK>();
-      instance.start.mockImplementation(() => {});
-      instance.shutdown.mockResolvedValue();
-
-      (instance as any).getTracerProvider = vi.fn().mockReturnValue(mock());
-      sdkInstances.push({ options, instance });
-      return instance;
-    }
-  }
-
-  class MockOTLPTraceExporter {
-    options: Record<string, unknown>;
-
-    constructor(options: Record<string, unknown>) {
-      this.options = options;
-    }
-  }
-
-  class MockOTLPMetricExporter {
-    options: Record<string, unknown>;
-
-    constructor(options: Record<string, unknown>) {
-      this.options = options;
-    }
-  }
-
-  class MockPeriodicExportingMetricReader {
-    options: Record<string, unknown>;
-
-    constructor(options: Record<string, unknown>) {
-      this.options = options;
-    }
-  }
-
-  // Clear module cache first
-  vi.resetModules();
-
-  // Create mock function that captures calls and store it globally
-  globalMockInitialize = vi.fn((options?: Record<string, unknown>) => {
-    traceloopInitializeCalls.push(options || {});
-  });
-
-  const mockTraceloop = {
-    initialize: globalMockInitialize,
-    instrumentations: [{ name: 'openai' }, { name: 'langchain' }],
-  };
-
-  // Mock node-require to intercept safeRequire('@traceloop/node-server-sdk').
-  // vi.doMock on the traceloop module itself doesn't work because safeRequire
-  // uses native require() which bypasses vitest's module interception.
-  const nodeRequireMock = () => ({
-    safeRequire: vi.fn((id: string) => {
-      if (id === '@traceloop/node-server-sdk') {
-        return mockTraceloop;
-      }
-      return undefined;
-    }),
-    requireModule: vi.fn((id: string) => {
-      const err = new Error(`Cannot find module '${id}'`);
-      (err as NodeJS.ErrnoException).code = 'MODULE_NOT_FOUND';
-      throw err;
-    }),
-    nodeRequire: vi.fn(),
-  });
-
-  vi.doMock('./node-require', nodeRequireMock);
-  vi.doMock('./node-require.ts', nodeRequireMock);
-
-  vi.doMock('@opentelemetry/sdk-node', () => ({
-    NodeSDK: MockNodeSDK,
-  }));
-
-  vi.doMock('@opentelemetry/exporter-trace-otlp-http', () => ({
-    OTLPTraceExporter: MockOTLPTraceExporter,
-  }));
+import { afterEach, describe, expect, it, vi } from 'vitest';
 
-  vi.doMock('@opentelemetry/exporter-metrics-otlp-http', () => ({
-    OTLPMetricExporter: MockOTLPMetricExporter,
-  }));
+type InitModule = typeof import('./init');
 
-  vi.doMock('@opentelemetry/sdk-metrics', () => ({
-    PeriodicExportingMetricReader: MockPeriodicExportingMetricReader,
-  }));
+async function loadInitModule(): Promise<InitModule> {
+  vi.resetModules();
+  return import('./init');
+}
 
-  const mod = await import('./init');
+function createSdkFactory() {
+  const calls: Array<Record<string, unknown>> = [];
+  const getTracerProvider = vi.fn(() => ({ id: 'mock-tracer-provider' }));
+  const start = vi.fn();
+  const shutdown = vi.fn(async () => {});
 
   return {
-    init: mod.init,
-    getConfig: mod.getConfig,
-    sdkInstances,
-    traceloopInitializeCalls,
-    mockTraceloop,
+    calls,
+    getTracerProvider,
+    start,
+    shutdown,
+    sdkFactory: (options: Record<string, unknown>) => {
+      calls.push(options);
+      return {
+        start,
+        shutdown,
+        getTracerProvider,
+      } as never;
+    },
   };
 }
 
 describe('init() OpenLLMetry integration', () => {
-  beforeEach(() => {
-    vi.resetModules();
-    traceloopInitializeCalls.length = 0;
-    globalMockInitialize = null;
-  });
-
   afterEach(() => {
-    for (const mod of mockedModules) {
-      vi.doUnmock(mod);
-    }
     vi.clearAllMocks();
     delete process.env.AUTOTEL_METRICS;
     delete process.env.NODE_ENV;
@@ -140,29 +38,53 @@ describe('init() OpenLLMetry integration', () => {
   });
 
   it('should not initialize OpenLLMetry when disabled', async () => {
-    const { init, traceloopInitializeCalls } = await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
+
+    mod._setOptionalRequireForTesting(() => ({
+      initialize: (options?: Record<string, unknown>) =>
+        traceloopInitializeCalls.push(options ?? {}),
+    }));
 
-    init({ service: 'test-app' });
+    mod.init({ service: 'test-app', sdkFactory: sdk.sdkFactory });
 
     expect(traceloopInitializeCalls).toHaveLength(0);
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should initialize OpenLLMetry when enabled', async () => {
-    const { init, traceloopInitializeCalls } = await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
+
+    mod._setOptionalRequireForTesting(() => ({
+      initialize: (options?: Record<string, unknown>) =>
+        traceloopInitializeCalls.push(options ?? {}),
+    }));
 
-    init({
+    mod.init({
       service: 'test-app',
       openllmetry: { enabled: true },
+      sdkFactory: sdk.sdkFactory,
     });
 
     expect(traceloopInitializeCalls).toHaveLength(1);
     expect(traceloopInitializeCalls[0]).toBeDefined();
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should pass OpenLLMetry options to initialize', async () => {
-    const { init, traceloopInitializeCalls } = await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
+
+    mod._setOptionalRequireForTesting(() => ({
+      initialize: (options?: Record<string, unknown>) =>
+        traceloopInitializeCalls.push(options ?? {}),
+    }));
 
-    init({
+    mod.init({
       service: 'test-app',
       openllmetry: {
         enabled: true,
@@ -171,6 +93,7 @@ describe('init() OpenLLMetry integration', () => {
           apiKey: 'test-key',
         },
       },
+      sdkFactory: sdk.sdkFactory,
     });
 
     expect(traceloopInitializeCalls).toHaveLength(1);
@@ -178,112 +101,94 @@ describe('init() OpenLLMetry integration', () => {
       disableBatch: true,
       apiKey: 'test-key',
     });
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should reuse autotel tracer provider when OpenLLMetry is enabled', async () => {
-    const { init, traceloopInitializeCalls, sdkInstances } =
-      await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
 
-    init({
+    mod._setOptionalRequireForTesting(() => ({
+      initialize: (options?: Record<string, unknown>) =>
+        traceloopInitializeCalls.push(options ?? {}),
+    }));
+
+    mod.init({
       service: 'test-app',
       openllmetry: { enabled: true },
+      sdkFactory: sdk.sdkFactory,
     });
 
-    expect(sdkInstances).toHaveLength(1);
-    const sdkInstance = sdkInstances[0].instance;
-
     expect(traceloopInitializeCalls).toHaveLength(1);
     const callOptions = traceloopInitializeCalls[0];
     expect(callOptions).toBeDefined();
-    expect((sdkInstance as any).getTracerProvider).toHaveBeenCalled();
+    expect(sdk.getTracerProvider).toHaveBeenCalled();
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should add OpenLLMetry instrumentations when selectiveInstrumentation is false', async () => {
-    const { init, sdkInstances, mockTraceloop } = await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const mockTraceloop = {
+      initialize: vi.fn(),
+      instrumentations: [{ name: 'openai' }, { name: 'langchain' }],
+    };
 
-    init({
+    mod._setOptionalRequireForTesting(() => mockTraceloop);
+
+    mod.init({
       service: 'test-app',
       openllmetry: { enabled: true },
       autoInstrumentations: false,
+      sdkFactory: sdk.sdkFactory,
     });
 
-    const options = sdkInstances.at(-1)?.options as Record<string, unknown>;
+    const options = sdk.calls.at(-1) as Record<string, unknown>;
     const instrumentations = options.instrumentations as unknown[];
 
     expect(instrumentations).toBeDefined();
     expect(mockTraceloop.instrumentations).toBeDefined();
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should handle missing @traceloop/node-server-sdk gracefully', async () => {
-    vi.resetModules();
-
-    // Mock node-require to return undefined for traceloop (simulating not installed)
-    const nodeRequireMock = () => ({
-      safeRequire: vi.fn(() => undefined),
-      requireModule: vi.fn((id: string) => {
-        const err = new Error(`Cannot find module '${id}'`);
-        (err as NodeJS.ErrnoException).code = 'MODULE_NOT_FOUND';
-        throw err;
-      }),
-      nodeRequire: vi.fn(),
-    });
-
-    vi.doMock('./node-require', nodeRequireMock);
-    vi.doMock('./node-require.ts', nodeRequireMock);
-
-    vi.doMock('@opentelemetry/sdk-node', () => ({
-      NodeSDK: class {
-        constructor() {
-          const instance = mockDeep<NodeSDK>();
-          instance.start.mockImplementation(() => {});
-          instance.shutdown.mockResolvedValue();
-          return instance;
-        }
-      },
-    }));
-
-    vi.doMock('@opentelemetry/exporter-trace-otlp-http', () => ({
-      OTLPTraceExporter: class {
-        constructor() {}
-      },
-    }));
-
-    vi.doMock('@opentelemetry/exporter-metrics-otlp-http', () => ({
-      OTLPMetricExporter: class {
-        constructor() {}
-      },
-    }));
-
-    vi.doMock('@opentelemetry/sdk-metrics', () => ({
-      PeriodicExportingMetricReader: class {
-        constructor() {}
-      },
-    }));
-
-    const { init } = await import('./init');
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    mod._setOptionalRequireForTesting(() => undefined);
 
     expect(() => {
-      init({
+      mod.init({
         service: 'test-app',
         openllmetry: { enabled: true },
+        sdkFactory: sdk.sdkFactory,
       });
     }).not.toThrow();
+    mod._resetOptionalRequireForTesting();
   });
 
   it('should initialize OpenLLMetry after SDK start', async () => {
-    const { init, sdkInstances, traceloopInitializeCalls } =
-      await loadInitWithMocks();
+    const mod = await loadInitModule();
+    const sdk = createSdkFactory();
+    const traceloopInitializeCalls: Array<Record<string, unknown>> = [];
+
+    mod._setOptionalRequireForTesting(() => ({
+      initialize: (options?: Record<string, unknown>) =>
+        traceloopInitializeCalls.push(options ?? {}),
+    }));
 
-    init({
+    mod.init({
       service: 'test-app',
       openllmetry: { enabled: true },
+      sdkFactory: sdk.sdkFactory,
     });
 
     // Verify SDK started (synchronously in init)
-    expect(sdkInstances).toHaveLength(1);
-    expect(sdkInstances[0].instance.start).toHaveBeenCalled();
+    expect(sdk.calls).toHaveLength(1);
+    expect(sdk.start).toHaveBeenCalled();
 
     // Verify OpenLLMetry was initialized (synchronously via safeRequire)
     expect(traceloopInitializeCalls).toHaveLength(1);
+    mod._resetOptionalRequireForTesting();
   });
 });

package/src/init.ts

@@ -1022,6 +1022,7 @@ let logger: Logger = silentLogger; // Silent by default - no spam
 let validationConfig: Partial<ValidationConfig> | null = null;
 let eventsConfig: EventsConfig | null = null;
 let _stringRedactor: StringRedactor | null = null;
+let _optionalRequire: typeof safeRequire = safeRequire;
 
 /**
  * Resolve metrics flag with env var override support
@@ -1471,7 +1472,7 @@ export function init(cfg: AutotelConfig): void {
 
   // Initialize OpenLLMetry if enabled (after SDK starts to reuse tracer provider)
   if (mergedConfig.openllmetry?.enabled) {
-    const traceloop = safeRequire<{
+    const traceloop = _optionalRequire<{
       initialize?: (options?: Record<string, unknown>) => void;
     }>('@traceloop/node-server-sdk');
 
@@ -1833,6 +1834,22 @@ export function getStringRedactor(): StringRedactor | null {
   return _stringRedactor;
 }
 
+/**
+ * @internal Override optional require for deterministic tests.
+ */
+export function _setOptionalRequireForTesting(
+  loader: typeof safeRequire,
+): void {
+  _optionalRequire = loader;
+}
+
+/**
+ * @internal Reset optional require override.
+ */
+export function _resetOptionalRequireForTesting(): void {
+  _optionalRequire = safeRequire;
+}
+
 /**
  * Get SDK instance (for shutdown)
  */

package/src/test-span-collector.ts

@@ -124,7 +124,7 @@ function isSerializable(v: unknown): v is SerializableValue {
   return false;
 }
 
-function serializeSpan(span: ReadableSpan): SerializedSpan {
+export function serializeSpan(span: ReadableSpan): SerializedSpan {
   const attrs: Record<string, SerializableValue> = {};
   for (const [k, v] of Object.entries(span.attributes)) {
     if (isSerializable(v)) attrs[k] = v;