build: small refactoring

Author: matteobruni

engine/src/Utils/HexColorManager.ts

@@ -1,14 +1,26 @@
 import type { IColor, IRangeColor, IRgb, IRgba } from "../Core/Interfaces/Colors.js";
 import type { IColorManager } from "../Core/Interfaces/IColorManager.js";
 
+/**
+ * Indexes for accessing color components from regex capture groups.
+ * Uses 1-based indexing as index 0 contains the full match.
+ */
 enum RgbIndexes {
     r = 1,
     g = 2,
     b = 3,
     a = 4,
 }
 
+const shorthandHexRegex = /^#?([a-f\d])([a-f\d])([a-f\d])([a-f\d])?$/i,
+    hexRegex = /^#?([a-f\d]{2})([a-f\d]{2})([a-f\d]{2})([a-f\d]{2})?$/i,
+    hexRadix = 16,
+    defaultAlpha = 1,
+    alphaFactor = 0xff;
+
 /**
+ * Manages hexadecimal color string parsing and conversion to RGB/RGBA format.
+ * Implements the IColorManager interface for handling hex color values.
  */
 export class HexColorManager implements IColorManager {
     readonly key;
@@ -40,25 +52,20 @@ export class HexColorManager implements IColorManager {
             return;
         }
 
-        const shorthandRegex = /^#?([a-f\d])([a-f\d])([a-f\d])([a-f\d])?$/i,
-            hexFixed = hexColor.replace(shorthandRegex, (_, r: string, g: string, b: string, a: string) => {
+        const hexFixed = hexColor.replace(shorthandHexRegex, (_, r: string, g: string, b: string, a: string) => {
                 return r + r + g + g + b + b + (a !== undefined ? a + a : "");
             }),
-            regex = /^#?([a-f\d]{2})([a-f\d]{2})([a-f\d]{2})([a-f\d]{2})?$/i,
-            result = regex.exec(hexFixed),
-            radix = 16,
-            defaultAlpha = 1,
-            alphaFactor = 0xff;
+            result = hexRegex.exec(hexFixed);
 
         return result
             ? {
                   a:
                       result[RgbIndexes.a] !== undefined
-                          ? parseInt(result[RgbIndexes.a], radix) / alphaFactor
+                          ? parseInt(result[RgbIndexes.a], hexRadix) / alphaFactor
                           : defaultAlpha,
-                  b: parseInt(result[RgbIndexes.b], radix),
-                  g: parseInt(result[RgbIndexes.g], radix),
-                  r: parseInt(result[RgbIndexes.r], radix),
+                  b: parseInt(result[RgbIndexes.b], hexRadix),
+                  g: parseInt(result[RgbIndexes.g], hexRadix),
+                  r: parseInt(result[RgbIndexes.r], hexRadix),
               }
             : undefined;
     }

plugins/colors/namedColor/src/NamedColorManager.ts

@@ -1,4 +1,11 @@
-import { type IColor, type IColorManager, type IRangeColor, type IRgb, type IRgba } from "@tsparticles/engine";
+import {
+    type IColor,
+    type IColorManager,
+    type IRangeColor,
+    type IRgb,
+    type IRgba,
+    getLogger,
+} from "@tsparticles/engine";
 
 const namedColors = new Map<string, IRgb>([
     ["aliceblue", { r: 240, g: 248, b: 255 }],
@@ -180,6 +187,8 @@ export class NamedColorManager implements IColorManager {
         const rgbColor = namedColors.get(input.toLowerCase());
 
         if (!rgbColor) {
+            getLogger().error("Color not found", input);
+
             return undefined;
         }