@akinon/eslint-plugin-projectzero 2.0.0-beta.2 → 2.0.0-beta.20

package/CHANGELOG.md

@@ -1,18 +1,126 @@
 # @akinon/eslint-plugin-projectzero
 
-## 2.0.0-beta.2
+## 2.0.0-beta.20
 
-## 2.0.0-beta.1
+## 1.125.0
+
+## 1.124.0
+
+## 1.123.0
+
+## 1.122.0
+
+## 1.121.0
+
+## 1.120.0
+
+## 1.119.0
+
+## 1.118.0
+
+## 1.117.0
+
+## 1.116.0
+
+## 1.115.0
+
+## 1.114.0
+
+## 1.113.0
+
+## 1.112.0
+
+## 1.111.0
+
+## 1.110.0
+
+## 1.109.0
+
+## 1.108.0
+
+## 1.107.0
+
+## 1.106.0
+
+## 1.105.0
+
+### Minor Changes
+
+- 62779ee: ZERO-3668: Refactor check-menu-depth rule to simplify depth validation logic and remove unused error messages
+
+## 1.104.0
+
+## 1.103.0
+
+## 1.102.0
+
+## 1.101.0
+
+## 1.100.0
+
+### Minor Changes
+
+- e57cd93: ZERO-3460: prevent installment loop on payment method switch
+
+## 1.99.0
 
 ### Minor Changes
 
-- ZERO-3091: Upgrade Next.js to v15 and React to v19
+- d58538b: ZERO-3638: Enhance RC pipeline: add fetch, merge, and pre-release setup with conditional commit
+
+## 1.98.0
+
+## 1.97.0
+
+## 1.96.0
+
+## 1.95.0
+
+### Minor Changes
+
+- 3b47394: ZERO-3582: Enhance check-menu-depth rule to enforce depth parameter in getMenu calls and add corresponding error messages
+
+## 1.94.0
+
+## 1.93.0
+
+## 1.92.0
+
+## 1.91.0
+
+## 1.90.0
+
+## 1.89.0
+
+## 1.88.0
+
+## 1.87.0
+
+## 1.86.0
+
+## 1.85.0
+
+### Minor Changes
+
+- 443ea19: ZERO-3254: Add ESLint rule to enforce correct order of pre-order middlewares
+
+## 1.84.0
+
+## 1.83.0
+
+## 1.82.0
+
+## 1.81.0
+
+## 1.80.0
+
+## 1.79.0
 
-## 2.0.0-beta.0
+## 1.78.0
 
-### Major Changes
+## 1.77.0
 
-- be6c09d: ZERO-3114: Create beta version.
+## 1.76.0
 
 ## 1.75.0
 

package/dist/rules/check-pre-order-middleware-order.js

@@ -0,0 +1,87 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const utils_1 = require("@typescript-eslint/utils");
+const CORRECT_PRE_ORDER_MIDDLEWARE_ORDER = [
+    'shippingStepMiddleware',
+    'installmentOptionMiddleware',
+    'paymentOptionResetMiddleware',
+    'paymentOptionMiddleware',
+    'attributeBasedShippingOptionMiddleware',
+    'dataSourceShippingOptionMiddleware',
+    'shippingOptionMiddleware',
+    'setAddressMiddleware',
+    'deliveryOptionMiddleware',
+    'setPreOrderMiddleware',
+    'redirectionMiddleware',
+    'preOrderValidationMiddleware'
+];
+exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
+    create(context) {
+        function checkPreOrderMiddlewareOrder(node) {
+            if (!isPreOrderMiddlewaresDeclaration(node)) {
+                return;
+            }
+            const arrayExpression = node.init;
+            const foundOrder = extractMiddlewareNames(arrayExpression);
+            validateMiddlewareOrder(foundOrder, node);
+        }
+        function isPreOrderMiddlewaresDeclaration(node) {
+            var _a;
+            return (node.id.type === utils_1.AST_NODE_TYPES.Identifier &&
+                node.id.name === 'preOrderMiddlewares' &&
+                ((_a = node.init) === null || _a === void 0 ? void 0 : _a.type) === utils_1.AST_NODE_TYPES.ArrayExpression);
+        }
+        function extractMiddlewareNames(arrayExpression) {
+            return arrayExpression.elements
+                .map((element) => {
+                if ((element === null || element === void 0 ? void 0 : element.type) === utils_1.AST_NODE_TYPES.Identifier) {
+                    return element.name;
+                }
+                return null;
+            })
+                .filter((name) => Boolean(name));
+        }
+        function validateMiddlewareOrder(foundOrder, node) {
+            for (let i = 0; i < foundOrder.length; i++) {
+                const middleware = foundOrder[i];
+                const correctIndex = CORRECT_PRE_ORDER_MIDDLEWARE_ORDER.indexOf(middleware);
+                if (correctIndex === -1) {
+                    context.report({
+                        node,
+                        messageId: 'unknownMiddleware',
+                        data: { middleware }
+                    });
+                    break;
+                }
+                if (i !== correctIndex) {
+                    context.report({
+                        node,
+                        messageId: 'incorrectPreOrderMiddlewarePosition',
+                        data: {
+                            middleware,
+                            currentPosition: i + 1,
+                            expectedPosition: correctIndex + 1
+                        }
+                    });
+                    break;
+                }
+            }
+        }
+        return {
+            VariableDeclarator: checkPreOrderMiddlewareOrder
+        };
+    },
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Enforces the correct order of pre-order middlewares',
+            recommended: 'error'
+        },
+        schema: [],
+        messages: {
+            incorrectPreOrderMiddlewarePosition: '{{middleware}} is at position {{currentPosition}}, but it should be at position {{expectedPosition}}.',
+            unknownMiddleware: '{{middleware}} is not recognized as a valid pre-order middleware.'
+        }
+    },
+    defaultOptions: []
+});

package/dist/rules/check-sentry-options.js

@@ -33,15 +33,6 @@ exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
                                         }
                                     }
                                 }
-                                if (key === 'tracesSampleRate') {
-                                    const traceRate = property.value.value;
-                                    if (traceRate !== 1.0) {
-                                        context.report({
-                                            node: property,
-                                            messageId: 'incorrectTraceRate'
-                                        });
-                                    }
-                                }
                             }
                         });
                     }
@@ -51,8 +42,7 @@ exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
     },
     meta: {
         messages: {
-            invalidDsn: 'Sentry DSN should not be empty.',
-            incorrectTraceRate: 'Sentry tracesSampleRate should be set to 1.0.'
+            invalidDsn: 'Sentry DSN should not be empty.'
         },
         type: 'problem',
         fixable: 'code',

package/dist/rules/index.js

@@ -17,6 +17,7 @@ const invalid_imports_1 = __importDefault(require("./invalid-imports"));
 const case_warning_1 = __importDefault(require("./case-warning"));
 const check_sentry_options_1 = __importDefault(require("./check-sentry-options"));
 const check_menu_depth_1 = __importDefault(require("./check-menu-depth"));
+const check_pre_order_middleware_order_1 = __importDefault(require("./check-pre-order-middleware-order"));
 const rules = {
     'client-url': client_url_1.default,
     'image-import': image_import_1.default,
@@ -30,6 +31,7 @@ const rules = {
     'invalid-imports': invalid_imports_1.default,
     'case-warning': case_warning_1.default,
     'check-sentry-options': check_sentry_options_1.default,
-    'check-menu-depth': check_menu_depth_1.default
+    'check-menu-depth': check_menu_depth_1.default,
+    'check-pre-order-middleware-order': check_pre_order_middleware_order_1.default
 };
 exports.rules = rules;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akinon/eslint-plugin-projectzero",
-  "version": "2.0.0-beta.2",
+  "version": "2.0.0-beta.20",
   "private": false,
   "description": "ESLint plugin for Project Zero Next",
   "main": "dist/index.js",
@@ -13,7 +13,7 @@
   "author": "",
   "license": "ISC",
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "8.18.2",
-    "@typescript-eslint/parser": "8.18.2"
+    "@typescript-eslint/eslint-plugin": "5.44.0",
+    "@typescript-eslint/parser": "5.44.0"
   }
 }

package/rules/check-pre-order-middleware-order.ts

@@ -0,0 +1,116 @@
+import {
+  AST_NODE_TYPES,
+  ESLintUtils,
+  TSESTree
+} from '@typescript-eslint/utils';
+import { RuleContext } from '@typescript-eslint/utils/dist/ts-eslint';
+
+const CORRECT_PRE_ORDER_MIDDLEWARE_ORDER = [
+  'shippingStepMiddleware',
+  'installmentOptionMiddleware',
+  'paymentOptionResetMiddleware',
+  'paymentOptionMiddleware',
+  'attributeBasedShippingOptionMiddleware',
+  'dataSourceShippingOptionMiddleware',
+  'shippingOptionMiddleware',
+  'setAddressMiddleware',
+  'deliveryOptionMiddleware',
+  'setPreOrderMiddleware',
+  'redirectionMiddleware',
+  'preOrderValidationMiddleware'
+];
+
+type MessageIds = 'incorrectPreOrderMiddlewarePosition' | 'unknownMiddleware';
+type Options = [];
+
+export default ESLintUtils.RuleCreator.withoutDocs({
+  create(context: RuleContext<MessageIds, Options>) {
+    function checkPreOrderMiddlewareOrder(
+      node: TSESTree.VariableDeclarator
+    ): void {
+      if (!isPreOrderMiddlewaresDeclaration(node)) {
+        return;
+      }
+
+      const arrayExpression = node.init as TSESTree.ArrayExpression;
+      const foundOrder = extractMiddlewareNames(arrayExpression);
+
+      validateMiddlewareOrder(foundOrder, node);
+    }
+
+    function isPreOrderMiddlewaresDeclaration(
+      node: TSESTree.VariableDeclarator
+    ): boolean {
+      return (
+        node.id.type === AST_NODE_TYPES.Identifier &&
+        node.id.name === 'preOrderMiddlewares' &&
+        node.init?.type === AST_NODE_TYPES.ArrayExpression
+      );
+    }
+
+    function extractMiddlewareNames(
+      arrayExpression: TSESTree.ArrayExpression
+    ): string[] {
+      return arrayExpression.elements
+        .map((element) => {
+          if (element?.type === AST_NODE_TYPES.Identifier) {
+            return element.name;
+          }
+          return null;
+        })
+        .filter((name): name is string => Boolean(name));
+    }
+
+    function validateMiddlewareOrder(
+      foundOrder: string[],
+      node: TSESTree.Node
+    ): void {
+      for (let i = 0; i < foundOrder.length; i++) {
+        const middleware = foundOrder[i];
+        const correctIndex =
+          CORRECT_PRE_ORDER_MIDDLEWARE_ORDER.indexOf(middleware);
+
+        if (correctIndex === -1) {
+          context.report({
+            node,
+            messageId: 'unknownMiddleware',
+            data: { middleware }
+          });
+          break;
+        }
+
+        if (i !== correctIndex) {
+          context.report({
+            node,
+            messageId: 'incorrectPreOrderMiddlewarePosition',
+            data: {
+              middleware,
+              currentPosition: i + 1,
+              expectedPosition: correctIndex + 1
+            }
+          });
+          break;
+        }
+      }
+    }
+
+    return {
+      VariableDeclarator: checkPreOrderMiddlewareOrder
+    };
+  },
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Enforces the correct order of pre-order middlewares',
+      recommended: 'error'
+    },
+    schema: [],
+    messages: {
+      incorrectPreOrderMiddlewarePosition:
+        '{{middleware}} is at position {{currentPosition}}, but it should be at position {{expectedPosition}}.',
+      unknownMiddleware:
+        '{{middleware}} is not recognized as a valid pre-order middleware.'
+    }
+  },
+  defaultOptions: []
+});

package/rules/check-sentry-options.ts

@@ -42,16 +42,6 @@ export default ESLintUtils.RuleCreator.withoutDocs({
                     }
                   }
                 }
-
-                if (key === 'tracesSampleRate') {
-                  const traceRate = (property.value as TSESTree.Literal).value;
-                  if (traceRate !== 1.0) {
-                    context.report({
-                      node: property,
-                      messageId: 'incorrectTraceRate'
-                    });
-                  }
-                }
               }
             });
           }
@@ -61,8 +51,7 @@ export default ESLintUtils.RuleCreator.withoutDocs({
   },
   meta: {
     messages: {
-      invalidDsn: 'Sentry DSN should not be empty.',
-      incorrectTraceRate: 'Sentry tracesSampleRate should be set to 1.0.'
+      invalidDsn: 'Sentry DSN should not be empty.'
     },
     type: 'problem',
     fixable: 'code',

package/rules/index.ts

@@ -11,6 +11,7 @@ import invalidImports from './invalid-imports';
 import caseWarning from './case-warning';
 import checkSentryOptions from './check-sentry-options';
 import checkMenuDepth from './check-menu-depth';
+import checkPreOrderMiddlewareOrder from './check-pre-order-middleware-order';
 
 const rules = {
   'client-url': clientUrl,
@@ -25,7 +26,8 @@ const rules = {
   'invalid-imports': invalidImports,
   'case-warning': caseWarning,
   'check-sentry-options': checkSentryOptions,
-  'check-menu-depth': checkMenuDepth
+  'check-menu-depth': checkMenuDepth,
+  'check-pre-order-middleware-order': checkPreOrderMiddlewareOrder
 };
 
 export { rules };