Update workflows (#431)

* Update build.yml

* Update package.json

* Update build.yml

* Create validate.yml

* Update build.yml

* Update validate.yml

* Update deploy.yml

* Add eslint-plugin-react

* style: format code with Prettier

This commit fixes the style issues introduced in a85398e according to the output
from Prettier.

Details: #431

* Fix settings

* style: format code with Prettier

This commit fixes the style issues introduced in 47fbba2 according to the output
from Prettier.

Details: #431

* Fix issues

* Restore config

* style: format code with Prettier

This commit fixes the style issues introduced in 3dbfd51 according to the output
from Prettier.

Details: #431

* Add baseline-browser-mapping

* Fix ESlint

* style: format code with Prettier

This commit fixes the style issues introduced in 0ab08f9 according to the output
from Prettier.

Details: #431

* Fix eslint problems

* style: format code with Prettier

This commit fixes the style issues introduced in be25b73 according to the output
from Prettier.

Details: #431

* Fix ESLint errors and warnings

* style: format code with Prettier

This commit fixes the style issues introduced in ccfa622 according to the output
from Prettier.

Details: #431

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: gstraccini[bot] <150967461+gstraccini[bot]@users.noreply.github.com>

Author: 3 people

.github/workflows/build.yml

@@ -9,29 +9,5 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  build:
-    name: Build and Test
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v6
-        with:
-          fetch-depth: 0
-
-      - name: Set up Node.js
-        uses: actions/setup-node@v6
-        with:
-          node-version: 22
-          cache: "npm"
-      
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Build
-        run: npm run build
-
-      # - name: Test
-      #   run: npm test
-        
-           
+  validate:
+    uses: ./.github/workflows/validate.yml

.github/workflows/deploy.yml

@@ -6,7 +6,7 @@ on:
   workflow_dispatch:
 
 concurrency:
-  group: 'pages'
+  group: pages
   cancel-in-progress: true
 
 permissions:
@@ -15,8 +15,12 @@ permissions:
   id-token: write
 
 jobs:
+  validate:
+    uses: ./.github/workflows/validate.yml
+
   build:
     name: Build
+    needs: validate
     runs-on: ubuntu-latest
     outputs:
       fullSemVer: ${{ steps.gitversion.outputs.fullSemVer }}
@@ -30,42 +34,32 @@ jobs:
       - name: Install GitVersion
         uses: gittools/actions/gitversion/setup@v4.5.0
         with:
-          versionSpec: "6.x"
+          versionSpec: 6.x
 
       - name: Determine Version
         id: gitversion
         uses: gittools/actions/gitversion/execute@v4.5.0
         with:
           useConfigFile: true
 
-      - name: Set up Node.js
+      - name: Setup Node.js
         uses: actions/setup-node@v6
         with:
           node-version: 22
-          cache: "npm"
+          cache: npm
 
       - name: Install dependencies
         run: npm ci
 
       - name: Build
-        run: npm run build
+        run: CI=true npm run build
 
       - name: Upload build artifacts
         uses: actions/upload-artifact@v7
         with:
           name: build-output
           path: dist/
 
-      # - name: Test
-      #   run: npm test
-
-      # - name: Upload test results
-      #   if: always()
-      #   uses: actions/upload-artifact@v7
-      #   with:
-      #     name: test-results
-      #     path: test-results.json
-
   deploy:
     name: Deploy GH Pages
     needs: build
@@ -77,18 +71,18 @@ jobs:
         with:
           name: build-output
           path: ./dist
-      
+
       - name: Setup Pages
         uses: actions/configure-pages@v6
-        
+
       - name: Upload artifact
         uses: actions/upload-pages-artifact@v4
         with:
-          path: './dist'
-          
+          path: ./dist
+
       - name: Deploy to GitHub Pages
         uses: actions/deploy-pages@v5
-  
+
   create_release:
     name: Create release
     needs: [build, deploy]

.github/workflows/validate.yml

@@ -0,0 +1,44 @@
+name: Validate project
+
+on:
+  workflow_call:
+
+jobs:
+  validate:
+    name: Validate
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v6
+        with:
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: 22
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Type check
+        run: npm run typecheck
+
+      - name: Lint
+        run: npm run lint
+
+      - name: Build
+        run: CI=true npm run build
+
+      # optional
+      # - name: Test
+      #   run: npm test
+
+      # - name: Upload test results
+      #   if: always()
+      #   uses: actions/upload-artifact@v7
+      #   with:
+      #     name: test-results
+      #     path: test-results.json

eslint.config.js

@@ -1,28 +1,81 @@
 import js from "@eslint/js";
 import globals from "globals";
+import react from "eslint-plugin-react";
 import reactHooks from "eslint-plugin-react-hooks";
 import reactRefresh from "eslint-plugin-react-refresh";
-import tseslint from "typescript-eslint";
+import tsPlugin from "@typescript-eslint/eslint-plugin";
+import tsParser from "@typescript-eslint/parser";
 
-export default tseslint.config(
-  { ignores: ["dist"] },
+export default [
   {
-    extends: [js.configs.recommended, ...tseslint.configs.recommended],
     files: ["**/*.{ts,tsx}"],
+    ignores: ["dist"],
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        ecmaVersion: 2020,
+        sourceType: "module",
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+      globals: globals.browser,
+    },
+    plugins: {
+      react,
+      "react-hooks": reactHooks,
+      "react-refresh": reactRefresh,
+      "@typescript-eslint": tsPlugin,
+    },
+    rules: {
+      ...js.configs.recommended.rules,
+      ...tsPlugin.configs.recommended.rules,
+      ...reactHooks.configs.recommended.rules,
+
+      // Catch missing JSX imports
+      "react/jsx-no-undef": "error",
+
+      // React Refresh
+      "react-refresh/only-export-components": [
+        "warn",
+        { allowConstantExport: true },
+      ],
+
+      // Optional TypeScript stricter rules
+      "@typescript-eslint/explicit-function-return-type": "warn",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        { argsIgnorePattern: "^_" },
+      ],
+    },
+    settings: {
+      react: {
+        version: "detect",
+      },
+    },
+  },
+  {
+    files: ["**/*.{js,jsx}"],
     languageOptions: {
       ecmaVersion: 2020,
+      sourceType: "module",
       globals: globals.browser,
+      parserOptions: { ecmaFeatures: { jsx: true } },
     },
     plugins: {
+      react,
       "react-hooks": reactHooks,
       "react-refresh": reactRefresh,
     },
     rules: {
+      ...js.configs.recommended.rules,
       ...reactHooks.configs.recommended.rules,
+      "react/jsx-no-undef": "error",
       "react-refresh/only-export-components": [
         "warn",
         { allowConstantExport: true },
       ],
     },
+    settings: { react: { version: "detect" } },
   },
-);
+];