test: enable full traces locally for Playwright UI debugging

Change trace from 'on-first-retry' to 'on' locally so Playwright UI mode
captures trace data for all test runs (not just retries). CI keeps the
efficient 'on-first-retry' setting via the CI env var.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: JamesPHoughton

playwright/playwright.config.mjs

@@ -36,7 +36,7 @@ export default defineConfig({
   ],
 
   use: {
-    trace: 'on-first-retry',
+    trace: process.env.CI ? 'on-first-retry' : 'on',
     screenshot: 'only-on-failure',
     video: 'retain-on-failure',