@adobe/spacecat-shared-utils 1.82.2 → 1.83.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-utils-v1.83.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.82.3...@adobe/spacecat-shared-utils-v1.83.0) (2025-12-10)
+
+
+### Features
+
+* Implement Structured (JSON) Logging for Spacecat Audits ([#1232](https://github.com/adobe/spacecat-shared/issues/1232)) ([7eae4d6](https://github.com/adobe/spacecat-shared/commit/7eae4d62fe9f0592f8124082fc66e9754803dd2b))
+
+# [@adobe/spacecat-shared-utils-v1.82.3](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.82.2...@adobe/spacecat-shared-utils-v1.82.3) (2025-12-10)
+
+
+### Bug Fixes
+
+* heading and toc mappers ([#1230](https://github.com/adobe/spacecat-shared/issues/1230)) ([f767fb3](https://github.com/adobe/spacecat-shared/commit/f767fb319c97ed64848a1c5fc5913693a52290d9))
+
 # [@adobe/spacecat-shared-utils-v1.82.2](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.82.1...@adobe/spacecat-shared-utils-v1.82.2) (2025-12-09)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-utils",
-  "version": "1.82.2",
+  "version": "1.83.0",
   "description": "Shared modules of the Spacecat Services - utils",
   "type": "module",
   "exports": {

package/src/locale-detect/locale-detect.js

@@ -41,6 +41,7 @@ export async function detectLocale(config) {
   const $ = cheerio.load(html);
 
   // Execute language detection indicators
+  /* c8 ignore start */
   for (const indicator of indicatorFuncs) {
     const results = indicator({ baseUrl: parsedBaseUrl, headers, $ });
     indicatorResults.push(...results);
@@ -56,6 +57,7 @@ export async function detectLocale(config) {
     }
     return acc;
   }, { region: {}, language: {} });
+  /* c8 ignore stop */
   const region = Object.keys(summary.region).length > 0 ? Object.keys(summary.region).sort((a, b) => summary.region[b] - summary.region[a])[0] : 'US';
   const language = Object.keys(summary.language).length > 0 ? Object.keys(summary.language).sort((a, b) => summary.language[b] - summary.language[a])[0] : 'en';
 

package/src/log-wrapper.js

@@ -13,64 +13,89 @@
 import { getTraceId } from './xray.js';
 
 /**
- * A higher-order function that wraps a given function and enhances logging by appending
- * a `jobId` and `traceId` to log messages when available. This improves traceability of logs
- * associated with specific jobs or processes.
- *
- * The wrapper checks if a `log` object exists in the `context` and whether the `message`
- * contains a `jobId`. It also extracts the AWS X-Ray trace ID if available. If found, log
- * methods (e.g., `info`, `error`, etc.) will prepend the `jobId` and/or `traceId` to all log
- * statements. All existing code using `context.log` will automatically include these markers.
+ * Check if a value is a plain object (not Array, not Error, not null, not other special objects)
+ * @param {*} value - The value to check
+ * @returns {boolean} - True if the value is a plain object
+ */
+function isPlainObject(value) {
+  return typeof value === 'object'
+    && value !== null
+    && !Array.isArray(value)
+    && !(value instanceof Error)
+    && value.constructor === Object;
+}
+
+/**
+ * A higher-order function that wraps a given function and enhances logging by converting
+ * all logs to JSON format and appending `jobId` and `traceId` to log messages when available.
  *
- * @param {function} fn - The original function to be wrapped, called with the provided
- * message and context after logging enhancement.
- * @returns {function(object, object): Promise<Response>} - A wrapped function that enhances
- * logging and returns the result of the original function.
+ * All log messages are automatically converted to structured JSON format:
+ * - String messages become: { message: "...", jobId: "...", traceId: "..." }
+ * - Object messages are merged with: { ...yourObject, jobId: "...", traceId: "..." }
  *
- * `context.log` will be enhanced in place to include `jobId` and/or `traceId` prefixed to all
- * log messages. No code changes needed - existing `context.log` calls work automatically.
+ * @param {function} fn - The original function to be wrapped
+ * @returns {function(object, object): Promise<Response>} - A wrapped function with JSON logging
  */
 export function logWrapper(fn) {
   return async (message, context) => {
     const { log } = context;
 
     if (log && !context.contextualLog) {
-      const markers = [];
+      const markers = {};
 
       // Extract jobId from message if available
       if (typeof message === 'object' && message !== null && 'jobId' in message) {
-        const { jobId } = message;
-        markers.push(`[jobId=${jobId}]`);
+        markers.jobId = message.jobId;
       }
 
       // Extract traceId from AWS X-Ray
       const traceId = getTraceId();
       if (traceId) {
-        markers.push(`[traceId=${traceId}]`);
+        markers.traceId = traceId;
       }
 
-      // If we have markers, enhance the log object directly
-      if (markers.length > 0) {
-        const markerString = markers.join(' ');
+      // Define log levels
+      const logLevels = ['info', 'error', 'debug', 'warn', 'trace', 'verbose', 'silly', 'fatal'];
 
-        // Define log levels
-        const logLevels = ['info', 'error', 'debug', 'warn', 'trace', 'verbose', 'silly', 'fatal'];
+      // Wrap all log methods to output structured JSON
+      context.log = logLevels.reduce((accumulator, level) => {
+        if (typeof log[level] === 'function') {
+          accumulator[level] = (...args) => {
+            // If first argument is a plain object, merge with markers
+            if (args.length > 0 && isPlainObject(args[0])) {
+              return log[level]({ ...markers, ...args[0] });
+            }
 
-        // Enhance context.log directly to include markers in all log statements
-        context.log = logLevels.reduce((accumulator, level) => {
-          if (typeof log[level] === 'function') {
-            accumulator[level] = (...args) => {
-              // If first argument is a string (format string), prepend the marker to it
-              if (args.length > 0 && typeof args[0] === 'string') {
-                const enhancedArgs = [`${markerString} ${args[0]}`, ...args.slice(1)];
-                return log[level](...enhancedArgs);
+            // If first argument is a string, convert to structured format
+            if (args.length > 0 && typeof args[0] === 'string') {
+              const logObject = {
+                ...markers,
+                message: args[0],
+              };
+
+              // If second argument is a plain object, merge it into the log object
+              if (args.length > 1 && isPlainObject(args[1])) {
+                Object.assign(logObject, args[1]);
+
+                // If there are more arguments after the object, add them as 'data'
+                if (args.length > 2) {
+                  logObject.data = args.slice(2);
+                }
+              } else if (args.length > 1) {
+                // If there are additional arguments but second is not a plain object,
+                // add all additional args as 'data'
+                logObject.data = args.slice(1);
               }
-              return log[level](...args);
-            };
-          }
-          return accumulator;
-        }, {});
-      }
+
+              return log[level](logObject);
+            }
+
+            // For other types (arrays, primitives, Error objects), wrap in object
+            return log[level]({ ...markers, data: args });
+          };
+        }
+        return accumulator;
+      }, {});
 
       // Mark that we've processed this context
       context.contextualLog = context.log;