Release/minor (#840)

## User-Facing Changes

<!-- will be used as a changelog entry -->

## Description

<!-- link relevant GitHub issues -->
<!-- add `docs` label if this PR requires documentation updates -->
<!-- add relevant metric tracking for experimental / new features -->

## Checklist

- [ ] The web version was tested and it is running ok
- [ ] The desktop version was tested and it is running ok
- [ ] This change is covered by unit tests
- [ ] Files constants.ts, types.ts and *.style.ts have been checked and
relevant code snippets have been relocated

---------

Signed-off-by: Bezerra Luiz, (Luiz.Bezerra@ctw.bmwgroup.com) <luiz.bezerra+QCOM@ctw.bmwgroup.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: ctw-joao-luis <Joao.Luis@ctw.bmwgroup.com>
Signed-off-by: Thomas Sedlmayer <tsedlmayer@pmsfit.de>
Signed-off-by: Taeseung Sohn <paulsohnjp@gmail.com>
Signed-off-by: Gabriela Almeida <gabriela.almeida@criticaltechworks.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Gabriela Almeida <gabriela.almeida@criticaltechworks.com>
Co-authored-by: Alexandre Neuwald CTW <Alexandre.Neuwald@ctw.bmwgroup.com>
Co-authored-by: Luiz Bezerra <luiz.bezerra@ctw.bmwgroup.com>
Co-authored-by: Bezerra Luiz, (Luiz.Bezerra@ctw.bmwgroup.com) <luiz.bezerra+QCOM@ctw.bmwgroup.com>
Co-authored-by: Alexandre Neuwald <alexandre.neuwald@criticaltechworks.com>
Co-authored-by: Bruno Abreu <bmachadorj@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Rodrigo CTW <rodrigo.PE.rodrigues@ctw.bmwgroup.com>
Co-authored-by: Bezerra Luiz, (Luiz.Bezerra@ctw.bmwgroup.com) <198787532+luiz-bezerra-ctw-bmwgroup-com_QCOM@users.noreply.github.com>
Co-authored-by: Thomas Sedlmayer <tsedlmayer@pmsfit.de>
Co-authored-by: Taeseung Sohn <paulsohnjp@gmail.com>
Co-authored-by: Planck-Ho <wenloong.he@gmail.com>

Author: 13 people

.eslintrc.ci.yaml

@@ -14,3 +14,12 @@ rules:
   import/no-cycle:
     - error
     - ignoreExternal: true
+
+overrides:
+  - files:
+      - "e2e/ci-helpers/**/*.ts"
+    rules:
+      no-restricted-syntax: off
+      import/order: off
+      "@typescript-eslint/no-unnecessary-condition": off
+      "@typescript-eslint/strict-boolean-expressions": off

.eslintrc.yaml

@@ -30,6 +30,7 @@ settings:
 rules:
   "@lichtblick/license-header": ["error", { licenseType: "MPL-2.0" }]
   "@lichtblick/prefer-hash-private": off
+  '@typescript-eslint/no-unnecessary-type-conversion': off
 
   tss-unused-classes/unused-classes: error
 

.github/workflows/ci.yml

@@ -49,15 +49,15 @@ jobs:
     name: ${{ matrix.config.name }} (${{ matrix.os }})
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: actions/setup-node@v6
         with:
           node-version: 20
       - run: corepack enable yarn
 
       - name: Restore cache
-        uses: actions/cache@v4
+        uses: actions/cache@v5
         with:
           path: |
             .yarn/cache
@@ -69,10 +69,18 @@ jobs:
 
       - run: ${{ matrix.config.command }}
 
-  e2e:
+  e2e-desktop:
     runs-on: ubuntu-latest
 
-    name: e2e (ubuntu-latest)
+    # Run tests in parallel across multiple shards
+    # This distributes tests across multiple GitHub Action jobs
+    # Each shard runs independently, reducing total CI time
+    strategy:
+      fail-fast: false
+      matrix:
+        shard: [1, 2, 3, 4]
+
+    name: e2e-desktop (ubuntu-latest) [${{ matrix.shard }}/4]
 
     # Set a fixed timezone to ensure consistent test results.
     # Lichtblick uses the system's local timezone, which can cause test assertions
@@ -81,15 +89,15 @@ jobs:
       TZ: Europe/Lisbon
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: actions/setup-node@v6
         with:
           node-version: 20
       - run: corepack enable yarn
 
       - name: Restore cache
-        uses: actions/cache@v4
+        uses: actions/cache@v5
         with:
           path: |
             .yarn/cache
@@ -106,10 +114,133 @@ jobs:
 
       # It must be dev build, otherwise playwright wont't have all data-testid avaiable for icons
       # https://mui.com/material-ui/migration/upgrade-to-v7/#svgicons-data-testid-removed
-      - name: test:e2e:desktop:ci (ubuntu-latest)
+      - name: Build desktop app
+        run: yarn desktop:build:dev
+
+      - name: Run E2E Desktop tests - Shard ${{ matrix.shard }}/4
         uses: coactions/setup-xvfb@v1
         with:
           working-directory: ./
-          run: |
-            yarn desktop:build:dev
-            yarn test:e2e:desktop:ci
+          run: yarn test:e2e:desktop:ci --shard=${{ matrix.shard }}/4
+
+      - name: Upload Desktop E2E Test Reports (Shard ${{ matrix.shard }})
+        if: always()
+        uses: actions/upload-artifact@v6
+        with:
+          name: blob-report-desktop-${{ matrix.shard }}
+          path: blob-report
+          retention-days: 1
+
+  e2e-web:
+    runs-on: ubuntu-latest
+
+    name: e2e-web (ubuntu-latest)
+
+    env:
+      TZ: Europe/Lisbon
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: 20
+      - run: corepack enable yarn
+
+      - name: Restore cache
+        uses: actions/cache@v5
+        with:
+          path: |
+            .yarn/cache
+            **/node_modules
+          key: v5-${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v5-${{ runner.os }}-yarn-
+
+      - run: yarn install --immutable
+
+      - name: Install Playwright Browsers
+        run: npx playwright install --with-deps chromium
+
+      - name: Run E2E Web tests
+        run: yarn test:e2e:web:ci
+
+      - name: Upload Web E2E Test Reports
+        if: always()
+        uses: actions/upload-artifact@v6
+        with:
+          name: e2e-web-reports
+          path: e2e/reports/web/
+          retention-days: 7
+          if-no-files-found: warn
+
+  # Merge reports from all shards and generate summary
+  e2e-report:
+    if: always()
+    needs: [e2e-desktop, e2e-web]
+    runs-on: ubuntu-latest
+
+    name: e2e-report (merge all reports)
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: 20
+      - run: corepack enable yarn
+
+      - name: Restore cache
+        uses: actions/cache@v5
+        with:
+          path: |
+            .yarn/cache
+            **/node_modules
+          key: v5-${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v5-${{ runner.os }}-yarn-
+
+      - run: yarn install --immutable
+
+      - name: Download all desktop blob reports
+        uses: actions/download-artifact@v7
+        with:
+          pattern: blob-report-desktop-*
+          path: all-blob-reports-desktop
+          merge-multiple: true
+
+      - name: Download web reports
+        uses: actions/download-artifact@v7
+        with:
+          name: e2e-web-reports
+          path: e2e/reports/web
+        continue-on-error: true
+
+      - name: Merge Desktop Playwright reports
+        run: |
+          mkdir -p e2e/reports/desktop
+
+          echo "Downloaded desktop blob reports:"
+          ls -la all-blob-reports-desktop/ || echo "No desktop blob reports found"
+
+          # Merge blob reports into JSON format
+          npx playwright merge-reports --reporter json ./all-blob-reports-desktop > e2e/reports/desktop/results.json
+
+          # Also generate HTML report
+          npx playwright merge-reports --reporter html ./all-blob-reports-desktop
+
+          echo "Merged desktop report size:"
+          ls -lh e2e/reports/desktop/results.json
+
+      - name: Generate E2E Test Summary
+        run: |
+          echo "Generating E2E test summary from merged reports..."
+          yarn ts-node e2e/ci-helpers/generate-test-summary.ts >> $GITHUB_STEP_SUMMARY
+
+      - name: Upload Merged E2E Test Reports
+        if: always()
+        uses: actions/upload-artifact@v6
+        with:
+          name: e2e-test-reports-merged
+          path: |
+            e2e/reports/
+            playwright-report/
+          retention-days: 7

.github/workflows/cloudflare-pages.yml

@@ -17,7 +17,7 @@ jobs:
       ENABLE_EXPERIMENTAL_COREPACK: 1
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
         with:
           submodules: true
 

.github/workflows/dependabot-fix.yml

@@ -25,7 +25,7 @@ jobs:
       YARN_ENABLE_SCRIPTS: false
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
         with:
           # Using a Personal Access Token here is required to trigger workflows on our new commit.
           # The default GitHub token doesn't trigger any workflows.

.github/workflows/enforce-branch-naming.yml

@@ -20,7 +20,7 @@ jobs:
               exit 1
             fi
           elif [[ "${GITHUB_BASE_REF}" == "develop" ]]; then
-            if [[ ! "${GITHUB_HEAD_REF}" =~ ^(feature/|bugfix/|dependabot/) ]]; then
+            if [[ ! "${GITHUB_HEAD_REF}" =~ ^(feature/|bugfix/|dependabot/|sync/) ]]; then
               echo "Source branch must start with 'feature/' or 'bugfix/' (or 'dependabot/') when targeting 'develop'."
               exit 1
             fi

.github/workflows/gh-pages.yml

@@ -23,7 +23,7 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
         with:
           ref: main
 

.github/workflows/post-release.yml

@@ -12,7 +12,7 @@ jobs:
 
     steps:
       - name: Checkout ${{ github.event.release.tag_name }}
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           ref: ${{ github.event.release.tag_name }}
 
@@ -39,7 +39,7 @@ jobs:
 
     steps:
       - name: Checkout ${{ github.event.release.tag_name }}
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           ref: ${{ github.event.release.tag_name }}
 

.github/workflows/pre-build.yml

@@ -13,7 +13,7 @@ jobs:
 
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           token: ${{ secrets.LICHTBLICK_GITHUB_TOKEN }}
 
@@ -45,21 +45,21 @@ jobs:
         run: yarn run package:darwin
 
       - name: Upload Windows artifact
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v6
         with:
           name: lichtblick-${{ env.version }}-windows
           path: dist/lichtblick-${{ env.version }}-win.exe
           retention-days: 30
 
       - name: Upload Linux artifact
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v6
         with:
           name: lichtblick-${{ env.version }}-debian-amd64
           path: dist/lichtblick-${{ env.version }}-linux-amd64.deb
           retention-days: 30
 
       - name: Upload MacOS artifact
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v6
         with:
           name: lichtblick-${{ env.version }}-macos
           path: dist/lichtblick-${{ env.version }}-mac-universal.dmg

.github/workflows/prerelease.yml

@@ -40,7 +40,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           token: ${{ secrets.LICHTBLICK_GITHUB_TOKEN }}
           ref: ${{ github.event.inputs.branch }}