@tiagojct/subsub 0.9.0 → 0.10.0

package/src/subsub.ts

@@ -1,15 +1,17 @@
 /**
  * Sub-Sub: a Zotero librarian and research assistant for pi.
  *
- * - Two modes with different tool sets: librarian (changes the library, no web)
- *   and researcher (reads the library, searches PubMed/OpenAlex, writes vault notes).
+ * - One assistant: it reads the library, searches PubMed, Europe PMC and
+ *   OpenAlex, writes notes in the Sub-Sub folder and, with library changes on,
+ *   changes the library. /library on|off is the switch (saved in the settings).
  * - An approval gate: every library change is previewed by the server (dry run)
  *   and shown to the user; nothing is applied without a yes.
  * - Profiles (reader, scholar, author, editor) set what Sub-Sub writes and which
  *   tools it offers; /profile changes it.
- * - Commands: /librarian, /researcher, /profile, /subsub, /history, /undo.
+ * - Commands: /library, /profile, /subsub, /history, /undo. /librarian and
+ *   /researcher (the two modes before 0.10) turn library changes on and off.
  * - Add-on: with "addons": ["starbuck"], Starbuck runs as a third server
- *   ("verify") for reference checks; its tools belong to researcher mode.
+ *   ("verify") for reference checks.
  *
  * The Python servers start in session_start (not in the factory) and stop in
  * session_shutdown. Tools are registered once and use the current bridge.
@@ -25,7 +27,6 @@ import {
 	findUv,
 	isProfile,
 	loadConfig,
-	type Mode,
 	parseEnvFile,
 	type Profile,
 	saveConfig,
@@ -36,7 +37,7 @@ import {
 	who,
 } from "./config.ts";
 import { profileList, profileSpec, promptBlocked } from "./profiles.ts";
-import { coerceArgs, loosenArrays } from "./args.ts";
+import { coerceArgs, loosenArrays, resolvePathArgs } from "./args.ts";
 import { applyEdits, checkLiteratureNote } from "./notes.ts";
 import { buildPolicy, judgePath, normalizeToolPath, realish, within } from "./paths.ts";
 import { describeArgs, formatPreview } from "./preview.ts";
@@ -52,7 +53,6 @@ export interface SubsubDeps {
 }
 
 const PACKAGE_DIR = join(dirname(fileURLToPath(import.meta.url)), "..");
-const MODE_ENTRY = "subsub-mode";
 const BENCH_ENTRY = "subsub-bench";
 
 function ownVersion(): string {
@@ -127,9 +127,12 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 
 	if (deps.bridge) registerTools(deps.bridge);
 
-	pi.registerFlag("librarian", { description: "Start Sub-Sub in librarian mode", type: "boolean", default: false });
+	pi.registerFlag("read-only", { description: "Start Sub-Sub with library changes off (this run only)", type: "boolean", default: false });
+	pi.registerFlag("librarian", { description: "Start Sub-Sub with library changes on (this run only)", type: "boolean", default: false });
 
-	let mode: Mode = cfg.defaultMode;
+	let libraryChanges = cfg.libraryChanges;
+	const benchRun = Boolean(process.env.SUBSUB_BENCH_OUT);
+	const toolOpts = () => ({ libraryChanges, profile, bench: benchRun });
 	let profile: Profile = cfg.profile;
 	const user = who(cfg);
 	/** True when started with the `subsub` command (not as a package inside plain pi). */
@@ -151,7 +154,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 		if (!webView || !ctx || ctx.mode !== "rpc") return;
 		const down = bridge ? Object.keys(bridge.errors) : ["zotero", "scholar"];
 		const state = {
-			mode,
+			libraryChanges,
 			profile,
 			profileLabel: profileSpec(profile).label,
 			model: ctx.model ? { provider: ctx.model.provider, id: ctx.model.id } : null,
@@ -172,7 +175,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 	function applyTheme(ctx: ExtensionContext): void {
 		if (!lookOn(ctx) || cfg.themes === false) return;
 		scheme ??= schemeFromAnsi(ctx.ui.theme.getFgAnsi("text"), ctx.ui.theme.name);
-		const name = themeName(mode, scheme, cfg.themes);
+		const name = themeName(scheme, cfg.themes || undefined);
 		const theme = name ? ctx.ui.getTheme(name) : undefined;
 		// A Theme object (not a name) changes the theme for this run only; the saved setting stays.
 		if (theme) ctx.ui.setTheme(theme);
@@ -218,7 +221,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 					const th = ctx.ui.theme;
 					return headerLines({
 						version,
-						mode,
+						libraryChanges,
 						profile: profileSpec(profile).label,
 						model: modelId,
 						library,
@@ -241,28 +244,37 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 	function status(ctx: ExtensionContext): void {
 		modelId = ctx.model ? `${ctx.model.id}` : "no model";
 		const down = bridge ? Object.keys(bridge.errors) : ["zotero", "scholar"];
-		ctx.ui.setStatus("subsub", `Sub-Sub: ${mode} | ${profileSpec(profile).label} | ${modelId}${down.length ? ` | not running: ${down.join(", ")}` : ""}`);
-		if (standalone) ctx.ui.setTitle(`Sub-Sub: ${mode}`);
+		const lib = libraryChanges ? "library changes on" : "read only";
+		ctx.ui.setStatus("subsub", `Sub-Sub: ${lib} | ${profileSpec(profile).label} | ${modelId}${down.length ? ` | not running: ${down.join(", ")}` : ""}`);
+		if (standalone) ctx.ui.setTitle(libraryChanges ? "Sub-Sub" : "Sub-Sub (read only)");
 		headerTui?.requestRender();
 		webCtx = ctx;
 		emitWeb();
 	}
 
-	async function applyMode(ctx: ExtensionContext, next: Mode, remember: boolean): Promise<void> {
-		mode = next;
-		pi.setActiveTools(toolsFor(mode, registered(), profile));
-		const spec = cfg.models[mode];
-		if (spec) {
-			const i = spec.indexOf("/");
-			const model = i > 0 ? ctx.modelRegistry.find(spec.slice(0, i), spec.slice(i + 1)) : undefined;
-			const ok = model ? await pi.setModel(model) : false;
-			if (!ok) ctx.ui.notify(`Sub-Sub: cannot use ${spec} (unknown model or no key); keeping the current model.`, "warning");
+	/** Set the tools for the switch and the profile; save the switch when asked. */
+	function applyLibrary(ctx: ExtensionContext, on: boolean, save: boolean): void {
+		libraryChanges = on;
+		pi.setActiveTools(toolsFor(registered(), toolOpts()));
+		if (save) {
+			try {
+				saveConfig({ libraryChanges: on });
+			} catch (err) {
+				ctx.ui.notify(`Sub-Sub: the setting was not saved: ${(err as Error).message}`, "warning");
+			}
 		}
-		if (remember) pi.appendEntry(MODE_ENTRY, { mode });
-		applyTheme(ctx);
 		status(ctx);
 	}
 
+	async function applyModel(ctx: ExtensionContext): Promise<void> {
+		const spec = cfg.model;
+		if (!spec) return;
+		const i = spec.indexOf("/");
+		const model = i > 0 ? ctx.modelRegistry.find(spec.slice(0, i), spec.slice(i + 1)) : undefined;
+		const ok = model ? await pi.setModel(model) : false;
+		if (!ok) ctx.ui.notify(`Sub-Sub: cannot use ${spec} (unknown model or no key); keeping the current model.`, "warning");
+	}
+
 	pi.on("session_start", async (_event, ctx) => {
 		if (!bridge) {
 			const b = new Bridge();
@@ -270,14 +282,12 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 			bridge = b;
 			registerTools(b);
 		}
-		let restored: Mode | undefined;
-		for (const e of ctx.sessionManager.getBranch() as Array<{ customType?: string; data?: { mode?: unknown } }>) {
-			const m = e.customType === MODE_ENTRY ? e.data?.mode : undefined;
-			if (m === "librarian" || m === "researcher") restored = m;
-		}
 		// Judge light or dark before Sub-Sub changes the theme.
 		if (lookOn(ctx)) scheme ??= schemeFromAnsi(ctx.ui.theme.getFgAnsi("text"), ctx.ui.theme.name);
-		await applyMode(ctx, restored ?? (pi.getFlag("librarian") ? "librarian" : cfg.defaultMode), false);
+		await applyModel(ctx);
+		applyTheme(ctx);
+		// The flags change this run only; the saved setting stays.
+		applyLibrary(ctx, pi.getFlag("read-only") ? false : pi.getFlag("librarian") ? true : cfg.libraryChanges, false);
 		if (lookOn(ctx)) {
 			setHeader(ctx);
 			void refreshLibrary();
@@ -308,7 +318,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 	});
 
 	pi.on("before_agent_start", async (event, ctx) => ({
-		systemPrompt: `${event.systemPrompt}\n\n${systemAddition(mode, { ...cfg, profile }, ctx.cwd)}`,
+		systemPrompt: `${event.systemPrompt}\n\n${systemAddition({ ...cfg, profile }, ctx.cwd, libraryChanges)}`,
 	}));
 
 	/** Model test (subsub-bench): record every change instead of asking, and apply none. */
@@ -321,8 +331,9 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 		// Previews and the real call must see the same, repaired arguments.
 		const schema = schemas.get(name);
 		if (schema) coerceArgs(schema, input);
+		resolvePathArgs(name, input, (p) => normalizeToolPath(p, ctx.cwd));
 		if (SERVER_PREFIXES.some((p) => name.startsWith(p))) {
-			const why = unavailableReason(name, mode, registered(), profile);
+			const why = unavailableReason(name, registered(), toolOpts());
 			if (why) {
 				if (benchOut) record({ kind: "wrong_mode", tool: name, input });
 				return { block: true, reason: why };
@@ -436,19 +447,39 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 		};
 	}
 
+	const libraryNote = (on: boolean) =>
+		on
+			? "Sub-Sub: library changes on. Every change shows a preview first."
+			: "Sub-Sub: library changes off. Sub-Sub reads, searches and writes notes; it does not change the library.";
+
+	pi.registerCommand("library", {
+		description: "Sub-Sub: turn library changes on or off (/library on, /library off)",
+		handler: async (args, ctx) => {
+			const a = args.trim().toLowerCase();
+			const on = a === "on" ? true : a === "off" ? false : a === "" ? !libraryChanges : undefined;
+			if (on === undefined) {
+				ctx.ui.notify("Type /library on or /library off.", "warning");
+				return;
+			}
+			applyLibrary(ctx, on, true);
+			ctx.ui.notify(libraryNote(on), "info");
+		},
+	});
+
+	// The two modes before 0.10, kept so that old habits and notes still work.
 	pi.registerCommand("librarian", {
-		description: "Sub-Sub: switch to the librarian (changes the library, no web)",
+		description: "Sub-Sub: same as /library on",
 		handler: async (_args, ctx) => {
-			await applyMode(ctx, "librarian", true);
-			ctx.ui.notify("Sub-Sub: librarian mode", "info");
+			applyLibrary(ctx, true, true);
+			ctx.ui.notify(`${libraryNote(true)} (/librarian is now /library on.)`, "info");
 		},
 	});
 
 	pi.registerCommand("researcher", {
-		description: "Sub-Sub: switch to the researcher (search, notes; library read-only)",
+		description: "Sub-Sub: same as /library off",
 		handler: async (_args, ctx) => {
-			await applyMode(ctx, "researcher", true);
-			ctx.ui.notify("Sub-Sub: researcher mode", "info");
+			applyLibrary(ctx, false, true);
+			ctx.ui.notify(`${libraryNote(false)} (/researcher is now /library off.)`, "info");
 		},
 	});
 
@@ -471,7 +502,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 			} catch (err) {
 				saved = ` Not saved: ${(err as Error).message}`;
 			}
-			await applyMode(ctx, mode, false);
+			applyLibrary(ctx, libraryChanges, false);
 			ctx.ui.notify(`Sub-Sub: ${profileSpec(profile).label} profile (${profileSpec(profile).summary}).${saved}`, "info");
 		},
 	});
@@ -479,7 +510,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 	pi.registerCommand("subsub", {
 		description: "Sub-Sub: Zotero connection and library overview",
 		handler: async (_args, ctx) => {
-			const lines = [`Mode: ${mode}. Profile: ${profileSpec(profile).label}. Model: ${ctx.model?.id ?? "none"}.`];
+			const lines = [`Library changes: ${libraryChanges ? "on" : "off"}. Profile: ${profileSpec(profile).label}. Model: ${ctx.model?.id ?? "none"}.`];
 			if (!bridge) {
 				ctx.ui.notify(`${lines[0]}\nThe Zotero servers are not running. Try /reload.`, "warning");
 				return;

package/src/web.ts

@@ -23,7 +23,8 @@ import { extname, join, normalize, resolve, sep } from "node:path";
 import { StringDecoder } from "node:string_decoder";
 import { agentDirFor, chooseCwd, PACKAGE_DIR, packageVersion } from "./cli.ts";
 import { expand, loadConfig, saveConfig, type SubsubConfig, withStarbuck } from "./config.ts";
-import { TESTED_MODELS } from "./init.ts";
+import { TESTED_MODEL } from "./init.ts";
+import { obsidianRoot } from "./layout.ts";
 import { realish, within } from "./paths.ts";
 
 // ---------------------------------------------------------------- options
@@ -567,12 +568,9 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 					if (busy) return send(res, 409, { error: "busy" });
 					const body = await readJson(req);
 					const spec = `${String(body.provider)}/${String(body.id)}`;
-					// Sub-Sub keeps one model per mode: remember it for the current mode, then restart pi so the
-					// extension reads the new setting.
-					const mode = (subsubState as { mode?: string } | null)?.mode === "librarian" ? "librarian" : "researcher";
-					const models = { ...(loadConfig(env).models ?? {}), [mode]: spec };
-					if (body.both) models[mode === "librarian" ? "researcher" : "librarian"] = spec;
-					saveConfig({ models }, env);
+					// Remember the model (and drop the per-mode models of versions before 0.10), then restart pi
+					// so the extension reads the new setting.
+					saveConfig({ model: spec, models: undefined }, env);
 					await agent.request({ type: "set_model", provider: String(body.provider), modelId: String(body.id) });
 					await restartAgent();
 					return send(res, 200, { ok: true });
@@ -596,7 +594,7 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 					const body = await readJson(req);
 					saveApiKey(agentDir, String(body.provider ?? ""), String(body.key ?? ""));
 					const current = loadConfig(env);
-					if (body.provider === "opencode-go" && Object.keys(current.models ?? {}).length === 0) saveConfig({ models: TESTED_MODELS }, env);
+					if (body.provider === "opencode-go" && !current.model) saveConfig({ model: TESTED_MODEL, models: undefined }, env);
 					await restartAgent();
 					return send(res, 200, { ok: true });
 				}
@@ -610,7 +608,7 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 					if (!inside || !existsSync(target)) return send(res, 404, { error: "not found in the Sub-Sub folder" });
 					// Documents only: a file that the model wrote must never be run.
 					if (!OPENABLE.has(extname(target).toLowerCase())) return send(res, 403, { error: "Sub-Sub opens only notes and documents (.md, .txt, .pdf, .bib, .csv, .qmd)." });
-					if (vault && within(target, vault) && existsSync(join(vault, ".obsidian"))) openExternal(`obsidian://open?path=${encodeURIComponent(target)}`);
+					if (vault && within(target, vault) && obsidianRoot(vault)) openExternal(`obsidian://open?path=${encodeURIComponent(target)}`);
 					else openExternal(target);
 					return send(res, 200, { ok: true });
 				}

package/templates/Zotero agent.md

@@ -1,10 +1,10 @@
-Shared rules for the Sub-Sub librarian and researcher. Sub-Sub reads this file at the start of every session. Edit it to change how notes look or where they go. Tag list: [[Zotero tags]].
+Shared rules for Sub-Sub. Sub-Sub reads this file at the start of every session. Edit it to change how notes look or where they go. Tag list: [[Zotero tags]].
 
-## Roles
+## What Sub-Sub does
 
-- The librarian changes the library: tags, imports, metadata, PDFs, collections, notes in Zotero. It does not search the web.
-- The researcher searches PubMed and OpenAlex, reads the library and writes notes in this folder. It cannot change the library, with one exception: it can add a short linked note to an item (attach_note).
-- Hand-off: the researcher adds works to Inbox/Zotero import queue.md. The user ticks the lines. The librarian imports the ticked lines.
+- Sub-Sub reads the library, searches PubMed, Europe PMC and OpenAlex, and writes notes in this folder.
+- With library changes on, it also changes the library: tags, imports, metadata, PDFs, collections, notes in Zotero. Every change shows a preview and needs the user's yes.
+- Works found in a search are imported directly when the user chooses them. For a list to choose from later, Sub-Sub writes Inbox/Zotero import queue.md; the user ticks the lines and Sub-Sub imports them.
 
 ## Safety
 
@@ -85,5 +85,5 @@ created: YYYY-MM-DD
 - Structure: scope and question; synthesis by theme; disagreements; gaps; a table of included items (citekey, year, design, main finding, source: full text or abstract).
 - For a tag ("what does my library say about X"), include every item with the tag, one table row each. If an item is left out, say which and why.
 - Cite with [@citekey] after each claim. Every claim must come from an included item.
-- Works that are not in the library go in a separate section "Not in the library", with DOI or PMID, and are proposed for the import queue.
+- Works that are not in the library go in a separate section "Not in the library", with DOI or PMID, and are proposed for import.
 - Link existing literature notes as [[citekey]].

package/web/app.css

@@ -87,9 +87,8 @@
 	--warn: #d6a84e;
 }
 
-/* The mode sets the accent: blue for the researcher, rust for the librarian. */
+/* One accent, blue. Rust marks the switch that lets Sub-Sub change the library. */
 body { --accent: var(--researcher); --accent-soft: var(--researcher-soft); }
-body[data-mode="librarian"] { --accent: var(--librarian); --accent-soft: var(--librarian-soft); }
 
 /* ---------------------------------------------------------------- base */
 
@@ -220,27 +219,20 @@ select:hover { border-color: var(--rule-strong); }
 .brand .mark { color: var(--accent); }
 .brand .hyphen { color: var(--accent); }
 
-.modes {
-	display: inline-flex;
-	padding: 3px;
-	border-radius: 8px;
-	background: var(--surface);
+/* The library switch in the top bar: rust when Sub-Sub may change the library. */
+.lib-switch {
+	padding: 0.3rem 0.7rem 0.3rem 0.5rem;
 	border: 1px solid var(--rule);
-}
-.modes button {
-	padding: 0.28rem 0.8rem;
-	border-radius: 5px;
-	font-size: 0.86rem;
+	border-radius: 999px;
+	background: var(--surface);
+	font-size: 0.84rem;
 	color: var(--muted);
-	border: 0;
-}
-.modes button:hover { color: var(--text); }
-.modes button[aria-pressed="true"] {
-	background: var(--raised);
-	color: var(--accent);
-	font-weight: 600;
-	box-shadow: 0 1px 2px rgb(0 0 0 / 0.08), 0 0 0 1px var(--rule);
+	white-space: nowrap;
 }
+.lib-switch:hover { border-color: var(--rule-strong); color: var(--text); }
+.top .lib-switch input:checked + .track { background: var(--librarian); }
+body[data-library="on"] .lib-switch { color: var(--text); }
+body[data-library="off"] .top { border-top-color: var(--rule-strong); }
 
 .status {
 	margin-left: auto;
@@ -316,7 +308,8 @@ select:hover { border-color: var(--rule-strong); }
 	border-radius: 5px;
 }
 .actions button:hover { background: var(--bg); }
-.actions hr { border: 0; border-top: 1px solid var(--rule); margin: 0.4rem 0.5rem; }
+.actions .group { margin: 0.55rem 0 0.15rem; padding: 0 0.5rem; font-size: 0.74rem; font-weight: 600; color: var(--muted); }
+.actions .group:first-child { margin-top: 0; }
 
 .side .grow { flex: 1; display: flex; flex-direction: column; min-height: 8rem; }
 .side-subhead { display: flex; justify-content: space-between; align-items: baseline; padding-right: 0.3rem; }
@@ -862,7 +855,8 @@ body > p.empty { text-align: center; color: var(--muted); padding: 30vh 1rem 0;
 
 @media (max-width: 30rem) {
 	.brand > span { display: none; }
-	.modes button { padding: 0.28rem 0.6rem; }
+	.lib-switch > span:last-child { display: none; }
+	.lib-switch { padding: 0.35rem 0.5rem; }
 }
 
 @media (prefers-reduced-motion: reduce) {

package/web/app.js

@@ -7,7 +7,7 @@ import { ACTIONS, STRINGS, SUGGESTIONS, TOOLS } from "/i18n.js";
 // ---------------------------------------------------------------- state
 
 let lang = "en";
-let state = null; // Sub-Sub's own state: mode, profile, model, library, down
+let state = null; // Sub-Sub's own state: libraryChanges, profile, model, library, down
 let session = null; // pi's session state
 let busy = false;
 let commands = [];
@@ -169,13 +169,15 @@ function applyLang() {
 	updateLayoutUI();
 }
 
-function mode() {
-	return state?.mode === "librarian" ? "librarian" : "researcher";
+/** Library changes on (the default until the state arrives). */
+function libOn() {
+	return state?.libraryChanges !== false;
 }
 
 function renderHeader() {
-	document.body.dataset.mode = mode();
-	for (const b of document.querySelectorAll(".modes button")) b.setAttribute("aria-pressed", String(b.dataset.mode === mode()));
+	document.body.dataset.library = libOn() ? "on" : "off";
+	$("library-switch").checked = libOn();
+	$("library-switch-label").title = t("libraryHint");
 	if (state?.profile) $("profile").value = $("profile-side").value = state.profile;
 	$("starbuck-toggle").checked = Boolean(state?.starbuck);
 	$("starbuck-label").title = t("starbuckHint");
@@ -203,7 +205,7 @@ function renderHeader() {
 		lib.classList.add("down");
 	}
 	lib.title = txt.textContent;
-	$("input").placeholder = mode() === "librarian" ? t("placeholderLibrarian") : t("placeholderResearcher");
+	$("input").placeholder = libOn() ? t("placeholder") : t("placeholderReadOnly");
 	if (log.querySelector(".empty")) {
 		clearEmpty();
 		renderEmpty();
@@ -221,12 +223,17 @@ function hasCommand(name) {
 function renderActions() {
 	const box = $("actions");
 	box.replaceChildren();
-	const list = [...ACTIONS[mode()]].filter(
-		(a) => hasCommand(a.cmd) && (!a.needs || state?.[a.needs]) && (!a.profiles || a.profiles.includes(state?.profile ?? "scholar")),
-	);
-	for (const a of list) box.append(actionButton(a));
-	box.append(el("hr"));
-	for (const a of ACTIONS.both.filter((x) => hasCommand(x.cmd))) box.append(actionButton(a));
+	const shown = (a) =>
+		hasCommand(a.cmd) &&
+		(!a.changes || libOn()) &&
+		(!a.needs || state?.[a.needs]) &&
+		(!a.profiles || a.profiles.includes(state?.profile ?? "scholar"));
+	for (const [group, label] of [["research", "groupResearch"], ["library", "groupLibrary"]]) {
+		const list = ACTIONS[group].filter(shown);
+		if (!list.length) continue;
+		box.append(el("p", { class: "group", text: t(label) }));
+		for (const a of list) box.append(actionButton(a));
+	}
 }
 
 function actionButton(a) {
@@ -326,17 +333,17 @@ function exportConversation() {
 			if (text && text !== "/subsub-refresh") msgs.push(`### User\n\n${text}\n`);
 		} else if (m.classList.contains("assistant")) {
 			const text = Array.from(m.querySelectorAll(".md")).map((d) => d.innerText).join("\n\n");
-			if (text) msgs.push(`### Sub-Sub (${mode()})\n\n${text}\n`);
+			if (text) msgs.push(`### Sub-Sub\n\n${text}\n`);
 		}
 	}
 	if (!msgs.length) return;
 	const date = new Date().toISOString().slice(0, 10);
-	const content = `# Sub-Sub Research Session (${date})\n\n**Mode**: ${mode()}  \n**Profile**: ${state?.profile ?? "scholar"}  \n\n---\n\n${msgs.join("\n---\n\n")}`;
+	const content = `# Sub-Sub Research Session (${date})\n\n**Profile**: ${state?.profile ?? "scholar"}  \n\n---\n\n${msgs.join("\n---\n\n")}`;
 	const blob = new Blob([content], { type: "text/markdown;charset=utf-8" });
 	const url = URL.createObjectURL(blob);
 	const a = document.createElement("a");
 	a.href = url;
-	a.download = `Sub-Sub-${mode()}-${date}.md`;
+	a.download = `Sub-Sub-${date}.md`;
 	document.body.append(a);
 	a.click();
 	a.remove();
@@ -369,14 +376,14 @@ function clearEmpty() {
 
 function renderEmpty() {
 	if (log.children.length) return;
-	const s = SUGGESTIONS[mode()][lang] ?? SUGGESTIONS[mode()].en;
+	const s = SUGGESTIONS.filter((x) => !x.changes || libOn()).map((x) => x[lang] ?? x.en);
 	log.append(
 		el(
 			"div",
 			{ class: "empty" },
-			el("p", { class: "kicker", text: t(mode()) }),
+			el("p", { class: "kicker", text: libOn() ? t("assistant") : `${t("assistant")} · ${t("readOnly")}` }),
 			el("h1", { text: t("emptyTitle") }),
-			el("p", { class: "lede", text: t(mode() === "librarian" ? "emptyLibrarian" : "emptyResearcher") }),
+			el("p", { class: "lede", text: t(libOn() ? "emptyOn" : "emptyOff") }),
 			el(
 				"div",
 				{ class: "chips" },
@@ -708,7 +715,8 @@ function onEvent(ev) {
 			break;
 		case "extension_ui_request":
 			if (ev.method === "notify") {
-				if (/^Sub-Sub: (librarian|researcher) mode$/.test(ev.message)) break;
+				// The switch shows the state; these confirmations would only repeat it.
+				if (/^Sub-Sub: library changes (on|off)\./.test(ev.message)) break;
 				note(ev.message, ev.notifyType === "error" ? "error" : ev.notifyType === "warning" ? "warning" : "info", /\n/.test(ev.message));
 			} else {
 				dialogs.push(ev);
@@ -893,7 +901,7 @@ async function openModels() {
 					onclick: async () => {
 						if (busy) return note(t("busy"), "warning");
 						try {
-							await api("/api/model", { provider: m.provider, id: m.id, both: $("model-both").checked });
+							await api("/api/model", { provider: m.provider, id: m.id });
 							dlg.close();
 						} catch (err) {
 							note(err.message, "error");
@@ -935,15 +943,17 @@ function autosize() {
 
 async function sendMessage(text) {
 	const message = text.trim();
-	if (!message) return;
+	if (!message) return false;
 	const name = message.startsWith("/") ? message.slice(1).split(/\s/)[0] : "";
 	const isTemplate = name && commands.some((c) => c.name === name && c.source === "prompt");
 	if (isTemplate) sentTemplates.push(message);
 	try {
 		await api("/api/prompt", { message });
+		return true;
 	} catch (err) {
 		if (isTemplate) sentTemplates.pop();
 		note(err.message, "error");
+		return false;
 	}
 }
 
@@ -1032,11 +1042,12 @@ $("new-btn").addEventListener("click", async () => {
 		note(err.message, "error");
 	}
 });
-for (const b of document.querySelectorAll(".modes button")) {
-	b.addEventListener("click", () => {
-		if (b.dataset.mode !== mode()) sendMessage(`/${b.dataset.mode}`);
-	});
-}
+$("library-switch").addEventListener("change", async (e) => {
+	const on = e.target.checked;
+	if (on === libOn()) return;
+	const ok = await sendMessage(`/library ${on ? "on" : "off"}`);
+	if (!ok) e.target.checked = libOn();
+});
 $("profile").addEventListener("change", (e) => sendMessage(`/profile ${e.target.value}`));
 $("starbuck-toggle").addEventListener("change", async (e) => {
 	const on = e.target.checked;