Merge pull request #115 from BinaryStudioAcademy/development

development to production

Author: v0ldemar01

.eslintrc.yml

@@ -4,14 +4,24 @@ parserOptions:
   ecmaVersion: 2022
   sourceType: module
   project:
-    - ./client/jsconfig.json
+    - ./tsconfig.json
+    - ./shared/tsconfig.json
+    - ./client/tsconfig.json
+    - ./server/tsconfig.json
 
 settings:
   node: true
 
 extends:
   - eslint:recommended
   - plugin:import/recommended
+  - plugin:import/typescript
+  - plugin:@typescript-eslint/recommended
+  - plugin:@typescript-eslint/recommended-type-checked
+  - plugin:@typescript-eslint/strict
+  - plugin:@typescript-eslint/strict-type-checked
+  - plugin:@typescript-eslint/stylistic
+  - plugin:@typescript-eslint/stylistic-type-checked
   - plugin:unicorn/recommended
   - plugin:sonarjs/recommended
 
@@ -56,9 +66,7 @@ rules:
   import/no-duplicates:
     - error
   no-unused-vars:
-    - error
-    - argsIgnorePattern: ^_
-      ignoreRestSiblings: true
+    - off
   sonarjs/cognitive-complexity:
     - error
     - 18
@@ -68,19 +76,74 @@ rules:
   object-curly-spacing:
     - error
     - always
-  semi:
+  '@typescript-eslint/no-unused-vars':
+    - error
+    - argsIgnorePattern: ^_
+      ignoreRestSiblings: true
+  '@typescript-eslint/lines-between-class-members':
+    - error
+  '@typescript-eslint/padding-line-between-statements':
+    - error
+    - blankLine: never
+      prev: export
+      next: export
+    - blankLine: always
+      prev:
+        - const
+        - class
+      next: export
+    - blankLine: always
+      prev: '*'
+      next:
+        - switch
+        - class
+        - function
+        - if
+        - return
+        - try
+        - interface
+        - type
+  '@typescript-eslint/consistent-type-definitions':
+    - off
+  '@typescript-eslint/non-nullable-type-assertion-style':
+    - off
+  '@typescript-eslint/no-unnecessary-condition':
+    - off
+  '@typescript-eslint/return-await':
+    - error
+    - always
+  '@typescript-eslint/quotes':
+    - error
+    - single
+  '@typescript-eslint/consistent-type-imports':
+    - error
+  '@typescript-eslint/consistent-type-exports':
+    - error
+  '@typescript-eslint/no-unnecessary-type-assertion':
+    - off
+  '@typescript-eslint/explicit-function-return-type':
+    - error
+    - allowTypedFunctionExpressions: true
+  '@typescript-eslint/no-empty-interface':
+    - error
+    - allowSingleExtends: true
+  '@typescript-eslint/explicit-member-accessibility':
+    - error
+  '@typescript-eslint/no-misused-promises':
+    - error
+    - checksVoidReturn: false
+  '@typescript-eslint/object-curly-spacing':
+    - error
+    - always
+  '@typescript-eslint/semi':
     - error
     - always
 
 overrides:
   - files:
-      - project.config.cjs
-      - commitlint.config.cjs
-      - dangerfile.cjs
-    parserOptions:
-      sourceType: commonjs
-    env:
-      node: true
+      - project.config.ts
+      - commitlint.config.ts
+      - dangerfile.ts
     rules:
       'import/no-default-export':
         - off

.github/workflows/ci-pr.yml

@@ -44,7 +44,7 @@ jobs:
 
       - name: Lint Commits
         run: |
-          npx commitlint --from ${{ github.event.pull_request.base.sha }} --to ${{ github.event.pull_request.head.sha }} --verbose -g ./commitlint.config.cjs
+          npx commitlint --from ${{ github.event.pull_request.base.sha }} --to ${{ github.event.pull_request.head.sha }} --verbose -g ./commitlint.config.ts
 
       - name: Add Folder Labels
         uses: actions/labeler@v4
@@ -62,4 +62,4 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
         run: |
-          npx danger ci --failOnErrors --text-only -d ./dangerfile.cjs
+          npx danger ci --failOnErrors --text-only

.lintstagedrc.yml

@@ -1,6 +1,6 @@
 '*': npm run lint:editorconfig && npm run lint:fs
-'*.{js,jsx,scss,json,md,html}': npm run prettify
-shared/**/*.js: npm run lint:shared:js
-client/src/**/*.{js,jsx}: npm run lint:client:js
+'*.{ts,tsx,scss,json,md,html}': npm run prettify
+shared/**/*.ts: npm run lint:shared:js
+client/src/**/*.{ts,tsx}: npm run lint:client:js
 client/src/**/*.scss: npm run lint:client:css
-server/**/*.js: npm run lint:server:js
+server/**/*.ts: npm run lint:server:js

.nvmrc

@@ -1 +1 @@
-18
+18.18

.prettierignore

@@ -0,0 +1,2 @@
+build
+dist

client/.env.example

@@ -6,5 +6,5 @@ VITE_API_SERVER=http://localhost:3001
 VITE_SOCKET_SERVER=ws://localhost:3001
 VITE_API_PATH=/api
 VITE_SOCKET_PATH=/socket.io
-VITE_PORT=3000
-VITE_HOST=localhost
+VITE_APP_PORT=3000
+VITE_APP_HOST=localhost

client/.eslintrc.yml

@@ -7,14 +7,15 @@ settings:
 
 parserOptions:
   project:
-    - ./jsconfig.json
+    - ./tsconfig.json
 
 extends:
   - ../.eslintrc.yml
   - plugin:react/recommended
   - plugin:react/jsx-runtime
   - plugin:react-hooks/recommended
   - plugin:jsx-a11y/recommended
+  - plugin:import/typescript
 
 rules:
   react/jsx-no-bind:
@@ -29,10 +30,12 @@ rules:
     - error
   import/no-unresolved:
     - off
+  react/prop-types:
+    - off
 
 overrides:
   - files:
-      - vite.config.js
+      - vite.config.ts
     rules:
       import/no-default-export:
         - off

client/.stylelintrc.yml

@@ -7,7 +7,6 @@ plugins:
 
 rules:
   selector-class-pattern: null
-  declaration-colon-newline-after: null
   selector-pseudo-class-no-unknown:
     - true
     - ignorePseudoClasses:

client/index.html

@@ -1,4 +1,4 @@
-<!DOCTYPE html>
+<!doctype html>
 <html lang="en">
   <head>
     <meta charset="utf-8" />
@@ -27,7 +27,7 @@
   <body>
     <noscript>You need to enable JavaScript to run this app.</noscript>
     <div id="root"></div>
-    <script type="module" src="./src/index.jsx"></script>
+    <script type="module" src="./src/index.tsx"></script>
     <!--
       This HTML file is a template.
       If you open it directly in the browser, you will see an empty page.