From baec9a7585fc87b3b3930372b3685659b54a2cee Mon Sep 17 00:00:00 2001
From: Etienne Lescot <etiennelescot@gmail.com>
Date: Fri, 27 Mar 2026 16:28:53 +0100
Subject: [PATCH] fix: focusable element when webcam expanded with no devices,
 add error test

- LaunchWindow: render sr-only <select> when webcamExpanded but
  cameraDevices.length === 0 (loading/error/empty), so keyboard users
  always have a focusable element even in no-camera states
- useCameraDevices.test: add error-branch test asserting error message,
  empty devices array and isLoading=false when enumerateDevices rejects
---
 src/components/launch/LaunchWindow.tsx |  2 +-
 src/hooks/useCameraDevices.test.ts     | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/components/launch/LaunchWindow.tsx b/src/components/launch/LaunchWindow.tsx
index 0d7ae23..f1b66b8 100644
--- a/src/components/launch/LaunchWindow.tsx
+++ b/src/components/launch/LaunchWindow.tsx
@@ -365,7 +365,7 @@ export function LaunchWindow() {
 											/>
 										</>
 									))}
-								{!webcamExpanded && (
+								{(!webcamExpanded || cameraDevices.length === 0) && (
 									<select
 										value={webcamDeviceId || selectedCameraId}
 										onChange={(e) => {
diff --git a/src/hooks/useCameraDevices.test.ts b/src/hooks/useCameraDevices.test.ts
index 9026ce6..5ca21bc 100644
--- a/src/hooks/useCameraDevices.test.ts
+++ b/src/hooks/useCameraDevices.test.ts
@@ -71,6 +71,19 @@ describe("useCameraDevices", () => {
 		expect(mockGetUserMedia).not.toHaveBeenCalled();
 	});
 
+	it("should set error state when enumeration fails", async () => {
+		mockEnumerateDevices.mockRejectedValueOnce(new Error("Permission denied"));
+
+		const { result } = renderHook(() => useCameraDevices(true));
+
+		await waitFor(() => {
+			expect(result.current.error).toBe("Permission denied");
+		});
+
+		expect(result.current.devices).toHaveLength(0);
+		expect(result.current.isLoading).toBe(false);
+	});
+
 	it("should fall back to first available device when selected device is unplugged", async () => {
 		const { result } = renderHook(() => useCameraDevices(true));