fix: test-index should use parseArgs instead

Author: lishaduck

package.json

@@ -79,7 +79,7 @@
   ],
   "license": "BSD-3-Clause",
   "lint-staged": {
-    "*.js": "pnpm run lint-arg --fix"
+    "*.js": "eslint --fix"
   },
   "type": "module",
   "main": "./dist/index.cjs",
@@ -146,14 +146,13 @@
     "create-rule": "babel-node ./src/bin/generateRule.js",
     "create-options": "node ./src/bin/generateOptions.mjs",
     "install-offline": "pnpm install --prefer-offline --no-audit",
-    "lint": "pnpm run lint-arg -- .",
-    "lint-arg": "eslint",
-    "lint-fix": "pnpm run lint-arg --fix .",
+    "lint": "eslint",
+    "lint-fix": "eslint --fix",
     "prepare": "husky",
     "test-no-cov": "BABEL_ENV=test mocha",
     "test": "c8 pnpm run test-no-cov",
     "test-cov": "TIMING=1 c8 --reporter text pnpm run test-no-cov",
-    "test-index": "pnpm run test-no-cov -- test/rules/index.js"
+    "test-index": "pnpm run test-no-cov test/rules/index.js"
   },
   "version": "1.0.0"
 }

test/rules/index.js

@@ -8,6 +8,9 @@ import {
   readFileSync,
 } from 'fs';
 import defaultsDeep from 'lodash.defaultsdeep';
+import {
+  parseArgs,
+} from 'node:util';
 import {
   join,
 } from 'path';
@@ -31,7 +34,33 @@ const main = async () => {
     throw new Error('TypeScript guard');
   }
 
-  for (const ruleName of process.env.npm_config_rule ? process.env.npm_config_rule.split(',') : ruleNames) {
+  /**
+   * @satisfies {import('node:util').ParseArgsOptionsConfig}
+   */
+  const options = {
+    invalid: {
+      type: 'string',
+    },
+    rule: {
+      type: 'string',
+    },
+    valid: {
+      type: 'string',
+    },
+  };
+
+  const {
+    values: {
+      invalid,
+      rule: rules,
+      valid,
+    },
+  } = parseArgs({
+    allowPositionals: true,
+    options,
+  });
+
+  for (const ruleName of rules ? rules.split(/[, ]/v) : ruleNames) {
     if (semver.gte(ESLint.version, '8.0.0') && ruleName === 'check-examples') {
       // Uses the processor instead for higher versions
       continue;
@@ -124,24 +153,24 @@ const main = async () => {
       return assertion;
     });
 
-    if (process.env.npm_config_invalid) {
-      const indexes = process.env.npm_config_invalid.split(',');
+    if (invalid) {
+      const indexes = invalid.split(/[, ]/v);
       assertions.invalid = assertions.invalid.filter((_assertion, idx) => {
         return indexes.includes(String(idx)) ||
           indexes.includes(String(idx - assertions.invalid.length));
       });
-      if (!process.env.npm_config_valid) {
+      if (!valid) {
         assertions.valid = [];
       }
     }
 
-    if (process.env.npm_config_valid) {
-      const indexes = process.env.npm_config_valid.split(',');
+    if (valid) {
+      const indexes = valid.split(/[, ]/v);
       assertions.valid = assertions.valid.filter((_assertion, idx) => {
         return indexes.includes(String(idx)) ||
           indexes.includes(String(idx - assertions.valid.length));
       });
-      if (!process.env.npm_config_invalid) {
+      if (!invalid) {
         assertions.invalid = [];
       }
     }