[#376] Support ESM

Author: feelform

lib/agent-builder.d.ts

@@ -5,6 +5,7 @@
  */
 
 import { Config } from './config';
+import type { Logging } from './utils/log/types';
 import AgentInfo = require('./data/dto/agent-info');
 
 interface TraceObject {
@@ -44,6 +45,7 @@ declare class AgentBuilder {
 
   setConfig(config: Config): AgentBuilder;
   setDataSender(dataSender: DataSender): AgentBuilder;
+  setLogger(logger: Logging): AgentBuilder;
   addService(service: () => void): AgentBuilder;
   disableStatsScheduler(): AgentBuilder;
   disablePingScheduler(): AgentBuilder;

lib/utils/log/log-builder.js

@@ -87,7 +87,7 @@ class LogBuilder {
     build() {
         if (!this.config) {
             const noConfigLog = new NoConfigLog(this.name, this.level)
-            noConfigLog.appenders = this.appenders
+            noConfigLog.appenders = [...this.appenders]
             return noConfigLog
         }
 

lib/utils/log/types.d.ts

@@ -66,7 +66,7 @@ export declare class Logger implements Logging {
     isDebug(): boolean;
     isInfo(): boolean;
 
-    getLogger(log?: Log | string): Logger;
+    getLogger(log?: Log | string): Logging;
 }
 
 export declare class ChildLogger implements Logging {

test/utils/log/logger.test.js

@@ -59,6 +59,41 @@ test('appenders', (t) => {
     t.equal(testLog2.appenders.length, 1, 'appenders length with valid appender')
 })
 
+test('no config logs clone appenders array', (t) => {
+    t.plan(3)
+
+    const builder = new LogBuilder('no-config-appenders')
+    const firstAppender = {
+        name: 'first-appender',
+        loggingLevel: levels.DEBUG,
+        debug: (msg) => msg,
+        info: (msg) => msg,
+        warn: (msg) => msg,
+        error: (msg) => msg
+    }
+
+    builder.addAppender(firstAppender)
+
+    const initialLog = builder.build()
+
+    t.equal(initialLog.getAppenders().length, 1, 'no config log should include existing appenders')
+
+    const secondAppender = {
+        name: 'second-appender',
+        loggingLevel: levels.DEBUG,
+        debug: (msg) => msg,
+        info: (msg) => msg,
+        warn: (msg) => msg,
+        error: (msg) => msg
+    }
+
+    builder.addAppender(secondAppender)
+    const refreshedLog = builder.build()
+
+    t.equal(initialLog.getAppenders().length, 1, 'modifying builder appenders should not affect existing no config log')
+    t.equal(refreshedLog.getAppenders().length, 2, 'subsequent builds should reflect new appenders')
+})
+
 test('addAppender validation', (t) => {
     t.plan(8)