@gentbajko/slopify 3.25.9 → 4.1.0

package/dist/slices/narration/pronunciation.js

@@ -1,5 +1,7 @@
 import { remark } from "remark";
 import remarkGfm from "remark-gfm";
+import { internalError } from "../../kernel/errors.js";
+import { createRows, editSettings } from "../../kernel/ports/settings-places.js";
 import { remembered } from "../../kernel/remembered.js";
 export const currentPronunciationRules = 2;
 export function pronunciationRulesOf(config) {
@@ -153,7 +155,7 @@ function parseGlossary(markdown, language, rules = currentPronunciationRules) {
 export function skippedGlossaryNotice(skipped, unpronounced = 0) {
     if (skipped.length === 0 && unpronounced === 0)
         return null;
-    const where = "in the Pronunciation Glossary at the end of the article in Edit project → Article";
+    const where = `in the Pronunciation Glossary at the end of the article in ${editSettings.article}`;
     const blank = unpronounced === 0
         ? ""
         : `${unpronounced === 1 ? "1 name has" : `${unpronounced} names have`} no pronunciation given and ${unpronounced === 1 ? "is" : "are"} read as written. To give ${unpronounced === 1 ? "it one" : "them one"}, fill in its IPA ${where}.`;
@@ -178,7 +180,7 @@ export function skippedSpeakerPronunciationsNotice(rows) {
         return null;
     const parts = rows.map((row) => `${row.speaker}: ${row.skipped.map((one) => `entry ${one.row}: ${one.reason}`).join("; ")}`);
     return (`Speaker pronunciations: some entries are skipped and read as ordinary text (${parts.join(". ")}). ` +
-        "The rest are used. To use them, fix those entries under Speakers → Pronunciations for that speaker (Play → Narration, or Edit project → Providers).");
+        `The rest are used. To use them, fix those entries under Speakers → Pronunciations for that speaker (${createRows.narration}, or ${editSettings.narration}).`);
 }
 // Remembered (`kernel/remembered.ts`): every plan rebuild matches the whole narration again.
 export const pronunciationMatches = remembered(1024, (source, entries, possessives) => `${possessives === true ? "p" : "-"}\u0000${source}\u0000${JSON.stringify(entries)}`, matchPronunciations);
@@ -211,7 +213,7 @@ function matchPronunciations(source, entries, possessives = false) {
             continue;
         const entry = ordered.find((_, index) => match[index + 1] !== undefined);
         if (entry === undefined)
-            throw new Error("Slopify hit an internal error (a pronunciation glossary term has no pronunciation). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+            throw new Error(internalError("a pronunciation glossary term has no pronunciation"));
         const possessive = possessives ? match[ordered.length + 1] : undefined;
         matches.push({
             start: match.index,
@@ -237,7 +239,7 @@ export function pronunciationSpans(source, entries, possessives = false) {
         for (const [index, word] of words.entries()) {
             const ipa = match.entry.ipa[index];
             if (ipa === undefined)
-                throw new Error("Slopify hit an internal error (a pronunciation glossary term is missing a word). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+                throw new Error(internalError("a pronunciation glossary term is missing a word"));
             const start = match.start + word.index;
             const spoken = match.possessive !== undefined && index === words.length - 1 ? withPossessive(ipa) : ipa;
             spans.push({ start, end: start + word[0].length, text: `/${spoken}/` });

package/dist/slices/narration/steering.js

@@ -1,3 +1,4 @@
+import { internalError } from "../../kernel/errors.js";
 import { sourceSentences } from "./preparation.js";
 const cannotFit = {
     ok: false,
@@ -9,14 +10,14 @@ function atoms(source, from, to, spans) {
         const span = spans.get(at);
         if (span !== undefined) {
             if (span.end > to || span.end <= at)
-                throw new Error("Slopify hit an internal error (a pronunciation fix spans more than one word). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+                throw new Error(internalError("a pronunciation fix spans more than one word"));
             result.push({ text: span.text, spokenText: source.slice(at, span.end) });
             at = span.end;
         }
         else {
             const point = source.codePointAt(at);
             if (point === undefined)
-                throw new Error("Slopify hit an internal error (a narration position is outside the article text). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+                throw new Error(internalError("a narration position is outside the article text"));
             const text = String.fromCodePoint(point);
             // An empty span is a later word of a multi-word alias (its spoken form sits on the first
             // word), so the whitespace in front of it would leave two spaces where one word was.
@@ -45,7 +46,7 @@ function checkedSpans(source, pronunciation) {
             span.end <= span.start ||
             span.end > source.length ||
             !/^[^\s\uD800-\uDFFF]+$/u.test(source.slice(span.start, span.end)))
-            throw new Error("Slopify hit an internal error (pronunciation fixes overlap or split a word). Try again; if it happens again, use Download diagnostics in Settings and report it.");
+            throw new Error(internalError("pronunciation fixes overlap or split a word"));
         spans.set(span.start, span);
         end = span.end;
     }

package/dist/slices/notifications/notifier.js

@@ -1,3 +1,4 @@
+import { errorText } from "../../kernel/errors.js";
 import { redact } from "../../kernel/log.js";
 import { webhookBody } from "./address.js";
 import { autoPassNoticeText, noticeOf, noticeText, reviewNoticeText, runNotices, topicsNoticeText, } from "./rules.js";
@@ -84,7 +85,7 @@ export function createRunNotifier(deps) {
             .catch((error) => {
             deps.log.write("warn", "notification.failed", {
                 ...about,
-                detail: `${kind} notification not delivered: ${error instanceof Error ? error.message : String(error)}`,
+                detail: `${kind} notification not delivered: ${errorText(error)}`,
             });
         })
             .finally(() => {
@@ -123,7 +124,7 @@ export function createRunNotifier(deps) {
             catch (error) {
                 deps.log.write("warn", "notification.failed", {
                     projectId: event.projectId,
-                    detail: `notification skipped: ${error instanceof Error ? error.message : String(error)}`,
+                    detail: `notification skipped: ${errorText(error)}`,
                 });
             }
         },
@@ -138,7 +139,7 @@ export function createRunNotifier(deps) {
             }
             catch (error) {
                 deps.log.write("warn", "notification.failed", {
-                    detail: `topics notification skipped: ${error instanceof Error ? error.message : String(error)}`,
+                    detail: `topics notification skipped: ${errorText(error)}`,
                 });
             }
         },

package/dist/slices/notifications/rules.js

@@ -29,6 +29,7 @@ const reviewItemNames = {
     narration: "the narration",
     thumbnail: "the thumbnail",
     shorts: "a short",
+    promise: "the article's opening (promise check)",
 };
 export function reviewNoticeText(subject) {
     const title = subject.title.trim() === "" ? "Untitled project" : subject.title.trim();
@@ -48,6 +49,8 @@ export function reviewItemName(itemKey, imageOrder) {
         return "the article";
     if (itemKey === "narration")
         return "the narration";
+    if (itemKey === "promise")
+        return "the article's opening (promise check)";
     if (itemKey === "thumbnail:image")
         return "the thumbnail";
     const variant = /^thumbnail:image:(\d+)$/.exec(itemKey);
@@ -139,8 +142,8 @@ export function shortReason(reason) {
 }
 export function noticeText(kind, subject) {
     const text = runNoticeText(kind, subject);
-    return subject.startedBy === "agent"
-        ? { ...text, headline: `${text.headline} · started by agent` }
+    return subject.startedBy === "agent" || subject.startedBy === "assistant"
+        ? { ...text, headline: `${text.headline} · started by ${subject.startedBy}` }
         : text;
 }
 function runNoticeText(kind, subject) {

package/dist/slices/notifications/settings.js

@@ -1,19 +1,11 @@
-import { readSetting, writeSetting } from "../settings/repo.js";
+import { z } from "zod";
+import { deleteSetting, readJsonSetting, writeSetting } from "../settings/repo.js";
 import { notificationUrlProblem } from "./address.js";
 // Its own row in the key/value `settings` table. Backups leave it out
 // (`slices/storage/portable.ts`): an ntfy topic in the URL is as good as a password.
 export const notificationUrlKey = "notificationUrl";
 export function readNotificationUrl(db) {
-    const stored = readSetting(db, notificationUrlKey);
-    if (stored === undefined)
-        return null;
-    let value;
-    try {
-        value = JSON.parse(stored);
-    }
-    catch {
-        return null;
-    }
+    const value = readJsonSetting(db, notificationUrlKey, z.unknown(), null);
     // A hand-edited row that no longer passes the rules is treated as not set rather than
     // POSTed to.
     return typeof value === "string" && value !== "" && notificationUrlProblem(value) === undefined
@@ -26,7 +18,7 @@ export function saveNotificationUrl(db, raw) {
         return { ok: false, message: problem };
     const url = raw.trim();
     if (url === "") {
-        db.prepare("DELETE FROM settings WHERE key = ?").run(notificationUrlKey);
+        deleteSetting(db, notificationUrlKey);
         return { ok: true, url: null };
     }
     writeSetting(db, notificationUrlKey, JSON.stringify(url));

package/dist/slices/notifications/subject.js

@@ -3,17 +3,18 @@ import { projectById, stagesOf } from "../admission/repo.js";
 import { stepFailedWhileWaiting } from "../admission/state-words.js";
 import { lastStarter } from "../agents/repo.js";
 import { projectHolds } from "../reviews/holds.js";
-import { uploadedAndSettled } from "../uploads/repo.js";
+import { isTemplateSample } from "../template-studio/repo.js";
+import { settledProjects } from "../uploads/repo.js";
 import { settledStatus } from "../uploads/rules.js";
 import { atItsDay, failuresOf } from "./rules.js";
 // What a notification says about a project, read at the moment it goes out, by the rule the
 // Projects list reads it with (`edge/http/projects.ts`).
 export function noticeSubject(db, projectId) {
     const project = projectById(db, projectId);
-    if (project === undefined)
+    if (project === undefined || isTemplateSample(db, projectId))
         return undefined;
     const stages = stagesOf(db, projectId);
-    const settled = uploadedAndSettled(db, projectId);
+    const settled = settledProjects(db, projectId);
     const holds = projectHolds(db, { projectId, standings: new Map([[projectId, stages]]), settled });
     const status = settledStatus(derive(stages, project.paused), settled.has(projectId));
     const failures = failuresOf(stages);

package/dist/slices/onboarding/copy.js

@@ -1,6 +1,7 @@
 import { copyFileSync, lstatSync, mkdirSync, readdirSync, rmSync } from "node:fs";
 import { dirname, join, relative, sep } from "node:path";
 import { transact } from "../../kernel/db/tx.js";
+import { internalError } from "../../kernel/errors.js";
 import { executionPlan } from "../rebuild/runtime-plan.js";
 import { currentRevisionId } from "../revisions/repo.js";
 import { getRevisionView } from "../revisions/view.js";
@@ -11,7 +12,7 @@ import { projectDir } from "../storage/layout.js";
 // ceiling: one pass per layer of the step graph (text, narration, timing, picks, prompts,
 // images, renders); far more than any plan has.
 const rebindPasses = 40;
-export function copyProject(deps, sourceId, title) {
+export function copyProject(deps, sourceId, title, { renamed = false } = {}) {
     const source = projectDir(deps.paths, sourceId);
     const tables = projectTables.map((table) => [table, projectRows(deps.db, table, sourceId)]);
     const ids = new Map([[sourceId, deps.ids.next()]]);
@@ -33,6 +34,9 @@ export function copyProject(deps, sourceId, title) {
     const projectId = ids.get(sourceId) ?? sourceId;
     const target = projectDir(deps.paths, projectId);
     const at = deps.clock.now().toISOString();
+    const named = (row) => renamed && typeof row.config === "string"
+        ? { ...row, config: renameConfig(row.config, title) }
+        : row;
     try {
         for (const file of listFiles(source)) {
             if (file === projectMarker)
@@ -51,7 +55,11 @@ export function copyProject(deps, sourceId, title) {
                         column,
                         typeof value === "string" ? remap(value) : value,
                     ]));
-                    insert(deps.db, table, table === "projects" ? { ...copied, title, created_at: at, updated_at: at } : copied);
+                    insert(deps.db, table, table === "projects"
+                        ? { ...named(copied), title, created_at: at, updated_at: at }
+                        : table === "project_revisions"
+                            ? named(copied)
+                            : copied);
                 }
             rebind(deps, projectId);
         });
@@ -62,6 +70,16 @@ export function copyProject(deps, sourceId, title) {
     }
     return { projectId };
 }
+// A stored config (JSON) under a new title, keeping what it was made about.
+function renameConfig(config, title) {
+    const parsed = JSON.parse(config);
+    if (typeof parsed !== "object" || parsed === null || !("title" in parsed))
+        return config;
+    const before = parsed;
+    if (typeof before.title !== "string" || before.title === title)
+        return config;
+    return JSON.stringify({ ...before, title, subjectTitle: before.subjectTitle ?? before.title });
+}
 // Re-stamps the copy's finished steps with the fingerprints its own ids give them.
 function rebind(deps, projectId) {
     const revisionId = currentRevisionId(deps.db, projectId);
@@ -104,7 +122,7 @@ function rebind(deps, projectId) {
                     .prepare(`UPDATE ${table} SET ${column}=replace(${column},?,?) WHERE project_id=? AND instr(${column},?)>0`)
                     .run(before, after, projectId, before);
     }
-    throw new Error("Slopify hit an internal error (the copy of the sample kept changing while its steps were matched up). Delete the copy and try Make my own copy again; if it happens again, use Download diagnostics in Settings and report it.");
+    throw new Error(internalError("the copy of the sample kept changing while its steps were matched up", "Delete the copy and try Make my own copy again"));
 }
 function requestOf(payload) {
     try {

package/dist/slices/onboarding/first-run.js

@@ -1,26 +1,35 @@
-import { readinessIsUsable } from "../../kernel/ports/model.js";
 import { templateById } from "../project-templates/repo.js";
-import { starterPacks } from "./packs.js";
+import { listVoices } from "../settings/repo.js";
+import { notTemplateSample } from "../template-studio/repo.js";
+import { starterPacks, starterSet } from "./packs.js";
+import { shortReadiness } from "./quick-short.js";
 import { sampleProjectIds } from "./sample.js";
 import { firstRunDismissed, readPackRecords } from "./state.js";
 // The first-run screen shows on a fresh install and never again once it is skipped or a real
-// project (anything but the samples) exists. Reading it writes nothing: a real project that
+// project (anything but the samples and the template studio's hidden projects) exists. Reading it writes nothing: a real project that
 // exists before the screen was recorded as done is reported as `settle`, and the client
 // records it with POST /dismiss, so deleting every project later does not bring it back.
-export function firstRunView(db, statuses) {
+export function firstRunView(db, statuses, 
+// How to install the computer's voice here, for the checklist's voice line.
+install) {
     const samples = sampleProjectIds(db);
     const real = db
-        .prepare("SELECT 1 FROM projects WHERE id IS NOT ? AND id IS NOT ? AND id IS NOT ? LIMIT 1")
+        .prepare(`SELECT 1 FROM projects WHERE id IS NOT ? AND id IS NOT ? AND id IS NOT ? AND ${notTemplateSample()} LIMIT 1`)
         .get(samples.library, samples.audiobook, samples.podcast) !== undefined;
     const dismissed = firstRunDismissed(db);
     const packs = readPackRecords(db);
     return {
         show: !real && !dismissed,
         settle: real && !dismissed,
-        voice: firstVoice(statuses),
+        // Every starter pack suggests the same voice, so the starter set's stands for all of them.
+        needs: shortReadiness({
+            statuses,
+            voices: listVoices(db),
+            suggested: starterSet.voice,
+            install,
+        }),
         sampleProjectId: samples.library,
         samples,
-        clis: detectedClis(statuses),
         // A pack counts as added only with its template: the first short adds a pack's prompts and
         // voice but no template, and Add pack must still be able to add that template later (or
         // again, after it was deleted).
@@ -37,36 +46,3 @@ export function firstRunView(db, statuses) {
         }),
     };
 }
-// Who would narrate a first short: a keyed voice provider when one has a key, else the
-// computer's own voice when a speech program was found.
-export function firstVoice(statuses) {
-    const keyed = statuses.find((row) => row.family === "tts" && row.readiness.kind === "keyed" && readinessIsUsable(row.readiness));
-    const system = statuses.find((row) => row.readiness.kind === "local")?.readiness;
-    const local = system?.kind === "local" ? system : undefined;
-    return {
-        keyed: keyed?.displayName ?? null,
-        system: {
-            available: local?.available === true,
-            engine: local?.engine ?? null,
-            issue: local?.issue ?? null,
-        },
-    };
-}
-// What this machine can make without a key: each agent CLI Slopify found, and whether it
-// answers (installed, a supported version, signed in as far as the probe can tell).
-export function detectedClis(statuses) {
-    return ["claude-code", "codex", "gemini"].map((id) => {
-        const status = statuses.find((row) => row.id === id);
-        const readiness = status?.readiness;
-        const cli = readiness?.kind === "cli" ? readiness : undefined;
-        return {
-            id,
-            name: status?.displayName ?? id,
-            installed: cli?.installed === true,
-            ready: cli?.installed === true && cli.issue === undefined,
-            version: cli?.version ?? null,
-            issue: cli?.issue ?? null,
-            draws: id === "codex",
-        };
-    });
-}

package/dist/slices/onboarding/full-video.js

@@ -26,7 +26,7 @@ export function fullVideoDraft(deps, input) {
     const config = project.config;
     if (!usesShortMode(config))
         return { ok: false, reason: "not-short" };
-    const topic = config.values.topic?.trim() || project.title;
+    const topic = config.values.Topic?.trim() || config.values.topic?.trim() || project.title;
     const pack = shortPack(deps, config.articlePrompt ?? "");
     const document = packDocument(deps, pack);
     if (document === undefined)
@@ -46,7 +46,8 @@ export function fullVideoDraft(deps, input) {
             form: {
                 ...form,
                 title: project.title,
-                values: { ...form.values, topic },
+                // {{Topic}} since 4.0; a pack installed before asks for {{topic}}.
+                values: { ...form.values, Topic: topic, topic },
                 llm: provider(config.llm, form.llm),
                 images: provider(config.images, form.images),
                 audio: config.audio === undefined

package/dist/slices/onboarding/install.js

@@ -1,9 +1,11 @@
 import { transact } from "../../kernel/db/tx.js";
+import { internalError } from "../../kernel/errors.js";
 import { detectSlots } from "../admission/substitute.js";
 import { nameMax } from "../library/model.js";
 import { insertPrompt, promptById, promptByName } from "../library/repo.js";
 import { templateById, templateSummaries } from "../project-templates/repo.js";
 import { createTemplate } from "../project-templates/service.js";
+import { readProviderDefaults } from "../settings/first-run.js";
 import { insertVoice, listVoices } from "../settings/repo.js";
 import { packById } from "./packs.js";
 import { readPackRecords, writePackRecord } from "./state.js";
@@ -45,10 +47,10 @@ export function installPack(deps, packId, options = { template: true }) {
             const created = createTemplate(deps, {
                 id,
                 name: freeTemplateName(deps, `${pack.name} starter`),
-                document: packTemplate(pack, names, voice.voiceId),
+                document: packTemplate(pack, names, voice.voiceId, readProviderDefaults(deps.db)),
             });
             if (!created.ok)
-                throw new Error(`Slopify hit an internal error (the ${pack.name} pack's template was refused: ${created.reason}). Try again; if it happens again, use Download diagnostics in Settings and report it.`);
+                throw new Error(internalError(`the ${pack.name} pack's template was refused: ${created.reason}`));
             template = id;
             added = true;
         }
@@ -71,7 +73,7 @@ export function installPack(deps, packId, options = { template: true }) {
         };
     });
 }
-function placePrompt(deps, kind, name, body, at) {
+export function placePrompt(deps, kind, name, body, at) {
     for (let n = 1;; n += 1) {
         const suffix = n === 1 ? "" : ` (${String(n)})`;
         const candidate = `${name.slice(0, nameMax - suffix.length).trimEnd()}${suffix}`;
@@ -109,7 +111,7 @@ function placeVoice(deps, pack, before) {
     });
     return { id, voiceId: pack.voice.voiceId, added: true };
 }
-function freeTemplateName(deps, name) {
+export function freeTemplateName(deps, name) {
     const taken = new Set(templateSummaries(deps.db).map((row) => row.name.toLowerCase()));
     for (let n = 1;; n += 1) {
         const candidate = n === 1 ? name : `${name} (${String(n)})`;

package/dist/slices/onboarding/model.js

@@ -10,6 +10,25 @@ export const sampleProjectsSchema = z.object({
     audiobook: z.string().nullable(),
     podcast: z.string().nullable(),
 });
+// What a 60-second short needs that nothing on this computer does yet, with what fixes it. The
+// first-run checklist and the refusal of "Make a 60-second short" say the same gaps.
+export const shortGapSchema = z.object({
+    need: z.enum(["text", "voice", "images"]),
+    // What is missing and what fixes it, in a sentence or two.
+    message: z.string(),
+    // A key for that part, or (the voice) a voice saved for the voice service that has a key.
+    fix: z.enum(["key", "saved-voice"]),
+    // The voice only: how to install the computer's own free voice from a terminal, where that
+    // can be done.
+    install: z.object({ text: z.string(), commands: z.array(z.string()) }).optional(),
+});
+// Who does one part of a first short, by name ("Claude Code CLI", "OpenAI", "eSpeak NG"), and
+// how: with the person's own sign-in to a tool on this computer, a saved key, or (the voice)
+// the computer's own speech. Null while nothing can.
+const shortPartSchema = z.object({
+    by: z.string().nullable(),
+    how: z.enum(["sign-in", "key", "computer"]).nullable(),
+});
 export const firstRunViewSchema = z
     .object({
     show: z.boolean(),
@@ -17,31 +36,17 @@ export const firstRunViewSchema = z
     // with POST /dismiss, so deleting every project later does not bring the screen back.
     // Reading this view writes nothing.
     settle: z.boolean(),
-    // Who narrates a first short: a voice provider with a saved key, else the computer's own.
-    voice: z.object({
-        // The keyed voice provider that would be used, by name ("OpenAI"), or null.
-        keyed: z.string().nullable(),
-        system: z.object({
-            available: z.boolean(),
-            // The speech program found ("eSpeak NG", "macOS voices").
-            engine: z.string().nullable(),
-            // Why none can be used, with the fix.
-            issue: z.string().nullable(),
-        }),
+    // The three things a short needs (the script, a voice and the images), each with who would
+    // do it, and the gaps, as "Make a 60-second short" would refuse them.
+    needs: z.object({
+        text: shortPartSchema,
+        voice: shortPartSchema,
+        images: shortPartSchema,
+        gaps: z.array(shortGapSchema),
     }),
     // The Library of Alexandria's project, as before the demos; `samples` has all three.
     sampleProjectId: z.string().nullable(),
     samples: sampleProjectsSchema,
-    clis: z.array(z.object({
-        id: z.enum(["claude-code", "codex", "gemini"]),
-        name: z.string(),
-        installed: z.boolean(),
-        ready: z.boolean(),
-        version: z.string().nullable(),
-        issue: z.string().nullable(),
-        // Codex draws the images too.
-        draws: z.boolean(),
-    })),
     packs: z.array(z.object({
         id: z.string(),
         name: z.string(),
@@ -63,22 +68,6 @@ export const quickShortInputSchema = z
 export const fullVideoInputSchema = z
     .object({ projectId: z.string().min(1).max(64), draftId: z.uuid() })
     .strict();
-export const quickShortResultSchema = z.object({
-    projectId: z.string(),
-    replayed: z.boolean(),
-});
-export const installedPackSchema = z.object({
-    packId: z.string(),
-    added: z.boolean(),
-    prompts: z.record(z.string(), z.string()),
-    templateId: z.string().nullable(),
-    voice: z.string().nullable(),
-});
-export const sampleCopySchema = z.object({ projectId: z.string() });
-export const sampleStateSchema = z.object({
-    projectId: z.string().nullable(),
-    samples: sampleProjectsSchema,
-});
 // Make my own copy of one sample; without a project, the Library of Alexandria.
 export const sampleCopyInputSchema = z
     .object({ projectId: z.string().max(60).optional() })