fix: E2E tests and keyboard handling for new sidebar design

- Update E2E tests to click expand button instead of project row
- Fix keyboard handler to ignore events bubbling from child buttons
  (addresses Codex review feedback)

Author: kylecarbs

src/browser/components/ProjectSidebar.tsx

@@ -515,6 +515,10 @@ const ProjectSidebarInner: React.FC<ProjectSidebarProps> = ({
                           selected={false}
                           onClick={() => handleAddWorkspace(projectPath)}
                           onKeyDown={(e: React.KeyboardEvent) => {
+                            // Ignore key events from child buttons
+                            if (e.target instanceof HTMLElement && e.target !== e.currentTarget) {
+                              return;
+                            }
                             if (e.key === "Enter" || e.key === " ") {
                               e.preventDefault();
                               handleAddWorkspace(projectPath);

tests/e2e/utils/ui.ts

@@ -92,7 +92,9 @@ export function createWorkspaceUI(page: Page, context: DemoProjectConfig): Works
       const workspaceItem = workspaceItems.first();
       const isVisible = await workspaceItem.isVisible().catch(() => false);
       if (!isVisible) {
-        await projectItem.click();
+        // Click the expand/collapse button within the project item
+        const expandButton = projectItem.getByRole("button", { name: /expand project/i });
+        await expandButton.click();
         await workspaceItem.waitFor({ state: "visible" });
       }