From 20ceaa1c6ade4950ed7d16caa5269ad2f4805db4 Mon Sep 17 00:00:00 2001 From: amankumarsingh77 Date: Tue, 5 May 2026 23:23:59 +0530 Subject: feat: tab ownership — isolate harness tabs from the user's browsing MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Track which page targets this harness session opened so the agent can never attach to, switch to, or close a tab the user opened themselves. New tabs are spawned in a dedicated Chrome window (newWindow:true on first open, openerId chaining after) so harness activity stays visually separate. - new OwnershipRegistry (src/cdp/ownership.ts) tracks owned targetIds and the harness window seed; mutations notify a single listener that persists to BrowserState (ownedTargetIds, harnessWindowTargetId) - attachFirstPage prefers an already-owned live target; otherwise opens a fresh harness-owned tab in its own window instead of grabbing whatever page the user is on - session subscribes Target.setDiscoverTargets and reaps targetDestroyed events so the registry stays in sync with reality - listTabs returns owned:boolean per tab and reconciles dead ids - browser_list_tabs defaults to scope:"owned"; pass scope:"all" to inspect the user's other tabs (read-only — they cannot be switched to or closed) - browser_switch_tab refuses non-owned tabs, with a clear remediation hint - browser_new_tab opens inside the harness window via openerId, falling back to a fresh window when the seed has been closed - new browser_close_tab tool — owned tabs only, supports unique hex prefix Co-Authored-By: Claude Opus 4.7 (1M context) --- src/state.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'src/state.ts') diff --git a/src/state.ts b/src/state.ts index 9dcef69..0629f40 100644 --- a/src/state.ts +++ b/src/state.ts @@ -10,9 +10,14 @@ import type { ExtensionAPI, ExtensionContext } from "@mariozechner/pi-coding-age export type BrowserState = { readonly namespace: string; readonly remoteBrowserId?: string; + readonly ownedTargetIds?: ReadonlyArray; + readonly harnessWindowTargetId?: string; }; -export const defaultState = (namespace = "default"): BrowserState => ({ namespace }); +export const defaultState = (namespace = "default"): BrowserState => ({ + namespace, + ownedTargetIds: [], +}); export const persistState = (pi: ExtensionAPI, state: BrowserState): void => { pi.appendEntry("browser-harness-state", state); -- cgit v1.2.3