@diffohq/diffo 0.18.1 → 0.19.0

package/dist/cli.mjs

@@ -308,6 +308,23 @@ function describeAnchor(anchor) {
 	if (anchor.kind === "layer") return `layer "${anchor.title}"`;
 	return `${anchor.path}:${anchorSpan(anchor)} (${anchor.side} side)`;
 }
+/** It rides every poll prompt on a pull request, so it stays short. */
+const VOICE_MAX = 1500;
+/**
+* A stored or submitted voice as the prompt uses it: trimmed, or null for
+* "the default" (absent, empty, not a string, or over the cap, which a
+* client never sends but an edited DB row could hold).
+*/
+function normalizeVoice(value) {
+	if (typeof value !== "string") return null;
+	const text = value.trim();
+	if (text === "" || text.length > 1500) return null;
+	return text;
+}
+/** The voice the agent reads: the reviewer's own, or the default. */
+function effectiveVoice(value) {
+	return normalizeVoice(value) ?? "Write it the way I'd type a review comment at the line: one or two short sentences, the point first, then stop. \"This resets the streak on late completions. Should it?\" is a comment. A paragraph that states the finding, explains why it matters and proposes the fix is a report; I rewrite those every time.\n\nSound like a teammate, not a model. Direct about the issue, concrete about the fix, kind to the person. Plain words, contractions, sentences of different lengths. \"I think\" or \"probably\" when I'm actually unsure, never softening for its own sake. A real question is often the whole comment. Start a nit with \"nit:\". Name the fix only when it isn't obvious; if it fits the anchored lines, put it in a ```suggestion block instead of describing it.\n\nGiveaways to avoid: em-dashes; \"Consider...\" or \"It might be worth...\"; \"not X but Y\"; \"I noticed that\"; \"it's important to note\"; stacked hedges (\"perhaps we might want to\"); lists of three for the sake of three; bold; praise before the point; restating what the diff shows; \"robust\", \"leverage\", \"ensure\", \"streamline\"; trailing \"-ing\" riders (\"...ensuring consistency\"); sign-offs like \"let me know\".";
+}
 //#endregion
 //#region src/server/prompt.ts
 /** A comment can anchor to a long range; the frozen text keeps its head. */
@@ -445,7 +462,7 @@ const TAB_TITLE = {
 };
 function nextStepFor(kind, actionable) {
 	if (kind === "submitted") return `Nothing to act on — tell the user the review was submitted, then run \`${CLI_COMMANDS.poll}\` again to keep listening (${POLL_STANCE}), or \`${CLI_COMMANDS.end}\` if they are done.`;
-	if (kind === "layers") return `Post the outline with \`${CLI_COMMANDS.layers}\` (or pipe it to \`${CLI_COMMANDS.layersStdin}\`), then run \`${CLI_COMMANDS.poll}\` again to keep listening (${POLL_STANCE}).`;
+	if (kind === "layers") return `Post the outline with \`${CLI_COMMANDS.layers}\` — ${LAYERS.post} (\`${CLI_COMMANDS.layersStdin}\`). If the post is refused, say so to the reviewer in chat — they are watching the Layers tab for it. Then run \`${CLI_COMMANDS.poll}\` again to keep listening (${POLL_STANCE}).`;
 	if (kind === "cleared") return `Post the guide if this changeset warrants one, then run \`${CLI_COMMANDS.firstPoll}\` again to keep listening (${POLL_STANCE}) — this is a fresh round, so give it a fresh title.`;
 	if (kind === "finish" && actionable === 0) return `Nothing to act on — run \`${CLI_COMMANDS.poll}\` again to keep listening (${POLL_STANCE}); the reviewer may follow up.`;
 	return `${kind === "finish" ? "The reviewer is done reading — work the whole batch. " : ""}Act on each thread, reply with \`${CLI_COMMANDS.reply}\`, then run \`${CLI_COMMANDS.poll}\` again to keep listening.`;
@@ -520,9 +537,10 @@ const HELP_AGENT_PR = `Reviewing a pull request (\`diffo <PR URL | owner/repo#N
   when they were wrong (say so privately), not on pushback against your own
   reply — redraft the one already in the thread instead — and not when asked
   but you have nothing worth saying to the author: say so privately, never
-  invent one. Keep it to one or two short sentences, the point first, the
-  way a person types a review comment; no preamble, no "Suggest …", no
-  report.
+  invent one. Write it in the reviewer's voice: their standing instructions
+  for how their comments read (Settings → Voice) come with every delivery.
+  By default, one or two short sentences, the point first, the way a person
+  types a review comment; no preamble, no "Suggest …", no report.
 - The guide and layers work as above, built from the description, the commits
   and the diff; skip the guide when the description already orients.
 - When the reviewer submits, a poll returns a \`"kind": "submitted"\` notice:
@@ -593,6 +611,13 @@ const LAYERS = {
 	stance: GUIDE.stance,
 	/** A post is the whole list. */
 	replace: "a post replaces the whole list, never merges; ids are kept for titles that match, so a re-post never moves the reviewer's place",
+	/** How the payload reaches the CLI. Two commands when it is long, because a
+	* harness that screens each command on its own (Claude Code's auto mode, for
+	* one) refuses a post it cannot trace: a command that writes the file and
+	* pipes it in one go, or a pipe from a file the session never visibly wrote.
+	* The refusal also tells the agent to stop, so the reviewer sees "Outlining…"
+	* until the server gives up on it (see OUTLINE_STALL_MS). */
+	post: "inline with --json when it fits on a command line; when it is long, write the JSON to a file in one command and pipe that file to --stdin in the next — never write and post in the same command, and never pipe a file this session did not itself write, because a harness that screens commands one at a time refuses a post it cannot trace back to you",
 	/** What earns a line in a layer's `decisions`, and the bar for it. The
 	* reviewer reads these before the diff — or instead of it — so every entry
 	* must be something they could act on, never a retelling of the change. */
@@ -612,7 +637,8 @@ const AUTHOR_DOCTRINE = {
 * addressed to the author in the reviewer's voice; nothing of it posts until the
 * reviewer adds it to their review and submits.
 */
-const PR_COMMENT_DOCTRINE = `## Drafting a review comment for the author
+function prCommentDoctrine(voice) {
+	return `## Drafting a review comment for the author
 
 The reviewer's private messages are of two kinds. A question to you ("what
 calls this?") wants an answer. A finding stated about the code ("this resets
@@ -637,23 +663,26 @@ request is the one thing the reviewer cannot tell apart from a finding. On a
 comment of your own (\`${CLI} comment\`), the same flag carries a finding you
 can show, never a hunch.
 
-Write it the way a person types a review comment at the line: one or two
-short sentences, the point first, done. "This resets the streak on late
-completions — should it?" is a review comment. A paragraph that states the
-finding, explains why it matters and proposes the fix is not; it is a report,
-and the reviewer rewrites it every time. No preamble ("This reads as…"), no
-"Suggest …" or "Consider …" framing, no restating what the diff already
-shows, no examples in parentheses, no hedging. Name the fix only when it is
-not obvious from the finding, and when it is local to the anchored lines put
-it in a \`\`\`suggestion block instead of prose. A question is often the
-whole comment. Nothing about you, the worktree, tests you ran, or Diffo —
-that evidence goes in --message, where the reviewer reads it. Match how this
-reviewer writes: their own review comments on this pull request are quoted
-under "The reviewer's voice" when there are any.
+Write it in the reviewer's voice. Their standing instructions for how a
+review comment of theirs reads (Settings → Voice in Diffo; every delivery
+carries the current text, so follow the latest):
+
+${indent(effectiveVoice(voice))}
+
+Nothing about you, the worktree, tests you ran, or Diffo — that evidence
+goes in --message, where the reviewer reads it. Their own review comments
+on this pull request are quoted under "The reviewer's voice" when there are
+any; match those too.
 
 Nothing you attach reaches GitHub. It appears under your reply with Add as
 PR comment draft / Edit / Dismiss; the reviewer decides, and the submit
 notice tells you what happened to each.`;
+}
+/** The doctrine as shipped before the voice could be set: the default voice. */
+const PR_COMMENT_DOCTRINE = prCommentDoctrine();
+function indent(text) {
+	return text.split("\n").map((line) => line === "" ? "" : `    ${line}`).join("\n");
+}
 const PR_DOCTRINE = {
 	guide: {
 		when: "the pull request description leaves a cold reader without the shape of the change — skip when the description already orients",
@@ -668,8 +697,15 @@ const PR_DOCTRINE = {
 	fix: "- `issue` threads want a fix you cannot push: work it out in the worktree, verify it the cheapest honest way, restore the worktree (`git checkout -- .`), and reply with a ```suggestion block plus what you checked. Never commit or push to the pull request.",
 	prComment: PR_COMMENT_DOCTRINE
 };
-function doctrineFor(changeset) {
-	return changeset?.pr ? PR_DOCTRINE : AUTHOR_DOCTRINE;
+/** The doctrine for a review, with the reviewer's voice folded into the PR
+* comment rule when they set one; the author's doctrine has no voice to carry. */
+function doctrineFor(changeset, voice) {
+	if (!changeset?.pr) return AUTHOR_DOCTRINE;
+	const custom = normalizeVoice(voice);
+	return custom === null ? PR_DOCTRINE : {
+		...PR_DOCTRINE,
+		prComment: prCommentDoctrine(custom)
+	};
 }
 /**
 * The pull request, framed for an agent that did not write it. Every PR-mode
@@ -1021,7 +1057,7 @@ function prLines(ctx) {
 }
 function buildThreadPrompt(thread, ctx) {
 	const siblings = siblingLines(ctx.siblings ?? []);
-	const D = doctrineFor(ctx.changeset);
+	const D = doctrineFor(ctx.changeset, ctx.voiceSetting);
 	const pr = ctx.changeset?.pr;
 	return [
 		pr ? `A reviewer is reading pull request #${pr.number} in \`${ctx.repo.name}\` with you as their copilot, and asked you this privately.` : `A reviewer is reading your changes in \`${ctx.repo.name}\` (branch \`${ctx.repo.branch}\`) and sent you this review thread.`,
@@ -1038,7 +1074,7 @@ function buildThreadPrompt(thread, ctx) {
 }
 function buildCoalescedPrompt(threads, ctx) {
 	const siblings = siblingLines(ctx.siblings ?? []);
-	const D = doctrineFor(ctx.changeset);
+	const D = doctrineFor(ctx.changeset, ctx.voiceSetting);
 	const pr = ctx.changeset?.pr;
 	return [
 		pr ? `The reviewer of pull request #${pr.number} in \`${ctx.repo.name}\` sent you new private messages on ${threads.length} threads.` : `The reviewer sent new messages on ${threads.length} review threads in \`${ctx.repo.name}\` (branch \`${ctx.repo.branch}\`).`,
@@ -1097,7 +1133,7 @@ function buildSubmittedPrompt(ctx, submitted) {
 function buildLayersRequestPrompt(ctx, existing) {
 	const refresh = existing !== null && existing.length > 0;
 	const titles = refresh ? existing.map((l) => `"${l.title}"`).join(", ") : "";
-	const D = doctrineFor(ctx.changeset);
+	const D = doctrineFor(ctx.changeset, ctx.voiceSetting);
 	return [
 		refresh ? `The reviewer asked you to refresh the layers in \`${ctx.repo.name}\` (branch \`${ctx.repo.branch}\`): the code moved since you outlined it, and files outside the outline have been gathering under "Since your review". Re-post the whole list as the change stands now.` : `The reviewer asked for layers in \`${ctx.repo.name}\` (branch \`${ctx.repo.branch}\`): outline the changeset as steps to read in order. The Layers tab reads "Outlining…" until you post.`,
 		"",
@@ -1114,14 +1150,14 @@ function buildLayersRequestPrompt(ctx, existing) {
 		"",
 		`Shape: ${LAYERS.shape}`,
 		"",
-		`Post it: \`${CLI_COMMANDS.layers}\`, or pipe the JSON to \`${CLI_COMMANDS.layersStdin}\` when it is long. Nothing else is owed for this item — no reply, no comment.`,
+		`Post it: \`${CLI_COMMANDS.layers}\` — ${D.layers.post} (\`${CLI_COMMANDS.layersStdin}\`). If the post is refused, tell the reviewer in chat; they are watching the Layers tab for it. Nothing else is owed for this item — no reply, no comment.`,
 		"",
 		`Then run \`${CLI_COMMANDS.poll}\` again to keep listening (${POLL_STANCE}).`,
 		""
 	].join("\n");
 }
 function buildClearedPrompt(ctx) {
-	const D = doctrineFor(ctx.changeset);
+	const D = doctrineFor(ctx.changeset, ctx.voiceSetting);
 	return [
 		`The reviewer cleared the review in \`${ctx.repo.name}\` (branch \`${ctx.repo.branch}\`): the previous round landed, and its threads and guide are gone. What the reviewer sees now is a fresh round.`,
 		"",
@@ -1159,7 +1195,7 @@ function buildFinishPrompt(threads, ctx, coverage) {
 		"leaves the reviewer with nothing to do.",
 		""
 	] : [];
-	const D = doctrineFor(ctx.changeset);
+	const D = doctrineFor(ctx.changeset, ctx.voiceSetting);
 	const pr = ctx.changeset?.pr;
 	const parts = [
 		pr ? `The reviewer finished reading pull request #${pr.number} in \`${repo.name}\` and is handing you what they still have for you privately.` : `A reviewer finished reading your changes in \`${repo.name}\` (branch \`${repo.branch}\`).`,
@@ -1468,8 +1504,8 @@ The loop:
    Flag it at open with \`diffo layers --suggest "<why, in one line>"\` and
    say so in your handoff ("say layers and I'll outline it"). Post the
    outline only when the reviewer asks, in chat or through the poll as a
-   \`"kind": "layers"\` payload: \`diffo layers --json '<Layer[]>'\`
-   (or pipe it to \`diffo layers --stdin\`). Each layer is ${LAYERS.what}.
+   \`"kind": "layers"\` payload: \`diffo layers --json '<Layer[]>'\` —
+   ${LAYERS.post} (\`diffo layers --stdin\`). Each layer is ${LAYERS.what}.
    Summary: ${LAYERS.summary}. Diagram: ${LAYERS.diagram}.
    Order: ${LAYERS.order}. ${LAYERS.mechanical}. ${LAYERS.stance}.
    ${LAYERS.replace}. \`diffo help layers\` has the shape.
@@ -1675,7 +1711,8 @@ Examples:
 
 Usage: diffo layers --suggest ["<why, in one line>"]   at open: this read benefits from layers
        diffo layers --json '<Layer[]>'                 post the outline, replacing the last one
-       … | diffo layers --stdin                        the same payload, piped
+       … | diffo layers --stdin                        the same payload, piped — from a file
+                                                       written in a command of its own
 
 Each layer is ${LAYERS.what}.
 Summary: ${LAYERS.summary}.
@@ -2470,6 +2507,7 @@ const execGh = (args) => new Promise((resolvePromise, reject) => {
 	execFile("gh", args, {
 		encoding: "utf-8",
 		maxBuffer: MAX_BUFFER$1,
+		windowsHide: true,
 		env: {
 			...process.env,
 			GH_PROMPT_DISABLED: "1"
@@ -2947,7 +2985,8 @@ function git(root, args) {
 			"pipe",
 			"pipe"
 		],
-		maxBuffer: MAX_BUFFER
+		maxBuffer: MAX_BUFFER,
+		windowsHide: true
 	});
 }
 const execFileAsync = promisify(execFile);
@@ -2963,7 +3002,8 @@ async function gitAsync(root, args) {
 	const { stdout } = await execFileAsync("git", args, {
 		cwd: root,
 		encoding: "utf-8",
-		maxBuffer: MAX_BUFFER
+		maxBuffer: MAX_BUFFER,
+		windowsHide: true
 	});
 	return stdout;
 }
@@ -3018,7 +3058,8 @@ async function branchExistsAsync(root, branch) {
 		], {
 			cwd: root,
 			encoding: "utf-8",
-			maxBuffer: MAX_BUFFER
+			maxBuffer: MAX_BUFFER,
+			windowsHide: true
 		});
 		return true;
 	} catch (err) {
@@ -3479,7 +3520,8 @@ function getBaseFileBytes(root, ref, relPath) {
 				"pipe",
 				"pipe"
 			],
-			maxBuffer: MAX_BUFFER
+			maxBuffer: MAX_BUFFER,
+			windowsHide: true
 		});
 	} catch {
 		return null;
@@ -3583,6 +3625,7 @@ function fetchSync(root, args) {
 			"pipe"
 		],
 		maxBuffer: MAX_BUFFER,
+		windowsHide: true,
 		timeout: FETCH_TIMEOUT_MS,
 		env: FETCH_ENV
 	});
@@ -3603,6 +3646,7 @@ async function fetchPrHeadAsync(root, number, remote = "origin") {
 		cwd: root,
 		encoding: "utf-8",
 		maxBuffer: MAX_BUFFER,
+		windowsHide: true,
 		timeout: FETCH_TIMEOUT_MS,
 		env: FETCH_ENV
 	});
@@ -3621,6 +3665,7 @@ async function fetchBranchAsync(root, branch, remote = "origin") {
 		cwd: root,
 		encoding: "utf-8",
 		maxBuffer: MAX_BUFFER,
+		windowsHide: true,
 		timeout: FETCH_TIMEOUT_MS,
 		env: FETCH_ENV
 	});
@@ -4356,6 +4401,11 @@ const STALL_AFTER_MS = 3e5;
 const REPLY_GRACE_MS = 9e4;
 /** A re-poll within this window of a delivery reads as "hasn't started yet". */
 const BATCH_SETTLE_MS = 2e3;
+/** An outline owed this long with no post reads as stalled: the reviewer gets
+* the reason and their button back instead of "Outlining…" for good. Shorter
+* than a reply's stall — an agent whose harness refused the post is told to
+* stop, and nothing else ends the wait: no re-poll, no reply clock. */
+const OUTLINE_STALL_MS = 18e4;
 /** Signal 0 probes existence without touching the process; EPERM still proves it exists. */
 function pidAlive(pid) {
 	try {
@@ -4370,6 +4420,7 @@ var DeliveryQueue = class DeliveryQueue {
 	replyGraceMs;
 	liveness;
 	batchSettleMs;
+	outlineStallMs;
 	buckets = /* @__PURE__ */ new Map();
 	scope = "";
 	waiter = null;
@@ -4378,6 +4429,7 @@ var DeliveryQueue = class DeliveryQueue {
 	deliveredAt = /* @__PURE__ */ new Map();
 	listeners = /* @__PURE__ */ new Set();
 	stalled = false;
+	outlineTimer = null;
 	stallTimer = null;
 	betweenPolls = false;
 	/** Why the agent is parked between polls: a reply just landed, its poll
@@ -4401,11 +4453,12 @@ var DeliveryQueue = class DeliveryQueue {
 	batchWatch = null;
 	waiterWatch = null;
 	batchListeners = /* @__PURE__ */ new Set();
-	constructor(stallAfterMs = STALL_AFTER_MS, replyGraceMs = REPLY_GRACE_MS, liveness = {}, batchSettleMs = BATCH_SETTLE_MS) {
+	constructor(stallAfterMs = STALL_AFTER_MS, replyGraceMs = REPLY_GRACE_MS, liveness = {}, batchSettleMs = BATCH_SETTLE_MS, outlineStallMs = OUTLINE_STALL_MS) {
 		this.stallAfterMs = stallAfterMs;
 		this.replyGraceMs = replyGraceMs;
 		this.liveness = liveness;
 		this.batchSettleMs = batchSettleMs;
+		this.outlineStallMs = outlineStallMs;
 	}
 	onBatchClosed(listener) {
 		this.batchListeners.add(listener);
@@ -4513,13 +4566,19 @@ var DeliveryQueue = class DeliveryQueue {
 	}
 	/** Where the reviewer's layers request stands (see `LayersRequest`). */
 	layersRequest() {
-		if (this.outlining() !== null) return "outlining";
+		if (this.outlining() !== null) return this.outlineStalled() ? "stalled" : "outlining";
 		if (this.buckets.get(this.scope)?.layers) return "queued";
 		return null;
 	}
 	outlining() {
 		return this.buckets.get(this.scope)?.outlining ?? null;
 	}
+	/** Derived, not flagged: the clock is the delivery itself, so a stall needs
+	* no state to clear — only `armOutlineWatch` to announce the crossing. */
+	outlineStalled() {
+		const since = this.outlining();
+		return since !== null && Date.now() - since >= this.outlineStallMs;
+	}
 	presenceDetail() {
 		return {
 			state: this.presence(),
@@ -4529,7 +4588,7 @@ var DeliveryQueue = class DeliveryQueue {
 	}
 	reason() {
 		if (this.awaitingReply) return this.stalled ? "stalled" : "delivered";
-		if (this.outlining() !== null) return "delivered";
+		if (this.outlining() !== null) return this.outlineStalled() ? "stalled" : "delivered";
 		if (this.waiter) return "polling";
 		if (this.betweenPolls) return this.parked;
 		return this.lastDetach ?? "no-agent";
@@ -4584,6 +4643,20 @@ var DeliveryQueue = class DeliveryQueue {
 		if (this.stallTimer !== null) clearTimeout(this.stallTimer);
 		this.stallTimer = null;
 	}
+	armOutlineWatch() {
+		this.clearOutlineWatch();
+		const since = this.outlining();
+		const timer = setTimeout(() => {
+			this.outlineTimer = null;
+			if (this.outlining() === since) this.notify();
+		}, this.outlineStallMs);
+		timer.unref?.();
+		this.outlineTimer = timer;
+	}
+	clearOutlineWatch() {
+		if (this.outlineTimer !== null) clearTimeout(this.outlineTimer);
+		this.outlineTimer = null;
+	}
 	armGrace(why = "replied", windowMs = this.replyGraceMs) {
 		this.clearGrace();
 		this.betweenPolls = true;
@@ -4689,7 +4762,12 @@ var DeliveryQueue = class DeliveryQueue {
 	* the cleared heads-up: a boolean per scope, surfaced once real feedback is
 	* answered. */
 	enqueueLayers() {
-		this.bucket().layers = true;
+		const bucket = this.bucket();
+		bucket.layers = true;
+		if (this.outlineStalled()) {
+			bucket.outlining = null;
+			this.clearOutlineWatch();
+		}
 		const queued = this.waiter === null;
 		this.wake();
 		if (queued) this.notify();
@@ -4703,6 +4781,7 @@ var DeliveryQueue = class DeliveryQueue {
 			bucket.layers = false;
 			bucket.outlining = null;
 		}
+		this.clearOutlineWatch();
 		this.notify();
 	}
 	/** The agent posted layers. Concludes an outstanding request; a spontaneous
@@ -4712,6 +4791,7 @@ var DeliveryQueue = class DeliveryQueue {
 		const since = bucket?.outlining ?? null;
 		if (bucket === void 0 || since === null) return null;
 		bucket.outlining = null;
+		this.clearOutlineWatch();
 		if (!this.awaitingReply && !this.betweenPolls) this.armGrace();
 		this.notify();
 		return Date.now() - since;
@@ -4734,7 +4814,10 @@ var DeliveryQueue = class DeliveryQueue {
 			if (this.batch.sawReply || Date.now() - this.batch.deliveredAt >= this.batchSettleMs) this.closeBatch("repoll");
 		}
 		const owing = this.buckets.get(this.scope);
-		if (owing !== void 0 && owing.outlining !== null && Date.now() - owing.outlining >= this.batchSettleMs) owing.outlining = null;
+		if (owing !== void 0 && owing.outlining !== null && Date.now() - owing.outlining >= this.batchSettleMs) {
+			owing.outlining = null;
+			this.clearOutlineWatch();
+		}
 		if (this.hasPending()) return Promise.resolve("data");
 		return new Promise((resolve) => {
 			const waiter = (outcome) => resolve(outcome);
@@ -4813,6 +4896,7 @@ var DeliveryQueue = class DeliveryQueue {
 			const bucket = this.bucket();
 			bucket.layers = false;
 			bucket.outlining = Date.now();
+			this.armOutlineWatch();
 			this.notify();
 			return;
 		}
@@ -4885,6 +4969,7 @@ var DeliveryQueue = class DeliveryQueue {
 		this.closeBatch("ended");
 		this.awaitingReply = false;
 		for (const bucket of this.buckets.values()) bucket.outlining = null;
+		this.clearOutlineWatch();
 		this.stalled = false;
 		this.clearStall();
 		this.clearGrace();
@@ -7423,10 +7508,12 @@ function createApp(ctx, store, review, queue) {
 		const value = ctx.uiSettings?.get(key) ?? null;
 		return value !== null && values.has(value) ? value : null;
 	};
+	const storedVoice = () => normalizeVoice(ctx.uiSettings?.get("voice"));
 	app.get("/api/settings", (c) => {
 		return c.json({
 			theme: stored("theme", THEME_VALUES),
-			critter: stored("critter", CRITTER_VALUES)
+			critter: stored("critter", CRITTER_VALUES),
+			voice: storedVoice()
 		});
 	});
 	app.put("/api/settings", async (c) => {
@@ -7434,11 +7521,14 @@ function createApp(ctx, store, review, queue) {
 		const body = await c.req.json().catch(() => null);
 		const theme = body?.theme;
 		const critter = body?.critter;
+		const voice = body?.voice;
 		const themeOk = typeof theme === "string" && THEME_VALUES.has(theme);
 		const critterOk = typeof critter === "string" && CRITTER_VALUES.has(critter);
-		if (theme !== void 0 && !themeOk || critter !== void 0 && !critterOk || !(themeOk || critterOk)) return c.json({ error: "need {theme?: \"system\" | \"light\" | \"dark\", critter?: \"on\" | \"off\"}, at least one" }, 400);
+		const voiceOk = typeof voice === "string" && voice.trim().length <= 1500;
+		if (theme !== void 0 && !themeOk || critter !== void 0 && !critterOk || voice !== void 0 && !voiceOk || !(themeOk || critterOk || voiceOk)) return c.json({ error: `need {theme?: "system" | "light" | "dark", critter?: "on" | "off", voice?: string (up to ${VOICE_MAX} characters, empty for the default)}, at least one` }, 400);
 		if (themeOk) ctx.uiSettings.set("theme", theme);
 		if (critterOk) ctx.uiSettings.set("critter", critter);
+		if (voiceOk) ctx.uiSettings.set("voice", voice.trim());
 		return c.json({ ok: true });
 	});
 	const reviewKind = () => reviewKindOf(ctx.spec, ctx.pr !== void 0);
@@ -7475,7 +7565,10 @@ function createApp(ctx, store, review, queue) {
 	const promptCtx = (...excludeThreadIds) => ({
 		repo: repoInfo(),
 		changeset: store?.get() ?? null,
-		...ctx.pr && review ? { voice: review.get().threads } : {},
+		...ctx.pr && review ? {
+			voice: review.get().threads,
+			voiceSetting: storedVoice()
+		} : {},
 		siblings: review?.get().threads.filter((t) => !excludeThreadIds.includes(t.id) && (t.state === "open" || t.state === "sent")) ?? []
 	});
 	const deliverThreads = (threadIds) => {
@@ -8563,6 +8656,7 @@ function spawnDaemonProcess(opts) {
 			logFd,
 			logFd
 		],
+		windowsHide: true,
 		env: {
 			...opts.env ?? process.env,
 			DIFFO_DAEMON: "1"

package/dist/client/assets/{abnfDiagram-WEYOIA3C-BITPutiq.js → abnfDiagram-WEYOIA3C-C4F66yrf.js}

@@ -1 +1 @@
-import{m as e,t}from"./mermaid-parser.core-LynYB9P7.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-D88qhP-M.js";import"./chunk-DSNLSHJZ-dqouHFOm.js";import{n as i,r as a,t as o}from"./chunk-2E7WHVMD-BH_-CWp4.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-vlyR5lnz.js";var c=e().RailroadAbnf.parser.LangiumParser,l=n(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=n(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=n(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=n(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=n(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=n(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=n(e=>{s(e,o),e.title&&o.setTitle(e.title),e.rules.map(e=>o.addRule(m(e)))},`populateDb`),g={parser:{parse:n(e=>{o.clear(),r.debug(`[ABNF Parser] Starting Langium parse`);let n=c.parse(e);if(n.lexerErrors.length>0||n.parserErrors.length>0)throw new t(n);let i=n.value;r.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),r.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:o}},db:o,renderer:a,styles:i};export{g as diagram};
+import{m as e,t}from"./mermaid-parser.core-C6WMZqm3.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-wNqkFAEP.js";import"./chunk-DSNLSHJZ-DiIzqoT2.js";import{n as i,r as a,t as o}from"./chunk-2E7WHVMD-DfTh-gtY.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-BOSV4eXG.js";var c=e().RailroadAbnf.parser.LangiumParser,l=n(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=n(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=n(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=n(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=n(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=n(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=n(e=>{s(e,o),e.title&&o.setTitle(e.title),e.rules.map(e=>o.addRule(m(e)))},`populateDb`),g={parser:{parse:n(e=>{o.clear(),r.debug(`[ABNF Parser] Starting Langium parse`);let n=c.parse(e);if(n.lexerErrors.length>0||n.parserErrors.length>0)throw new t(n);let i=n.value;r.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),r.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:o}},db:o,renderer:a,styles:i};export{g as diagram};

package/dist/client/assets/{arc-XLwEHJAC.js → arc-DBJ10kEG.js}

@@ -1 +1 @@
-import{n as e,t}from"./path-fybaL0A-.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-BhcyxeU7.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i){if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};
+import{n as e,t}from"./path-fybaL0A-.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-F1lvNP36.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i){if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};

package/dist/client/assets/architecture-WOLXFQ4H-C2zkRoho.js

@@ -0,0 +1 @@
+import"./chunk-NGNAAXSQ-Bw6tHVuc.js";export{M as createArchitectureServices}from"./mermaid-parser.core-C6WMZqm3.js";