feat(workspaces): open workspace in a new browser tab

The 'Open' button on a workspace card did client-side navigation
(onNavigate -> pushState), opening the workspace in the SAME tab.
Replace the JS click handler with a native anchor using target="_blank"
(+ rel=noopener noreferrer) so clicking Open opens the workspace in a
NEW browser tab. The new tab does a full load and routes to /<id> via
the existing parsePath -> App wiring.

The card no longer needs the onNavigate prop (its only navigation was
the Open button), so drop it from WorkspaceCard + WorkspacesHome and
update the card tests to assert the new-tab link attributes instead of
the onNavigate call.

1 files changed, 14 insertions, 13 deletions

diff --git a/src/features/workspaces/ui/WorkspaceCard.test.ts b/src/features/workspaces/ui/WorkspaceCard.test.ts
index 385856d..0264d5f 100644
--- a/src/features/workspaces/ui/WorkspaceCard.test.ts
+++ b/src/features/workspaces/ui/WorkspaceCard.test.ts
@@ -45,9 +45,8 @@ function fakeStore() {
 describe("WorkspaceCard", () => {
 	it("renders the title, slug, and an Open link", () => {
 		const store = fakeStore() as unknown as WorkspaceStore;
-		const onNavigate = vi.fn();
 		render(WorkspaceCard, {
-			props: { ws: fakeEntry(), store, onNavigate },
+			props: { ws: fakeEntry(), store },
 		});
 		expect(screen.getByText("My Workspace")).toBeInTheDocument();
 		expect(screen.getByText("/my-ws")).toBeInTheDocument();
@@ -57,7 +56,7 @@ describe("WorkspaceCard", () => {
 	it("double-clicking the title reveals an edit input", async () => {
 		const user = userEvent.setup();
 		const store = fakeStore() as unknown as WorkspaceStore;
-		render(WorkspaceCard, { props: { ws: fakeEntry(), store, onNavigate: vi.fn() } });
+		render(WorkspaceCard, { props: { ws: fakeEntry(), store } });
 
 		await user.dblClick(screen.getByText("My Workspace"));
 		expect(screen.getByLabelText("Workspace title")).toHaveValue("My Workspace");
@@ -66,7 +65,7 @@ describe("WorkspaceCard", () => {
 	it("renames via the store on Enter", async () => {
 		const user = userEvent.setup();
 		const store = fakeStore() as unknown as WorkspaceStore;
-		render(WorkspaceCard, { props: { ws: fakeEntry(), store, onNavigate: vi.fn() } });
+		render(WorkspaceCard, { props: { ws: fakeEntry(), store } });
 
 		await user.dblClick(screen.getByText("My Workspace"));
 		const input = screen.getByLabelText("Workspace title");
@@ -80,7 +79,7 @@ describe("WorkspaceCard", () => {
 		const user = userEvent.setup();
 		const store = fakeStore() as unknown as WorkspaceStore;
 		render(WorkspaceCard, {
-			props: { ws: fakeEntry({ defaultCwd: "/old" }), store, onNavigate: vi.fn() },
+			props: { ws: fakeEntry({ defaultCwd: "/old" }), store },
 		});
 
 		const input = screen.getByLabelText("Default working directory");
@@ -99,7 +98,7 @@ describe("WorkspaceCard", () => {
 		const user = userEvent.setup();
 		const store = fakeStore() as unknown as WorkspaceStore;
 		render(WorkspaceCard, {
-			props: { ws: fakeEntry({ defaultCwd: "/old" }), store, onNavigate: vi.fn() },
+			props: { ws: fakeEntry({ defaultCwd: "/old" }), store },
 		});
 
 		const input = screen.getByLabelText("Default working directory");
@@ -109,13 +108,15 @@ describe("WorkspaceCard", () => {
 		expect(store.setDefaultCwd).toHaveBeenCalledWith("my-ws", null);
 	});
 
-	it("the Open link calls onNavigate with the workspace path (no full-card nav)", async () => {
-		const user = userEvent.setup();
+	it("the Open link opens the workspace in a new browser tab (no same-tab navigation)", () => {
 		const store = fakeStore() as unknown as WorkspaceStore;
-		const onNavigate = vi.fn();
-		render(WorkspaceCard, { props: { ws: fakeEntry(), store, onNavigate } });
-
-		await user.click(screen.getByRole("link", { name: "Open" }));
-		expect(onNavigate).toHaveBeenCalledWith("/my-ws");
+		render(WorkspaceCard, { props: { ws: fakeEntry(), store } });
+
+		const open = screen.getByRole("link", { name: "Open" });
+		// Native new-tab link: href points at the workspace, and target="_blank"
+		// opens it in a new browser tab rather than client-side navigating.
+		expect(open).toHaveAttribute("href", "/my-ws");
+		expect(open).toHaveAttribute("target", "_blank");
+		expect(open.getAttribute("rel") ?? "").toMatch(/noopener/);
 	});
 });