fix(oci): log terminal object storage failures as errors

Author: waleedlatif1

apps/sim/lib/internal/oci-object-storage/execute-tool.test.ts

@@ -132,13 +132,8 @@ describe('OCI Object Storage tool execution boundary', () => {
       expect(response.status).toBe(status)
       expect(body.error).toContain(text)
       expect(JSON.stringify(body)).not.toContain('secret-key-canary')
-      if (status >= 500) {
-        expect(logger.error).toHaveBeenCalledOnce()
-        expect(logger.warn).not.toHaveBeenCalled()
-      } else {
-        expect(logger.warn).toHaveBeenCalledOnce()
-        expect(logger.error).not.toHaveBeenCalled()
-      }
+      expect(logger.error).toHaveBeenCalledOnce()
+      expect(logger.warn).not.toHaveBeenCalled()
     }
   )
 

apps/sim/lib/internal/oci-object-storage/execute-tool.ts

@@ -120,11 +120,7 @@ export const executeOciObjectStorageTool: InternalToolOperationHandler<
       toolId: request.toolId,
       status: normalized.status,
     }
-    if (normalized.status >= 500) {
-      logger.error('OCI Object Storage operation failed', logContext)
-    } else {
-      logger.warn('OCI Object Storage operation failed', logContext)
-    }
+    logger.error('OCI Object Storage operation failed', logContext)
     return Response.json(
       { success: false, error: normalized.message },
       { status: normalized.status }