Fix sandbox configuration serialization and add comprehensive logging

Author: Kanishka

src/controllers/configuration.js

@@ -118,10 +118,12 @@ function ConfigurationController(ctx) {
 
       // Update sandbox configurations for each audit type
       Object.keys(sandboxConfigs).forEach((auditType) => {
+        console.log(`Updating sandbox config for audit type: ${auditType}`, sandboxConfigs[auditType]);
         config.updateSandboxAuditConfig(auditType, sandboxConfigs[auditType]);
       });
 
       // Save the updated configuration
+      console.log('Saving updated configuration with sandbox audits:', config.state?.sandboxAudits);
       await config.save();
 
       return ok({

src/dto/configuration.js

@@ -20,19 +20,18 @@ export const ConfigurationDto = {
    * @returns {{
    * }}
    */
-  toJSON: (configuration) => ({
-    version: configuration.getVersion(),
-    jobs: configuration.getJobs(),
-    ...(configuration.getHandlers() ? { handlers: configuration.getHandlers() } : {}),
-    queues: configuration.getQueues(),
-    ...(configuration.getSlackRoles() ? { slackRoles: configuration.getSlackRoles() } : {}),
-    ...(configuration.getEnabledSandboxAudits && configuration.getEnabledSandboxAudits().length > 0
-      ? {
-        sandboxAudits: configuration.getEnabledSandboxAudits().reduce((acc, auditType) => {
-          acc[auditType] = configuration.getSandboxAuditConfig(auditType);
-          return acc;
-        }, {}),
-      }
-      : {}),
-  }),
+  toJSON: (configuration) => {
+    const result = {
+      version: configuration.getVersion(),
+      jobs: configuration.getJobs(),
+      ...(configuration.getHandlers() ? { handlers: configuration.getHandlers() } : {}),
+      queues: configuration.getQueues(),
+      ...(configuration.getSlackRoles() ? { slackRoles: configuration.getSlackRoles() } : {}),
+      ...(configuration.state?.sandboxAudits
+        ? { sandboxAudits: configuration.state.sandboxAudits }
+        : {}),
+    };
+    console.log('[ConfigurationDto] Serializing configuration with sandboxAudits:', configuration.state?.sandboxAudits);
+    return result;
+  },
 };

test/controllers/configurations.test.js

@@ -89,6 +89,14 @@ describe('Configurations Controller', () => {
         if (auditType === 'meta-tags') return { expire: '15' };
         return null;
       },
+      state: {
+        sandboxAudits: {
+          enabledAudits: {
+            cwv: { expire: '10' },
+            'meta-tags': { expire: '15' },
+          },
+        },
+      },
     },
   ];