fix: rework how modules are imported...

Author: GuillaumeDecMeetsMore

clients/javascript/lib/baseClient.ts

@@ -14,6 +14,9 @@ import {
   UnprocessableEntityError,
   sanitizeErrorData,
 } from './helpers/errors'
+import type cacheableLookupType from 'cacheable-lookup'
+import type httpsType from 'node:https'
+import type httpType from 'node:http'
 
 /**
  * Configuration for the keep alive feature
@@ -400,17 +403,17 @@ export const createAxiosInstanceBackend = async (
     typeof module !== 'undefined' &&
     module.exports
   ) {
-    // This is a dynamic import to avoid loading this module in the browser
-    const CacheableLookupLib = await import('cacheable-lookup')
+    // Import the module here to avoid loading this module in the browser
+    const CacheableLookup: typeof cacheableLookupType = require('cacheable-lookup')
 
     // Create a cacheable lookup instance
     // Goal is to make DNS lookup fully async + avoid hitting the limit of 4 UV threads
     // See https://marmelab.com/blog/2025/07/28/dns-in-nodejs.html (for example) on the subject
-    const cacheableLookup = new CacheableLookupLib.default()
+    const cacheableLookup = new CacheableLookup()
 
     if (args.baseUrl.startsWith('https')) {
-      // This is a dynamic import to avoid loading the https module in the browser
-      const https = await import('node:https')
+      // Import the module here to avoid loading this module in the browser
+      const https: typeof httpsType = require('node:https')
       // Default values are what we evaluated to be good for our load
       const httpsAgent = new https.Agent({
         keepAlive: true,
@@ -422,8 +425,8 @@ export const createAxiosInstanceBackend = async (
       cacheableLookup.install(httpsAgent)
       config.httpsAgent = httpsAgent
     } else {
-      // This is a dynamic import to avoid loading the http module in the browser
-      const http = await import('node:http')
+      // Import the module here to avoid loading this module in the browser
+      const http: typeof httpType = require('node:http')
       // Default values are what we evaluated to be good for our load
       const httpAgent = new http.Agent({
         keepAlive: true,

clients/javascript/tests/helpers/fixtures.ts

@@ -15,6 +15,10 @@ export const setupAccount = async (options?: { timeout?: number }) => {
 
   const config: Parameters<typeof NitteiClient>[0] = {
     apiKey: account.secretApiKey,
+    // Test the keep alive feature at the same time
+    keepAlive: {
+      enabled: true,
+    },
   }
   if (options?.timeout) {
     config.timeout = options.timeout