@akinon/eslint-plugin-projectzero 1.42.0-rc.5 → 1.42.0-rc.7

package/CHANGELOG.md

@@ -1,7 +1,15 @@
 # @akinon/eslint-plugin-projectzero
 
+## 1.42.0-rc.7
+
+## 1.42.0-rc.6
+
 ## 1.42.0-rc.5
 
+### Minor Changes
+
+- 5a4c607: ZERO-2764: Add case-warning rule to eslint-plugin-projectzero
+
 ## 1.42.0-rc.4
 
 ## 1.42.0-rc.3

package/configs/core.ts

@@ -10,6 +10,7 @@ export default {
     '@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,23 +1,25 @@
-"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',
-        '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',
+    '@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'"
+      }
+    ]
+  }
 };

package/dist/rules/case-warning.js

@@ -0,0 +1,32 @@
+"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/index.js

@@ -1,31 +1,39 @@
-"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 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 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
+  '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
 };
 exports.rules = rules;

package/package.json

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

package/rules/case-warning.ts

@@ -0,0 +1,39 @@
+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/index.ts

@@ -9,6 +9,7 @@ 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,7 +22,8 @@ const rules = {
   'check-middleware-order': checkMiddlewareOrder,
   'urls-without-slash': urlsWithoutSlash,
   'invalid-imports': invalidImports,
-  'check-menu-depth': checkMenuDepth
+  'check-menu-depth': checkMenuDepth,
+  'case-warning': caseWarning
 };
 
 export { rules };