feat: always record Playwright video + upload artifacts on success

- Change video from 'retain-on-failure' to 'on' in playwright.config.ts
- Change artifact upload from 'if: failure()' to 'if: always()' in
  both CI workflows (master + PR)
- Video recordings now available as downloadable artifacts for every
  test run, regardless of pass/fail status

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: renemadsen

.github/workflows/dotnet-core-master.yml

@@ -196,7 +196,7 @@ jobs:
       if: ${{ failure() }}
       run: cat docker_run_log
     - name: Archive Playwright report
-      if: failure()
+      if: always()
       uses: actions/upload-artifact@v4
       with:
         name: playwright-report-${{ matrix.test }}

.github/workflows/dotnet-core-pr.yml

@@ -185,7 +185,7 @@ jobs:
       if: ${{ failure() }}
       run: cat docker_run_log
     - name: Archive Playwright report
-      if: failure()
+      if: always()
       uses: actions/upload-artifact@v4
       with:
         name: playwright-report-${{ matrix.test }}

docs/superpowers/specs/2026-04-09-playwright-always-record-video-design.md

@@ -0,0 +1,27 @@
+# Playwright Always-Record Video Design
+
+## Context
+
+Playwright tests currently only capture video/screenshot on failure (`retain-on-failure`). This makes it impossible to review test behavior when tests pass. We want video recordings always available as CI artifacts, similar to Cypress video recording.
+
+## Changes
+
+### 1. playwright.config.ts - Always record video
+
+Change `video` from `'retain-on-failure'` to `'on'`. Keep screenshot as `'only-on-failure'` (screenshots are less useful when video exists). Skip trace (large files, not needed for general monitoring).
+
+### 2. CI workflow - Upload artifacts always
+
+Change artifact upload condition from `if: failure()` to `if: always()` in both workflow files. This ensures video recordings are available as downloadable artifacts even when all tests pass.
+
+### Files to modify
+
+| File | Change |
+|------|--------|
+| `eform-client/playwright.config.ts` | `video: 'retain-on-failure'` -> `video: 'on'` |
+| `.github/workflows/dotnet-core-master.yml` | `if: failure()` -> `if: always()` on artifact upload step |
+| `.github/workflows/dotnet-core-pr.yml` | Same change |
+
+## Verification
+
+After merge, check any CI run's artifacts - every playwright-report-{a..o} artifact should contain video files (`.webm`) regardless of test outcome.

eform-client/playwright.config.ts

@@ -6,7 +6,7 @@ export default defineConfig({
   use: {
     baseURL: 'http://localhost:4200',
     viewport: { width: 1920, height: 1080 },
-    video: 'retain-on-failure',
+    video: 'on',
     screenshot: 'only-on-failure',
   },
   reporter: [