fix(web): stabilize flaky E2E tests

sessions.spec.js: Add per-RPC timeout (5s) to sendRpcFromPage so a
stuck WebSocket call can't consume the entire 45s test timeout. Reduce
retries from 40 to 10.

sandboxes.spec.js: Wait for container row to render before clicking
Delete. Add explicit timeout to "No containers found" assertion.

settings-nav.spec.js: Wait for WebSocket connection in settings section
load tests. Replace unreliable requestAnimationFrame waits with
setTimeout(100) in mockChannelsStatus. Add explicit timeout to matrix
senders visibility assertion.

Author: penso

crates/web/ui/e2e/specs/sandboxes.spec.js

@@ -238,7 +238,7 @@ test.describe("Sandboxes page – Running Containers", () => {
 
 		await navigateAndWait(page, "/settings/sandboxes");
 		await expect(page.getByRole("button", { name: "Refresh", exact: true })).toBeVisible();
-		await expect(page.getByText("No containers found.")).toBeVisible();
+		await expect(page.getByText("No containers found.")).toBeVisible({ timeout: 10_000 });
 
 		expect(pageErrors).toEqual([]);
 	});
@@ -396,7 +396,8 @@ test.describe("Sandboxes page – Container error handling", () => {
 		await navigateAndWait(page, "/settings/sandboxes");
 		await expect.poll(() => containerListFetches, { timeout: 10_000 }).toBeGreaterThan(0);
 
-		// Click the delete button
+		// Wait for the container row to render before clicking delete
+		await expect(page.getByText("moltis-sandbox-ghost")).toBeVisible({ timeout: 10_000 });
 		await page.getByRole("button", { name: "Delete", exact: true }).click();
 
 		// Error message should appear

crates/web/ui/e2e/specs/sessions.spec.js

@@ -16,29 +16,33 @@ function isRetryableRpcError(message) {
 
 async function sendRpcFromPage(page, method, params) {
 	let lastResponse = null;
-	for (let attempt = 0; attempt < 40; attempt++) {
+	for (let attempt = 0; attempt < 10; attempt++) {
 		if (attempt > 0) {
 			await waitForWsConnected(page);
 		}
 		lastResponse = await page
 			.evaluate(
-				async ({ methodName, methodParams }) => {
+				async ({ methodName, methodParams, timeoutMs }) => {
 					var appScript = document.querySelector('script[type="module"][src*="js/app.js"]');
 					if (!appScript) throw new Error("app module script not found");
 					var appUrl = new URL(appScript.src, window.location.origin);
 					var prefix = appUrl.href.slice(0, appUrl.href.length - "js/app.js".length);
 					var helpers = await import(`${prefix}js/helpers.js`);
-					return helpers.sendRpc(methodName, methodParams);
+					var rpc = helpers.sendRpc(methodName, methodParams);
+					var timeout = new Promise((_, reject) => setTimeout(() => reject(new Error("RPC timeout")), timeoutMs));
+					return Promise.race([rpc, timeout]);
 				},
 				{
 					methodName: method,
 					methodParams: params,
+					timeoutMs: 5000,
 				},
 			)
 			.catch((error) => ({ ok: false, error: { message: error?.message || String(error) } }));
 
 		if (lastResponse?.ok) return lastResponse;
-		if (!isRetryableRpcError(lastResponse?.error?.message)) return lastResponse;
+		if (!isRetryableRpcError(lastResponse?.error?.message) && !lastResponse?.error?.message?.includes("RPC timeout"))
+			return lastResponse;
 	}
 	return lastResponse;
 }

crates/web/ui/e2e/specs/settings-nav.spec.js

@@ -90,8 +90,7 @@ async function mockChannelsStatus(page, { channels, senders = [], allowRetryOwne
 					} else {
 						await channelsPage.prefetchChannels();
 					}
-					await new Promise((resolve) => requestAnimationFrame(() => resolve()));
-					await new Promise((resolve) => requestAnimationFrame(() => resolve()));
+					await new Promise((resolve) => setTimeout(resolve, 100));
 				},
 				{ channels, senders, allowRetryOwnership, label },
 			);
@@ -224,6 +223,7 @@ test.describe("Settings navigation", () => {
 		test(`settings/${section.id} loads without errors`, async ({ page }) => {
 			const pageErrors = watchPageErrors(page);
 			await navigateAndWait(page, `/settings/${section.id}`);
+			await waitForWsConnected(page);
 
 			await expect(page).toHaveURL(new RegExp(`/settings/${section.id}$`));
 
@@ -1305,10 +1305,10 @@ test.describe("Settings navigation", () => {
 			});
 			state.setConnected(true);
 			await channelsPage.prefetchChannels();
-			await new Promise((resolve) => requestAnimationFrame(() => resolve()));
+			await new Promise((resolve) => setTimeout(resolve, 100));
 		});
 
-		await expect(page.getByText("Matrix (moltis-testbot)", { exact: true })).toBeVisible();
+		await expect(page.getByText("Matrix (moltis-testbot)", { exact: true })).toBeVisible({ timeout: 10_000 });
 		await page.getByRole("tab", { name: /Senders/ }).click();
 		await expect.poll(() => page.locator(".senders-table tbody tr").count(), { timeout: 10_000 }).toBe(1);
 		await expect(page.getByText("Alice", { exact: true })).toBeVisible();