@vvlad1973/simple-logger 2.2.0 → 2.3.0

package/src/helpers/helpers.ts

@@ -92,19 +92,86 @@ export function colorizeLevel(level: LoggerLevel, label: string): string {
 
 /**
  * Prepares the log call by rearranging the provided arguments into a standardized format.
+ * For external loggers (like pino), ensures context object comes BEFORE message string.
  *
  * @param {unknown[]} args - The array of arguments to prepare for the log call.
  * @return {PreparedLogCall} The prepared log call arguments, or null if the input is invalid.
  */
 export function prepareLogCall(args: unknown[]): PreparedLogCall {
-  if (typeof args[0] === 'string') {
-    return [args[0], ...args.slice(1)];
-  }
+  // Case 1: First arg is object (context) - standard pino format
   if (typeof args[0] === 'object' && args[0] !== null) {
     const msg = typeof args[1] === 'string' ? args[1] : undefined;
     const rest = args.slice(msg ? 2 : 1);
     return [args[0], msg, ...rest];
   }
+
+  // Case 2: First arg is string (message)
+  if (typeof args[0] === 'string') {
+    const msg = args[0];
+    // Check if second arg is context object - if so, swap them for pino
+    if (typeof args[1] === 'object' && args[1] !== null && !Array.isArray(args[1])) {
+      const context = args[1];
+      const rest = args.slice(2);
+      return [context, msg, ...rest];
+    }
+    // Otherwise keep as is (message only or message with non-object args)
+    return [msg, ...args.slice(1)];
+  }
+
   return null;
 }
 
+/**
+ * Safely wraps a value for logging to ensure it's a proper context object
+ *
+ * Use this helper ONLY for single values of unknown type that may be:
+ * - undefined/null
+ * - primitives (string, number, boolean)
+ * - Error objects (will be wrapped as an object with err property)
+ *
+ * DO NOT use for:
+ * - Composite objects with multiple fields (they're already safe)
+ * - Objects you construct yourself (use them directly)
+ *
+ * @param value - Value to wrap (can be Error, primitive, undefined, or object)
+ * @returns Safe object for logging or undefined if value is nullish
+ *
+ * @example
+ * ```typescript
+ * // Use for unknown single values
+ * catch (error) {
+ *   logger.error(safeLogContext(error), 'Failed');
+ * }
+ *
+ * catch (reason) {
+ *   logger.error(safeLogContext(reason), 'Rejected');
+ * }
+ *
+ * // DO NOT use for composite objects
+ * logger.error({ err, userId }, 'Failed'); // Direct, no helper needed
+ * ```
+ */
+export function safeLogContext(value: unknown): Record<string, unknown> {
+  // Nullish values - return empty record
+  if (value === null || value === undefined) {
+    return {} as Record<string, unknown>;
+  }
+
+  // Error objects - wrap in standard "err" field
+  if (value instanceof Error) {
+    return { err: value };
+  }
+
+  // Already an object (not array) - validate it's a plain object
+  if (typeof value === 'object' && !Array.isArray(value)) {
+    // Use type guard to safely return the object
+    if (isPlainObject(value)) {
+      return value;
+    }
+    // For non-plain objects (class instances, etc), wrap them
+    return { value };
+  }
+
+  // Primitives or arrays - wrap in "value" field
+  return { value };
+}

package/src/index.ts

@@ -4,4 +4,5 @@ export default SimpleLogger;
 export { SimpleLogger };
 
 export type * from './types/simple-logger.types.js';
-export * from './constants/constants.js';
+export * from './constants/constants.js';
+export * from './helpers/helpers.js'

package/tsconfig.json

@@ -11,7 +11,8 @@
     "sourceMap": true,
     "resolveJsonModule": true,
     "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true
+    "forceConsistentCasingInFileNames": true,
+    "types": ["node", "vitest/globals"]
   },
   "include": [
     "src/**/*"

package/vitest.config.ts

@@ -12,6 +12,7 @@ export default defineConfig({
     include: ['src/**/*.test.ts'],
     exclude: ['node_modules/**', 'dist/**', 'docs/**'],
     coverage: {
+      enabled: true,
       provider: 'v8',
       reporter: ['text', 'json', 'html', 'lcov'],
       include: ['src/**/*.ts'],
@@ -22,7 +23,6 @@ export default defineConfig({
         '**/*.test.ts',
         '**/*.config.ts',
         '**/*.config.js',
-        'src/__test__/**'
       ],
       thresholds: {
         lines: 80,