codex-chatgpt-control 0.5.1-alpha.3 → 0.5.1-alpha.4

package/dist/src/commands/configuration.js

@@ -6,6 +6,7 @@ import { tabIdFromPage } from "../browser/attach.js";
 import { contextFromPage } from "./context.js";
 import { detectExperience, openExperience } from "./experience.js";
 import { setMode } from "./modes.js";
+import { closeChatPopover, inspectChatPopover, readChatPopover, selectChatPopoverEffort, selectChatPopoverModel, selectChatPopoverSpeed } from "./chat-popover.js";
 import { ensurePage } from "./session.js";
 const WORK_AXES = ["model", "effort", "speed"];
 const CONFIGURATION_CONTROL_DISCOVERY_TIMEOUT_MS = 5_000;
@@ -13,11 +14,11 @@ const CONFIGURATION_CONTROL_POLL_MS = 250;
 const CONFIGURATION_SELECTION_MAX_ATTEMPTS = 6;
 const CONFIGURATION_SELECTION_RETRY_MS = 400;
 const CONFIGURATION_AXIS_ORDER = [
+    "modelVersion",
     "model",
     "intelligence",
     "effort",
     "speed",
-    "modelVersion",
 ];
 export async function inspectConfiguration(env, args = {}) {
     const boot = await ensurePage(env);
@@ -49,11 +50,25 @@ export async function inspectConfiguration(env, args = {}) {
             };
         }
         const experience = detected.data.experience;
+        const initialChatPopover = experience !== "unknown" ? (await readChatPopover(page)).snapshot : undefined;
         const initialPanel = await readConfigurationPanel(page);
         const rootOpened = experience !== "unknown" && await waitForConfigurationRoot(page, experience, args.timeoutMs);
         if (rootOpened) {
             await page.waitForTimeout?.(150);
         }
+        if (experience !== "unknown" && (await readChatPopover(page)).snapshot !== undefined) {
+            try {
+                const chat = await inspectChatPopover(page);
+                const data = configurationInspectionFromPopover(experience, detected.data.evidence, chat);
+                if (args.includeOptions === false)
+                    data.options = {};
+                return resultOk(data, await contextFromPage(page, { experience, selectorProfile: data.selectorProfile }), data.verified ? [] : ["The owned configuration views did not provide unique active model and effort evidence."]);
+            }
+            finally {
+                if (initialChatPopover === undefined)
+                    await closeConfigurationMenus(page);
+            }
+        }
         const workAdvancedOpened = experience !== "work"
             || (rootOpened && await ensureWorkAdvancedPanel(page));
         if (experience === "work" && workAdvancedOpened) {
@@ -95,6 +110,32 @@ export async function inspectConfiguration(env, args = {}) {
         return resultError(error instanceof Error ? error : new Error(String(error)), await contextFromPage(page));
     }
 }
+/** Shared owned popover, with surface-specific model semantics and observed axes only. */
+export function configurationInspectionFromPopover(experience, evidence, popover) {
+    const data = { experience, selectorProfile: experience === "work" ? "work_basic_v1" : "chat_simplified_v1",
+        availableAxes: [], active: {}, options: {}, verified: false, evidence };
+    if (popover?.slider !== undefined) {
+        data.availableAxes.push("effort");
+        if (popover.effort !== undefined) {
+            data.active.effort = popover.effort;
+            data.options.effort = [{ id: normalizeConfigurationId(popover.effort), label: popover.effort, selected: true }];
+        }
+    }
+    if (popover !== undefined && popover.modelOptions.length > 0) {
+        const axis = experience === "work" ? "model" : "modelVersion";
+        data.availableAxes.push(axis);
+        if (popover.activeModel !== undefined)
+            data.active[axis] = popover.activeModel;
+        data.options[axis] = popover.modelOptions.map(option => ({ id: normalizeConfigurationId(option.label), label: option.label, selected: option.checked }));
+    }
+    if (experience === "work" && popover?.speed !== undefined) {
+        data.availableAxes.push("speed");
+        data.active.speed = popover.speed;
+        data.options.speed = ["Standard", "Fast"].map(label => ({ id: normalizeConfigurationId(label), label, selected: label === popover.speed }));
+    }
+    data.verified = popover?.effort !== undefined && popover.activeModel !== undefined;
+    return data;
+}
 async function waitForConfigurationRoot(page, experience, timeoutMs) {
     const discoveryMs = Math.min(timeoutMs ?? CONFIGURATION_CONTROL_DISCOVERY_TIMEOUT_MS, CONFIGURATION_CONTROL_DISCOVERY_TIMEOUT_MS);
     const attempts = Math.max(1, Math.ceil(Math.max(0, discoveryMs) / CONFIGURATION_CONTROL_POLL_MS));
@@ -154,7 +195,15 @@ export async function applyConfiguration(env, args) {
         }
         const selected = [];
         for (const [axis, requested] of selectionEntries(desired)) {
-            const active = activeConfigurationValue(before, axis);
+            // A model change can reset effort and alter its available range.
+            // Re-observe between axes instead of trusting the initial snapshot.
+            const currentResult = selected.length === 0 ? beforeResult : await inspectConfiguration(env, { includeOptions: false });
+            if (!currentResult.ok || currentResult.data === undefined)
+                return forwardFailure(currentResult);
+            const current = currentResult.data;
+            if (current.experience !== before.experience)
+                return configurationFailure(page, before, desired, selected, "The composer experience changed during configuration selection.", "experience_mismatch");
+            const active = activeConfigurationValue(current, axis);
             if (active !== undefined && configurationValueMatches(active, requested)) {
                 selected.push({ axis, requested, selected: active });
                 continue;
@@ -228,22 +277,37 @@ export function configurationInspectionFromSurface(experience, detectedProfile,
         const simplified = chatMenuLooksSimplified(menuItems);
         selectorProfile = simplified ? "chat_simplified_v1" : detectedProfile;
         const axis = simplified ? "intelligence" : "effort";
-        if (menuItems.length > 0 || panel.openerLabel !== undefined) {
-            availableAxes.push(axis);
-        }
-        if (panel.openerLabel !== undefined) {
-            active[axis] = panel.openerLabel;
-        }
+        const modelRows = menuItems.filter(item => item.role === "menuitemradio"
+            && item.hasPopup !== true
+            && (/^(?:gpt[\s-]|o\d+(?:\b|$)|\d+(?:\.\d+)?$)/i.test(item.label)
+                || localeLabels.modeOptions.latest.some(label => visibleLabelMatches(item.label, label))));
         const chatOptions = menuItems
-            .filter(item => !isConfigurationAxisRow(item.label))
+            .filter(item => !isConfigurationAxisRow(item.label)
+            && !modelRows.includes(item)
+            && item.hasPopup !== true
+            && !/^gpt[\s-]/i.test(item.label)
+            && item.ariaLabel !== "Select model")
             .map(menuItemToOption);
-        if (chatOptions.length > 0) {
+        const selectedEffort = chatOptions.filter(option => option.selected === true);
+        const openerValue = panel.openerValue ?? panel.openerLabel;
+        if (chatOptions.length > 0 || openerValue !== undefined)
+            availableAxes.push(axis);
+        if (selectedEffort.length === 1)
+            active[axis] = selectedEffort[0].label;
+        else if (openerValue !== undefined
+            && !isConfigurationAxisRow(openerValue)
+            && !/^(?:thinking effort|select model|power)$/i.test(openerValue))
+            active[axis] = openerValue;
+        if (chatOptions.length > 0)
             options[axis] = chatOptions;
-        }
-        const modelRows = menuItems.filter(item => /^gpt[\s-]/i.test(item.label) || item.hasPopup === true);
-        if (modelRows.length > 0) {
+        const modelOpeners = menuItems.filter(item => item.role !== "menuitemradio"
+            && /^gpt[\s-]/i.test(item.label));
+        if (modelRows.length > 0 || modelOpeners.length > 0) {
             availableAxes.push("modelVersion");
-            options.modelVersion = modelRows.map(menuItemToOption);
+            options.modelVersion = [...modelRows, ...modelOpeners].map(menuItemToOption);
+            const checked = modelRows.filter(item => item.checked === true);
+            if (checked.length === 1)
+                active.modelVersion = checked[0].label;
         }
     }
     return {
@@ -268,6 +332,24 @@ async function selectWorkAxis(env, axis, requested, timeoutMs) {
     if (!WORK_AXES.includes(axis)) {
         return undefined;
     }
+    const initialPopover = (await readChatPopover(page)).snapshot;
+    await openConfigurationRoot(page, "work");
+    if ((await readChatPopover(page)).snapshot !== undefined) {
+        try {
+            const labels = configurationSemanticLabels(requested);
+            if (axis === "model")
+                return await selectChatPopoverModel(page, labels);
+            if (axis === "effort")
+                return await selectChatPopoverEffort(page, labels);
+            if (axis === "speed")
+                return await selectChatPopoverSpeed(page, labels);
+            return undefined;
+        }
+        finally {
+            if (initialPopover === undefined)
+                await closeConfigurationMenus(page);
+        }
+    }
     const retryWindowMs = Math.min(timeoutMs ?? CONFIGURATION_CONTROL_DISCOVERY_TIMEOUT_MS, CONFIGURATION_CONTROL_DISCOVERY_TIMEOUT_MS);
     const attempts = Math.max(2, Math.min(CONFIGURATION_SELECTION_MAX_ATTEMPTS, Math.ceil(Math.max(0, retryWindowMs) / CONFIGURATION_SELECTION_RETRY_MS) + 1));
     for (let attempt = 0; attempt < attempts; attempt += 1) {
@@ -424,6 +506,22 @@ function workAxisOptionLabelMatches(axis, label) {
     return candidates.some(candidate => visibleLabelMatches(label, candidate));
 }
 async function selectChatAxis(env, axis, requested, timeoutMs) {
+    const page = env.page;
+    const initialPopover = (await readChatPopover(page)).snapshot;
+    await openConfigurationRoot(page, "chat");
+    if ((await readChatPopover(page)).snapshot !== undefined) {
+        try {
+            if (axis === "modelVersion")
+                return await selectChatPopoverModel(page, configurationSemanticLabels(requested));
+            if (axis === "effort" || axis === "intelligence" || axis === "model")
+                return await selectChatPopoverEffort(page, configurationSemanticLabels(requested));
+            return undefined;
+        }
+        finally {
+            if (initialPopover === undefined)
+                await closeConfigurationMenus(page);
+        }
+    }
     const legacyArgs = axis === "modelVersion"
         ? { modelVersion: requested }
         : axis === "intelligence"
@@ -443,6 +541,8 @@ async function selectChatAxis(env, axis, requested, timeoutMs) {
     return result.ok ? result.data?.selected.at(-1) : undefined;
 }
 async function openConfigurationRoot(page, experience) {
+    if (experience !== "unknown" && (await readChatPopover(page)).snapshot !== undefined)
+        return true;
     const existing = await readConfigurationPanel(page);
     if (existing.axisRows.length > 0) {
         return true;
@@ -654,6 +754,7 @@ async function readConfigurationPanel(page) {
             const html = control;
             return {
                 label: normalize(control.getAttribute("aria-label") ?? html.innerText ?? control.textContent ?? ""),
+                value: normalize(html.innerText ?? control.textContent ?? ""),
                 testId: control.getAttribute("data-testid") ?? ""
             };
         })
@@ -666,6 +767,8 @@ async function readConfigurationPanel(page) {
         };
         if (openerCandidates.length === 1 && openerCandidates[0]?.label.length) {
             result.openerLabel = openerCandidates[0].label;
+            if (openerCandidates[0].value.length > 0)
+                result.openerValue = openerCandidates[0].value;
         }
         return result;
     }, localeLabels.configurationAxes).catch(() => ({ axisRows: [], advancedVisible: false }));
@@ -832,6 +935,10 @@ function normalizeConfigurationId(value) {
         .trim();
 }
 async function closeConfigurationMenus(page) {
+    if ((await readChatPopover(page)).snapshot !== undefined) {
+        await closeChatPopover(page);
+        return;
+    }
     if (!await pressConfigurationEscape(page))
         return;
     await page.waitForTimeout?.(50);

package/dist/src/commands/context.js

@@ -1,11 +1,18 @@
 import { countPageMessages } from "../dom/messages.js";
 import { parseConversationId } from "../browser/page-state.js";
 import { withTimeout } from "./timeouts.js";
+import { chatGPTAttachmentContextUrl } from "../browser/chatgpt-url.js";
 export async function contextFromPage(page, partial = {}, options = {}) {
+    const partialAttachmentUrl = chatGPTAttachmentContextUrl(partial.url);
+    const safePartial = partialAttachmentUrl === undefined ? partial : { ...partial, url: partialAttachmentUrl };
     if (page === undefined || options.minimal === true) {
-        return { timestamp: new Date().toISOString(), ...partial };
+        return { timestamp: new Date().toISOString(), ...safePartial };
+    }
+    const url = typeof page.url === "function" ? await Promise.resolve().then(() => page.url()).catch(() => safePartial.url) : safePartial.url;
+    const attachmentUrl = chatGPTAttachmentContextUrl(url);
+    if (attachmentUrl !== undefined) {
+        return { timestamp: new Date().toISOString(), ...safePartial, url: attachmentUrl };
     }
-    const url = typeof page.url === "function" ? await Promise.resolve(page.url()).catch(() => partial.url) : partial.url;
     const title = typeof page.title === "function"
         ? await withTimeout(page.title(), 1000, "Timed out while reading page title.").catch(() => partial.title)
         : partial.title;
@@ -16,7 +23,7 @@ export async function contextFromPage(page, partial = {}, options = {}) {
     const conversationId = url !== undefined ? parseConversationId(url) : partial.conversationId;
     const context = {
         timestamp: new Date().toISOString(),
-        ...partial
+        ...safePartial
     };
     if (url !== undefined) {
         context.url = url;

package/dist/src/commands/doctor.js

@@ -161,7 +161,7 @@ function downloadCheck(env) {
     if (page === undefined)
         return unknown("Download readiness requires a bootstrapped ChatGPT page.");
     return typeof page.waitForEvent === "function"
-        ? ok("Browser download events are available.")
+        ? ok("The browser exposes a download event API; a completed download receipt has not been verified.")
         : unsupported("The active browser page does not expose download events.");
 }
 async function clipboardCheck() {

package/dist/src/commands/experience.d.ts

@@ -3,6 +3,11 @@ type SurfaceSnapshot = {
     url: string;
     composerLabels: string[];
     mainControls: string[];
+    /** Controls outside overlays, scoped to the composer (or main fallback). */
+    composerControls?: string[];
+    /** Separate visible control roots; unrelated overlays cannot combine axes. */
+    controlGroups?: string[][];
+    rootBudgetExceeded?: true;
     mainText: string;
     selectedSurfaceLabels?: string[];
 };

package/dist/src/commands/experience.js

@@ -7,15 +7,30 @@ import { ensurePage } from "./session.js";
 const CHATGPT_HOME = "https://chatgpt.com/";
 const EXPERIENCE_CONTROL_DISCOVERY_TIMEOUT_MS = 15_000;
 const EXPERIENCE_POLL_MS = 250;
+const EXPERIENCE_READINESS_TIMEOUT_MS = 1500;
 export async function detectExperience(env, args = {}) {
-    void args;
     const boot = await ensurePage(env);
     if (!boot.ok) {
         return boot;
     }
     const page = env.page;
     try {
-        const data = detectExperienceFromSnapshot(await readSurfaceSnapshot(page));
+        const readinessMs = Math.max(0, Math.min(args.timeoutMs ?? EXPERIENCE_READINESS_TIMEOUT_MS, EXPERIENCE_CONTROL_DISCOVERY_TIMEOUT_MS));
+        const readinessDeadline = Date.now() + readinessMs;
+        let data = detectExperienceFromSnapshot(await readSurfaceSnapshot(page));
+        // An empty post-navigation capture is loading evidence, not a final
+        // classification. Never wait out a known blocker or a genuine score tie.
+        for (let attempt = 1; data.experience === "unknown" && data.evidence.length === 0
+            && Date.now() < readinessDeadline
+            && attempt < pollAttempts(readinessMs, EXPERIENCE_POLL_MS); attempt += 1) {
+            const blocker = await experiencePageBlocker(page, data);
+            if (blocker !== undefined)
+                return blocker;
+            if (page.waitForTimeout === undefined || Date.now() >= readinessDeadline)
+                break;
+            await page.waitForTimeout(Math.min(EXPERIENCE_POLL_MS, readinessDeadline - Date.now()));
+            data = detectExperienceFromSnapshot(await readSurfaceSnapshot(page));
+        }
         return resultOk(data, await contextFromPage(page, {
             experience: data.experience,
             selectorProfile: data.selectorProfile
@@ -177,9 +192,14 @@ async function navigateConversationToSurfaceHome(page, timeoutMs) {
     return true;
 }
 export function detectExperienceFromSnapshot(snapshot) {
+    if (snapshot.rootBudgetExceeded === true) {
+        return {
+            experience: "unknown", selectorProfile: "unknown", confidence: "low",
+            evidence: [{ source: "control", label: "Experience surface root budget exceeded" }]
+        };
+    }
     const evidence = [];
     const composerLabels = snapshot.composerLabels.map(normalizeForLabelMatch);
-    const controls = snapshot.mainControls.map(normalizeForLabelMatch);
     const mainText = normalizeForLabelMatch(snapshot.mainText);
     const selectedSurfaceLabels = (snapshot.selectedSurfaceLabels ?? []).map(normalizeForLabelMatch);
     const url = snapshot.url.toLowerCase();
@@ -201,13 +221,18 @@ export function detectExperienceFromSnapshot(snapshot) {
     for (const label of chatComposer) {
         evidence.push({ source: "composer", label });
     }
-    const workAxisCount = ["model", "effort", "speed"]
-        .filter(axis => hasAnyLabel(controls, localeLabels.configurationAxes[axis]))
-        .length;
-    if (workAxisCount >= 2) {
-        evidence.push({ source: "control", label: `Work configuration axes (${workAxisCount}/3)` });
+    // Model and effort also occur in the Chat composer popover. Only the
+    // complete Work axis group, including speed in the same visible pane,
+    // supplies Work evidence. Do not combine labels across unrelated overlays.
+    const workAxes = (snapshot.controlGroups ?? [snapshot.mainControls]).some(group => {
+        const groupLabels = group.map(normalizeForLabelMatch);
+        return ["model", "effort", "speed"].every(axis => hasAnyLabel(groupLabels, localeLabels.configurationAxes[axis]));
+    });
+    if (workAxes) {
+        evidence.push({ source: "control", label: "Work configuration axes (3/3)" });
     }
-    const workConfigurationOpener = controls.some(label => /\b(?:gpt[\s-]?\d|\d+(?:\.\d+)+|sol|luna|terra)\b/i.test(label)
+    const composerControls = (snapshot.composerControls ?? snapshot.mainControls).map(normalizeForLabelMatch);
+    const workConfigurationOpener = composerControls.some(label => /\b(?:gpt[\s-]?\d|\d+(?:\.\d+)+|sol|luna|terra)\b/i.test(label)
         && hasAnyLabel([label], [
             ...localeLabels.configurationOptions.light,
             ...localeLabels.configurationOptions.medium,
@@ -227,7 +252,7 @@ export function detectExperienceFromSnapshot(snapshot) {
     }
     const workScore = workComposer.length * 4
         + (workSurfaceSelected ? 10 : 0)
-        + (workAxisCount >= 2 ? 4 : 0)
+        + (workAxes ? 6 : 0)
         // The active Work task drops the Chat/Work radio and keeps the shared
         // "Chat with ChatGPT" textbox name. Its compound model + effort opener is
         // therefore strong enough to disambiguate that continuation surface.
@@ -288,7 +313,19 @@ export async function readSurfaceSnapshot(page) {
         const normalize = (value) => value.replace(/\s+/g, " ").trim();
         const normalizeComparable = (value) => normalize(value).toLocaleLowerCase();
         const wantedSurfaceLabels = new Set(surfaceOptionLabels.map(normalizeComparable));
-        const composerRoots = Array.from(document.querySelectorAll("main form, main [data-testid*='composer' i], main [class*='composer' i]"));
+        const composerRoots = Array.from(document.querySelectorAll("main form, main [data-testid*='composer' i], main [class*='composer' i]")).filter(visible);
+        const main = document.querySelector("main");
+        const overlayRoots = Array.from(document.querySelectorAll("[role='menu'], [role='listbox'], [data-radix-popper-content-wrapper], [data-radix-menu-content]")).filter(visible);
+        const composerControlRoots = composerRoots.length > 0 ? composerRoots : main === null ? [] : [main];
+        const effectiveControlRoots = Array.from(new Set([...composerControlRoots, ...overlayRoots]));
+        // Reject oversized captures before expanding overlapping subtrees. A
+        // truncated prefix could omit contradictory surface evidence.
+        if (effectiveControlRoots.length > 32) {
+            return {
+                composerLabels: [], mainControls: [], composerControls: [], controlGroups: [],
+                mainText: "", selectedSurfaceLabels: [], rootBudgetExceeded: true
+            };
+        }
         const composerNodes = composerRoots.flatMap(root => [
             root,
             ...Array.from(root.querySelectorAll("textarea, [contenteditable='true'], [role='textbox'], input"))
@@ -299,18 +336,29 @@ export async function readSurfaceSnapshot(page) {
             .map(normalize)
             .filter(Boolean)))
             .slice(0, 16);
-        const main = document.querySelector("main");
-        const overlayRoots = Array.from(document.querySelectorAll("[role='menu'], [role='listbox'], [data-radix-popper-content-wrapper], [data-radix-menu-content]")).filter(visible);
-        const controlRoots = Array.from(new Set([...composerRoots, ...overlayRoots]));
-        const effectiveControlRoots = controlRoots.length > 0
-            ? controlRoots
-            : main === null ? [] : [main];
-        const mainControls = Array.from(new Set(effectiveControlRoots.flatMap(root => Array.from(root.querySelectorAll("button, [role='button'], [role='menuitem'], [role='menuitemradio'], [role='option']")))
+        const overlayRootSet = new Set(overlayRoots);
+        const owningOverlay = (node) => {
+            let current = node;
+            while (current !== null) {
+                if (overlayRootSet.has(current))
+                    return current;
+                current = current.parentElement;
+            }
+            return undefined;
+        };
+        const controlsFor = (root) => Array.from(new Set(Array.from(root.querySelectorAll("button, [role='button'], [role='menuitem'], [role='menuitemradio'], [role='option']"))
             .filter(visible)
+            // An ancestor composer/popover must not aggregate independent or
+            // nested menus. Each overlay contributes only its own nearest scope.
+            .filter(node => owningOverlay(node) === (overlayRootSet.has(root) ? root : undefined))
             .map(labelFor)
             .map(normalize)
-            .filter(Boolean)))
-            .slice(0, 120);
+            .filter(Boolean))).slice(0, 120);
+        const rootGroups = new Map(effectiveControlRoots.map(root => [root, controlsFor(root)]));
+        const controlGroups = [...rootGroups.values()];
+        const mainControls = Array.from(new Set(controlGroups.flat())).slice(0, 120);
+        const composerControls = Array.from(new Set(composerControlRoots
+            .flatMap(root => rootGroups.get(root) ?? []))).slice(0, 120);
         const surfaceTextNodes = main === null ? [] : Array.from(main.querySelectorAll("h1, h2, h3, form, [data-testid*='composer' i], [class*='composer' i]"))
             .filter(visible)
             .slice(0, 32);
@@ -321,7 +369,7 @@ export async function readSurfaceSnapshot(page) {
             .map(normalize)
             .filter(label => wantedSurfaceLabels.has(normalizeComparable(label)))))
             .slice(0, 4);
-        return { composerLabels, mainControls, mainText, selectedSurfaceLabels };
+        return { composerLabels, mainControls, composerControls, controlGroups, mainText, selectedSurfaceLabels };
     }, [
         ...localeLabels.experienceOptions.chat,
         ...localeLabels.experienceOptions.work,

package/dist/src/commands/files.js

@@ -4,7 +4,7 @@ import { createHash } from "node:crypto";
 import path from "node:path";
 import { downloadLatestArtifact, locatorCountWithTimeout } from "./artifacts.js";
 import { ACTIVE_COMPOSER_FILE_INPUT_CLICK_EXPRESSION } from "../browser/active-composer-file-input.js";
-import { waitForDownloadFromClick } from "../browser/downloads.js";
+import { isTerminalDownloadCode, isTerminalDownloadError, waitForDownloadFromClick } from "../browser/downloads.js";
 import { nodeErrorCode, resultError, resultOk } from "../errors.js";
 import { addFilesButton, cssSelectors, requiredLocator } from "../dom/selectors.js";
 import { escapeRegExp, localeLabels } from "../dom/locale-labels.js";
@@ -987,7 +987,7 @@ export async function downloadLatestFile(env, args) {
         }
         if (count === 0) {
             const artifactDownload = await downloadLatestArtifact(env, args);
-            if (artifactDownload.ok) {
+            if (artifactDownload.ok || isTerminalDownloadCode(artifactDownload.blocker?.code)) {
                 return artifactDownload;
             }
             return {
@@ -1008,7 +1008,7 @@ export async function downloadLatestFile(env, args) {
         return resultOk(downloaded, await contextFromPage(page));
     }
     catch (error) {
-        return resultError(error instanceof Error ? error : new Error(String(error)), await contextFromPage(page));
+        return resultError(error instanceof Error ? error : new Error(String(error)), await contextFromPage(page, {}, { minimal: isTerminalDownloadError(error) }));
     }
 }
 async function tryGeneratedFilePreviewDownload(page, args) {
@@ -1039,8 +1039,12 @@ async function tryGeneratedFilePreviewDownload(page, args) {
         await affordance.click({ timeoutMs: localGuardTimeout(timeoutMs, 10000) });
         const labelledPreview = requiredLocator(page, `section[aria-label="${escapeCssAttribute(selected.filename)}"]`);
         const workbookPreviews = requiredLocator(page, "section[data-testid^='popcorn-']");
-        const workbookPreview = workbookPreviews.filter?.({ hasText: selected.filename }) ?? workbookPreviews;
-        const download = await waitForPreviewDownloadControl(page, [labelledPreview, workbookPreview], timeoutMs);
+        // A provider without locator filtering cannot tie a workbook preview to
+        // the selected filename. Keep the explicitly labelled preview usable.
+        const workbookPreview = typeof workbookPreviews.filter === "function"
+            ? workbookPreviews.filter({ hasText: selected.filename })
+            : undefined;
+        const download = await waitForPreviewDownloadControl(page, workbookPreview === undefined ? [labelledPreview] : [labelledPreview, workbookPreview], timeoutMs);
         if (download === undefined) {
             throw new Error(`The artifact preview for ${selected.filename} did not expose a visible Download control.`);
         }
@@ -1048,7 +1052,7 @@ async function tryGeneratedFilePreviewDownload(page, args) {
         return resultOk(downloaded, await contextFromPage(page));
     }
     catch (error) {
-        return resultError(error instanceof Error ? error : new Error(String(error)), await contextFromPage(page));
+        return resultError(error instanceof Error ? error : new Error(String(error)), await contextFromPage(page, {}, { minimal: isTerminalDownloadError(error) }));
     }
 }
 async function inspectGeneratedFileAffordances(page, timeoutMs) {

package/dist/src/commands/modes.js

@@ -6,6 +6,7 @@ import { normalizeLabel, normalizeWhitespace } from "../dom/visible-text.js";
 import { contextFromPage } from "./context.js";
 import { discoverPowerSlider, observedPowerSlider, resolvePowerTarget } from "./power-discovery.js";
 import { ensurePage } from "./session.js";
+import { closeChatPopover, readChatPopover, selectChatPopoverEffort, selectChatPopoverModel, setChatPopoverView } from "./chat-popover.js";
 const DEFAULT_MODE_EFFORT = "Thinking";
 const CURRENT_MODE_LABELS = dedupeLabels([
     ...localeLabels.modeLabels,
@@ -14,7 +15,7 @@ const CURRENT_MODE_LABELS = dedupeLabels([
 ]);
 const MODE_OPENER_LABELS = [...CURRENT_MODE_LABELS.filter(label => label !== "Pro"), ...localeLabels.modeOpenerExtra];
 const MODEL_VERSION_FAMILY_PATTERN = /^gpt[\s-]/i;
-const MODEL_VERSION_LABEL_PATTERN = /^(?:o\d+|\d+(?:\.\d+)?)$/i;
+const MODEL_VERSION_LABEL_PATTERN = /^(?:gpt[\s-].+|o\d+|\d+(?:\.\d+)?|latest)$/i;
 const CANONICAL_INTELLIGENCE_ORDER = new Map([
     ["instant", 0],
     ["medium", 1],
@@ -53,6 +54,7 @@ export async function setMode(env, args) {
     }
     const page = env.page;
     try {
+        const initialPopover = (await readChatPopover(page)).snapshot;
         const requested = requestedModeSelections(args);
         const requestedVersion = requestedModelVersion(args);
         const requestedForOpening = requestedVersion === undefined ? requested : [...requested, requestedModeSelection(requestedVersion)];
@@ -64,6 +66,34 @@ export async function setMode(env, args) {
             return selectorDrift(page, "No unique ChatGPT mode menu opener was found.");
         }
         await page.waitForTimeout?.(250);
+        const scopedPopover = (await readChatPopover(page)).snapshot;
+        if (scopedPopover !== undefined) {
+            try {
+                const selected = [];
+                const candidates = [];
+                if (requestedVersion !== undefined) {
+                    const model = await selectChatPopoverModel(page, [requestedVersion]);
+                    if (model === undefined)
+                        return selectorDrift(page, `Model version "${requestedVersion}" did not verify in the owned Chat model view.`);
+                    selected.push(model);
+                    candidates.push(model);
+                }
+                for (const request of requested) {
+                    const effort = await selectChatPopoverEffort(page, request.labels);
+                    if (effort === undefined)
+                        return selectorDrift(page, `Mode option "${request.requested}" did not verify in the owned Chat effort control.`);
+                    selected.push(effort);
+                    candidates.push(effort);
+                }
+                return resultOk({ selected, candidates }, await contextFromPage(page));
+            }
+            finally {
+                if (initialPopover !== undefined)
+                    await setChatPopoverView(page, initialPopover.view);
+                else
+                    await closeChatPopover(page);
+            }
+        }
         let candidates = await enumerateVisibleMenuItems(page);
         const observedCandidates = [...candidates];
         const selected = [];
@@ -288,6 +318,8 @@ async function waitForModeMenu(page, requested, timeoutMs) {
     const deadline = Date.now() + timeoutMs;
     let modeButtons = [];
     do {
+        if ((await readChatPopover(page)).snapshot !== undefined)
+            return { opened: true, alreadySelected: [], modeButtons };
         modeButtons = await visibleModeButtonLabelList(page);
         const alreadySelected = findAlreadySelectedModes(modeButtons, requested);
         if (alreadySelected.length === requested.length) {
@@ -668,8 +700,8 @@ async function selectModelVersion(page, requestedVersion, currentCandidates, tim
         : { candidates: candidates.map(candidate => candidate.label) };
 }
 function isModelVersionSubmenuOpener(item) {
-    return item.hasPopup === true
-        || (item.role !== "menuitemradio" && MODEL_VERSION_FAMILY_PATTERN.test(item.label));
+    return item.role !== "menuitemradio"
+        && (MODEL_VERSION_FAMILY_PATTERN.test(item.label) || item.ariaLabel === "Select model");
 }
 async function clickResolvedMenuItem(page, item) {
     if (item.testId !== undefined && await clickIfUniqueMenuControl(page.locator?.(`[data-testid="${escapeAttributeValue(item.testId)}"]`), item)) {
@@ -681,7 +713,7 @@ async function clickResolvedMenuItem(page, item) {
     return clickMenuItem(page, item);
 }
 async function openModelVersionSubmenu(page, candidates) {
-    const submenuOpeners = candidates.filter(item => item.hasPopup === true || MODEL_VERSION_FAMILY_PATTERN.test(item.label));
+    const submenuOpeners = candidates.filter(isModelVersionSubmenuOpener);
     if (submenuOpeners.length === 0) {
         return false;
     }