fix: include time and scope in log file (#1951)

vscode-neovim · May 12, 2024 · cccdf99 · cccdf99
1 parent 1fc6752
commit cccdf99
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/logger.ts b/src/logger.ts
@@ -87,11 +87,10 @@ export class Logger implements Disposable {
             return p + (i > 0 ? " " : "") + c;
         }, "");
 
-        if (this.fd) {
-            fs.appendFileSync(this.fd, msg + "\n");
-        }
-        if (this.logToConsole) {
-            console[level == LogLevel.error ? "error" : "log"](`${getTimestamp()} ${scope}: ${msg}`);
+        if (this.fd || this.logToConsole) {
+            const logMsg = `${getTimestamp()} ${scope}: ${msg}`;
+            this.fd && fs.appendFileSync(this.fd, logMsg + "\n");
+            this.logToConsole && console[level == LogLevel.error ? "error" : "log"](logMsg);
         }
 
         // Half-baked attempt to avoid infinite loop.