fix(utils): correct logWrapper format string handling

The logWrapper was prepending trace/job IDs as a separate argument instead
of concatenating them with the format string. This caused format placeholders
(%s, %d, etc.) to not be interpolated, appearing literally in logs.

Changes:
- Merge marker string with format string (first arg) instead of prepending as separate arg
- Add tests to verify format string placeholder interpolation works correctly

Before: log.info('[traceId=123]', 'Found %d items', 42)
After:  log.info('[traceId=123] Found %d items', 42)

Fixes format string interpolation in all log levels (info, error, debug, warn, etc.)

Related: #1154

Author: HollywoodTonight

packages/spacecat-shared-utils/src/log-wrapper.js

@@ -59,7 +59,13 @@ export function logWrapper(fn) {
         // 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) => log[level](markerString, ...args);
+            accumulator[level] = (...args) => {
+              // If first argument is a string (format string), prepend the marker to it
+              if (args.length > 0 && typeof args[0] === 'string') {
+                args[0] = `${markerString} ${args[0]}`;
+              }
+              return log[level](...args);
+            };
           }
           return accumulator;
         }, {});

packages/spacecat-shared-utils/test/log-wrapper.test.js

@@ -234,4 +234,31 @@ describe('logWrapper tests', () => {
     // context.log should remain unchanged
     expect(mockContext.log).to.equal(originalLog);
   });
+
+  // Test format string interpolation
+  logLevels.forEach((level) => {
+    it(`should properly handle format strings with placeholders in ${level} log`, async () => {
+      getTraceIdStub.returns('1-abc-def');
+      const originalLog = mockContext.log;
+      const wrappedFn = logWrapper(mockFnFromSqs);
+
+      await wrappedFn(message, mockContext);
+
+      // Log with format string placeholders
+      mockContext.log[level]('Found %d items for site %s', 42, 'example.com');
+
+      // Verify that the enhanced log was called with the correct format string and args
+      expect(originalLog[level].calledOnce).to.be.true;
+      const callArgs = originalLog[level].getCall(0).args;
+      
+      // First arg should be the format string with markers prepended
+      expect(callArgs[0]).to.include('[jobId=');
+      expect(callArgs[0]).to.include('[traceId=1-abc-def]');
+      expect(callArgs[0]).to.include('Found %d items for site %s');
+      
+      // Subsequent args should be the format string arguments
+      expect(callArgs[1]).to.equal(42);
+      expect(callArgs[2]).to.equal('example.com');
+    });
+  });
 });