fix(LLMO-4070): handle Configuration.findLatest errors in patchAuditForSite

findLatest throws (not returns null) when S3 throws NoSuchBucket or
AccessDenied. Wrap in try-catch so unavailable configuration skips the
audit-type check instead of surfacing a 500.

Author: noelruiz34

src/controllers/audits.js

@@ -218,7 +218,12 @@ function AuditsController(ctx) {
       return forbidden('User does not have access to this site');
     }
 
-    const configuration = await Configuration.findLatest();
+    let configuration;
+    try {
+      configuration = await Configuration.findLatest();
+    } catch {
+      // Configuration unavailable — skip audit-type registration check
+    }
     if (configuration) {
       const registeredAudits = configuration.getHandlers();
       if (!registeredAudits[auditType]) {

test/controllers/audits.test.js

@@ -856,6 +856,23 @@ describe('Audits Controller', () => {
       );
     });
 
+    it('proceeds without audit-type check when Configuration.findLatest throws', async () => {
+      mockDataAccess.Configuration.findLatest.rejects(new Error('S3 bucket not found'));
+      const auditType = 'broken-backlinks';
+
+      const context = {
+        params: { siteId, auditType },
+        data: {},
+      };
+
+      const result = await auditsController.patchAuditForSite(context);
+
+      // S3 unavailability must not surface as 500 — controller continues and hits normal validation
+      expect(result.status).to.equal(400);
+      const error = await result.json();
+      expect(error).to.have.property('message', 'No updates provided');
+    });
+
     it('merges manual overwrites correctly', async () => {
       const auditType = 'broken-backlinks';
       const manualOverwrites = [