diff --git a/web/shared.ts b/web/shared.ts index add7537..ecac19b 100644 --- a/web/shared.ts +++ b/web/shared.ts @@ -17,7 +17,16 @@ export type BrowserTarget = { export const browserTabPath = (id: string) => `/browser/${encodeURIComponent(id)}`; /** The previewed tab, or null on the bare `/browser` route — the list with nothing open. */ -export const useSelectedTabId = (): string | null => useParams<{ tabId?: string }>().tabId ?? null; +/** + * Which tab is open, from the URL. + * + * The param is `section`, not `tabId`. A plugin gets exactly two routes from the shell — `/` and + * `//:section` — so `:section` is the only param name that ever arrives, whatever the plugin + * means by it. This read `tabId` while it was a platform screen with its own `/browser/:tabId` route, + * and kept reading it after becoming a plugin: the id was in the URL, the route matched, and the panel + * saw undefined. The preview said "Select a tab to preview" with a tab plainly selected. + */ +export const useSelectedTabId = (): string | null => useParams<{ section?: string }>().section ?? null; // Both panels want this list and React Query serves both observers from one key, so the preview being // able to tell "no tab open" from "that tab is gone" costs no extra request. It needs to: a CDP target