@tiagojct/subsub 0.9.1 → 0.11.0

package/src/look.ts

@@ -4,7 +4,6 @@
  * Only used when Sub-Sub runs as its own command in the terminal UI.
  */
 
-import type { Mode } from "./config.ts";
 
 export const BANNER = [
 	"╔═╗╦ ╦╔╗    ╔═╗╦ ╦╔╗",
@@ -25,7 +24,8 @@ export const QUOTES: Array<[string, string]> = [
 	["Call me Ishmael.", "Moby-Dick, ch. 1"],
 ];
 
-export const MODE_THEMES: Record<Mode, string> = { librarian: "subsub-try-works", researcher: "subsub-glauca" };
+/** The terminal theme, without -dark/-light. subsub-try-works is the other one shipped. */
+export const DEFAULT_THEME = "subsub-glauca";
 
 export type Scheme = "dark" | "light";
 
@@ -47,8 +47,7 @@ export function schemeFromAnsi(fgAnsi: string, themeName?: string): Scheme {
 	return "dark";
 }
 
-export function themeName(mode: Mode, scheme: Scheme, map: Partial<Record<Mode, string>> = MODE_THEMES): string | undefined {
-	const base = map[mode];
+export function themeName(scheme: Scheme, base: string | undefined = DEFAULT_THEME): string | undefined {
 	return base ? `${base}-${scheme}` : undefined;
 }
 
@@ -84,7 +83,7 @@ export function libraryLine(st: LibraryState): { text: string; warn: boolean } {
 /** Header lines. Each line is cut to the width before it is coloured. */
 export function headerLines(opts: {
 	version: string;
-	mode: Mode;
+	mode: "librarian" | "researcher";
 	/** Profile label, e.g. "Scholar". */
 	profile?: string;
 	model: string;

package/src/profiles.ts

@@ -5,7 +5,7 @@
  *
  * - reader:  explains each step; reading notes are quotes with page numbers and
  *            questions; no summaries or syntheses; no bulk library changes.
- * - scholar: literature notes, syntheses, searches, alerts; the full librarian.
+ * - scholar: literature notes, syntheses, searches, alerts; all library changes.
  * - author:  scholar plus manuscript work: citation check, bibliography, comments
  *            on the argument. Does not write the manuscript's paragraphs.
  * - editor:  everything, including drafting text for a manuscript on request.
@@ -46,13 +46,13 @@ export const PROFILE_SPECS: Record<Profile, ProfileSpec> = {
 			"- A literature note is a reading note: the full reference, then direct quotes with page numbers (or section names), and under each quote one question for {{user}} to answer. Do not write summaries, syntheses, conclusions or paragraphs that {{user}} could hand in as their own text. When a prompt or the shared rules ask for a literature note or a synthesis, make a reading note instead and say so.",
 			"- For searches, list works with one line each on what they study, not on what they conclude.",
 			"- If {{user}} asks for a summary, a synthesis or text for an assignment, say that the Reader profile does not write these, and offer quotes and questions. {{User}} can change the profile with /profile.",
-			"- Librarian: at most 10 items per review note. Explain each proposed tag in the Reason column.",
+			"- Tag review notes: at most 10 items. Explain each proposed tag in the Reason column.",
 		].join("\n"),
 	},
 	scholar: {
 		name: "scholar",
 		label: "Scholar",
-		summary: "literature notes, syntheses, searches and alerts; the full librarian",
+		summary: "literature notes, syntheses, searches and alerts; all library changes",
 		without: MANUSCRIPT,
 		withoutPrompts: ["review"],
 		batch: 25,
@@ -60,7 +60,7 @@ export const PROFILE_SPECS: Record<Profile, ProfileSpec> = {
 			"Profile: Scholar.",
 			"- Write literature notes and syntheses from what you read, with a source for every claim.",
 			"- Do not draft text for {{user}}'s manuscripts, theses or assignments. If asked, say that the Author and Editor profiles help with manuscripts (/profile).",
-			"- Librarian: 25 items per review note.",
+			"- Tag review notes: 25 items.",
 		].join("\n"),
 	},
 	author: {
@@ -75,7 +75,7 @@ export const PROFILE_SPECS: Record<Profile, ProfileSpec> = {
 			"- Write literature notes and syntheses from what you read, with a source for every claim.",
 			"- Help with {{user}}'s manuscripts: check citations against the library, export the bibliography, and comment on the argument, the structure and the evidence that is missing, with the sources that could support each point.",
 			"- Do not write the manuscript's paragraphs. Say what is missing and where the support is; {{user}} writes the text. The Editor profile drafts text (/profile).",
-			"- Librarian: 25 items per review note.",
+			"- Tag review notes: 25 items.",
 		].join("\n"),
 	},
 	editor: {
@@ -89,7 +89,7 @@ export const PROFILE_SPECS: Record<Profile, ProfileSpec> = {
 			"Profile: Editor. All tools.",
 			"- Write literature notes and syntheses from what you read, with a source for every claim.",
 			"- When {{user}} asks, draft text for a manuscript: cite library items as [@citekey], use only claims from sources you read, and mark drafted text clearly so that {{user}} can check it.",
-			"- Librarian: 25 items per review note unless {{user}} asks for more.",
+			"- Tag review notes: 25 items unless {{user}} asks for more.",
 		].join("\n"),
 	},
 };

package/src/prompt.ts

@@ -53,13 +53,13 @@ export function fill(text: string, cfg: Pick<SubsubConfig, "userName" | "about"
 }
 
 export function roleText(mode: Mode, cfg?: Pick<SubsubConfig, "userName" | "about" | "profile">): string {
-	const raw = readIf(join(ROLE_DIR, `${mode}.md`)) ?? `You are the ${mode}.`;
+	const raw = readIf(join(ROLE_DIR, `${mode}.md`)) ?? `You are Sub-Sub, the ${mode}.`;
 	return cfg ? fill(raw, cfg) : raw;
 }
 
 export function systemAddition(mode: Mode, cfg: SubsubConfig, cwd: string): string {
 	const user = who(cfg);
-	const parts = [`# Sub-Sub: ${mode} mode`, roleText(mode, cfg)];
+	const parts = [`# Sub-Sub: ${mode === "librarian" ? "Librarian" : "Researcher"} mode`, roleText(mode, cfg)];
 	const shared = readIf(cfg.sharedRules);
 	if (shared) {
 		parts.push(

package/src/roles.ts

@@ -7,7 +7,7 @@ import { profileSpec } from "./profiles.ts";
 
 export const FILE_TOOLS = ["read", "edit", "write", "grep", "find", "ls"];
 
-/** Prefixes of the tools that Sub-Sub's servers provide (the gate checks the mode for these). */
+/** Prefixes of the tools that Sub-Sub's servers provide (the gate checks the mode and the profile for these). */
 export const SERVER_PREFIXES = ["zotero_", "scholar_", "verify_"];
 
 export const ZOTERO_READ = [
@@ -48,27 +48,48 @@ export const PREVIEW_WRITES = new Set([
 /** Writes without a dry run that still need a yes. */
 export const CONFIRM_WRITES = new Set(["scholar_export_bibliography"]);
 
-export function toolsFor(mode: Mode, registered: string[], profile: Profile = "editor"): string[] {
+/** Zotero tools that look at the library or write only a note in the Sub-Sub folder. */
+export const ZOTERO_CHECKS = ["tag_audit", "audit_metadata", "find_duplicates", "check_retractions", "missing_pdfs", "write_tag_review"].map(
+	(n) => `zotero_${n}`,
+);
+
+
+export interface ToolOptions {
+	profile?: Profile;
+	/** The model test: also the bakeoff_ tools. */
+	bench?: boolean;
+}
+
+/**
+ * The tools of a mode. The Librarian manages the library: Zotero tools and files.
+ * The Researcher has the same, plus searches, notes and reference checks. The
+ * profile then removes tools. Every library change is still previewed (gateKind).
+ */
+export function toolsFor(mode: Mode, registered: string[], opts: ToolOptions = {}): string[] {
 	const has = new Set(registered);
-	const without = new Set(profileSpec(profile).without);
-	const wanted =
-		mode === "librarian"
-			? [...registered.filter((n) => n.startsWith("zotero_")), ...FILE_TOOLS]
-			: [
-					...ZOTERO_READ,
-					...registered.filter((n) => n.startsWith("scholar_") || n.startsWith("verify_")),
-					...FILE_TOOLS,
-				];
+	const without = new Set(profileSpec(opts.profile ?? "editor").without);
+	const zotero = registered.filter((n) => n.startsWith("zotero_") && (opts.bench || !n.startsWith("zotero_bakeoff_")));
+	const wanted = [
+		...ZOTERO_READ,
+		...ZOTERO_CHECKS,
+		...zotero,
+		...(mode === "researcher" ? registered.filter((n) => n.startsWith("scholar_") || n.startsWith("verify_")) : []),
+		...FILE_TOOLS,
+	];
 	return [...new Set(wanted)].filter((n) => has.has(n) && !without.has(n));
 }
 
 /** Why a Sub-Sub tool is not available now, or undefined when it is. */
-export function unavailableReason(name: string, mode: Mode, registered: string[], profile: Profile): string | undefined {
-	if (toolsFor(mode, registered, profile).includes(name)) return undefined;
+export function unavailableReason(name: string, mode: Mode, registered: string[], opts: ToolOptions = {}): string | undefined {
+	if (toolsFor(mode, registered, opts).includes(name)) return undefined;
+	const profile = opts.profile ?? "editor";
 	if (profileSpec(profile).without.includes(name)) {
 		return `${name} is not part of the ${profileSpec(profile).label} profile. The user can change the profile with /profile.`;
 	}
-	return `${name} is not available in ${mode} mode. The user can switch with /${otherMode(mode)}.`;
+	if (mode === "librarian" && toolsFor("researcher", registered, opts).includes(name)) {
+		return `${name} is part of the Researcher, which also does everything the Librarian does. The user can switch with /researcher.`;
+	}
+	return `${name} is not available.`;
 }
 
 export type GateKind = "preview" | "confirm" | "path" | null;
@@ -84,7 +105,3 @@ export function gateKind(toolName: string, input: Record<string, unknown>): Gate
 	if (toolName === "write" || toolName === "edit") return "path";
 	return null;
 }
-
-export function otherMode(mode: Mode): Mode {
-	return mode === "librarian" ? "researcher" : "librarian";
-}

package/src/subsub.ts

@@ -1,15 +1,16 @@
 /**
  * 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).
+ * - Two modes: the Librarian manages the library (Zotero tools only); the
+ *   Researcher does the same and also searches PubMed, Europe PMC and OpenAlex
+ *   and writes notes. Each mode can have its own model.
  * - 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.
  * - 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,9 +26,9 @@ import {
 	findUv,
 	isProfile,
 	loadConfig,
-	type Mode,
 	parseEnvFile,
 	type Profile,
+	type Mode,
 	saveConfig,
 	serverCommand,
 	starbuckCommand,
@@ -44,6 +45,8 @@ import { headerLines, type LibraryState, paintPreview, QUOTES, type Scheme, sche
 import { systemAddition } from "./prompt.ts";
 import { gateKind, SERVER_PREFIXES, toolsFor, unavailableReason } from "./roles.ts";
 
+const MODE_ENTRY = "subsub-mode";
+
 export interface SubsubDeps {
 	config?: SubsubConfig;
 	/** A ready bridge (tests). When given, Sub-Sub does not start or stop servers. */
@@ -52,7 +55,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 +129,11 @@ 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("librarian", { description: "Start Sub-Sub in the Librarian mode", type: "boolean", default: false });
 
 	let mode: Mode = cfg.defaultMode;
+	const benchRun = Boolean(process.env.SUBSUB_BENCH_OUT);
+	const toolOpts = () => ({ 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). */
@@ -172,7 +176,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);
@@ -250,7 +254,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 
 	async function applyMode(ctx: ExtensionContext, next: Mode, remember: boolean): Promise<void> {
 		mode = next;
-		pi.setActiveTools(toolsFor(mode, registered(), profile));
+		pi.setActiveTools(toolsFor(mode, registered(), toolOpts()));
 		const spec = cfg.models[mode];
 		if (spec) {
 			const i = spec.indexOf("/");
@@ -259,7 +263,6 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 			if (!ok) ctx.ui.notify(`Sub-Sub: cannot use ${spec} (unknown model or no key); keeping the current model.`, "warning");
 		}
 		if (remember) pi.appendEntry(MODE_ENTRY, { mode });
-		applyTheme(ctx);
 		status(ctx);
 	}
 
@@ -277,6 +280,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 		}
 		// Judge light or dark before Sub-Sub changes the theme.
 		if (lookOn(ctx)) scheme ??= schemeFromAnsi(ctx.ui.theme.getFgAnsi("text"), ctx.ui.theme.name);
+		applyTheme(ctx);
 		await applyMode(ctx, restored ?? (pi.getFlag("librarian") ? "librarian" : cfg.defaultMode), false);
 		if (lookOn(ctx)) {
 			setHeader(ctx);
@@ -323,7 +327,7 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 		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, mode, registered(), toolOpts());
 			if (why) {
 				if (benchOut) record({ kind: "wrong_mode", tool: name, input });
 				return { block: true, reason: why };
@@ -438,18 +442,18 @@ export async function createSubsub(pi: ExtensionAPI, deps: SubsubDeps = {}): Pro
 	}
 
 	pi.registerCommand("librarian", {
-		description: "Sub-Sub: switch to the librarian (changes the library, no web)",
+		description: "Sub-Sub: switch to the Librarian (manages the library: tags, imports, metadata, PDFs)",
 		handler: async (_args, ctx) => {
 			await applyMode(ctx, "librarian", true);
-			ctx.ui.notify("Sub-Sub: librarian mode", "info");
+			ctx.ui.notify("Sub-Sub: Librarian mode. For searches and notes, switch to /researcher.", "info");
 		},
 	});
 
 	pi.registerCommand("researcher", {
-		description: "Sub-Sub: switch to the researcher (search, notes; library read-only)",
+		description: "Sub-Sub: switch to the Researcher (searches, notes, reviews, and everything the Librarian does)",
 		handler: async (_args, ctx) => {
 			await applyMode(ctx, "researcher", true);
-			ctx.ui.notify("Sub-Sub: researcher mode", "info");
+			ctx.ui.notify("Sub-Sub: Researcher mode. It searches, writes notes and also changes the library, with a preview.", "info");
 		},
 	});
 

package/src/web.ts

@@ -568,12 +568,11 @@ 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.
+					// 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";
-					const models = { ...(loadConfig(env).models ?? {}), [mode]: spec };
+					const models: Record<string, string> = { ...(loadConfig(env).models ?? {}), [mode]: spec };
 					if (body.both) models[mode === "librarian" ? "researcher" : "librarian"] = spec;
-					saveConfig({ models }, env);
+					saveConfig({ models, model: undefined }, env);
 					await agent.request({ type: "set_model", provider: String(body.provider), modelId: String(body.id) });
 					await restartAgent();
 					return send(res, 200, { ok: true });
@@ -597,7 +596,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" && Object.keys(current.models).length === 0) saveConfig({ models: TESTED_MODELS, model: undefined }, env);
 					await restartAgent();
 					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 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
 
@@ -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,7 +87,7 @@
 	--warn: #d6a84e;
 }
 
-/* The mode sets the accent: blue for the researcher, rust for the librarian. */
+/* 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); }
 
@@ -316,7 +316,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; }

package/web/app.js

@@ -175,7 +175,10 @@ function mode() {
 
 function renderHeader() {
 	document.body.dataset.mode = mode();
-	for (const b of document.querySelectorAll(".modes button")) b.setAttribute("aria-pressed", String(b.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");
@@ -221,12 +224,18 @@ 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.needs || state?.[a.needs]) &&
+		(!a.profiles || a.profiles.includes(state?.profile ?? "scholar"));
+	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) }));
+		for (const a of list) box.append(actionButton(a));
+	}
 }
 
 function actionButton(a) {
@@ -326,17 +335,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,7 +378,7 @@ function clearEmpty() {
 
 function renderEmpty() {
 	if (log.children.length) return;
-	const s = SUGGESTIONS[mode()][lang] ?? SUGGESTIONS[mode()].en;
+	const s = SUGGESTIONS[mode()].map((x) => x[lang] ?? x.en);
 	log.append(
 		el(
 			"div",
@@ -708,7 +717,8 @@ function onEvent(ev) {
 			break;
 		case "extension_ui_request":
 			if (ev.method === "notify") {
-				if (/^Sub-Sub: (librarian|researcher) mode$/.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);
@@ -935,15 +945,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;
 	}
 }
 

package/web/i18n.js

@@ -3,8 +3,6 @@
 
 export const STRINGS = {
 	en: {
-		researcher: "Researcher",
-		librarian: "Librarian",
 		profile: "Profile",
 		model: "Model",
 		noModel: "No model",
@@ -14,8 +12,6 @@ export const STRINGS = {
 		noConversations: "No earlier conversations.",
 		send: "Send",
 		stop: "Stop",
-		placeholderResearcher: "Ask about your library, or type / for commands",
-		placeholderLibrarian: "Ask for a change to your library, or type / for commands",
 		yes: "Yes",
 		no: "No",
 		ok: "OK",
@@ -44,7 +40,6 @@ export const STRINGS = {
 		connect: "Connect",
 		modelTitle: "Model",
 		chooseModel: "Choose a model",
-		forBoth: "Use it in both modes",
 		noModels: "No models yet. Connect a provider below.",
 		connectProvider: "Connect a provider",
 		provider: "Provider",
@@ -83,13 +78,20 @@ export const STRINGS = {
 		starbuckOn: "Reference checks are on. Use /verify, or ask the researcher to check a manuscript's references.",
 		starbuckOff: "Reference checks are off.",
 		composerHint: "Enter sends. Shift+Enter starts a new line. / shows the commands.",
-		emptyResearcher: "The researcher reads your library and searches PubMed, Europe PMC and OpenAlex. It cannot change your library.",
-		emptyLibrarian: "The librarian changes your library. You see a preview of each change, and nothing changes until you approve it.",
 		previewKicker: "Preview",
+		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",
 	},
 	pt: {
-		researcher: "Investigador",
-		librarian: "Bibliotecário",
 		profile: "Perfil",
 		model: "Modelo",
 		noModel: "Sem modelo",
@@ -99,8 +101,6 @@ export const STRINGS = {
 		noConversations: "Ainda não há conversas anteriores.",
 		send: "Enviar",
 		stop: "Parar",
-		placeholderResearcher: "Pergunte sobre a sua biblioteca, ou escreva / para ver os comandos",
-		placeholderLibrarian: "Peça uma alteração à biblioteca, ou escreva / para ver os comandos",
 		yes: "Sim",
 		no: "Não",
 		ok: "OK",
@@ -129,7 +129,6 @@ export const STRINGS = {
 		connect: "Ligar",
 		modelTitle: "Modelo",
 		chooseModel: "Escolha um modelo",
-		forBoth: "Usar nos dois modos",
 		noModels: "Ainda não há modelos. Ligue um fornecedor abaixo.",
 		connectProvider: "Ligar um fornecedor",
 		provider: "Fornecedor",
@@ -168,16 +167,27 @@ export const STRINGS = {
 		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.",
-		emptyResearcher: "O investigador lê a sua biblioteca e pesquisa na PubMed, Europe PMC e OpenAlex. Não pode alterar a biblioteca.",
-		emptyLibrarian: "O bibliotecário altera a sua biblioteca. Vê uma pré-visualização de cada alteração, e nada muda até a aprovar.",
 		previewKicker: "Pré-visualização",
+		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",
 	},
 };
 
-/** Buttons in the side panel. `fill` puts the command in the message box; `send` runs it at once. */
+/**
+ * Buttons in the side panel, in two groups. `fill` puts the command in the message box; `send` runs it at once.
+ * The Librarian sees the Library group only. `profiles` and `needs` limit a button to some profiles or add-ons.
+ */
 export const ACTIONS = {
-	researcher: [
-		{ cmd: "subsub", send: true, en: "Library overview", pt: "Resumo da biblioteca" },
+	research: [
 		{ cmd: "lit", fill: true, profiles: ["scholar", "author", "editor"], en: "Literature review", pt: "Revisão da literatura" },
 		{ cmd: "compare", fill: true, profiles: ["scholar", "author", "editor"], en: "Compare sources", pt: "Comparar fontes" },
 		{ cmd: "lit-note", fill: true, en: "Literature note", pt: "Nota de leitura" },
@@ -190,28 +200,28 @@ export const ACTIONS = {
 		{ cmd: "request-copy", fill: true, en: "Ask an author for a copy", pt: "Pedir uma cópia ao autor" },
 		{ cmd: "digest", send: true, profiles: ["scholar", "author", "editor"], en: "Digest of the week", pt: "Resumo da semana" },
 	],
-	librarian: [
+	library: [
 		{ cmd: "subsub", send: true, en: "Library overview", pt: "Resumo da biblioteca" },
 		{ 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" },
-	],
-	both: [
 		{ cmd: "history", send: true, en: "Recent changes", pt: "Alterações recentes" },
 		{ cmd: "undo", send: true, en: "Undo the last change", pt: "Desfazer a última alteração" },
 	],
 };
 
-/** Suggestions on an empty conversation. */
+/** Suggestions on an empty conversation, per mode. */
 export const SUGGESTIONS = {
-	researcher: {
-		en: ["What do I have on ", "Find recent systematic reviews on ", "Which items have no PDF?"],
-		pt: ["O que tenho sobre ", "Procure revisões sistemáticas recentes sobre ", "Que itens não têm PDF?"],
-	},
-	librarian: {
-		en: ["Tag the next 10 items without a topic", "Import 10.1136/bmj.n71", "Find duplicates"],
-		pt: ["Etiquete os próximos 10 itens sem tópico", "Importe 10.1136/bmj.n71", "Procure duplicados"],
-	},
+	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. */

package/web/index.html

@@ -110,7 +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" checked> <span data-i18n="forBoth">Use it in both modes</span></label>
+		<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">