@automattic/eslint-plugin-wpvip 0.13.1 → 1.0.0

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@automattic/eslint-plugin-wpvip",
-  "version": "0.13.1",
+  "version": "1.0.0",
   "description": "ESLint plugin for internal WordPress VIP projects",
   "main": "index.js",
   "scripts": {
     "check-types": "tsc -p tsconfig.check.json",
     "cmd:format": "prettier --ignore-path .gitignore '**/*.(js|json|jsx|md|ts|tsx|yml|yaml)'",
     "cmd:jest": "npm run link-plugin && jest",
-    "cmd:lint": "npm run link-plugin && eslint --ext 'js,jsx,ts,tsx'",
+    "cmd:lint": "npm run link-plugin && eslint",
     "format": "npm run cmd:format -- --write",
     "format:check": "npm run cmd:format -- --check",
     "jest": "npm run cmd:jest",
@@ -15,7 +15,7 @@
     "link-plugin": "mkdir -p ./node_modules/@automattic; ln -fns $(pwd) ./node_modules/@automattic/eslint-plugin-wpvip",
     "lint": "npm run cmd:lint .",
     "lint:fix": "npm run cmd:lint . -- --fix",
-    "test": "npm run jest"
+    "test": "NODE_OPTIONS=--experimental-vm-modules npm run jest"
   },
   "repository": {
     "type": "git",
@@ -32,37 +32,36 @@
   },
   "homepage": "https://github.com/Automattic/eslint-config-wpvip#readme",
   "dependencies": {
-    "@babel/eslint-parser": "7.24.7",
-    "@rushstack/eslint-patch": "1.10.3",
-    "@typescript-eslint/eslint-plugin": "7.15.0",
-    "@typescript-eslint/parser": "7.15.0",
-    "eslint-config-prettier": "8.7.0",
-    "eslint-import-resolver-typescript": "3.6.1",
-    "eslint-plugin-import": "2.29.1",
-    "eslint-plugin-jest": "28.5.0",
-    "eslint-plugin-jsdoc": "46.8.2",
-    "eslint-plugin-json": "3.1.0",
-    "eslint-plugin-jsx-a11y": "6.7.1",
-    "eslint-plugin-prettier": "4.2.1",
-    "eslint-plugin-promise": "6.1.1",
-    "eslint-plugin-react": "7.33.2",
-    "eslint-plugin-react-hooks": "4.6.0",
-    "eslint-plugin-security": "1.7.1",
-    "eslint-plugin-unused-imports": "3.2.0",
+    "@babel/eslint-parser": "~7.28.5",
+    "@typescript-eslint/parser": "~8.48.1",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-import-resolver-typescript": "4.4.4",
+    "eslint-plugin-import": "2.32.0",
+    "eslint-plugin-jest": "29.2.1",
+    "eslint-plugin-jsdoc": "61.4.1",
+    "eslint-plugin-json": "4.0.1",
+    "eslint-plugin-jsx-a11y": "6.10.2",
+    "eslint-plugin-prettier": "5.5.4",
+    "eslint-plugin-promise": "7.2.1",
+    "eslint-plugin-react": "7.37.5",
+    "eslint-plugin-react-hooks": "7.0.1",
+    "eslint-plugin-security": "3.0.1",
+    "eslint-plugin-unused-imports": "4.3.0",
     "find-package-json": "1.2.0",
-    "globals": "13.23.0"
+    "globals": "16.5.0",
+    "typescript-eslint": "^8.48.1"
   },
   "peerDependencies": {
-    "eslint": "^8.56.0"
+    "eslint": "^9.7.0"
   },
   "devDependencies": {
-    "@tsconfig/node18": "18.2.2",
-    "@types/eslint": "8.44.8",
-    "@types/jest": "29.5.12",
-    "eslint": "8.56.0",
+    "@tsconfig/node20": "^20.1.8",
+    "@types/eslint": "9.6.1",
+    "@types/jest": "29.5.14",
+    "eslint": "^9.39.1",
     "jest": "29.7.0",
-    "prettier": "npm:wp-prettier@2.8.5",
-    "ts-jest": "29.1.5",
-    "typescript": "5.5.3"
+    "prettier": "npm:wp-prettier@3.0.3",
+    "ts-jest": "29.4.6",
+    "typescript": "5.9.3"
   }
 }

package/plugin.js

@@ -0,0 +1,12 @@
+const rules = require( './rules' );
+
+/** @type {import('eslint').ESLint.Plugin & { configs: Record<string, import('eslint').Linter.Config[]> }} */
+const plugin = {
+	meta: {
+		name: '@automattic/wpvip',
+	},
+	configs: {},
+	rules,
+};
+
+module.exports = plugin;

package/rules/nestjs-route-prefix.js

@@ -5,7 +5,19 @@
 const nestJsControllerDecorators = [ 'Controller' ];
 const nestJsRouteDecorators = [ 'Get', 'Post', 'Put', 'Delete', 'Patch', 'Options', 'Head' ];
 
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
+	meta: {
+		type: 'problem',
+		docs: {
+			description: 'Enforce route prefixes for Nest.js route decorators',
+		},
+		messages: {
+			controllerPrefix: '@Controller should not have route prefixes.',
+			decoratorSlash: `@{{ decoratorName }} should have a route starting with '/' and not ending with '/'.`,
+		},
+		schema: [],
+	},
 	create( context ) {
 		return {
 			Decorator( node ) {
@@ -21,7 +33,7 @@ module.exports = {
 					if ( argument && argument.type === 'Literal' && argument.value !== '' ) {
 						context.report( {
 							node,
-							message: '@Controller should not have route prefixes.',
+							messageId: 'controllerPrefix',
 						} );
 					}
 					return;
@@ -36,7 +48,8 @@ module.exports = {
 					) {
 						context.report( {
 							node,
-							message: `@${ decoratorName } should have a route starting with '/' and not ending with '/'.`,
+							messageId: 'decoratorSlash',
+							data: { decoratorName },
 						} );
 					}
 				}

package/rules/no-async-foreach.js

@@ -7,7 +7,18 @@
  * https://www.npmjs.com/package/eslint-plugin-no-async-foreach
  */
 
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
+	meta: {
+		type: 'problem',
+		docs: {
+			description: 'Avoid passing an async function to Array.prototype.forEach',
+		},
+		messages: {
+			avoidAsyncForEach: 'Avoid passing an async function to Array.prototype.forEach',
+		},
+		schema: [],
+	},
 	create( context ) {
 		return {
 			ExpressionStatement( node ) {
@@ -21,7 +32,7 @@ module.exports = {
 					} );
 					if ( functionArguments ) {
 						if ( functionArguments.async ) {
-							context.report( node, 'Avoid passing an async function to Array.prototype.forEach' );
+							context.report( { node, messageId: 'avoidAsyncForEach' } );
 						}
 					}
 				}

package/rules/no-unguarded-get-range-at.js

@@ -3,6 +3,7 @@
  * https://github.com/WordPress/gutenberg/blob/%40wordpress/eslint-plugin%4014.1.0/packages/eslint-plugin/rules/no-unguarded-get-range-at.js
  */
 
+/** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
 	meta: {
 		type: 'problem',

package/init.js

@@ -1 +0,0 @@
-require( '@rushstack/eslint-patch/modern-module-resolution' );