Merge pull request #397 from ChildMindInstitute/dev

Merge changes from the dev branch into the stage branch

.env.example

@@ -0,0 +1,8 @@
+NODE_ENV=development
+VITE_ENV=dev
+VITE_API_HOST=http://localhost:8000/
+VITE_ADMIN_PANEL_HOST=http://localhost:3000/
+VITE_SECURE_LOCAL_STORAGE_HASH_KEY=ML_SECURE
+VITE_SECURE_LOCAL_STORAGE_PREFIX=ML_SECURE
+VITE_IV_LENGTH=16
+VITE_BUILD_VERSION=dev-build

.eslintignore

@@ -1,7 +1,11 @@
-node_modules/
-dist/
-.prettierrc.js
-.eslintrc.js
-env.d.ts
-public
-.gitignore
+# Ignore everything
+/*
+
+# Except for the src directory
+!src
+
+# And any top-level JS or TS files
+!/*.js
+!/*.jsx
+!/*.ts
+!/*.tsx

.eslintrc.js

@@ -6,7 +6,14 @@ module.exports = {
     jest: true,
   },
   parser: '@typescript-eslint/parser',
-  plugins: ['react', 'react-hooks', '@typescript-eslint', 'prettier', 'eslint-plugin-import'],
+  plugins: [
+    'react',
+    'react-hooks',
+    '@typescript-eslint',
+    'prettier',
+    'eslint-plugin-import',
+    'unused-imports',
+  ],
   overrides: [
     {
       files: ['*.ts', '*.tsx'],
@@ -24,41 +31,16 @@ module.exports = {
         'plugin:@typescript-eslint/eslint-recommended',
         'plugin:@typescript-eslint/recommended',
         'plugin:@typescript-eslint/recommended-requiring-type-checking',
+        'plugin:prettier/recommended',
       ],
       rules: {
-        'linebreak-style': ['error', 'unix'],
-        'no-empty-function': 'off',
-        '@typescript-eslint/no-empty-function': 'off',
-        '@typescript-eslint/no-explicit-any': 'off',
-        '@typescript-eslint/no-unsafe-assignment': 'off',
-        '@typescript-eslint/no-unsafe-call': 'off',
-        '@typescript-eslint/no-floating-promises': 'off',
+        // Error
         '@typescript-eslint/no-misused-promises': ['error', { checksVoidReturn: false }],
-        'prettier/prettier': 'error',
-        'react/display-name': 'off',
-        'react/prop-types': 'off',
         'react-hooks/rules-of-hooks': 'error',
-        'react-hooks/exhaustive-deps': 'warn',
-        'react/jsx-uses-react': 'off',
-        'react/react-in-jsx-scope': 'off',
-        'no-console': "warn",
-        'no-case-declarations': 'off',
-        '@typescript-eslint/unbound-method': 'off',
-        '@typescript-eslint/no-unsafe-member-access': 'off',
-        '@typescript-eslint/no-unsafe-argument': 'off',
-        '@typescript-eslint/restrict-template-expressions': 'off',
-        '@typescript-eslint/no-unsafe-return': 'off',
-        '@typescript-eslint/restrict-plus-operands': 'off',
-        '@typescript-eslint/no-non-null-assertion': 'off',
         'import/order': [
           'error',
           {
-            groups: [
-              ['external', 'builtin'],
-              'internal',
-              ['sibling', 'parent'],
-              'index',
-            ],
+            groups: [['external', 'builtin'], 'internal', ['sibling', 'parent'], 'index'],
             pathGroups: [
               {
                 pattern: '@(react)',
@@ -77,7 +59,73 @@ module.exports = {
               caseInsensitive: true,
             },
           },
-        ]
+        ],
+        'unused-imports/no-unused-imports': 'error',
+        'import/no-cycle': 'error',
+        'constructor-super': 'error',
+        'no-this-before-super': 'error',
+        'no-useless-computed-key': 'error',
+        'no-useless-constructor': 'off',
+        'no-useless-rename': 'error',
+        'no-var': 'error',
+        'object-shorthand': 'error',
+        'prefer-template': 'error',
+        'no-restricted-syntax': ['error', 'ForInStatement', 'SequenceExpression'],
+        'no-caller': 'error',
+        'no-template-curly-in-string': 'error',
+        'array-callback-return': 'error',
+        'no-eval': 'error',
+        'no-extend-native': 'error',
+        eqeqeq: ['error', 'always'],
+        'no-lone-blocks': 'error',
+        'no-proto': 'error',
+        'no-script-url': 'error',
+        'no-self-compare': 'error',
+        'no-sequences': 'error',
+        'no-throw-literal': 'error',
+        'no-undef-init': 'error',
+        'no-nested-ternary': 'error',
+        'no-unneeded-ternary': 'error',
+        'no-debugger': 'error',
+        'no-empty': 'error',
+        'no-unused-labels': 'error',
+        'prefer-const': 'error',
+
+        // Warn
+        '@typescript-eslint/no-empty-function': 'warn',
+        '@typescript-eslint/no-explicit-any': 'warn',
+        '@typescript-eslint/no-unsafe-assignment': 'warn',
+        '@typescript-eslint/no-unsafe-call': 'warn',
+        '@typescript-eslint/no-floating-promises': 'warn',
+        'react/display-name': 'warn',
+        'react/prop-types': 'warn',
+        'react-hooks/exhaustive-deps': 'warn',
+        'no-console': 'warn',
+        '@typescript-eslint/no-non-null-assertion': 'warn',
+        'unused-imports/no-unused-vars': [
+          'warn',
+          {
+            vars: 'all',
+            varsIgnorePattern: '^_',
+            args: 'after-used',
+            argsIgnorePattern: '^_',
+          },
+        ],
+
+        // Off
+        'react/jsx-uses-react': 'off',
+        'react/react-in-jsx-scope': 'off',
+        'no-case-declarations': 'off',
+        '@typescript-eslint/unbound-method': 'off',
+        '@typescript-eslint/no-unsafe-member-access': 'off',
+        '@typescript-eslint/no-unsafe-argument': 'off',
+        '@typescript-eslint/restrict-template-expressions': 'off',
+        '@typescript-eslint/no-unsafe-return': 'off',
+        '@typescript-eslint/restrict-plus-operands': 'off',
+        '@typescript-eslint/no-unused-vars': 'off',
+        'no-extra-semi': 'off',
+        'prefer-destructuring': 'off',
+        camelcase: 'off',
       },
     },
   ],
@@ -86,4 +134,4 @@ module.exports = {
       version: 'detect',
     },
   },
-};
+};

.github/workflows/ci.yml

@@ -0,0 +1,54 @@
+name: CI
+on: [pull_request]
+
+env:
+  NODE_VERSION: 20.11.0
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ env.NODE_VERSION }}
+
+      - uses: actions/cache@v4
+        with:
+          path: |
+            ~/node_modules
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/yarn.lock') }}
+
+      - run: yarn install --frozen-lockfile
+
+      - uses: wearerequired/lint-action@v2
+        with:
+          eslint: true
+          eslint_extensions: mjs,js,jsx,mts,ts,tsx
+          prettier: true
+          prettier_extensions: mjs,js,jsx,mts,ts,tsx
+          continue_on_error: false
+  tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ env.NODE_VERSION }}
+
+      - name: Install deps
+        run: yarn install --frozen-lockfile
+
+      - uses: actions/cache@v4
+        with:
+          path: |
+            ~/node_modules
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/yarn.lock') }}
+
+      - name: Running tests
+        env:
+          REACT_APP_API_DOMAIN: http://localhost:8080
+        run: yarn test

.gitignore

@@ -26,4 +26,4 @@ coverage
 *.sw?
 .env
 .env*
-
+!.env.example

.prettierignore

@@ -1,3 +1,13 @@
-/node_modules
-/public
-/src/i18n/
+# Ignore everything
+/*
+
+# Except for the src directory
+!src
+
+# And any top-level JS or TS files
+!/*.js
+!/*.mjs
+!/*.jsx
+!/*.ts
+!/*.mts
+!/*.tsx

.prettierrc.js

@@ -1,11 +1,13 @@
 module.exports = {
-  arrowParens: 'avoid',
-  bracketSameLine: true,
+  arrowParens: 'always',
+  bracketSameLine: false,
   bracketSpacing: true,
-  singleQuote: false,
+  singleQuote: true,
+  jsxSingleQuote: false,
   trailingComma: 'all',
-  printWidth: 120,
+  semi: true,
+  printWidth: 100,
   tabWidth: 2,
   useTabs: false,
-  semi: false
-}
+  endOfLine: 'lf',
+};

Jenkinsfile.groovy

@@ -1 +1 @@
-build( job: env.MindloggerWebRefactorNewBuilder )
+build( job: env.MindloggerWebRefactorBuilder )