opendevbrowser 0.0.17 → 0.0.18

package/extension/dist/ops/ops-session-store.js

@@ -3,12 +3,21 @@ import { createCoordinatorId, TargetSessionCoordinator } from "./target-session-
 export class OpsRefStore {
     refsByTarget = new Map();
     snapshotByTarget = new Map();
+    refCounterByTarget = new Map();
+    nextRef(targetId) {
+        const next = (this.refCounterByTarget.get(targetId) ?? 0) + 1;
+        this.refCounterByTarget.set(targetId, next);
+        return `r${next}`;
+    }
     setSnapshot(targetId, entries) {
         const map = new Map();
+        const snapshotId = createCoordinatorId();
         for (const entry of entries) {
-            map.set(entry.ref, entry);
+            map.set(entry.ref, {
+                ...entry,
+                snapshotId
+            });
         }
-        const snapshotId = createCoordinatorId();
         this.refsByTarget.set(targetId, map);
         this.snapshotByTarget.set(targetId, snapshotId);
         return { snapshotId, targetId, count: entries.length };
@@ -33,11 +42,13 @@ export class OpsRefStore {
 }
 export class OpsSessionStore {
     coordinator = new TargetSessionCoordinator();
-    createSession(ownerClientId, tabId, leaseId, info, options) {
+    createSession(ownerClientId, tabId, leaseId, info, options, sessionId) {
         const parallelismPolicy = options?.parallelismPolicy ?? DEFAULT_OPS_PARALLELISM_POLICY;
         return this.coordinator.createSession(ownerClientId, tabId, leaseId, info, {
             refStore: new OpsRefStore(),
             syntheticTargets: new Map(),
+            dialogs: new Map(),
+            fileChoosers: new Map(),
             consoleEvents: [],
             networkEvents: [],
             networkRequests: new Map(),
@@ -53,7 +64,7 @@ export class OpsSessionStore {
             frozenSignals: 0,
             parallelismPolicy,
             parallelismState: createOpsGovernorState(parallelismPolicy, "extensionOpsHeaded")
-        });
+        }, sessionId);
     }
     get(sessionId) {
         return this.coordinator.get(sessionId);
@@ -71,8 +82,8 @@ export class OpsSessionStore {
         return this.coordinator.addTarget(sessionId, tabId, info);
     }
     removeTarget(sessionId, targetId) {
-        const target = this.coordinator.removeTarget(sessionId, targetId);
         const session = this.requireSession(sessionId);
+        const target = this.coordinator.removeTarget(sessionId, targetId);
         if (!target)
             return null;
         session.targetQueues.delete(targetId);
@@ -80,6 +91,8 @@ export class OpsSessionStore {
         session.targetQueueOldestAt.delete(targetId);
         session.refStore.clearTarget(targetId);
         session.syntheticTargets.delete(targetId);
+        session.dialogs.delete(targetId);
+        session.fileChoosers.delete(targetId);
         return target;
     }
     getTargetIdByTabId(sessionId, tabId) {
@@ -103,6 +116,71 @@ export class OpsSessionStore {
     listNamedTargets(sessionId) {
         return this.coordinator.listNamedTargets(sessionId);
     }
+    upsertSyntheticTarget(sessionId, target) {
+        const session = this.requireSession(sessionId);
+        const existing = session.syntheticTargets.get(target.targetId);
+        const nextTarget = {
+            ...(existing ?? {}),
+            ...target,
+            tabId: target.tabId,
+            type: target.type,
+            attachedAt: target.attachedAt
+        };
+        session.syntheticTargets.set(target.targetId, nextTarget);
+        return nextTarget;
+    }
+    getSyntheticTarget(sessionId, targetId) {
+        return this.requireSession(sessionId).syntheticTargets.get(targetId) ?? null;
+    }
+    listSyntheticTargets(sessionId) {
+        return Array.from(this.requireSession(sessionId).syntheticTargets.values());
+    }
+    findSyntheticTargetBySessionId(sessionId, childSessionId) {
+        const session = this.requireSession(sessionId);
+        for (const target of session.syntheticTargets.values()) {
+            if (target.sessionId === childSessionId) {
+                return target;
+            }
+        }
+        return null;
+    }
+    removeSyntheticTarget(sessionId, targetId) {
+        const session = this.requireSession(sessionId);
+        const existing = session.syntheticTargets.get(targetId) ?? null;
+        if (!existing) {
+            return null;
+        }
+        if (session.activeTargetId === targetId) {
+            session.activeTargetId = null;
+        }
+        session.syntheticTargets.delete(targetId);
+        session.refStore.clearTarget(targetId);
+        session.dialogs.delete(targetId);
+        session.fileChoosers.delete(targetId);
+        return existing;
+    }
+    getDialog(sessionId, targetId) {
+        return this.requireSession(sessionId).dialogs.get(targetId) ?? null;
+    }
+    setDialog(sessionId, targetId, dialog) {
+        const session = this.requireSession(sessionId);
+        session.dialogs.set(targetId, dialog);
+        return dialog;
+    }
+    clearDialog(sessionId, targetId) {
+        this.requireSession(sessionId).dialogs.delete(targetId);
+    }
+    getFileChooser(sessionId, targetId) {
+        return this.requireSession(sessionId).fileChoosers.get(targetId) ?? null;
+    }
+    setFileChooser(sessionId, targetId, chooser) {
+        const session = this.requireSession(sessionId);
+        session.fileChoosers.set(targetId, chooser);
+        return chooser;
+    }
+    clearFileChooser(sessionId, targetId) {
+        this.requireSession(sessionId).fileChoosers.delete(targetId);
+    }
     requireSession(sessionId) {
         return this.coordinator.requireSession(sessionId);
     }

package/extension/dist/ops/snapshot-builder.js

@@ -1,4 +1,4 @@
 import { buildSnapshotFromCdp } from "./snapshot-shared.js";
-export async function buildSnapshot(send, mode, mainFrameOnly = true, maxNodes) {
-    return await buildSnapshotFromCdp(send, mode, mainFrameOnly, maxNodes);
+export async function buildSnapshot(send, mode, createRef, mainFrameOnly = true, maxNodes) {
+    return await buildSnapshotFromCdp(send, mode, createRef, mainFrameOnly, maxNodes);
 }

package/extension/dist/ops/snapshot-shared.js

@@ -103,7 +103,7 @@ export const selectorFunction = function () {
     return buildPathSelector(this);
 };
 const SELECTOR_FUNCTION = selectorFunction.toString();
-export async function buildSnapshotFromCdp(send, mode, mainFrameOnly = true, maxNodes) {
+export async function buildSnapshotFromCdp(send, mode, createRef, mainFrameOnly = true, maxNodes) {
     await send("Accessibility.enable", {});
     await send("DOM.enable", {});
     const result = await send("Accessibility.getFullAXTree", {});
@@ -132,7 +132,7 @@ export async function buildSnapshotFromCdp(send, mode, mainFrameOnly = true, max
         const selector = await resolveSelector(send, node.backendDOMNodeId);
         if (!selector)
             continue;
-        const ref = `r${entries.length + 1}`;
+        const ref = createRef();
         const name = redactText(extractValue(node.name));
         const value = redactText(extractValue(node.value));
         const disabled = isTruthyProperty(node.properties, "disabled");

package/extension/dist/ops/target-session-coordinator.js

@@ -8,7 +8,8 @@ export class TargetSessionCoordinator {
             targetId,
             tabId,
             url: info?.url,
-            title: info?.title
+            title: info?.title,
+            openerTargetId: undefined
         };
         const createdAt = Date.now();
         const session = {
@@ -61,7 +62,8 @@ export class TargetSessionCoordinator {
             targetId,
             tabId,
             url: info?.url,
-            title: info?.title
+            title: info?.title,
+            openerTargetId: info?.openerTargetId
         };
         session.targets.set(targetId, target);
         this.tabToSession.set(tabId, sessionId);
@@ -85,7 +87,7 @@ export class TargetSessionCoordinator {
         }
         if (session.activeTargetId === targetId) {
             const [first] = session.targets.keys();
-            session.activeTargetId = first ?? "";
+            session.activeTargetId = first ?? null;
         }
         return target;
     }

package/extension/dist/popup.js

@@ -1,6 +1,6 @@
 import { DEFAULT_AUTO_CONNECT, DEFAULT_AUTO_PAIR, DEFAULT_DISCOVERY_PORT, DEFAULT_NATIVE_ENABLED, DEFAULT_PAIRING_ENABLED, DEFAULT_PAIRING_TOKEN, DEFAULT_RELAY_PORT } from "./relay-settings.js";
 import { logError } from "./logging.js";
-import { describeAnnotationItem, filterAnnotationPayload } from "./annotation-payload.js";
+import { describeAnnotationItem, filterAnnotationPayload, formatAnnotationDispatchReceipt } from "./annotation-payload.js";
 const statusEl = document.getElementById("status");
 const statusIndicator = document.getElementById("statusIndicator");
 const statusPill = document.getElementById("statusPill");
@@ -55,10 +55,20 @@ if (!statusEl
     || !annotationNote) {
     throw new Error("Popup DOM missing required elements");
 }
-const defaultNote = "Local relay only. Tokens stay on-device.";
+const defaultNote = "Local relay only. Page data and tokens stay on-device.";
+const defaultHealthNote = "Health check pending.";
 const defaultAnnotationNote = "No annotations captured yet.";
 const LAST_ANNOTATION_META_KEY = "annotationLastMeta";
+const MISSING_EXTENSION_RECEIVER_MESSAGE = "Could not establish connection. Receiving end does not exist.";
 let lastAnnotationPayload = null;
+let currentHealthNote = defaultHealthNote;
+const normalizePopupErrorMessage = (error) => {
+    const message = error instanceof Error ? error.message : String(error ?? "Unknown error");
+    if (message.includes(MISSING_EXTENSION_RECEIVER_MESSAGE)) {
+        return "Extension background is unavailable. Reload the extension and retry.";
+    }
+    return message;
+};
 const setNote = (message) => {
     const next = message && message.trim() ? message : defaultNote;
     statusNote.textContent = next;
@@ -67,6 +77,11 @@ const setAnnotationNote = (message) => {
     const next = message && message.trim() ? message : defaultAnnotationNote;
     annotationNote.textContent = next;
 };
+const setHealthNote = (message) => {
+    const next = message && message.trim() ? message : defaultHealthNote;
+    currentHealthNote = next;
+    healthNote.textContent = next;
+};
 const setStatus = (status) => {
     const isConnected = status === "connected";
     statusEl.textContent = isConnected ? "Connected" : "Disconnected";
@@ -95,7 +110,7 @@ const setHealth = (health) => {
         setHealthValue(healthInjected, "Unknown", "off");
         setHealthValue(healthCdp, "Unknown", "off");
         setHealthValue(healthPairing, "Unknown", "off");
-        healthNote.textContent = "Health check pending.";
+        setHealthNote();
         return;
     }
     setHealthValue(healthRelay, health.ok ? "Online" : "Offline", health.ok ? "ok" : "warn");
@@ -106,26 +121,28 @@ const setHealth = (health) => {
     setHealthValue(healthPairing, health.pairingRequired ? "Required" : "Not required", health.pairingRequired ? "warn" : "ok");
     if (health.lastHandshakeError) {
         const detail = health.lastHandshakeError.message ? ` ${health.lastHandshakeError.message}` : "";
-        healthNote.textContent = `Last handshake error: ${health.lastHandshakeError.code}.${detail}`;
+        setHealthNote(`Last handshake error: ${health.lastHandshakeError.code}.${detail}`);
     }
     else if (!health.ok) {
-        healthNote.textContent = `Relay health: ${formatReason(health.reason)}.`;
+        setHealthNote(`Relay health: ${formatReason(health.reason)}.`);
     }
     else {
-        healthNote.textContent = "Relay health OK.";
+        setHealthNote("Relay health OK.");
     }
 };
 const setInjectionStatus = (injected, detail) => {
     if (injected === null) {
         setHealthValue(healthInjected, "Unknown", "off");
+        healthNote.textContent = currentHealthNote;
         return;
     }
     if (injected) {
         setHealthValue(healthInjected, "Injected", "ok");
+        healthNote.textContent = currentHealthNote;
         return;
     }
     setHealthValue(healthInjected, "Not injected", "warn");
-    if (detail && healthNote.textContent === "Relay health OK.") {
+    if (detail && healthNote.textContent === currentHealthNote) {
         healthNote.textContent = `Annotation UI: ${detail}`;
     }
 };
@@ -158,8 +175,8 @@ const setNativeHealth = (health, enabled) => {
     }
     const formatted = formatNativeHealth(health);
     setHealthValue(healthNative, formatted.label, formatted.tone);
-    if (formatted.note && healthNote.textContent === "Relay health OK.") {
-        healthNote.textContent = `${healthNote.textContent} ${formatted.note}`;
+    if (formatted.note && currentHealthNote === "Relay health OK.") {
+        setHealthNote(`${currentHealthNote} ${formatted.note}`);
     }
 };
 const applyStatus = (response) => {
@@ -173,7 +190,7 @@ const sendMessage = (message) => {
         chrome.runtime.sendMessage(message, (response) => {
             const lastError = chrome.runtime.lastError;
             if (lastError) {
-                reject(new Error(lastError.message));
+                reject(new Error(normalizePopupErrorMessage(lastError.message)));
                 return;
             }
             if (!response) {
@@ -184,6 +201,17 @@ const sendMessage = (message) => {
         });
     });
 };
+const getPopupTabId = async () => {
+    try {
+        const tabs = await chrome.tabs.query({ active: true, currentWindow: true });
+        const tabId = tabs[0]?.id;
+        return typeof tabId === "number" ? tabId : undefined;
+    }
+    catch (error) {
+        logError("popup.tab_query", error, { code: "tab_query_failed" });
+        return undefined;
+    }
+};
 const setStorage = (items) => {
     return new Promise((resolve) => {
         chrome.storage.local.set(items, () => resolve());
@@ -191,7 +219,11 @@ const setStorage = (items) => {
 };
 const refreshInjectionStatus = async () => {
     try {
-        const response = await sendMessage({ type: "annotation:probe" });
+        const tabId = await getPopupTabId();
+        const response = await sendMessage({
+            type: "annotation:probe",
+            ...(typeof tabId === "number" ? { tabId } : {})
+        });
         setInjectionStatus(response.injected, response.detail);
     }
     catch (error) {
@@ -462,7 +494,7 @@ const copyPayloadToClipboard = async (payload, message) => {
     await copyTextToClipboard(JSON.stringify(payload, null, 2));
     setAnnotationNote(message);
 };
-const sendPayloadToAgent = async (payload, source, label, successMessage) => {
+const sendPayloadToAgent = async (payload, source, label) => {
     const response = await sendMessage({
         type: "annotation:sendPayload",
         payload,
@@ -472,7 +504,7 @@ const sendPayloadToAgent = async (payload, source, label, successMessage) => {
     if (!response.ok) {
         throw new Error(response.error?.message ?? "Agent dispatch failed.");
     }
-    setAnnotationNote(successMessage);
+    setAnnotationNote(formatAnnotationDispatchReceipt(response.receipt));
 };
 const copySelectedAnnotation = async (annotationIds, successMessage = "Copied annotation payload to clipboard.") => {
     const response = await loadAnnotationPayload(true);
@@ -499,7 +531,8 @@ const sendSelectedAnnotation = async (annotationIds, source, label, successMessa
     const payload = withPopupContext(annotationIds && annotationIds.length > 0
         ? filterAnnotationPayload(response.payload, annotationIds, { includeScreenshots: Boolean(response.payload.screenshots?.length) })
         : response.payload);
-    await sendPayloadToAgent(payload, source, label, successMessage);
+    void successMessage;
+    await sendPayloadToAgent(payload, source, label);
 };
 const toggle = async () => {
     const isConnected = statusEl.textContent === "Connected";
@@ -562,9 +595,11 @@ annotationStartButton.addEventListener("click", () => {
     (async () => {
         setAnnotationNote("Starting annotation...");
         const options = buildPopupAnnotationOptions();
+        const tabId = await getPopupTabId();
         const response = await sendMessage({
             type: "annotation:start",
-            options
+            options,
+            ...(typeof tabId === "number" ? { tabId } : {})
         });
         if (!response.ok) {
             const message = response.error?.message ?? "Annotation start failed.";