@mhmdhammoud/meritt-utils 1.5.7 → 1.5.8

package/dist/__tests__/logger.test.js

@@ -109,7 +109,7 @@ describe('route and format logs', () => {
             key1: 'val1',
         }));
     });
-    test('remap reserved elastic field names dynamically', () => {
+    test('remap reserved elastic field names and reduce objects to scalars', () => {
         //@ts-ignore
         jest.spyOn(pino_1.pino, 'destination').mockReturnValue(PINO_DESTINATION);
         //@ts-ignore
@@ -122,12 +122,10 @@ describe('route and format logs', () => {
                 _index: 'bad-index',
             },
         });
+        // Top-level fields use scalars only (avoids ES document_parsing_exception)
         expect(PINO.info).toHaveBeenCalledWith(expect.objectContaining({
             mongo_id: 'abc123',
-            nested: expect.objectContaining({
-                mongo_id: 'nested-1',
-                es_index: 'bad-index',
-            }),
+            nested: 'nested-1',
         }));
         expect(PINO.info).not.toHaveBeenCalledWith(expect.objectContaining({
             _id: expect.anything(),

package/dist/lib/logger.js

@@ -77,6 +77,37 @@ const isPlainObject = (v) => v !== null &&
     !Array.isArray(v) &&
     !(v instanceof Error) &&
     !(v instanceof Date);
+/**
+ * Reduces object values to scalars for top-level ES fields.
+ * ES text/keyword fields reject nested objects; use _id or truncated JSON.
+ */
+const toScalarForTopLevel = (value) => {
+    if (value === null || value === undefined)
+        return null;
+    if (typeof value === 'string' ||
+        typeof value === 'number' ||
+        typeof value === 'boolean')
+        return value;
+    if (value instanceof Date)
+        return value.toISOString();
+    if (isObjectIdLike(value))
+        return value.toHexString();
+    if (value instanceof Error)
+        return value.message;
+    if (isPlainObject(value) && '_id' in value) {
+        const id = value._id;
+        if (id != null) {
+            if (typeof id === 'string')
+                return id;
+            if (isObjectIdLike(id))
+                return id.toHexString();
+            if (typeof id === 'object' && id !== null && 'toString' in id)
+                return String(id.toString());
+        }
+    }
+    const str = typeof value === 'object' ? JSON.stringify(value) : String(value);
+    return str.length > 200 ? `${str.slice(0, 200)}...` : str;
+};
 /**
  * Recursively sanitize log values for Elasticsearch safety.
  * - Remaps reserved key names (e.g. `_id` -> `mongo_id`)
@@ -382,7 +413,7 @@ class Logger {
             ecs.trace_id = trace.traceId;
         }
         // Structured context: flatten single plain object as top-level fields
-        // Sanitize values to avoid ES mapping conflicts (e.g. Error objects → serializable shape)
+        // Use scalars only for top-level ES fields; text/keyword mappings reject nested objects
         let detail;
         if (args.length === 1 &&
             isPlainObject(args[0]) &&
@@ -390,11 +421,11 @@ class Logger {
             const obj = args[0];
             for (const [k, v] of Object.entries(obj)) {
                 const key = toSafeElasticFieldName(k);
-                ecs[key] = sanitizeForElastic(v);
+                ecs[key] = toScalarForTopLevel(v);
             }
         }
         else {
-            detail = isLocal ? args : JSON.stringify(args);
+            detail = isLocal ? args : JSON.stringify(sanitizeForElastic(args));
         }
         // Legacy fields for backward compatibility (component, code, msg)
         const base = {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@mhmdhammoud/meritt-utils",
-	"version": "1.5.7",
+	"version": "1.5.8",
 	"description": "",
 	"main": "./dist/index.js",
 	"private": false,

package/src/__tests__/logger.test.ts

@@ -90,7 +90,7 @@ describe('route and format logs', () => {
 		)
 	})
 
-	test('remap reserved elastic field names dynamically', () => {
+	test('remap reserved elastic field names and reduce objects to scalars', () => {
 		//@ts-ignore
 		jest.spyOn(pino, 'destination').mockReturnValue(PINO_DESTINATION)
 		//@ts-ignore
@@ -105,13 +105,11 @@ describe('route and format logs', () => {
 			},
 		})
 
+		// Top-level fields use scalars only (avoids ES document_parsing_exception)
 		expect(PINO.info).toHaveBeenCalledWith(
 			expect.objectContaining({
 				mongo_id: 'abc123',
-				nested: expect.objectContaining({
-					mongo_id: 'nested-1',
-					es_index: 'bad-index',
-				}),
+				nested: 'nested-1',
 			})
 		)
 		expect(PINO.info).not.toHaveBeenCalledWith(

package/src/lib/logger.ts

@@ -52,6 +52,36 @@ const isPlainObject = (v: unknown): v is Record<string, unknown> =>
 	!(v instanceof Error) &&
 	!(v instanceof Date)
 
+/**
+ * Reduces object values to scalars for top-level ES fields.
+ * ES text/keyword fields reject nested objects; use _id or truncated JSON.
+ */
+const toScalarForTopLevel = (
+	value: unknown
+): string | number | boolean | null => {
+	if (value === null || value === undefined) return null
+	if (
+		typeof value === 'string' ||
+		typeof value === 'number' ||
+		typeof value === 'boolean'
+	)
+		return value
+	if (value instanceof Date) return value.toISOString()
+	if (isObjectIdLike(value)) return value.toHexString()
+	if (value instanceof Error) return value.message
+	if (isPlainObject(value) && '_id' in value) {
+		const id = (value as { _id?: unknown })._id
+		if (id != null) {
+			if (typeof id === 'string') return id
+			if (isObjectIdLike(id)) return id.toHexString()
+			if (typeof id === 'object' && id !== null && 'toString' in id)
+				return String((id as { toString: () => string }).toString())
+		}
+	}
+	const str = typeof value === 'object' ? JSON.stringify(value) : String(value)
+	return str.length > 200 ? `${str.slice(0, 200)}...` : str
+}
+
 /**
  * Recursively sanitize log values for Elasticsearch safety.
  * - Remaps reserved key names (e.g. `_id` -> `mongo_id`)
@@ -462,7 +492,7 @@ class Logger {
 		}
 
 		// Structured context: flatten single plain object as top-level fields
-		// Sanitize values to avoid ES mapping conflicts (e.g. Error objects → serializable shape)
+		// Use scalars only for top-level ES fields; text/keyword mappings reject nested objects
 		let detail: unknown
 		if (
 			args.length === 1 &&
@@ -472,10 +502,10 @@ class Logger {
 			const obj = args[0]
 			for (const [k, v] of Object.entries(obj)) {
 				const key = toSafeElasticFieldName(k)
-				ecs[key] = sanitizeForElastic(v)
+				ecs[key] = toScalarForTopLevel(v)
 			}
 		} else {
-			detail = isLocal ? args : JSON.stringify(args)
+			detail = isLocal ? args : JSON.stringify(sanitizeForElastic(args))
 		}
 
 		// Legacy fields for backward compatibility (component, code, msg)