@diffohq/diffo 0.8.0 → 0.9.0

package/README.md

@@ -60,8 +60,9 @@ That's the whole install. From then on, one command in any session:
 
 Every time, the agent opens the review and hands you the URL. It all runs on your
 machine, and nothing needs to be committed or pushed first. A pull request needs the
-[GitHub CLI](https://cli.github.com) signed in, and that is the only time Diffo touches
-the network.
+[GitHub CLI](https://cli.github.com) signed in. Otherwise the only thing that leaves your
+machine is two small anonymous usage events per review, [listed in full](https://diffohq.github.io/diffo/telemetry)
+and off with `diffo telemetry off`.
 
 New here? [**Your first review, end to end**](https://diffohq.github.io/diffo/tutorial) takes about five minutes.
 
@@ -135,10 +136,11 @@ read, and the agent is there to answer, explain, and fix.
   <img alt="Diffo's architecture: your agent writes the code and opens the review; a local Diffo server watches the changeset and serves it to your browser; your comments and Finish review return to the agent through diffo poll, and its answers and fixes land back in the review live." src="assets/how-it-works-light.svg" width="100%">
 </picture>
 
-One process on your machine, bound to loopback: no account, no cloud, no telemetry, and
-no model API. Diffo spawns no agents of its own; the one you're already talking to stays
-attached through the `diffo` CLI, so your comments land in the session that holds the
-context. The viewer itself is a real diff viewer, unified and split, keyboard-first, with
+One process on your machine, bound to loopback: no account, no cloud, and no model API.
+The one thing it reports is [anonymous usage data](https://diffohq.github.io/diffo/telemetry),
+two events per review, never code or paths, off with one command. Diffo spawns no agents
+of its own; the one you're already talking to stays attached through the `diffo` CLI, so
+your comments land in the session that holds the context. The viewer itself is a real diff viewer, unified and split, keyboard-first, with
 GitHub's conventions. The CLI, the flags, and everything under the hood are in the docs.
 
 ## Docs

package/dist/cli.mjs

@@ -17,9 +17,24 @@ import { stream, streamSSE } from "hono/streaming";
 * token only: under Claude Code the transient `zsh -c` carries
 * `~/.claude/shell-snapshots/…` in its arguments, which would match HARNESS. */
 const SHELL = /^-?(?:\/[^ ]*\/)?(?:sh|bash|zsh|fish|dash|ksh|csh|tcsh)(?:\s|$)/;
+/** The coding agents whose processes the ancestor walk recognises. Also the
+* vocabulary of the `agent` field in usage data, so a name never travels unless
+* it is on this list. */
+const HARNESS_NAMES = [
+	"claude",
+	"cursor",
+	"codex",
+	"copilot",
+	"windsurf",
+	"zed",
+	"aider",
+	"goose",
+	"cline",
+	"amp"
+];
 /** A harness name at the start of a path segment or token, never inside a word:
 * `/Users/sam/example/` must not read as `amp`. */
-const HARNESS = /(?:^|[/\s@_.-])(?:claude|cursor|codex|copilot|windsurf|zed|aider|goose|cline|amp)/i;
+const HARNESS = new RegExp(`(?:^|[/\\s@_.-])(${HARNESS_NAMES.join("|")})`, "i");
 function ownInvocation() {
 	return {
 		script: process.argv[1] ?? "",
@@ -34,15 +49,26 @@ function isOwnLauncher(command, own) {
 	const tail = own.args.length > 0 ? `${base} ${own.args.join(" ")}` : own.script;
 	return command === tail || command.endsWith(` ${tail}`) || command.endsWith(`/${tail}`);
 }
-function pickSessionAncestor(ancestors, own = ownInvocation()) {
+function pickSession(ancestors, own) {
 	for (const row of ancestors) {
 		const command = row.command.trim();
 		if (SHELL.test(command)) continue;
 		if (isOwnLauncher(command, own)) continue;
-		if (HARNESS.test(command)) return row.pid;
+		const match = HARNESS.exec(command);
+		if (match) return {
+			pid: row.pid,
+			harness: match[1].toLowerCase()
+		};
 	}
 	return null;
 }
+function pickSessionAncestor(ancestors, own = ownInvocation()) {
+	return pickSession(ancestors, own)?.pid ?? null;
+}
+/** Which agent the session is, by the same walk that finds it. */
+function pickSessionHarness(ancestors, own = ownInvocation()) {
+	return pickSession(ancestors, own)?.harness ?? null;
+}
 function readRow(pid) {
 	try {
 		const ppid = Number.parseInt(execFileSync("ps", [
@@ -87,6 +113,16 @@ function detectSessionPid() {
 		return null;
 	}
 }
+/** The agent this CLI runs under, or null from a plain terminal (and on Windows,
+* where the walk is not available). Only ever a name from HARNESS_NAMES. */
+function detectHarness() {
+	if (process.platform === "win32") return null;
+	try {
+		return pickSessionHarness(readAncestors(process.ppid));
+	} catch {
+		return null;
+	}
+}
 //#endregion
 //#region src/shared/review.ts
 const THREAD_INTENTS = ["fix", "question"];
@@ -1019,6 +1055,8 @@ For the reviewer:
   setup              Register diffo with the coding agents on this machine
                      (Claude Code, Cursor, VS Code, Copilot, and the shared
                      ~/.agents skills dir read by Codex, Gemini, Amp, Goose, …)
+  telemetry          Show whether anonymous usage data is on, or turn it
+                     off / on (telemetry off · telemetry on)
 
 For the agent (the AI that wrote the change):
   poll               Wait for the reviewer's feedback (blocking long-poll;
@@ -1358,7 +1396,33 @@ are left as they are, and opening a review keeps installed skills fresh
 automatically after upgrades.
 
 Example:
-  diffo setup`
+  diffo setup`,
+	telemetry: `diffo telemetry: anonymous usage data — see it, turn it off, turn it on
+
+Usage: diffo telemetry [status|on|off]
+
+Diffo reports two small anonymous events per review, "opened" and "finished":
+its version, your OS and Node major, the kind of review (working tree, branch,
+or pull request), which agent opened it, counts (threads, comments, whether
+layers were used, a coarse duration), and whether it runs from a source
+checkout (dev: true, so the maintainers' own use is filtered out). Never code,
+paths, repository or branch names, comment text, or anything about the pull
+request. One random id per machine tells returning machines from new ones;
+turning reporting off deletes it. Nothing is sent until the review page has
+shown you the notice once, and nothing from the review that showed it.
+
+  status   what is on, why it is off, and where the data goes (the default)
+  off      stop reporting on this machine, for good (a machine that has
+           reported sends one last event saying so, then nothing)
+  on       start again (a fresh id, not the old history)
+
+DIFFO_TELEMETRY_DISABLED=1 or DO_NOT_TRACK=1 turns it off for one process or
+for a fleet; CI does the same. DIFFO_TELEMETRY_DEBUG=1 writes every payload to
+the server log before it is sent. The full list, and where it goes:
+https://diffohq.github.io/diffo/telemetry
+
+Example:
+  diffo telemetry off`
 };
 function helpFor(topic) {
 	return topic && VERB_HELP[topic] || HELP_TEXT;
@@ -1370,6 +1434,7 @@ const VERBS = /* @__PURE__ */ new Set([
 	"layers",
 	"end",
 	"setup",
+	"telemetry",
 	"status",
 	"stop",
 	"clean",
@@ -1653,9 +1718,38 @@ function parseCleanVerb(rest) {
 		all: parsed.value.values.all === true
 	};
 }
+function parseTelemetryVerb(rest) {
+	const parsed = tryParse(() => parseArgs({
+		args: rest,
+		allowPositionals: true,
+		options: { help: {
+			type: "boolean",
+			short: "h"
+		} }
+	}));
+	if (!parsed.ok) return {
+		kind: "error",
+		message: parsed.message
+	};
+	const { values, positionals } = parsed.value;
+	if (values.help) return {
+		kind: "help",
+		topic: "telemetry"
+	};
+	const action = positionals[0] ?? "status";
+	if (positionals.length > 1 || action !== "status" && action !== "on" && action !== "off") return {
+		kind: "error",
+		message: "telemetry takes one of: status, on, off"
+	};
+	return {
+		kind: "telemetry",
+		action
+	};
+}
 function parseVerb(verb, rest) {
 	if (verb === "layers") return parseLayersVerb(rest);
 	if (verb === "clean") return parseCleanVerb(rest);
+	if (verb === "telemetry") return parseTelemetryVerb(rest);
 	const parsed = tryParse(() => parseArgs({
 		args: rest,
 		allowPositionals: true,
@@ -5993,6 +6087,286 @@ var ChangesetStore = class {
 	}
 };
 //#endregion
+//#region src/server/telemetry.ts
+/**
+* Anonymous usage data, Homebrew-style: on by default, announced in the review
+* page before the first event leaves, one command or variable to turn off. Two
+* events per review, `review_opened` and `review_finished`, and one last
+* `telemetry_disabled` when a machine that has reported turns it off; `send`
+* below is the authoritative list of what they carry, and the docs page links here. Never
+* paths, repo or branch names, code, comment text, or anything IP-derived. The
+* one identifier is a random UUID per machine, in a file of its own
+* (`~/.diffo/telemetry-id`) rather than the database: `DIFFO_DB` moves the
+* database, and one machine must count as one machine wherever it keeps state.
+*/
+const TELEMETRY_DOCS_URL = "https://diffohq.github.io/diffo/telemetry";
+/** PostHog Cloud, EU region. */
+const TELEMETRY_ENDPOINT = "https://eu.i.posthog.com/capture/";
+/** Public and write-only by design: adds events, reads nothing. */
+const TELEMETRY_KEY = "phc_nZqoJQbyccraPZwxsHaaAUPjf5z8k2QqdnWYedyhQzCx";
+/** A blocked host must never be felt. */
+const SEND_TIMEOUT_MS = 2e3;
+function defaultMachineIdPath() {
+	return join(homedir(), ".diffo", "telemetry-id");
+}
+const UUID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+/** One UUID and a newline, owner-readable like the database next to it. */
+function fileMachineIdStore(path = defaultMachineIdPath()) {
+	const read = () => {
+		try {
+			const id = readFileSync(path, "utf8").trim();
+			return UUID.test(id) ? id : null;
+		} catch {
+			return null;
+		}
+	};
+	const write = (id, flag) => {
+		try {
+			mkdirSync(dirname(path), {
+				recursive: true,
+				mode: 448
+			});
+			writeFileSync(path, `${id}\n`, {
+				flag,
+				mode: 384
+			});
+			return id;
+		} catch (err) {
+			if (flag === "wx" && err.code === "EEXIST") return read() ?? write(id, "w");
+			return null;
+		}
+	};
+	return {
+		read,
+		claim: (id) => write(id, "wx"),
+		clear() {
+			try {
+				rmSync(path, { force: true });
+			} catch {}
+		}
+	};
+}
+const KEY = {
+	enabled: "telemetry.enabled",
+	notice: "telemetry.notice",
+	noticeAt: "telemetry.notice_at"
+};
+const DEBUG_VAR = "DIFFO_TELEMETRY_DEBUG";
+/** `0`, `false` and the empty string mean unset, as most tools read them. */
+function isSet(value) {
+	if (value === void 0) return false;
+	const v = value.trim().toLowerCase();
+	return v !== "" && v !== "0" && v !== "false";
+}
+/** The variable that turns usage data off, in precedence order, or null. */
+function envOptOut(env) {
+	if (isSet(env.DO_NOT_TRACK)) return "DO_NOT_TRACK";
+	if (isSet(env.DIFFO_TELEMETRY_DISABLED)) return "DIFFO_TELEMETRY_DISABLED";
+	if (isSet(env.CI)) return "CI";
+	return null;
+}
+function reviewKindOf(spec, pr) {
+	if (pr) return "pr";
+	return spec.kind === "branch" ? "branch" : "tree";
+}
+const AGENT_NAMES = /* @__PURE__ */ new Set([...HARNESS_NAMES, "unknown"]);
+/** Whatever the CLI reported, narrowed to a name this build knows. */
+function agentNameOf(raw) {
+	return typeof raw === "string" && AGENT_NAMES.has(raw) ? raw : "other";
+}
+/** Coarse on purpose: low cardinality, and nobody's afternoon timed. */
+function durationBucket(ms) {
+	const min = ms / 6e4;
+	if (min < 5) return "<5m";
+	if (min < 15) return "5-15m";
+	if (min < 30) return "15-30m";
+	if (min < 60) return "30-60m";
+	return "1h+";
+}
+var Telemetry = class {
+	store;
+	ids;
+	env;
+	fetchImpl;
+	now;
+	log;
+	dev;
+	base;
+	/** Decided at open and kept for the finish, so the review that first shows the
+	* notice reports nothing at all. Undefined until the first open. */
+	armedReview;
+	openedAt;
+	inFlight = [];
+	/** The id this process settled on when the disk refused to keep one. */
+	fallbackId;
+	constructor(deps) {
+		this.store = deps.store;
+		this.ids = deps.ids;
+		this.env = deps.env ?? process.env;
+		this.fetchImpl = deps.fetch ?? globalThis.fetch;
+		this.now = deps.now ?? Date.now;
+		this.log = deps.log ?? (() => {});
+		this.dev = deps.dev ?? false;
+		this.base = {
+			version: deps.version ?? VERSION,
+			platform: deps.platform ?? process.platform,
+			arch: deps.arch ?? process.arch,
+			node: deps.nodeMajor ?? Number.parseInt(process.versions.node.split(".")[0] ?? "0", 10),
+			dev: this.dev
+		};
+	}
+	status() {
+		const shared = {
+			notice: this.notice(),
+			machineId: this.readMachineId(),
+			dev: this.dev,
+			docs: TELEMETRY_DOCS_URL
+		};
+		const variable = envOptOut(this.env);
+		if (variable !== null) return {
+			enabled: false,
+			source: "env",
+			variable,
+			...shared
+		};
+		if (this.store.get(KEY.enabled) === "false") return {
+			enabled: false,
+			source: "setting",
+			...shared
+		};
+		return {
+			enabled: true,
+			source: "default",
+			...shared
+		};
+	}
+	/**
+	* Either way the machine id goes: back on means a new history, not the old one.
+	* Turning off a machine that has reported sends one last `telemetry_disabled`,
+	* so opt-outs can be counted; a machine that never reported, or one an
+	* environment variable already silenced, sends nothing. Returns whether it did.
+	*/
+	setEnabled(on, source) {
+		const farewell = !on && this.status().enabled && this.readMachineId() !== null;
+		if (farewell) this.send("telemetry_disabled", { source });
+		this.store.set(KEY.enabled, on ? "true" : "false");
+		this.ids.clear();
+		this.fallbackId = void 0;
+		if (on) this.store.set(KEY.notice, "acknowledged");
+		return farewell;
+	}
+	/** The review page rendered the notice; nothing is sent before this. */
+	noticeShown() {
+		if (this.notice() !== "pending") return;
+		this.store.set(KEY.notice, "shown");
+		this.store.set(KEY.noticeAt, new Date(this.now()).toISOString());
+	}
+	/** The reviewer dismissed the notice; it stays gone. */
+	acknowledge() {
+		if (this.notice() === "pending") this.noticeShown();
+		this.store.set(KEY.notice, "acknowledged");
+	}
+	/** A `diffo` open landed. Decides whether this review reports, and says so. */
+	reviewOpened(props) {
+		const armed = this.armed();
+		this.armedReview = armed;
+		this.openedAt = this.now();
+		if (armed) this.send("review_opened", {
+			kind: props.kind,
+			agent: props.agent
+		});
+		return armed;
+	}
+	/** Only for a review that was opened reporting, and is still on. */
+	reviewFinished(props) {
+		if (this.armedReview !== true || !this.armed()) return;
+		this.send("review_finished", {
+			kind: props.kind,
+			threads: props.threads,
+			comments: props.comments,
+			layers: props.layers,
+			...props.event ? { event: props.event } : {},
+			...this.openedAt !== void 0 ? { duration: durationBucket(this.now() - this.openedAt) } : {}
+		});
+	}
+	/** Every send so far, settled. Tests, and an orderly shutdown. */
+	async flush() {
+		await Promise.all(this.inFlight.splice(0));
+	}
+	notice() {
+		const raw = this.store.get(KEY.notice);
+		return raw === "shown" || raw === "acknowledged" ? raw : "pending";
+	}
+	/** On, and the notice has been on a screen at least once. */
+	armed() {
+		return this.status().enabled && this.notice() !== "pending";
+	}
+	readMachineId() {
+		return this.ids.read() ?? this.fallbackId ?? null;
+	}
+	/** The id on disk, minted on the first send; a disk that refuses it gets one
+	* id per process rather than one per event. */
+	machineId() {
+		const existing = this.readMachineId();
+		if (existing) return existing;
+		const id = randomUUID();
+		const claimed = this.ids.claim(id);
+		if (claimed === null) this.fallbackId = id;
+		return claimed ?? id;
+	}
+	send(event, properties) {
+		const payload = {
+			api_key: TELEMETRY_KEY,
+			event,
+			distinct_id: this.machineId(),
+			timestamp: new Date(this.now()).toISOString(),
+			properties: {
+				$lib: "diffo",
+				$lib_version: this.base.version,
+				$process_person_profile: false,
+				$geoip_disable: true,
+				...this.base,
+				...properties
+			}
+		};
+		if (isSet(this.env[DEBUG_VAR])) this.log(`telemetry: ${JSON.stringify(payload)}`);
+		let request;
+		try {
+			request = this.fetchImpl(TELEMETRY_ENDPOINT, {
+				method: "POST",
+				headers: { "Content-Type": "application/json" },
+				body: JSON.stringify(payload),
+				signal: AbortSignal.timeout(SEND_TIMEOUT_MS)
+			});
+		} catch {
+			request = Promise.resolve();
+		}
+		const debug = isSet(this.env[DEBUG_VAR]);
+		this.inFlight.push(request.then((res) => {
+			if (debug) this.log(`telemetry: ${event} → ${res?.status ?? "sent"}`);
+		}, (err) => {
+			if (debug) this.log(`telemetry: ${event} failed: ${err?.message ?? String(err)}`);
+		}));
+	}
+};
+/** `diffo telemetry [status|on|off]`, for a human's terminal. */
+function describeTelemetry(status, asked, farewell = false) {
+	const lines = [];
+	if (status.enabled) {
+		lines.push(asked === "on" ? "usage data: on (turned on)" : "usage data: on", "  what: two anonymous events per review (opened, finished) — version, OS,", "        kind of review, agent, counts. Never code, paths, or comment text", "  off:  diffo telemetry off, or DIFFO_TELEMETRY_DISABLED=1 (DO_NOT_TRACK=1 counts too)");
+		if (status.notice === "pending") lines.push("  note: nothing has been sent yet; the review page shows a notice first");
+		if (status.dev) lines.push("  dev:  running from a source checkout; every event carries dev: true");
+	} else {
+		const why = status.source === "env" ? `${status.variable} is set` : asked === "off" ? "turned off" : "turned off with `diffo telemetry off`";
+		lines.push(`usage data: off (${why})`);
+		if (asked === "off" && farewell) lines.push("  sent: one last event saying this machine turned it off; nothing follows");
+		if (asked === "on" && status.source === "env") lines.push(`  note: it stays off while ${status.variable} is set`);
+		if (status.source === "setting") lines.push("  on:   diffo telemetry on");
+	}
+	lines.push(`  more: ${status.docs}`);
+	return lines.join("\n");
+}
+//#endregion
 //#region src/server/watcher.ts
 const DEBOUNCE_MS = 300;
 const MAX_COOLDOWN_MS = 5e3;
@@ -6185,6 +6559,31 @@ function createApp(ctx, store, review, queue) {
 		ctx.uiSettings.set("theme", theme);
 		return c.json({ ok: true });
 	});
+	const reviewKind = () => reviewKindOf(ctx.spec, ctx.pr !== void 0);
+	app.get("/api/telemetry", (c) => {
+		if (!ctx.telemetry) return c.json({ error: "telemetry unavailable" }, 503);
+		return c.json(ctx.telemetry.status());
+	});
+	app.put("/api/telemetry", async (c) => {
+		if (!ctx.telemetry) return c.json({ error: "telemetry unavailable" }, 503);
+		const body = await c.req.json().catch(() => null);
+		if (typeof body?.enabled === "boolean") ctx.telemetry.setEnabled(body.enabled, "ui");
+		else if (body?.notice === "shown") ctx.telemetry.noticeShown();
+		else if (body?.notice === "acknowledged") ctx.telemetry.acknowledge();
+		else return c.json({ error: "need {enabled: boolean} or {notice: \"shown\" | \"acknowledged\"}" }, 400);
+		return c.json(ctx.telemetry.status());
+	});
+	app.post("/api/telemetry/opened", async (c) => {
+		const body = await c.req.json().catch(() => null);
+		const armed = ctx.telemetry?.reviewOpened({
+			kind: reviewKind(),
+			agent: agentNameOf(body?.agent)
+		}) ?? false;
+		return c.json({
+			ok: true,
+			armed
+		});
+	});
 	const repoInfo = () => store?.get().repo ?? {
 		path: resolve(ctx.root),
 		name: "",
@@ -6543,6 +6942,13 @@ function createApp(ctx, store, review, queue) {
 			queue.enqueueFinish(coverage);
 		}
 		review.clearWithheld(batch.map((t) => t.id));
+		ctx.telemetry?.reviewFinished({
+			kind: reviewKind(),
+			threads: batch.length,
+			comments: batch.reduce((n, t) => n + t.messages.filter((m) => m.author === "reviewer").length, 0),
+			layers: (review.get().layers?.items.length ?? 0) > 0,
+			...event !== null ? { event } : {}
+		});
 		return c.json({
 			threads: review.get().threads,
 			prompt,
@@ -6870,6 +7276,15 @@ function startServer(options) {
 	queue.rescope(store.get().repo.branch);
 	rehydrateQueue(review, queue);
 	const log = (message) => console.log(`[${(/* @__PURE__ */ new Date()).toLocaleTimeString("en-GB", { hour12: false })}] ${message}`);
+	const telemetry = options.telemetry ?? new Telemetry({
+		store: {
+			get: (key) => db.getUiSetting(key),
+			set: (key, value) => db.setUiSetting(key, value)
+		},
+		ids: fileMachineIdStore(),
+		log,
+		dev: SRC_STAMP !== null || IS_DEV
+	});
 	queue.subscribe((presence) => log(`agent ${presence}`));
 	const unsubscribeBatch = queue.onBatchClosed((closed) => {
 		const took = Math.round((closed.closedAt - closed.deliveredAt) / 1e3);
@@ -6931,6 +7346,7 @@ function startServer(options) {
 		...options,
 		...prContext ? { pr: prContext } : {},
 		idle,
+		telemetry,
 		onShutdownRequest: options.onShutdownRequest ?? (() => process.exit(0)),
 		uiSettings: {
 			get: (key) => db.getUiSetting(key),
@@ -6963,6 +7379,7 @@ function startServer(options) {
 		store,
 		review,
 		queue,
+		telemetry,
 		stopWatching: async () => {
 			idle?.stop();
 			puller?.stop();
@@ -7686,6 +8103,21 @@ function printBaseHint(spec, files) {
 	if (!hint) return;
 	console.log(`this branch has ${hint.commits} ${hint.commits === 1 ? "commit" : "commits"} since ${hint.base}. Review them with \`diffo --base ${hint.base}\``);
 }
+/**
+* Tells the server a review was opened, and by which agent — the server decides
+* whether that is reported (see server/telemetry.ts). Best effort: an open never
+* waits on, or fails over, usage data.
+*/
+async function announceOpen(port) {
+	try {
+		await fetch(apiUrl(port, "/api/telemetry/opened"), {
+			method: "POST",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify({ agent: detectHarness() ?? "unknown" }),
+			signal: AbortSignal.timeout(2e3)
+		});
+	} catch {}
+}
 async function fetchChangesetInfo(port) {
 	try {
 		return await (await fetch(apiUrl(port, "/api/changeset"), { signal: AbortSignal.timeout(2e3) })).json();
@@ -7829,6 +8261,26 @@ if (command.kind === "stop") {
 	}
 	process.exit(0);
 }
+if (command.kind === "telemetry") {
+	const db = new DiffoDb();
+	try {
+		const telemetry = new Telemetry({
+			store: {
+				get: (key) => db.getUiSetting(key),
+				set: (key, value) => db.setUiSetting(key, value)
+			},
+			ids: fileMachineIdStore(),
+			log: (line) => process.stderr.write(`diffo: ${line}\n`),
+			dev: SRC_STAMP !== null || process.env.ENV === "development"
+		});
+		const farewell = command.action === "status" ? false : telemetry.setEnabled(command.action === "on", "cli");
+		console.log(describeTelemetry(telemetry.status(), command.action, farewell));
+		await telemetry.flush();
+	} finally {
+		db.close();
+	}
+	process.exit(0);
+}
 if (command.kind === "poll") {
 	const port = await requireServer();
 	process.stderr.write("diffo: waiting for the reviewer. Keep this process attended: a tracked\nbackground task or the foreground, never detached. If it dies, just\nre-run it; feedback is held in the review and survives.\n");
@@ -8047,6 +8499,7 @@ if (existing !== null && takeOverPort === null) {
 	const url = reviewUrl(existing.port);
 	console.log(prOpen ? "diffo is already watching this pull request" : "diffo is already watching this repo");
 	await warnSpecMismatch(existing.port, spec);
+	await announceOpen(existing.port);
 	console.log(`→ ${url}`);
 	await printAgentNextStep(existing.port, prOpen?.pr);
 	process.exit(0);
@@ -8066,6 +8519,7 @@ if (existing !== null && takeOverPort !== null) {
 }
 if (!command.foreground) {
 	const daemonPort = await requireServer(command.port, spec.kind === "branch" ? spec.base : void 0, SRC_STAMP, prOpen ? command.target : void 0);
+	await announceOpen(daemonPort);
 	const url = reviewUrl(daemonPort);
 	await printChangesetSummary(daemonPort);
 	console.log(`→ ${url}`);
@@ -8107,6 +8561,10 @@ try {
 	}
 	throw err;
 }
+started.telemetry.reviewOpened({
+	kind: reviewKindOf(spec, prOpen !== null),
+	agent: detectHarness() ?? "unknown"
+});
 const { stats, repo } = started.store.get();
 const url = reviewUrl(port);
 const label = prOpen ? `pull request #${prOpen.pr.number} "${prOpen.pr.title}"` : specLabel(spec);

package/dist/client/assets/{abnfDiagram-O67JEVCF-BUIFF3Lv.js → abnfDiagram-O67JEVCF-Dh4is2Nj.js}

@@ -1 +1 @@
-import{m as e,t}from"./mermaid-parser.core-CCoRJPAh.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-BWz11yEV.js";import"./chunk-O7XYJQB3-Bb8gIxaB.js";import{n as i,r as a,t as o}from"./chunk-GTNCS2PH-BWkvnYRd.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-SnjyC0ZV.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-CwcOI9nG.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-Bmfs-TFn.js";import"./chunk-O7XYJQB3-BjcjInBL.js";import{n as i,r as a,t as o}from"./chunk-GTNCS2PH-at0ENjhB.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-BabKb2iQ.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-x6Ulf5MI.js → arc-CMN0Gb1g.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-uIEEXDIm.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-Cezvykkn.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-7GRP2DOG-DaIFNiR2.js

@@ -0,0 +1 @@
+import"./chunk-FOHPRMQF-DzxwRta7.js";import{M as e}from"./mermaid-parser.core-CwcOI9nG.js";export{e as createArchitectureServices};