Actually pass init option to logger

Author: lionel-

apps/lsp/src/config.ts

@@ -146,16 +146,12 @@ export class ConfigurationManager extends Disposable {
     this._settings = defaultSettings();
   }
 
-  public init(logLevel?: string) {
-    const initLogLevel = Logger.parseLogLevel(
-      logLevel ?? "warn"
-    );
-
+  public init(logLevel: LogLevel) {
     this._settings = {
       ...this._settings,
       quarto: {
         ...this._settings.quarto,
-        logLevel: initLogLevel,
+        logLevel,
       }
     };
   }

apps/lsp/src/index.ts

@@ -75,7 +75,11 @@ let mdLs: IMdLanguageService | undefined;
 connection.onInitialize((params: InitializeParams) => {
   // Set log level from initialization options if provided so that we use the
   // expected level as soon as possible
-  configManager.init(params.initializationOptions?.logLevel);
+  const initLogLevel = Logger.parseLogLevel(
+    params.initializationOptions?.logLevel ?? "warn"
+  );
+  logger.init(initLogLevel);
+  configManager.init(initLogLevel);
 
   // We're connected, log messages via LSP
   logger.setConnection(connection);

apps/lsp/src/logging.ts

@@ -46,7 +46,7 @@ export class Logger extends Disposable implements ILogger {
     return JSON.stringify(data, undefined, 2);
   }
 
-  private _logLevel: LogLevel;
+  private _logLevel = LogLevel.Warn;
   private _connection?: Connection;
   private _config?: ConfigurationManager;
 
@@ -56,6 +56,10 @@ export class Logger extends Disposable implements ILogger {
     super();
   }
 
+  init(logLevel: LogLevel): void {
+    this._logLevel = logLevel;
+  }
+
   setConnection(connection: Connection) {
     this._connection = connection;
     this.logInfo('LSP is now connected');