stabilize ui browser suite on linux

Author: KSemenenko

ci-workflows-release-pr-validation.plan.md

@@ -217,6 +217,16 @@ Root-cause note:
   - `node /Users/ksemenenko/Developer/PrompterLive/tests/PrompterLive.App.UITests/bin/Debug/net10.0/.playwright/package/cli.js install chromium`
   - `dotnet test /Users/ksemenenko/Developer/PrompterLive/tests/PrompterLive.App.UITests/PrompterLive.App.UITests.csproj --no-build`
   - `dotnet format /Users/ksemenenko/Developer/PrompterLive/PrompterLive.slnx`
+- GitHub runs `23814810144` and `23814814816` proved that workflow-level isolation alone was not enough; the browser suite still failed under GitHub Linux with 25-28 flaky editor, go-live, and teleprompter tests.
+- The remaining GitHub-only root causes were narrowed to:
+  - mac-only keyboard shortcuts in UI tests (`Meta+A`, `Meta+Z`, `Meta+Shift+Z`) that broke Linux editor shortcuts and typing flows
+  - browser-suite self-contention from `4` parallel xUnit workers inside one process on the slower GitHub runner
+  - route/readiness assertions still relying on Playwright's short default timeouts instead of the suite's WASM-specific timeout budget
+- Local validation after the browser-suite stability fixes passed:
+  - `dotnet build /Users/ksemenenko/Developer/PrompterLive/PrompterLive.slnx -warnaserror`
+  - `dotnet test /Users/ksemenenko/Developer/PrompterLive/tests/PrompterLive.Core.Tests/PrompterLive.Core.Tests.csproj --no-build`
+  - `dotnet test /Users/ksemenenko/Developer/PrompterLive/tests/PrompterLive.App.Tests/PrompterLive.App.Tests.csproj --no-build`
+  - `dotnet test /Users/ksemenenko/Developer/PrompterLive/tests/PrompterLive.App.UITests/PrompterLive.App.UITests.csproj --no-build`
 
 ## Final Validation Skills And Commands
 

tests/PrompterLive.App.UITests/AGENTS.md

@@ -31,7 +31,7 @@
 - This suite is the primary acceptance gate for the product.
 - Major workflows must be covered by long scenario tests, not only narrow regression tests.
 - Major scenario tests must save screenshots under `output/playwright/`.
-- This suite resolves its origin at runtime and uses one `dotnet test` process with up to `4` parallel xUnit workers.
+- This suite resolves its origin at runtime and uses one `dotnet test` process with up to `2` parallel xUnit workers.
 - Do not keep separate concurrent `dotnet build` or `dotnet test` processes alive against the same test assets.
 - Prefer `PrompterLive.Shared.Contracts.AppRoutes`, `UiTestIds`, and other named constants over inline route or selector strings.
 - Use `data-testid` first. Raw text, role-name, and CSS selectors are allowed only when no stable `data-testid` contract exists yet and the missing contract is fixed in the same task.

tests/PrompterLive.App.UITests/Infrastructure/StandaloneAppFixture.cs

@@ -53,7 +53,10 @@ public async Task<IPage> NewPageAsync()
         });
         await ConfigureMediaHarnessAsync(context);
         _contexts.Add(context);
-        return await context.NewPageAsync();
+        var page = await context.NewPageAsync();
+        page.SetDefaultNavigationTimeout(BrowserTestConstants.Timing.DefaultNavigationTimeoutMs);
+        page.SetDefaultTimeout(BrowserTestConstants.Timing.ExtendedVisibleTimeoutMs);
+        return page;
     }
 
     private static class SharedRuntime

tests/PrompterLive.App.UITests/Infrastructure/TestAssemblyConfiguration.cs

@@ -4,5 +4,5 @@ namespace PrompterLive.App.UITests;
 
 internal static class UiTestExecution
 {
-    public const int MaxParallelThreads = 4;
+    public const int MaxParallelThreads = 2;
 }

tests/PrompterLive.App.UITests/Support/BrowserTestConstants.cs

@@ -412,6 +412,7 @@ public static class Localization
 
     public static class Timing
     {
+        public const int DefaultNavigationTimeoutMs = 20_000;
         public const int FastVisibleTimeoutMs = 5_000;
         public const int DefaultVisibleTimeoutMs = 10_000;
         public const int ExtendedVisibleTimeoutMs = 15_000;
@@ -448,10 +449,10 @@ public static class Regexes
     public static class Keyboard
     {
         public const string ArrowRight = "ArrowRight";
-        public const string SelectAll = "Meta+A";
+        public const string SelectAll = "ControlOrMeta+A";
         public const string Backspace = "Backspace";
-        public const string Undo = "Meta+Z";
-        public const string Redo = "Meta+Shift+Z";
+        public const string Undo = "ControlOrMeta+Z";
+        public const string Redo = "ControlOrMeta+Shift+Z";
         public const string Shift = "Shift";
     }