@alextheman/eslint-plugin 1.8.2 → 1.10.0

package/dist/index.cjs

@@ -3706,7 +3706,7 @@ module.exports = __toCommonJS(index_exports);
 
 // package.json
 var name = "@alextheman/eslint-plugin";
-var version = "1.8.2";
+var version = "1.10.0";
 
 // src/configs/alexPluginBase.ts
 function createAlexPluginBaseConfig(plugin2) {
@@ -3717,7 +3717,8 @@ function createAlexPluginBaseConfig(plugin2) {
       },
       rules: {
         "@alextheman/no-namespace-imports": "error",
-        "@alextheman/no-relative-imports": "error"
+        "@alextheman/no-relative-imports": "error",
+        "@alextheman/use-object-shorthand": "error"
       }
     }
   ];
@@ -3982,59 +3983,125 @@ var noPluginConfigAccessFromSrcConfigs = create_rule_default({
 var no_plugin_configs_access_from_src_configs_default = noPluginConfigAccessFromSrcConfigs;
 
 // src/rules/no-relative-imports.ts
-var import_path = __toESM(require("path"), 1);
 var noRelativeImports = create_rule_default({
   name: "no-relative-imports",
   meta: {
     docs: { description: "Forbid the use of relative imports" },
     messages: {
       message: 'Relative import from "{{source}}" is not allowed.',
-      stupidPath: 'Who the hell imports from "{{source}}"?! Know your own project directory, Goddamnit!'
+      stupidPath: "For the love of God, please do not mix relative path parts in your import statements like that! How can you possibly be ok with {{source}}?!"
     },
     type: "suggestion",
-    fixable: "code",
-    schema: []
+    schema: [
+      {
+        type: "object",
+        properties: {
+          depth: {
+            type: "number"
+          }
+        },
+        additionalProperties: false
+      }
+    ]
   },
-  defaultOptions: [],
+  defaultOptions: [{ depth: void 0 }],
   create(context) {
+    var _a;
+    const depth = (_a = context.options[0]) == null ? void 0 : _a.depth;
+    if (depth !== void 0) {
+      if (depth % 1 !== 0) {
+        throw new Error("NON_INTEGER_DEPTH_NOT_ALLOWED");
+      }
+      if (depth < 0) {
+        throw new Error("NEGATIVE_DEPTH_NOT_ALLOWED");
+      }
+    }
     return {
       ImportDeclaration(node) {
         if (node.source.value.includes("./") || node.source.value.includes("../")) {
-          if (!node.source.value.startsWith("./") && !node.source.value.startsWith("../")) {
-            context.report({
+          if (depth === void 0) {
+            return context.report({
+              node,
+              messageId: "message",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+          const importPathParts = node.source.value.split("/");
+          if (importPathParts.includes(".") && importPathParts.includes("..")) {
+            return context.report({
               node,
               messageId: "stupidPath",
               data: {
                 source: node.source.value
               }
             });
-            return null;
           }
+          if (importPathParts.includes(".") && importPathParts[0] !== "." || importPathParts.includes("..") && importPathParts[0] !== "..") {
+            return context.report({
+              node,
+              messageId: "stupidPath",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+          if (depth === 0 && importPathParts[0] === ".") {
+            return;
+          }
+          let endOfRelativePathFound = false;
+          for (const part of importPathParts.slice(0, depth + 1)) {
+            if (part !== "..") {
+              endOfRelativePathFound = true;
+              break;
+            }
+          }
+          if (!endOfRelativePathFound) {
+            return context.report({
+              node,
+              messageId: "message",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+        }
+      }
+    };
+  }
+});
+var no_relative_imports_default = noRelativeImports;
+
+// src/rules/use-object-shorthand.ts
+var import_utils2 = require("@typescript-eslint/utils");
+var useObjectShorthand = create_rule_default({
+  name: "use-object-shorthand",
+  meta: {
+    docs: {
+      description: 'Encourage the use of object shorthand (e.g. const property = "Hello"; const object = { property });'
+    },
+    messages: {
+      useShorthand: "{ {{source}} } is not allowed. Please use the object shorthand."
+    },
+    type: "suggestion",
+    fixable: "code",
+    schema: []
+  },
+  defaultOptions: [],
+  create(context) {
+    return {
+      Property(node) {
+        if (node.key.type === import_utils2.AST_NODE_TYPES.Identifier && node.value.type === import_utils2.AST_NODE_TYPES.Identifier && node.key.name === node.value.name && !node.shorthand) {
           context.report({
             node,
-            messageId: "message",
+            messageId: "useShorthand",
             data: {
-              source: node.source.value
+              source: context.sourceCode.getText(node)
             },
             fix(fixer) {
-              if (!context.parserOptions.tsconfigRootDir) {
-                return null;
-              }
-              const fullImportPath = import_path.default.resolve(
-                import_path.default.dirname(context.physicalFilename),
-                node.source.value
-              );
-              const projectRelativePath = import_path.default.relative(
-                context.parserOptions.tsconfigRootDir,
-                fullImportPath
-              );
-              if (projectRelativePath.startsWith("..")) {
-                return null;
-              }
-              return fixer.replaceText(
-                node.source,
-                `${node.source.raw[0]}${import_path.default.posix.normalize(projectRelativePath)}${node.source.raw[0]}`
-              );
+              const key = node.key;
+              return fixer.replaceTextRange([node.range[0], node.range[1]], key.name);
             }
           });
         }
@@ -4042,13 +4109,14 @@ var noRelativeImports = create_rule_default({
     };
   }
 });
-var no_relative_imports_default = noRelativeImports;
+var use_object_shorthand_default = useObjectShorthand;
 
 // src/rules/index.ts
 var rules_default = {
   "no-namespace-imports": no_namespace_imports_default,
   "no-relative-imports": no_relative_imports_default,
-  "no-plugin-configs-access-from-src-configs": no_plugin_configs_access_from_src_configs_default
+  "no-plugin-configs-access-from-src-configs": no_plugin_configs_access_from_src_configs_default,
+  "use-object-shorthand": use_object_shorthand_default
 };
 
 // src/index.ts

package/dist/index.d.cts

@@ -2,7 +2,7 @@ import { Linter } from 'eslint';
 import { Config } from 'prettier';
 
 var name = "@alextheman/eslint-plugin";
-var version = "1.8.2";
+var version = "1.10.0";
 
 declare const prettierRules: Config;
 

package/dist/index.d.ts

@@ -2,7 +2,7 @@ import { Linter } from 'eslint';
 import { Config } from 'prettier';
 
 var name = "@alextheman/eslint-plugin";
-var version = "1.8.2";
+var version = "1.10.0";
 
 declare const prettierRules: Config;
 

package/dist/index.js

@@ -3692,7 +3692,7 @@ var require_globals2 = __commonJS({
 
 // package.json
 var name = "@alextheman/eslint-plugin";
-var version = "1.8.2";
+var version = "1.10.0";
 
 // src/configs/alexPluginBase.ts
 function createAlexPluginBaseConfig(plugin2) {
@@ -3703,7 +3703,8 @@ function createAlexPluginBaseConfig(plugin2) {
       },
       rules: {
         "@alextheman/no-namespace-imports": "error",
-        "@alextheman/no-relative-imports": "error"
+        "@alextheman/no-relative-imports": "error",
+        "@alextheman/use-object-shorthand": "error"
       }
     }
   ];
@@ -3968,59 +3969,125 @@ var noPluginConfigAccessFromSrcConfigs = create_rule_default({
 var no_plugin_configs_access_from_src_configs_default = noPluginConfigAccessFromSrcConfigs;
 
 // src/rules/no-relative-imports.ts
-import path from "path";
 var noRelativeImports = create_rule_default({
   name: "no-relative-imports",
   meta: {
     docs: { description: "Forbid the use of relative imports" },
     messages: {
       message: 'Relative import from "{{source}}" is not allowed.',
-      stupidPath: 'Who the hell imports from "{{source}}"?! Know your own project directory, Goddamnit!'
+      stupidPath: "For the love of God, please do not mix relative path parts in your import statements like that! How can you possibly be ok with {{source}}?!"
     },
     type: "suggestion",
-    fixable: "code",
-    schema: []
+    schema: [
+      {
+        type: "object",
+        properties: {
+          depth: {
+            type: "number"
+          }
+        },
+        additionalProperties: false
+      }
+    ]
   },
-  defaultOptions: [],
+  defaultOptions: [{ depth: void 0 }],
   create(context) {
+    var _a;
+    const depth = (_a = context.options[0]) == null ? void 0 : _a.depth;
+    if (depth !== void 0) {
+      if (depth % 1 !== 0) {
+        throw new Error("NON_INTEGER_DEPTH_NOT_ALLOWED");
+      }
+      if (depth < 0) {
+        throw new Error("NEGATIVE_DEPTH_NOT_ALLOWED");
+      }
+    }
     return {
       ImportDeclaration(node) {
         if (node.source.value.includes("./") || node.source.value.includes("../")) {
-          if (!node.source.value.startsWith("./") && !node.source.value.startsWith("../")) {
-            context.report({
+          if (depth === void 0) {
+            return context.report({
+              node,
+              messageId: "message",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+          const importPathParts = node.source.value.split("/");
+          if (importPathParts.includes(".") && importPathParts.includes("..")) {
+            return context.report({
               node,
               messageId: "stupidPath",
               data: {
                 source: node.source.value
               }
             });
-            return null;
           }
+          if (importPathParts.includes(".") && importPathParts[0] !== "." || importPathParts.includes("..") && importPathParts[0] !== "..") {
+            return context.report({
+              node,
+              messageId: "stupidPath",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+          if (depth === 0 && importPathParts[0] === ".") {
+            return;
+          }
+          let endOfRelativePathFound = false;
+          for (const part of importPathParts.slice(0, depth + 1)) {
+            if (part !== "..") {
+              endOfRelativePathFound = true;
+              break;
+            }
+          }
+          if (!endOfRelativePathFound) {
+            return context.report({
+              node,
+              messageId: "message",
+              data: {
+                source: node.source.value
+              }
+            });
+          }
+        }
+      }
+    };
+  }
+});
+var no_relative_imports_default = noRelativeImports;
+
+// src/rules/use-object-shorthand.ts
+import { AST_NODE_TYPES } from "@typescript-eslint/utils";
+var useObjectShorthand = create_rule_default({
+  name: "use-object-shorthand",
+  meta: {
+    docs: {
+      description: 'Encourage the use of object shorthand (e.g. const property = "Hello"; const object = { property });'
+    },
+    messages: {
+      useShorthand: "{ {{source}} } is not allowed. Please use the object shorthand."
+    },
+    type: "suggestion",
+    fixable: "code",
+    schema: []
+  },
+  defaultOptions: [],
+  create(context) {
+    return {
+      Property(node) {
+        if (node.key.type === AST_NODE_TYPES.Identifier && node.value.type === AST_NODE_TYPES.Identifier && node.key.name === node.value.name && !node.shorthand) {
           context.report({
             node,
-            messageId: "message",
+            messageId: "useShorthand",
             data: {
-              source: node.source.value
+              source: context.sourceCode.getText(node)
             },
             fix(fixer) {
-              if (!context.parserOptions.tsconfigRootDir) {
-                return null;
-              }
-              const fullImportPath = path.resolve(
-                path.dirname(context.physicalFilename),
-                node.source.value
-              );
-              const projectRelativePath = path.relative(
-                context.parserOptions.tsconfigRootDir,
-                fullImportPath
-              );
-              if (projectRelativePath.startsWith("..")) {
-                return null;
-              }
-              return fixer.replaceText(
-                node.source,
-                `${node.source.raw[0]}${path.posix.normalize(projectRelativePath)}${node.source.raw[0]}`
-              );
+              const key = node.key;
+              return fixer.replaceTextRange([node.range[0], node.range[1]], key.name);
             }
           });
         }
@@ -4028,13 +4095,14 @@ var noRelativeImports = create_rule_default({
     };
   }
 });
-var no_relative_imports_default = noRelativeImports;
+var use_object_shorthand_default = useObjectShorthand;
 
 // src/rules/index.ts
 var rules_default = {
   "no-namespace-imports": no_namespace_imports_default,
   "no-relative-imports": no_relative_imports_default,
-  "no-plugin-configs-access-from-src-configs": no_plugin_configs_access_from_src_configs_default
+  "no-plugin-configs-access-from-src-configs": no_plugin_configs_access_from_src_configs_default,
+  "use-object-shorthand": use_object_shorthand_default
 };
 
 // src/index.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alextheman/eslint-plugin",
-  "version": "1.8.2",
+  "version": "1.10.0",
   "description": "",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",