@steipete/oracle 0.14.0 → 0.15.0

package/dist/src/browser/actions/thinkingTime.js

@@ -1,6 +1,10 @@
 import { MENU_CONTAINER_SELECTOR, MENU_ITEM_SELECTOR, MODEL_BUTTON_SELECTOR, } from "../constants.js";
 import { logDomFailure } from "../domDebug.js";
 import { buildClickDispatcher } from "./domEvents.js";
+const BROWSER_THINKING_LOG_PREFIX = "[browser] Thinking time:";
+function formatBrowserThinkingLog(message) {
+    return `${BROWSER_THINKING_LOG_PREFIX} ${message.replace(/^Thinking time:\s*/, "")}`;
+}
 /**
  * Surfaces the model-picker snapshot captured alongside a failed detection.
  *
@@ -26,13 +30,14 @@ export async function ensureThinkingTime(Runtime, level, logger, desiredModel) {
     const result = await evaluateThinkingTimeSelection(Runtime, level, desiredModel);
     const capitalizedLevel = level.charAt(0).toUpperCase() + level.slice(1);
     const targetModelKind = inferThinkingTargetModelKind(desiredModel);
-    const strictProEffort = targetModelKind === "pro" && level === "extended";
+    const observedModelKind = result && "modelKind" in result ? result.modelKind : null;
+    const strictProEffort = (targetModelKind === "pro" || observedModelKind === "pro") && level === "extended";
     switch (result?.status) {
         case "already-selected":
-            logger(`Thinking time: ${result.label ?? capitalizedLevel} (already selected)`);
+            logger(formatBrowserThinkingLog(`${result.label ?? capitalizedLevel} (already selected)`));
             return;
         case "switched":
-            logger(`Thinking time: ${result.label ?? capitalizedLevel}`);
+            logger(formatBrowserThinkingLog(result.label ?? capitalizedLevel));
             return;
         case "chip-not-found":
         case "menu-not-found":
@@ -50,7 +55,7 @@ export async function ensureThinkingTime(Runtime, level, logger, desiredModel) {
             if (strictProEffort) {
                 throw new Error(`${message}; refusing to submit without confirmed Pro Extended.`);
             }
-            logger(`${message}; continuing with ChatGPT default.`);
+            logger(formatBrowserThinkingLog(`${message}; continuing with ChatGPT default.`));
             return;
         }
         default: {
@@ -59,7 +64,7 @@ export async function ensureThinkingTime(Runtime, level, logger, desiredModel) {
             if (strictProEffort) {
                 throw new Error(`Thinking time: unknown outcome selecting ${capitalizedLevel}; refusing to submit without confirmed Pro Extended.`);
             }
-            logger(`Thinking time: unknown outcome selecting ${capitalizedLevel}; continuing with ChatGPT default.`);
+            logger(formatBrowserThinkingLog(`unknown outcome selecting ${capitalizedLevel}; continuing with ChatGPT default.`));
             return;
         }
     }
@@ -75,10 +80,10 @@ export async function ensureThinkingTimeIfAvailable(Runtime, level, logger, desi
         const capitalizedLevel = level.charAt(0).toUpperCase() + level.slice(1);
         switch (result?.status) {
             case "already-selected":
-                logger(`Thinking time: ${result.label ?? capitalizedLevel} (already selected)`);
+                logger(formatBrowserThinkingLog(`${result.label ?? capitalizedLevel} (already selected)`));
                 return true;
             case "switched":
-                logger(`Thinking time: ${result.label ?? capitalizedLevel}`);
+                logger(formatBrowserThinkingLog(result.label ?? capitalizedLevel));
                 return true;
             case "chip-not-found":
             case "menu-not-found":
@@ -86,12 +91,12 @@ export async function ensureThinkingTimeIfAvailable(Runtime, level, logger, desi
             case "selection-unverified":
             case "model-kind-not-found":
                 if (logger.verbose) {
-                    logger(`Thinking time: ${result.status.replaceAll("-", " ")}; continuing with default.`);
+                    logger(formatBrowserThinkingLog(`${result.status.replaceAll("-", " ")}; continuing with default.`));
                 }
                 return false;
             default:
                 if (logger.verbose) {
-                    logger("Thinking time: unknown outcome; continuing with default.");
+                    logger(formatBrowserThinkingLog("unknown outcome; continuing with default."));
                 }
                 return false;
         }
@@ -99,7 +104,7 @@ export async function ensureThinkingTimeIfAvailable(Runtime, level, logger, desi
     catch (error) {
         const message = error instanceof Error ? error.message : String(error);
         if (logger.verbose) {
-            logger(`Thinking time selection failed (${message}); continuing with default.`);
+            logger(formatBrowserThinkingLog(`selection failed (${message}); continuing with default.`));
             await logDomFailure(Runtime, logger, "thinking-time");
         }
         return false;
@@ -130,10 +135,10 @@ function buildThinkingTimeExpression(level, desiredModel) {
 
     // Bilingual matchers: English level token + observed Chinese variants.
     const LEVEL_TOKENS = {
-      light: ['light', '轻'],
-      standard: ['standard', '标准'],
-      extended: ['extended', '扩展', '深度', '加强'],
-      heavy: ['heavy', '重度', '加重', '高'],
+      light: ['light', 'instant', '轻'],
+      standard: ['standard', 'medium', '标准'],
+      extended: ['extended', 'high', '扩展', '深度', '加强'],
+      heavy: ['heavy', 'extra high', '重度', '加重', '高'],
     };
     const targetTokens = LEVEL_TOKENS[TARGET_LEVEL] || [TARGET_LEVEL];
 
@@ -152,11 +157,36 @@ function buildThinkingTimeExpression(level, desiredModel) {
       .replace(/[^a-z0-9\\u4e00-\\u9fa5]+/g, ' ')
       .replace(/\\s+/g, ' ')
       .trim();
+    const hasToken = (text, token) => normalize(text).split(' ').includes(token);
     const matchesLevel = (text) => {
       const t = normalize(text);
-      return targetTokens.some((tok) => t.includes(String(tok).toLowerCase()));
+      if (!t) return false;
+      return targetTokens.some((tok) => {
+        const token = normalize(tok);
+        if (!token) return false;
+        if (token === 'high') return hasToken(t, 'high') && !hasToken(t, 'extra');
+        if (token === 'extra high') return hasToken(t, 'extra') && hasToken(t, 'high');
+        return t === token || hasToken(t, token) || t.includes(token);
+      });
+    };
+    const matchesAnyEffortLevel = (text) => {
+      const normalizedText = normalize(text);
+      if (!normalizedText) return false;
+      for (const tokens of Object.values(LEVEL_TOKENS)) {
+        for (const rawToken of tokens) {
+          const token = normalize(rawToken);
+          if (!token) continue;
+          if (token.includes(' ')) {
+            if (token.split(' ').every((part) => hasToken(normalizedText, part))) return true;
+          } else if (/^[a-z0-9]+$/.test(token)) {
+            if (hasToken(normalizedText, token)) return true;
+          } else if (normalizedText.includes(token)) {
+            return true;
+          }
+        }
+      }
+      return false;
     };
-    const hasToken = (text, token) => normalize(text).split(' ').includes(token);
     const optionIsSelected = (node) => {
       if (!(node instanceof HTMLElement)) return false;
       const ariaChecked = node.getAttribute('aria-checked');
@@ -180,9 +210,28 @@ function buildThinkingTimeExpression(level, desiredModel) {
         );
       } catch {}
     };
+    const dispatchHoverSequence = (target) => {
+      if (!target || !(target instanceof EventTarget)) return false;
+      const types = ['pointerover', 'pointerenter', 'mouseover', 'mouseenter', 'pointermove', 'mousemove'];
+      for (const type of types) {
+        try {
+          const common = { bubbles: true, cancelable: true, view: window };
+          const event =
+            type.startsWith('pointer') && 'PointerEvent' in window
+              ? new PointerEvent(type, { ...common, pointerId: 1, pointerType: 'mouse' })
+              : new MouseEvent(type, common);
+          target.dispatchEvent(event);
+        } catch {}
+      }
+      try {
+        target.focus?.();
+      } catch {}
+      return true;
+    };
 
     const TRAILING_SELECTOR = '[data-model-picker-thinking-effort-action="true"]';
     const INTELLIGENCE_MENU_SELECTOR = '[data-testid="composer-intelligence-picker-content"]';
+    const PRO_EFFORT_TRIGGER_SELECTOR = '[data-testid="composer-intelligence-pro-thinking-effort-trigger"]';
 
     const findModelButton = () => document.querySelector(MODEL_BUTTON_SELECTOR);
     const findTrailingButtons = () => Array.from(document.querySelectorAll(TRAILING_SELECTOR));
@@ -270,13 +319,57 @@ function buildThinkingTimeExpression(level, desiredModel) {
         return { error: redactDiagnosticText(err && err.message ? err.message : err) };
       }
     };
+    const modelKindFromNode = (button) => {
+      const label = normalize(
+        (button?.textContent ?? '') + ' ' + (button?.getAttribute?.('aria-label') ?? ''),
+      );
+      if (hasToken(label, 'pro')) return 'pro';
+      if (hasToken(label, 'thinking')) return 'thinking';
+      if (hasToken(label, 'instant')) return 'instant';
+      return null;
+    };
+    const currentModelKind = () => modelKindFromNode(findModelButton());
+    const effectiveTargetModelKind = () => TARGET_MODEL_KIND || currentModelKind();
+    const isIntelligenceEffortMenu = (menu) => {
+      if (menu?.getAttribute?.('data-testid') === 'composer-intelligence-picker-content') {
+        return true;
+      }
+      const label = menu?.querySelector?.('.__menu-label, [class*="menu-label"]');
+      return normalize(label?.textContent ?? '').includes('intelligence');
+    };
     const failure = (status, extra = {}) => ({
       status,
+      modelKind: effectiveTargetModelKind(),
       ...extra,
       diagnostic: collectPickerDiagnostic(),
     });
-    const findOptionInMenu = (menu) => {
-      for (const item of menu.querySelectorAll(MENU_ITEM_SELECTOR)) {
+    const findOptionInMenu = (menu, modelKindOverride = null) => {
+      const items = Array.from(menu.querySelectorAll(MENU_ITEM_SELECTOR));
+      const modelKind = modelKindOverride || effectiveTargetModelKind();
+      if (modelKind === 'pro') {
+        for (const item of items) {
+          const itemText = normalize(
+            (item.textContent ?? '') + ' ' + (item.getAttribute?.('aria-label') ?? ''),
+          );
+          if (
+            hasToken(itemText, 'pro') &&
+            (matchesLevel(item.textContent ?? '') ||
+              matchesLevel(item.getAttribute?.('aria-label') ?? ''))
+          ) {
+            return item;
+          }
+        }
+        if (isIntelligenceEffortMenu(menu)) {
+          return null;
+        }
+      }
+      for (const item of items) {
+        const itemText = normalize(
+          (item.textContent ?? '') + ' ' + (item.getAttribute?.('aria-label') ?? ''),
+        );
+        if (modelKind && modelKind !== 'pro' && hasToken(itemText, 'pro')) {
+          continue;
+        }
         if (
           matchesLevel(item.textContent ?? '') ||
           matchesLevel(item.getAttribute?.('aria-label') ?? '')
@@ -296,14 +389,21 @@ function buildThinkingTimeExpression(level, desiredModel) {
     };
     const isEffortMenu = (menu) => {
       if (!isVisible(menu)) return false;
+      if (menu.getAttribute?.('data-testid') === 'composer-intelligence-picker-content') return true;
       const label = menu.querySelector?.('.__menu-label, [class*="menu-label"]');
       const labelText = normalize(label?.textContent ?? '');
       return (
+        labelText.includes('intelligence') ||
         labelText.includes('thinking time') ||
         labelText.includes('thinking effort') ||
         countEffortLevels(menu) >= 2
       );
     };
+    const isProEffortMenu = (menu) => {
+      if (!isVisible(menu)) return false;
+      const text = normalize(menu?.textContent ?? '');
+      return text.includes('pro standard') && text.includes('pro extended');
+    };
     const controlledMenu = (trigger) => {
       const id = trigger?.getAttribute?.('aria-controls');
       if (!id) return null;
@@ -318,9 +418,69 @@ function buildThinkingTimeExpression(level, desiredModel) {
       }
       return null;
     };
-    const selectAndVerify = async (trigger, findOption) => {
+    const controlledProEffortMenu = (trigger) => {
+      const id = trigger?.getAttribute?.('aria-controls');
+      if (!id) return null;
+      const menu = document.getElementById?.(id);
+      return isProEffortMenu(menu) ? menu : null;
+    };
+    const findVisibleProEffortMenu = (trigger) => {
+      const controlled = controlledProEffortMenu(trigger);
+      if (controlled) return controlled;
+      for (const menu of document.querySelectorAll(MENU_CONTAINER_SELECTOR)) {
+        if (isProEffortMenu(menu)) return menu;
+      }
+      return null;
+    };
+    const matchesProEffortLevel = (node) => {
+      const text = normalize(
+        (node?.textContent ?? '') + ' ' + (node?.getAttribute?.('aria-label') ?? ''),
+      );
+      if (TARGET_LEVEL === 'standard') {
+        return text.includes('pro') && text.includes('standard');
+      }
+      if (TARGET_LEVEL === 'extended') {
+        return text.includes('pro') && text.includes('extended');
+      }
+      return false;
+    };
+    const findProEffortOptionInMenu = (menu) => {
+      for (const item of menu.querySelectorAll(MENU_ITEM_SELECTOR)) {
+        if (matchesProEffortLevel(item)) return item;
+      }
+      return null;
+    };
+    const currentProEffortPillMatchesTarget = (trigger, modelKindOverride = null) => {
+      const button = trigger?.matches?.('button.__composer-pill') ? trigger : findModelButton();
+      if ((modelKindOverride || TARGET_MODEL_KIND || modelKindFromNode(button)) !== 'pro') {
+        return false;
+      }
+      const label = normalize(button?.textContent ?? '');
+      if (TARGET_LEVEL === 'standard') {
+        return hasToken(label, 'pro') && !hasToken(label, 'extended');
+      }
+      if (TARGET_LEVEL === 'extended') {
+        return hasToken(label, 'pro') && hasToken(label, 'extended');
+      }
+      return false;
+    };
+    const currentEffortPillMatchesTarget = (trigger, modelKindOverride = null) => {
+      if (currentProEffortPillMatchesTarget(trigger, modelKindOverride)) return true;
+      const button = trigger?.matches?.('button.__composer-pill') ? trigger : findModelButton();
+      if ((modelKindOverride || TARGET_MODEL_KIND || modelKindFromNode(button)) === 'pro') {
+        return false;
+      }
+      const label = (button?.textContent ?? '') + ' ' + (button?.getAttribute?.('aria-label') ?? '');
+      return matchesLevel(label);
+    };
+    const selectAndVerify = async (trigger, findOption, modelKindOverride = null) => {
       const option = findOption();
-      if (!option) return failure('option-not-found');
+      const triggerModelKind =
+        modelKindOverride ||
+        TARGET_MODEL_KIND ||
+        modelKindFromNode(trigger) ||
+        effectiveTargetModelKind();
+      if (!option) return failure('option-not-found', { modelKind: triggerModelKind });
       const label = option.textContent?.trim?.() || null;
       if (optionIsSelected(option)) {
         closeOpenMenus();
@@ -334,6 +494,10 @@ function buildThinkingTimeExpression(level, desiredModel) {
         closeOpenMenus();
         return { status: 'switched', label: refreshed.textContent?.trim?.() || label };
       }
+      if (currentEffortPillMatchesTarget(trigger, triggerModelKind)) {
+        closeOpenMenus();
+        return { status: 'switched', label };
+      }
 
       if (!refreshed && trigger && trigger.getAttribute?.('aria-expanded') !== 'true') {
         dispatchClickSequence(trigger);
@@ -346,29 +510,56 @@ function buildThinkingTimeExpression(level, desiredModel) {
           closeOpenMenus();
           return { status: 'switched', label: selected.textContent?.trim?.() || label };
         }
+        if (currentEffortPillMatchesTarget(trigger, triggerModelKind)) {
+          closeOpenMenus();
+          return { status: 'switched', label };
+        }
         await sleep(100);
       }
-      const result = failure('selection-unverified');
+      const result = failure('selection-unverified', { modelKind: triggerModelKind });
       closeOpenMenus();
       return result;
     };
+    const selectProEffortFromSubmenu = async () => {
+      if (TARGET_MODEL_KIND !== 'pro' || (TARGET_LEVEL !== 'standard' && TARGET_LEVEL !== 'extended')) {
+        return null;
+      }
+      const trigger = document.querySelector(PRO_EFFORT_TRIGGER_SELECTOR);
+      if (!trigger) {
+        return null;
+      }
+      dispatchHoverSequence(trigger);
+      if (trigger.getAttribute?.('aria-expanded') !== 'true') {
+        dispatchClickSequence(trigger);
+      }
+      const deadline = performance.now() + MAX_WAIT_MS;
+      while (performance.now() < deadline) {
+        const menu = findVisibleProEffortMenu(trigger);
+        if (menu) {
+          return selectAndVerify(trigger, () => {
+            const currentMenu = findVisibleProEffortMenu(trigger);
+            return currentMenu ? findProEffortOptionInMenu(currentMenu) : null;
+          });
+        }
+        await sleep(100);
+      }
+      return null;
+    };
 
     // Current ChatGPT exposes a standalone Pro or Thinking composer pill whose
     // controlled menu contains the effort levels. Prefer this ownership boundary
     // before probing older model-picker layouts.
     const COMPOSER_EFFORT_PILL_SELECTORS = [
-      'form button.__composer-pill[aria-haspopup="menu"]',
-      '[data-testid="composer-footer-actions"] button[aria-haspopup="menu"]',
-      '.__composer-pill-composite button[aria-haspopup="menu"]',
+      'form button.__composer-pill',
+      '[data-testid="composer-footer-actions"] button.__composer-pill',
+      '.__composer-pill-composite button.__composer-pill',
     ];
     const findComposerEffortPill = () => {
-      const candidates = [];
       const seen = new Set();
       for (const selector of COMPOSER_EFFORT_PILL_SELECTORS) {
         for (const button of document.querySelectorAll(selector)) {
           if (seen.has(button) || !isVisible(button)) continue;
           seen.add(button);
-          if (button.getAttribute?.('aria-haspopup') !== 'menu') continue;
           if (button.getAttribute?.('data-testid') === 'model-switcher-dropdown-button') continue;
           const label = normalize(
             (button.getAttribute?.('aria-label') ?? '') + ' ' +
@@ -378,21 +569,72 @@ function buildThinkingTimeExpression(level, desiredModel) {
           if (
             (TARGET_MODEL_KIND === 'pro' && hasToken(label, 'pro') && !hasToken(label, 'thinking')) ||
             (TARGET_MODEL_KIND === 'thinking' && hasToken(label, 'thinking') && !hasToken(label, 'pro')) ||
-            (!TARGET_MODEL_KIND && hasToken(label, 'thinking'))
+            (!TARGET_MODEL_KIND && hasToken(label, 'thinking')) ||
+            (button.matches?.('button.__composer-pill') && matchesAnyEffortLevel(label))
           ) {
             return button;
           }
-          candidates.push(button);
         }
       }
-      const composerPills = candidates.filter((button) =>
-        button.matches?.('button.__composer-pill'),
+      return null;
+    };
+    let composerEffortPill = findComposerEffortPill();
+    let modelBtn = findModelButton();
+    const modelKindFromLegacyTrailing = (trailing) => {
+      const row = trailing.closest?.(
+        '[role="menuitem"], [role="menuitemradio"], [data-radix-collection-item]',
+      );
+      const idText = normalize(
+        (row?.getAttribute?.('data-testid') ?? '') + ' ' +
+        (trailing.getAttribute?.('data-testid') ?? '')
       );
-      return composerPills.length === 1 ? composerPills[0] : null;
+      if (!idText.includes('model switcher')) return null;
+      const modelPart = normalize(idText.replace(/\\bthinking effort\\b.*$/, ''));
+      if (hasToken(modelPart, 'pro')) return 'pro';
+      if (hasToken(modelPart, 'thinking')) return 'thinking';
+      if (hasToken(modelPart, 'instant')) return 'instant';
+      return null;
     };
-
-    const composerEffortPill = findComposerEffortPill();
+    const legacyEffortOwnerIsReady = () => {
+      if (
+        TARGET_MODEL_KIND === 'pro' &&
+        TARGET_LEVEL === 'extended' &&
+        isVisible(document.querySelector(INTELLIGENCE_MENU_SELECTOR))
+      ) {
+        return true;
+      }
+      const expectedKind = TARGET_MODEL_KIND || modelKindFromNode(modelBtn);
+      return Boolean(
+        expectedKind &&
+        findTrailingButtons().some(
+          (button) => isVisible(button) && modelKindFromLegacyTrailing(button) === expectedKind,
+        ),
+      );
+    };
+    let attemptedModelButton =
+      modelBtn?.getAttribute?.('aria-expanded') === 'true' ? modelBtn : null;
+    const effortOwnerDeadline = performance.now() + MAX_WAIT_MS;
+    while (!composerEffortPill && performance.now() < effortOwnerDeadline) {
+      if (
+        modelBtn &&
+        attemptedModelButton !== modelBtn &&
+        modelBtn.getAttribute?.('aria-expanded') !== 'true'
+      ) {
+        dispatchClickSequence(modelBtn);
+        attemptedModelButton = modelBtn;
+        await sleep(INITIAL_WAIT_MS);
+      }
+      if (modelBtn && legacyEffortOwnerIsReady()) break;
+      await sleep(100);
+      composerEffortPill = findComposerEffortPill();
+      modelBtn = findModelButton();
+      if (modelBtn?.getAttribute?.('aria-expanded') === 'true') {
+        attemptedModelButton = modelBtn;
+      }
+    }
     if (composerEffortPill) {
+      if (attemptedModelButton && attemptedModelButton !== composerEffortPill) closeOpenMenus();
+      const composerModelKind = TARGET_MODEL_KIND || modelKindFromNode(composerEffortPill);
       if (composerEffortPill.getAttribute?.('aria-expanded') !== 'true') {
         dispatchClickSequence(composerEffortPill);
         await sleep(INITIAL_WAIT_MS);
@@ -401,14 +643,24 @@ function buildThinkingTimeExpression(level, desiredModel) {
       while (performance.now() < deadline) {
         const menu = findVisibleEffortMenu(composerEffortPill);
         if (menu) {
-          return selectAndVerify(composerEffortPill, () => {
-            const currentMenu = findVisibleEffortMenu(composerEffortPill);
-            return currentMenu ? findOptionInMenu(currentMenu) : null;
-          });
+          const proEffortResult = await selectProEffortFromSubmenu();
+          if (proEffortResult) {
+            return proEffortResult;
+          }
+          return selectAndVerify(
+            composerEffortPill,
+            () => {
+              const currentMenu = findVisibleEffortMenu(composerEffortPill);
+              return currentMenu ? findOptionInMenu(currentMenu, composerModelKind) : null;
+            },
+            composerModelKind,
+          );
         }
         await sleep(100);
       }
-      const result = failure('menu-not-found');
+      const result = failure('menu-not-found', {
+        modelKind: composerModelKind,
+      });
       closeOpenMenus();
       return result;
     }
@@ -447,22 +699,7 @@ function buildThinkingTimeExpression(level, desiredModel) {
         (trailing.getAttribute?.('data-testid') ?? '')
       );
     };
-    const testIdTextForTrailing = (trailing) => {
-      const row = rowForTrailing(trailing) || findEffortRow(trailing);
-      return normalize(
-        (row?.getAttribute?.('data-testid') ?? '') + ' ' +
-        (trailing.getAttribute?.('data-testid') ?? '')
-      );
-    };
-    const modelKindFromTrailing = (trailing) => {
-      const idText = testIdTextForTrailing(trailing);
-      if (!idText.includes('model switcher')) return null;
-      const modelPart = normalize(idText.replace(/\\bthinking effort\\b.*$/, ''));
-      if (hasToken(modelPart, 'pro')) return 'pro';
-      if (hasToken(modelPart, 'thinking')) return 'thinking';
-      if (hasToken(modelPart, 'instant')) return 'instant';
-      return null;
-    };
+    const modelKindFromTrailing = modelKindFromLegacyTrailing;
     const trailingMatchesTargetModelKind = (trailing) => {
       if (!TARGET_MODEL_KIND) return false;
       const idKind = modelKindFromTrailing(trailing);
@@ -499,12 +736,19 @@ function buildThinkingTimeExpression(level, desiredModel) {
       return null;
     };
 
-    const modelBtn = findModelButton();
+    const modelButtonDeadline = performance.now() + MAX_WAIT_MS;
+    while (!modelBtn && performance.now() < modelButtonDeadline) {
+      await sleep(100);
+      modelBtn = findModelButton();
+    }
     if (!modelBtn) {
       return failure('chip-not-found');
     }
     // Open model menu (idempotent — leaves it open if already open).
-    if (modelBtn.getAttribute('aria-expanded') !== 'true') {
+    if (
+      modelBtn.getAttribute('aria-expanded') !== 'true' &&
+      !legacyEffortOwnerIsReady()
+    ) {
       dispatchClickSequence(modelBtn);
       await sleep(INITIAL_WAIT_MS);
     }
@@ -512,15 +756,9 @@ function buildThinkingTimeExpression(level, desiredModel) {
     // ---------- COMPATIBILITY UI: unified "Intelligence" effort picker ----------
     // One observed ChatGPT layout replaced the per-model trailing buttons with a single
     // "Intelligence" menu ([data-testid="composer-intelligence-picker-content"]),
-    // whose role="menuitemradio" rows are the effort tiers. For the Pro model the
-    // combined "Pro Extended" row carries aria-checked when active, and Pro
-    // sub-options live behind
-    // [data-testid="composer-intelligence-pro-thinking-effort-trigger"]. We
-    // confirm Pro Extended by the radio's checked state (real proof), never by
-    // the composer-pill label. The new non-pro tiers (Instant/Medium/High/Extra
-    // High) don't map cleanly onto light/standard/extended/heavy, so we only
-    // drive this picker for the strict Pro Extended target and let other levels
-    // fall through to the legacy paths below.
+    // whose role="menuitemradio" rows are the effort tiers. We verify the checked
+    // radio instead of trusting the composer-pill label; non-Pro targets also
+    // explicitly skip Pro rows before matching effort labels.
     if (TARGET_MODEL_KIND === 'pro' && TARGET_LEVEL === 'extended') {
       const matchesProExtended = (node) => {
         const text = normalize(

package/dist/src/browser/artifacts.js

@@ -1,6 +1,7 @@
 import fs from "node:fs/promises";
 import path from "node:path";
 import { getOracleHomeDir } from "../oracleHome.js";
+import { isDeepResearchIncompleteText } from "./deepResearchResult.js";
 const ARTIFACTS_DIRNAME = "artifacts";
 function sanitizePathSegment(value, fallback) {
     const sanitized = value
@@ -85,16 +86,9 @@ export async function writeBinaryBrowserArtifact(params) {
         sourceUrl: params.sourceUrl,
     };
 }
-function isToolOnlyPlaceholder(text) {
-    const normalized = text.toLowerCase().replace(/\s+/g, " ").trim();
-    return (normalized === "called tool" ||
-        normalized === "used tool" ||
-        normalized === "użyto narzędzia" ||
-        normalized === "narzędzie wywołane");
-}
 export async function saveDeepResearchReportArtifact(params) {
     const report = params.reportMarkdown.trim();
-    if (report.length < 40 || isToolOnlyPlaceholder(report)) {
+    if (report.length < 40 || isDeepResearchIncompleteText(report)) {
         return null;
     }
     return writeTextBrowserArtifact({