logixia 1.10.2 → 1.10.3

package/dist/index.mjs

@@ -1,5 +1,5 @@
-import { p as internalWarn, u as safeToString } from "./transport.manager-CaL4XuLD.mjs";
-import { A as redactObject, B as LogLevel, C as setTraceId, D as registerForShutdown, E as flushOnExit, F as isError, G as createExpressContextMiddleware, H as globalPluginRegistry, I as normalizeError, K as createFastifyContextHook, L as serializeError, M as getActiveOtelContext, N as getOtelMetaFields, O as resetShutdownHandlers, P as initOtelBridge, R as DEFAULT_LOG_COLORS, S as runWithTraceId, T as deregisterFromShutdown, U as usePlugin, V as PluginRegistry, W as LogixiaContext, _ as _setActiveContextKey, b as getCurrentTraceId, c as KafkaTraceInterceptor, d as __decorateMetadata, f as LogixiaLogger, g as TraceContext, h as TRACE_CONTEXT_KEY, i as WebSocketTraceInterceptor, j as disableOtelBridge, k as applyRedaction, l as LogixiaLoggerService, m as DEFAULT_TRACE_HEADERS, n as LOGIXIA_LOGGER_PREFIX, o as resolveResponseHeader, p as createLogger$1, r as LogixiaLoggerModule, s as __decorateParam, t as LOGIXIA_LOGGER_CONFIG, u as __decorate, v as createTraceMiddleware, w as traceStorage, x as getTraceContextKey, y as extractTraceId, z as DEFAULT_LOG_LEVELS } from "./logitron-logger.module-Bt_Jei1V.mjs";
+import { p as internalWarn, u as safeToString } from "./transport.manager-Cij_sA-b.mjs";
+import { A as redactObject, B as LogLevel, C as setTraceId, D as registerForShutdown, E as flushOnExit, F as isError, G as createExpressContextMiddleware, H as globalPluginRegistry, I as normalizeError, K as createFastifyContextHook, L as serializeError, M as getActiveOtelContext, N as getOtelMetaFields, O as resetShutdownHandlers, P as initOtelBridge, R as DEFAULT_LOG_COLORS, S as runWithTraceId, T as deregisterFromShutdown, U as usePlugin, V as PluginRegistry, W as LogixiaContext, _ as _setActiveContextKey, b as getCurrentTraceId, c as KafkaTraceInterceptor, d as __decorateMetadata, f as LogixiaLogger, g as TraceContext, h as TRACE_CONTEXT_KEY, i as WebSocketTraceInterceptor, j as disableOtelBridge, k as applyRedaction, l as LogixiaLoggerService, m as DEFAULT_TRACE_HEADERS, n as LOGIXIA_LOGGER_PREFIX, o as resolveResponseHeader, p as createLogger$1, r as LogixiaLoggerModule, s as __decorateParam, t as LOGIXIA_LOGGER_CONFIG, u as __decorate, v as createTraceMiddleware, w as traceStorage, x as getTraceContextKey, y as extractTraceId, z as DEFAULT_LOG_LEVELS } from "./logitron-logger.module-BBC9nO5q.mjs";
 import "./search-DOvSI-mb.mjs";
 import { Catch, Inject, Optional } from "@nestjs/common";
 
@@ -227,8 +227,11 @@ const InjectLogger = () => Inject(`${LOGIXIA_LOGGER_PREFIX}SERVICE`);
 /**
 * Method decorator that auto-logs entry, exit, duration, and errors.
 *
-* Works on both async and sync methods. Attaches to the logger found on the
-* class instance via a `logger` property (the conventional NestJS name).
+* Preserves the original method's sync/async contract: a synchronous method
+* stays synchronous (returns its value directly, with logs emitted
+* fire-and-forget), and an async method is awaited so exit/error logs reflect
+* the resolved result. Attaches to the logger found on the class instance via a
+* `logger` property (the conventional NestJS name).
 *
 * @example
 * ```ts
@@ -246,7 +249,23 @@ function LogMethod(options = {}) {
 		const className = ((_constructor = target.constructor) === null || _constructor === void 0 ? void 0 : _constructor.name) ?? "Unknown";
 		const label = options.label ?? `${className}.${methodName}`;
 		let _warnedNoLogger = false;
-		descriptor.value = async function(...args) {
+		const reportLogFailure = (phase, err) => {
+			process.stderr.write(`[logixia] @LogMethod(${label}) ${phase} log failed: ${String(err)}\n`);
+		};
+		const emit = (logger$1, phase, message, data) => {
+			const logFnRaw = logger$1[level];
+			const p = (typeof logFnRaw === "function" ? logFnRaw : logger$1.debug).bind(logger$1)(message, data);
+			if (p && typeof p.catch === "function") p.catch((e) => reportLogFailure(phase, e));
+		};
+		const emitError = (logger$1, error, start) => {
+			const err = error instanceof Error ? error : new Error(String(error));
+			const errLog = logger$1.error(err, {
+				method: label,
+				durationMs: Date.now() - start
+			});
+			if (errLog && typeof errLog.catch === "function") errLog.catch((e) => reportLogFailure("error", e));
+		};
+		descriptor.value = function(...args) {
 			const logger$1 = this.logger ?? LogixiaLoggerModule._globalLogger ?? void 0;
 			if (!logger$1 && !_warnedNoLogger) {
 				_warnedNoLogger = true;
@@ -255,36 +274,31 @@ function LogMethod(options = {}) {
 			const start = Date.now();
 			const entry = { method: label };
 			if (logArgs && args.length > 0) entry["args"] = args;
-			const reportLogFailure = (phase, err) => {
-				process.stderr.write(`[logixia] @LogMethod(${label}) ${phase} log failed: ${String(err)}\n`);
-			};
-			if (logger$1) {
-				const logFnRaw = logger$1[level];
-				await (typeof logFnRaw === "function" ? logFnRaw : logger$1.debug).bind(logger$1)(`→ ${label}`, entry).catch((e) => reportLogFailure("entry", e));
-			}
-			try {
-				const result = await originalMethod.apply(this, args);
+			if (logger$1) emit(logger$1, "entry", `→ ${label}`, entry);
+			const buildExit = (result$1) => {
 				const exit = {
 					method: label,
 					durationMs: Date.now() - start
 				};
-				if (logResult) exit["result"] = result;
-				if (logger$1) {
-					const logFnRaw = logger$1[level];
-					await (typeof logFnRaw === "function" ? logFnRaw : logger$1.debug).bind(logger$1)(`← ${label}`, exit).catch((e) => reportLogFailure("exit", e));
-				}
-				return result;
+				if (logResult) exit["result"] = result$1;
+				return exit;
+			};
+			let result;
+			try {
+				result = originalMethod.apply(this, args);
 			} catch (error) {
-				if (logger$1 && logErrors) {
-					const err = error instanceof Error ? error : new Error(String(error));
-					const errLog = logger$1.error(err, {
-						method: label,
-						durationMs: Date.now() - start
-					});
-					if (errLog !== void 0 && errLog !== null && typeof errLog.catch === "function") errLog.catch((e) => reportLogFailure("error", e));
-				}
+				if (logger$1 && logErrors) emitError(logger$1, error, start);
 				throw error;
 			}
+			if (result && typeof result.then === "function") return result.then((resolved) => {
+				if (logger$1) emit(logger$1, "exit", `← ${label}`, buildExit(resolved));
+				return resolved;
+			}, (error) => {
+				if (logger$1 && logErrors) emitError(logger$1, error, start);
+				throw error;
+			});
+			if (logger$1) emit(logger$1, "exit", `← ${label}`, buildExit(result));
+			return result;
 		};
 		return descriptor;
 	};
@@ -344,6 +358,21 @@ LogixiaExceptionFilter = __decorate([
 
 //#endregion
 //#region src/formatters/json.formatter.ts
+/**
+* Build a JSON.stringify replacer that replaces circular references with the
+* string '[Circular]' instead of throwing. A fresh replacer must be created per
+* stringify call because it holds per-serialization state (the seen set).
+*/
+function createCircularReplacer() {
+	const seen = /* @__PURE__ */ new WeakSet();
+	return function(_key, value) {
+		if (typeof value === "object" && value !== null) {
+			if (seen.has(value)) return "[Circular]";
+			seen.add(value);
+		}
+		return value;
+	};
+}
 var JsonFormatter = class {
 	constructor(options = {}) {
 		this.includeTimestamp = options.includeTimestamp ?? true;
@@ -371,7 +400,8 @@ var JsonFormatter = class {
 			hostname: process.env.HOSTNAME || "unknown",
 			version: process.version
 		};
-		return this.prettyPrint ? JSON.stringify(formatted, null, 2) : JSON.stringify(formatted);
+		const replacer = createCircularReplacer();
+		return this.prettyPrint ? JSON.stringify(formatted, replacer, 2) : JSON.stringify(formatted, replacer);
 	}
 	serializePayload(payload) {
 		const serialized = {};
@@ -474,11 +504,11 @@ var TextFormatter = class TextFormatter {
 				if (typeof value === "string") return `${key}="${value}"`;
 				if (typeof value === "number" || typeof value === "boolean") return `${key}=${value}`;
 				if (value instanceof Date) return `${key}=${value.toISOString()}`;
-				if (typeof value === "object") return `${key}=${JSON.stringify(value)}`;
+				if (typeof value === "object") return `${key}=${stripControls(safeToString(value))}`;
 				return `${key}=${String(value)}`;
 			}).join(" ");
 		} catch {
-			return JSON.stringify(payload);
+			return stripControls(safeToString(payload));
 		}
 	}
 	/**
@@ -598,7 +628,7 @@ function buildLabelKey(config, entry) {
 	const payload = entry.payload ?? {};
 	for (const name of labelNames) {
 		const raw = name === "level" ? entry.level : payload[name];
-		pairs[name] = raw !== void 0 && raw !== null ? String(raw) : "";
+		pairs[sanitizePromName(name)] = raw !== void 0 && raw !== null ? String(raw) : "";
 	}
 	return JSON.stringify(pairs);
 }
@@ -611,6 +641,18 @@ function escapeLabel(value) {
 	return value.replace(/\\/g, "\\\\").replace(/"/g, "\\\"").replace(/\n/g, "\\n");
 }
 /**
+* Coerce an arbitrary string into a valid Prometheus metric or label name
+* (`[a-zA-Z_][a-zA-Z0-9_]*`). Invalid characters become underscores and a
+* leading digit is prefixed with `_`. Without this, a user-supplied name like
+* `my-metric` or a label like `status code` would emit output Prometheus cannot
+* parse, breaking the ENTIRE scrape endpoint, not just that metric.
+*/
+function sanitizePromName(name) {
+	let safe = name.replace(/\W/g, "_");
+	if (safe.length > 0 && /\d/.test(safe[0])) safe = `_${safe}`;
+	return safe.length > 0 ? safe : "_";
+}
+/**
 * A logixia plugin that extracts Prometheus-compatible metrics from log entries.
 *
 * Implements `LogixiaPlugin` — pass directly to `logger.use()`:
@@ -684,7 +726,7 @@ var MetricsPlugin = class {
 	render() {
 		const lines = [];
 		for (const [rawName, config] of Object.entries(this.map)) {
-			const metricName = `logixia_${rawName}`;
+			const metricName = `logixia_${sanitizePromName(rawName)}`;
 			const state = this.metricState.get(rawName);
 			if (!state) continue;
 			const helpText = config.help ?? rawName.replace(/_/g, " ");