chore: improve performance abd reduce unpacked package size

Author: webdiscus

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 2.1.1 (20225-07-12)
+
+- chore: improve performance
+- chore: reduce unpacked package size
+
 ## 2.1.0 (20225-07-11)
 
 - feat: add support negated flags with prefix `--no-`, e.g. `--no-color` -> `{ color: false }`

README.md

@@ -13,6 +13,7 @@ Supports all standard CLI flag formats and named positional arguments.
 
 ## Features
 
+- Supports ESM, CommonJS, TypeScript
 - Short `-f` and long `--flag` flags
 - Grouped short boolean flags `-xyz`
 - Values via `--key=value` or `--key value`

bench/index.js

@@ -19,7 +19,7 @@ const mri = require('mri');
 console.timeEnd('mri');
 
 console.time('flaget');
-const flaget = require('flaget2');
+const flaget = require('flaget');
 console.timeEnd('flaget');
 
 console.log('\nBenchmark:');

bench/package-lock.json

@@ -1,8 +1,10 @@
 {
+  "scripts": {
+    "start": "node index.js"
+  },
   "devDependencies": {
     "benchmark": "2.1.4",
-    "flaget2": "file:../dist",
-    "flaget": "2.1.0",
+    "flaget": "file:../dist",
     "minimist": "1.2.8",
     "mri": "1.2.0",
     "nopt": "8.1.0",

bench/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flaget",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "A lightweight CLI flag/argument parser for Node.js. Supports all standard flag formats and named positional arguments.",
   "keywords": [
     "cli",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "flaget",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "Tiny CLI flag parser",
   "keywords": [
     "cli",

package.npm.json

@@ -30,7 +30,7 @@ const cast = (val) => {
 };
 
 /**
- * Assign a flag val to both original and camelCase keys.
+ * Assign a flag value to both original and camelCase keys.
  * @param {{flags: Object<string, any>}} res The result object containing the `flags` property.
  * @param {string} key The original flag key.
  * @param {any} val The value to assign to the flag.
@@ -66,7 +66,7 @@ const getArray = (raw, i, val) => {
   }
 
   // offset is difference between current index "j" and start index "i"
-  return { values, offset: j - i };
+  return { values, offset: j - i};
 };
 
 /**
@@ -99,41 +99,36 @@ const flaget = (options = {}) => {
    * @returns {number} The updated index.
    */
   const parseLong = (arg, i) => {
-    // performance optimized:
-    // 1) using indexOf + slice is ~10% faster than split('=')
-    // 2) splitting first, then slicing is ~30% faster than slicing first, then splitting
-    //let [rawKey, rawVal] = arg.split('='); // 10% slower, but 50 bytes smaller
-    let j = arg.indexOf('=');
-    let [rawKey, rawVal] = ~j ? [arg.slice(0, j), arg.slice(++j)] : [arg];
-
-    let isNo = rawKey.startsWith('--no-');
-    let key = rawKey.slice(isNo ? 5 : 2);
-    key = alias[key] || key;
-
-    // negated or boolean flag
-    if (isNo || boolean.includes(key)) {
-      setFlag(res, key, !isNo);
-      return i;
-    }
+      // performance optimized:
+      // 1) using indexOf + slice is ~10% faster than split('=')
+      // 2) splitting first, then slicing is ~30% faster than slicing first, then splitting
+      //let [rawKey, rawVal] = arg.split('='); // 10% slower, but 50 bytes smaller
+      let j = arg.indexOf('=');
+      let isNo = arg.startsWith('--no-');
+      let key = arg.slice(isNo ? 5 : 2, ~j ? j : void 0);
+      let val = ~j ? arg.slice(++j) : null;
+
+      key = alias[key] || key;
+
+      // negated or boolean flag
+      if (isNo || boolean.includes(key)) {
+        setFlag(res, key, !isNo);
+        return i;
+      }
 
-    // flag with muli-value
-    if (array.includes(key)) {
-      let { values, offset } = getArray(raw, i, rawVal);
-      setFlag(res, key, (res.flags[key] || []).concat(values));
-      return i + offset;
-    }
+      // flag with muli-value
+      if (array.includes(key)) {
+        let { values, offset } = getArray(raw, i, val);
+        setFlag(res, key, (res.flags[key] || []).concat(values));
+        return i + offset;
+      }
 
-    let val = true;
-    if (rawVal != null) {
-      val = cast(rawVal);
-    } else if (isArg(raw, i)) {
-      val = cast(raw[++i]);
-    }
+      val = val != null ? cast(val) : isArg(raw, i) ? cast(raw[++i]) : true;
 
-    setFlag(res, key, val);
+      setFlag(res, key, val);
 
-    return i;
-  };
+      return i;
+    };
 
   /**
    * Parse short CLI flags (-a, -abc, -f value).