fix: remove sharding from CI workflow, run tests in single job

Author: SFARPak

.github/workflows/ci.yml

@@ -17,19 +17,7 @@ defaults:
 
 jobs:
   test:
-    # Why Mac and Windows?
-    # I can't run electron playwright on ubuntu-latest and
-    # Linux support for Dyad is experimental so not as important
-    # as Mac + Windows
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [
-            { name: "macos", image: "macos-latest" },
-          ]
-        shard: [1, 2, 3, 4]
-        shardTotal: [4]
-    runs-on: ${{ matrix.os.image }}
+    runs-on: macos-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
@@ -87,60 +75,15 @@ jobs:
         run: npm run pre:e2e
       - name: Prep test server
         run: cd testing/fake-llm-server && npm install && npm run build && cd -
-      - name: E2E tests (Shard ${{ matrix.shard }}/4)
+      - name: E2E tests
         # You can add debug logging to make it easier to see what's failing
         # by adding "DEBUG=pw:browser" in front.
-        # Use blob reporter for sharding and merge capabilities
-        run: DEBUG=pw:browser npx playwright test --shard=${{ matrix.shard }}/${{ matrix.shardTotal }}
-      - name: Upload shard results
+        run: DEBUG=pw:browser npx playwright test
+      - name: Upload test results
         uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
         if: ${{ !cancelled() }}
         with:
-          name: blob-report-${{ matrix.os.name }}-shard-${{ matrix.shard }}
-          path: blob-report
+          name: test-results
+          path: test-results
           retention-days: 1
 
-  merge-reports:
-    # Merge reports after playwright-tests, even if some shards have failed
-    if: ${{ !cancelled() }}
-    needs: [test]
-
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: lts/*
-      - name: Install dependencies
-        run: npm install --no-audit --no-fund --progress=false
-
-      - name: Download blob reports from GitHub Actions Artifacts
-        uses: actions/download-artifact@v4
-        with:
-          path: all-blob-reports
-          pattern: blob-report-*
-          merge-multiple: true
-
-      - name: Debug - List downloaded blob reports
-        run: |
-          echo "Contents of all-blob-reports directory:"
-          ls -la all-blob-reports/
-          echo "File sizes and details:"
-          find all-blob-reports/ -type f -exec ls -lh {} \; || echo "No files found"
-
-      - name: Merge into HTML Report
-        run: PLAYWRIGHT_HTML_OUTPUT_DIR=playwright-report npx playwright merge-reports --config=merge.config.ts ./all-blob-reports
-
-      - name: Debug - List playwright-report contents
-        run: |
-          echo "Contents of playwright-report directory:"
-          ls -la playwright-report/ || echo "playwright-report directory does not exist"
-          echo "Current directory contents:"
-          ls -la
-
-      - name: Upload HTML report
-        uses: actions/upload-artifact@v4
-        with:
-          name: html-report--attempt-${{ github.run_attempt }}
-          path: playwright-report
-          retention-days: 3