@camox/api-contract 0.72.0 → 0.75.0

package/dist/{patrol.d.ts → specialists.d.ts}

@@ -1,13 +1,13 @@
-//#region src/patrol.d.ts
+//#region src/specialists.d.ts
 /**
- * The Patrol (ADR 0024): every patroller of a project, the work each did as reports, as the
+ * The Specialists (ADR 0024): every specialist of a project, the work each did as reports, as the
  * studio shows it at the top of the Agent Sidebar. The API reads it for one scope, a draft or
  * Live, and names each report's subject, so the studio needs no entity loaded to show it.
  */
-/** Every patroller's stable id. The registry in the API lists them all. */
-export declare const PATROLLER_IDS: readonly ["translator", "seoManager", "linkChecker"];
-/** A patroller's stable id. */
-export type PatrollerId = (typeof PATROLLER_IDS)[number];
+/** Every specialist's stable id. The registry in the API lists them all. */
+export declare const SPECIALIST_IDS: readonly ["translator", "seoManager", "linkChecker"];
+/** A specialist's stable id. */
+export type SpecialistId = (typeof SPECIALIST_IDS)[number];
 /**
  * What a structured report says was done. Each has a failed form, by the report's outcome. A
  * merge with Live's localization reads as a translation.
@@ -15,7 +15,7 @@ export type PatrollerId = (typeof PATROLLER_IDS)[number];
 export type ReportVerb = "translated" | "mergedWithLive" | "wroteSeo" | "describedFile" | "publishedCatchUp" | "removedLink" | "fixedLink";
 /** What a report can be about. */
 export type ReportEntity = "page" | "block" | "sharedContent" | "repeatableItem" | "record" | "file";
-/** A report's subject, named as the Canvas names it, read when the Patrol is. */
+/** A report's subject, named as the Canvas names it, read when the Specialists is. */
 export type ReportSubject = {
   entity: ReportEntity;
   id: string;
@@ -44,10 +44,10 @@ export declare const reportLocale: (summary: {
 /** Whether the work was done, or failed. A failure shows in its report only. */
 export type ReportOutcome = "done" | "failed";
 /**
- * Where a patroller is on duty, in drafts or on Live: it works as you go, checks when you
- * publish, or is off there (ADR 0034).
+ * Where a specialist is on duty, in drafts or on Live: it works as you go, or is off there (ADR
+ * 0034). None checks when you publish: publishing never waits on Camox (ADR 0043).
  */
-export type PatrollerDuty = "asYouGo" | "atPublish" | "off";
+export type SpecialistDuty = "asYouGo" | "off";
 /**
  * Where the studio shows a report's subject: a page on the Canvas, with the block and repeatable
  * item to select on it, a collection record, or a file.
@@ -66,7 +66,7 @@ export type ReportPlace = {
   fileId: string;
 };
 export type ReportView = {
-  /** Unique among the patroller's reports in the view. */
+  /** Unique among the specialist's reports in the view. */
   id: string;
   summary: ReportSummary;
   outcome: ReportOutcome;
@@ -84,6 +84,7 @@ export type ReportView = {
  * What a finding is about (ADR 0034), by its kind:
  * - `missingPage`: a link to a page that doesn't exist, the page's name as it was last known.
  * - `brokenLink`: an external link that failed, and how.
+ * - `missingMetaTitle`: a page whose meta title is written by hand, and empty.
  * A link's `url` is as content stores it: a page link's is `camox:page:<id>`, and `text` is what
  * the link reads.
  */
@@ -97,6 +98,8 @@ export type FindingAbout = {
   url: string;
   text: string;
   failure: LinkFailure;
+} | {
+  kind: "missingMetaTitle";
 };
 /** How a link failed: an HTTP error status, no connection (DNS included), or no answer in time. */
 export type LinkFailure = {
@@ -129,14 +132,17 @@ export type FindingView = {
   fieldKind: string | null;
   /** When it was first raised, in milliseconds since the epoch. */
   raisedAt: number;
-  /** Whether it's Live's. Live's are fixed in the Patrol Draft. */
+  /**
+   * Whether it's Live's. A draft shows Live's findings on the content it still holds, and fixes
+   * them there.
+   */
   live: boolean;
   /** Where its content shows, as for reports. */
   place: ReportPlace | null;
 };
 /**
  * A finding as one sentence, as the studio, emails, Briefs and refusals word it: "“About us” in
- * Footer links to About, a page that no longer exists".
+ * Footer links to About, a page that no longer exists", or "Pricing has no meta title".
  */
 export declare function findingSentence(finding: Pick<FindingView, "about" | "subject">): string;
 /**
@@ -159,27 +165,27 @@ export type DismissalView = {
 export type ApplyResult = {
   applied: boolean;
 };
-/** How many subjects a patroller is on in one locale, a locale's tag. */
+/** How many subjects a specialist is on in one locale, a locale's tag. */
 export type LocaleCount = {
   locale: string;
   count: number;
 };
 /**
- * What a patroller is on: the one subject when there's one, and how many subjects in each
+ * What a specialist is on: the one subject when there's one, and how many subjects in each
  * locale, the most first. A subject in several locales counts in each.
  */
-export type PatrollerWorking = {
+export type SpecialistWorking = {
   subject: ReportSubject | null;
   locales: LocaleCount[];
 };
-export type PatrollerView = {
-  id: PatrollerId;
+export type SpecialistView = {
+  id: SpecialistId;
   /** Its role, as editors know it: "Translator", "SEO manager". */
   role: string;
   /** Its duty in the scope viewed. Off there, it still has its reports. */
-  duty: PatrollerDuty;
+  duty: SpecialistDuty;
   /** Null when it isn't working in the scope. */
-  working: PatrollerWorking | null;
+  working: SpecialistWorking | null;
   /**
    * When it last finished a check in the scope, whether it found anything or not, or null when it
    * never checked there.
@@ -202,17 +208,15 @@ export type PatrollerView = {
   /** The project's dismissals of its findings, the newest first. */
   dismissals: DismissalView[];
 };
+/** Every specialist of the registry, the latest activity first; never-worked ones last. */
+export type SpecialistsView = {
+  specialists: SpecialistView[];
+};
 /**
- * Every patroller of the registry, the latest activity first; never-worked ones last. With the
- * project's open Patrol Draft, where Live's findings are fixed, if any.
+ * URL search param the studio opens a specialist's findings from, by its id, as the email about
+ * Live's new findings links to them. The studio drops it once opened.
  */
-export type PatrolView = {
-  patrollers: PatrollerView[];
-  patrolDraft: {
-    id: string;
-    name: string | null;
-  } | null;
-};
-/** How many reports the Patrol shows of each patroller; the rest are counted. */
+export declare const SPECIALIST_SEARCH_PARAM = "camox-specialist";
+/** How many reports the Specialists shows of each specialist; the rest are counted. */
 export declare const REPORTS_SHOWN = 5;
 //#endregion

package/dist/{patrol.js → specialists.js}

@@ -1,19 +1,19 @@
-//#region src/patrol.ts
+//#region src/specialists.ts
 /**
-* The Patrol (ADR 0024): every patroller of a project, the work each did as reports, as the
+* The Specialists (ADR 0024): every specialist of a project, the work each did as reports, as the
 * studio shows it at the top of the Agent Sidebar. The API reads it for one scope, a draft or
 * Live, and names each report's subject, so the studio needs no entity loaded to show it.
 */
-/** Every patroller's stable id. The registry in the API lists them all. */
-const PATROLLER_IDS = [
+/** Every specialist's stable id. The registry in the API lists them all. */
+const SPECIALIST_IDS = [
 	"translator",
 	"seoManager",
 	"linkChecker"
 ];
 /** The locale a summary's work was in: a structured summary's, if any. A text summary has none. */
 const reportLocale = (summary) => summary.kind === "structured" ? summary.locale : null;
-/** What a finding is about, after where it is: "links to About, a page that no longer exists". */
-function aboutSentence(about) {
+/** What a link finding is about, after where it is: "links to About, a page that no longer exists". */
+function linkSentence(about) {
 	switch (about.kind) {
 		case "missingPage": return about.page === null ? "links to a page that no longer exists" : `links to ${about.page}, a page that no longer exists`;
 		case "brokenLink": {
@@ -25,20 +25,29 @@ function aboutSentence(about) {
 }
 /**
 * A finding as one sentence, as the studio, emails, Briefs and refusals word it: "“About us” in
-* Footer links to About, a page that no longer exists".
+* Footer links to About, a page that no longer exists", or "Pricing has no meta title".
 */
 function findingSentence(finding) {
-	return `“${finding.about.text}” in ${finding.subject.name} ${aboutSentence(finding.about)}`;
+	const { about, subject } = finding;
+	if (about.kind === "missingMetaTitle") return `${subject.name} has no meta title`;
+	return `“${about.text}” in ${subject.name} ${linkSentence(about)}`;
 }
 /**
 * A finding as one sentence where its content is already shown, as the studio's sidebar shows it
 * by its field: "“About us” links to About, a page that no longer exists".
 */
 function findingSentenceInPlace(finding) {
-	return `“${finding.about.text}” ${aboutSentence(finding.about)}`;
+	const { about } = finding;
+	if (about.kind === "missingMetaTitle") return "The page has no meta title";
+	return `“${about.text}” ${linkSentence(about)}`;
 }
-/** How many reports the Patrol shows of each patroller; the rest are counted. */
+/**
+* URL search param the studio opens a specialist's findings from, by its id, as the email about
+* Live's new findings links to them. The studio drops it once opened.
+*/
+const SPECIALIST_SEARCH_PARAM = "camox-specialist";
+/** How many reports the Specialists shows of each specialist; the rest are counted. */
 const REPORTS_SHOWN = 5;
 
 //#endregion
-export { PATROLLER_IDS, REPORTS_SHOWN, findingSentence, findingSentenceInPlace, reportLocale };
+export { REPORTS_SHOWN, SPECIALIST_IDS, SPECIALIST_SEARCH_PARAM, findingSentence, findingSentenceInPlace, reportLocale };

package/dist/threads.d.ts

@@ -1,7 +1,7 @@
 import { DraftOutcome } from "./drafts.js";
 import { z } from "zod";
-import { AgentEvent } from "@earendil-works/pi-durable";
 import { Message } from "@earendil-works/pi-ai";
+import { AgentEvent } from "@earendil-works/pi-durable";
 //#region src/threads.d.ts
 /**
  * The kind of the transcript entry recording an editor's message, with its sender and the
@@ -35,8 +35,7 @@ export declare const RECONCILE_ENTRY = "camox.reconcile";
 /**
  * The kind of the transcript entry opening a draft's thread a Live thread handed a request off
  * to: the Brief, which the model reads ahead of the editor's messages, and editors see collapsed
- * after them, so the request reads on as they sent it. In a draft Camox opened, such as its Patrol Draft, Camox's own messages are Briefs
- * too, with their `operationId` as data, and start the agent's run, which acts for Camox.
+ * after them, so the request reads on as they sent it.
  */
 export declare const BRIEF_ENTRY = "camox.brief";
 /**
@@ -109,6 +108,30 @@ export declare const codeRequestDataSchema: z.ZodObject<{
   reconciling: z.ZodOptional<z.ZodBoolean>;
 }, z.core.$strip>;
 export type CodeRequestData = z.infer<typeof codeRequestDataSchema>;
+/**
+ * The draft thread's agent's tool asking the Connected Agents following its turn questions, which
+ * their waiting calls come back with as the agent works on (ADR 0042). The model calls it itself,
+ * never from a script.
+ */
+export declare const ASK_TOOL = "askConnectedAgent";
+/**
+ * The kind of the transcript entry recording questions the agent asked the Connected Agents
+ * following its turn, with a `questionsDataSchema` as data. Neither the model nor editors read
+ * it: questions are never a thread's messages, and never end its turn.
+ */
+export declare const QUESTIONS_ENTRY = "camox.questions";
+/**
+ * What a questions entry records: the questions, in the order the agent asked them, and the
+ * Connected Agents they were asked of, each by its member's id and client id.
+ */
+export declare const questionsDataSchema: z.ZodObject<{
+  questions: z.ZodArray<z.ZodString>;
+  to: z.ZodArray<z.ZodObject<{
+    memberId: z.ZodString;
+    clientId: z.ZodString;
+  }, z.core.$strip>>;
+}, z.core.$strip>;
+export type QuestionsData = z.infer<typeof questionsDataSchema>;
 /** Whether `name` is a tool handing work to an agent of its own (`HANDING_OFF_TOOLS`). */
 export declare const handsOff: (name: string) => boolean;
 /**
@@ -167,9 +190,17 @@ export declare const connectedAgentClientSchema: z.ZodObject<{
   verified: z.ZodBoolean;
 }, z.core.$strip>;
 export type ConnectedAgentClient = z.infer<typeof connectedAgentClientSchema>;
+/** An Attachment a Connected Agent's message carries: an image the agent sees with it. */
+export declare const messageAttachmentSchema: z.ZodObject<{
+  id: z.ZodString;
+  filename: z.ZodString;
+  mimeType: z.ZodString;
+  url: z.ZodString;
+}, z.core.$strip>;
+export type MessageAttachmentData = z.infer<typeof messageAttachmentSchema>;
 /**
- * What a Connected Agent's message entry records: the member it acts as, its client, and the
- * operation id it was sent with.
+ * What a Connected Agent's message entry records: the member it acts as, its client, the
+ * operation id it was sent with, and whether it answers the agent's questions (ADR 0042).
  */
 export declare const connectedAgentMessageDataSchema: z.ZodObject<{
   member: z.ZodObject<{
@@ -182,6 +213,13 @@ export declare const connectedAgentMessageDataSchema: z.ZodObject<{
     verified: z.ZodBoolean;
   }, z.core.$strip>;
   operationId: z.ZodOptional<z.ZodString>;
+  answer: z.ZodOptional<z.ZodBoolean>;
+  attachments: z.ZodOptional<z.ZodArray<z.ZodObject<{
+    id: z.ZodString;
+    filename: z.ZodString;
+    mimeType: z.ZodString;
+    url: z.ZodString;
+  }, z.core.$strip>>>;
 }, z.core.$strip>;
 export type ConnectedAgentMessageData = z.infer<typeof connectedAgentMessageDataSchema>;
 /** What a push's entry records: who pushed, their commits' subjects, oldest first, and the build. */
@@ -315,7 +353,9 @@ export type ThreadMessage = {
 } |
 /**
  * A member's Connected Agent's message: the member's own AI, in its client, writing as itself.
- * Never shown as the member's own words.
+ * Never shown as the member's own words. An answer to the agent's questions shows as the
+ * context the draft was built on, collapsed, as a Brief does, rather than as a request
+ * (ADR 0042).
  */
 {
   id: string;
@@ -323,6 +363,9 @@ export type ThreadMessage = {
   member: ThreadSender;
   client: ConnectedAgentClient;
   text: string;
+  answer: boolean;
+  /** The images it carries for the agent to see, if any (ADR 0044). */
+  attachments?: MessageAttachmentData[];
 } |
 /** Camox's own message, starting a turn it began itself: why it did. */
 {
@@ -474,6 +517,8 @@ type ThreadStep = {
   member: ThreadSender;
   client: ConnectedAgentClient;
   text: string;
+  answer: boolean;
+  attachments?: MessageAttachmentData[];
 } |
 /** Camox starting a turn of its own, saying why. */
 {

package/dist/threads.js

@@ -33,8 +33,7 @@ const RECONCILE_ENTRY = "camox.reconcile";
 /**
 * The kind of the transcript entry opening a draft's thread a Live thread handed a request off
 * to: the Brief, which the model reads ahead of the editor's messages, and editors see collapsed
-* after them, so the request reads on as they sent it. In a draft Camox opened, such as its Patrol Draft, Camox's own messages are Briefs
-* too, with their `operationId` as data, and start the agent's run, which acts for Camox.
+* after them, so the request reads on as they sent it.
 */
 const BRIEF_ENTRY = "camox.brief";
 /**
@@ -101,6 +100,29 @@ const CODE_REQUEST_ENTRY = "camox.code-request";
 /** What a Code Agent's request records: as an Artifact Agent's, and whether Camox reconciles. */
 const codeRequestDataSchema = artifactRequestDataSchema.extend({ reconciling: z.boolean().optional() });
 /**
+* The draft thread's agent's tool asking the Connected Agents following its turn questions, which
+* their waiting calls come back with as the agent works on (ADR 0042). The model calls it itself,
+* never from a script.
+*/
+const ASK_TOOL = "askConnectedAgent";
+/**
+* The kind of the transcript entry recording questions the agent asked the Connected Agents
+* following its turn, with a `questionsDataSchema` as data. Neither the model nor editors read
+* it: questions are never a thread's messages, and never end its turn.
+*/
+const QUESTIONS_ENTRY = "camox.questions";
+/**
+* What a questions entry records: the questions, in the order the agent asked them, and the
+* Connected Agents they were asked of, each by its member's id and client id.
+*/
+const questionsDataSchema = z.object({
+	questions: z.array(z.string()).min(1),
+	to: z.array(z.object({
+		memberId: z.string(),
+		clientId: z.string()
+	}))
+});
+/**
 * The agent's tools handing work to an agent of its own, in a conversation beside the thread's,
 * whose answer they return: editors see what that agent does as the agent's.
 */
@@ -160,14 +182,24 @@ const connectedAgentClientSchema = z.object({
 	name: z.string(),
 	verified: z.boolean()
 });
+/** An Attachment a Connected Agent's message carries: an image the agent sees with it. */
+const messageAttachmentSchema = z.object({
+	id: z.string(),
+	filename: z.string(),
+	mimeType: z.string(),
+	url: z.string()
+});
 /**
-* What a Connected Agent's message entry records: the member it acts as, its client, and the
-* operation id it was sent with.
+* What a Connected Agent's message entry records: the member it acts as, its client, the
+* operation id it was sent with, and whether it answers the agent's questions (ADR 0042).
 */
 const connectedAgentMessageDataSchema = z.object({
 	member: threadSenderSchema,
 	client: connectedAgentClientSchema,
-	operationId: z.string().optional()
+	operationId: z.string().optional(),
+	answer: z.boolean().optional(),
+	/** The images the message carries for the agent to see, as studios list them (ADR 0044). */
+	attachments: z.array(messageAttachmentSchema).optional()
 });
 /** What a push's entry records: who pushed, their commits' subjects, oldest first, and the build. */
 const pushDataSchema = z.object({
@@ -395,7 +427,9 @@ function threadStepOf(entry) {
 					name: data.client.name,
 					verified: data.client.verified
 				},
-				text: editorTextOf(message.content)
+				text: editorTextOf(message.content),
+				answer: data.answer ?? false,
+				...data.attachments?.length && { attachments: data.attachments }
 			};
 		}
 		if (entry.kind === "camox.brief") return {
@@ -500,7 +534,9 @@ function threadViewOf({ steps, running, toolCalls = [], callingTools = false, ch
 					from: "connectedAgent",
 					member: step.member,
 					client: step.client,
-					text: step.text
+					text: step.text,
+					answer: step.answer,
+					...step.attachments && { attachments: step.attachments }
 				});
 				waiting = step.id;
 				break;
@@ -736,4 +772,4 @@ function reduceThreadFrame(state, frame) {
 }
 
 //#endregion
-export { ARTIFACT_REQUEST_ENTRY, ARTIFACT_TOOL, BRIEF_ENTRY, BUILD_STATUS_ENTRY, BUILD_TOOL, CHANGING_TOOLS, CODE_REQUEST_ENTRY, CODE_TOOL, CONNECTED_AGENT_MESSAGE_ENTRY, DRAFT_CARD_ENTRY, DRAFT_ENDED_ENTRY, EDITOR_MESSAGE_ENTRY, EMPTY_THREAD_STATE, EMPTY_THREAD_VIEW, LOOK_TOOL, PUSH_ENTRY, RECONCILE_ENTRY, RUN_STOPPED_ENTRY, START_DRAFT_TOOL, THREAD_ENDED_CLOSE_CODE, agentActivitySchema, artifactRequestDataSchema, buildStatusDataSchema, buildStatusSchema, codeRequestDataSchema, connectedAgentClientSchema, connectedAgentMessageDataSchema, editorMessageDataSchema, editorSelectionSchema, editorTextOf, editorWhereSchema, handOffDetailsSchema, handsOff, isSender, pushDataSchema, pushOutcomeSchema, reconcileDataSchema, reduceThreadEvents, reduceThreadFrame, reduceThreadState, startedDraftCardSchema, startedDraftSchema, textOf, threadClientMessageSchema, threadSenderSchema, threadViewOf, touchKindSchema, touchSchema, touchedBatchSchema };
+export { ARTIFACT_REQUEST_ENTRY, ARTIFACT_TOOL, ASK_TOOL, BRIEF_ENTRY, BUILD_STATUS_ENTRY, BUILD_TOOL, CHANGING_TOOLS, CODE_REQUEST_ENTRY, CODE_TOOL, CONNECTED_AGENT_MESSAGE_ENTRY, DRAFT_CARD_ENTRY, DRAFT_ENDED_ENTRY, EDITOR_MESSAGE_ENTRY, EMPTY_THREAD_STATE, EMPTY_THREAD_VIEW, LOOK_TOOL, PUSH_ENTRY, QUESTIONS_ENTRY, RECONCILE_ENTRY, RUN_STOPPED_ENTRY, START_DRAFT_TOOL, THREAD_ENDED_CLOSE_CODE, agentActivitySchema, artifactRequestDataSchema, buildStatusDataSchema, buildStatusSchema, codeRequestDataSchema, connectedAgentClientSchema, connectedAgentMessageDataSchema, editorMessageDataSchema, editorSelectionSchema, editorTextOf, editorWhereSchema, handOffDetailsSchema, handsOff, isSender, messageAttachmentSchema, pushDataSchema, pushOutcomeSchema, questionsDataSchema, reconcileDataSchema, reduceThreadEvents, reduceThreadFrame, reduceThreadState, startedDraftCardSchema, startedDraftSchema, textOf, threadClientMessageSchema, threadSenderSchema, threadViewOf, touchKindSchema, touchSchema, touchedBatchSchema };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camox/api-contract",
-  "version": "0.72.0",
+  "version": "0.75.0",
   "license": "MIT",
   "files": [
     "src",
@@ -44,9 +44,9 @@
       "types": "./dist/locales.d.ts",
       "default": "./dist/locales.js"
     },
-    "./patrol": {
-      "types": "./dist/patrol.d.ts",
-      "default": "./dist/patrol.js"
+    "./specialists": {
+      "types": "./dist/specialists.d.ts",
+      "default": "./dist/specialists.js"
     },
     "./router": {
       "types": "./dist/router.d.ts",
@@ -55,6 +55,10 @@
     "./layout-routes": {
       "types": "./dist/layout-routes.d.ts",
       "default": "./dist/layout-routes.js"
+    },
+    "./claim-links": {
+      "types": "./dist/claim-links.d.ts",
+      "default": "./dist/claim-links.js"
     }
   },
   "dependencies": {

package/src/claim-links.ts

@@ -0,0 +1,23 @@
+import { projectWebsiteUrl } from "./projects";
+
+/** URL search param carrying a Claim Link's token on the website it links to. */
+export const CLAIM_LINK_SEARCH_PARAM = "camox-claim";
+
+/**
+ * Where a Claim Link stands: `open` until it's claimed, revoked by an operator, or blocked, when a
+ * claim was refused because someone who isn't an operator joined its organization (ADR 0041).
+ */
+export type ClaimLinkState = "open" | "claimed" | "revoked" | "blocked";
+
+/**
+ * A Claim Link: the project's address, its Custom Domain if it has one, else its `camox.site`
+ * subdomain, with the link's token.
+ */
+export function claimLinkUrl(
+  project: { slug: string; customDomain: string | null },
+  token: string,
+) {
+  const url = new URL("/", projectWebsiteUrl(project));
+  url.searchParams.set(CLAIM_LINK_SEARCH_PARAM, token);
+  return url.toString();
+}

package/src/drafts.ts

@@ -164,12 +164,6 @@ export type EndedDraft = {
  */
 export const DRAFT_ENDED_ERROR_CODE = "DRAFT_ENDED";
 
-/**
- * oRPC error code for a publish whose token no longer matches Live: Live changed since the
- * preview, so nothing was applied and the caller reviews the new preview.
- */
-export const CONFLICTS_CHANGED_ERROR = "CONFLICTS_CHANGED";
-
 /**
  * oRPC error code for a publish of a draft with code whose revision isn't the one the review
  * showed: the agent left a newer one, or the draft is already publishing. The caller reviews
@@ -179,15 +173,16 @@ export const REVISION_CHANGED_ERROR = "REVISION_CHANGED";
 
 /**
  * Why a publish with code went back to the review instead of landing: the agent left a revision
- * after the one the review showed, Live's conflicts changed, the draft got a blocker, Live's code
- * moved and the draft's merge of it clashed (the "doesn't merge with Live's" blocker, which
- * Camox's turn fixes), Live's code moved and the draft took too long to merge it, or the publish
- * failed on Camox's side.
+ * after the one the review showed, the draft got a blocker, Live's code moved and the draft's
+ * merge of it clashed (the "doesn't merge with Live's" blocker, which Camox's turn fixes), Live's
+ * code moved and the draft took too long to merge it, or the publish failed on Camox's side.
+ * `conflictsChanged` is only stored by publishes before ADR 0043, which aborted when Live changed.
  */
 export const PUBLISH_ABORT_REASONS = [
   "revisionChanged",
   "conflictsChanged",
   "blocked",
+  "codeDoesNotBuild",
   "codeDoesNotMerge",
   "liveCodeMoved",
   "failed",
@@ -198,36 +193,13 @@ export type PublishAbortReason = (typeof PUBLISH_ABORT_REASONS)[number];
 /** oRPC error code for a publish refused because the draft still has blockers. */
 export const PUBLISH_BLOCKED_ERROR = "PUBLISH_BLOCKED";
 
-/**
- * oRPC error code for a publish refused because the draft still has pending translations: every
- * locale goes live together, and there is no publishing anyway (ADR 0033).
- */
-export const TRANSLATIONS_PENDING_ERROR = "TRANSLATIONS_PENDING";
-
-/**
- * oRPC error code for a publish refused because the draft has open findings, each to fix or for a
- * member to dismiss: there is no publishing past them (ADR 0033).
- */
-export const FINDINGS_OPEN_ERROR = "FINDINGS_OPEN";
-
-/**
- * oRPC error code for a publish refused because a patroller's check at publish hasn't read the
- * draft's latest changes yet: publishing waits for it (ADR 0033).
- */
-export const CHECKS_RUNNING_ERROR = "CHECKS_RUNNING";
-
 /**
  * Why the publish tool published nothing of a draft with changes, and returned a fresh preview to
- * review again, by the error code of each: Live changed since the preview, the draft has blockers,
- * translations are still running, the draft has open findings, a check at publish is still
- * running, or the draft's code changed since the preview.
+ * review again, by the error code of each: the draft has blockers, or the draft's code changed
+ * since the preview.
  */
 export const NOT_PUBLISHED_CODES = {
-  conflictsChanged: CONFLICTS_CHANGED_ERROR,
   blocked: PUBLISH_BLOCKED_ERROR,
-  translating: TRANSLATIONS_PENDING_ERROR,
-  findings: FINDINGS_OPEN_ERROR,
-  checking: CHECKS_RUNNING_ERROR,
   revisionChanged: REVISION_CHANGED_ERROR,
 } as const;
 
@@ -339,7 +311,7 @@ const LOCALE_WRITES: Record<keyof Router["locales"], boolean> = {
 
 // Applying a finding's fix edits content as the member would; dismissing one is the project's,
 // never content (ADR 0034).
-const PATROL_WRITES: Record<keyof Router["patrol"], boolean> = {
+const SPECIALISTS_WRITES: Record<keyof Router["specialists"], boolean> = {
   get: false,
   applyFix: true,
   dismissFinding: false,
@@ -352,7 +324,9 @@ const CONTENT_WRITES: Partial<Record<keyof Router, Record<string, boolean>>> = {
   collectionDefinitions: COLLECTION_WRITES,
   files: FILE_WRITES,
   pages: PAGE_WRITES,
-  patrol: PATROL_WRITES,
+  specialists: SPECIALISTS_WRITES,
+  // The old name of `specialists`, for studios not yet upgraded (ADR 0019).
+  patrol: SPECIALISTS_WRITES,
   repeatableItems: ITEM_WRITES,
 };
 

package/src/query-keys.ts

@@ -59,7 +59,11 @@ export const queryKeys = {
     // The draft a viewer is in, scoped to it like content.
     get: ["camox", "drafts", "get"],
   },
-  // The Patrol of a scope: its reports and its patrollers' pending work.
+  // The Specialists of a scope: their reports and pending work.
+  specialists: {
+    get: ["camox", "specialists"],
+  },
+  // Where studios not yet upgraded to the specialists cache them (ADR 0019).
   patrol: {
     get: ["camox", "patrol"],
   },