Release/0.9.4 (#166)

* bumps version to 0.9.4

* changes to cql-ls v4.6.0

* fix logging issues

* fixes issue with missing "testCasesToExclude" option in config.json file

* updates change log

Author: raleigh-g-thompson

CHANGELOG.md

@@ -1,5 +1,16 @@
 # Change Log
 
+
+## v0.9.4 (prerelease)
+
+Date: 2026-05-14
+
+* fixes issue with missing "testCasesToExclude" option in config.json file
+* fix logging issues, most logs were defaulting to info level and not honoring actual defined level
+* bumps CQL LS to v4.6.0
+* bumps version to 0.9.4
+
+
 ## v0.9.3 (prerelease)
 
 Date: 2026-04-21

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cql",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "displayName": "Clinical Quality Language (CQL)",
   "description": "Syntax highlighting, linting, and execution for the HL7 Clinical Quality Language (CQL) for VS Code",
   "publisher": "cqframework",
@@ -658,7 +658,7 @@
     "cql-language-server": {
       "groupId": "org.opencds.cqf.cql.ls",
       "artifactId": "cql-ls-server",
-      "version": "4.5.0"
+      "version": "4.6.0"
     }
   },
   "devDependencies": {

src/commands/execute-cql.ts

@@ -560,7 +560,11 @@ export function loadTestConfig(testConfigPath: Uri): TestConfig {
       log.error('Error parsing config file', errors);
       return { testCasesToExclude: [] };
     }
-    return parsed;
+    // Ensure testCasesToExclude is always defined to prevent iteration errors
+    return {
+      ...parsed,
+      testCasesToExclude: parsed.testCasesToExclude ?? []
+    };
   } catch (error) {
     log.error('Error reading config file', error);
     return { testCasesToExclude: [] };

src/log-services/multi-transport-logger.ts

@@ -9,6 +9,7 @@ export class MultiTransportLogger implements LogOutputChannel {
   private readonly _logger: WinstonLogger;
   private readonly _logFileBaseName: string;
   private readonly _logFileStorageUri: Uri;
+  private _currentBlockLevel: string | undefined;
 
   readonly logLevel: LogLevel;
   readonly onDidChangeLogLevel: Event<LogLevel>;
@@ -35,7 +36,11 @@ export class MultiTransportLogger implements LogOutputChannel {
     });
 
     this._logger = createLogger({
-      level: 'debug', // Default level
+      // Use the transport's own level set so 'trace' is a recognised level name
+      // and maps correctly to outputChannel.trace() instead of falling through
+      // to appendLine() in the default case.
+      levels: LogOutputChannelTransport.config.levels,
+      level: MultiTransportLogger.toWinstonLevel(this._outputChannel.logLevel),
       format: format.combine(
         format.timestamp({ format: 'YYYY-MM-DD HH:mm:ss' }),
         format.errors({ stack: true }),
@@ -49,6 +54,26 @@ export class MultiTransportLogger implements LogOutputChannel {
         this._dailyFileTransport,
       ],
     });
+
+    this._outputChannel.onDidChangeLogLevel(level => {
+      if (level === LogLevel.Off) {
+        this._logger.silent = true;
+      } else {
+        this._logger.silent = false;
+        this._logger.level = MultiTransportLogger.toWinstonLevel(level);
+      }
+    });
+  }
+
+  private static toWinstonLevel(level: LogLevel): string {
+    switch (level) {
+      case LogLevel.Trace:   return 'trace';
+      case LogLevel.Debug:   return 'debug';
+      case LogLevel.Info:    return 'info';
+      case LogLevel.Warning: return 'warn';
+      case LogLevel.Error:   return 'error';
+      default:               return 'info';
+    }
   }
 
   getOuptutChannelName(): string {
@@ -86,8 +111,44 @@ export class MultiTransportLogger implements LogOutputChannel {
   appendLine(value: string): void {
     const lines = value.split('\n');
 
+    // Logback server stderr pattern:
+    // %-4relative [%thread] %-5level %logger{35} %msg %n
+    // e.g. "1234 [main] WARN  org.opencds.SomeClass Some message"
+    const logbackPattern = /^\d+\s+\[\S+\]\s+(TRACE|DEBUG|INFO|WARN|ERROR)\s+/i;
+
+    // vscode-languageclient LSP trace pattern: "[Trace - HH:MM:SS AM] ..."
+    // The header and its params body arrive in separate appendLine() calls, so
+    // _currentBlockLevel persists the detected level across calls.
+    const lspTracePattern = /^\[(Trace|Debug|Info|Warn|Error)\s+-\s+/i;
+
+    // If this call starts with an LSP trace header, update the persisted level.
+    for (const line of lines) {
+      if (!line.trim()) continue;
+      const m = lspTracePattern.exec(line);
+      if (m) this._currentBlockLevel = m[1].toUpperCase();
+      break;
+    }
+
+    const logAtLevel = (level: string | undefined, line: string) => {
+      switch (level) {
+        case 'TRACE': this._logger.log('trace', line); break;
+        case 'DEBUG': this._logger.debug(line); break;
+        case 'WARN':  this._logger.warn(line);  break;
+        case 'ERROR': this._logger.error(line); break;
+        default:      this._logger.info(line);  break;
+      }
+    };
+
     lines.forEach(line => {
-      if (line) this._logger.info(line);
+      if (!line) return;
+      const logbackMatch = logbackPattern.exec(line);
+      if (logbackMatch) {
+        // Server log lines carry their own level — use it directly.
+        logAtLevel(logbackMatch[1].toUpperCase(), line);
+      } else {
+        // LSP trace lines (header + continuation) use the persisted block level.
+        logAtLevel(this._currentBlockLevel, line);
+      }
     });
   }