@simplysm/eslint-plugin 10.0.59 → 10.0.61

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@simplysm/eslint-plugin",
-  "version": "10.0.59",
+  "version": "10.0.61",
   "description": "심플리즘 패키지 - ESLINT 플러그인",
   "author": "김석래",
   "repository": {
@@ -12,7 +12,7 @@
   "type": "module",
   "main": "src/index.cjs",
   "engines": {
-    "node": "^18"
+    "node": "^16"
   },
   "dependencies": {
     "@typescript-eslint/eslint-plugin": "^6.7.3",
@@ -21,8 +21,8 @@
     "eslint": "^8.50.0",
     "eslint-import-resolver-typescript": "^3.6.1",
     "eslint-module-utils": "^2.8.0",
-    "eslint-plugin-deprecation": "^2.0.0",
     "eslint-plugin-import": "^2.28.1",
+    "eslint-plugin-react": "^7.33.2",
     "eslint-plugin-react-refresh": "^0.4.3",
     "typescript": "^4.9.5"
   }

package/src/configs/root.cjs

@@ -32,6 +32,8 @@ module.exports = {
         "no-unused-vars": ["error"],
         "no-undef": ["error"],
         // "linebreak-style": ["error", "unix"]
+
+        "arrow-parens": ["error"]
       },
     },
     {
@@ -45,10 +47,10 @@ module.exports = {
       },
       plugins: [
         "@typescript-eslint",
-        "deprecation",
         "import",
         "@simplysm",
-        "react-refresh"
+        "react-refresh",
+        "react"
       ],
       settings: {
         "import/parsers": {
@@ -58,26 +60,22 @@ module.exports = {
           "typescript": {
             project: "packages/*/tsconfig.json",
           }
+        },
+        "react": {
+          version: "detect"
         }
       },
       rules: {
         // 기본
         "no-console": ["warn"],
         "no-warning-comments": ["warn"],
+        "arrow-parens": ["error"],
         // "linebreak-style": ["error", "unix"],
 
-        // Deprecation
-        // "deprecation/deprecation": ["warn"], // 느림
-
         // import
         "import/no-extraneous-dependencies": ["error"], // 느림
         // "import/no-duplicates": ["error"], // 느림
 
-        // 심플리즘
-        "@simplysm/ts-no-throw-not-implement-error": ["warn"],
-        "@simplysm/ts-no-self-entry-import": ["error"],
-        "@simplysm/ts-no-external-import": ["error"],
-
         // REACT
         "react-refresh/only-export-components": ["warn"],
 
@@ -111,12 +109,40 @@ module.exports = {
         "@typescript-eslint/no-unused-expressions": ["error"],
         "@typescript-eslint/no-unused-vars": ["error", {args: "none"}],
         "@typescript-eslint/strict-boolean-expressions": ["error", {
-          allowString: true,
-          allowNumber: true,
+          // allowString: true,
+          // allowNumber: true,
           allowNullableBoolean: true,
           allowNullableObject: true
         }],
-        "@typescript-eslint/prefer-ts-expect-error": ["error"]
+        "@typescript-eslint/prefer-ts-expect-error": ["error"],
+
+        // REACT
+        "react/jsx-curly-brace-presence": ["error", {props: "always", propElementValues: "always"}],
+        "react/jsx-key": ["error", {
+          checkFragmentShorthand: true,
+          checkKeyMustBeforeSpread: true,
+          warnOnDuplicates: true
+        }],
+        "react/jsx-no-useless-fragment": "error",
+        "react/jsx-pascal-case": "error",
+        "react/jsx-wrap-multilines": ["error", {
+          "declaration": "parens",
+          "assignment": "parens",
+          "return": "parens",
+          "arrow": "parens",
+          "condition": "parens",
+          "logical": "parens",
+          "prop": "parens"
+        }],
+        "react/no-array-index-key": "error",
+        "react/no-deprecated": "warn",
+
+
+        // 심플리즘
+        "@simplysm/ts-no-throw-not-implement-error": ["warn"],
+        "@simplysm/ts-no-self-entry-import": ["error"],
+        "@simplysm/ts-no-external-import": ["error"],
+        "@simplysm/jsx-no-prop-object": ["error"],
       }
     }
   ]

package/src/index.cjs

@@ -5,6 +5,7 @@ module.exports = {
   rules: {
     "ts-no-external-import": require("./rules/ts-no-external-import.cjs"),
     "ts-no-self-entry-import": require("./rules/ts-no-self-entry-import.cjs"),
-    "ts-no-throw-not-implement-error": require("./rules/ts-no-throw-not-implement-error.cjs")
+    "ts-no-throw-not-implement-error": require("./rules/ts-no-throw-not-implement-error.cjs"),
+    "jsx-no-prop-object": require("./rules/jsx-no-direct-object.cjs")
   }
 };

package/src/rules/jsx-no-direct-object.cjs

@@ -0,0 +1,58 @@
+module.exports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "'jsx'에 직접적으로 Object/Array/Function를 입력할 수 없음",
+    },
+    fixable: false,
+    schema: [],
+  },
+  create: (context) => {
+    let scope = context.getScope();
+    let variables = scope.variables;
+
+    while (scope.type !== 'global') {
+      scope = scope.upper;
+      variables = scope.variables.concat(variables);
+    }
+    if (scope.childScopes.length) {
+      variables = scope.childScopes[0].variables.concat(variables);
+      if (scope.childScopes[0].childScopes.length) {
+        variables = scope.childScopes[0].childScopes[0].variables.concat(variables);
+      }
+    }
+    variables = variables.reverse();
+
+    return {
+      JSXExpressionContainer(node) {
+        if (
+          node.type !== 'JSXExpressionContainer'
+          || node.expression.type === "ArrayExpression"
+          || node.expression.type === "ObjectExpression"
+          || node.expression.type === "ArrowFunctionExpression"
+          || node.expression.type === "FunctionExpression"
+        ) {
+          context.report({
+            node,
+            message: "'jsx'에 직접적으로 Object/Array/Function을 넣을 수 없습니다. (필요시, $.obj, $.fn, $.getter 활용)"
+          });
+        }
+        else if (node.expression.type === "Identifier") {
+          const variable = variables.find((item) => item.name === node.expression.name);
+
+          if (
+            variable?.defs[0]?.node.init?.type === "ArrayExpression"
+            || variable?.defs[0]?.node.init?.type === "ObjectExpression"
+            || variable?.defs[0]?.node.init?.type === "ArrowFunctionExpression"
+            || variable?.defs[0]?.node.init?.type === "FunctionExpression"
+          ) {
+            context.report({
+              node,
+              message: "'jsx'에 직접적으로 Object/Array/Function을 넣을 수 없습니다. (필요시, $.obj, $.fn, $.getter 활용)"
+            });
+          }
+        }
+      }
+    };
+  },
+};

package/src/rules/ts-no-throw-not-implement-error.cjs

@@ -1,4 +1,4 @@
-const { AST_NODE_TYPES } = require("@typescript-eslint/experimental-utils");
+const {AST_NODE_TYPES} = require("@typescript-eslint/experimental-utils");
 
 module.exports = {
   meta: {