simple-merge-class-names 1.0.9 → 2.0.0

package/README.md

@@ -12,6 +12,8 @@ A straightforward utility for merging CSS class names in `React + Tailwind` and
 -   [Where This Package Excels](#where-this-package-excels)
 -   [Source Code](#source-code)
     -   [Argument Handling](#argument-handling)
+        -   [Changes From Version 1.X.X](#changes-from-version-1xx)
+-   [Testing](#testing)
 -   [Production Considerations](#production-considerations)
 -   [License](#license)
 
@@ -105,23 +107,48 @@ While similar packages exist (`clsx`) with better features and potentially impro
  * mergeClassNames - A straightforward utility for merging CSS class names in React + Tailwind, and other JavaScript projects.
  ...
  */
-const isDefined = (val) => val !== undefined && val !== null;
+
+const isTypeString = (val) => typeof val === "string";
 
 const isNotEmptyString = (val) => val !== "";
 
 export const mergeClassNames = (...args) => {
-    const space = " ";
-    const values = args.filter(
-        (val) => isDefined(val) && isNotEmptyString(val)
-    );
-    const className = values.join(space);
+    const space = "\x20"; // " "; ASCII code for single space character;
+
+    const stringsOnly = args.filter((val) => isTypeString(val));
+
+    const trimmed = stringsOnly.map((val) => val.trim());
+
+    const nonEmpty = trimmed.filter((val) => isNotEmptyString(val));
+
+    const className = nonEmpty.join(space);
     return className;
 };
 ```
 
 ### Argument Handling
 
-`mergeClassNames` accepts multiple arguments but filters out `null`, `undefined`, and empty strings (`""`). Remaining values are either strings or are _implicitly converted_ to strings by the JavaScript engine, then joined with spaces to produce the final class name string.
+`mergeClassNames` only accepts **_non-empty string values_**, everything else like empty strings (`""`), `null`, `undefined`, numbers, objects and arrays is _ignored_. This ensures stricter and predictable output.
+
+#### Breaking Changes From Version 1.X.X
+
+In pervious versions, arguments that were not strings were implicitly converted to strings by the JavaScript engine.
+
+### Testing
+
+This project uses `Vitest` as the test runner for fast, modern testing.
+
+#### Run All Testing Once
+
+```bash
+pnpm test
+```
+
+#### Run Tests In Watch Mode
+
+```bash
+pnpm test:watch
+```
 
 ## Production Considerations
 

package/mergeClassNames.js

@@ -37,15 +37,19 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-const isDefined = (val) => val !== undefined && val !== null;
+const isTypeString = (val) => typeof val === "string";
 
 const isNotEmptyString = (val) => val !== "";
 
 export const mergeClassNames = (...args) => {
-    const space = " ";
-    const values = args.filter(
-        (val) => isDefined(val) && isNotEmptyString(val)
-    );
-    const className = values.join(space);
+    const space = "\x20"; // " "; ASCII code for single space character;
+
+    const stringsOnly = args.filter((val) => isTypeString(val));
+
+    const trimmed = stringsOnly.map((val) => val.trim());
+
+    const nonEmpty = trimmed.filter((val) => isNotEmptyString(val));
+
+    const className = nonEmpty.join(space);
     return className;
 };

package/package.json

@@ -1,11 +1,19 @@
 {
     "name": "simple-merge-class-names",
-    "version": "1.0.9",
+    "version": "2.0.0",
     "description": "Example usage: <div className = {mergeClassNames('flex', 'flex-col')}/>",
     "main": "mergeClassNames.js",
     "exports": "./mergeClassNames.js",
+    "files": [
+        "package.json",
+        "mergeClassNames.js",
+        "LICENSE.txt",
+        "README.md"
+    ],
     "scripts": {
-        "test": "echo \"Error: no test specified\" && exit 1"
+        "test": "vitest",
+        "test:watch": "vitest --watch",
+        "test:ui": "vitest --ui"
     },
     "keywords": [
         "merge",
@@ -23,5 +31,8 @@
     "bugs": {
         "url": "https://github.com/new-AF/simple-merge-class-names/issues"
     },
-    "homepage": "https://github.com/new-AF/simple-merge-class-names"
+    "homepage": "https://github.com/new-AF/simple-merge-class-names",
+    "devDependencies": {
+        "vitest": "^3.2.3"
+    }
 }