refactor: remove formatName utility and update related references

Author: KristjanESPERANTO

API/api.js

@@ -12,7 +12,6 @@ const path = require("node:path");
 const url = require("node:url");
 const {v4: uuid} = require("uuid");
 const express = require("express");
-const {formatName} = require("../lib/utils.js");
 
 const getActions = (content) => {
   const re = /notification ===? (?:"|')([A-Z_-]+?)(?:"|')|case (?:"|')([A-Z_-]+)(?:"|')/g;
@@ -534,7 +533,7 @@ module.exports = {
         id: `mc-${r.path}`,
         type: "menu",
         icon: "bars",
-        text: formatName(r.module),
+        text: r.module,
         items: []
       };
       for (const a of Object.keys(r.actions)) {

lib/utils.js

@@ -13,17 +13,6 @@ function capitalizeFirst (string) {
   return string.charAt(0).toUpperCase() + string.slice(1);
 }
 
-/**
- * Formats a module name by removing MMM- prefix and converting to title case.
- * @param {string} string - The module name to format
- * @returns {string} Formatted module name with spaces and title case
- */
-function formatName (string) {
-  if (!string) return string;
-  // Original logic: strip MMM-, split camelCase, convert delimiters to spaces & capitalize
-  return string.replaceAll("MMM-", "").replaceAll(/([a-z])([A-Z])/g, "$1 $2").replaceAll(/(^|[-_])(\w)/g, ($0, $1, $2) => ($1 && " ") + $2.toUpperCase());
-}
-
 /**
  * Checks if a string contains a pattern (null-safe).
  * @param {string} pattern - The pattern to search for
@@ -35,4 +24,4 @@ function includes (pattern, string) {
   return string.includes(pattern);
 }
 
-module.exports = {capitalizeFirst, formatName, includes};
+module.exports = {capitalizeFirst, includes};

tests/unit/api.answerModuleApi.test.js

@@ -15,7 +15,6 @@ function makeContext (overrides = {}) {
     sendResponse: () => {},
     checkInitialized: () => true,
     translate: (s) => s,
-    formatName: (s) => s,
     thisConfig: {},
     ...overrides
   };

tests/unit/api.delayedFlow.test.js

@@ -14,7 +14,6 @@ function makeContext (overrides = {}) {
     checkInitialized: () => true,
     checkDelay: apiModule.checkDelay,
     translate: (s) => s,
-    formatName: (s) => s,
     delayedQuery: () => {},
     thisConfig: {},
     ...overrides

tests/unit/api.helpers.test.js

@@ -14,7 +14,6 @@ function makeContext (overrides = {}) {
     sendResponse: () => {},
     checkInitialized: () => true,
     translate: (s) => s,
-    formatName: (s) => s,
     thisConfig: {},
     ...overrides
   };

tests/unit/utils.test.js

@@ -1,7 +1,7 @@
 /* Unit tests for lib/utils.js using Node's built-in test runner */
 const assert = require("node:assert/strict");
 const {test, describe} = require("node:test");
-const {capitalizeFirst, formatName, includes} = require("../../lib/utils");
+const {capitalizeFirst, includes} = require("../../lib/utils");
 
 describe("utils", () => {
   describe("capitalizeFirst", () => {
@@ -13,15 +13,6 @@ describe("utils", () => {
     });
   });
 
-  describe("formatName", () => {
-    test("removes MMM- prefix and splits camelCase", () => {
-      assert.equal(formatName("MMM-Remote-Control"), "Remote Control");
-    });
-    test("converts hyphen/underscore to spaces and capitalizes", () => {
-      assert.equal(formatName("my-module_name"), "My Module Name");
-    });
-  });
-
   describe("includes", () => {
     test("returns true when substring is found", () => {
       assert.equal(includes("test", "this is a test string"), true);