@dev.fast/whiteboard 0.2.5-preview.20261013.113 → 0.2.5-preview.20261015.116

package/dist/headless-host-Cp37C-v0.js

@@ -0,0 +1,2 @@
+import { i as withHeadlessServerLock, n as runHeadlessServer, r as takeOverServerHome, t as HeadlessServerBusyError } from "./headless-host-CVXTSIlx.js";
+export { HeadlessServerBusyError, runHeadlessServer, takeOverServerHome, withHeadlessServerLock };

package/dist/input-error-OLgB_h31.js

@@ -0,0 +1,11 @@
+//#region src/review-api/input-error.ts
+/** Deliberately safe to show to API clients, unlike filesystem/provider errors. */
+var ReviewInputError = class extends Error {
+	status;
+	constructor(message, status = 400) {
+		super(message);
+		this.status = status;
+	}
+};
+//#endregion
+export { ReviewInputError as t };

package/dist/{local-data-Dy1pNtLF.js → local-data-yfLp5PiJ.js}

@@ -1,9 +1,10 @@
-import { C as ReviewInputError, _ as readFileAtCommit, b as resolveRevision, c as diffTrees, f as gitCommonDir, g as parseGitRemote, h as listTrackedFilesAtCommit, i as diff, l as diffWorkingTree, m as listCommitRange, n as createBlobBatchReader, o as diffFileSummariesTrees, p as jjRevisionIsConflicted, r as detectLocalVcs, s as diffFileSummariesWorkingTree, t as LocalVcsToolsMissingError, u as git, v as resolveRepoContext, y as resolveRepoContextSync } from "./src-DiHP11l3.js";
+import { _ as parseGitRemote, a as diff, b as resolveRepoContextSync, c as diffFileSummariesWorkingTree, d as git, g as listTrackedFilesAtCommit, h as listCommitRange, i as detectLocalVcs, l as diffTrees, m as jjRevisionIsConflicted, n as createBlobBatchReader, p as gitCommonDir, s as diffFileSummariesTrees, t as LocalVcsToolsMissingError, u as diffWorkingTree, v as readFileAtCommit, x as resolveRevision, y as resolveRepoContext } from "./src-BMPuxWv4.js";
 import { c as jsonString, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import { _ as SCRATCHPAD_REVIEW_ID, b as structuralChangeCounts, g as ReviewSelectedDiffSchema, m as ReviewApiSelectionSourceSchema, y as decodeStructuralDiffEvent } from "./src-WCbabvdS.js";
-import { s as safeStorageSegment } from "./review-home-paths-Y9mYWkCI.js";
-import { $ as unionIntervals, A as applyLensEdit, C as resourceReferences, D as summarizeEdit, E as sourceReferences, F as checkSourcePath, I as fileLineRangeSchema, K as sourceAnchors, L as requireVisibleSource, N as lensSelections, O as checkReferences, P as SourceRangeError, Q as subtractIntervals, R as sliceSourceRange, S as resourceReference, X as intersectIntervals, Y as emptyCoverage, a as defineSoftwareMap, b as pinsSchema, c as traceSchema, d as applyEdit, et as updateCoverage, f as assignFreshIds, g as explicitPins, h as elements, i as SoftwareModelValidationError, j as lensEditSchema, l as textIncludesQuote, m as editSchema, n as migrateStoredDocument, p as documentSchema, q as coverageSchema, r as migrateStoredLenses, t as liftFileLenses, u as anchorPins, v as isUnit, w as reviewTargetSchema } from "./stored-document-migration-u4L_1VZa.js";
-import { D as processIsAlive, O as withFileLock, P as writePrivateJsonAtomic } from "./s3-config-CAxhO9u_.js";
+import { C as ReviewApiSelectionSourceSchema, M as structuralChangeCounts, O as ReviewSelectedDiffSchema, j as decodeStructuralDiffEvent, k as SCRATCHPAD_REVIEW_ID } from "./src-Bgvf1sEN.js";
+import { s as safeStorageSegment } from "./review-home-paths-zwevIq4I.js";
+import { $ as unionIntervals, A as applyLensEdit, C as resourceReferences, D as summarizeEdit, E as sourceReferences, F as checkSourcePath, I as fileLineRangeSchema, K as sourceAnchors, L as requireVisibleSource, N as lensSelections, O as checkReferences, P as SourceRangeError, Q as subtractIntervals, R as sliceSourceRange, S as resourceReference, X as intersectIntervals, Y as emptyCoverage, a as defineSoftwareMap, b as pinsSchema, c as traceSchema, d as applyEdit, et as updateCoverage, f as assignFreshIds, g as explicitPins, h as elements, i as SoftwareModelValidationError, j as lensEditSchema, l as textIncludesQuote, m as editSchema, n as migrateStoredDocument, p as documentSchema, q as coverageSchema, r as migrateStoredLenses, t as liftFileLenses, u as anchorPins, v as isUnit, w as reviewTargetSchema } from "./stored-document-migration-CFOvjRwW.js";
+import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
+import { A as withFileLock, I as writePrivateJsonAtomic, O as processIsAlive } from "./s3-config-DDWWMh5x.js";
 import { t as errorMessage } from "./error-message-OtiDonty.js";
 import { r as isMissingFileError } from "./fs-utils-BMPLt0cr.js";
 import { createRequire } from "node:module";
@@ -1665,26 +1666,30 @@ var ReviewActivity = class {
 };
 /** A semantic selection, independent of comment/thread creation. */
 const AgentSelectionSchema = z.strictObject({
-	target: z.discriminatedUnion("kind", [z.strictObject({
-		kind: z.literal("text"),
-		quote: z.string(),
-		/** Where the quote is: characters `start` to `end` of the text of the
-		* review block `blockId`, which was `text` then. The block keeps its
-		* id across versions; comparing its text then and now finds the quote
-		* again, or tells that it changed. */
-		anchor: z.strictObject({
-			blockId: z.string().min(1).max(200),
-			start: z.number().int().nonnegative(),
-			end: z.number().int().positive(),
-			text: z.string().max(2e4)
-		}).refine((anchor) => anchor.start < anchor.end, "an empty anchor").optional()
-	}), z.strictObject({
-		kind: z.literal("code"),
-		path: z.string(),
-		side: z.enum(["base", "head"]),
-		startLine: z.number().int().positive(),
-		endLine: z.number().int().positive()
-	})]),
+	target: z.discriminatedUnion("kind", [
+		z.strictObject({
+			kind: z.literal("text"),
+			quote: z.string(),
+			/** Where the quote is: characters `start` to `end` of the text of the
+			* review block `blockId`, which was `text` then. The block keeps its
+			* id across versions; comparing its text then and now finds the quote
+			* again, or tells that it changed. */
+			anchor: z.strictObject({
+				blockId: z.string().min(1).max(200),
+				start: z.number().int().nonnegative(),
+				end: z.number().int().positive(),
+				text: z.string().max(2e4)
+			}).refine((anchor) => anchor.start < anchor.end, "an empty anchor").optional()
+		}),
+		z.strictObject({
+			kind: z.literal("code"),
+			path: z.string(),
+			side: z.enum(["base", "head"]),
+			startLine: z.number().int().positive(),
+			endLine: z.number().int().positive()
+		}),
+		z.strictObject({ kind: z.literal("review") })
+	]),
 	title: z.string().max(500),
 	detail: z.string().max(24e3).optional(),
 	revision: z.string().max(200).optional(),
@@ -1694,6 +1699,7 @@ const AgentSelectionSchema = z.strictObject({
 function selectionMarkdown(selection, sourceExcerpt = "", diffPaths) {
 	const target = selection.target;
 	if (selection.selectedDiff) return selectedDiffMarkdown(selection.selectedDiff, diffPaths);
+	if (target.kind === "review") return "";
 	if (target.kind === "text") return target.quote.split(/\r?\n/).map((line) => `> ${line}`).join("\n");
 	return [
 		`## ${selection.title}`,
@@ -1872,6 +1878,13 @@ const statusSchema = z.enum([
 	"idle",
 	"failed"
 ]);
+/** What an agent may do without asking: nothing that changes the checkout,
+* what its own judgment calls safe, or everything. */
+const askPermissionsSchema = z.enum([
+	"read-only",
+	"auto",
+	"bypass"
+]);
 /** Everything the Ask panel renders. A watcher gets it whole once, then
 * follows it through `AskChange`s. */
 const askThreadStateSchema = z.object({
@@ -1884,8 +1897,7 @@ const askThreadStateSchema = z.object({
 	signIn: z.string().optional(),
 	/** The mode that keeps the agent from changing files was accepted. */
 	readOnly: z.boolean(),
-	/** The agent edits and runs commands without asking. */
-	bypass: z.boolean(),
+	permissions: askPermissionsSchema,
 	head: z.string(),
 	/** The checkout the agent works in. */
 	cwd: z.string(),
@@ -1913,7 +1925,7 @@ const askChangeSchema = z.discriminatedUnion("type", [
 		type: z.literal("set"),
 		status: statusSchema.optional(),
 		readOnly: z.boolean().optional(),
-		bypass: z.boolean().optional(),
+		permissions: askPermissionsSchema.optional(),
 		choices: askChoicesSchema.optional(),
 		commands: z.array(askCommandSchema).optional(),
 		accepts: askAcceptsSchema.optional(),
@@ -1969,7 +1981,7 @@ function applyAskChange(state, change) {
 				...rest,
 				status: change.status ?? state.status,
 				readOnly: change.readOnly ?? state.readOnly,
-				bypass: change.bypass ?? state.bypass
+				permissions: change.permissions ?? state.permissions
 			};
 			const choices = change.choices ?? state.choices;
 			if (choices) next.choices = choices;
@@ -2037,8 +2049,8 @@ const askRecordSchema = z.object({
 	cwd: z.string(),
 	/** What the panel showed last; absent until a turn ends or it closes. */
 	entries: z.array(askEntrySchema).optional(),
-	/** The agent edits and runs commands without asking. */
-	bypass: z.boolean().optional()
+	/** What the agent may do without asking, when it reopens. */
+	permissions: askPermissionsSchema.optional()
 });
 const rowSchema = z.object({
 	id: z.string(),
@@ -2053,7 +2065,8 @@ const rowSchema = z.object({
 	created_at: z.string(),
 	updated_at: z.string(),
 	entries: z.string().nullable(),
-	bypass: z.number()
+	bypass: z.number(),
+	permissions: z.string().nullable()
 }).transform((row) => askRecordSchema.parse({
 	id: row.id,
 	reviewId: row.review_id,
@@ -2067,7 +2080,7 @@ const rowSchema = z.object({
 	createdAt: row.created_at,
 	updatedAt: row.updated_at,
 	entries: row.entries === null ? void 0 : JSON.parse(row.entries),
-	bypass: row.bypass === 1
+	permissions: row.permissions ?? (row.bypass === 1 ? "bypass" : "read-only")
 }));
 /** Saved Ask conversations, per review. Shared reviews are not in
 * `reviews`, so there is no foreign key; deleting a review deletes its rows. */
@@ -2093,9 +2106,12 @@ var AskHistory = class {
       updated_at TEXT NOT NULL,
       entries TEXT,
       bypass INTEGER NOT NULL DEFAULT 0,
+      permissions TEXT,
       UNIQUE(agent, session_id));
     CREATE INDEX IF NOT EXISTS ask_conversations_review ON ask_conversations(review_id, updated_at);`);
-		if (!db.prepare("PRAGMA table_info(ask_conversations)").all().some((column) => String(column.name) === "bypass")) db.exec("ALTER TABLE ask_conversations ADD COLUMN bypass INTEGER NOT NULL DEFAULT 0");
+		const columns = new Set(db.prepare("PRAGMA table_info(ask_conversations)").all().map((column) => String(column.name)));
+		if (!columns.has("bypass")) db.exec("ALTER TABLE ask_conversations ADD COLUMN bypass INTEGER NOT NULL DEFAULT 0");
+		if (!columns.has("permissions")) db.exec("ALTER TABLE ask_conversations ADD COLUMN permissions TEXT");
 		db.exec("CREATE TABLE IF NOT EXISTS ask_agent_offers(agent TEXT PRIMARY KEY, offer TEXT NOT NULL)");
 		db.exec("CREATE TABLE IF NOT EXISTS ask_agent_model_offers(agent TEXT NOT NULL, model TEXT NOT NULL, offer TEXT NOT NULL, PRIMARY KEY(agent, model))");
 	}
@@ -2113,13 +2129,13 @@ var AskHistory = class {
 	}
 	save(record) {
 		this.assertWritable(record.reviewId);
-		this.db.prepare(`INSERT OR REPLACE INTO ask_conversations(id,review_id,agent,session_id,version,head,cwd,selection,title,created_at,updated_at,bypass)
-        VALUES(?,?,?,?,?,?,?,?,?,?,?,?)`).run(record.id, record.reviewId, record.agent, record.sessionId, record.version, record.head, record.cwd, JSON.stringify(record.selection), record.title, record.createdAt, record.updatedAt, record.bypass ? 1 : 0);
+		this.db.prepare(`INSERT OR REPLACE INTO ask_conversations(id,review_id,agent,session_id,version,head,cwd,selection,title,created_at,updated_at,bypass,permissions)
+        VALUES(?,?,?,?,?,?,?,?,?,?,?,?,?)`).run(record.id, record.reviewId, record.agent, record.sessionId, record.version, record.head, record.cwd, JSON.stringify(record.selection), record.title, record.createdAt, record.updatedAt, record.permissions === "bypass" ? 1 : 0, record.permissions ?? null);
 	}
-	/** Whether reopening the conversation bypasses permissions. */
-	setBypass(id, bypass) {
+	/** What the agent may do without asking when the conversation reopens. */
+	setPermissions(id, permissions) {
 		this.assertThreadWritable(id);
-		this.db.prepare("UPDATE ask_conversations SET bypass=? WHERE id=?").run(bypass ? 1 : 0, id);
+		this.db.prepare("UPDATE ask_conversations SET bypass=?, permissions=? WHERE id=?").run(permissions === "bypass" ? 1 : 0, permissions, id);
 	}
 	/** Points a conversation at a new session, when its agent could not
 	* reopen the one it had. */
@@ -2160,7 +2176,7 @@ var AskHistory = class {
 	/** Newest first. */
 	list(reviewId) {
 		return this.db.prepare("SELECT * FROM ask_conversations WHERE review_id=? ORDER BY updated_at DESC").all(reviewId).map((row) => {
-			const { reviewId: _reviewId, sessionId: _sessionId, version: _version, cwd: _cwd, entries, bypass: _bypass, ...entry } = rowSchema.parse(row);
+			const { reviewId: _reviewId, sessionId: _sessionId, version: _version, cwd: _cwd, entries, permissions: _permissions, ...entry } = rowSchema.parse(row);
 			const question = entries?.find((entry) => entry.kind === "user")?.text;
 			if (question?.trim()) entry.question = question;
 			return entry;
@@ -2187,19 +2203,21 @@ function pullRequestKey(url) {
 	return url.toLowerCase();
 }
 /** Omission preserves identity; null detaches it without changing import metadata. */
-function setPullRequest(snapshot, url) {
+function setPullRequest(snapshot, url, title) {
 	if (url === void 0) return;
 	if (url === null) {
 		if (snapshot.origin) {
 			delete snapshot.origin.pullRequestUrl;
 			delete snapshot.origin.pullRequestNumber;
+			delete snapshot.origin.pullRequestTitle;
 		}
 		return;
 	}
 	snapshot.origin = {
 		...snapshot.origin,
 		pullRequestUrl: url,
-		pullRequestNumber: Number(url.split("/").at(-1))
+		pullRequestNumber: Number(url.split("/").at(-1)),
+		pullRequestTitle: title ?? (snapshot.origin?.pullRequestUrl === url ? snapshot.origin.pullRequestTitle : void 0)
 	};
 }
 //#endregion
@@ -2677,8 +2695,8 @@ var ReviewStore = class {
 			listener();
 		} catch {}
 	}
-	/** Managed records are discoverable even if their preparation stamp was lost. */
-	tutorialIds() {
+	/** Reviews of the tutorial earlier versions shipped, for their removal. */
+	legacyTutorialIds() {
 		return this.db.prepare(`SELECT reviews.id FROM reviews JOIN versions ON versions.review_id=reviews.id AND versions.version=reviews.version WHERE json_extract(versions.snapshot,'$.origin.tutorial') = 1`).all().map((row) => String(row.id));
 	}
 	list(mode = "structural") {
@@ -2696,8 +2714,7 @@ var ReviewStore = class {
         JOIN versions ON versions.review_id=reviews.id AND versions.version=reviews.version
         LEFT JOIN review_attention ON review_attention.review_id=reviews.id
         LEFT JOIN repositories ON repositories.id=json_extract(versions.snapshot,'$.pins.repositoryId')
-        WHERE COALESCE(json_extract(versions.snapshot,'$.origin.tutorial'), 0) = 0
-        ${id === void 0 ? "" : "AND reviews.id=?"}
+        ${id === void 0 ? "" : "WHERE reviews.id=?"}
         ORDER BY reviews.rowid`).all(...id === void 0 ? [] : [id]).map((row) => {
 			const summary = JSON.parse(String(row.summary));
 			if (summary.pins) summary.target ??= {
@@ -2763,13 +2780,14 @@ var ReviewStore = class {
 	inspect(id, targetId, version) {
 		return inspectSnapshot(this.read(id, version), targetId);
 	}
-	/** The host can seed a managed document; transport callers only supply a command. */
+	/** The host can seed a document; transport callers only supply a command. */
 	execute(input, initial) {
 		if (this.closing) return Promise.reject(/* @__PURE__ */ new Error("Review store is closing."));
 		const command = commandSchema.parse(input);
 		if (initial && command.operation.type !== "create") throw new ReviewInputError("Initial content requires a create command.");
 		const pullRequest = this.startPullRequest(command);
 		pullRequest?.catch(() => {});
+		const pullRequestTitle = pullRequest ? pullRequest.then(({ title }) => title || void 0, () => void 0) : this.startPullRequestTitle(command);
 		const run = this.pending.then(async () => {
 			const op = command.operation;
 			this.purgeExpired();
@@ -2821,7 +2839,7 @@ var ReviewStore = class {
 			}
 			const id = op.type !== "create" ? op.reviewId : op.kind === "scratchpad" ? SCRATCHPAD_ID : randomUUID();
 			const previous = op.type === "create" ? void 0 : this.read(id);
-			let snapshot = op.type === "create" ? createdSnapshot(id, op, resolvedTarget, fromPullRequest?.title) : structuredClone(previous);
+			let snapshot = op.type === "create" ? createdSnapshot(id, op, resolvedTarget, fromPullRequest && (fromPullRequest.title || `PR #${op.pullRequestUrl.split("/").at(-1)}`)) : structuredClone(previous);
 			delete snapshot.trashedAt;
 			delete snapshot.sourceUnavailable;
 			delete snapshot.lastEdit;
@@ -2839,6 +2857,10 @@ var ReviewStore = class {
 					};
 				}
 			}
+			if (requestedTarget) snapshot.origin = {
+				...snapshot.origin,
+				baseBranch: requestedTarget.base === resolvedTarget?.pins.base ? void 0 : requestedTarget.base
+			};
 			switch (op.type) {
 				case "create":
 					if (initial) {
@@ -2849,13 +2871,13 @@ var ReviewStore = class {
 						};
 						for (const block of snapshot.document) assignFreshIds(block, (prefix) => `${prefix}-${++nextId}`);
 					}
-					setPullRequest(snapshot, op.pullRequestUrl);
+					setPullRequest(snapshot, op.pullRequestUrl, await pullRequestTitle);
 					break;
 				case "rename":
 					snapshot.title = op.title;
 					break;
 				case "set_target":
-					setPullRequest(snapshot, op.pullRequestUrl ?? (op.pullRequestUrl === null || snapshot.pins?.repositoryId !== resolvedTarget.pins.repositoryId ? null : void 0));
+					setPullRequest(snapshot, op.pullRequestUrl ?? (op.pullRequestUrl === null || snapshot.pins?.repositoryId !== resolvedTarget.pins.repositoryId ? null : void 0), await pullRequestTitle);
 					snapshot.staleSources = [];
 					snapshot.target = resolvedTarget.target;
 					snapshot.pins = resolvedTarget.pins;
@@ -2942,12 +2964,16 @@ var ReviewStore = class {
 			preferred: existing && this.read(existing).pins?.repositoryId
 		});
 	}
+	startPullRequestTitle(command) {
+		const op = command.operation;
+		if (op.type !== "create" && op.type !== "set_target" || !op.pullRequestUrl) return Promise.resolve(void 0);
+		return this.providers.pullRequestTitle?.(op.pullRequestUrl) ?? Promise.resolve(void 0);
+	}
 	/** Reviews whose PR is this one, newest version first. Summaries only. */
 	reviewsForPullRequest(url) {
 		return this.db.prepare(`SELECT reviews.id FROM reviews
         JOIN versions ON versions.review_id=reviews.id AND versions.version=reviews.version
         WHERE lower(json_extract(versions.snapshot,'$.origin.pullRequestUrl'))=?
-          AND COALESCE(json_extract(versions.snapshot,'$.origin.tutorial'), 0) = 0
         ORDER BY json_extract(versions.snapshot,'$.createdAt') DESC, reviews.rowid DESC`).all(pullRequestKey(url)).map((row) => String(row.id));
 	}
 	assertReusableReview(reviewId) {
@@ -3114,7 +3140,7 @@ function inspectSnapshot(snapshot, targetId) {
 }
 //#endregion
 //#region src/review-api/workspaces.ts
-const OWNED_ELSEWHERE = "Another Desktop owns this review's language workspaces. Quit that Desktop, then retry.";
+const OWNED_ELSEWHERE = "Another Whiteboard server is managing this review's language workspaces. Retry once it stops.";
 /** Local lifecycle only: source and authored history never depend on preparation.
 * Status/queue/process lifecycle follows #334, retaining the legacy prepare config
 * and per-review managed checkout layout instead of a new settings system.
@@ -3147,10 +3173,20 @@ var ReviewWorkspaces = class {
 			this.db.close();
 			throw error;
 		}
-		this.stop = store.subscribeCatalog(() => {
+		const stopCatalog = store.subscribeCatalog(() => {
 			this.collect();
 			this.releaseTrashed();
 		});
+		const stopDeleted = store.subscribe(({ reviewId, deleted }) => {
+			if (!deleted) return;
+			const repositories = this.repositoryDirs();
+			repositories.catch(() => void 0);
+			this.release(reviewId, repositories, () => !this.hasReview(reviewId));
+		});
+		this.stop = () => {
+			stopCatalog();
+			stopDeleted();
+		};
 		this.collect();
 		this.releaseTrashed();
 	}
@@ -3169,7 +3205,11 @@ var ReviewWorkspaces = class {
 		if (!released.length) return;
 		const repositories = this.repositoryDirs();
 		repositories.catch(() => void 0);
-		for (const reviewId of released) this.release(reviewId, repositories);
+		for (const reviewId of released) this.release(reviewId, repositories, () => this.stillTrashed(reviewId));
+	}
+	/** Restored since the move to Trash: its checkouts may be in use. */
+	stillTrashed(reviewId) {
+		return this.store.trashedIds().includes(reviewId);
 	}
 	async repositoryDirs() {
 		const dirs = new Set(this.all().map((item) => item.repository));
@@ -3183,21 +3223,21 @@ var ReviewWorkspaces = class {
 	released(reviewId) {
 		return this.releasing.get(reviewId) ?? Promise.resolve();
 	}
-	release(reviewId, repositories = this.repositoryDirs()) {
+	release(reviewId, repositories, stillReleased) {
 		const requests = [...this.requests.values()];
 		const done = this.cleanup.then(async () => {
 			await Promise.allSettled(requests);
-			await this.releaseCheckouts(reviewId, await repositories);
+			await this.releaseCheckouts(reviewId, await repositories, stillReleased);
 		}).catch((error) => {
-			console.error(`Could not free the pinned checkouts of trashed review ${reviewId}:`, error);
+			console.error(`Could not free the pinned checkouts of review ${reviewId}:`, error);
 		}).finally(() => {
 			if (this.releasing.get(reviewId) === done) this.releasing.delete(reviewId);
 		});
 		this.releasing.set(reviewId, done);
 		this.cleanup = done;
 	}
-	async releaseCheckouts(reviewId, repositories) {
-		if (!this.store.trashedIds().includes(reviewId)) return;
+	async releaseCheckouts(reviewId, repositories, stillReleased) {
+		if (!stillReleased()) return;
 		const environments = this.all().filter((item) => item.reviewId === reviewId);
 		const checkouts = repositories.filter((repository) => existsSync(reviewManagedCheckoutRoot(repository, reviewId)));
 		if (!environments.length && !checkouts.length || !this.claim(reviewId)) return;
@@ -3292,7 +3332,7 @@ var ReviewWorkspaces = class {
 	async retryCleanup(id) {
 		const environment = this.get(id);
 		if (!environment || environment.state !== "cleanup-failed") throw new ReviewInputError("Cleanup failure not found.", 404);
-		if (this.store.has(environment.reviewId)) this.release(environment.reviewId);
+		if (this.store.has(environment.reviewId)) this.release(environment.reviewId, this.repositoryDirs(), () => this.stillTrashed(environment.reviewId));
 		else this.collect(id);
 		await this.cleanup;
 	}
@@ -3466,6 +3506,7 @@ var ReviewWorkspaces = class {
 						]);
 					}
 					if (environment.rootPath) await removeReviewPrepareArtifacts(environment.rootPath);
+					if (environment.repository && !this.all().some((other) => other.id !== environment.id && other.reviewId === environment.reviewId && other.repository === environment.repository)) await removeReviewManagedCheckouts(environment.repository, environment.reviewId);
 					this.db.prepare("DELETE FROM pinned_environments WHERE id=?").run(environment.id);
 					this.db.prepare("DELETE FROM workspace_leases WHERE review_id=? AND owner=? AND NOT EXISTS(SELECT 1 FROM pinned_environments WHERE value->>'reviewId'=?)").run(environment.reviewId, this.ownerId, environment.reviewId);
 				} catch (error) {
@@ -4263,9 +4304,18 @@ var LocalReviewData = class {
 				base,
 				head
 			},
-			title: pullRequest.title.trim() || `PR #${address.number}`
+			title: pullRequest.title.trim()
 		};
 	}
+	async pullRequestTitle(url) {
+		const { provider, address } = resolvePullRequest(url);
+		try {
+			const { title } = await provider.readPullRequest(address, this.options.pullRequests ?? defaultPullRequestDeps);
+			return title.trim() || void 0;
+		} catch {
+			return;
+		}
+	}
 	/** A registered checkout with a remote for host/owner/repo, and that remote. */
 	async pullRequestCheckout(provider, address, repository, deps) {
 		const registered = this.store.repositories();
@@ -4685,6 +4735,7 @@ function openLocalReviewStore(databasePath, options = {}) {
 		projectSource: (snapshot, pins) => data.projectSource(snapshot, pins),
 		resolveTarget: (target, pinned) => data.resolveTarget(target, pinned),
 		resolvePullRequest: (url, repository) => data.resolvePullRequest(url, repository),
+		pullRequestTitle: (url) => data.pullRequestTitle(url),
 		headBranch: (pins, headRef) => data.headBranch(pins, headRef),
 		sourcePins: (snapshot) => data.sourcePins(snapshot),
 		unavailableAnchors: (snapshot) => data.unavailableAnchors(snapshot),
@@ -4729,4 +4780,4 @@ async function readWorkspace(file) {
 	}
 }
 //#endregion
-export { invalidateStructuralComparisons as _, inspectSnapshot as a, askAgentIds as c, askQuestionSchema as d, AgentSelectionSchema as f, activityUpdateSchema as g, activityEndSchema as h, commandSchema as i, askChoiceKinds as l, activityBeginSchema as m, uploadSchema as n, initializeReviewStoreSchema as o, selectionMarkdown as p, SCRATCHPAD_ID as r, applyAskChange as s, openLocalReviewStore as t, askPicksSchema as u, diffrExecutable as v, diffrMissingError as y };
+export { activityUpdateSchema as _, inspectSnapshot as a, diffrMissingError as b, askAgentIds as c, askPicksSchema as d, askQuestionSchema as f, activityEndSchema as g, activityBeginSchema as h, commandSchema as i, askChoiceKinds as l, selectionMarkdown as m, uploadSchema as n, initializeReviewStoreSchema as o, AgentSelectionSchema as p, SCRATCHPAD_ID as r, applyAskChange as s, openLocalReviewStore as t, askPermissionsSchema as u, invalidateStructuralComparisons as v, diffrExecutable as y };

package/dist/{mcp-B8kuRSXM.js → mcp-DBbDwG4y.js}

@@ -1,7 +1,7 @@
-import { C as ReviewInputError } from "./src-DiHP11l3.js";
-import { a as toolResultText, s as ReviewApiError } from "./agent-client-P3J4y8Zn.js";
-import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-fdA0RPIT.js";
-import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-CJHBCHMh.js";
+import { a as toolResultText, s as ReviewApiError } from "./agent-client-DVd-prfp.js";
+import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
+import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-BXKEwhfi.js";
+import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-BXWzQG8T.js";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ListToolsRequestSchema } from "@modelcontextprotocol/sdk/types.js";

package/dist/{package-paths-B9PDoAl6.js → package-paths-BsuI4WUf.js}

@@ -1,5 +1,5 @@
 import { c as jsonString, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import "./review-home-paths-Y9mYWkCI.js";
+import "./review-home-paths-zwevIq4I.js";
 import { readFileSync } from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";

package/dist/profile-BSOkA6A-.js

@@ -0,0 +1,2 @@
+import { t as openReviewProfile } from "./profile-pAzg7fxC.js";
+export { openReviewProfile };

package/dist/{profile-tbGWuWHu.js → profile-pAzg7fxC.js}

@@ -1,6 +1,6 @@
-import { o as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-Dy1pNtLF.js";
-import "./review-home-paths-Y9mYWkCI.js";
-import { O as withFileLock } from "./s3-config-CAxhO9u_.js";
+import { o as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-yfLp5PiJ.js";
+import "./review-home-paths-zwevIq4I.js";
+import { A as withFileLock } from "./s3-config-DDWWMh5x.js";
 import { access, mkdir } from "node:fs/promises";
 import path from "node:path";
 import { DatabaseSync } from "node:sqlite";

package/dist/remote-attach-BAt2eZx3.js

@@ -0,0 +1,137 @@
+import { i as readReviewPackageVersion } from "./package-paths-BsuI4WUf.js";
+import { a as readReviewServerHealth, l as serverNotReady } from "./server-discovery-CifQ3yQS.js";
+import { c as stopBackgroundServer, o as recordedBackgroundServer, r as ensureBackgroundServer } from "./background-server-Cw7CcfaZ.js";
+import { t as ensureRemoteLanguageServer } from "./remote-language-server-DQgWUNeV.js";
+import path from "node:path";
+import { spawn } from "node:child_process";
+import { gt, valid } from "semver";
+//#region src/remote-toolchains.ts
+const PROBE_TIMEOUT_MS = 1e4;
+const TOOLCHAINS = {
+	rust: [["cargo", "--version"], ["rustc", "--version"]],
+	swift: [["swift", "--version"]],
+	csharp: [["dotnet", "--version"]],
+	go: [["go", "version"]]
+};
+function probe(command, env, timeoutMs) {
+	const shell = env.SHELL || "/bin/sh";
+	const flags = ["csh", "tcsh"].includes(path.basename(shell)) ? ["-ic"] : [
+		"-i",
+		"-l",
+		"-c"
+	];
+	return new Promise((resolve) => {
+		const child = spawn(shell, [...flags, command.join(" ")], {
+			env,
+			detached: true,
+			stdio: "ignore"
+		});
+		const timer = setTimeout(() => {
+			child.removeAllListeners("exit");
+			try {
+				process.kill(-child.pid, "SIGKILL");
+			} catch {}
+			resolve(`${command.join(" ")} did not answer within ${timeoutMs / 1e3} s`);
+		}, timeoutMs);
+		child.once("error", (error) => {
+			clearTimeout(timer);
+			resolve(`the login shell ${shell} did not run: ${error.message}`);
+		});
+		child.once("exit", (code) => {
+			clearTimeout(timer);
+			resolve(code === 0 ? "found" : code === 127 ? "missing" : `${command.join(" ")} failed with exit code ${code ?? "none"}`);
+		});
+	});
+}
+async function missingToolchains(groups, env, { timeoutMs = PROBE_TIMEOUT_MS, toolchains = TOOLCHAINS } = {}) {
+	const entries = await Promise.all(groups.map(async (group) => {
+		const commands = Object.hasOwn(toolchains, group) ? toolchains[group] : [];
+		const outcomes = await Promise.all(commands.map((command) => probe(command, env, timeoutMs)));
+		const missing = commands.filter((_, i) => outcomes[i] === "missing").map(([tool]) => tool);
+		const failures = outcomes.filter((outcome) => outcome !== "found" && outcome !== "missing");
+		const details = [...missing.length > 0 ? [`${missing.join(" and ")} ${missing.length > 1 ? "were" : "was"} not found on the login shell's PATH`] : [], ...failures];
+		return details.length > 0 ? [[group, details.join("; ")]] : [];
+	}));
+	return new Map(entries.flat());
+}
+//#endregion
+//#region src/remote-attach.ts
+async function remoteAttach(input) {
+	const extensions = new AbortController();
+	const language = ensureRemoteLanguageServer({
+		env: input.env,
+		packageRoot: input.packageRoot,
+		groups: input.groups,
+		signal: extensions.signal,
+		ensure: input.ensureExtensions,
+		installTimeoutMs: input.installTimeoutMs,
+		cli: input.cli
+	});
+	const toolchains = missingToolchains(input.groups ?? [], input.env);
+	let server;
+	let previousVersion;
+	let incompatibleRunning;
+	try {
+		const version = input.version ?? readReviewPackageVersion(import.meta.url);
+		const other = input.replace ? await otherVersionRunning(input.stateDir, version) : void 0;
+		if (other && (other.startedBy === "user" || newer(other.version, version))) incompatibleRunning = other;
+		else if (other) {
+			await stopBackgroundServer({ serverPid: other.pid });
+			previousVersion = other.version;
+		}
+		server = await ensureBackgroundServer({
+			stateDir: input.stateDir,
+			env: input.env,
+			startedBy: "desktop",
+			cli: input.cli
+		});
+	} catch (error) {
+		extensions.abort();
+		await Promise.all([language, toolchains]);
+		throw error;
+	}
+	const { discovery, started } = server;
+	const { languageServer, languageServerDetail, languageServerPending, languageGroups } = await language;
+	const missing = await toolchains;
+	const health = await readReviewServerHealth(discovery);
+	if (!health) throw serverNotReady(input.stateDir);
+	return {
+		event: "remote.attach",
+		version: health.version ?? null,
+		commit: health.commit ?? null,
+		serverId: health.serverId ?? null,
+		url: discovery.url,
+		token: discovery.token,
+		startedServer: started,
+		languageServer,
+		...languageServerDetail !== void 0 && { languageServerDetail },
+		...languageServerPending && { languageServerPending },
+		languageGroups: languageGroups.map(({ group, installed, detail }) => {
+			const details = [detail, missing.get(group)].filter(Boolean).join("; ");
+			return {
+				group,
+				installed,
+				...details && { detail: details }
+			};
+		}),
+		...previousVersion !== void 0 && {
+			replaced: true,
+			previousVersion
+		},
+		...incompatibleRunning && { incompatibleRunning }
+	};
+}
+const newer = (running, own) => valid(running) !== null && valid(own) !== null && gt(running, own);
+async function otherVersionRunning(stateDir, version) {
+	const recorded = await recordedBackgroundServer(stateDir);
+	if (!recorded) return void 0;
+	const { discovery, health } = recorded;
+	const running = health.version ?? "unknown";
+	return running === version ? void 0 : {
+		version: running,
+		pid: discovery.serverPid,
+		startedBy: discovery.startedBy
+	};
+}
+//#endregion
+export { remoteAttach };

package/dist/remote-extensions-2Cxf3hMo.js

@@ -0,0 +1,2 @@
+import { t as ensureRemoteExtensions } from "./remote-extensions-BiNx6NQK.js";
+export { ensureRemoteExtensions };