autotel 7.10.1 → 8.0.0

package/dist/{init-BvUtwZNx.js → init-CKHEf5g2.js}

@@ -8,7 +8,7 @@ import { PolicyLogRecordProcessor, policySpanFilter, setPolicies, watchPolicyFil
 import { SpanNameNormalizingProcessor } from "./span-name-normalizer.js";
 import { AttributeRedactingProcessor, REDACTOR_PRESETS, normalizeAttributeRedactorConfig } from "./attribute-redacting-processor.js";
 import { t as PrettyConsoleExporter } from "./pretty-console-exporter-DqKl_q9z.js";
-import { t as CanonicalLogLineProcessor } from "./canonical-log-line-processor-BQ6NeZAk.js";
+import { t as CanonicalLogLineProcessor } from "./canonical-log-line-processor-CntKMtpJ.js";
 import { n as subscribeChannel } from "./channel-BobQKsZd.js";
 import { resetMetrics } from "./metric.js";
 import { DiagConsoleLogger, DiagLogLevel, TraceFlags, context, createContextKey, diag, metrics, propagation, trace } from "@opentelemetry/api";
@@ -1959,15 +1959,15 @@ const silentLogger = {
 function wrapLogger(base, silent, minLevel) {
 	if (silent) return silentLogger;
 	const threshold = LOG_LEVELS[minLevel];
-	const wrap = (fn, level) => {
+	const wrap = (level) => {
 		if (LOG_LEVELS[level] < threshold) return (() => {});
-		return ((...args) => fn(...args));
+		return ((...args) => base[level](...args));
 	};
 	return {
-		debug: wrap(base.debug, "debug"),
-		info: wrap(base.info, "info"),
-		warn: wrap(base.warn, "warn"),
-		error: wrap(base.error, "error")
+		debug: wrap("debug"),
+		info: wrap("info"),
+		warn: wrap("warn"),
+		error: wrap("error")
 	};
 }
 
@@ -4137,20 +4137,68 @@ function toPackageName(name) {
 	return name.startsWith("@opentelemetry/instrumentation-") ? name : `@opentelemetry/instrumentation-${name}`;
 }
 /**
-* Detect if we're running in ESM mode
+* Whether the app's entry runs as ESM: its extension when it has an explicit
+* one (`.mjs`/`.cjs`, and `.mts`/`.cts` under tsx), else the cwd
+* package.json's `type`.
 */
-function isESMMode() {
+function isESMMode(entry = process.argv[1] ?? "") {
+	if (/\.c[jt]s$/.test(entry)) return false;
+	if (/\.m[jt]s$/.test(entry)) return true;
 	try {
 		const fs = requireModule("node:fs");
-		try {
-			return JSON.parse(fs.readFileSync(`${process.cwd()}/package.json`, "utf8")).type === "module";
-		} catch {
-			return false;
-		}
+		return JSON.parse(fs.readFileSync(`${process.cwd()}/package.json`, "utf8")).type === "module";
 	} catch {
 		return false;
 	}
 }
+/** Set by `autotel/register` so `init()` can tell the ESM hook is in place. */
+const REGISTER_FLAG = Symbol.for("autotel.register");
+/**
+* Whether the OTel ESM loader hook is registered: by `autotel/register`, or
+* by pointing node at a `hook.mjs` directly.
+*/
+function isEsmHookLoaded(flags = `${process.execArgv.join(" ")} ${process.env.NODE_OPTIONS ?? ""}`) {
+	return globalThis[REGISTER_FLAG] === true || flags.includes("hook.mjs");
+}
+const LOGGER_PACKAGES = [
+	"pino",
+	"winston",
+	"bunyan"
+];
+function loadedModulePaths() {
+	try {
+		const { createRequire } = requireModule("node:module");
+		return Object.entries(createRequire(`${process.cwd()}/`).cache).filter(([, mod]) => mod?.loaded).map(([path]) => path);
+	} catch {
+		return [];
+	}
+}
+/**
+* Names in the array form that loaded nothing: a typo, or a library
+* auto-instrumentations-node has no instrumentation for (`fastify`, `next`).
+* Upstream only reports these through `diag`, which is silent by default.
+* Names a manual instrumentation replaced are not missing.
+*/
+function missingInstrumentations(names, loaded, manualInstrumentationNames = /* @__PURE__ */ new Set()) {
+	const loadedNames = new Set(loaded.map((i) => i.instrumentationName));
+	const replaced = new Set([...manualInstrumentationNames].map((className) => INSTRUMENTATION_CLASS_TO_PACKAGE.get(className)));
+	return names.filter((name) => {
+		const packageName = toPackageName(name);
+		return !loadedNames.has(packageName) && !replaced.has(packageName);
+	});
+}
+/**
+* Logger packages the config instruments that were already loaded when
+* `init()` ran. Instrumentations patch a package as it loads, so these never
+* get trace context: the logger module was imported before `init()`.
+*/
+function loggersLoadedBeforeInit(integrations, loadedPaths = loadedModulePaths()) {
+	if (integrations === false) return [];
+	const requested = integrations === true ? LOGGER_PACKAGES : Array.isArray(integrations) ? integrations : Object.entries(integrations).filter(([, options]) => options.enabled !== false).map(([name]) => name);
+	const wanted = new Set(requested.map((name) => toPackageName(name)));
+	const paths = loadedPaths.map((path) => path.replaceAll("\\", "/"));
+	return LOGGER_PACKAGES.filter((name) => wanted.has(toPackageName(name)) && paths.some((path) => path.includes(`/node_modules/${name}/`)));
+}
 /**
 * Lazy-load auto-instrumentations (optional peer dependency)
 * Only loads when integrations config is truthy, avoiding ~40+ package imports at startup.
@@ -4179,11 +4227,12 @@ See: https://github.com/jagreehal/autotel#esm-setup`);
 * @internal
 */
 let _autoInstrumentationsLoader = null;
+const HTTP_PACKAGE = "@opentelemetry/instrumentation-http";
 /**
 * Get auto-instrumentations based on simple integration names
 * Excludes instrumentations that are manually provided to avoid conflicts
 */
-function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @__PURE__ */ new Set()) {
+function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @__PURE__ */ new Set(), ignoredServerPorts = []) {
 	if (integrations === false) return [];
 	const getNodeAutoInstrumentations = _autoInstrumentationsLoader ? _autoInstrumentationsLoader() : loadNodeAutoInstrumentations();
 	const exclusionConfig = {};
@@ -4198,7 +4247,18 @@ function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @
 		if (packageName in exclusionConfig) continue;
 		config[packageName] = options;
 	}
-	return getNodeAutoInstrumentations(withAutotelDefaults(config));
+	const http = config[HTTP_PACKAGE] ?? {};
+	if (ignoredServerPorts.length > 0 && http.enabled !== false) {
+		const userHook = http.ignoreIncomingRequestHook;
+		config[HTTP_PACKAGE] = {
+			...http,
+			ignoreIncomingRequestHook: (request) => ignoredServerPorts.includes(request.socket?.localPort ?? -1) || (userHook?.(request) ?? false)
+		};
+	}
+	const resolved = withAutotelDefaults(config);
+	if (!Array.isArray(integrations)) return getNodeAutoInstrumentations(resolved);
+	const listed = new Set(Object.keys(config).filter((name) => config[name]?.enabled === true));
+	return getNodeAutoInstrumentations(new Proxy(resolved, { get: (target, name) => typeof name === "string" && listed.has(name) ? target[name] : { enabled: false } }));
 }
 
 //#endregion
@@ -4363,6 +4423,10 @@ function init(cfg) {
 	const minLevel = mergedConfig.minLevel ?? "info";
 	const baseLogger = mergedConfig.logger || silentLogger;
 	logger = wrapLogger(baseLogger, silent, minLevel);
+	const setupWarn = (msg) => {
+		if (mergedConfig.logger) logger.warn({}, msg);
+		else if (!silent) console.warn(msg);
+	};
 	if (initialized) logger.warn({}, "[autotel] init() called again - last config wins. This may cause unexpected behavior.");
 	config = mergedConfig;
 	validationConfig = mergedConfig.validation || null;
@@ -4371,19 +4435,24 @@ function init(cfg) {
 	const version = mergedConfig.version || detectVersion();
 	const environment = mergedConfig.environment || process.env.NODE_ENV || "development";
 	const metricsEnabled = resolveMetricsFlag(mergedConfig.metrics);
-	const canonicalLogger = mergedConfig.canonicalLogLines?.logger ?? mergedConfig.logger;
+	const canonicalLogger = mergedConfig.canonicalLogLines?.logger;
 	const hasCanonicalLogger = Array.isArray(canonicalLogger) ? canonicalLogger.length > 0 : canonicalLogger !== void 0;
 	const canonicalUsesOtel = mergedConfig.canonicalLogLines?.enabled === true && (mergedConfig.canonicalLogLines.otel ?? !hasCanonicalLogger);
 	const logsEnabled = resolveLogsFlag(mergedConfig.logs ?? (canonicalUsesOtel ? true : "auto"));
+	let devtoolsPort;
 	if (devtoolsConfig.enabled && devtoolsConfig.embedded) {
 		const devtoolsModule = _optionalRequire("autotel-devtools");
 		if (devtoolsModule?.createDevtools) {
+			if (devtoolsConfig.port === 0) throw new Error("[autotel] devtools.embedded needs a fixed devtools.port; 0 is not supported");
 			const devtoolsInstance = devtoolsModule.createDevtools({
 				port: devtoolsConfig.port,
 				host: devtoolsConfig.host,
-				verbose: devtoolsConfig.verbose
+				verbose: devtoolsConfig.verbose,
+				maxPortTries: 1
 			});
+			devtoolsInstance.ready?.catch((error) => setupWarn(`[autotel] autotel-devtools could not start (${error instanceof Error ? error.message : String(error)}). Telemetry is still sent to ${endpoint}; free the port or set devtools.port.`));
 			_devtoolsClose = devtoolsInstance.close;
+			devtoolsPort = devtoolsInstance.port;
 			endpoint = `http://${devtoolsConfig.host}:${devtoolsInstance.port}`;
 			logger.info({}, `[autotel] autotel-devtools embedded server started at ${endpoint}`);
 		} else logger.warn({}, "[autotel] devtools.embedded requested but autotel-devtools is not installed. Falling back to endpoint-only mode.");
@@ -4496,17 +4565,22 @@ function init(cfg) {
 	if (mergedConfig.policies && logRecordProcessors) logRecordProcessors = logRecordProcessors.map((processor) => new PolicyLogRecordProcessor(processor));
 	let finalInstrumentations = mergedConfig.instrumentations ? [...mergedConfig.instrumentations] : [];
 	if (mergedConfig.autoInstrumentations !== void 0 && mergedConfig.autoInstrumentations !== false) {
-		if (isESMMode()) logger.info({}, "[autotel] ESM mode detected. For auto-instrumentation to work:\n  1. Install @opentelemetry/auto-instrumentations-node as a direct dependency\n  2. Import autotel/register FIRST in your instrumentation file\n  3. Use getNodeAutoInstrumentations() directly instead of autoInstrumentations\n  See: https://github.com/jagreehal/autotel#esm-setup");
+		if (isESMMode() && !isEsmHookLoaded()) setupWarn("[autotel] ESM app without the OTel loader hook: packages loaded with `import` (pino, express, ...) will not be instrumented. Start node with `--import autotel/register`. See: https://github.com/jagreehal/autotel#esm-setup");
+		for (const name of loggersLoadedBeforeInit(mergedConfig.autoInstrumentations)) setupWarn(`[autotel] ${name} was loaded before init(), so its logs get no trace context. Run init() first, and don't create your ${name} logger in the module that calls it.`);
 		try {
 			const manualInstrumentationNames = getInstrumentationNames(mergedConfig.instrumentations ?? []);
 			if (manualInstrumentationNames.size > 0) {
 				const manualNames = [...manualInstrumentationNames].join(", ");
 				logger.info({}, `[autotel] Detected manual instrumentations (${manualNames}). These will take precedence over auto-instrumentations. Tip: Set autoInstrumentations:false if you want full manual control, or remove manual configs to use auto-instrumentations.`);
 			}
-			const autoInstrumentations = getAutoInstrumentations(mergedConfig.autoInstrumentations, manualInstrumentationNames);
+			const autoInstrumentations = getAutoInstrumentations(mergedConfig.autoInstrumentations, manualInstrumentationNames, devtoolsPort === void 0 ? [] : [devtoolsPort]);
+			if (Array.isArray(mergedConfig.autoInstrumentations)) {
+				const missing = missingInstrumentations(mergedConfig.autoInstrumentations, autoInstrumentations ?? [], manualInstrumentationNames);
+				if (missing.length > 0) setupWarn(`[autotel] autoInstrumentations: no instrumentation loaded for ${missing.join(", ")}. Check the spelling; @opentelemetry/auto-instrumentations-node has none for some libraries (fastify, next).`);
+			}
 			if (autoInstrumentations && autoInstrumentations.length > 0) finalInstrumentations = [...finalInstrumentations, ...autoInstrumentations];
 		} catch (error) {
-			logger.warn({}, `[autotel] Failed to configure auto-instrumentations: ${error instanceof Error ? error.message : String(error)}`);
+			setupWarn(`[autotel] Failed to configure auto-instrumentations: ${error instanceof Error ? error.message : String(error)}`);
 		}
 	}
 	const autotelSampler = mergedConfig.sampler ?? (mergedConfig.sampling ? resolveSamplingPreset(mergedConfig.sampling) : void 0);

package/dist/{init-CHanKvOq.d.cts → init-D5k8hBRg.d.cts}

@@ -1,11 +1,11 @@
 import { a as AttributeRedactorPreset, r as AttributeRedactorConfig } from "./attribute-redacting-processor-CvBE0z1p.cjs";
-import { s as Logger } from "./logger-C9UT1aZ_.cjs";
-import { g as SamplingPreset, m as Sampler$1 } from "./sampling-Dud4XU2m.cjs";
+import { s as Logger } from "./logger-b0qgWFy7.cjs";
+import { g as SamplingPreset, m as Sampler$1 } from "./sampling-CLJmXMw0.cjs";
 import { o as EventSubscriber } from "./event-subscriber-DMACQA-1.cjs";
 import { r as SpanFilterPredicate } from "./filtering-span-processor-BYvLMiNz.cjs";
 import { r as Policy } from "./policy-C4QcZnqv.cjs";
 import { r as SpanNameNormalizerConfig } from "./span-name-normalizer-CN1-8C76.cjs";
-import { t as CanonicalLogLineOptions } from "./canonical-log-line-processor-DLeAQjgb.cjs";
+import { t as CanonicalLogLineOptions } from "./canonical-log-line-processor-C-RdV5af.cjs";
 import { BasicTracerProvider, ReadableSpan, SpanExporter, SpanProcessor } from "@opentelemetry/sdk-trace-base";
 import { Attributes, Sampler, TextMapPropagator } from "@opentelemetry/api";
 import { Resource, ResourceDetector } from "@opentelemetry/resources";
@@ -289,10 +289,11 @@ interface CanonicalLogLinesConfig {
   enabled: boolean;
   /**
    * Logger(s) to emit canonical log lines through. Pass an array to fan out to
-   * several. When omitted, lines go to the OTel Logs API instead.
+   * several. When omitted, lines go to the OTel Logs API instead. The
+   * top-level `logger` is for autotel's diagnostics and is not used here.
    *
-   * Note this falls back to the top-level `logger` on AutotelConfig, so
-   * setting that alone also diverts canonical log lines away from OTLP.
+   * Called as `logger[level](fields, message)`, pino's signature (bunyan's
+   * too). Winston takes `(message, meta)`, so wrap it.
    */
   logger?: Logger | Logger[];
   /**
@@ -488,7 +489,7 @@ interface AutotelConfig {
    * })
    * ```
    *
-   * @example Enable specific auto-instrumentations
+   * @example Enable only these (the array is an allowlist; include `'http'` for request spans)
    * ```typescript
    * init({
    *   service: 'my-app',
@@ -496,7 +497,7 @@ interface AutotelConfig {
    * })
    * ```
    *
-   * @example Configure specific auto-instrumentations
+   * @example Everything, with per-package overrides (unlisted packages still load)
    * ```typescript
    * init({
    *   service: 'my-app',
@@ -867,7 +868,8 @@ interface AutotelConfig {
    * Logger instance for internal autotel diagnostic messages
    *
    * This logger is used by autotel internally to log initialization, warnings,
-   * and debug information. Any logger with info/warn/error/debug methods works.
+   * and debug information. It is called pino-style, `(fields, message)`, so pino
+   * and bunyan work as is; wrap a `(message, meta)` logger such as winston.
    *
    * **For OTel instrumentation of your application logs**, use the `autoInstrumentations` option:
    * - `autoInstrumentations: ['pino']` - Injects traceId/spanId into Pino logs
@@ -875,18 +877,17 @@ interface AutotelConfig {
    *
    * Default: silent logger (no-op)
    *
-   * @example Pino with OTel instrumentation
+   * @example Pino with trace context (no `logger` needed)
    * ```typescript
-   * import pino from 'pino'
+   * // telemetry.ts - imported first, before anything that creates the logger
    * import { init } from 'autotel'
+   * init({ service: 'my-app', autoInstrumentations: ['pino'], logs: true })
    *
-   * const logger = pino({ level: 'info' })
-   * init({
-   *   service: 'my-app',
-   *   logger,                       // For autotel's internal logs
-   *   autoInstrumentations: ['pino'] // For OTel trace context in YOUR logs
-   * })
+   * // logger.ts - created after init(), so pino is patched as it loads
+   * import pino from 'pino'
+   * export const logger = pino()
    * ```
+   * Under ESM, start node with `--import autotel/register` too.
    *
    * @example Custom logger for autotel diagnostics
    * ```typescript
@@ -900,18 +901,10 @@ interface AutotelConfig {
    * ```
    *
    * @remarks
-   * This is also the fallback logger for canonical log lines, so setting it
-   * sends them here INSTEAD of the OTel Logs API — and therefore stops them
-   * reaching an OTLP logs backend such as Loki. On a platform whose log view
-   * reads stdout you usually want both, which is what
-   * `canonicalLogLines.otel` is for:
-   *
-   * ```typescript
-   * init({
-   *   service: 'my-app',
-   *   canonicalLogLines: { enabled: true, logger, otel: true },
-   * })
-   * ```
+   * Don't pass your app logger here to get trace context: creating it before
+   * `init()` loads its package before the instrumentation, so it is never
+   * patched. Canonical log lines do not use this logger either; set
+   * `canonicalLogLines.logger` (or `pretty`) for those.
    */
   logger?: Logger;
   /**

package/dist/{init-Cx1-ejLn.cjs → init-DF4YrpSB.cjs}

@@ -9,7 +9,7 @@ const require_policy = require('./policy.cjs');
 const require_span_name_normalizer = require('./span-name-normalizer.cjs');
 const require_attribute_redacting_processor = require('./attribute-redacting-processor.cjs');
 const require_pretty_console_exporter = require('./pretty-console-exporter-CMzlrRNg.cjs');
-const require_canonical_log_line_processor = require('./canonical-log-line-processor-BU03jJeA.cjs');
+const require_canonical_log_line_processor = require('./canonical-log-line-processor-CBRe-VsO.cjs');
 const require_channel = require('./channel-D0eaVqn3.cjs');
 const require_metric = require('./metric.cjs');
 let _opentelemetry_api = require("@opentelemetry/api");
@@ -1964,15 +1964,15 @@ const silentLogger = {
 function wrapLogger(base, silent, minLevel) {
 	if (silent) return silentLogger;
 	const threshold = LOG_LEVELS[minLevel];
-	const wrap = (fn, level) => {
+	const wrap = (level) => {
 		if (LOG_LEVELS[level] < threshold) return (() => {});
-		return ((...args) => fn(...args));
+		return ((...args) => base[level](...args));
 	};
 	return {
-		debug: wrap(base.debug, "debug"),
-		info: wrap(base.info, "info"),
-		warn: wrap(base.warn, "warn"),
-		error: wrap(base.error, "error")
+		debug: wrap("debug"),
+		info: wrap("info"),
+		warn: wrap("warn"),
+		error: wrap("error")
 	};
 }
 
@@ -4142,20 +4142,68 @@ function toPackageName(name) {
 	return name.startsWith("@opentelemetry/instrumentation-") ? name : `@opentelemetry/instrumentation-${name}`;
 }
 /**
-* Detect if we're running in ESM mode
+* Whether the app's entry runs as ESM: its extension when it has an explicit
+* one (`.mjs`/`.cjs`, and `.mts`/`.cts` under tsx), else the cwd
+* package.json's `type`.
 */
-function isESMMode() {
+function isESMMode(entry = process.argv[1] ?? "") {
+	if (/\.c[jt]s$/.test(entry)) return false;
+	if (/\.m[jt]s$/.test(entry)) return true;
 	try {
 		const fs = require_node_require.requireModule("node:fs");
-		try {
-			return JSON.parse(fs.readFileSync(`${process.cwd()}/package.json`, "utf8")).type === "module";
-		} catch {
-			return false;
-		}
+		return JSON.parse(fs.readFileSync(`${process.cwd()}/package.json`, "utf8")).type === "module";
 	} catch {
 		return false;
 	}
 }
+/** Set by `autotel/register` so `init()` can tell the ESM hook is in place. */
+const REGISTER_FLAG = Symbol.for("autotel.register");
+/**
+* Whether the OTel ESM loader hook is registered: by `autotel/register`, or
+* by pointing node at a `hook.mjs` directly.
+*/
+function isEsmHookLoaded(flags = `${process.execArgv.join(" ")} ${process.env.NODE_OPTIONS ?? ""}`) {
+	return globalThis[REGISTER_FLAG] === true || flags.includes("hook.mjs");
+}
+const LOGGER_PACKAGES = [
+	"pino",
+	"winston",
+	"bunyan"
+];
+function loadedModulePaths() {
+	try {
+		const { createRequire } = require_node_require.requireModule("node:module");
+		return Object.entries(createRequire(`${process.cwd()}/`).cache).filter(([, mod]) => mod?.loaded).map(([path]) => path);
+	} catch {
+		return [];
+	}
+}
+/**
+* Names in the array form that loaded nothing: a typo, or a library
+* auto-instrumentations-node has no instrumentation for (`fastify`, `next`).
+* Upstream only reports these through `diag`, which is silent by default.
+* Names a manual instrumentation replaced are not missing.
+*/
+function missingInstrumentations(names, loaded, manualInstrumentationNames = /* @__PURE__ */ new Set()) {
+	const loadedNames = new Set(loaded.map((i) => i.instrumentationName));
+	const replaced = new Set([...manualInstrumentationNames].map((className) => INSTRUMENTATION_CLASS_TO_PACKAGE.get(className)));
+	return names.filter((name) => {
+		const packageName = toPackageName(name);
+		return !loadedNames.has(packageName) && !replaced.has(packageName);
+	});
+}
+/**
+* Logger packages the config instruments that were already loaded when
+* `init()` ran. Instrumentations patch a package as it loads, so these never
+* get trace context: the logger module was imported before `init()`.
+*/
+function loggersLoadedBeforeInit(integrations, loadedPaths = loadedModulePaths()) {
+	if (integrations === false) return [];
+	const requested = integrations === true ? LOGGER_PACKAGES : Array.isArray(integrations) ? integrations : Object.entries(integrations).filter(([, options]) => options.enabled !== false).map(([name]) => name);
+	const wanted = new Set(requested.map((name) => toPackageName(name)));
+	const paths = loadedPaths.map((path) => path.replaceAll("\\", "/"));
+	return LOGGER_PACKAGES.filter((name) => wanted.has(toPackageName(name)) && paths.some((path) => path.includes(`/node_modules/${name}/`)));
+}
 /**
 * Lazy-load auto-instrumentations (optional peer dependency)
 * Only loads when integrations config is truthy, avoiding ~40+ package imports at startup.
@@ -4184,11 +4232,12 @@ See: https://github.com/jagreehal/autotel#esm-setup`);
 * @internal
 */
 let _autoInstrumentationsLoader = null;
+const HTTP_PACKAGE = "@opentelemetry/instrumentation-http";
 /**
 * Get auto-instrumentations based on simple integration names
 * Excludes instrumentations that are manually provided to avoid conflicts
 */
-function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @__PURE__ */ new Set()) {
+function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @__PURE__ */ new Set(), ignoredServerPorts = []) {
 	if (integrations === false) return [];
 	const getNodeAutoInstrumentations = _autoInstrumentationsLoader ? _autoInstrumentationsLoader() : loadNodeAutoInstrumentations();
 	const exclusionConfig = {};
@@ -4203,7 +4252,18 @@ function getAutoInstrumentations(integrations, manualInstrumentationNames = /* @
 		if (packageName in exclusionConfig) continue;
 		config[packageName] = options;
 	}
-	return getNodeAutoInstrumentations(withAutotelDefaults(config));
+	const http = config[HTTP_PACKAGE] ?? {};
+	if (ignoredServerPorts.length > 0 && http.enabled !== false) {
+		const userHook = http.ignoreIncomingRequestHook;
+		config[HTTP_PACKAGE] = {
+			...http,
+			ignoreIncomingRequestHook: (request) => ignoredServerPorts.includes(request.socket?.localPort ?? -1) || (userHook?.(request) ?? false)
+		};
+	}
+	const resolved = withAutotelDefaults(config);
+	if (!Array.isArray(integrations)) return getNodeAutoInstrumentations(resolved);
+	const listed = new Set(Object.keys(config).filter((name) => config[name]?.enabled === true));
+	return getNodeAutoInstrumentations(new Proxy(resolved, { get: (target, name) => typeof name === "string" && listed.has(name) ? target[name] : { enabled: false } }));
 }
 
 //#endregion
@@ -4368,6 +4428,10 @@ function init(cfg) {
 	const minLevel = mergedConfig.minLevel ?? "info";
 	const baseLogger = mergedConfig.logger || silentLogger;
 	logger = wrapLogger(baseLogger, silent, minLevel);
+	const setupWarn = (msg) => {
+		if (mergedConfig.logger) logger.warn({}, msg);
+		else if (!silent) console.warn(msg);
+	};
 	if (initialized) logger.warn({}, "[autotel] init() called again - last config wins. This may cause unexpected behavior.");
 	config = mergedConfig;
 	validationConfig = mergedConfig.validation || null;
@@ -4376,19 +4440,24 @@ function init(cfg) {
 	const version = mergedConfig.version || detectVersion();
 	const environment = mergedConfig.environment || process.env.NODE_ENV || "development";
 	const metricsEnabled = resolveMetricsFlag(mergedConfig.metrics);
-	const canonicalLogger = mergedConfig.canonicalLogLines?.logger ?? mergedConfig.logger;
+	const canonicalLogger = mergedConfig.canonicalLogLines?.logger;
 	const hasCanonicalLogger = Array.isArray(canonicalLogger) ? canonicalLogger.length > 0 : canonicalLogger !== void 0;
 	const canonicalUsesOtel = mergedConfig.canonicalLogLines?.enabled === true && (mergedConfig.canonicalLogLines.otel ?? !hasCanonicalLogger);
 	const logsEnabled = resolveLogsFlag(mergedConfig.logs ?? (canonicalUsesOtel ? true : "auto"));
+	let devtoolsPort;
 	if (devtoolsConfig.enabled && devtoolsConfig.embedded) {
 		const devtoolsModule = _optionalRequire("autotel-devtools");
 		if (devtoolsModule?.createDevtools) {
+			if (devtoolsConfig.port === 0) throw new Error("[autotel] devtools.embedded needs a fixed devtools.port; 0 is not supported");
 			const devtoolsInstance = devtoolsModule.createDevtools({
 				port: devtoolsConfig.port,
 				host: devtoolsConfig.host,
-				verbose: devtoolsConfig.verbose
+				verbose: devtoolsConfig.verbose,
+				maxPortTries: 1
 			});
+			devtoolsInstance.ready?.catch((error) => setupWarn(`[autotel] autotel-devtools could not start (${error instanceof Error ? error.message : String(error)}). Telemetry is still sent to ${endpoint}; free the port or set devtools.port.`));
 			_devtoolsClose = devtoolsInstance.close;
+			devtoolsPort = devtoolsInstance.port;
 			endpoint = `http://${devtoolsConfig.host}:${devtoolsInstance.port}`;
 			logger.info({}, `[autotel] autotel-devtools embedded server started at ${endpoint}`);
 		} else logger.warn({}, "[autotel] devtools.embedded requested but autotel-devtools is not installed. Falling back to endpoint-only mode.");
@@ -4501,17 +4570,22 @@ function init(cfg) {
 	if (mergedConfig.policies && logRecordProcessors) logRecordProcessors = logRecordProcessors.map((processor) => new require_policy.PolicyLogRecordProcessor(processor));
 	let finalInstrumentations = mergedConfig.instrumentations ? [...mergedConfig.instrumentations] : [];
 	if (mergedConfig.autoInstrumentations !== void 0 && mergedConfig.autoInstrumentations !== false) {
-		if (isESMMode()) logger.info({}, "[autotel] ESM mode detected. For auto-instrumentation to work:\n  1. Install @opentelemetry/auto-instrumentations-node as a direct dependency\n  2. Import autotel/register FIRST in your instrumentation file\n  3. Use getNodeAutoInstrumentations() directly instead of autoInstrumentations\n  See: https://github.com/jagreehal/autotel#esm-setup");
+		if (isESMMode() && !isEsmHookLoaded()) setupWarn("[autotel] ESM app without the OTel loader hook: packages loaded with `import` (pino, express, ...) will not be instrumented. Start node with `--import autotel/register`. See: https://github.com/jagreehal/autotel#esm-setup");
+		for (const name of loggersLoadedBeforeInit(mergedConfig.autoInstrumentations)) setupWarn(`[autotel] ${name} was loaded before init(), so its logs get no trace context. Run init() first, and don't create your ${name} logger in the module that calls it.`);
 		try {
 			const manualInstrumentationNames = getInstrumentationNames(mergedConfig.instrumentations ?? []);
 			if (manualInstrumentationNames.size > 0) {
 				const manualNames = [...manualInstrumentationNames].join(", ");
 				logger.info({}, `[autotel] Detected manual instrumentations (${manualNames}). These will take precedence over auto-instrumentations. Tip: Set autoInstrumentations:false if you want full manual control, or remove manual configs to use auto-instrumentations.`);
 			}
-			const autoInstrumentations = getAutoInstrumentations(mergedConfig.autoInstrumentations, manualInstrumentationNames);
+			const autoInstrumentations = getAutoInstrumentations(mergedConfig.autoInstrumentations, manualInstrumentationNames, devtoolsPort === void 0 ? [] : [devtoolsPort]);
+			if (Array.isArray(mergedConfig.autoInstrumentations)) {
+				const missing = missingInstrumentations(mergedConfig.autoInstrumentations, autoInstrumentations ?? [], manualInstrumentationNames);
+				if (missing.length > 0) setupWarn(`[autotel] autoInstrumentations: no instrumentation loaded for ${missing.join(", ")}. Check the spelling; @opentelemetry/auto-instrumentations-node has none for some libraries (fastify, next).`);
+			}
 			if (autoInstrumentations && autoInstrumentations.length > 0) finalInstrumentations = [...finalInstrumentations, ...autoInstrumentations];
 		} catch (error) {
-			logger.warn({}, `[autotel] Failed to configure auto-instrumentations: ${error instanceof Error ? error.message : String(error)}`);
+			setupWarn(`[autotel] Failed to configure auto-instrumentations: ${error instanceof Error ? error.message : String(error)}`);
 		}
 	}
 	const autotelSampler = mergedConfig.sampler ?? (mergedConfig.sampling ? resolveSamplingPreset(mergedConfig.sampling) : void 0);

package/dist/{init-Biyby8Br.d.ts → init-DMjyzzbM.d.ts}

@@ -1,11 +1,11 @@
 import { a as AttributeRedactorPreset, r as AttributeRedactorConfig } from "./attribute-redacting-processor-rkV_WL-k.js";
-import { s as Logger } from "./logger-C9UT1aZ_.js";
-import { g as SamplingPreset, m as Sampler$1 } from "./sampling-B_8orhSB.js";
+import { s as Logger } from "./logger-b0qgWFy7.js";
+import { g as SamplingPreset, m as Sampler$1 } from "./sampling-DBlX8Nv5.js";
 import { o as EventSubscriber } from "./event-subscriber-DMACQA-1.js";
 import { r as SpanFilterPredicate } from "./filtering-span-processor-BDGe1myy.js";
 import { r as Policy } from "./policy-DNw_0qUq.js";
 import { r as SpanNameNormalizerConfig } from "./span-name-normalizer-DHcg1bSP.js";
-import { t as CanonicalLogLineOptions } from "./canonical-log-line-processor-DDGzm-wi.js";
+import { t as CanonicalLogLineOptions } from "./canonical-log-line-processor-BaFqHO5l.js";
 import { Attributes, Sampler, TextMapPropagator } from "@opentelemetry/api";
 import { Instrumentation } from "@opentelemetry/instrumentation";
 import { Resource, ResourceDetector } from "@opentelemetry/resources";
@@ -289,10 +289,11 @@ interface CanonicalLogLinesConfig {
   enabled: boolean;
   /**
    * Logger(s) to emit canonical log lines through. Pass an array to fan out to
-   * several. When omitted, lines go to the OTel Logs API instead.
+   * several. When omitted, lines go to the OTel Logs API instead. The
+   * top-level `logger` is for autotel's diagnostics and is not used here.
    *
-   * Note this falls back to the top-level `logger` on AutotelConfig, so
-   * setting that alone also diverts canonical log lines away from OTLP.
+   * Called as `logger[level](fields, message)`, pino's signature (bunyan's
+   * too). Winston takes `(message, meta)`, so wrap it.
    */
   logger?: Logger | Logger[];
   /**
@@ -488,7 +489,7 @@ interface AutotelConfig {
    * })
    * ```
    *
-   * @example Enable specific auto-instrumentations
+   * @example Enable only these (the array is an allowlist; include `'http'` for request spans)
    * ```typescript
    * init({
    *   service: 'my-app',
@@ -496,7 +497,7 @@ interface AutotelConfig {
    * })
    * ```
    *
-   * @example Configure specific auto-instrumentations
+   * @example Everything, with per-package overrides (unlisted packages still load)
    * ```typescript
    * init({
    *   service: 'my-app',
@@ -867,7 +868,8 @@ interface AutotelConfig {
    * Logger instance for internal autotel diagnostic messages
    *
    * This logger is used by autotel internally to log initialization, warnings,
-   * and debug information. Any logger with info/warn/error/debug methods works.
+   * and debug information. It is called pino-style, `(fields, message)`, so pino
+   * and bunyan work as is; wrap a `(message, meta)` logger such as winston.
    *
    * **For OTel instrumentation of your application logs**, use the `autoInstrumentations` option:
    * - `autoInstrumentations: ['pino']` - Injects traceId/spanId into Pino logs
@@ -875,18 +877,17 @@ interface AutotelConfig {
    *
    * Default: silent logger (no-op)
    *
-   * @example Pino with OTel instrumentation
+   * @example Pino with trace context (no `logger` needed)
    * ```typescript
-   * import pino from 'pino'
+   * // telemetry.ts - imported first, before anything that creates the logger
    * import { init } from 'autotel'
+   * init({ service: 'my-app', autoInstrumentations: ['pino'], logs: true })
    *
-   * const logger = pino({ level: 'info' })
-   * init({
-   *   service: 'my-app',
-   *   logger,                       // For autotel's internal logs
-   *   autoInstrumentations: ['pino'] // For OTel trace context in YOUR logs
-   * })
+   * // logger.ts - created after init(), so pino is patched as it loads
+   * import pino from 'pino'
+   * export const logger = pino()
    * ```
+   * Under ESM, start node with `--import autotel/register` too.
    *
    * @example Custom logger for autotel diagnostics
    * ```typescript
@@ -900,18 +901,10 @@ interface AutotelConfig {
    * ```
    *
    * @remarks
-   * This is also the fallback logger for canonical log lines, so setting it
-   * sends them here INSTEAD of the OTel Logs API — and therefore stops them
-   * reaching an OTLP logs backend such as Loki. On a platform whose log view
-   * reads stdout you usually want both, which is what
-   * `canonicalLogLines.otel` is for:
-   *
-   * ```typescript
-   * init({
-   *   service: 'my-app',
-   *   canonicalLogLines: { enabled: true, logger, otel: true },
-   * })
-   * ```
+   * Don't pass your app logger here to get trace context: creating it before
+   * `init()` loads its package before the instrumentation, so it is never
+   * patched. Canonical log lines do not use this logger either; set
+   * `canonicalLogLines.logger` (or `pretty`) for those.
    */
   logger?: Logger;
   /**

package/dist/instrumentation.cjs

@@ -1,5 +1,5 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
-const require_init = require('./init-Cx1-ejLn.cjs');
+const require_init = require('./init-DF4YrpSB.cjs');
 const require_node_require = require('./node-require-BPsB09eC.cjs');
 let _opentelemetry_resources = require("@opentelemetry/resources");
 let _opentelemetry_exporter_logs_otlp_http = require("@opentelemetry/exporter-logs-otlp-http");

package/dist/instrumentation.js

@@ -1,4 +1,4 @@
-import { C as TailSamplingSpanProcessor, r as getLogger } from "./init-BvUtwZNx.js";
+import { C as TailSamplingSpanProcessor, r as getLogger } from "./init-CKHEf5g2.js";
 import { n as safeRequire, t as requireModule } from "./node-require-CkK_xzL-.js";
 import { detectResources, hostDetector, processDetector, resourceFromAttributes } from "@opentelemetry/resources";
 import { OTLPLogExporter } from "@opentelemetry/exporter-logs-otlp-http";