@tiagojct/subsub 0.10.0 → 0.11.1

package/src/web.ts

@@ -22,8 +22,8 @@ import { homedir } from "node:os";
 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_MODEL } from "./init.ts";
+import { configPath, expand, loadConfig, readConfigFile, saveConfig, type SubsubConfig, withStarbuck } from "./config.ts";
+import { TESTED_MODELS } from "./init.ts";
 import { obsidianRoot } from "./layout.ts";
 import { realish, within } from "./paths.ts";
 
@@ -336,7 +336,7 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 	env.PI_CODING_AGENT_DIR = agentDir; // for the session list below
 
 	const childEnv: NodeJS.ProcessEnv = { ...env, SUBSUB_WEB: "1" };
-	const baseArgs = [bin, "--mode", "rpc", "--here", ...(opts.librarian ? ["--librarian"] : []), ...opts.piArgs];
+	const baseArgs = [bin, "--mode", "rpc", "--here", ...opts.piArgs];
 	const agent = new Agent(process.execPath, baseArgs, cwd, childEnv);
 
 	// ---- state shared with the pages
@@ -346,8 +346,20 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 	let busy = false;
 	let lastSeen = Date.now();
 	let restarting = false;
+	let switching = false; // a restart for a new model, add-on or key is under way
+	let lastSession: string | undefined;
 	let refreshAfterRun = false;
 
+	/** The mode to start pi in: the one on screen, else the --librarian flag. A session's own mode still wins. */
+	function modeArgs(): string[] {
+		const shown = (subsubState as { mode?: string } | null)?.mode;
+		if (shown === "librarian" || shown === "researcher") return [`--${shown}`];
+		return opts.librarian ? ["--librarian"] : [];
+	}
+	function startAgent(): void {
+		agent.start([...modeArgs(), ...(lastSession && existsSync(lastSession) ? ["--session", lastSession] : [])]);
+	}
+
 	function broadcast(ev: unknown): void {
 		const data = `data: ${JSON.stringify(ev)}\n\n`;
 		for (const c of clients) c.write(data);
@@ -384,12 +396,13 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 	agent.onExit = (code) => {
 		if (restarting) return;
 		busy = false;
+		for (const id of dialogs.keys()) broadcast({ type: "dialog_closed", id });
 		dialogs.clear();
 		broadcast({ type: "agent_exit", code, stderr: agent.stderr.slice(-1500) });
 		log(`pi stopped (code ${code}).\n${agent.stderr.slice(-2000)}`);
 	};
 
-	agent.start();
+	startAgent();
 
 	async function snapshot(): Promise<Record<string, unknown>> {
 		const out: Record<string, unknown> = {
@@ -403,6 +416,7 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 		};
 		try {
 			out.session = await agent.request({ type: "get_state" });
+			lastSession = (out.session as { sessionFile?: string }).sessionFile ?? lastSession;
 			out.messages = ((await agent.request({ type: "get_messages" })) as { messages: unknown[] }).messages;
 			out.commands = ((await agent.request({ type: "get_commands" })) as { commands: unknown[] }).commands;
 		} catch (err) {
@@ -422,11 +436,10 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 	}
 
 	async function restartAgent(): Promise<void> {
-		let sessionFile: string | undefined;
 		try {
-			sessionFile = ((await agent.request({ type: "get_state" })) as { sessionFile?: string }).sessionFile;
+			lastSession = ((await agent.request({ type: "get_state" })) as { sessionFile?: string }).sessionFile ?? lastSession;
 		} catch {
-			/* start fresh */
+			/* keep the last one known */
 		}
 		restarting = true;
 		for (const id of dialogs.keys()) broadcast({ type: "dialog_closed", id });
@@ -434,10 +447,23 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 		await agent.stop();
 		restarting = false;
 		busy = false;
-		agent.start(sessionFile && existsSync(sessionFile) ? ["--session", sessionFile] : []);
+		startAgent();
 		broadcast(await snapshot());
 	}
 
+	/** Run a change that restarts pi, one at a time. */
+	async function withRestart(res: ServerResponse, change: () => Promise<void> | void): Promise<void> {
+		if (busy || switching) return send(res, 409, { error: "busy" });
+		switching = true;
+		try {
+			await change();
+			await restartAgent();
+		} finally {
+			switching = false;
+		}
+		return send(res, 200, { ok: true });
+	}
+
 	// ---- idle stop
 	let closing = false;
 	const idleTimer = setInterval(() => {
@@ -565,38 +591,43 @@ export async function startWeb(opts: WebOptions, env: NodeJS.ProcessEnv = proces
 					return send(res, 200, { models: Array.isArray(data) ? data : (data.models ?? []) });
 				}
 				case "POST /api/model": {
-					if (busy) return send(res, 409, { error: "busy" });
+					if (busy || switching) return send(res, 409, { error: "busy" });
 					const body = await readJson(req);
 					const spec = `${String(body.provider)}/${String(body.id)}`;
-					// 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 });
+					// Remember the model for the current mode (or both), then restart pi so the extension reads it.
+					const mode = (subsubState as { mode?: string } | null)?.mode === "librarian" ? "librarian" : "researcher";
+					// Start from what the file says, not from the built-in defaults, so a model the user never chose is not saved.
+					const raw = readConfigFile(configPath(env));
+					const saved = raw.models && typeof raw.models === "object" ? (raw.models as Record<string, string>) : typeof raw.model === "string" && raw.model ? { librarian: raw.model, researcher: raw.model } : {};
+					const models: Record<string, string> = { ...saved, [mode]: spec };
+					if (body.both) models[mode === "librarian" ? "researcher" : "librarian"] = spec;
+					saveConfig({ models, model: undefined }, env);
+					return withRestart(res, async () => {
+						await agent.request({ type: "set_model", provider: String(body.provider), modelId: String(body.id) });
+					});
 				}
 				case "POST /api/addons": {
 					// Turn an add-on on or off, then restart pi so the extension starts or stops its server.
-					if (busy) return send(res, 409, { error: "busy" });
+					if (busy || switching) return send(res, 409, { error: "busy" });
 					const body = await readJson(req);
 					if (typeof body.starbuck !== "boolean") return send(res, 400, { error: "starbuck must be true or false" });
-					saveConfig({ addons: withStarbuck(loadConfig(env).addons, body.starbuck) }, env);
-					await restartAgent();
-					return send(res, 200, { ok: true });
+					return withRestart(res, () => {
+						saveConfig({ addons: withStarbuck(loadConfig(env).addons, body.starbuck) }, env);
+					});
 				}
 				case "POST /api/restart":
-					if (agent.proc) return send(res, 200, { ok: true });
-					agent.start();
+					if (agent.proc || switching) return send(res, 200, { ok: true });
+					startAgent();
 					broadcast(await snapshot());
 					return send(res, 200, { ok: true });
 				case "POST /api/login": {
-					if (busy) return send(res, 409, { error: "busy" });
+					if (busy || switching) return send(res, 409, { error: "busy" });
 					const body = await readJson(req);
-					saveApiKey(agentDir, String(body.provider ?? ""), String(body.key ?? ""));
-					const current = loadConfig(env);
-					if (body.provider === "opencode-go" && !current.model) saveConfig({ model: TESTED_MODEL, models: undefined }, env);
-					await restartAgent();
-					return send(res, 200, { ok: true });
+					return withRestart(res, () => {
+						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, model: undefined }, env);
+					});
 				}
 				case "POST /api/open": {
 					const body = await readJson(req);

package/templates/Zotero agent.md

@@ -2,8 +2,8 @@ Shared rules for Sub-Sub. Sub-Sub reads this file at the start of every session.
 
 ## What Sub-Sub does
 
-- 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.
+- Sub-Sub has two modes. The Librarian manages the library: tags, imports, metadata, PDFs, collections, notes in Zotero. The Researcher reads the library, searches PubMed, Europe PMC and OpenAlex, writes notes in this folder, and also does everything the Librarian does.
+- Every library 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

package/web/app.css

@@ -87,8 +87,9 @@
 	--warn: #d6a84e;
 }
 
-/* One accent, blue. Rust marks the switch that lets Sub-Sub change the library. */
+/* The mode sets the accent: blue for the Researcher, rust for the Librarian. */
 body { --accent: var(--researcher); --accent-soft: var(--researcher-soft); }
+body[data-mode="librarian"] { --accent: var(--librarian); --accent-soft: var(--librarian-soft); }
 
 /* ---------------------------------------------------------------- base */
 
@@ -219,20 +220,27 @@ select:hover { border-color: var(--rule-strong); }
 .brand .mark { color: var(--accent); }
 .brand .hyphen { color: var(--accent); }
 
-/* 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);
-	border-radius: 999px;
+.modes {
+	display: inline-flex;
+	padding: 3px;
+	border-radius: 8px;
 	background: var(--surface);
-	font-size: 0.84rem;
+	border: 1px solid var(--rule);
+}
+.modes button {
+	padding: 0.28rem 0.8rem;
+	border-radius: 5px;
+	font-size: 0.86rem;
 	color: var(--muted);
-	white-space: nowrap;
+	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);
 }
-.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;
@@ -855,8 +863,7 @@ body > p.empty { text-align: center; color: var(--muted); padding: 30vh 1rem 0;
 
 @media (max-width: 30rem) {
 	.brand > span { display: none; }
-	.lib-switch > span:last-child { display: none; }
-	.lib-switch { padding: 0.35rem 0.5rem; }
+	.modes button { padding: 0.28rem 0.6rem; }
 }
 
 @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: libraryChanges, profile, model, library, down
+let state = null; // Sub-Sub's own state: mode, profile, model, library, down
 let session = null; // pi's session state
 let busy = false;
 let commands = [];
@@ -169,15 +169,16 @@ function applyLang() {
 	updateLayoutUI();
 }
 
-/** Library changes on (the default until the state arrives). */
-function libOn() {
-	return state?.libraryChanges !== false;
+function mode() {
+	return state?.mode === "librarian" ? "librarian" : "researcher";
 }
 
 function renderHeader() {
-	document.body.dataset.library = libOn() ? "on" : "off";
-	$("library-switch").checked = libOn();
-	$("library-switch-label").title = t("libraryHint");
+	document.body.dataset.mode = mode();
+	for (const b of document.querySelectorAll(".modes button")) {
+		b.setAttribute("aria-pressed", String(b.dataset.mode === mode()));
+		b.title = t(b.dataset.mode === "librarian" ? "librarianHint" : "researcherHint");
+	}
 	if (state?.profile) $("profile").value = $("profile-side").value = state.profile;
 	$("starbuck-toggle").checked = Boolean(state?.starbuck);
 	$("starbuck-label").title = t("starbuckHint");
@@ -205,7 +206,7 @@ function renderHeader() {
 		lib.classList.add("down");
 	}
 	lib.title = txt.textContent;
-	$("input").placeholder = libOn() ? t("placeholder") : t("placeholderReadOnly");
+	$("input").placeholder = mode() === "librarian" ? t("placeholderLibrarian") : t("placeholderResearcher");
 	if (log.querySelector(".empty")) {
 		clearEmpty();
 		renderEmpty();
@@ -225,10 +226,11 @@ function renderActions() {
 	box.replaceChildren();
 	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 groups = mode() === "librarian" ? [["library", "groupLibrary"]] : [["research", "groupResearch"], ["library", "groupLibrary"]];
+	for (const [group, label] of groups) {
 		const list = ACTIONS[group].filter(shown);
 		if (!list.length) continue;
 		box.append(el("p", { class: "group", text: t(label) }));
@@ -376,14 +378,14 @@ function clearEmpty() {
 
 function renderEmpty() {
 	if (log.children.length) return;
-	const s = SUGGESTIONS.filter((x) => !x.changes || libOn()).map((x) => x[lang] ?? x.en);
+	const s = SUGGESTIONS[mode()].map((x) => x[lang] ?? x.en);
 	log.append(
 		el(
 			"div",
 			{ class: "empty" },
-			el("p", { class: "kicker", text: libOn() ? t("assistant") : `${t("assistant")} · ${t("readOnly")}` }),
+			el("p", { class: "kicker", text: t(mode()) }),
 			el("h1", { text: t("emptyTitle") }),
-			el("p", { class: "lede", text: t(libOn() ? "emptyOn" : "emptyOff") }),
+			el("p", { class: "lede", text: t(mode() === "librarian" ? "emptyLibrarian" : "emptyResearcher") }),
 			el(
 				"div",
 				{ class: "chips" },
@@ -648,6 +650,7 @@ function onEvent(ev) {
 			setBusy(!!ev.busy);
 			dialogs.length = 0;
 			for (const d of ev.dialogs ?? []) dialogs.push(d);
+			dropStaleDialog();
 			nextDialog();
 			loadSessions();
 			break;
@@ -715,8 +718,8 @@ function onEvent(ev) {
 			break;
 		case "extension_ui_request":
 			if (ev.method === "notify") {
-				// The switch shows the state; these confirmations would only repeat it.
-				if (/^Sub-Sub: library changes (on|off)\./.test(ev.message)) break;
+				// The mode buttons show the mode; these confirmations would only repeat it.
+				if (/^Sub-Sub: (Librarian|Researcher) mode\./.test(ev.message)) break;
 				note(ev.message, ev.notifyType === "error" ? "error" : ev.notifyType === "warning" ? "warning" : "info", /\n/.test(ev.message));
 			} else {
 				dialogs.push(ev);
@@ -741,6 +744,8 @@ function onEvent(ev) {
 			break;
 		case "agent_exit":
 			setBusy(false);
+			dialogs.length = 0;
+			dropStaleDialog();
 			log.append(
 				el(
 					"div",
@@ -774,6 +779,7 @@ function dialogTitle(d) {
 	if ((m = raw.match(/^apply (\S+)\?$/))) return t("apply", { x: toolLabel(m[1]) });
 	if ((m = raw.match(/^run (\S+)\?$/))) return t("run", { x: toolLabel(m[1]) });
 	if (raw === "undo?") return t("undo");
+	if ((m = raw.match(/^write the Starbuck report to (.+)\?$/))) return t("starbuckReport", { x: m[1] });
 	if ((m = raw.match(/^(write|edit) (.+)\?$/))) return t("write", { x: m[2] });
 	return raw;
 }
@@ -795,6 +801,14 @@ function paintPreview(pre, text) {
 	}
 }
 
+/** Close the dialog on screen if Sub-Sub no longer waits for it (pi restarted, or another tab answered). */
+function dropStaleDialog() {
+	if (shownDialog && !dialogs.some((d) => d.id === shownDialog.id)) {
+		shownDialog = null;
+		if ($("dialog").open) $("dialog").close();
+	}
+}
+
 function nextDialog() {
 	if (shownDialog || !dialogs.length) return;
 	const d = dialogs[0];
@@ -819,6 +833,7 @@ function nextDialog() {
 			note(err.message, "error");
 		}
 	};
+	$("dialog-form").onsubmit = (e) => e.preventDefault();
 	if (d.method === "confirm") {
 		buttons.append(
 			el("span", { class: "modal-shortcut-hint", text: t("confirmShortcutHint") }),
@@ -832,14 +847,25 @@ function nextDialog() {
 		const field = d.method === "editor" ? el("textarea", {}) : el("input", { type: "text", placeholder: d.placeholder ?? "" });
 		field.value = d.prefill ?? "";
 		body.append(field);
+		$("dialog-form").onsubmit = (e) => {
+			e.preventDefault();
+			answer({ value: field.value });
+		};
 		buttons.append(
 			el("button", { type: "button", class: "ghost", text: t("cancel"), onclick: () => answer({ cancelled: true }) }),
 			el("button", { type: "button", class: "primary", text: t("ok"), onclick: () => answer({ value: field.value }) }),
 		);
 	}
+	// Keys typed just before the dialog opened (the user was writing a message) must not answer it.
+	const openedAt = performance.now();
+	const yesKeys = lang === "pt" ? ["s", "S", "y", "Y"] : ["y", "Y"];
 	dlg.onkeydown = (e) => {
 		if (d.method === "confirm") {
-			if (e.key === "y" || e.key === "Y" || (e.key === "Enter" && (e.metaKey || e.ctrlKey))) {
+			if (e.repeat || performance.now() - openedAt < 700) {
+				if (!e.metaKey && !e.ctrlKey && e.key.length === 1) e.preventDefault();
+				return;
+			}
+			if (yesKeys.includes(e.key) || (e.key === "Enter" && (e.metaKey || e.ctrlKey))) {
 				e.preventDefault();
 				answer({ confirmed: true });
 			} else if (e.key === "n" || e.key === "N") {
@@ -865,6 +891,9 @@ function showModels() {
 
 async function openModels() {
 	const dlg = $("model-dialog");
+	// Name the mode the choice is saved for; "both modes" is a choice for this time only.
+	dlg.querySelector("h2").textContent = t(mode() === "librarian" ? "modelTitleLibrarian" : "modelTitleResearcher");
+	$("model-both").checked = false;
 	const ul = $("models");
 	ul.replaceChildren(el("li", { class: "empty-models", text: "…" }));
 	const sel = $("login-provider");
@@ -901,7 +930,7 @@ async function openModels() {
 					onclick: async () => {
 						if (busy) return note(t("busy"), "warning");
 						try {
-							await api("/api/model", { provider: m.provider, id: m.id });
+							await api("/api/model", { provider: m.provider, id: m.id, both: $("model-both").checked });
 							dlg.close();
 						} catch (err) {
 							note(err.message, "error");
@@ -1042,12 +1071,11 @@ $("new-btn").addEventListener("click", async () => {
 		note(err.message, "error");
 	}
 });
-$("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();
-});
+for (const b of document.querySelectorAll(".modes button")) {
+	b.addEventListener("click", () => {
+		if (b.dataset.mode !== mode()) sendMessage(`/${b.dataset.mode}`);
+	});
+}
 $("profile").addEventListener("change", (e) => sendMessage(`/profile ${e.target.value}`));
 $("starbuck-toggle").addEventListener("change", async (e) => {
 	const on = e.target.checked;

package/web/i18n.js

@@ -21,6 +21,7 @@ export const STRINGS = {
 		run: "Run {x}?",
 		undo: "Undo this change?",
 		write: "Allow writing to {x}?",
+		starbuckReport: "Write the Starbuck report to {x}?",
 		approveHint: "Nothing changes until you select Yes.",
 		zoteroChecking: "Zotero: checking",
 		zoteroDown: "Zotero is not running",
@@ -39,6 +40,8 @@ export const STRINGS = {
 		connectModel: "Connect a model to start.",
 		connect: "Connect",
 		modelTitle: "Model",
+		modelTitleResearcher: "Model for the Researcher",
+		modelTitleLibrarian: "Model for the Librarian",
 		chooseModel: "Choose a model",
 		noModels: "No models yet. Connect a provider below.",
 		connectProvider: "Connect a provider",
@@ -79,14 +82,15 @@ export const STRINGS = {
 		starbuckOff: "Reference checks are off.",
 		composerHint: "Enter sends. Shift+Enter starts a new line. / shows the commands.",
 		previewKicker: "Preview",
-		assistant: "Assistant",
-		readOnly: "Read only",
-		libraryChanges: "Library changes",
-		libraryHint: "On: Sub-Sub can import, tag and repair items, and every change shows a preview first. Off: it only reads, searches and writes notes.",
-		placeholder: "Ask about your library, or type / for commands",
-		placeholderReadOnly: "Ask about your library (read only), or type / for commands",
-		emptyOn: "Sub-Sub reads your library, searches PubMed, Europe PMC and OpenAlex, and writes notes. It also imports and tags: every change shows a preview, and nothing changes until you approve it.",
-		emptyOff: "Library changes are off. Sub-Sub reads your library, searches and writes notes, but does not change the library. Turn them on with the switch at the top.",
+		researcher: "Researcher",
+		librarian: "Librarian",
+		researcherHint: "Searches, reads and writes notes, and also changes the library (every change is previewed).",
+		librarianHint: "Manages the library only: tags, imports, metadata, PDFs. No literature search.",
+		placeholderResearcher: "Ask about your library or the literature, or type / for commands",
+		placeholderLibrarian: "Ask for a change to your library, or type / for commands",
+		emptyResearcher: "The Researcher reads your library, searches PubMed, Europe PMC and OpenAlex and writes notes. It also imports, tags and repairs items: every change shows a preview, and nothing changes until you approve it.",
+		emptyLibrarian: "The Librarian manages your library: tags, imports, metadata and PDFs. Every change shows a preview, and nothing changes until you approve it. For searches and notes, switch to the Researcher.",
+		forBoth: "Use it in both modes",
 		groupResearch: "Research",
 		groupLibrary: "Library",
 	},
@@ -109,6 +113,7 @@ export const STRINGS = {
 		run: "Executar {x}?",
 		undo: "Desfazer esta alteração?",
 		write: "Permitir escrever em {x}?",
+		starbuckReport: "Escrever o relatório do Starbuck em {x}?",
 		approveHint: "Nada muda até selecionar Sim.",
 		zoteroChecking: "Zotero: a verificar",
 		zoteroDown: "O Zotero não está aberto",
@@ -127,6 +132,8 @@ export const STRINGS = {
 		connectModel: "Ligue um modelo para começar.",
 		connect: "Ligar",
 		modelTitle: "Modelo",
+		modelTitleResearcher: "Modelo do Investigador",
+		modelTitleLibrarian: "Modelo do Bibliotecário",
 		chooseModel: "Escolha um modelo",
 		noModels: "Ainda não há modelos. Ligue um fornecedor abaixo.",
 		connectProvider: "Ligar um fornecedor",
@@ -160,21 +167,22 @@ export const STRINGS = {
 		searchConversations: "Procurar conversas",
 		export: "Exportar",
 		exportTitle: "Exportar conversa",
-		confirmShortcutHint: "Y para aprovar, N para recusar",
+		confirmShortcutHint: "S para aprovar, N para recusar",
 		starbuck: "Verificação de referências (Starbuck)",
 		starbuckHint: "Verifica se as obras citadas existem, correspondem à citação e não foram retratadas. O Sub-Sub reinicia quando muda esta opção.",
 		starbuckOn: "A verificação de referências está ligada. Use /verify, ou peça ao investigador que verifique as referências de um manuscrito.",
 		starbuckOff: "A verificação de referências está desligada.",
 		composerHint: "Enter envia. Shift+Enter muda de linha. / mostra os comandos.",
 		previewKicker: "Pré-visualização",
-		assistant: "Assistente",
-		readOnly: "Só leitura",
-		libraryChanges: "Alterações à biblioteca",
-		libraryHint: "Ligado: o Sub-Sub pode importar, etiquetar e corrigir itens, e cada alteração mostra primeiro uma pré-visualização. Desligado: só lê, pesquisa e escreve notas.",
-		placeholder: "Pergunte sobre a sua biblioteca, ou escreva / para ver os comandos",
-		placeholderReadOnly: "Pergunte sobre a sua biblioteca (só leitura), ou escreva / para ver os comandos",
-		emptyOn: "O Sub-Sub lê a sua biblioteca, pesquisa na PubMed, Europe PMC e OpenAlex e escreve notas. Também importa e etiqueta: cada alteração mostra uma pré-visualização, e nada muda até a aprovar.",
-		emptyOff: "As alterações à biblioteca estão desligadas. O Sub-Sub lê a biblioteca, pesquisa e escreve notas, mas não altera a biblioteca. Ligue-as no interruptor no topo.",
+		researcher: "Investigador",
+		librarian: "Bibliotecário",
+		researcherHint: "Pesquisa, lê e escreve notas, e também altera a biblioteca (cada alteração é pré-visualizada).",
+		librarianHint: "Só gere a biblioteca: etiquetas, importações, metadados, PDF. Não pesquisa a literatura.",
+		placeholderResearcher: "Pergunte sobre a sua biblioteca ou a literatura, ou escreva / para ver os comandos",
+		placeholderLibrarian: "Peça uma alteração à biblioteca, ou escreva / para ver os comandos",
+		emptyResearcher: "O Investigador lê a sua biblioteca, pesquisa na PubMed, Europe PMC e OpenAlex e escreve notas. Também importa, etiqueta e corrige itens: cada alteração mostra uma pré-visualização, e nada muda até a aprovar.",
+		emptyLibrarian: "O Bibliotecário gere a sua biblioteca: etiquetas, importações, metadados e PDF. Cada alteração mostra uma pré-visualização, e nada muda até a aprovar. Para pesquisar e escrever notas, mude para o Investigador.",
+		forBoth: "Usar nos dois modos",
 		groupResearch: "Investigação",
 		groupLibrary: "Biblioteca",
 	},
@@ -182,7 +190,7 @@ export const STRINGS = {
 
 /**
  * Buttons in the side panel, in two groups. `fill` puts the command in the message box; `send` runs it at once.
- * `changes`: shown only with library changes on. `profiles` and `needs` limit a button to some profiles or add-ons.
+ * The Librarian sees the Library group only. `profiles` and `needs` limit a button to some profiles or add-ons.
  */
 export const ACTIONS = {
 	research: [
@@ -200,21 +208,27 @@ export const ACTIONS = {
 	],
 	library: [
 		{ cmd: "subsub", send: true, en: "Library overview", pt: "Resumo da biblioteca" },
-		{ cmd: "tag-batch 10", send: true, changes: true, en: "Tag 10 items", pt: "Etiquetar 10 itens" },
-		{ cmd: "clean-tags", send: true, changes: true, en: "Clean up tags", pt: "Limpar etiquetas" },
-		{ cmd: "import-queue", send: true, changes: true, en: "Import the queue", pt: "Importar a fila" },
+		{ cmd: "tag-batch 10", send: true, en: "Tag 10 items", pt: "Etiquetar 10 itens" },
+		{ cmd: "clean-tags", send: true, en: "Clean up tags", pt: "Limpar etiquetas" },
+		{ cmd: "import-queue", send: true, en: "Import the queue", pt: "Importar a fila" },
 		{ cmd: "history", send: true, en: "Recent changes", pt: "Alterações recentes" },
-		{ cmd: "undo", send: true, changes: true, en: "Undo the last change", pt: "Desfazer a última alteração" },
+		{ cmd: "undo", send: true, en: "Undo the last change", pt: "Desfazer a última alteração" },
 	],
 };
 
-/** Suggestions on an empty conversation. `changes`: only with library changes on. */
-export const SUGGESTIONS = [
-	{ en: "What do I have on ", pt: "O que tenho sobre " },
-	{ en: "Find recent systematic reviews on ", pt: "Procure revisões sistemáticas recentes sobre " },
-	{ en: "Tag the next 10 items without a topic", pt: "Etiquete os próximos 10 itens sem tópico", changes: true },
-	{ en: "Import 10.1136/bmj.n71", pt: "Importe 10.1136/bmj.n71", changes: true },
-];
+/** Suggestions on an empty conversation, per mode. */
+export const SUGGESTIONS = {
+	researcher: [
+		{ en: "What do I have on ", pt: "O que tenho sobre " },
+		{ en: "Find recent systematic reviews on ", pt: "Procure revisões sistemáticas recentes sobre " },
+		{ en: "Which items have no PDF?", pt: "Que itens não têm PDF?" },
+	],
+	librarian: [
+		{ en: "Tag the next 10 items without a topic", pt: "Etiquete os próximos 10 itens sem tópico" },
+		{ en: "Import 10.1136/bmj.n71", pt: "Importe 10.1136/bmj.n71" },
+		{ en: "Find duplicates", pt: "Procure duplicados" },
+	],
+};
 
 /** Plain names for the tools, shown while they run. Unknown tools keep their name. */
 export const TOOLS = {

package/web/index.html

@@ -10,12 +10,15 @@
 	<link rel="stylesheet" href="/app.css">
 	<script type="module" src="/app.js"></script>
 </head>
-<body data-library="on">
+<body data-mode="researcher">
 	<div class="app">
 		<header class="top">
 			<button type="button" class="icon-btn" id="menu-btn" aria-controls="side" aria-expanded="true" data-i18n-title="menu" title="Menu"><svg viewBox="0 0 24 24" aria-hidden="true"><rect x="3.5" y="4.5" width="17" height="15" rx="2"/><path d="M9.5 4.5v15"/></svg><span class="visually-hidden" data-i18n="menu">Menu</span></button>
 			<div class="brand"><svg class="mark" viewBox="0 0 24 24" aria-hidden="true"><rect x="3" y="5" width="18" height="14" rx="1.5"/><path d="M6.5 9h7M6.5 12h11"/><circle cx="12" cy="16" r="1.1"/></svg><span>Sub<span class="hyphen">-</span>Sub</span></div>
-			<label class="switch lib-switch" id="library-switch-label"><input type="checkbox" role="switch" id="library-switch" checked><span class="track" aria-hidden="true"></span><span data-i18n="libraryChanges">Library changes</span></label>
+			<div class="modes" role="group" aria-label="Mode">
+				<button type="button" data-mode="researcher" aria-pressed="true" data-i18n="researcher">Researcher</button>
+				<button type="button" data-mode="librarian" aria-pressed="false" data-i18n="librarian">Librarian</button>
+			</div>
 			<div class="status">
 				<span class="library" id="library"><span class="library-dot" aria-hidden="true"></span><span class="library-text"></span></span>
 				<label class="profile wide-only"><span class="visually-hidden" data-i18n="profile">Profile</span>
@@ -107,6 +110,7 @@
 			<button type="button" class="link-btn" id="model-close" data-i18n="close">Close</button>
 		</div>
 		<h3 data-i18n="chooseModel">Choose a model</h3>
+		<label class="check"><input type="checkbox" id="model-both"> <span data-i18n="forBoth">Use it in both modes</span></label>
 		<ul class="models" id="models"></ul>
 		<h3 data-i18n="connectProvider">Connect a provider</h3>
 		<form id="login-form" class="login">