fix: improve logging stability for complex objects

src/main/core/log/loggable.ts

@@ -10,14 +10,24 @@ import { type Logger } from './logger.js'
  *  Abstract class which enforces that all child classes define a logger instance.
  */
 export class Loggable {
-  protected logger: Logger
+  // JS-private notation is used to make this field non-enumerable
+  readonly #logger: Logger
+
+  /**
+   * Gets the logger instance associated with this Loggable.
+   *
+   * @returns The logger instance.
+   */
+  protected get logger() {
+    return this.#logger
+  }
 
   /**
    * Constructs a loggable class.
    *
    * @param logger - The logger to use during logging.
    */
   public constructor(logger: Logger) {
-    this.logger = logger
+    this.#logger = logger
   }
 }

src/main/core/log/logger.ts

@@ -18,7 +18,9 @@ const DRAIN_INTERVAL = 250 // ms
 export class Logger {
   private readonly filePath: string
   private readonly buffer: string[]
-  private readonly drainer: NodeJS.Timeout
+
+  // JS-private notation is used to make this field non-enumerable
+  readonly #drainer: NodeJS.Timeout
 
   /**
    * Constructs a Logger instance.
@@ -28,7 +30,7 @@ export class Logger {
   public constructor(filePath: string) {
     this.filePath = filePath
     this.buffer = []
-    this.drainer = setInterval(() => {
+    this.#drainer = setInterval(() => {
       void this.flush()
     }, DRAIN_INTERVAL)
 
@@ -48,7 +50,7 @@ export class Logger {
    * Flushes the logger and clears the log draining interval.
    */
   public async close() {
-    clearInterval(this.drainer)
+    clearInterval(this.#drainer)
     await this.flush()
   }