]*\bopen\b/);
+ });
+
+ test("the v1/base/v2 radiogroup left Models and lives in the Subagents delegation section", () => {
+ expect(models).not.toContain("models-v2-mode-row");
+ expect(models).not.toContain("const setMultiAgentMode");
+ expect(models).not.toContain("v2HelpOpen");
+ expect(delegation).toContain('role="radiogroup" aria-label={t("models.v2Label")}');
+ expect(delegation).toContain("onUltraModeSave({ multiAgentMode: mode })");
+ // The long help text is reachable from a focusable Tooltip, not a modal.
+ expect(delegation).toContain('content={t("models.v2Help")}');
+ });
+
+ test("the subtitle and order hint became focusable tooltips with accessible names", () => {
+ expect(models).not.toContain("models-order-hint");
+ expect(models).toContain('content={t("models.subtitle")}');
+ expect(models).toContain('{t("models.subtitleAria")}');
+ expect(models).toContain('content={t("models.orderHint")}');
+ expect(models).toContain('{t("models.orderHintAria")}');
+ // Non-catalog subtitles only in the empty state.
+ expect(models).toContain('{tab !== "catalog" && tabIsEmpty && ');
+ });
+
+ test("the page head no longer carries a third restart button", () => {
+ const head = models.slice(models.indexOf('className="page-head"'), models.indexOf(" {
+ test("edit and all-on/all-off stay inline; aliases, custom model, presets and cap fold into the details", () => {
+ const at = models.indexOf('');
+ const more = models.indexOf('', at);
+ expect(more).toBeGreaterThan(at);
+ const inline = models.slice(at, more);
+ expect(inline).toContain("models-alias-edit");
+ expect(inline).toContain('{t("models.allOn")}');
+ expect(inline).toContain('{t("models.allOff")}');
+ const folded = models.slice(more, models.indexOf(" ", more));
+ expect(folded).toContain('label={t("models.useDefaultAliases")}');
+ expect(folded).toContain('{t("models.customAdd")}');
+ expect(folded).toContain('aria-label={t("models.presetLabel")}');
+ expect(folded).toContain("models-cap-cluster");
+ // Labelled disclosure, never a menu role.
+ expect(folded).toContain('aria-label={t("models.groupMore")}');
+ expect(folded).not.toContain('role="menu"');
+ });
+
+ test("both disclosures have summary styling that hides the native marker", () => {
+ expect(css).toContain(".models-advanced > summary::-webkit-details-marker { display: none; }");
+ expect(css).toContain(".models-group-more > summary::-webkit-details-marker { display: none; }");
+ });
+});
diff --git a/gui/tests/models-keep-native-v1-placement.test.ts b/gui/tests/models-keep-native-v1-placement.test.ts
index 4e3969df58..ec1cde3c91 100644
--- a/gui/tests/models-keep-native-v1-placement.test.ts
+++ b/gui/tests/models-keep-native-v1-placement.test.ts
@@ -68,13 +68,16 @@ test("v2 setting writes adopt the response instead of a no-op refetch", () => {
expect(helper).not.toContain("void loadV2()");
});
-test("both v2 surface setters route through the shared writer", () => {
+test("the keep-native setter routes through the shared writer; the mode switch left for Subagents", () => {
const setters = modelsSource.slice(
- modelsSource.indexOf("const setMultiAgentMode"),
+ modelsSource.indexOf("const setKeepNativeChatGptOnV1"),
modelsSource.indexOf("const putV2Threads"),
);
- expect(setters).toContain("putV2Setting({ multiAgentMode: mode })");
expect(setters).toContain("putV2Setting({ keepNativeChatGptOnV1: next })");
expect(setters).not.toContain("void loadV2()");
+ // The v1/base/v2 radiogroup is a delegation setting and lives on Subagents now
+ // (030): Models no longer owns a multiAgentMode writer.
+ expect(modelsSource).not.toContain("const setMultiAgentMode");
+ expect(modelsSource).not.toContain("putV2Setting({ multiAgentMode: mode })");
});
diff --git a/gui/tests/subagents-ultra-mode.test.tsx b/gui/tests/subagents-ultra-mode.test.tsx
index 73e34907c0..2c12cbd379 100644
--- a/gui/tests/subagents-ultra-mode.test.tsx
+++ b/gui/tests/subagents-ultra-mode.test.tsx
@@ -173,3 +173,50 @@ test("a save refresh from an old API server cannot overwrite a newer server", as
expect(ultraSwitch().disabled).toBe(true);
expect(ultraSwitch().getAttribute("aria-pressed")).toBe("false");
});
+
+test("clicking a surface mode on Subagents PUTs /api/v2 with multiAgentMode and re-reads it", async () => {
+ // Server starts in v1; after the PUT the re-read returns the new mode, and the radiogroup
+ // must follow the server, not the click.
+ v2Responses = [{ ok: true, body: { enabled: false, multiAgentMode: "v1", multiAgentModeHintText: null } }];
+ let serverMode: "v1" | "default" | "v2" = "v1";
+ const previousFetch = globalThis.fetch;
+ Object.defineProperty(globalThis, "fetch", {
+ configurable: true,
+ value: async (url: string, init?: RequestInit) => {
+ requests.push({ url: String(url), init });
+ const path = new URL(String(url), "http://localhost/").pathname;
+ if (path === "/api/v2") {
+ if (init?.method === "PUT") {
+ const body = JSON.parse(String(init.body)) as { multiAgentMode?: typeof serverMode };
+ if (body.multiAgentMode) serverMode = body.multiAgentMode;
+ return response({ ok: true });
+ }
+ return response({ enabled: false, multiAgentMode: serverMode, multiAgentModeHintText: null });
+ }
+ return (previousFetch as typeof fetch)(url, init);
+ },
+ });
+
+ root = createRoot(container);
+ await act(async () => {
+ root!.render();
+ });
+ await act(async () => { await new Promise(r => testWindow.setTimeout(r, 30)); });
+
+ const group = container.querySelector('[role="radiogroup"][aria-label="Sub-agent mode"]')
+ ?? [...container.querySelectorAll('[role="radiogroup"]')].find(g => g.querySelector('[role="radio"]'));
+ expect(group).toBeDefined();
+ const radios = [...group!.querySelectorAll('[role="radio"]')];
+ expect(radios.map(r => r.getAttribute("aria-checked"))).toEqual(["true", "false", "false"]);
+
+ await act(async () => { radios[2]!.click(); });
+ await act(async () => { await new Promise(r => testWindow.setTimeout(r, 30)); });
+
+ const put = requests.find(r => r.init?.method === "PUT" && r.url.endsWith("/api/v2"));
+ expect(put).toBeDefined();
+ expect(JSON.parse(String(put!.init!.body))).toEqual({ multiAgentMode: "v2" });
+ // The follow-up GET landed and the radiogroup reflects the server.
+ const gets = requests.filter(r => (r.init?.method ?? "GET") === "GET" && r.url.endsWith("/api/v2"));
+ expect(gets.length).toBeGreaterThanOrEqual(2);
+ expect(radios.map(r => r.getAttribute("aria-checked"))).toEqual(["false", "false", "true"]);
+});