fix: update logger library

Author: tscpp

package-lock.json

@@ -47,6 +47,7 @@
     "version": "node ./dist/cli/main.js version"
   },
   "dependencies": {
+    "@eliassko/logger": "^1.0.2",
     "@manypkg/get-packages": "^2.2.1",
     "conventional-commits-parser": "^6.0.0",
     "detect-indent": "^7.0.1",
@@ -86,8 +87,7 @@
     "typescript-eslint": "^7.13.0",
     "yargs": "^17.7.2"
   },
-  "publishConfig": {
-  },
+  "publishConfig": {},
   "overrides": {
     "eslint": "$eslint"
   }

package.json

@@ -1,5 +1,5 @@
 import yargs, { Argv, CommandModule } from "yargs";
-import { logger, LogLevel, Log } from "../lib/logger.js";
+import logger from "../lib/logger.js";
 import isCI from "is-ci";
 import { ENV_PREFIX, SCRIPT_NAME } from "./constants.js";
 import init from "./commands/init.js";
@@ -8,8 +8,8 @@ import promote from "./commands/promote.js";
 import status from "./commands/status.js";
 import version from "./commands/version.js";
 import { appendFileSync, openSync } from "node:fs";
-import { Chalk, supportsColor } from "chalk";
 import stripAnsi from "strip-ansi";
+import { Log, LogLevel, formatLog, printLog } from "@eliassko/logger";
 
 function createCLI(argv: string[]) {
   const cli = yargs(argv);
@@ -138,21 +138,16 @@ function createCLI(argv: string[]) {
       );
 
       // Avoid throwing errors.
-      logger.exitGracefully = true;
+      logger.throw = false;
 
       // Pipe logger to console
-      logger
-        .clone()
-        .pipe(createLogFormatter({ colorize: true }))
-        .pipe(createLogWriter(logLevel));
+      logger.onLog((log) => printLog(formatLog(log)), logLevel);
 
       // Pipe logger to debug log
       if (args.debugFile) {
-        logger
-          .clone()
-          .pipe(createLogFormatter())
-          .pipe(createLogUncolorizer())
-          .pipe(createLogFileWriter("conver.debug.log", LogLevel.Debug));
+        const writeLog = createLogWriter("conver.debug.log");
+
+        logger.onLog((log) => writeLog(uncolorizeLog(formatLog(log))));
       }
 
       if (args.force) {
@@ -221,84 +216,18 @@ function toLogLevel(value: string): LogLevel {
   }
 }
 
-interface FormatLogOptions {
-  colorize?: boolean | undefined;
-}
-
-function createLogFormatter(options?: FormatLogOptions) {
-  return (log: Log): Log => {
-    const chalk = new Chalk({
-      level: options?.colorize && supportsColor ? supportsColor.level : 0,
-    });
-
-    let text = log.text;
-
-    switch (log.level) {
-      case LogLevel.Debug:
-        text = chalk.dim(applyLinePrefix(text, "[DBUG] "));
-        break;
-
-      case LogLevel.Verbose:
-        text = chalk.dim(applyLinePrefix(text, "[VERB] "));
-        break;
-
-      case LogLevel.Info:
-        text = applyLinePrefix(text, chalk.blue("[INFO] "));
-        break;
-
-      case LogLevel.Warning:
-        text = applyLinePrefix(text, chalk.yellow("[WARN] "));
-        break;
-
-      case LogLevel.Error:
-        text = applyLinePrefix(text, chalk.red("[ERR!] "));
-        break;
-    }
-
-    return {
-      ...log,
-      text,
-    };
+function uncolorizeLog(log: Log): Log {
+  const text = stripAnsi(log.text);
+  return {
+    ...log,
+    text,
   };
-
-  function applyLinePrefix(text: string, prefix: string) {
-    return text
-      .split("\n")
-      .map((line) => prefix + line)
-      .join("\n");
-  }
 }
 
-function createLogUncolorizer() {
-  return (log: Log): Log => {
-    const text = stripAnsi(log.text);
-    return {
-      ...log,
-      text,
-    };
-  };
-}
-
-function createLogFileWriter(path: string, level: LogLevel) {
+function createLogWriter(path: string) {
   const fd = openSync(path, "w");
 
   return (log: Log): void => {
-    if (log.level > level) {
-      return;
-    }
-
     appendFileSync(fd, log.text + "\n");
   };
 }
-
-function createLogWriter(level: LogLevel) {
-  return (log: Log): void => {
-    if (log.level > level) {
-      return;
-    }
-
-    const stream =
-      log.level <= LogLevel.Warning ? process.stderr : process.stdout;
-    stream.write(log.text + "\n");
-  };
-}

src/cli/cli.ts

@@ -1,7 +1,7 @@
 import { writeFile } from "node:fs/promises";
 import { declareCommand } from "../cli.js";
 import { existsSync } from "node:fs";
-import { logger } from "../../lib/logger.js";
+import logger from "../../lib/logger.js";
 import { ExecaError } from "execa";
 import enquirer from "enquirer";
 import isTTY from "../../lib/utils/is-tty.js";

src/cli/commands/init.ts

@@ -2,7 +2,7 @@ import { SemVer } from "semver";
 import { declareCommand } from "../cli.js";
 import enquirer from "enquirer";
 import isTTY from "../../lib/utils/is-tty.js";
-import { logger } from "../../lib/logger.js";
+import logger from "../../lib/logger.js";
 import { isPreRelease } from "../../lib/utils/version.js";
 import { getWorkspace } from "../../lib/workspace.js";
 import { join } from "node:path";

src/cli/commands/pre.ts

@@ -1,5 +1,5 @@
 import isTTY from "../../lib/utils/is-tty.js";
-import { logger } from "../../lib/logger.js";
+import logger from "../../lib/logger.js";
 import { renderList } from "../../lib/utils/tui.js";
 import { getWorkspace } from "../../lib/workspace.js";
 import { declareCommand } from "../cli.js";

src/cli/commands/promote.ts

@@ -1,6 +1,6 @@
 import chalk from "chalk";
 import { declareCommand } from "../cli.js";
-import { logger } from "../../lib/logger.js";
+import logger from "../../lib/logger.js";
 import { renderTable, renderVersioning } from "../../lib/utils/tui.js";
 import { createVersioningPlan } from "../../lib/versioning.js";
 import { getWorkspace } from "../../lib/workspace.js";

src/cli/commands/status.ts

@@ -10,7 +10,7 @@ import chalk from "chalk";
 import { renderVersioning } from "../../lib/utils/tui.js";
 import enquirer from "enquirer";
 import isTTY from "../../lib/utils/is-tty.js";
-import { logger } from "../../lib/logger.js";
+import logger from "../../lib/logger.js";
 import {
   JSONCEdit,
   modifyJsoncFile,

src/cli/commands/version.ts

@@ -1,6 +1,6 @@
 import { CommitParser as ConventionalCommitsParser } from "conventional-commits-parser";
 import { createGit } from "./utils/git.js";
-import { logger } from "./logger.js";
+import logger from "./logger.js";
 import { ExecaError } from "execa";
 import { renderCommitList } from "./utils/tui.js";
 import { Options } from "./options.js";

src/lib/history.ts

@@ -1,59 +1,16 @@
-import { inspect } from "node:util";
-import Pipeline from "./utils/pipeline.js";
+import { Logger } from "@eliassko/logger";
 
-export enum LogLevel {
-  Silent = 0,
-  Error,
-  Warning,
-  Info,
-  Verbose,
-  Debug,
-}
-
-export interface Log {
-  text: string;
-  level: LogLevel;
-}
-
-export class Logger extends Pipeline<Log> {
-  exitGracefully = false;
-
-  debug(...data: unknown[]) {
-    return this.write({ text: format(data), level: LogLevel.Debug });
-  }
+export class CustomLogger extends Logger {
+  throw = true;
 
-  verbose(...data: unknown[]) {
-    return this.write({ text: format(data), level: LogLevel.Verbose });
-  }
-
-  info(text: string) {
-    return this.write({ text, level: LogLevel.Info });
-  }
-
-  warn(text: string) {
-    return this.write({ text, level: LogLevel.Warning });
-  }
-
-  error(text: string) {
-    return this.write({ text, level: LogLevel.Error });
-  }
-
-  fatal(text: string): never {
+  override fatal(text: string): never {
     this.error(text);
-
-    if (this.exitGracefully) {
-      process.exit(1);
-    } else {
-      process.stderr.write("\n");
+    if (this.throw) {
       throw new Error(text);
+    } else {
+      process.exit(1);
     }
   }
 }
 
-function format(data: unknown[]) {
-  return data
-    .map((object) => (typeof object === "string" ? object : inspect(object)))
-    .join(" ");
-}
-
-export const logger = new Logger();
+export default new CustomLogger();