@adecore/agents-react 0.21.1 → 0.21.2

package/dist/chat/ui/Composer.js

@@ -49,8 +49,7 @@ const NOTICE_MS = 4000;
 const PROMPT_HISTORY = 50;
 // Commands the composer handles itself; the CLI's own ones are sent through as text.
 const LOCAL_COMMANDS = ['model', 'compact', 'clear'];
-/* Files and paths dropped on the card become attachments and chips there. CodeMirror would paste a
-   file's text, or the drag's own text, at the drop point on top of that. */
+/* The card turns dropped files and paths into attachments and chips; CodeMirror would also paste their text at the drop point. */
 const LEAVE_DROPS_TO_THE_CARD = EditorView.domEventHandlers({
     drop: (event) => event.dataTransfer !== null && (event.dataTransfer.types.includes('Files') || carriesMentions(event.dataTransfer.types))
 });
@@ -88,6 +87,18 @@ function hintedPlaceholder(lead, joiner, hints) {
     root.append(lead, extra);
     return root;
 }
+function sendLabelKey(busy, canSteer, delivery) {
+    if (!busy) {
+        return 'composer.send';
+    }
+    if (!canSteer) {
+        return 'composer.queueButton';
+    }
+    return delivery === 'steer' ? 'composer.steerButton' : 'composer.queueWithSteerButton';
+}
+function sameQuery(left, right) {
+    return left?.start === right?.start && left?.query === right?.query;
+}
 function splitPath(path) {
     const slash = path.lastIndexOf('/');
     return slash < 0 ? { name: path, dir: '' } : { name: path.slice(slash + 1), dir: path.slice(0, slash) };
@@ -125,7 +136,6 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
     const providers = useProviders((s) => s.providers);
     const { approvals, questions } = usePendingRequests(chatId);
     const order = useChatRow(chatId, (row) => row?.order);
-    // The structure, which a delta leaves alone. The prompts and the requests are items of their own.
     const items = useChatRow(chatId, (row) => row?.structure);
     const chats = chatHost().useReferableChats();
     const placeholderLead = chatHost().useComposerPlaceholder(scope.id, chatId);
@@ -136,7 +146,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
     const capabilities = provider?.capabilities;
     const compaction = capabilities?.compaction;
     const dismissedTurnId = useResumeCompactionDismissal(chatId);
-    /* The cheap half of the offer, so a chat that can never make one keeps no timer. */
+    // The cheap half of the offer, so a chat that can never make one keeps no timer.
     const couldOfferCompaction = compaction !== undefined && compaction !== 'none' && info.usage.contextTokens >= RESUME_COMPACTION_TOKENS;
     const now = useNow(60_000, couldOfferCompaction);
     const compactionOffer = !couldOfferCompaction
@@ -156,13 +166,9 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
     const quote = draft.quote;
     // The CLI announces its session on the first message, so anything before that is still a blank chat.
     const started = info.agentSessionId !== null || info.usage.turns > 0;
-    /*
-     * A chat that has started keeps its provider (the CLI holds the thread), and one opened for a
-     * named CLI was never going to change it. Either way the models stay switchable. The CLI takes
-     * the new one on the restart the next send does anyway. With one provider in the list the
-     * picker drops its group headers on its own, so it reads as that CLI's own catalog.
-     */
     const accountChoice = useAccountChoice(info.provider, info.account);
+    // A chat that has started keeps its provider (the CLI holds the thread), and so does one opened for a
+    // named CLI; its models stay switchable, since the CLI takes a new one on the restart the next send does.
     const pickable = started || providerFixed
         ? providers.filter((entry) => entry.kind === info.provider)
         : providers.filter((entry) => entry.installed && entry.capabilities.chat);
@@ -197,11 +203,11 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
         }
         let stale = false;
         const search = () => {
-            const search = chatHost().searchFiles;
-            if (search === null) {
+            const searchFiles = chatHost().searchFiles;
+            if (searchFiles === null) {
                 return;
             }
-            search(scope.id, info.cwd, mention.query, MENTION_RESULTS)
+            searchFiles(scope.id, info.cwd, mention.query, MENTION_RESULTS)
                 .then((files) => {
                 if (!stale) {
                     setSearched({ query: mention.query, files });
@@ -292,15 +298,25 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
         setDraft((current) => ({ ...current, quote: '' }));
         inputRef.current?.focus();
     };
-    const configure = (patch) => {
+    const showError = (error) => {
+        setNotice(error instanceof Error ? error.message : String(error));
+    };
+    /* One configuration at a time; a second pick while one is on its way is dropped. */
+    const runConfiguration = (work) => {
         if (configuring.current) {
             return;
         }
         configuring.current = true;
         setConfigurationPending(true);
-        void actions
-            .configure(chatId, patch)
-            .then(() => {
+        void work()
+            .catch(showError)
+            .finally(() => {
+            configuring.current = false;
+            setConfigurationPending(false);
+        });
+    };
+    const configure = (patch) => {
+        runConfiguration(() => actions.configure(chatId, patch).then(() => {
             if (patch.selection) {
                 rememberChatSelection(info.provider, patch.selection);
             }
@@ -310,40 +326,24 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
             if (patch.account) {
                 rememberChatAccount(scope.id, info.provider, patch.account);
             }
-        })
-            .catch((error) => setNotice(error instanceof Error ? error.message : String(error)))
-            .finally(() => {
-            configuring.current = false;
-            setConfigurationPending(false);
-        });
+        }));
     };
     const chooseAccount = (account) => configure({ account });
     /* A model of another CLI re-points the whole chat; one of this CLI's own is a configure. */
     const chooseModel = (provider, slug) => {
         const selection = { model: slug, options: {} };
         if (provider !== info.provider) {
-            if (configuring.current) {
-                return;
-            }
-            configuring.current = true;
-            setConfigurationPending(true);
-            void onRetarget(provider, selection)
-                .then(() => rememberChatSelection(provider, selection))
-                .catch((error) => setNotice(error instanceof Error ? error.message : String(error)))
-                .finally(() => {
-                configuring.current = false;
-                setConfigurationPending(false);
-            });
+            runConfiguration(() => onRetarget(provider, selection).then(() => rememberChatSelection(provider, selection)));
             return;
         }
         configure({ selection });
     };
     const stop = (shiftKey) => {
         if (composerStopOf(shiftKey) === 'turn') {
-            void actions.stopTurn(chatId, false).catch((error) => setNotice(error instanceof Error ? error.message : String(error)));
+            void actions.stopTurn(chatId, false).catch(showError);
             return;
         }
-        chatHost().confirm.stopSubagents(scope.id, chatId, () => void actions.stopTurn(chatId, true).catch((error) => setNotice(error instanceof Error ? error.message : String(error))));
+        chatHost().confirm.stopSubagents(scope.id, chatId, () => void actions.stopTurn(chatId, true).catch(showError));
     };
     /* The host decides whether a turn is in the way; only its refusal asks the person first. */
     const clearThread = async () => {
@@ -380,17 +380,16 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
                 return false;
         }
     };
-    const same = (a, b) => a?.start === b?.start && a?.query === b?.query;
     /* Which picker the caret opens: `@` for a file, `$` for a skill, or neither. In code a sigil is just text. */
     const trackTriggers = (value, selection, state) => {
         const empty = selection.from === selection.to;
         const tree = empty ? (ensureSyntaxTree(state, selection.from, 50) ?? syntaxTree(state)) : null;
         const caret = tree !== null && !inCode(tree, value, selection.from) ? selection.from : null;
         const skill = caret === null ? null : findSkillQuery(value, caret);
-        setSkillQuery((current) => (same(current, skill) ? current : skill));
+        setSkillQuery((current) => (sameQuery(current, skill) ? current : skill));
         // A CLI that does not expand `@path` gets the text as it is, so the picker stays out of the way.
         const next = caret === null || capabilities?.mentions === false ? null : findMentionQuery(value, caret);
-        setMention((current) => (same(current, next) ? current : next));
+        setMention((current) => (sameQuery(current, next) ? current : next));
     };
     // The inserted text reaches the editor with the render it causes, so the caret waits a frame for it.
     const moveCaret = (caret) => {
@@ -438,8 +437,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
         setSkillQuery(null);
         moveCaret(result.caret);
     };
-    /* Paths dragged in from a list of files. There is no mention query to insert into, so they land
-       at the end of the prompt and join the draft's mentions, which is what `chat.send` carries. */
+    /* Paths dragged in from a list of files. With no mention query to insert into, they land at the end of the prompt. */
     const addMentions = (paths) => {
         const fresh = paths.filter((path) => path !== '');
         if (fresh.length === 0 || capabilities?.mentions === false) {
@@ -620,6 +618,18 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
         setText(history[next].text, history[next].mentions);
         return true;
     };
+    /* The arrows walk an open picker round; true when the key was one of them. */
+    const moveInMenu = (key, count) => {
+        if (key === 'ArrowDown') {
+            setMenuIndex((i) => (i + 1) % count);
+            return true;
+        }
+        if (key === 'ArrowUp') {
+            setMenuIndex((i) => (i - 1 + count) % count);
+            return true;
+        }
+        return false;
+    };
     /* Runs before the editor's own keymap; true means the key is handled and its default prevented. */
     const onKeyDown = (e, view) => {
         // Shift with the paste shortcut pastes a large text inline after all; any other key forgets it.
@@ -634,8 +644,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
             setText('', [], []);
             return true;
         }
-        // The composer keeps the keyboard while you type, but the shortcuts that move between views,
-        // panels and the palette stay the app's, untouched: Mod+Shift+Enter would otherwise send as well.
+        // The app's own shortcuts stay the app's while you type: Mod+Shift+Enter would otherwise send as well.
         if (chatHost().isAppShortcut(e)) {
             return false;
         }
@@ -653,12 +662,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
             return true;
         }
         if (commandMenuOpen) {
-            if (e.key === 'ArrowDown') {
-                setMenuIndex((i) => (i + 1) % commands.length);
-                return true;
-            }
-            if (e.key === 'ArrowUp') {
-                setMenuIndex((i) => (i - 1 + commands.length) % commands.length);
+            if (moveInMenu(e.key, commands.length)) {
                 return true;
             }
             if (e.key === 'Tab') {
@@ -673,12 +677,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
             }
         }
         if (skillMenuOpen) {
-            if (e.key === 'ArrowDown') {
-                setMenuIndex((i) => (i + 1) % skillMatches.length);
-                return true;
-            }
-            if (e.key === 'ArrowUp') {
-                setMenuIndex((i) => (i - 1 + skillMatches.length) % skillMatches.length);
+            if (moveInMenu(e.key, skillMatches.length)) {
                 return true;
             }
             if (e.key === 'Tab' || e.key === 'Enter') {
@@ -687,12 +686,7 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
             }
         }
         if (mentionMenuOpen && mentionCount > 0) {
-            if (e.key === 'ArrowDown') {
-                setMenuIndex((i) => (i + 1) % mentionCount);
-                return true;
-            }
-            if (e.key === 'ArrowUp') {
-                setMenuIndex((i) => (i - 1 + mentionCount) % mentionCount);
+            if (moveInMenu(e.key, mentionCount)) {
                 return true;
             }
             if (e.key === 'Tab' || e.key === 'Enter') {
@@ -856,11 +850,5 @@ export function Composer({ chatId, info, focused, answerPromptsElsewhere, disabl
                                         addFiles(Array.from(e.target.files ?? []));
                                         e.target.value = '';
                                         inputRef.current?.focus();
-                                    } }), _jsx(RunSettings, { disabled: configurationPending || !writable, providers: pickable, provider: info.provider, selection: info.selection, model: model, runtimeMode: info.runtimeMode, effectiveRuntimeMode: info.effectiveRuntimeMode, permissionMode: info.permissionMode, open: modelPickerOpen, onOpenChange: setModelPickerOpen, onModel: chooseModel, onOption: (id, value) => configure({ selection: { ...info.selection, options: { ...info.selection.options, [id]: value } } }), onMode: (runtimeMode) => configure({ runtimeMode }), usage: info.usage, compactDisabled: busy || !writable, onCompact: compact, account: accountChoice === null ? null : { choice: accountChoice, started, onPick: chooseAccount } }), _jsx(StashPicker, { onRestore: restoreStashed }), Slot !== null && (_jsx(ErrorBoundary, { label: t('composer.slotFailed'), resetKeys: [chatId], compact: true, className: "relative rounded-lg", children: _jsx(Slot, { scopeId: scope.id, chatId: chatId, disabled: !writable, insert: insertIntoDraft }) })), _jsx("span", { className: "grow" }), _jsxs("div", { className: "flex h-9 shrink-0 items-center gap-0.5 rounded-full bg-surface-hover", children: [_jsx("div", { ref: setDictationToolbar, className: "flex items-center pl-1 empty:hidden [&_.icon-btn]:rounded-full" }), busy && (_jsx(Tooltip, { label: composerStopLabel(), name: true, children: _jsx("button", { className: "flex h-9 w-9 shrink-0 items-center justify-center rounded-full bg-status-error text-accent-text", onClick: (event) => stop(event.shiftKey), children: _jsx(Icon, { icon: Square, size: 16 }) }) })), (!busy || !isEmptyDraft(draft)) && (_jsx(Tooltip, { label: busy
-                                                ? t(canSteer
-                                                    ? sendDelivery === 'steer'
-                                                        ? 'composer.steerButton'
-                                                        : 'composer.queueWithSteerButton'
-                                                    : 'composer.queueButton')
-                                                : t('composer.send'), kbd: KEY_SHORTCUTS.modEnter, name: true, children: _jsx("button", { className: "flex h-9 w-9 shrink-0 items-center justify-center rounded-full bg-accent text-accent-text hover:brightness-90 disabled:opacity-50 disabled:hover:brightness-100", disabled: isEmptyDraft(draft) || !writable || guard.tooLong, onClick: (event) => submit(event.altKey), children: _jsx(Icon, { icon: ArrowUp, size: 16 }) }) }))] })] })] }) }), _jsx(PromptDialog, { open: confirmClear, title: t('composer.clear.title'), description: t('composer.clear.description'), confirmLabel: t('composer.clear.confirm'), danger: true, fallbackMessage: t('composer.notice.clearFailed'), onConfirm: forceClearThread, onOpenChange: () => setConfirmClear(false) })] }));
+                                    } }), _jsx(RunSettings, { disabled: configurationPending || !writable, providers: pickable, provider: info.provider, selection: info.selection, model: model, runtimeMode: info.runtimeMode, effectiveRuntimeMode: info.effectiveRuntimeMode, permissionMode: info.permissionMode, open: modelPickerOpen, onOpenChange: setModelPickerOpen, onModel: chooseModel, onOption: (id, value) => configure({ selection: { ...info.selection, options: { ...info.selection.options, [id]: value } } }), onMode: (runtimeMode) => configure({ runtimeMode }), usage: info.usage, compactDisabled: busy || !writable, onCompact: compact, account: accountChoice === null ? null : { choice: accountChoice, started, onPick: chooseAccount } }), _jsx(StashPicker, { onRestore: restoreStashed }), Slot !== null && (_jsx(ErrorBoundary, { label: t('composer.slotFailed'), resetKeys: [chatId], compact: true, className: "relative rounded-lg", children: _jsx(Slot, { scopeId: scope.id, chatId: chatId, disabled: !writable, insert: insertIntoDraft }) })), _jsx("span", { className: "grow" }), _jsxs("div", { className: "flex h-9 shrink-0 items-center gap-0.5 rounded-full bg-surface-hover", children: [_jsx("div", { ref: setDictationToolbar, className: "flex items-center pl-1 empty:hidden [&_.icon-btn]:rounded-full" }), busy && (_jsx(Tooltip, { label: composerStopLabel(), name: true, children: _jsx("button", { className: "flex h-9 w-9 shrink-0 items-center justify-center rounded-full bg-status-error text-accent-text", onClick: (event) => stop(event.shiftKey), children: _jsx(Icon, { icon: Square, size: 16 }) }) })), (!busy || !isEmptyDraft(draft)) && (_jsx(Tooltip, { label: t(sendLabelKey(busy, canSteer, sendDelivery)), kbd: KEY_SHORTCUTS.modEnter, name: true, children: _jsx("button", { className: "flex h-9 w-9 shrink-0 items-center justify-center rounded-full bg-accent text-accent-text hover:brightness-90 disabled:opacity-50 disabled:hover:brightness-100", disabled: isEmptyDraft(draft) || !writable || guard.tooLong, onClick: (event) => submit(event.altKey), children: _jsx(Icon, { icon: ArrowUp, size: 16 }) }) }))] })] })] }) }), _jsx(PromptDialog, { open: confirmClear, title: t('composer.clear.title'), description: t('composer.clear.description'), confirmLabel: t('composer.clear.confirm'), danger: true, fallbackMessage: t('composer.notice.clearFailed'), onConfirm: forceClearThread, onOpenChange: () => setConfirmClear(false) })] }));
 }

package/dist/chat/ui/Markdown.js

@@ -54,10 +54,8 @@ function languageOf(className) {
 }
 const INLINE_CODE = 'rounded-sm bg-surface-sunken px-1 py-px font-mono text-code';
 /*
- * A path an agent wrote down, opened in the preview panel. It keeps the shape of the code chip it
- * would otherwise be, so a sentence full of them does not start dancing; only the color says it can
- * be clicked. `data-file-path` is what the thread's own right-click menu reads, so a link answers to
- * both ways of opening it.
+ * A path an agent wrote down. It keeps the shape of the code chip it would otherwise be, so a sentence
+ * full of them does not shift; `data-file-path` is what the thread's right-click menu reads.
  */
 function FileLink({ target, className, children }) {
     const cwd = useFileLinkCwd();
@@ -72,8 +70,10 @@ function InlineCode({ text }) {
     }
     return (_jsx(FileLink, { target: target, className: `${INLINE_CODE} text-accent hover:underline`, children: text }));
 }
-/* The path a link target spells, with the one scheme that still means a file on the host's machine
-   taken off. A href nobody encoded is left as it is, since decoding throws on a stray percent. */
+/*
+ * The path a link target spells, without the one scheme that still means a file on the host's machine.
+ * A href nobody encoded is left as it is, since decoding throws on a stray percent.
+ */
 function hrefPath(href) {
     const bare = href.replace(/^file:\/\//, '');
     try {
@@ -89,6 +89,9 @@ function MarkdownLink({ href, children }) {
     if (target !== null) {
         return (_jsx(FileLink, { target: target, className: "text-accent underline underline-offset-2", children: children }));
     }
+    return _jsx(ExternalLink, { href: href, children: children });
+}
+function ExternalLink({ href, children }) {
     return (_jsx("a", { href: href, target: "_blank", rel: "noreferrer", children: children }));
 }
 function MarkdownCode({ className, children, plain = false }) {
@@ -118,9 +121,7 @@ const plainComponents = {
     code({ className, children }) {
         return (_jsx(MarkdownCode, { className: className, plain: true, children: children }));
     },
-    a({ href, children }) {
-        return (_jsx("a", { href: href, target: "_blank", rel: "noreferrer", children: children }));
-    }
+    a: ExternalLink
 };
 // A reply sits under the heading of its message, an `h3`, so its own headings start one level below
 // that. Only `aria-level` moves. The tag stays, since the styles and a copy of the thread read it.
@@ -186,8 +187,7 @@ const PLUGINS_WITH_BREAKS = [remarkGfm, remarkBreaks];
 const MESSAGE_PLUGINS = [remarkGfm, remarkHtmlAsText, remarkBreaks];
 const FADE_PLUGINS = [rehypeFadeWords];
 const NO_PLUGINS = [];
-/* One block of a reply. It renders no element of its own, so prose still sees the paragraphs as
-   direct children and keeps its first and last margins. */
+/* One block of a reply. It renders no element of its own, so prose keeps its first and last margins. */
 const ReplyBlock = memo(function ReplyBlock({ text, fade, open, reply }) {
     return (_jsx(CodeStreamingContext.Provider, { value: open, children: _jsx(ShellCodeBlockReplyContext.Provider, { value: reply ?? null, children: _jsx(ReactMarkdown, { remarkPlugins: REPLY_PLUGINS, rehypePlugins: fade ? FADE_PLUGINS : NO_PLUGINS, components: replyComponents, children: text }) }) }));
 });

package/dist/chat/ui/Pickers.js

@@ -13,11 +13,9 @@ function matches(model, query) {
     return model.name.toLowerCase().includes(query) || model.slug.toLowerCase().includes(query);
 }
 /*
- * Which model answers: the provider's mark plus the model's short name
- * as the trigger, a search field over the popup, one group per provider and the legacy models
- * behind an expander. Before the first message the list carries every installed CLI, so picking
- * another provider's model is also how a chat picks its provider. A chat bound to one CLI passes
- * only that CLI, which drops the group headers and leaves its own catalog to choose from.
+ * Which model answers: a search field over one group per provider, with the legacy models behind an
+ * expander. Before the first message the list carries every installed CLI, so picking another
+ * provider's model is also how a chat picks its provider; with one CLI the group headers drop.
  */
 export function ModelPicker({ providers, provider, selection, open, onOpenChange, onChange, kbd = '/model', side = 'top', trigger = 'plain', finalFocus, footer }) {
     const { t } = useTranslation('agent-chat');
@@ -96,10 +94,8 @@ export function ModelPicker({ providers, provider, selection, open, onOpenChange
                             })] }), footer !== undefined && _jsx("div", { className: "border-t border-border-soft px-3 py-2 text-xs text-text-faint", children: footer })] })] }));
 }
 /*
- * Prompts put aside with Cmd+S. The shelf is the whole app's, not this chat's. On a canvas a
- * stashed prompt usually moves to another node, which is the reason to put it away in the first
- * place. Restoring drops the text, the mentions and the skills into this composer; the files a
- * draft held are named on the row but not kept, because their bytes never go to storage.
+ * Prompts put aside with Cmd+S, on a shelf the whole app shares, since a stashed prompt usually moves
+ * to another chat. A draft's files are named on the row but not kept: their bytes never go to storage.
  */
 export function StashPicker({ onRestore }) {
     const { t } = useTranslation('agent-chat');

package/dist/chat/ui/QuoteButton.js

@@ -13,9 +13,8 @@ function lastLineOf(range) {
     return rects.at(-1) ?? null;
 }
 /*
- * A button under a selection in an answer that hands it to the composer. Quoting on the selection
- * alone grew the composer while the person was still selecting, which moved the thread under the
- * pointer. It only shows once the pointer is up, so a drag never has it jump along.
+ * A button under a selection in an answer that hands it to the composer. It shows once the pointer is
+ * up: quoting on the selection alone grew the composer mid-drag and moved the thread under the pointer.
  */
 export function QuoteButton({ thread, frame, taker }) {
     const { t } = useTranslation('agent-chat');

package/dist/chat/ui/RunSettings.js

@@ -47,6 +47,12 @@ function contextTone(fraction) {
 function modelValue(provider, slug) {
     return `${provider}/${slug}`;
 }
+/* Each provider's current or legacy models; a provider with none of them is left out. */
+function modelGroupsOf(providers, legacy) {
+    return providers
+        .map((entry) => ({ entry, models: entry.models.filter((row) => (row.legacy === true) === legacy) }))
+        .filter((group) => group.models.length > 0);
+}
 /*
  * Everything that decides how the next turn runs, behind one pill: the model, its own knobs and the
  * permission mode. Model and effective permissions stay visible as the composer narrows.
@@ -68,8 +74,8 @@ export function RunSettings({ disabled = false, providers, provider, selection,
         summary.push({ id: CONTEXT_OPTION, label: formatTokens(usage.contextWindow ?? 0), ring: true });
     }
     const grouped = providers.length > 1;
-    const live = providers.map((entry) => ({ entry, models: entry.models.filter((row) => !row.legacy) })).filter((group) => group.models.length > 0);
-    const legacy = providers.map((entry) => ({ entry, models: entry.models.filter((row) => row.legacy) })).filter((group) => group.models.length > 0);
+    const live = modelGroupsOf(providers, false);
+    const legacy = modelGroupsOf(providers, true);
     const chosenLegacy = owner?.models.find((row) => row.legacy && row.slug === selection.model);
     const chosenModel = modelValue(provider, selection.model);
     const shownMode = effectiveRuntimeMode ?? runtimeMode;

package/dist/chat/ui/Scrubber.js

@@ -34,9 +34,8 @@ const CARD_ACTIONS = {
 const NOTHING_FOUND = [];
 /*
  * The strip at the left edge of a chat view: a tick per message of the person, per wake by tasks and
- * per other message with a bookmark, the ones on screen bright and the rest dimmed, a bookmark in the accent, growing only under the pointer. A single hover card follows the pointer along the strip, anchored at the tick
- * it points at, so a thousand ticks are a thousand spans and not a thousand popups. While a find is
- * open, a short mark at the strip's right edge stands beside every tick with a hit under it.
+ * per other bookmarked message, the ones on screen bright. One hover card follows the pointer, so a
+ * thousand ticks are a thousand spans and not a thousand popups. An open find marks every tick it hits.
  */
 export const Scrubber = memo(function Scrubber({ ticks, firstInView, lastInView, onPick, chat = null, found = NOTHING_FOUND, foundCurrent = null }) {
     const { t } = useTranslation('agent-chat');

package/dist/chat/ui/SubagentControls.js

@@ -7,9 +7,7 @@ import { breadcrumbOf, MAIN_AGENT, trailTo, useOpenableSubagents, useSubagentTra
 import { SubagentStopButton } from './SubagentStopButton.js';
 import { Icon, IconButton } from '@adecore/ui';
 const CRUMB_LINK = 'truncate text-text-muted hover:text-text';
-/*
- * The chat's own title where the bar draws it. While a sub-agent stands in the chat's place it is the first crumb of the way down, so pressing it goes back to the main agent.
- */
+/* The chat's own title in the bar; while a sub-agent stands in the chat's place, pressing it goes back to the main agent. */
 export function SubagentTitleCrumb({ chatId, className, children }) {
     const { trail, show } = useSubagentTrail(chatId);
     if (trail.length === 0) {

package/dist/chat/ui/Timeline.js

@@ -37,6 +37,11 @@ import { AgentIcon } from '../../agents/AgentIcon.js';
 import { useModelName } from '../../agents/model-name.js';
 import { ContextMenu, EmptyState, ErrorBoundary, Icon, SectionLabel } from '@adecore/ui';
 const ESTIMATED_ROW_PX = 56;
+/* A jump puts its row at the top and stops following the end, so the reader stays where they landed. */
+function showRowAtTop(virtualizer, followRef, index) {
+    followRef.current = false;
+    virtualizer.scrollToIndex(index, { align: 'start' });
+}
 /*
  * A thread before its first message: which model answers, what the lines into this node let it read,
  * and the folder it works in, so a first question can lean on all three.
@@ -191,7 +196,7 @@ export function Timeline({ chatId, composer, overlay }) {
     useEffect(() => registerTimeline(chatId, scrollRef.current, (enabled) => {
         followRef.current = enabled;
     }, () => coveredHeightRef.current), [chatId]);
-    // The client does not run the React Compiler, so its memoization rule has nothing to break here.
+    // Nothing here builds with the React Compiler, so its memoization rule has nothing to break.
     // oxlint-disable-next-line react/incompatible-library
     const virtualizer = useVirtualizer({
         count: rows.length,
@@ -209,11 +214,9 @@ export function Timeline({ chatId, composer, overlay }) {
     });
     const tail = lastReplyLength ?? rows.length;
     /*
-     * The rows are measured as they render, so the end of the list moves while it is drawn: a diff
-     * that highlights, an image that loads, a tool row that grows a line of output. `scrollToIndex`
-     * aims at where the row was when it was asked, which is what left the thread short of the end.
-     * The scroller's own bottom is a fact rather than an estimate, and the total size changing is
-     * what says a row was measured again, so this runs for every one of those.
+     * Rows are measured as they render, so the end moves while it is drawn (a diff that highlights, an
+     * image that loads). `scrollToIndex` aims at where a row was when asked and fell short; the
+     * scroller's own bottom is a fact, and a new total size says a row was measured again.
      */
     const totalSize = virtualizer.getTotalSize();
     useLayoutEffect(() => {
@@ -259,8 +262,7 @@ export function Timeline({ chatId, composer, overlay }) {
     const paddingLeft = threadPaddingLeft(frame.width, showsScrubber);
     /*
      * Where each message starts, from the virtualizer's measurements (an estimate for a row it never
-     * drew), so reading the scroll position costs no layout. The virtualizer renders this component on
-     * every scroll already.
+     * drew), so reading the scroll position costs no layout.
      */
     const measurements = virtualizer.measurementsCache;
     const starts = ticks.map((tick) => measurements[tick.rowIndex]?.start ?? tick.rowIndex * ESTIMATED_ROW_PX);
@@ -314,8 +316,7 @@ export function Timeline({ chatId, composer, overlay }) {
             setSeeking({ itemId: tick.bookmark.itemId, key: keyOf(chatId) });
             return;
         }
-        followRef.current = false;
-        virtualizer.scrollToIndex(tick.rowIndex, { align: 'start' });
+        showRowAtTop(virtualizer, followRef, tick.rowIndex);
     };
     useLayoutEffect(() => {
         if (seeking === null) {
@@ -336,8 +337,7 @@ export function Timeline({ chatId, composer, overlay }) {
             else {
                 setSeeking(null);
                 if (jump.kind === 'row') {
-                    followRef.current = false;
-                    virtualizer.scrollToIndex(jump.index, { align: 'start' });
+                    showRowAtTop(virtualizer, followRef, jump.index);
                     setLanding({ target: seeking.block, key: seeking.key });
                 }
             }
@@ -346,8 +346,7 @@ export function Timeline({ chatId, composer, overlay }) {
         const index = rows.findIndex((row) => row.id === seeking.itemId);
         if (index !== -1) {
             setSeeking(null);
-            followRef.current = false;
-            virtualizer.scrollToIndex(index, { align: 'start' });
+            showRowAtTop(virtualizer, followRef, index);
             return;
         }
         const item = items?.[seeking.itemId];
@@ -399,8 +398,7 @@ export function Timeline({ chatId, composer, overlay }) {
         const found = changes !== -1 ? changes : rows.findIndex((row) => row.id === opening?.id || row.id === `start-${turnId}` || row.id === `fold-${turnId}`);
         if (found !== -1) {
             setRevealing(null);
-            followRef.current = false;
-            virtualizer.scrollToIndex(found, { align: 'start' });
+            showRowAtTop(virtualizer, followRef, found);
             useTimelineFlash.getState().flash(keyOf(chatId), rows[found].id, changes !== -1);
             return;
         }
@@ -452,8 +450,7 @@ export function Timeline({ chatId, composer, overlay }) {
             return;
         }
         subagents.add(found.branch.id);
-        followRef.current = false;
-        virtualizer.scrollToIndex(found.index, { align: 'start' });
+        showRowAtTop(virtualizer, followRef, found.index);
     };
     return (_jsx(FileLinkContext.Provider, { value: info?.cwd ?? null, children: _jsxs("div", { ref: frameRef, className: "relative flex min-h-0 grow flex-col", "data-file-cwd": info?.cwd ?? '', "data-file-scope-id": scope.id, children: [searching && chatFind.bar, _jsxs(ContextMenu.Root, { children: [_jsx("div", { ref: scrollRef, 
                             // No CSS scroll-padding here: Chromium counts the sticky composer's caret as hidden behind

package/dist/chat/ui/TimelineMenu.js

@@ -9,11 +9,7 @@ import { useChatScope } from '../../scope.js';
 import { useChatRow } from '../../state/chats.js';
 import { openFileLink, useFileLinkCwd, useFileLinkScopeId } from './file-links.js';
 import { ContextMenu, copyText, DisabledReason, EDIT_SHORTCUTS, Icon, Kbd, selectAllWithin } from '@adecore/ui';
-/*
- * The menu behind a right-click in a thread. Copy is the reason it exists. Everything in a thread
- * is text a person may want out of it, and a whole message, a code block or the markdown an answer
- * was written in are each a different amount of that.
- */
+/* The menu behind a right-click in a thread, mostly for copying: a whole message, a code block or an answer's markdown. */
 export function TimelineMenuPopup({ target, thread, chatId = null }) {
     const { t } = useTranslation('agent-chat');
     const message = target.row === null ? null : messageTextOf(target.row);
@@ -36,10 +32,8 @@ export function TimelineMenuPopup({ target, thread, chatId = null }) {
     return (_jsxs(ContextMenu.Popup, { children: [_jsxs(ContextMenu.Item, { disabled: target.selection === '', onClick: () => copyText(target.selection), children: [_jsx(Icon, { icon: Copy, size: 14 }), " ", t('common.action.copy'), " ", _jsx(Kbd, { shortcut: EDIT_SHORTCUTS.copy })] }), message !== null && (_jsxs(ContextMenu.Item, { onClick: () => copyText(message), children: [_jsx(Icon, { icon: MessageSquare, size: 14 }), " ", t('timeline.menu.copyMessage')] })), target.code !== null && (_jsxs(ContextMenu.Item, { onClick: () => copyText(target.code ?? ''), children: [_jsx(Icon, { icon: Braces, size: 14 }), " ", t('timeline.menu.copyCode')] })), markdown !== null && (_jsxs(ContextMenu.Item, { onClick: () => copyText(markdown), children: [_jsx(Icon, { icon: FileText, size: 14 }), " ", t('timeline.menu.copyMarkdown')] })), chatId !== null && turnId !== null && fork !== null && (_jsx(DisabledReason, { reason: forkBlocked, children: _jsxs(ContextMenu.Item, { disabled: forkBlocked !== null, onClick: () => fork(chatId, turnId), children: [_jsx(Icon, { icon: GitFork, size: 14 }), " ", t('timeline.menu.forkFromHere')] }) })), chatId !== null && markable !== null && _jsx(BookmarkMenuItems, { chatId: chatId, itemId: markable }), _jsx(ContextMenu.Separator, {}), _jsxs(ContextMenu.Item, { onClick: () => selectAllWithin(thread.current), children: [_jsx(Icon, { icon: Scan, size: 14 }), " ", t('common.action.selectAll')] }), target.path !== null && fileLinks !== null && (_jsxs(_Fragment, { children: [_jsx(ContextMenu.Separator, {}), _jsxs(ContextMenu.Item, { onClick: openInPreview, children: [_jsx(Icon, { icon: Eye, size: 14 }), " ", t('timeline.menu.openInPreview')] })] }))] }));
 }
 /*
- * Place a bookmark on a message, or rename and remove the one it has: the same rows from the thread
- * and from the strip beside it. `onName` runs when a row opens the name field, which sits on the
- * bookmark's line over the message and so has to be on screen; the strip scrolls there, the thread
- * was clicked on it.
+ * Place a bookmark on a message, or rename and remove the one it has, from the thread and from the strip.
+ * `onName` runs when a row opens the name field, which sits over the message and so has to be on screen.
  */
 export function BookmarkMenuItems({ chatId, itemId, onName }) {
     const { t } = useTranslation('agent-chat');

package/dist/chat/ui/UnifiedDiff.js

@@ -6,9 +6,8 @@ import { chatHost } from '../../host.js';
 import { useDiffTheme } from './diff-theme.js';
 import { fullFileDiff } from './full-diff.js';
 /*
- * A patch the CLI reported itself (Codex writes unified diffs). The renderer needs a file header
- * to know what it is looking at, and a CLI that only sends hunks gets one from the path we know.
- * A diff without hunks at all is shown as its own lines, so nothing is ever swallowed.
+ * A patch the CLI reported itself. The renderer needs a file header, which a CLI that only sends hunks
+ * gets from the path we know; a diff without hunks is shown as its own lines, so nothing is swallowed.
  */
 function asPatch(change) {
     if (!/^@@/m.test(change.diff)) {
@@ -21,8 +20,7 @@ function asPatch(change) {
     const to = change.kind === 'delete' ? '/dev/null' : `b/${change.path}`;
     return `--- ${from}\n+++ ${to}\n${change.diff}`;
 }
-/* Lets a diff with a view of its own run to the bottom of it, so the scrollbar of a short one sits
-   there instead of under its last line; the lines themselves keep their height. */
+/* Lets a diff with a view of its own run to its bottom, so a short one's scrollbar sits there and not under its last line. */
 const FILL_CSS = `
 :host { display: flex; flex-direction: column; flex-grow: 1; }
 pre { flex-grow: 1; align-content: start; }

package/dist/chat/ui/VisualFrame.js

@@ -19,10 +19,9 @@ function personActsIn(frame) {
     return frame.ownerDocument.activeElement === frame && navigator.userActivation?.isActive !== false;
 }
 /*
- * One visual's page in a frame on the thread's own ground. The frame loads the app's sandbox host
- * page and the bridge hands it the page and the theme. The frame never gets `allow-same-origin`,
- * which keeps the page on an opaque origin even when the host page is served on the app's own. Its
- * box keeps one height while the page loads and when it fails, so nothing under it moves.
+ * One visual's page in a frame that loads the app's sandbox host page; the bridge hands it the page and
+ * the theme. The frame never gets `allow-same-origin`, which keeps the page on an opaque origin even
+ * when the host page is served on the app's own. Its box keeps one height while it loads or fails.
  */
 export function VisualFrame({ chatId, visual, fill = false, className }) {
     const { t } = useTranslation('agent-chat');