Update CI workflow: permissions, test steps, and Playwright

Adds explicit permissions for contents and pull-requests. Simplifies test steps by removing individual test commands in favor of a single coverage run. Limits Playwright browser install to Chromium only. Updates build step comments and references for clarity.

Author: edeno

.github/workflows/test.yml

@@ -6,6 +6,10 @@ on:
   pull_request:
     branches: [main]
 
+permissions:
+  contents: read
+  pull-requests: write
+
 jobs:
   test:
     name: Unit & Integration Tests
@@ -27,21 +31,8 @@ jobs:
       - name: Run linter
         run: npm run lint
 
-      - name: Run baseline tests
-        run: npm run test:baseline
-        continue-on-error: false
-
-      - name: Run unit tests (if they exist)
-        run: npm test -- run unit || echo "No unit tests found yet"
-        continue-on-error: true
-
-      - name: Run integration tests (if they exist)
-        run: npm run test:integration || echo "No integration tests found yet"
-        continue-on-error: true
-
       - name: Run all tests with coverage
-        run: npm run test:coverage -- run || echo "Tests passed with warnings"
-        continue-on-error: false
+        run: npm run test:coverage
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4
@@ -78,7 +69,7 @@ jobs:
         run: npm ci
 
       - name: Install Playwright browsers
-        run: npx playwright install --with-deps chromium firefox webkit
+        run: npx playwright install --with-deps chromium
 
       - name: Run E2E tests
         run: npm run test:e2e
@@ -164,10 +155,11 @@ jobs:
         run: npm ci
 
       - name: Build production bundle
-        # PHASE 0 TEMPORARY: Disable treating warnings as errors
+        # TODO: Remove CI=false after fixing ESLint warnings (target: Phase 3)
         # Create React App treats warnings as errors when CI=true
-        # Re-enable in Phase 3 (Code Quality & Refactoring) by removing CI=false
-        # Known warnings: unused variables in App.js, ArrayUpdateMenu.jsx, ListElement.jsx
+        # Known warnings: unused variables in App.js (lines 119, 122, 596, 618, 2852)
+        #                 unused variables in ArrayUpdateMenu.jsx (line 13)
+        # Issue: https://github.com/LorenFrankLab/rec_to_nwb_yaml_creator/issues/[TBD]
         run: CI=false npm run build
 
       - name: Upload build artifacts