perf: memoize strings

BREAKING CHANGE: remove (broken) support for empty namespace

Author: wojpawlik

deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@grammyjs/debug",
-  "version": "0.2.2",
+  "version": "0.3.0",
   "exports": "./src/mod.ts",
   "publish": {
     "exclude": [

src/env.ts

@@ -5,3 +5,5 @@ const prompts = (variable: string) =>
 
 export const env = (variable: string): string =>
 	(!prompts(variable) && (context.process?.env ?? context.env)?.[variable]) || "";
+
+export const noColor = context.Deno?.noColor ?? Boolean(env("NO_COLOR"));

src/mod.ts

@@ -1,14 +1,14 @@
-import { context } from "./types.ts";
 import { colorNs, selectColor } from "./colors.ts";
-import { env } from "./env.ts";
+import { env, noColor } from "./env.ts";
 import { Namespaces } from "./namespacing.ts";
+import { context } from "./types.ts";
 
 const DEBUG: string = env("DEBUG");
 const stderr = context.process?.stderr;
-const useColor = stderr?.isTTY && !env("NO_COLOR");
+const useColor = stderr?.isTTY && !noColor;
 const console = context.console.Console?.(stderr) ?? context.console;
 const useAnsi = ["Node.js", "Deno", "Bun"].includes(
-	context.navigator.userAgent.split("/", 1)[0],
+  context.navigator.userAgent.split("/", 1)[0],
 );
 
 /**
@@ -20,15 +20,13 @@ const namespaces: Namespaces = new Namespaces(DEBUG);
  * A debug logger instance.
  */
 export interface DebugFn {
-	(...data: unknown[]): void;
-	/**
-	 * Manually enable or disable logging for this namespace.
-	 */
-	enabled: boolean;
+  (...data: unknown[]): void;
+  /**
+   * Manually enable or disable logging for this namespace.
+   */
+  enabled: boolean;
 }
 
-const ns = (n: string) => (n ? n + " " : "");
-
 /**
  * Create a debug instance for a namespace.
  *
@@ -50,24 +48,26 @@ const ns = (n: string) => (n ? n + " " : "");
  * @returns A debug instance.
  */
 export function createDebug(namespace: string): DebugFn {
-	const color = selectColor(namespace);
-	const debugfn = (...data: unknown[]) => {
-		if (!debugfn.enabled) return;
-		const start = data.length ? data.shift() : "";
-		if (useAnsi) {
-			const name = useColor ? colorNs(namespace, color) : namespace;
-			console.debug(ns(name) + start, ...data);
-		} else {
-			console.debug(
-				`%c${ns(namespace)}%c${start}`,
-				`color: #${color.toString(16).padStart(6, "0")}`,
-				"color: inherit",
-				...data,
-			);
-		}
-	};
+  const color = selectColor(namespace);
+  const css = `color: #${color.toString(16).padStart(6, "0")}`;
+  const ansiNamespace = useColor ? colorNs(namespace, color) : namespace;
+
+  const debugfn = (...data: unknown[]) => {
+    if (!debugfn.enabled) return;
+    const start = data.length ? data.shift() : "";
+    if (useAnsi) {
+      console.debug(`${ansiNamespace} ${start}`, ...data);
+    } else {
+      console.debug(
+        `%c${namespace}%c ${start}`,
+        css,
+        "color: inherit",
+        ...data,
+      );
+    }
+  };
 
-	debugfn.enabled = namespaces.check(namespace);
+  debugfn.enabled = namespaces.check(namespace);
 
-	return debugfn;
+  return debugfn;
 }