@gentbajko/slopify 3.25.8 → 4.0.0

package/dist/slices/rebuild/runtime-subtitles.js

@@ -1,5 +1,6 @@
 import { readFileSync } from "node:fs";
 import { z } from "zod";
+import { internalError } from "../../kernel/errors.js";
 import { subtitleModelDir } from "../../kernel/paths.js";
 import { projectLanguage } from "../../kernel/ports/languages.js";
 import { SubtitleMismatch } from "../../kernel/ports/subtitles.js";
@@ -7,6 +8,7 @@ import { narrationAliasesOf } from "../admission/rules.js";
 import { captionFont, captionFontDeps } from "../fonts/coverage.js";
 import { resolveFont } from "../fonts/index.js";
 import { validateCues } from "../revisions/rules.js";
+import { donePiece, isReadyOutput, readyOutput } from "../revisions/view.js";
 import { discardPreparedAssets, writeAsset } from "../storage/assets.js";
 import { outputPath } from "../storage/layout.js";
 import { captionCues, serializeAss, serializeSrt, serializeVtt } from "../subtitles/captions.js";
@@ -47,6 +49,14 @@ const cuesSchema = z.object({
     })),
     omissions: wordsSchema.shape.omissions,
 });
+// The narration's word timing (`subtitles:timing`), read for a step that needs it. `purpose` says
+// what the step uses it for, in the sentence a missing timing fails with.
+export function readTimingWords(deps, context, view, purpose) {
+    const timing = readyOutput(view, { workKey: "subtitles:timing", role: "subtitle_words" });
+    if (timing === undefined)
+        throw new Error(`The narration's word timing, which ${purpose}, is missing. Use More → Render the video again in the Video section, then Try again.`);
+    return wordsSchema.parse(JSON.parse(readFileSync(outputPath(deps.paths, context.work.projectId, timing.output.path), "utf8")));
+}
 export async function executeSubtitleRecipe(deps, context, piece) {
     if (!context.maySubmit(piece.id))
         return "held";
@@ -58,7 +68,7 @@ export async function executeSubtitleRecipe(deps, context, piece) {
         return cues(deps, context, piece, snapshot);
     if (piece.key === "subtitles:files")
         return files(deps, context, piece, snapshot);
-    throw new Error("Slopify hit an internal error (unknown kind of caption step). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+    throw new Error(internalError("unknown kind of caption step"));
 }
 async function timing(deps, context, piece, snapshot) {
     const audio = await revisionAudio(deps, context, snapshot.view);
@@ -66,7 +76,7 @@ async function timing(deps, context, piece, snapshot) {
         throw new Error("Captions and the YouTube description need narration audio, but this project has none. Turn captions and the YouTube description off in Edit project, or turn narration on, then Try again.");
     const alignSubtitles = deps.alignSubtitles;
     if (alignSubtitles === undefined)
-        throw new Error("Slopify hit an internal error (the caption timing tool is missing from this build). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("the caption timing tool is missing from this build"));
     if (!context.maySubmit(piece.id))
         return "held";
     const words = [];
@@ -131,10 +141,7 @@ async function cues(deps, context, piece, snapshot) {
         value = { cues: [...manual.cues], omissions: [] };
     }
     else {
-        const output = view.outputs.find((one) => one.selected &&
-            one.available &&
-            one.state === "ready" &&
-            one.workKey === "subtitles:timing");
+        const output = readyOutput(view, { workKey: "subtitles:timing" });
         if (output === undefined)
             throw new Error("Caption timing is missing. Use More → Render the video again in the Video section, then Try again.");
         const timed = wordsSchema.parse(JSON.parse(readFileSync(outputPath(deps.paths, context.work.projectId, output.output.path), "utf8")));
@@ -150,8 +157,8 @@ async function files(deps, context, piece, snapshot) {
     const { view } = snapshot;
     const config = view.revision.config.subtitles;
     if (config === undefined || config.mode === "off")
-        throw new Error("Slopify hit an internal error (captions are off for this version of the project). Try again; if it happens again, use Download diagnostics in Settings and report it.");
-    const row = view.pieces.find((one) => one.key === "subtitles:cues" && one.selected && one.available && one.piece.state === "done");
+        throw new Error(internalError("captions are off for this version of the project"));
+    const row = donePiece(view, "subtitles:cues");
     if (row?.piece.payload === null || row?.piece.payload === undefined)
         throw new Error("The captions haven't been prepared yet. Use More → Render the video again in the Video section, then Try again.");
     const value = cuesSchema.parse(JSON.parse(row.piece.payload));
@@ -219,9 +226,7 @@ async function files(deps, context, piece, snapshot) {
         ])
             prepared.push(preparedText(deps, context, piece, role, filename, text));
         prepared.push(preparedResult(deps, context, piece, "subtitle_font", writeAsset(deps, context.work.projectId, `selected${font.extension}`, readFileSync(font.path))));
-        const media = view.outputs.find((one) => one.selected &&
-            one.available &&
-            one.state === "ready" &&
+        const media = view.outputs.find((one) => isReadyOutput(one) &&
             (one.output.role === "audio_export" ||
                 (one.output.role === "video" && config.mode === "files")));
         if (media !== undefined) {
@@ -295,9 +300,6 @@ function mismatchChunks(snapshot, segment) {
         return [];
     }
 }
-export function mismatchMessage(snapshot, segment, mismatch) {
-    return describeMismatch(mismatchChunks(snapshot, segment), segment, mismatch);
-}
 // The chunk that holds the words the audio stopped matching, by index; -1 when none does.
 export function mismatchChunk(chunks, mismatch) {
     const probe = comparable(mismatch.expected).split(" ").slice(0, 6).join(" ");

package/dist/slices/rebuild/runtime-voices.js

@@ -1,10 +1,11 @@
-import { mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { z } from "zod";
+import { internalError } from "../../kernel/errors.js";
 import { masterFile, masterReport } from "../loudness/loudnorm.js";
 import { masterGoal } from "../loudness/model.js";
 import { allocateAsset, discardPreparedAssets, sealAsset } from "../storage/assets.js";
-import { outputPath, projectDir } from "../storage/layout.js";
+import { projectDir } from "../storage/layout.js";
 import { audioExportArgs } from "../video/audio-export-args.js";
 import { runFfmpeg } from "../video/ffmpeg.js";
 import { audioChapters, audioFileArgs, ffmetadata } from "../voices/audio-files.js";
@@ -12,18 +13,17 @@ import { turnStarts } from "../voices/timing.js";
 import { exportSnapshot, revisionAudio } from "./runtime-export-inputs.js";
 import { lineLevelledAudio } from "./runtime-lines.js";
 import { preparedResult, publishResult } from "./runtime-publication.js";
-import { wordsSchema } from "./runtime-subtitles.js";
+import { readTimingWords } from "./runtime-subtitles.js";
+import { pieceAlreadyDone } from "./work-records.js";
 // The listening files of a multi-voice run: the MP3 and the M4B of the narration timeline,
 // with a chapter at each section of the script where its first turn is heard.
 export async function executeVoicesRecipe(deps, context, piece) {
-    if (deps.db
-        .prepare("SELECT state FROM revision_work_pieces WHERE id=? AND work_id=?")
-        .get(piece.id, context.work.workId)?.state === "done")
+    if (pieceAlreadyDone(deps.db, context.work.workId, piece.id))
         return "done";
     if (!context.maySubmit(piece.id))
         return "held";
     if (piece.key !== "voices:files" || piece.input.kind !== "local")
-        throw new Error("Slopify hit an internal error (unknown kind of listening-file step). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("unknown kind of listening-file step"));
     context.signal.throwIfAborted();
     const { view } = exportSnapshot(deps, context, piece);
     const sections = z
@@ -32,14 +32,7 @@ export async function executeVoicesRecipe(deps, context, piece) {
         .rest(z.unknown())
         .parse(piece.input.values)[2]
         .map(([title, firstTurn]) => ({ title, firstTurn }));
-    const timing = view.outputs.find((row) => row.selected &&
-        row.available &&
-        row.state === "ready" &&
-        row.workKey === "subtitles:timing" &&
-        row.output.role === "subtitle_words");
-    if (timing === undefined)
-        throw new Error("The narration's word timing, which the chapter markers come from, is missing. Use More → Render the video again in the Video section, then Try again.");
-    const { words } = wordsSchema.parse(JSON.parse(readFileSync(outputPath(deps.paths, context.work.projectId, timing.output.path), "utf8")));
+    const { words } = readTimingWords(deps, context, view, "the chapter markers come from");
     const audio = await revisionAudio(deps, context, view, { levelled: true });
     if (audio.length === 0)
         throw new Error("The MP3 and M4B files need narration audio, but this project has none. Turn narration on in Edit project, or turn Audio files off under Speakers (Play → Narration, or Edit project → Providers), then Try again.");

package/dist/slices/rebuild/runtime-youtube.js

@@ -1,42 +1,33 @@
-import { readFileSync } from "node:fs";
+import { internalError } from "../../kernel/errors.js";
 import { withLanguage } from "../../kernel/ports/languages.js";
-import { outputPath } from "../storage/layout.js";
 import { assembleDescription, checkDescriptionAnswer, descriptionMessages, tagsText, } from "../youtube/answer.js";
 import { defaultDescriptionPrompt } from "../youtube/model.js";
 import { keepsShape, titleShape } from "../youtube/titles.js";
 import { transcriptPassages, transcriptText } from "../youtube/transcript.js";
 import { exportSnapshot, revisionAudio } from "./runtime-export-inputs.js";
-import { savedCatalogue } from "./runtime-plan.js";
+import { workLlm } from "./runtime-plan.js";
 import { preparedTexts, publishResult } from "./runtime-publication.js";
-import { wordsSchema } from "./runtime-subtitles.js";
+import { readTimingWords } from "./runtime-subtitles.js";
+import { pieceAlreadyDone } from "./work-records.js";
 // Writes the YouTube description, tags, pinned comment and other titles from the saved word timing: the transcript's
 // passages carry their start in the final video, so the chapters the model picks are real
 // times. An answer that breaks YouTube's chapter or tag rules is a failed attempt, which the
 // provider wrapper asks again while attempts remain.
 export async function executeYoutubeRecipe(deps, context, providers, piece) {
-    if (deps.db
-        .prepare("SELECT state FROM revision_work_pieces WHERE id=? AND work_id=?")
-        .get(piece.id, context.work.workId)?.state === "done")
+    if (pieceAlreadyDone(deps.db, context.work.workId, piece.id))
         return "done";
     if (!context.maySubmit(piece.id))
         return "held";
     context.signal.throwIfAborted();
     const input = piece.input;
     if (input.kind !== "local" || input.operation !== "youtube-description-v1")
-        throw new Error("Slopify hit an internal error (the YouTube description step was set up wrongly). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("the YouTube description step was set up wrongly"));
     const { view } = exportSnapshot(deps, context, piece);
     const config = view.revision.config;
     const llm = config.llm;
     if (llm === undefined || llm.provider.trim() === "" || llm.model.trim() === "")
         throw new Error("The YouTube description needs an AI text model, and none is chosen. Choose one in Edit project → Providers, then Try again.");
-    const timing = view.outputs.find((row) => row.selected &&
-        row.available &&
-        row.state === "ready" &&
-        row.workKey === "subtitles:timing" &&
-        row.output.role === "subtitle_words");
-    if (timing === undefined)
-        throw new Error("The narration's word timing, which the YouTube description's chapters come from, is missing. Use More → Render the video again in the Video section, then Try again.");
-    const { words } = wordsSchema.parse(JSON.parse(readFileSync(outputPath(deps.paths, context.work.projectId, timing.output.path), "utf8")));
+    const { words } = readTimingWords(deps, context, view, "the YouTube description's chapters come from");
     const passages = transcriptPassages(words);
     if (passages.length === 0)
         throw new Error("The narration's word timing holds no words, so there is nothing to write the YouTube description from. Use More → Render the video again in the Video section, then Try again.");
@@ -56,25 +47,16 @@ export async function executeYoutubeRecipe(deps, context, providers, piece) {
         durationSeconds,
         transcript: transcriptText(passages),
     }), config.language);
-    const row = deps.db
-        .prepare("SELECT recipe_context FROM revision_work WHERE id=?")
-        .get(context.work.workId);
-    const model = savedCatalogue(row?.recipe_context).llm.find((one) => one.provider === llm.provider && one.id === llm.model);
+    const call = workLlm(deps.db, context.work.workId, llm, piece.key);
     if (!context.maySubmit(piece.id))
         return "held";
-    const answer = await providers.forPiece(piece.id).llm({
-        provider: llm.provider,
-        model: llm.model,
-        ...(llm.thinking === undefined ? {} : { thinking: llm.thinking }),
-        thinkingConfig: llm.thinking === undefined ? null : (model?.llm.thinking?.[llm.thinking] ?? null),
+    const answer = await providers.forPiece(piece.id).llm(call({
         messages,
-        webSearch: false,
-        previewLabel: piece.key,
         check: (value) => {
             const checked = checkDescriptionAnswer(value.text, durationSeconds, config.title, shape);
             return checked.ok ? undefined : checked.reason;
         },
-    });
+    }));
     if (!answer.ok)
         return "held";
     const checked = checkDescriptionAnswer(answer.value.text, durationSeconds, config.title);
@@ -103,6 +85,6 @@ export async function executeYoutubeRecipe(deps, context, providers, piece) {
         tokensIn: answer.value.usage?.inputTokens ?? 0,
         tokensOut: answer.value.usage?.outputTokens ?? 0,
         descriptions: 1,
-    });
+    }, context.work.projectId);
     return "done";
 }

package/dist/slices/rebuild/transition-repo.js

@@ -1,5 +1,6 @@
 import { z } from "zod";
 import { transact } from "../../kernel/db/tx.js";
+import { internalError } from "../../kernel/errors.js";
 import { currentRevisionId, revisionById } from "../revisions/repo.js";
 import { insertWorkPiece } from "./work-records.js";
 const reservationSchema = z.object({
@@ -19,10 +20,10 @@ export function transitionRevisionWork(deps, input) {
         const base = revisionById(db, input.projectId, input.baseRevisionId);
         const next = revisionById(db, input.projectId, input.revisionId);
         if (base === undefined || next === undefined)
-            throw new Error("Slopify hit an internal error (a project version to switch between is missing). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+            throw new Error(internalError("a project version to switch between is missing"));
         if (JSON.stringify(Object.entries(input.fingerprints).sort()) !==
             JSON.stringify(Object.entries(next.fingerprints).sort()))
-            throw new Error("Slopify hit an internal error (the planned steps don't match the saved project version). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+            throw new Error(internalError("the planned steps don't match the saved project version"));
         const reserved = db
             .prepare(`SELECT r.* FROM revision_work_reservations r JOIN revision_work w ON w.id=r.work_id WHERE r.project_id=? AND r.revision_id=? AND w.state IN ('pending','running','done')`)
             .all(input.projectId, input.baseRevisionId)
@@ -78,7 +79,7 @@ export function transitionRevisionWork(deps, input) {
                 continue;
             const exact = input.recipes?.find((row) => row.key === key);
             if (exact !== undefined && exact.fingerprint !== fp)
-                throw new Error("Slopify hit an internal error (a planned step doesn't match the saved project version). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+                throw new Error(internalError("a planned step doesn't match the saved project version"));
             // An undo, a restore or a save repeated while a step waits for Resume asks for a
             // step the project already has a row for. A finished one is the result itself; a
             // held copy nobody started is the same placeholder. Minting another each time left a
@@ -93,7 +94,7 @@ export function transitionRevisionWork(deps, input) {
                 .prepare("SELECT id FROM stages WHERE project_id=? AND kind=?")
                 .get(input.projectId, kind);
             if (stage === undefined)
-                throw new Error("Slopify hit an internal error (a planned step has no matching stage). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+                throw new Error(internalError("a planned step has no matching stage"));
             const workId = deps.ids.next();
             const pieceId = deps.ids.next();
             db.prepare(`INSERT INTO revision_work(id,project_id,revision_id,stage_id,kind,fingerprint,state,dispatch_state,created_at) VALUES (?,?,?,?,?,?,'pending','held',?)`).run(workId, input.projectId, input.revisionId, z.string().parse(stage.id), kind, fp, deps.clock.now().toISOString());
@@ -178,5 +179,5 @@ export function stageForKey(key) {
         return "video";
     if (prefix === "document")
         return "document";
-    throw new Error("Slopify hit an internal error (unknown kind of step). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+    throw new Error(internalError("unknown kind of step"));
 }

package/dist/slices/rebuild/work-records.js

@@ -42,3 +42,9 @@ export function insertWorkPiece(db, piece) {
     const input = recipeInputSchema.parse(piece.input);
     db.prepare(`INSERT INTO revision_work_pieces(id,work_id,work_key,request_fingerprint,fingerprint,input_json,continuation,generation_token,state,dispatch_state,submitted_at,logical_fingerprint) VALUES (?,?,?,?,?,?,?,?,?,?,?,?)`).run(piece.id, piece.workId, piece.key, piece.requestFingerprint, piece.fingerprint, JSON.stringify(input), piece.continuation, piece.generationToken, piece.state, piece.dispatchState, piece.submittedAt, piece.logicalFingerprint ?? null);
 }
+// A step run again after its piece finished (a restart, a second claim) has nothing to do.
+export function pieceAlreadyDone(db, workId, pieceId) {
+    return (db
+        .prepare("SELECT state FROM revision_work_pieces WHERE id=? AND work_id=?")
+        .get(pieceId, workId)?.state === "done");
+}

package/dist/slices/reruns/cascade.js

@@ -40,8 +40,7 @@ function clearsOf(action, kind) {
     // The previous video stays downloadable until the new render finishes. Clearing the video's
     // outputs here would take it away the moment the user pressed Re-render - or for the whole of
     // a re-narration, when the video comes along as a dependent. It is kept instead, and
-    // `slices/video/run.ts` replaces the file and the rows in one swap once ffmpeg has exited
-    // cleanly. A render that fails or is canceled leaves the old one standing.
+    // the render replaces the file and the rows in one swap once ffmpeg has exited cleanly. A render that fails or is canceled leaves the old one standing.
     if (kind === "video") {
         return "nothing";
     }

package/dist/slices/reruns/index.js

@@ -1,6 +1,7 @@
 import { rmSync } from "node:fs";
 import { z } from "zod";
 import { transact } from "../../kernel/db/tx.js";
+import { errorText, internalError } from "../../kernel/errors.js";
 import { allPiecesOf, deletePiece, deletePieces, setPiece, } from "../../kernel/runner/piece-repo.js";
 import { projectById, stagesOf } from "../admission/repo.js";
 import { resetStage } from "../admission/stage-writes.js";
@@ -101,8 +102,7 @@ export function deleteImage(deps, projectId, outputId) {
         return { ok: false, reason: "last-image" };
     }
     return apply(deps, projectId, { kind: "image-deleted" }, loaded, () => {
-        // `slices/images/run.ts` counts an image as landed only when its row and its file both
-        // survive, so a delete removes both - and the piece that planned it, or the next run
+        // An image counts as landed only when its row and its file both survive, so a delete removes both - and the piece that planned it, or the next run
         // of the stage would read the plan and make the image again.
         deleteOutput(deps.db, image.id);
         dropImagePiece(deps, loaded.stages, image.path);
@@ -205,7 +205,7 @@ function clearStage(deps, stage, outputs) {
 // runs the returned cleanup only after commit. Finished narration is left intact.
 export function clearUnfinishedAudio(deps, projectId) {
     if (currentRevisionId(deps.db, projectId) !== undefined)
-        throw new Error("Slopify hit an internal error (narration was changed the old way on a project that uses versions). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("narration was changed the old way on a project that uses versions"));
     const stage = stagesOf(deps.db, projectId).find((one) => one.kind === "audio");
     if (stage === undefined ||
         stage.state === "done" ||
@@ -219,8 +219,8 @@ export function clearUnfinishedAudio(deps, projectId) {
     return [() => removeFiles(deps, projectId, orphaned)];
 }
 // A regenerated image is replaced in place at the same index, so the piece keeps the prompt
-// and the index it was planned with and only stops naming a file: `slices/images/run.ts`
-// then sees one image of its plan still to make, and the file it named can be removed.
+// and the index it was planned with and only stops naming a file: the images stage then
+// sees one image of its plan still to make, and the file it named can be removed.
 function forgetImageFile(deps, stages, path) {
     for (const piece of imagePieces(deps, stages, path)) {
         if (piece.payload === null) {
@@ -274,7 +274,7 @@ function removeFiles(deps, projectId, orphaned) {
             // the action here would leave the user with a stage that cannot be re-run.
             deps.log.write("warn", "reruns.file", {
                 projectId,
-                detail: `${path}: ${messageOf(error)}`,
+                detail: `${path}: ${errorText(error)}`,
             });
         }
     }
@@ -291,6 +291,3 @@ function stageOf(stages, kind) {
 function imageOf(outputs, outputId) {
     return outputs.find((output) => output.id === outputId && output.role === "image");
 }
-function messageOf(error) {
-    return error instanceof Error ? error.message : String(error);
-}

package/dist/slices/reviews/external-model.js

@@ -1,3 +1,6 @@
+// Who may answer an external review: the agent it waits for, or the person. The in-app
+// assistant never does (`edge/http/external-reviews.ts`).
+export const externalAnswerers = ["person", "agent"];
 // ceiling: the owner's number. An hourly agent routine answers well inside two hours; a run
 // with no agent at all still finishes the same evening.
 export const externalReviewWaitMs = 2 * 60 * 60 * 1000;

package/dist/slices/reviews/external-repo.js

@@ -1,5 +1,6 @@
 import { z } from "zod";
 import { transact } from "../../kernel/db/tx.js";
+import { internalError } from "../../kernel/errors.js";
 import { liveProject } from "../admission/repo.js";
 import { externalReviewByFingerprint, externalReviewById } from "./external-rows.js";
 import { liveWork, reviewStanding, wantedBy } from "./external-waiting.js";
@@ -17,7 +18,7 @@ export function openExternalReview(db, input) {
      WHERE external_reviews.state IN ('superseded','canceled')`).run(input.id, input.projectId, input.revisionId, input.workId, input.pieceId, input.workKey, input.itemKey, input.stage, input.itemFingerprint, input.reviewFingerprint, JSON.stringify(input.material), input.waitingSince, input.deadlineAt);
     const saved = externalReviewByFingerprint(db, input.projectId, input.reviewFingerprint);
     if (saved === undefined)
-        throw new Error("Slopify hit an internal error (an external review could not be saved). Retry stage; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("an external review could not be saved"));
     return saved;
 }
 // Cancel stops a run's reviews waiting. Retried, its step asks again (`openExternalReview`) and

package/dist/slices/reviews/external-rows.js

@@ -1,6 +1,5 @@
 import { z } from "zod";
-import { actors } from "../agents/model.js";
-import { externalReviewStates } from "./external-model.js";
+import { externalAnswerers, externalReviewStates } from "./external-model.js";
 import { reviewStages } from "./model.js";
 const materialSchema = z.object({
     request: z.object({
@@ -28,7 +27,7 @@ const rowSchema = z.object({
     waiting_since: z.string(),
     deadline_at: z.string(),
     verdict: z.string().nullable(),
-    answered_by: z.enum(actors).nullable(),
+    answered_by: z.enum(externalAnswerers).nullable(),
     answered_at: z.string().nullable(),
 });
 // One stored review as the slice speaks of it; throws for a row that can't be read back.

package/dist/slices/reviews/model.js

@@ -1,8 +1,27 @@
 // Automatic reviews: a reviewer model looks at each finished item of a stage (the article,
-// every image, the narration, the thumbnail, every short) before the run moves on, and says
+// the opening's promise, every image, the narration, the thumbnail, every short) before the
+// run moves on, and says
 // pass or fail with its reasons. A failed item is flagged, or made again while its retries
 // last. Browser-safe: Play, Edit project and the project page read these names and limits too.
-export const reviewStages = ["article", "images", "narration", "thumbnail", "shorts"];
+// "promise" reads the written article's opening against the title and thumbnail before the
+// paid narration starts; it is last so every list saved before it keeps its order.
+export const reviewStages = [
+    "article",
+    "images",
+    "narration",
+    "thumbnail",
+    "shorts",
+    "promise",
+];
+// The stages in the order a run reaches them, as the Reviews rows list them.
+export const reviewStagesInRunOrder = [
+    "article",
+    "promise",
+    "images",
+    "narration",
+    "thumbnail",
+    "shorts",
+];
 // Off is the same as the stage having no entry, which is what every project saved before
 // reviews existed has.
 export const reviewModes = ["off", "flag", "redo"];
@@ -26,6 +45,7 @@ export const reviewStageLabels = {
     narration: "Narration",
     thumbnail: "Thumbnail",
     shorts: "Shorts",
+    promise: "Promise check (before narration)",
 };
 export const reviewModeLabels = {
     off: "Off",
@@ -47,9 +67,6 @@ export function reviewRetriesOf(settings) {
 export function reviewedByAgent(settings) {
     return settings?.reviewer === "agent";
 }
-export function reviewedStages(settings) {
-    return reviewStages.filter((stage) => reviewModeOf(settings, stage) !== "off");
-}
 // The key a stage's picked Review prompt renders under in `config.rendered`.
 export function reviewPromptKey(stage) {
     return `review.${stage}`;
@@ -57,12 +74,12 @@ export function reviewPromptKey(stage) {
 // A review's step is `review:` and the key of the item it reviews. The narration is
 // reviewed as a whole, so its item key names no step.
 export const narrationItemKey = "narration";
+// The promise check reviews the article, but as its own item, so its verdict never stands in
+// for the article review's.
+export const promiseItemKey = "promise";
 export function reviewKey(itemKey) {
     return `review:${itemKey}`;
 }
-export function reviewedItemKey(key) {
-    return key.startsWith("review:") ? key.slice("review:".length) : undefined;
-}
 export function reviewSettingsForm(settings) {
     return {
         provider: settings.provider,
@@ -116,4 +133,5 @@ export const defaultReviewPrompts = {
     narration: "Review this narration against the text it was read from. The transcript was heard back from the audio; the gaps list stretches of text the timing could not hear. Fail it when words, sentences or passages are missing or garbled in a way a listener would notice. Ignore punctuation, numbers written out, and small differences in how words were transcribed.",
     thumbnail: "Review this thumbnail as it would be seen on a phone, about 320 pixels wide. Fail it when its text (if any) cannot be read at that size, when the subject is unclear or cluttered, when a hand or body is malformed, or when it shows stray letters or watermarks.",
     shorts: "Review these images from one vertical short, as they would be seen full screen on a phone. Fail them when the subject is unclear, when a hand or body is malformed, when they show stray text, letters or watermarks, or when they do not fit the short's title.",
+    promise: "Review whether this video's opening keeps the promise its title and thumbnail make. Pass it when the first 30 seconds start delivering what the title and thumbnail promise and give the viewer a reason to keep watching. Fail it when the opening is a slow preamble, greets the viewer or talks about the channel (subscribe, welcome back) before the subject, is about something other than the title, or promises something the rest of the article never delivers. When you fail it, name the sentence from the article that should come first.",
 };

package/dist/slices/reviews/repo.js

@@ -1,4 +1,5 @@
 import { z } from "zod";
+import { internalError } from "../../kernel/errors.js";
 import { actors } from "../agents/model.js";
 import { reviewAnswerers, reviewStages } from "./model.js";
 const rowSchema = z.object({
@@ -56,7 +57,7 @@ export function saveVerdict(db, record) {
        created_at=excluded.created_at`).run(record.id, record.projectId, record.revisionId, record.itemKey, record.stage, record.itemFingerprint, record.reviewFingerprint, record.passed ? 1 : 0, JSON.stringify(record.reasons), record.outcome, record.attempt, record.action, record.actionAt, record.redoState, record.redoError, record.answeredBy, record.actionBy, record.createdAt);
     const saved = verdictByReview(db, record.projectId, record.reviewFingerprint);
     if (saved === undefined)
-        throw new Error("Slopify hit an internal error (a review's verdict could not be saved). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+        throw new Error(internalError("a review's verdict could not be saved"));
     return saved;
 }
 export function verdictByReview(db, projectId, reviewFingerprint) {

package/dist/slices/reviews/rules.js

@@ -6,6 +6,7 @@ export function stageMakesItems(draft, stage) {
     const { sources } = draft;
     switch (stage) {
         case "article":
+        case "promise":
             return sources.article === "generate";
         case "images":
             return sources.images === "generate";

package/dist/slices/reviews/view.js

@@ -1,9 +1,12 @@
-import { narrationItemKey } from "./model.js";
+import { narrationItemKey, promiseItemKey } from "./model.js";
 // The step whose output a verdict is about: the word timing for the narration (it is what
-// the reviewer heard), a short's render for the short, and the item's own step otherwise.
+// the reviewer heard), the article for the promise check, a short's render for the short, and
+// the item's own step otherwise.
 export function reviewedOutputKey(itemKey) {
     if (itemKey === narrationItemKey)
         return "subtitles:timing";
+    if (itemKey === promiseItemKey)
+        return "article:body";
     if (/^shorts:\d+$/.test(itemKey))
         return `${itemKey}:render`;
     return itemKey;

package/dist/slices/revisions/downloads.js

@@ -8,6 +8,7 @@ import { contentTypeOf, downloadName, fileVersion, slugOf, versionedName, } from
 import { outputPath } from "../storage/layout.js";
 import { projectTitle } from "../storage/repo.js";
 import { outputSchema } from "../storage/schema.js";
+import { assetPathOrUndefined } from "./mutation-assets.js";
 import { positionedOutput } from "./projection.js";
 import { outputsForRevision, revisionById } from "./repo.js";
 import { stagePieceSchema } from "./schema.js";
@@ -71,12 +72,10 @@ export function revisionSetZip(deps, projectId, revisionId, set, only) {
         .map((row) => ({ ...positionedOutput(revision, row), slot: row.slot }))
         .sort((a, b) => setOrder(a.output.role, a.output.meta) - setOrder(b.output.role, b.output.meta));
     for (const row of members) {
-        const asset = deps.db
-            .prepare("SELECT path FROM project_assets WHERE project_id=? AND id=?")
-            .get(projectId, row.assetId);
-        if (asset === undefined)
+        const saved = assetPathOrUndefined(deps, projectId, row.assetId);
+        if (saved === undefined)
             throw new Error("Revision image asset is not registered.");
-        const path = outputPath(deps.paths, projectId, z.string().parse(asset.path));
+        const path = outputPath(deps.paths, projectId, saved);
         if (statSync(path, { throwIfNoEntry: false })?.isFile() !== true)
             continue;
         // Images and video are already compressed, so they are stored rather than deflated.

package/dist/slices/revisions/mutation-assets.js

@@ -186,11 +186,14 @@ export async function measureAudio(deps, projectId, path) {
     return duration;
 }
 export function assetPath(deps, projectId, id) {
-    return z
-        .object({ path: z.string() })
-        .parse(deps.db
+    return z.string().parse(assetPathOrUndefined(deps, projectId, id));
+}
+// The same, or undefined when the project has no such asset.
+export function assetPathOrUndefined(deps, projectId, id) {
+    const path = deps.db
         .prepare("SELECT path FROM project_assets WHERE project_id=? AND id=?")
-        .get(projectId, id)).path;
+        .get(projectId, id)?.path;
+    return typeof path === "string" ? path : undefined;
 }
 // A thumbnail step that draws its thumbnails, so each can be replaced by a file of one's own.
 export function generatesThumbnail(config) {

package/dist/slices/revisions/mutation-cues.js

@@ -6,6 +6,7 @@ import { usesVoices } from "../voices/model.js";
 import { cueSpeaker } from "../voices/timing.js";
 import { assetPath, measureAudio } from "./mutation-assets.js";
 import { validateCues } from "./rules.js";
+import { readyOutput } from "./view.js";
 // Edited captions of a multi-voice run keep their speakers. A cue that comes back without one
 // (the caption editor edits text and times only) takes the speaker it had before, found by the
 // cue's id, and a new cue the speaker of the narration under it, from the word timing. A cue
@@ -22,11 +23,7 @@ export function withCueSpeakers(deps, base, edit) {
     const timing = () => {
         if (words !== undefined)
             return words;
-        const row = base.outputs.find((one) => one.selected &&
-            one.available &&
-            one.state === "ready" &&
-            one.workKey === "subtitles:timing" &&
-            one.output.role === "subtitle_words");
+        const row = readyOutput(base, { workKey: "subtitles:timing", role: "subtitle_words" });
         try {
             words =
                 row === undefined

package/dist/slices/revisions/mutations.js

@@ -26,7 +26,6 @@ import { bindNarrationSources, validateNarrationIntent, validateTemplateIntent }
 import { revisionEditSchema } from "./schema.js";
 import { keptSubject } from "./subject.js";
 import { withEarlierJoins } from "./view.js";
-export { bindUpload } from "./mutation-assets.js";
 export { restoreRevision } from "./restore.js";
 export async function saveRevision(deps, input) {
     const parsed = revisionEditSchema.safeParse(input.edit);

package/dist/slices/revisions/repo.js

@@ -1,5 +1,5 @@
 import { z } from "zod";
-import { agentRevisionIds } from "../agents/repo.js";
+import { agentRevisionIds, assistantRevisionIds } from "../agents/repo.js";
 import { projectRevisionSchema } from "./schema.js";
 export { insertManifestOutput, insertManifestPiece, outputsForRevision, piecesForRevision, selectOutputRecord, selectPieceRecord, } from "./manifest-repo.js";
 const revisionRow = z.object({
@@ -39,6 +39,7 @@ export function currentRevisionId(db, projectId) {
 export function listRevisionHistory(db, projectId) {
     const head = currentRevisionId(db, projectId);
     const byAgent = agentRevisionIds(db, projectId);
+    const byAssistant = assistantRevisionIds(db, projectId);
     return db
         .prepare("SELECT * FROM project_revisions WHERE project_id=? ORDER BY created_at DESC,id DESC")
         .all(projectId)
@@ -52,6 +53,7 @@ export function listRevisionHistory(db, projectId) {
             createdAt: revision.createdAt,
             current: revision.id === head,
             ...(byAgent.has(revision.id) ? { byAgent: true } : {}),
+            ...(byAssistant.has(revision.id) ? { byAssistant: true } : {}),
         };
     });
 }

package/dist/slices/revisions/schema.js

@@ -221,6 +221,7 @@ export const revisionSummarySchema = z.object({
     createdAt: z.string(),
     current: z.boolean(),
     byAgent: z.boolean().optional(),
+    byAssistant: z.boolean().optional(),
 });
 export const revisionMutationSuccessSchema = z.object({
     ok: z.literal(true),