camox 0.62.0 → 0.65.1

package/dist/features/studio/components/ReviewDraftSheet.js

@@ -2,19 +2,21 @@ import { useDraftScope } from "../../../lib/draft-scope.js";
 import { reads, useReadQuery, useStudioCache } from "../../../lib/studio-reads.js";
 import { cn } from "../../../lib/utils.js";
 import { getDraftReadClient } from "../../../lib/api-client.js";
-import { draftMutations } from "../../../lib/queries.js";
+import { draftMutations, patrolMutations } from "../../../lib/queries.js";
 import { previewStore } from "../../preview/previewStore.js";
+import { studioStore } from "../studioStore.js";
 import { localeLabel } from "../localeOptions.js";
 import { draftLabel, plural, timeAgo } from "../draftLabel.js";
+import { askAgentMessage, findingSentence, fixLabel } from "../findingSentence.js";
 import { useShowInCanvas } from "../showInCanvas.js";
 import { c } from "react/compiler-runtime";
 import * as React from "react";
-import { CONFLICTS_CHANGED_ERROR, REVISION_CHANGED_ERROR, TRANSLATIONS_PENDING_ERROR } from "@camox/api-contract/drafts";
+import { CHECKS_RUNNING_ERROR, CONFLICTS_CHANGED_ERROR, FINDINGS_OPEN_ERROR, REVISION_CHANGED_ERROR, TRANSLATIONS_PENDING_ERROR } from "@camox/api-contract/drafts";
 import { Fragment, jsx, jsxs } from "react/jsx-runtime";
 import { useMutation } from "@tanstack/react-query";
 import { Button } from "@camox/ui/button";
 import { Sheet, SheetContent, SheetDescription, SheetTitle } from "@camox/ui/sheet";
-import { CircleCheck, OctagonX, Replace, Trash2, TriangleAlert, Unlink } from "lucide-react";
+import { CircleAlert, CircleCheck, OctagonX, Replace, Trash2, TriangleAlert, Unlink } from "lucide-react";
 import { toast } from "@camox/ui/toaster";
 import { Skeleton } from "@camox/ui/skeleton";
 import { AlertDialog, AlertDialogAction, AlertDialogCancel, AlertDialogContent, AlertDialogDescription, AlertDialogFooter, AlertDialogHeader, AlertDialogTitle } from "@camox/ui/alert-dialog";
@@ -77,7 +79,6 @@ function blockerActionLabel(problem) {
 		case "unacceptedFileType": return isImage(problem) ? "Choose another image" : "Choose another file";
 		case "emptyRequiredReference": return "Choose a record";
 		case "danglingReference": return "Choose another record";
-		case "danglingPageLink": return "Fix the link";
 		case "duplicatePath": return "Fix the path";
 		case "missingMetaTitle": return "Write a meta title";
 		default: return "Show in Canvas";
@@ -144,7 +145,7 @@ function Card(t0) {
 	} else t6 = $[8];
 	let t7;
 	if ($[9] !== children) {
-		t7 = /* @__PURE__ */ jsx("div", {
+		t7 = children && /* @__PURE__ */ jsx("div", {
 			className: "text-muted-foreground text-sm",
 			children
 		});
@@ -248,6 +249,138 @@ function ProblemCards({ problems, tone, icon, onShow, onOpenThread }) {
 		}, `shared:${message}`)];
 	});
 }
+/** Where the Canvas shows a finding: its block or item on its page. */
+function findingTarget(finding) {
+	const { place } = finding;
+	if (place?.kind !== "page") return null;
+	return {
+		pageId: place.pageId,
+		locale: finding.locale,
+		blockId: place.blockId,
+		itemId: place.itemId,
+		field: place.blockId ? {
+			name: finding.field,
+			kind: null
+		} : null
+	};
+}
+/**
+* One card per open finding of the draft, which holds its publish (ADR 0033), with the Patrol's
+* actions: show it in the Canvas, apply its fix, or dismiss it.
+*/
+function FindingCards(t0) {
+	const $ = c(19);
+	const { projectId, findings, onChanged, onShow } = t0;
+	let t1;
+	if ($[0] === Symbol.for("react.memo_cache_sentinel")) {
+		t1 = patrolMutations.applyFix();
+		$[0] = t1;
+	} else t1 = $[0];
+	let t2;
+	if ($[1] !== onChanged) {
+		t2 = {
+			...t1,
+			onSuccess: (t3) => {
+				const { applied } = t3;
+				if (!applied) toast.info("The text changed since it was checked, so it's being checked again.");
+				onChanged();
+			},
+			onError: _temp
+		};
+		$[1] = onChanged;
+		$[2] = t2;
+	} else t2 = $[2];
+	const applyFix = useMutation(t2);
+	let t3;
+	if ($[3] === Symbol.for("react.memo_cache_sentinel")) {
+		t3 = patrolMutations.dismissFinding();
+		$[3] = t3;
+	} else t3 = $[3];
+	let t4;
+	if ($[4] !== onChanged) {
+		t4 = {
+			...t3,
+			onSuccess: onChanged,
+			onError: _temp2
+		};
+		$[4] = onChanged;
+		$[5] = t4;
+	} else t4 = $[5];
+	const dismiss = useMutation(t4);
+	const busy = applyFix.isPending || dismiss.isPending;
+	let t5;
+	if ($[6] !== applyFix || $[7] !== busy || $[8] !== dismiss || $[9] !== findings || $[10] !== onShow || $[11] !== projectId) {
+		let t6;
+		if ($[13] !== applyFix || $[14] !== busy || $[15] !== dismiss || $[16] !== onShow || $[17] !== projectId) {
+			t6 = (finding) => {
+				const target = findingTarget(finding);
+				return /* @__PURE__ */ jsx(Card, {
+					tone: "danger",
+					icon: CircleAlert,
+					title: findingSentence(finding),
+					action: /* @__PURE__ */ jsxs("span", {
+						className: "flex flex-wrap gap-1",
+						children: [
+							target && /* @__PURE__ */ jsx(Button, {
+								variant: "outline",
+								size: "sm",
+								onClick: () => onShow(target),
+								children: "Show"
+							}),
+							finding.fix && /* @__PURE__ */ jsx(Button, {
+								variant: "outline",
+								size: "sm",
+								disabled: busy,
+								onClick: () => applyFix.mutate({
+									projectId,
+									findingId: finding.id
+								}),
+								children: fixLabel(finding.fix)
+							}),
+							/* @__PURE__ */ jsx(Button, {
+								variant: "outline",
+								size: "sm",
+								onClick: () => studioStore.trigger.askAgent({ text: askAgentMessage(finding, finding.patroller.role) }),
+								children: "Ask the agent"
+							}),
+							/* @__PURE__ */ jsx(Button, {
+								variant: "ghost",
+								size: "sm",
+								disabled: busy,
+								onClick: () => dismiss.mutate({
+									projectId,
+									findingId: finding.id
+								}),
+								children: "Dismiss"
+							})
+						]
+					})
+				}, finding.id);
+			};
+			$[13] = applyFix;
+			$[14] = busy;
+			$[15] = dismiss;
+			$[16] = onShow;
+			$[17] = projectId;
+			$[18] = t6;
+		} else t6 = $[18];
+		t5 = findings.map(t6);
+		$[6] = applyFix;
+		$[7] = busy;
+		$[8] = dismiss;
+		$[9] = findings;
+		$[10] = onShow;
+		$[11] = projectId;
+		$[12] = t5;
+	} else t5 = $[12];
+	return t5;
+}
+function _temp2(error_0) {
+	return toast.error(error_0.message || "Could not dismiss it");
+}
+function _temp(error) {
+	return toast.error(error.message || "Could not apply the fix");
+}
 /** Who made Live's version of a conflict, as a sentence's subject: a member, a release, or none. */
 function whoChanged(conflict) {
 	if (conflict.release) return "A release";
@@ -450,9 +583,9 @@ function UseLiveVersionButton(t0) {
 * then lost changes: one card each, with no section heading.
 */
 function Cards(t0) {
-	const $ = c(33);
+	const $ = c(39);
 	const { projectId, preview, onChanged, onShow, onOpenThread } = t0;
-	const { blockers, conflicts, lost, changes, defaultLocale } = preview;
+	const { blockers, findings, conflicts, lost, changes, defaultLocale } = preview;
 	let t1;
 	if ($[0] !== preview.warnings) {
 		t1 = preview.warnings.filter(isShownWarning);
@@ -460,10 +593,10 @@ function Cards(t0) {
 		$[1] = t1;
 	} else t1 = $[1];
 	const warnings = t1;
-	if (blockers.length + warnings.length + conflicts.length + lost.length === 0) return null;
+	if (blockers.length + findings.length + warnings.length + conflicts.length + lost.length === 0) return null;
 	let t2;
 	if ($[2] !== changes) {
-		t2 = new Set(changes.filter(_temp).map(_temp2));
+		t2 = new Set(changes.filter(_temp3).map(_temp4));
 		$[2] = changes;
 		$[3] = t2;
 	} else t2 = $[3];
@@ -483,24 +616,38 @@ function Cards(t0) {
 		$[7] = t3;
 	} else t3 = $[7];
 	let t4;
-	if ($[8] !== onOpenThread || $[9] !== onShow || $[10] !== warnings) {
-		t4 = /* @__PURE__ */ jsx(ProblemCards, {
+	if ($[8] !== findings || $[9] !== onChanged || $[10] !== onShow || $[11] !== projectId) {
+		t4 = /* @__PURE__ */ jsx(FindingCards, {
+			projectId,
+			findings,
+			onChanged,
+			onShow
+		});
+		$[8] = findings;
+		$[9] = onChanged;
+		$[10] = onShow;
+		$[11] = projectId;
+		$[12] = t4;
+	} else t4 = $[12];
+	let t5;
+	if ($[13] !== onOpenThread || $[14] !== onShow || $[15] !== warnings) {
+		t5 = /* @__PURE__ */ jsx(ProblemCards, {
 			problems: warnings,
 			tone: "warning",
 			icon: TriangleAlert,
 			onShow,
 			onOpenThread
 		});
-		$[8] = onOpenThread;
-		$[9] = onShow;
-		$[10] = warnings;
-		$[11] = t4;
-	} else t4 = $[11];
-	let t5;
-	if ($[12] !== conflicts || $[13] !== defaultLocale || $[14] !== deleted || $[15] !== onChanged || $[16] !== projectId) {
-		let t6;
-		if ($[18] !== defaultLocale || $[19] !== deleted || $[20] !== onChanged || $[21] !== projectId) {
-			t6 = (conflict) => /* @__PURE__ */ jsxs(Card, {
+		$[13] = onOpenThread;
+		$[14] = onShow;
+		$[15] = warnings;
+		$[16] = t5;
+	} else t5 = $[16];
+	let t6;
+	if ($[17] !== conflicts || $[18] !== defaultLocale || $[19] !== deleted || $[20] !== onChanged || $[21] !== projectId) {
+		let t7;
+		if ($[23] !== defaultLocale || $[24] !== deleted || $[25] !== onChanged || $[26] !== projectId) {
+			t7 = (conflict) => /* @__PURE__ */ jsxs(Card, {
 				tone: "warning",
 				icon: Replace,
 				title: conflictTitle(conflict),
@@ -519,25 +666,25 @@ function Cards(t0) {
 					})
 				]
 			}, conflictKey(conflict));
-			$[18] = defaultLocale;
-			$[19] = deleted;
-			$[20] = onChanged;
-			$[21] = projectId;
-			$[22] = t6;
-		} else t6 = $[22];
-		t5 = conflicts.map(t6);
-		$[12] = conflicts;
-		$[13] = defaultLocale;
-		$[14] = deleted;
-		$[15] = onChanged;
-		$[16] = projectId;
-		$[17] = t5;
-	} else t5 = $[17];
-	let t6;
-	if ($[23] !== defaultLocale || $[24] !== lost) {
-		let t7;
-		if ($[26] !== defaultLocale) {
-			t7 = (change_1) => /* @__PURE__ */ jsxs(Card, {
+			$[23] = defaultLocale;
+			$[24] = deleted;
+			$[25] = onChanged;
+			$[26] = projectId;
+			$[27] = t7;
+		} else t7 = $[27];
+		t6 = conflicts.map(t7);
+		$[17] = conflicts;
+		$[18] = defaultLocale;
+		$[19] = deleted;
+		$[20] = onChanged;
+		$[21] = projectId;
+		$[22] = t6;
+	} else t6 = $[22];
+	let t7;
+	if ($[28] !== defaultLocale || $[29] !== lost) {
+		let t8;
+		if ($[31] !== defaultLocale) {
+			t8 = (change_1) => /* @__PURE__ */ jsxs(Card, {
 				tone: "warning",
 				icon: Unlink,
 				title: change_1.reason === "writtenAfterDeletion" ? change_1.label : `Loses the change to the ${change_1.label}`,
@@ -547,41 +694,43 @@ function Cards(t0) {
 				change_1.id,
 				change_1.locale
 			]));
-			$[26] = defaultLocale;
-			$[27] = t7;
-		} else t7 = $[27];
-		t6 = lost.map(t7);
-		$[23] = defaultLocale;
-		$[24] = lost;
-		$[25] = t6;
-	} else t6 = $[25];
-	let t7;
-	if ($[28] !== t3 || $[29] !== t4 || $[30] !== t5 || $[31] !== t6) {
-		t7 = /* @__PURE__ */ jsxs("ul", {
+			$[31] = defaultLocale;
+			$[32] = t8;
+		} else t8 = $[32];
+		t7 = lost.map(t8);
+		$[28] = defaultLocale;
+		$[29] = lost;
+		$[30] = t7;
+	} else t7 = $[30];
+	let t8;
+	if ($[33] !== t3 || $[34] !== t4 || $[35] !== t5 || $[36] !== t6 || $[37] !== t7) {
+		t8 = /* @__PURE__ */ jsxs("ul", {
 			className: "flex flex-col gap-2",
 			children: [
 				t3,
 				t4,
 				t5,
-				t6
+				t6,
+				t7
 			]
 		});
-		$[28] = t3;
-		$[29] = t4;
-		$[30] = t5;
-		$[31] = t6;
-		$[32] = t7;
-	} else t7 = $[32];
-	return t7;
+		$[33] = t3;
+		$[34] = t4;
+		$[35] = t5;
+		$[36] = t6;
+		$[37] = t7;
+		$[38] = t8;
+	} else t8 = $[38];
+	return t8;
 }
-function _temp2(change_0) {
+function _temp4(change_0) {
 	return JSON.stringify([
 		change_0.entity,
 		change_0.id,
 		change_0.locale
 	]);
 }
-function _temp(change) {
+function _temp3(change) {
 	return change.change === "deleted";
 }
 /** A quiet line saying something is done, with a check, or still running, with a spinner. */
@@ -617,11 +766,10 @@ function StatusLine(t0) {
 	return t3;
 }
 /** What Publish sends: the review's token, and the revision a draft with code shows. */
-const publishInput = (projectId, shown, publishAnyway) => ({
+const publishInput = (projectId, shown) => ({
 	projectId,
 	token: shown.token,
-	...shown.code?.revision && { revision: shown.code.revision },
-	...publishAnyway && { publishAnyway }
+	...shown.code?.revision && { revision: shown.code.revision }
 });
 /** Why a publish with code went back to the review, as the review says it. */
 const PUBLISH_ABORTED = {
@@ -632,14 +780,19 @@ const PUBLISH_ABORTED = {
 	liveCodeMoved: "Live's code changed while publishing, and the draft took too long to take it in, so the publish stopped. Try again.",
 	failed: "The publish couldn't finish on Camox's side, and Live is as it was. Try again."
 };
+/** The patrollers still checking the draft: "The Link checker is checking the draft." */
+function checkingSentence(roles) {
+	const who = listFormat.format(roles.map((role) => `the ${role}`));
+	return `${who[0].toUpperCase()}${who.slice(1)} ${roles.length > 1 ? "are" : "is"} checking the draft.`;
+}
 /**
 * What's fine or still running: the draft is up to date with Live, its latest code changes
-* building, its translations, and the page SEO being written.
+* building, its translations, its patrollers' checks, and the page SEO being written.
 */
 function StatusLines(t0) {
-	const $ = c(34);
-	const { preview, locales, waiting } = t0;
-	const { conflicts, translating, warnings, code } = preview;
+	const $ = c(36);
+	const { preview, locales } = t0;
+	const { conflicts, translating, checking, warnings, code } = preview;
 	let seoPages;
 	let t1;
 	let t2;
@@ -647,58 +800,56 @@ function StatusLines(t0) {
 	let t4;
 	let t5;
 	let t6;
-	if ($[0] !== code || $[1] !== conflicts.length || $[2] !== locales || $[3] !== translating || $[4] !== waiting || $[5] !== warnings) {
-		const translations = locales.filter(_temp3).map(_temp4);
+	if ($[0] !== code || $[1] !== conflicts.length || $[2] !== locales || $[3] !== translating || $[4] !== warnings) {
+		const translations = locales.filter(_temp5).map(_temp6);
 		let t7;
-		if ($[13] !== warnings) {
-			t7 = new Set(warnings.filter(_temp5).map(_temp6));
-			$[13] = warnings;
-			$[14] = t7;
-		} else t7 = $[14];
+		if ($[12] !== warnings) {
+			t7 = new Set(warnings.filter(_temp7).map(_temp8));
+			$[12] = warnings;
+			$[13] = t7;
+		} else t7 = $[13];
 		seoPages = t7.size;
 		const aborted = code?.publishAborted && !code.publishing ? code.publishAborted : null;
 		t1 = "flex flex-col gap-1.5";
-		if ($[15] !== aborted) {
+		if ($[14] !== aborted) {
 			t2 = aborted && /* @__PURE__ */ jsx("li", {
 				className: "text-sm text-amber-700 dark:text-amber-400",
 				children: PUBLISH_ABORTED[aborted] ?? PUBLISH_ABORTED.failed
 			});
-			$[15] = aborted;
-			$[16] = t2;
-		} else t2 = $[16];
-		if ($[17] !== code?.includesLive || $[18] !== conflicts.length) {
+			$[14] = aborted;
+			$[15] = t2;
+		} else t2 = $[15];
+		if ($[16] !== code?.includesLive || $[17] !== conflicts.length) {
 			t3 = conflicts.length === 0 && (code?.includesLive ?? true) && /* @__PURE__ */ jsx(StatusLine, {
 				done: true,
 				children: "Includes everything published to Live since this draft started"
 			});
-			$[17] = code?.includesLive;
-			$[18] = conflicts.length;
-			$[19] = t3;
-		} else t3 = $[19];
-		if ($[20] !== code?.building) {
+			$[16] = code?.includesLive;
+			$[17] = conflicts.length;
+			$[18] = t3;
+		} else t3 = $[18];
+		if ($[19] !== code?.building) {
 			t4 = code?.building && /* @__PURE__ */ jsx(StatusLine, {
 				done: false,
 				children: "Building the latest changes…"
 			});
-			$[20] = code?.building;
-			$[21] = t4;
-		} else t4 = $[21];
-		if ($[22] !== code?.publishing) {
+			$[19] = code?.building;
+			$[20] = t4;
+		} else t4 = $[20];
+		if ($[21] !== code?.publishing) {
 			t5 = code?.publishing && /* @__PURE__ */ jsx(StatusLine, {
 				done: false,
 				children: "Publishing the new code and content…"
 			});
-			$[22] = code?.publishing;
-			$[23] = t5;
-		} else t5 = $[23];
+			$[21] = code?.publishing;
+			$[22] = t5;
+		} else t5 = $[22];
 		t6 = translating.length > 0 ? /* @__PURE__ */ jsxs(StatusLine, {
 			done: false,
 			children: [
 				"Translating into ",
 				listFormat.format(translating.map(localeLabel)),
-				".",
-				" ",
-				waiting ? "Publishing once translated." : "Publishing waits for it."
+				". Publishing waits for it."
 			]
 		}) : translations.length > 0 && /* @__PURE__ */ jsxs(StatusLine, {
 			done: true,
@@ -708,27 +859,41 @@ function StatusLines(t0) {
 		$[1] = conflicts.length;
 		$[2] = locales;
 		$[3] = translating;
-		$[4] = waiting;
-		$[5] = warnings;
-		$[6] = seoPages;
-		$[7] = t1;
-		$[8] = t2;
-		$[9] = t3;
-		$[10] = t4;
-		$[11] = t5;
-		$[12] = t6;
+		$[4] = warnings;
+		$[5] = seoPages;
+		$[6] = t1;
+		$[7] = t2;
+		$[8] = t3;
+		$[9] = t4;
+		$[10] = t5;
+		$[11] = t6;
 	} else {
-		seoPages = $[6];
-		t1 = $[7];
-		t2 = $[8];
-		t3 = $[9];
-		t4 = $[10];
-		t5 = $[11];
-		t6 = $[12];
+		seoPages = $[5];
+		t1 = $[6];
+		t2 = $[7];
+		t3 = $[8];
+		t4 = $[9];
+		t5 = $[10];
+		t6 = $[11];
 	}
 	let t7;
-	if ($[24] !== seoPages) {
-		t7 = seoPages > 0 && /* @__PURE__ */ jsxs(StatusLine, {
+	if ($[23] !== checking) {
+		t7 = checking.length > 0 && /* @__PURE__ */ jsxs(StatusLine, {
+			done: false,
+			children: [
+				checkingSentence(checking.map(_temp9)),
+				" Publishing waits for",
+				" ",
+				checking.length > 1 ? "them" : "it",
+				"."
+			]
+		});
+		$[23] = checking;
+		$[24] = t7;
+	} else t7 = $[24];
+	let t8;
+	if ($[25] !== seoPages) {
+		t8 = seoPages > 0 && /* @__PURE__ */ jsxs(StatusLine, {
 			done: false,
 			children: [
 				"Writing SEO for ",
@@ -736,12 +901,12 @@ function StatusLines(t0) {
 				"…"
 			]
 		});
-		$[24] = seoPages;
-		$[25] = t7;
-	} else t7 = $[25];
-	let t8;
-	if ($[26] !== t1 || $[27] !== t2 || $[28] !== t3 || $[29] !== t4 || $[30] !== t5 || $[31] !== t6 || $[32] !== t7) {
-		t8 = /* @__PURE__ */ jsxs("ul", {
+		$[25] = seoPages;
+		$[26] = t8;
+	} else t8 = $[26];
+	let t9;
+	if ($[27] !== t1 || $[28] !== t2 || $[29] !== t3 || $[30] !== t4 || $[31] !== t5 || $[32] !== t6 || $[33] !== t7 || $[34] !== t8) {
+		t9 = /* @__PURE__ */ jsxs("ul", {
 			className: t1,
 			children: [
 				t2,
@@ -749,30 +914,36 @@ function StatusLines(t0) {
 				t4,
 				t5,
 				t6,
-				t7
+				t7,
+				t8
 			]
 		});
-		$[26] = t1;
-		$[27] = t2;
-		$[28] = t3;
-		$[29] = t4;
-		$[30] = t5;
-		$[31] = t6;
-		$[32] = t7;
-		$[33] = t8;
-	} else t8 = $[33];
-	return t8;
+		$[27] = t1;
+		$[28] = t2;
+		$[29] = t3;
+		$[30] = t4;
+		$[31] = t5;
+		$[32] = t6;
+		$[33] = t7;
+		$[34] = t8;
+		$[35] = t9;
+	} else t9 = $[35];
+	return t9;
+}
+function _temp9(t0) {
+	const { role } = t0;
+	return role;
 }
-function _temp6(warning_0) {
+function _temp8(warning_0) {
 	return warning_0.id;
 }
-function _temp5(warning) {
+function _temp7(warning) {
 	return warning.code === "metaTitlePending";
 }
-function _temp4(locale_0) {
+function _temp6(locale_0) {
 	return locale_0.tag;
 }
-function _temp3(locale) {
+function _temp5(locale) {
 	return !locale.isDefault;
 }
 /**
@@ -790,7 +961,7 @@ function DraftOverview(t0) {
 	if ($[0] === Symbol.for("react.memo_cache_sentinel")) {
 		t1 = {
 			...draftMutations.retryOverview(),
-			onError: _temp7
+			onError: _temp10
 		};
 		$[0] = t1;
 	} else t1 = $[0];
@@ -809,7 +980,7 @@ function DraftOverview(t0) {
 	if (overview.status === "ready") {
 		let t2;
 		if ($[2] !== overview.bullets) {
-			t2 = overview.bullets.map(_temp8);
+			t2 = overview.bullets.map(_temp11);
 			$[2] = overview.bullets;
 			$[3] = t2;
 		} else t2 = $[3];
@@ -850,7 +1021,7 @@ function DraftOverview(t0) {
 	if (overview.status === "failed") {
 		let t2;
 		if ($[11] !== changes) {
-			t2 = new Set(changes.map(_temp9));
+			t2 = new Set(changes.map(_temp12));
 			$[11] = changes;
 			$[12] = t2;
 		} else t2 = $[12];
@@ -953,13 +1124,13 @@ function DraftOverview(t0) {
 	} else t2 = $[33];
 	return t2;
 }
-function _temp9(change) {
+function _temp12(change) {
 	return change.entity === "locale" ? "Languages" : ownerName(change.owner, change.entity);
 }
-function _temp8(bullet, index) {
+function _temp11(bullet, index) {
 	return /* @__PURE__ */ jsx("li", { children: bullet }, index);
 }
-function _temp7(error) {
+function _temp10(error) {
 	return toast.error(error.message || "Could not summarize the draft");
 }
 /** Switches the studio to Live once its draft ended here, replacing the draft's URL entry. */
@@ -1014,7 +1185,7 @@ function useRenameDraft(projectId) {
 		t2 = {
 			mutationFn: t0,
 			onSuccess: t1,
-			onError: _temp10
+			onError: _temp13
 		};
 		$[5] = t0;
 		$[6] = t1;
@@ -1022,7 +1193,7 @@ function useRenameDraft(projectId) {
 	} else t2 = $[7];
 	return useMutation(t2);
 }
-function _temp10(error) {
+function _temp13(error) {
 	return toast.error(error.message || "Could not rename the draft");
 }
 /**
@@ -1127,7 +1298,7 @@ function DraftNameInput(t0) {
 * preview is shown. Once the draft ends, the studio switches to Live.
 */
 function ReviewDraftSheet(t0) {
-	const $ = c(101);
+	const $ = c(78);
 	const { projectId, draftId, open, startRenaming: t1, onOpenChange } = t0;
 	const startRenaming = t1 === void 0 ? false : t1;
 	const [confirmingDelete, setConfirmingDelete] = React.useState(false);
@@ -1267,7 +1438,13 @@ function ReviewDraftSheet(t0) {
 				return;
 			}
 			if (error.code === TRANSLATIONS_PENDING_ERROR) {
-				toast.info("Translations are still running. Wait for them, or publish anyway.");
+				toast.info("Translations are still running. Publishing waits for them.");
+				preview.refetch();
+				return;
+			}
+			const code = error.code;
+			if (code === FINDINGS_OPEN_ERROR || code === CHECKS_RUNNING_ERROR) {
+				toast.info(error.message);
 				preview.refetch();
 				return;
 			}
@@ -1292,66 +1469,10 @@ function ReviewDraftSheet(t0) {
 	const data = preview.data;
 	const headline = data?.overview.status === "ready" ? data.overview.headline : null;
 	const busy = publish.isPending;
-	const ready = !!data && data.blockers.length === 0 && !data.code?.building && !data.code?.publishing && !busy && !preview.isFetching;
+	const ready = !!data && data.blockers.length === 0 && data.translating.length === 0 && data.findings.length === 0 && data.checking.length === 0 && !data.code?.building && !data.code?.publishing && !busy && !preview.isFetching;
 	let t17;
-	if ($[34] !== data?.translating) {
-		t17 = data?.translating ?? [];
-		$[34] = data?.translating;
-		$[35] = t17;
-	} else t17 = $[35];
-	const translating = t17;
-	const [waiting, setWaiting] = React.useState(null);
-	const translated = !!data && !preview.isFetching && data.translating.length === 0;
-	if (waiting && translated) {
-		const appeared = data.conflicts.some((conflict) => !waiting.conflicts.has(conflictKey(conflict)));
-		if (data.blockers.length > 0 || appeared) setWaiting(null);
-	}
-	const { mutate } = publish;
-	let t18;
-	let t19;
-	if ($[36] !== busy || $[37] !== data || $[38] !== mutate || $[39] !== projectId || $[40] !== translated || $[41] !== waiting) {
-		t18 = () => {
-			if (!waiting || !translated || !data || busy) return;
-			mutate(publishInput(projectId, data), { onSettled: () => setWaiting(null) });
-		};
-		t19 = [
-			waiting,
-			translated,
-			data,
-			busy,
-			mutate,
-			projectId
-		];
-		$[36] = busy;
-		$[37] = data;
-		$[38] = mutate;
-		$[39] = projectId;
-		$[40] = translated;
-		$[41] = waiting;
-		$[42] = t18;
-		$[43] = t19;
-	} else {
-		t18 = $[42];
-		t19 = $[43];
-	}
-	React.useEffect(t18, t19);
-	let t20;
-	if ($[44] !== data || $[45] !== projectId || $[46] !== publish || $[47] !== translating.length) {
-		t20 = () => {
-			if (!data) return;
-			if (translating.length === 0) return publish.mutate(publishInput(projectId, data));
-			setWaiting({ conflicts: new Set(data.conflicts.map(conflictKey)) });
-		};
-		$[44] = data;
-		$[45] = projectId;
-		$[46] = publish;
-		$[47] = translating.length;
-		$[48] = t20;
-	} else t20 = $[48];
-	const publishWhole = t20;
-	let t21;
-	if ($[49] !== draft.data || $[50] !== headline || $[51] !== projectId) {
-		t21 = /* @__PURE__ */ jsx("div", {
+	if ($[34] !== draft.data || $[35] !== headline || $[36] !== projectId) {
+		t17 = /* @__PURE__ */ jsx("div", {
 			className: "flex flex-col gap-1 border-b p-4 pr-12",
 			children: headline ? /* @__PURE__ */ jsxs(Fragment, { children: [/* @__PURE__ */ jsxs(SheetDescription, {
 				className: "flex min-w-0 items-center gap-1",
@@ -1384,14 +1505,14 @@ function ReviewDraftSheet(t0) {
 				children: "Draft"
 			}), /* @__PURE__ */ jsx(SheetDescription, { children: draft.data ? `${draft.data.createdBy.name} · started ${timeAgo(draft.data.createdAt)}` : "Publishing applies this whole draft to Live." })] })
 		});
-		$[49] = draft.data;
-		$[50] = headline;
-		$[51] = projectId;
-		$[52] = t21;
-	} else t21 = $[52];
-	let t22;
-	if ($[53] !== data || $[54] !== locales || $[55] !== onOpenChange || $[56] !== preview || $[57] !== projectId || $[58] !== showInCanvas || $[59] !== waiting) {
-		t22 = data ? /* @__PURE__ */ jsxs("div", {
+		$[34] = draft.data;
+		$[35] = headline;
+		$[36] = projectId;
+		$[37] = t17;
+	} else t17 = $[37];
+	let t18;
+	if ($[38] !== data || $[39] !== locales || $[40] !== onOpenChange || $[41] !== preview || $[42] !== projectId || $[43] !== showInCanvas) {
+		t18 = data ? /* @__PURE__ */ jsxs("div", {
 			className: "flex min-h-0 flex-1 flex-col gap-6 overflow-y-auto p-4",
 			children: [
 				/* @__PURE__ */ jsx(DraftOverview, {
@@ -1413,161 +1534,137 @@ function ReviewDraftSheet(t0) {
 				}),
 				/* @__PURE__ */ jsx(StatusLines, {
 					preview: data,
-					locales: locales.data ?? [],
-					waiting: !!waiting
+					locales: locales.data ?? []
 				})
 			]
 		}) : /* @__PURE__ */ jsx("div", {
 			className: "flex flex-1 items-center justify-center",
 			children: /* @__PURE__ */ jsx(Spinner, { className: "text-muted-foreground" })
 		});
-		$[53] = data;
-		$[54] = locales;
-		$[55] = onOpenChange;
-		$[56] = preview;
-		$[57] = projectId;
-		$[58] = showInCanvas;
-		$[59] = waiting;
-		$[60] = t22;
-	} else t22 = $[60];
-	let t23;
-	let t24;
-	if ($[61] === Symbol.for("react.memo_cache_sentinel")) {
-		t23 = () => setConfirmingDelete(true);
-		t24 = /* @__PURE__ */ jsx(Trash2, {});
-		$[61] = t23;
-		$[62] = t24;
+		$[38] = data;
+		$[39] = locales;
+		$[40] = onOpenChange;
+		$[41] = preview;
+		$[42] = projectId;
+		$[43] = showInCanvas;
+		$[44] = t18;
+	} else t18 = $[44];
+	let t19;
+	let t20;
+	if ($[45] === Symbol.for("react.memo_cache_sentinel")) {
+		t19 = () => setConfirmingDelete(true);
+		t20 = /* @__PURE__ */ jsx(Trash2, {});
+		$[45] = t19;
+		$[46] = t20;
 	} else {
-		t23 = $[61];
-		t24 = $[62];
+		t19 = $[45];
+		t20 = $[46];
 	}
-	let t25;
-	if ($[63] !== busy) {
-		t25 = /* @__PURE__ */ jsxs(Button, {
+	let t21;
+	if ($[47] !== busy) {
+		t21 = /* @__PURE__ */ jsxs(Button, {
 			variant: "ghost",
 			className: "text-destructive",
 			disabled: busy,
-			onClick: t23,
-			children: [t24, "Delete draft"]
+			onClick: t19,
+			children: [t20, "Delete draft"]
 		});
-		$[63] = busy;
-		$[64] = t25;
-	} else t25 = $[64];
-	let t26;
-	if ($[65] !== data || $[66] !== projectId || $[67] !== publish || $[68] !== ready || $[69] !== translating.length) {
-		t26 = translating.length > 0 && /* @__PURE__ */ jsx(Button, {
-			variant: "outline",
+		$[47] = busy;
+		$[48] = t21;
+	} else t21 = $[48];
+	const t22 = !ready;
+	let t23;
+	if ($[49] !== data || $[50] !== projectId || $[51] !== publish) {
+		t23 = () => data && publish.mutate(publishInput(projectId, data));
+		$[49] = data;
+		$[50] = projectId;
+		$[51] = publish;
+		$[52] = t23;
+	} else t23 = $[52];
+	let t24;
+	if ($[53] !== publish.isPending) {
+		t24 = publish.isPending ? /* @__PURE__ */ jsx(Spinner, {}) : null;
+		$[53] = publish.isPending;
+		$[54] = t24;
+	} else t24 = $[54];
+	let t25;
+	if ($[55] !== t22 || $[56] !== t23 || $[57] !== t24) {
+		t25 = /* @__PURE__ */ jsxs(Button, {
 			className: "ml-auto",
-			disabled: !ready,
-			onClick: () => data && publish.mutate(publishInput(projectId, data, true)),
-			children: "Publish anyway"
-		});
-		$[65] = data;
-		$[66] = projectId;
-		$[67] = publish;
-		$[68] = ready;
-		$[69] = translating.length;
-		$[70] = t26;
-	} else t26 = $[70];
-	const t27 = translating.length === 0 && "ml-auto";
-	let t28;
-	if ($[71] !== t27) {
-		t28 = cn(t27);
-		$[71] = t27;
-		$[72] = t28;
-	} else t28 = $[72];
-	const t29 = !ready || !!waiting;
-	let t30;
-	if ($[73] !== publish.isPending || $[74] !== waiting) {
-		t30 = publish.isPending || waiting ? /* @__PURE__ */ jsx(Spinner, {}) : null;
-		$[73] = publish.isPending;
-		$[74] = waiting;
-		$[75] = t30;
-	} else t30 = $[75];
-	let t31;
-	if ($[76] !== publishWhole || $[77] !== t28 || $[78] !== t29 || $[79] !== t30) {
-		t31 = /* @__PURE__ */ jsxs(Button, {
-			className: t28,
-			disabled: t29,
-			onClick: publishWhole,
-			children: [t30, "Publish"]
+			disabled: t22,
+			onClick: t23,
+			children: [t24, "Publish"]
 		});
-		$[76] = publishWhole;
-		$[77] = t28;
-		$[78] = t29;
-		$[79] = t30;
-		$[80] = t31;
-	} else t31 = $[80];
-	let t32;
-	if ($[81] !== t25 || $[82] !== t26 || $[83] !== t31) {
-		t32 = /* @__PURE__ */ jsx("div", {
+		$[55] = t22;
+		$[56] = t23;
+		$[57] = t24;
+		$[58] = t25;
+	} else t25 = $[58];
+	let t26;
+	if ($[59] !== t21 || $[60] !== t25) {
+		t26 = /* @__PURE__ */ jsx("div", {
 			className: "flex flex-col gap-3 border-t p-4",
 			children: /* @__PURE__ */ jsxs("div", {
 				className: "flex items-center gap-2",
-				children: [
-					t25,
-					t26,
-					t31
-				]
+				children: [t21, t25]
 			})
 		});
-		$[81] = t25;
-		$[82] = t26;
-		$[83] = t31;
-		$[84] = t32;
-	} else t32 = $[84];
-	let t33;
-	if ($[85] !== initialFocus || $[86] !== t21 || $[87] !== t22 || $[88] !== t32) {
-		t33 = /* @__PURE__ */ jsxs(SheetContent, {
+		$[59] = t21;
+		$[60] = t25;
+		$[61] = t26;
+	} else t26 = $[61];
+	let t27;
+	if ($[62] !== initialFocus || $[63] !== t17 || $[64] !== t18 || $[65] !== t26) {
+		t27 = /* @__PURE__ */ jsxs(SheetContent, {
 			ref: popup,
 			initialFocus,
 			side: "right",
 			className: "w-full gap-0 data-[side=right]:sm:max-w-lg",
 			children: [
-				t21,
-				t22,
-				t32
+				t17,
+				t18,
+				t26
 			]
 		});
-		$[85] = initialFocus;
-		$[86] = t21;
-		$[87] = t22;
-		$[88] = t32;
-		$[89] = t33;
-	} else t33 = $[89];
-	let t34;
-	if ($[90] !== onOpenChange || $[91] !== open || $[92] !== t33) {
-		t34 = /* @__PURE__ */ jsx(Sheet, {
+		$[62] = initialFocus;
+		$[63] = t17;
+		$[64] = t18;
+		$[65] = t26;
+		$[66] = t27;
+	} else t27 = $[66];
+	let t28;
+	if ($[67] !== onOpenChange || $[68] !== open || $[69] !== t27) {
+		t28 = /* @__PURE__ */ jsx(Sheet, {
 			open,
 			onOpenChange,
-			children: t33
+			children: t27
 		});
-		$[90] = onOpenChange;
-		$[91] = open;
-		$[92] = t33;
-		$[93] = t34;
-	} else t34 = $[93];
-	let t35;
-	if ($[94] !== confirmingDelete || $[95] !== leave || $[96] !== projectId) {
-		t35 = /* @__PURE__ */ jsx(DeleteDraftDialog, {
+		$[67] = onOpenChange;
+		$[68] = open;
+		$[69] = t27;
+		$[70] = t28;
+	} else t28 = $[70];
+	let t29;
+	if ($[71] !== confirmingDelete || $[72] !== leave || $[73] !== projectId) {
+		t29 = /* @__PURE__ */ jsx(DeleteDraftDialog, {
 			projectId,
 			open: confirmingDelete,
 			onOpenChange: setConfirmingDelete,
 			onDeleted: leave
 		});
-		$[94] = confirmingDelete;
-		$[95] = leave;
-		$[96] = projectId;
-		$[97] = t35;
-	} else t35 = $[97];
-	let t36;
-	if ($[98] !== t34 || $[99] !== t35) {
-		t36 = /* @__PURE__ */ jsxs(Fragment, { children: [t34, t35] });
-		$[98] = t34;
-		$[99] = t35;
-		$[100] = t36;
-	} else t36 = $[100];
-	return t36;
+		$[71] = confirmingDelete;
+		$[72] = leave;
+		$[73] = projectId;
+		$[74] = t29;
+	} else t29 = $[74];
+	let t30;
+	if ($[75] !== t28 || $[76] !== t29) {
+		t30 = /* @__PURE__ */ jsxs(Fragment, { children: [t28, t29] });
+		$[75] = t28;
+		$[76] = t29;
+		$[77] = t30;
+	} else t30 = $[77];
+	return t30;
 }
 /** Asks before deleting the draft the studio is in, then deletes it. */
 function DeleteDraftDialog(t0) {
@@ -1587,7 +1684,7 @@ function DeleteDraftDialog(t0) {
 				onOpenChange(false);
 				onDeleted(ended);
 			},
-			onError: _temp11
+			onError: _temp14
 		};
 		$[1] = onDeleted;
 		$[2] = onOpenChange;
@@ -1666,7 +1763,7 @@ function DeleteDraftDialog(t0) {
 	} else t10 = $[25];
 	return t10;
 }
-function _temp11(error) {
+function _temp14(error) {
 	return toast.error(error.message || "Could not delete the draft");
 }