Merge pull request #107 from rootstrap/feature/eslint-new-rules

ESLint: Update rules

Author: guillermomachado

.eslintrc.js

@@ -3,38 +3,39 @@ const path = require('path');
 module.exports = {
   // Configuration for JavaScript files
   extends: [
-    'expo', 
-    'plugin:tailwindcss/recommended', 
+    'expo',
+    'plugin:tailwindcss/recommended',
     'prettier',
-    'eslint:recommended'
+    'eslint:recommended',
+    'plugin:@tanstack/query/recommended',
   ],
   env: {
     'jest/globals': true,
-    'node': true
+    node: true,
   },
   plugins: [
-    'unicorn', 
+    'unicorn',
     '@typescript-eslint',
     'unused-imports',
     'tailwindcss',
     'simple-import-sort',
     'sonarjs',
-    'jest'
+    'jest',
   ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
     project: './tsconfig.json',
   },
   ignorePatterns: [
-    "node_modules",
-    "*.config.js",
-    "docs",
-    "cli",
-    "android",
-    "ios",
-    "lint-staged.config.js",
-    "i18next-syntax-validation.js",
-    ".eslintrc.js"
+    'node_modules',
+    '*.config.js',
+    'docs',
+    'cli',
+    'android',
+    'ios',
+    'lint-staged.config.js',
+    'i18next-syntax-validation.js',
+    '.eslintrc.js',
   ],
   rules: {
     'import/no-duplicates': 'error',
@@ -81,7 +82,7 @@ module.exports = {
         argsIgnorePattern: '^_',
         varsIgnorePattern: '^_',
         caughtErrorsIgnorePattern: '^_',
-      }
+      },
     ],
     curly: [2, 'all'],
     'prefer-const': [
@@ -91,16 +92,20 @@ module.exports = {
       },
     ],
     'object-shorthand': 'error',
-    'arrow-body-style': ["error", "as-needed"],
-    'no-console': ['error', {allow: ['error']}],
+    'arrow-body-style': ['error', 'as-needed'],
+    'no-console': ['error', { allow: ['error'] }],
     'guard-for-in': 'error',
-    '@typescript-eslint/no-magic-numbers': ["error",
-       { ignoreArrayIndexes: true, 
-        ignoreEnums: true,
-        ignore: [-1, 0, 1]
-      }
+    '@typescript-eslint/no-magic-numbers': [
+      'error',
+      { ignoreArrayIndexes: true, ignoreEnums: true, ignore: [-1, 0, 1] },
     ],
-    '@typescript-eslint/prefer-nullish-coalescing': "error"
+    '@typescript-eslint/prefer-nullish-coalescing': 'error',
+    'react/jsx-fragments': ['error', 'syntax'],
+    'react/jsx-no-useless-fragment': 'error',
+    'react/no-children-prop': ['error', { allowFunctions: true }],
+    'no-nested-ternary': 'error',
+    'no-unneeded-ternary': 'error',
+    'prefer-template': 'error',
   },
   overrides: [
     // Configuration for  translations files (i18next)

package.json

@@ -114,6 +114,7 @@
     "@commitlint/config-conventional": "^19.2.2",
     "@dev-plugins/react-query": "^0.0.7",
     "@expo/config": "~9.0.2",
+    "@tanstack/eslint-plugin-query": "^5.62.1",
     "@testing-library/jest-dom": "^6.5.0",
     "@testing-library/react-native": "^12.7.2",
     "@types/i18n-js": "^3.8.9",
@@ -131,6 +132,7 @@
     "eslint-plugin-i18n-json": "^4.0.0",
     "eslint-plugin-jest": "^28.8.3",
     "eslint-plugin-prettier": "^5.2.1",
+    "eslint-plugin-react": "^7.37.2",
     "eslint-plugin-simple-import-sort": "^10.0.0",
     "eslint-plugin-sonarjs": "^1.0.4",
     "eslint-plugin-tailwindcss": "^3.15.2",