@akinon/eslint-plugin-projectzero 1.43.0-rc.6 → 1.43.0

package/CHANGELOG.md

@@ -1,33 +1,6 @@
 # @akinon/eslint-plugin-projectzero
 
-## 1.43.0-rc.6
-
-### Minor Changes
-
-- 5a4c607: ZERO-2764: Add case-warning rule to eslint-plugin-projectzero
-- 64699d3: ZERO-2761: Fix invalid import for plugin module
-- c474780: ZERO-2666: Add rule to check menu depth
-
-## 1.43.0-rc.5
-
-## 1.43.0-rc.4
-
-## 1.43.0-rc.3
-
-### Minor Changes
-
-- 64699d3: ZERO-2761: Fix invalid import for plugin module
-
-## 1.43.0-rc.2
-
-## 1.43.0-rc.1
-
-## 1.43.0-rc.0
-
-### Minor Changes
-
-- 5a4c607: ZERO-2764: Add case-warning rule to eslint-plugin-projectzero
-- c474780: ZERO-2666: Add rule to check menu depth
+## 1.43.0
 
 ## 1.42.0
 

package/configs/core.ts

@@ -9,8 +9,6 @@ export default {
     '@akinon/projectzero/meta-data-import': 'error',
     '@akinon/projectzero/urls-without-slash': 'error',
     '@akinon/projectzero/invalid-imports': 'error',
-    '@akinon/projectzero/check-menu-depth': 'error',
-    '@akinon/projectzero/case-warning': 'warn',
     'no-restricted-syntax': [
       'error',
       {

package/dist/configs/core.js

@@ -1,25 +1,22 @@
-'use strict';
-Object.defineProperty(exports, '__esModule', { value: true });
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = {
-  rules: {
-    '@akinon/projectzero/image-import': 'error',
-    '@akinon/projectzero/client-url': 'error',
-    '@akinon/projectzero/link-import': 'error',
-    '@akinon/projectzero/router-import': 'error',
-    '@akinon/projectzero/skip-trailing-slash-redirect': 'error',
-    '@akinon/projectzero/check-locale': 'error',
-    '@akinon/projectzero/meta-data-import': 'error',
-    '@akinon/projectzero/urls-without-slash': 'error',
-    '@akinon/projectzero/invalid-imports': 'error',
-    '@akinon/projectzero/check-menu-depth': 'error',
-    '@akinon/projectzero/case-warning': 'warn',
-    'no-restricted-syntax': [
-      'error',
-      {
-        selector:
-          "MemberExpression[property.name='COMMERCE_URL'][object.property.name='env'][object.object.name='process']",
-        message: "You should use it as 'Settings.commerceUrl'"
-      }
-    ]
-  }
+    rules: {
+        '@akinon/projectzero/image-import': 'error',
+        '@akinon/projectzero/client-url': 'error',
+        '@akinon/projectzero/link-import': 'error',
+        '@akinon/projectzero/router-import': 'error',
+        '@akinon/projectzero/skip-trailing-slash-redirect': 'error',
+        '@akinon/projectzero/check-locale': 'error',
+        '@akinon/projectzero/meta-data-import': 'error',
+        '@akinon/projectzero/urls-without-slash': 'error',
+        '@akinon/projectzero/invalid-imports': 'error',
+        'no-restricted-syntax': [
+            'error',
+            {
+                selector: "MemberExpression[property.name='COMMERCE_URL'][object.property.name='env'][object.object.name='process']",
+                message: "You should use it as 'Settings.commerceUrl'"
+            }
+        ]
+    }
 };

package/dist/rules/index.js

@@ -1,39 +1,29 @@
-'use strict';
-var __importDefault =
-  (this && this.__importDefault) ||
-  function (mod) {
-    return mod && mod.__esModule ? mod : { default: mod };
-  };
-Object.defineProperty(exports, '__esModule', { value: true });
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
 exports.rules = void 0;
-const client_url_1 = __importDefault(require('./client-url'));
-const image_import_1 = __importDefault(require('./image-import'));
-const link_import_1 = __importDefault(require('./link-import'));
-const router_import_1 = __importDefault(require('./router-import'));
-const skip_trailing_slash_redirect_1 = __importDefault(
-  require('./skip-trailing-slash-redirect')
-);
-const check_locale_1 = __importDefault(require('./check-locale'));
-const meta_data_import_1 = __importDefault(require('./meta-data-import'));
-const check_middleware_order_1 = __importDefault(
-  require('./check-middleware-order')
-);
-const urls_without_slash_1 = __importDefault(require('./urls-without-slash'));
-const invalid_imports_1 = __importDefault(require('./invalid-imports'));
-const check_menu_depth_1 = __importDefault(require('./check-menu-depth'));
-const case_warning_1 = __importDefault(require('./case-warning'));
+const client_url_1 = __importDefault(require("./client-url"));
+const image_import_1 = __importDefault(require("./image-import"));
+const link_import_1 = __importDefault(require("./link-import"));
+const router_import_1 = __importDefault(require("./router-import"));
+const skip_trailing_slash_redirect_1 = __importDefault(require("./skip-trailing-slash-redirect"));
+const check_locale_1 = __importDefault(require("./check-locale"));
+const meta_data_import_1 = __importDefault(require("./meta-data-import"));
+const check_middleware_order_1 = __importDefault(require("./check-middleware-order"));
+const urls_without_slash_1 = __importDefault(require("./urls-without-slash"));
+const invalid_imports_1 = __importDefault(require("./invalid-imports"));
 const rules = {
-  'client-url': client_url_1.default,
-  'image-import': image_import_1.default,
-  'link-import': link_import_1.default,
-  'router-import': router_import_1.default,
-  'skip-trailing-slash-redirect': skip_trailing_slash_redirect_1.default,
-  'check-locale': check_locale_1.default,
-  'meta-data-import': meta_data_import_1.default,
-  'check-middleware-order': check_middleware_order_1.default,
-  'urls-without-slash': urls_without_slash_1.default,
-  'invalid-imports': invalid_imports_1.default,
-  'check-menu-depth': check_menu_depth_1.default,
-  'case-warning': case_warning_1.default
+    'client-url': client_url_1.default,
+    'image-import': image_import_1.default,
+    'link-import': link_import_1.default,
+    'router-import': router_import_1.default,
+    'skip-trailing-slash-redirect': skip_trailing_slash_redirect_1.default,
+    'check-locale': check_locale_1.default,
+    'meta-data-import': meta_data_import_1.default,
+    'check-middleware-order': check_middleware_order_1.default,
+    'urls-without-slash': urls_without_slash_1.default,
+    'invalid-imports': invalid_imports_1.default
 };
 exports.rules = rules;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akinon/eslint-plugin-projectzero",
-  "version": "1.43.0-rc.6",
+  "version": "1.43.0",
   "private": false,
   "description": "ESLint plugin for Project Zero Next",
   "main": "dist/index.js",

package/rules/index.ts

@@ -8,8 +8,6 @@ import metaDataImport from './meta-data-import';
 import checkMiddlewareOrder from './check-middleware-order';
 import urlsWithoutSlash from './urls-without-slash';
 import invalidImports from './invalid-imports';
-import checkMenuDepth from './check-menu-depth';
-import caseWarning from './case-warning';
 
 const rules = {
   'client-url': clientUrl,
@@ -21,9 +19,7 @@ const rules = {
   'meta-data-import': metaDataImport,
   'check-middleware-order': checkMiddlewareOrder,
   'urls-without-slash': urlsWithoutSlash,
-  'invalid-imports': invalidImports,
-  'check-menu-depth': checkMenuDepth,
-  'case-warning': caseWarning
+  'invalid-imports': invalidImports
 };
 
 export { rules };

package/dist/rules/case-warning.js

@@ -1,32 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("@typescript-eslint/utils");
-exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
-    create(context) {
-        return {
-            CallExpression(node) {
-                if (node.callee.type === 'MemberExpression' &&
-                    node.callee.property.type === 'Identifier' &&
-                    (node.callee.property.name === 'toLocaleUpperCase' ||
-                        node.callee.property.name === 'toLocaleLowerCase')) {
-                    const args = node.arguments;
-                    if (args.length === 0) {
-                        context.report({
-                            node,
-                            messageId: 'noLocale'
-                        });
-                    }
-                }
-            }
-        };
-    },
-    meta: {
-        messages: {
-            noLocale: 'toLocaleUpperCase() or toLocaleLowerCase() should specify a locale for consistent behavior across different environments.'
-        },
-        type: 'problem',
-        fixable: 'code',
-        schema: []
-    },
-    defaultOptions: []
-});

package/dist/rules/check-menu-depth.js

@@ -1,43 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-const utils_1 = require("@typescript-eslint/utils");
-const types_1 = require("@typescript-eslint/types");
-exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
-    create: function (context) {
-        return {
-            CallExpression(node) {
-                const { callee, arguments: args } = node;
-                if (callee.type !== types_1.AST_NODE_TYPES.Identifier ||
-                    callee.name !== 'getMenu' ||
-                    args.length === 0 ||
-                    args[0].type !== types_1.AST_NODE_TYPES.ObjectExpression) {
-                    return;
-                }
-                const depthProperty = args[0].properties.find((prop) => {
-                    var _a;
-                    return prop.type === types_1.AST_NODE_TYPES.Property &&
-                        prop.key.type === types_1.AST_NODE_TYPES.Identifier &&
-                        prop.key.name === 'depth' &&
-                        ((_a = prop.value) === null || _a === void 0 ? void 0 : _a.type) === types_1.AST_NODE_TYPES.Literal &&
-                        typeof prop.value.value === 'number' &&
-                        prop.value.value > 3;
-                });
-                if (depthProperty) {
-                    context.report({
-                        node: depthProperty,
-                        messageId: 'depthMessage'
-                    });
-                }
-            }
-        };
-    },
-    meta: {
-        type: 'problem',
-        fixable: 'code',
-        messages: {
-            depthMessage: 'A request for a menu deeper than 3 levels from the server can lead to unnecessary memory usage and unwanted outcomes. Therefore, sending a request for a 3-level menu on the client side and making additional requests on hover for deeper levels may be more efficient.'
-        },
-        schema: []
-    },
-    defaultOptions: []
-});

package/rules/case-warning.ts

@@ -1,39 +0,0 @@
-import { ESLintUtils } from '@typescript-eslint/utils';
-import { TSESTree } from '@typescript-eslint/types';
-
-type MessageIds = 'noLocale';
-type Options = readonly [];
-
-export default ESLintUtils.RuleCreator.withoutDocs<Options, MessageIds>({
-  create(context) {
-    return {
-      CallExpression(node: TSESTree.CallExpression) {
-        if (
-          node.callee.type === 'MemberExpression' &&
-          node.callee.property.type === 'Identifier' &&
-          (node.callee.property.name === 'toLocaleUpperCase' ||
-            node.callee.property.name === 'toLocaleLowerCase')
-        ) {
-          const args = node.arguments;
-
-          if (args.length === 0) {
-            context.report({
-              node,
-              messageId: 'noLocale'
-            });
-          }
-        }
-      }
-    };
-  },
-  meta: {
-    messages: {
-      noLocale:
-        'toLocaleUpperCase() or toLocaleLowerCase() should specify a locale for consistent behavior across different environments.'
-    },
-    type: 'problem',
-    fixable: 'code',
-    schema: []
-  },
-  defaultOptions: []
-});

package/rules/check-menu-depth.ts

@@ -1,48 +0,0 @@
-import { ESLintUtils } from '@typescript-eslint/utils';
-import { AST_NODE_TYPES } from '@typescript-eslint/types';
-
-export default ESLintUtils.RuleCreator.withoutDocs({
-  create: function (context) {
-    return {
-      CallExpression(node) {
-        const { callee, arguments: args } = node;
-
-        if (
-          callee.type !== AST_NODE_TYPES.Identifier ||
-          callee.name !== 'getMenu' ||
-          args.length === 0 ||
-          args[0].type !== AST_NODE_TYPES.ObjectExpression
-        ) {
-          return;
-        }
-
-        const depthProperty = args[0].properties.find(
-          (prop) =>
-            prop.type === AST_NODE_TYPES.Property &&
-            prop.key.type === AST_NODE_TYPES.Identifier &&
-            prop.key.name === 'depth' &&
-            prop.value?.type === AST_NODE_TYPES.Literal &&
-            typeof prop.value.value === 'number' &&
-            prop.value.value > 3
-        );
-
-        if (depthProperty) {
-          context.report({
-            node: depthProperty,
-            messageId: 'depthMessage'
-          });
-        }
-      }
-    };
-  },
-  meta: {
-    type: 'problem',
-    fixable: 'code',
-    messages: {
-      depthMessage:
-        'A request for a menu deeper than 3 levels from the server can lead to unnecessary memory usage and unwanted outcomes. Therefore, sending a request for a 3-level menu on the client side and making additional requests on hover for deeper levels may be more efficient.'
-    },
-    schema: []
-  },
-  defaultOptions: []
-});