fix: properly merge reporter options from config and CLI

- Fixes issue with reporter options not being properly merged
- Handles multiple --reporter-option flags correctly
- Ensures CLI options override config file options
- Fixes #5532

Author: saranganet

lib/cli/options.js

@@ -331,34 +331,47 @@ const loadOptions = (argv = []) => {
   args._ = Array.from(new Set(args._));
 
   // Special handling for reporter-option: command line should override config file
-  // We need to check the original argv to see if reporter-option was specified via CLI
-  const originalArgs = Array.isArray(argv) ? argv : argv.split(' ');
-  const hasCliReporterOption = originalArgs.some(arg => 
-    arg === '--reporter-option' || arg === '--reporter-options' || 
-    arg.startsWith('--reporter-option=') || arg.startsWith('--reporter-options=')
-  );
-  
-  if (hasCliReporterOption && args['reporter-option'] && Array.isArray(args['reporter-option'])) {
-    // Find the CLI reporter-option by looking for the last occurrence in the original args
-    let cliOptionValue = null;
-    for (let i = originalArgs.length - 1; i >= 0; i--) {
-      if (originalArgs[i] === '--reporter-option' || originalArgs[i] === '--reporter-options') {
+  if (args['reporter-option'] && Array.isArray(args['reporter-option'])) {
+    // Start with reporter options from config file if they exist
+    const configOptions = args['reporter-option'].reduce((acc, opt) => {
+      if (typeof opt === 'string') {
+        const [key, ...values] = opt.split('=');
+        if (key && values.length > 0) {
+          acc[key] = values.join('=');
+        }
+      } else if (opt && typeof opt === 'object') {
+        Object.assign(acc, opt);
+      }
+      return acc;
+    }, {});
+
+    // Check if there are any CLI reporter options
+    const originalArgs = Array.isArray(argv) ? argv : String(argv).split(' ');
+    const cliOptions = {};
+    
+    // Process all CLI reporter options
+    for (let i = 0; i < originalArgs.length; i++) {
+      const arg = originalArgs[i];
+      if (arg === '--reporter-option' || arg === '--reporter-options') {
         if (i + 1 < originalArgs.length) {
-          cliOptionValue = originalArgs[i + 1];
+          const [key, ...values] = originalArgs[++i].split('=');
+          if (key && values.length > 0) {
+            cliOptions[key] = values.join('=');
+          }
+        }
+      } else if (arg.startsWith('--reporter-option=') || arg.startsWith('--reporter-options=')) {
+        const [key, ...values] = arg.substring(arg.indexOf('=') + 1).split('=');
+        if (key && values.length > 0) {
+          cliOptions[key] = values.join('=');
         }
-        break;
-      } else if (originalArgs[i].startsWith('--reporter-option=')) {
-        cliOptionValue = originalArgs[i].substring(originalArgs[i].indexOf('=') + 1);
-        break;
-      } else if (originalArgs[i].startsWith('--reporter-options=')) {
-        cliOptionValue = originalArgs[i].substring(originalArgs[i].indexOf('=') + 1);
-        break;
       }
     }
+
+    // Merge CLI options (which take precedence) with config options
+    const mergedOptions = { ...configOptions, ...cliOptions };
 
-    if (cliOptionValue) {
-      args['reporter-option'] = [cliOptionValue];
-    }
+    // Convert back to array of key=value strings
+    args['reporter-option'] = Object.entries(mergedOptions).map(([key, value]) => `${key}=${value}`);
   }
 
   return args;