Try to fix the workflow....

Author: leeyi45

.github/workflows/pull-request.yml

@@ -25,17 +25,20 @@ jobs:
             - 'devserver/**'
             libraries:
             - 'lib/**'
-            modules:
-            - src/{bundles,tabs}/**
+            tabs:
+            - src/tabs/**
+            bundles:
+            - src/bundles/**
             docs:
             - docs/**
     outputs:
       # if the workflow file was modified, then we rerun the entire job
       devserver: ${{ steps.filter.outputs.devserver || steps.filter.outputs.workflows }}
       # if the devserver needs to be tested bundles and tabs need to be rebuilt
-      modules: ${{ steps.filter.outputs.modules || steps.filter.outputs.workflows || steps.filter.outputs.devserver }}
+      modules: ${{ steps.filter.outputs.tabs || steps.filter.outputs.bundles || steps.filter.outputs.workflows || steps.filter.outputs.devserver }}
       libraries: ${{ steps.filter.outputs.libraries || steps.filter.outputs.workflows }}
       docs: ${{ steps.filter.outputs.docs || steps.filter.outputs.workflows }}
+      playwright: $${{ steps.filter.outputs.devserver || steps.filter.outputs.libraries || steps.filter.outputs.bundles }}
 
   test:
     name: Verify all tests pass and build success
@@ -61,6 +64,10 @@ jobs:
         - name: Install dependencies
           run: yarn install --immutable
 
+        - name: Install Playwright
+          if: needs.paths-filter.outputs.playwright == 'true'
+          run: yarn playwright install --with-deps
+
         - name: Run tsc for libraries
           if: needs.paths-filter.outputs.libraries == 'true'
           run: yarn workspaces foreach -ptW --from "./lib/*" run tsc
@@ -75,7 +82,7 @@ jobs:
 
         - name: Test bundles and tabs
           if: needs.paths-filter.outputs.modules == 'true'
-          run: playwright install --with-deps && yarn test:modules
+          run: yarn test:modules
 
         - name: Build manifest
           if: needs.paths-filter.outputs.devserver == 'true'

devserver/vite.config.ts

@@ -5,7 +5,7 @@ import fs from 'fs/promises';
 import pathlib from 'path';
 import react from '@vitejs/plugin-react';
 import { nodePolyfills } from 'vite-plugin-node-polyfills';
-import { loadEnv, defineConfig, type UserConfig, type ResolverFunction } from 'vite'
+import { loadEnv, defineConfig, type UserConfig } from 'vite'
 import { mergeConfig } from 'vitest/config';
 import type { BrowserCommand } from 'vitest/node';
 import rootConfig from '../vitest.config';
@@ -85,8 +85,7 @@ export default defineConfig(({ mode }) => {
           headless: true,
           provider: 'playwright',
           instances: [{
-            browser: 'chromium',
-            screenshotFailures: false
+            browser: 'chromium'
           }],
           commands: {
             setLocalStorage

lib/modules-lib/package.json

@@ -15,7 +15,8 @@
     "typedoc-plugin-rename-defaults": "^0.7.3",
     "typescript": "^5.8.2",
     "vitest": "^3.2.3",
-    "vitest-browser-react": "^1.0.0"
+    "vitest-browser-react": "^1.0.0",
+    "@vitejs/plugin-react": "^4.5.1"
   },
   "exports": {
     "./tabs/index": null,

lib/modules-lib/vitest.config.js

@@ -1,11 +1,12 @@
-// @ts-check
 // Modules-lib test config
+import react from '@vitejs/plugin-react';
 import { defineProject, mergeConfig } from 'vitest/config';
 import rootConfig from '../../vitest.config';
 
 export default mergeConfig(
   rootConfig,
   defineProject({
+    plugins: [react()],
     test: {
       name: 'Modules Library',
       include: [`${import.meta.dirname}/**/__tests__/*.test.{ts,tsx}`],