1 files changed, 14 insertions, 31 deletions
diff --git a/test-suite/unit/shared-window.test.mjs b/test-suite/unit/shared-window.test.mjs
index 7afeb77..66041f4 100644
--- a/test-suite/unit/shared-window.test.mjs
+++ b/test-suite/unit/shared-window.test.mjs
@@ -1,5 +1,5 @@
-// Agent sessions share one window: a new session's tab goes into the window that already holds a
-// "Pi Session:" group, and a window is created only when there is none.
+// Each session has its own window: a window shows only one tab, so sessions sharing one hid each
+// other's tabs (no real input, blank screenshots). A session's extra tabs go into its own window.
 import assert from "node:assert/strict";
 import fs from "node:fs";
 import vm from "node:vm";
@@ -35,44 +35,27 @@ function worker({ groups = [], windows = { 5: { id: 5, type: "normal" } } } = {}
 }
 
 const piGroup = (windowId, name = "Pi Session: claude-code 1 other") => ({ id: 1, windowId, title: name });
+const ops = (calls) => calls.filter((c) => c.method !== "storage.set").map((c) => c.method);
 
-test("a new session's tab goes into the window that already holds another Pi session", async () => {
+test("a new session gets its own window even when another session's window exists", async () => {
   const { w, calls } = worker({ groups: [piGroup(5)] });
   await w.createAutomationTarget("claude-code 2 mine", "Pi Session: claude-code 2 mine");
-  assert.deepEqual(calls.filter((c) => c.method !== "storage.set").map((c) => c.method), ["tabs.create"]);
-  assert.equal(calls[0].props.windowId, 5);
+  assert.deepEqual(ops(calls), ["windows.create"]);
 });
 
-test("with no Pi window yet, the first session creates one", async () => {
-  const { w, calls } = worker({ groups: [] });
+test("a session that already has a group adds its tab to that group's window", async () => {
+  const { w, calls } = worker({ groups: [piGroup(5, "Pi Session: claude-code 2 mine")] });
   await w.createAutomationTarget("claude-code 2 mine", "Pi Session: claude-code 2 mine");
-  assert.deepEqual(calls.filter((c) => c.method !== "storage.set").map((c) => c.method), ["windows.create"]);
+  assert.deepEqual(ops(calls), ["tabs.create"]);
+  assert.equal(calls.find((c) => c.method === "tabs.create").props.windowId, 5);
 });
 
-test("groups that are not Pi sessions do not attract agent tabs", async () => {
-  const { w, calls } = worker({ groups: [{ id: 2, windowId: 5, title: "Work" }] });
-  await w.createAutomationTarget("claude-code 2 mine", "Pi Session: claude-code 2 mine");
-  assert.deepEqual(calls.filter((c) => c.method !== "storage.set").map((c) => c.method), ["windows.create"]);
-});
-
-test("a Pi group whose window is gone, or is not a normal window, is skipped", async () => {
-  const { w, calls } = worker({ groups: [piGroup(7), piGroup(8)], windows: { 8: { id: 8, type: "popup" } } });
-  await w.createAutomationTarget("claude-code 2 mine", "Pi Session: claude-code 2 mine");
-  assert.deepEqual(calls.filter((c) => c.method !== "storage.set").map((c) => c.method), ["windows.create"]);
-});
-
-test("the shared window is never recorded as ours, so cleanup cannot close it", async () => {
-  const { w, calls } = worker({ groups: [piGroup(5)] });
-  await w.createAutomationTarget("claude-code 2 mine", "Pi Session: claude-code 2 mine");
-  const stored = calls.filter((c) => c.method === "storage.set").map((c) => c.json).join("");
-  assert.doesNotMatch(stored, /"windowId":5/);
-});
-
-test("chrome_tab new for a session with no group uses the shared window, not the focused one", async () => {
+test("chrome_tab new for a session with no group goes to the session's own window, not the focused one", async () => {
   const { w, calls } = worker({ groups: [piGroup(5)] });
   w.groupTab = async (tab) => tab;
   w.bringToFront = async () => {};
-  await w.dispatch("tab.new", { url: "about:blank", groupTitle: "Pi Session: claude-code 2 mine", sessionKey: "claude-code 2 mine" });
-  const created = calls.find((c) => c.method === "tabs.create");
-  assert.equal(created.props.windowId, 5);
+  await w.dispatch("tab.new", { url: "https://x.test/", groupTitle: "Pi Session: claude-code 2 mine", sessionKey: "claude-code 2 mine" });
+  assert.deepEqual(ops(calls), ["windows.create"]);
+  assert.equal(calls[0].props.url, "https://x.test/");
+  assert.equal(calls[0].props.focused, false);
 });