Implemented new 'Off' LogLevel that works the exact same way as LogLevel.None

Author: gabedemesa

.changeset/three-zoos-enter.md

@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+Added new 'Off' LogLevel that works the exact same way as LogLevel.None

packages/effect/src/LogLevel.ts

@@ -20,7 +20,7 @@ import type { Pipeable } from "./Pipeable.js"
  * @property label - A label associated with the log level.
  * @property syslog -The syslog severity level of the log level.
  */
-export type LogLevel = All | Fatal | Error | Warning | Info | Debug | Trace | None
+export type LogLevel = All | Fatal | Error | Warning | Info | Debug | Trace | None | Off
 
 /**
  * @since 2.0.0
@@ -111,6 +111,17 @@ export interface Trace extends Pipeable {
  */
 export interface None extends Pipeable {
   readonly _tag: "None"
+  readonly label: "NONE"
+  readonly syslog: 7
+  readonly ordinal: number
+}
+
+/**
+ * @since 2.0.0
+ * @category model
+ */
+export interface Off extends Pipeable {
+  readonly _tag: "Off"
   readonly label: "OFF"
   readonly syslog: 7
   readonly ordinal: number
@@ -164,6 +175,12 @@ export const Trace: LogLevel = core.logLevelTrace
  */
 export const None: LogLevel = core.logLevelNone
 
+/**
+ * @since 2.0.0
+ * @category constructors
+ */
+export const Off: LogLevel = core.logLevelOff
+
 /**
  * @since 2.0.0
  * @category constructors
@@ -277,6 +294,8 @@ export const fromLiteral = (literal: Literal): LogLevel => {
       return Info
     case "Trace":
       return Trace
+    case "Off":
+      return Off
     case "None":
       return None
     case "Warning":

packages/effect/src/internal/core.ts

@@ -1678,6 +1678,17 @@ export const logLevelTrace: LogLevel.LogLevel = {
 export const logLevelNone: LogLevel.LogLevel = {
   _tag: "None",
   syslog: 7,
+  label: "NONE",
+  ordinal: Number.MAX_SAFE_INTEGER,
+  pipe() {
+    return pipeArguments(this, arguments)
+  }
+}
+
+/** @internal */
+export const logLevelOff: LogLevel.LogLevel = {
+  _tag: "Off",
+  syslog: 7,
   label: "OFF",
   ordinal: Number.MAX_SAFE_INTEGER,
   pipe() {
@@ -1694,7 +1705,8 @@ export const allLogLevels: ReadonlyArray<LogLevel.LogLevel> = [
   logLevelWarning,
   logLevelError,
   logLevelFatal,
-  logLevelNone
+  logLevelNone,
+  logLevelOff
 ]
 
 // -----------------------------------------------------------------------------

packages/effect/src/internal/logger.ts

@@ -302,6 +302,7 @@ const colors = {
 
 const logLevelColors: Record<LogLevel.LogLevel["_tag"], ReadonlyArray<string>> = {
   None: [],
+  Off: [],
   All: [],
   Trace: [colors.gray],
   Debug: [colors.blue],
@@ -312,6 +313,7 @@ const logLevelColors: Record<LogLevel.LogLevel["_tag"], ReadonlyArray<string>> =
 }
 const logLevelStyle: Record<LogLevel.LogLevel["_tag"], string> = {
   None: "",
+  Off: "",
   All: "",
   Trace: "color:gray",
   Debug: "color:blue",

packages/effect/test/LogLevel.test.ts

@@ -10,6 +10,7 @@ describe("LogLevel", () => {
     strictEqual(LogLevel.fromLiteral("Fatal"), LogLevel.Fatal)
     strictEqual(LogLevel.fromLiteral("Info"), LogLevel.Info)
     strictEqual(LogLevel.fromLiteral("None"), LogLevel.None)
+    strictEqual(LogLevel.fromLiteral("Off"), LogLevel.Off)
     strictEqual(LogLevel.fromLiteral("Trace"), LogLevel.Trace)
     strictEqual(LogLevel.fromLiteral("Warning"), LogLevel.Warning)
   })