fix(LLMO-4070): log warning when Configuration.findLatest fails

Bare catch {} swallowed all errors silently. Log the error at warn level
so S3 unavailability is visible in logs while still allowing the request
to proceed without the audit-type registration check.

Author: noelruiz34

src/controllers/audits.js

@@ -221,8 +221,8 @@ function AuditsController(ctx) {
     let configuration;
     try {
       configuration = await Configuration.findLatest();
-    } catch {
-      // Configuration unavailable — skip audit-type registration check
+    } catch (err) {
+      context.log.warn(`Failed to load configuration, skipping audit-type check: ${err.message}`);
     }
     if (configuration) {
       const registeredAudits = configuration.getHandlers();

test/controllers/audits.test.js

@@ -859,10 +859,12 @@ 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 log = { warn: sinon.stub() };
 
       const context = {
         params: { siteId, auditType },
         data: {},
+        log,
       };
 
       const result = await auditsController.patchAuditForSite(context);
@@ -871,6 +873,8 @@ describe('Audits Controller', () => {
       expect(result.status).to.equal(400);
       const error = await result.json();
       expect(error).to.have.property('message', 'No updates provided');
+      expect(log.warn).to.have.been.calledOnce;
+      expect(log.warn.firstCall.args[0]).to.include('S3 bucket not found');
     });
 
     it('merges manual overwrites correctly', async () => {