chore: enable pnpm shellEmulator (#1438)

* chore: enable pnpm shellEmulator

This fixes `pnpm build` on Windows.
Otherwise, the string escapes in the `build` script are misinterpreted.

After enabling `shellEmulator`, husky needed to be moved off npm.

* fix: test-index should use parseArgs instead

Author: lishaduck

.husky/post-checkout

@@ -1,2 +1 @@
-
-npx run-if-changed
+run-if-changed

.husky/post-commit

@@ -1,2 +1 @@
-
-npx run-if-changed
+run-if-changed

.husky/post-merge

@@ -1,2 +1 @@
-
-npx run-if-changed
+run-if-changed

.husky/post-rewrite

@@ -1,2 +1 @@
-
-npx run-if-changed
+run-if-changed

.husky/pre-commit

@@ -1,2 +1 @@
-
-npx lint-staged
+lint-staged

.husky/pre-push

@@ -1,2 +1 @@
-
-npm test && npm run build && npm run check-docs
+pnpm test && pnpm run build && pnpm run check-docs

CONTRIBUTING.md

@@ -68,20 +68,20 @@ can be ignored by being preceded by `/* c8 ignore next */` (including
 for warnings where the block is never passed over (i.e., the block is always
 entered)). If you want to test without coverage at all, you can use
 `pnpm test-no-cov`. To only test rules rather than other files, you
-can use `npm run test-index` (`pnpm` doesn't work with this script currently).
+can use `pnpm run test-index`.
 
 To test specific rules, you can supply a comma-separated list with the `--rule`
 flag passed to `test-index`, e.g., for `check-examples` and `require-example`:
 
-`npm run test-index --rule=check-examples,require-example`.
+`pnpm run test-index --rule=check-examples,require-example`.
 
 You can further limit this by providing `--invalid` and/or `--valid` flags
 with a comma-separated list of 0-based indexes that you wish to include (also
 accepts negative offsets from the end, e.g., `-1` for the last item). For
 example, to check the first and third invalid tests of `check-examples`
 along with the second valid test, you can run:
 
-`npm run test-index --rule=require-example --invalid=0,2 --valid=1`.
+`pnpm run test-index --rule=require-example --invalid=0,2 --valid=1`.
 
 ## Requirements for PRs
 

package.json

@@ -50,7 +50,6 @@
     "c8": "^10.1.3",
     "camelcase": "^8.0.0",
     "chai": "^5.2.0",
-    "cross-env": "^7.0.3",
     "decamelize": "^6.0.0",
     "eslint": "9.29.0",
     "eslint-config-canonical": "~44.9.5",
@@ -80,11 +79,7 @@
   ],
   "license": "BSD-3-Clause",
   "lint-staged": {
-    ".eslintignore": [
-      "npm run lint-fix",
-      "git add ."
-    ],
-    "*.js": "npm run lint-arg -- --fix"
+    "*.js": "eslint --fix"
   },
   "type": "module",
   "main": "./dist/index.cjs",
@@ -140,25 +135,24 @@
     "url": "https://github.com/gajus/eslint-plugin-jsdoc/issues"
   },
   "run-if-changed": {
-    "package-lock.json": "npm run install-offline"
+    "package-lock.json": "pnpm run install-offline"
   },
   "scripts": {
     "tsc": "tsc",
     "tsc-build": "tsc -p tsconfig-prod.json",
-    "build": "rimraf ./dist && cross-env NODE_ENV=production babel ./src --out-file-extension .cjs --out-dir ./dist --copy-files --source-maps --ignore ./src/bin/*.js --no-copy-ignored && replace 'require\\(\"\\.(.*?)\\.[^.]*?\"\\)' 'require(\".$1.cjs\")' 'dist' -r --include=\"*.cjs\" && pnpm tsc-build",
+    "build": "rimraf ./dist && NODE_ENV=production babel ./src --out-file-extension .cjs --out-dir ./dist --copy-files --source-maps --ignore ./src/bin/*.js --no-copy-ignored && replace 'require\\(\"\\.(.*?)\\.[^.]*?\"\\)' 'require(\".$1.cjs\")' 'dist' -r --include=\"*.cjs\" && pnpm tsc-build",
     "check-docs": "babel-node ./src/bin/generateDocs.js --check",
-    "create-docs": "npm run create-options && babel-node ./src/bin/generateDocs.js",
+    "create-docs": "pnpm run create-options && babel-node ./src/bin/generateDocs.js",
     "create-rule": "babel-node ./src/bin/generateRule.js",
     "create-options": "node ./src/bin/generateOptions.mjs",
     "install-offline": "pnpm install --prefer-offline --no-audit",
-    "lint": "npm run lint-arg -- .",
-    "lint-arg": "eslint",
-    "lint-fix": "npm run lint-arg -- --fix .",
+    "lint": "eslint",
+    "lint-fix": "eslint --fix",
     "prepare": "husky",
-    "test-no-cov": "cross-env BABEL_ENV=test mocha",
-    "test": "c8 npm run test-no-cov",
-    "test-cov": "cross-env TIMING=1 c8 --reporter text npm run test-no-cov",
-    "test-index": "npm run test-no-cov -- test/rules/index.js"
+    "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"
   },
   "version": "1.0.0"
 }

pnpm-lock.yaml

@@ -1,3 +1,5 @@
 ignoredBuiltDependencies:
   - core-js
   - core-js-pure
+
+shellEmulator: true