add enum of log level

Author: hellyzh

src/logger/logger.ts

@@ -6,6 +6,12 @@ import * as winston from "winston";
 import TelemetryTransport from "./telemetry-transport";
 import UserNotificationTransport from "./user-notification-transport";
 
+export enum LogLevel {
+    info = "info",
+    warn = "warn",
+    error = "error",
+}
+
 function FilterErrorPath(line: string): string {
     if (line) {
         const values: string[] = line.split("/out/");
@@ -21,9 +27,9 @@ function FilterErrorPath(line: string): string {
 export function configure(context: vscode.ExtensionContext) {
     winston.configure({
         transports: [
-            new (winston.transports.File)({ level: "warn", filename: context.asAbsolutePath("arduino.log") }),
-            new TelemetryTransport({ level: "info", context }),
-            new UserNotificationTransport({ level: "info" }),
+            new (winston.transports.File)({ level: LogLevel.warn, filename: context.asAbsolutePath("arduino.log") }),
+            new TelemetryTransport({ level: LogLevel.info, context }),
+            new UserNotificationTransport({ level: LogLevel.info }),
         ],
     });
 }
@@ -54,7 +60,7 @@ export function silly(message: string, metadata?: any) {
 
 export function traceUserData(message: string, metadata?: any) {
     // use `info` as the log level and add a special flag in metadata
-    winston.log("info", message, { ...metadata, telemetry: true });
+    winston.log(LogLevel.info, message, { ...metadata, telemetry: true });
 }
 
 function traceErrorOrWarning(level: string, message: string, error: Error, metadata?: any) {
@@ -68,15 +74,15 @@ function traceErrorOrWarning(level: string, message: string, error: Error, metad
             firstLine = FilterErrorPath(firstLine ? firstLine.replace(/\\/g, "/") : "");
         }
     }
-    winston.log(level, "error", { ...metadata, message: error.message, errorLine: firstLine, telemetry: true });
+    winston.log(level, message, { ...metadata, message: error.message, errorLine: firstLine, telemetry: true });
 }
 
 export function traceError(message: string, error: Error, metadata?: any) {
-    traceErrorOrWarning("error", message, error, metadata);
+    traceErrorOrWarning(LogLevel.error, message, error, metadata);
 }
 
 export function traceWarning(message: string, error: Error, metadata?: any) {
-    traceErrorOrWarning("warn", message, error, metadata);
+    traceErrorOrWarning(LogLevel.warn, message, error, metadata);
 }
 
 export function notifyAndThrowUserError(errorCode: string, error: Error, message?: string) {

src/logger/telemetry-transport.ts

@@ -4,6 +4,7 @@
 import * as vscode from "vscode";
 import TelemetryReporter from "vscode-extension-telemetry";
 import * as winston from "winston";
+import { LogLevel } from "./logger";
 interface IPackageInfo {
     name: string;
     version: string;
@@ -60,7 +61,7 @@ export class TelemetryTransport extends winston.Transport {
                         }
                     }
                 }
-                if (level === "info") {
+                if (level === LogLevel.info) {
                     this.reporter.sendTelemetryEvent(message, properties, measures);
                 } else {
                     this.reporter.sendTelemetryErrorEvent(message, properties, measures, ["message", "notification", "errorLine"]);

src/logger/user-notification-transport.ts

@@ -3,6 +3,7 @@
 
 import * as vscode from "vscode";
 import * as winston from "winston";
+import { LogLevel } from "./logger";
 
 export default class UserNotificationTransport extends winston.Transport {
 
@@ -13,9 +14,9 @@ export default class UserNotificationTransport extends winston.Transport {
     protected log(level: string, message: string, metadata?: any, callback?: (arg1, arg2) => void) {
         if (metadata && metadata.showUser) {
             const notification = (metadata && metadata.notification) ? metadata.notification : message;
-            if (level === "warn") {
+            if (level === LogLevel.warn) {
                 vscode.window.showWarningMessage(notification);
-            } else if (level === "error") {
+            } else if (level === LogLevel.error) {
                 vscode.window.showErrorMessage(notification);
             } else {
                 winston.error(`Invalid error level '${level}' for user notification.`);