chore: use pnpm in package.json scripts (#11520)

* chore: use pnpm in package.json scripts

* chore: use pnpm in package.json scripts

* fix

* Update .husky/pre-commit

* Apply suggestions from code review

---------

Co-authored-by: neverland <[email protected]>

Author: ahabhgk

.husky/pre-commit

@@ -1 +1 @@
-npx lint-staged
+pnpx lint-staged

crates/rspack_plugin_javascript/src/parser_plugin/import_meta_plugin.rs

@@ -204,11 +204,11 @@ impl JavascriptParserPlugin for ImportMetaPlugin {
         // import.meta
         // warn when access import.meta directly
         parser.warning_diagnostics.push(Box::new(create_traceable_error(
-      "Critical dependency".into(),
-      "Accessing import.meta directly is unsupported (only property access or destructuring is supported)".into(),
-      parser.source_file,
-      span.into()
-    ).with_severity(Severity::Warning)));
+          "Critical dependency".into(),
+          "Accessing import.meta directly is unsupported (only property access or destructuring is supported)".into(),
+          parser.source_file,
+          span.into(),
+        ).with_severity(Severity::Warning)));
 
         let content = if parser.is_asi_position(span.lo()) {
           ";({})"

lint-staged.config.mjs

@@ -2,7 +2,7 @@ export default {
 	"*.rs": "rustfmt",
 	"*.{ts,tsx,js,mjs,yaml,yml}":
 		"node ./node_modules/prettier/bin/prettier.cjs --write",
-	"*.toml": "npx @taplo/cli format",
+	"*.toml": "pnpx @taplo/cli format",
 	"*.{ts,tsx,js,cts,cjs,mts,mjs}": [
 		"pnpm run lint:js",
 		// ignore staged file list; execute x without extra args

package.json

@@ -12,23 +12,23 @@
     "check-dependency-version": "pnpx check-dependency-version-consistency@5 . --ignore-dep chalk --ignore-package webpack-test --ignore-package webpack-examples",
     "build:js": "pnpm --filter \"@rspack/core\" build && pnpm --parallel --filter \"@rspack/*\" --filter \"create-rspack\" --filter \"!@rspack/core\" build",
     "build:js:canary": "pnpm --filter \"@rspack-canary/core\" build && pnpm --parallel --filter \"@rspack-canary/*\" --filter \"create-rspack-canary\" --filter \"!@rspack-canary/core\" build",
-    "build:cli:dev": "npm run build:binding:dev && npm run build:js",
-    "build:cli:dev:wasm": "pnpm --filter @rspack/binding build:dev:wasm && npm run build:js",
-    "build:cli:dev:browser": "pnpm --filter @rspack/binding build:dev:browser && npm run build:js",
-    "build:cli:release": "npm run build:binding:release && npm run build:js",
-    "build:cli:release:all": "pnpm --filter @rspack/binding build:release:all && npm run build:js",
-    "build:cli:release:arm64": "pnpm --filter @rspack/binding build:release:arm64 && npm run build:js",
-    "build:cli:release:x64": "pnpm --filter @rspack/binding build:release:x64 && npm run build:js",
-    "build:cli:release:linux": "pnpm --filter @rspack/binding build:release:linux && npm run build:js",
-    "build:cli:release:win": "pnpm --filter @rspack/binding build:release:win && npm run build:js",
-    "build:cli:release:wasm": "pnpm --filter @rspack/binding build:release:wasm && npm run build:js",
-    "build:cli:release:browser": "pnpm --filter @rspack/binding build:release:browser && npm run build:js",
+    "build:cli:dev": "pnpm run build:binding:dev && pnpm run build:js",
+    "build:cli:dev:wasm": "pnpm --filter @rspack/binding build:dev:wasm && pnpm run build:js",
+    "build:cli:dev:browser": "pnpm --filter @rspack/binding build:dev:browser && pnpm run build:js",
+    "build:cli:release": "pnpm run build:binding:release && pnpm run build:js",
+    "build:cli:release:all": "pnpm --filter @rspack/binding build:release:all && pnpm run build:js",
+    "build:cli:release:arm64": "pnpm --filter @rspack/binding build:release:arm64 && pnpm run build:js",
+    "build:cli:release:x64": "pnpm --filter @rspack/binding build:release:x64 && pnpm run build:js",
+    "build:cli:release:linux": "pnpm --filter @rspack/binding build:release:linux && pnpm run build:js",
+    "build:cli:release:win": "pnpm --filter @rspack/binding build:release:win && pnpm run build:js",
+    "build:cli:release:wasm": "pnpm --filter @rspack/binding build:release:wasm && pnpm run build:js",
+    "build:cli:release:browser": "pnpm --filter @rspack/binding build:release:browser && pnpm run build:js",
     "test:js": "pnpm -r run test",
     "format:rs": "cargo fmt --all",
     "format:js": "pnpm run format-ci:js --write",
     "format-ci:js": "node ./node_modules/prettier/bin/prettier.cjs \"packages/**/src/**/*.{ts,tsx,mts,cts,js,mjs,cjs,jsx}\" \"crates/rspack_plugin_runtime/**/*.{ts,js}\" \"scripts/**/*.{cjs,mjs,js}\"  \"*.mjs\" \"x\" --check",
-    "format-ci:toml": "npx @taplo/cli format --check '.cargo/*.toml' './crates/**/Cargo.toml' './Cargo.toml'",
-    "format:toml": "npx @taplo/cli format '.cargo/*.toml' './crates/**/Cargo.toml' './Cargo.toml'",
+    "format-ci:toml": "pnpx @taplo/cli format --check '.cargo/*.toml' './crates/**/Cargo.toml' './Cargo.toml'",
+    "format:toml": "pnpx @taplo/cli format '.cargo/*.toml' './crates/**/Cargo.toml' './Cargo.toml'",
     "lint:js": "pnpm run lint-ci:js --write",
     "lint-ci:js": "biome check --diagnostic-level=warn --no-errors-on-unmatched --max-diagnostics=none --error-on-warnings",
     "lint:rs": "node ./scripts/check_rust_dependency.cjs",
@@ -102,4 +102,4 @@
       "esbuild"
     ]
   }
-}
+}

tests/e2e/package.json

@@ -4,7 +4,7 @@
   "version": "0.1.8",
   "scripts": {
     "test": "playwright test",
-    "test:CI": "CI=1 npm run test"
+    "test:CI": "CI=1 pnpm run test"
   },
   "devDependencies": {
     "@babel/core": "^7.28.0",

x.mjs

@@ -202,7 +202,7 @@ rspackCommand
 	.action(async ({ debug }) => {
 		try {
 			if (!debug) {
-				await $`npx rspack ${getVariadicArgs()}`;
+				await $`pnpm rspack ${getVariadicArgs()}`;
 				return;
 			}
 			await launchRspackCli(getVariadicArgs());
@@ -234,7 +234,7 @@ jestCommand
 	.argument("[args...]", "Arguments pass through to rspack cli")
 	.action(async ({ debug }) => {
 		if (!debug) {
-			await $`npx jest ${getVariadicArgs()}`;
+			await $`pnpx jest ${getVariadicArgs()}`;
 			return;
 		}
 		await launchJestWithArgs(getVariadicArgs());