@agent-native/core 0.149.1 → 0.149.3

package/corpus/templates/slides/actions/patch-deck.ts

@@ -31,7 +31,12 @@ import {
   sourceImportForDeck,
   type SourceImportMetadata,
 } from "../server/lib/source-import.js";
+import { assertSlideAnimationsResolve } from "../server/lib/validate-slide-animations.js";
 import { ASPECT_RATIO_VALUES } from "../shared/aspect-ratios.js";
+import {
+  assertHumanReadableDeckTitle,
+  repairGeneratedDeckTitle,
+} from "../shared/deck-title.js";
 
 // ---------------------------------------------------------------------------
 // Per-deck write lock — same pattern as add-slide.ts so all client and agent
@@ -66,6 +71,29 @@ export function withDeckLock<T>(
 // Operation schemas
 // ---------------------------------------------------------------------------
 
+const SlideAnimationSchema = z.object({
+  id: z.string().min(1).describe("Stable ID for this ordered reveal step"),
+  elementIndex: z
+    .number()
+    .int()
+    .min(0)
+    .describe(
+      "0-based legacy child index. Keep it paired with elementPath for compatibility.",
+    ),
+  elementPath: z
+    .array(z.number().int().min(0))
+    .min(1)
+    .optional()
+    .describe(
+      "Preferred 0-based child-index path from the outer .fmd-slide wrapper. Required for agent-created or content-revised animations; re-read final HTML after content edits.",
+    ),
+  type: z
+    .enum(["appear", "fade", "slide-up", "zoom"])
+    .describe(
+      "Animation used when this step is revealed. Supported semantics: appear (immediate reveal), fade (opacity), slide-up (subtle upward motion), zoom (subtle scale). Do not invent other types.",
+    ),
+});
+
 const SlideFieldsSchema = z.object({
   content: z.string().optional(),
   notes: z.string().optional(),
@@ -75,8 +103,16 @@ const SlideFieldsSchema = z.object({
   imageLoading: z.boolean().optional(),
   imagePrompt: z.string().optional(),
   excalidrawData: z.string().optional(),
-  transition: z.string().optional(),
-  animations: z.array(z.unknown()).optional(),
+  transition: z
+    .enum(["instant", "none", "fade", "slide", "zoom"])
+    .optional()
+    .describe("Transition used when entering this slide"),
+  animations: z
+    .array(SlideAnimationSchema)
+    .optional()
+    .describe(
+      "Complete ordered on-click reveal list. Include every intended target in order; unlisted elements remain visible. Use elementPath from the final HTML and 0-based indexes.",
+    ),
 });
 
 /** Update fields on a single existing slide */
@@ -171,11 +207,51 @@ export function assertSourceImportOperationsPreserved(
   );
 }
 
+/**
+ * A deck-wide source restyle must be atomic from the agent's point of view:
+ * accepting a partial batch makes an apparently successful run indistinguishable
+ * from a run that quietly left the tail of the imported deck untouched.
+ */
+export function assertSourceImportSlidesCovered(
+  metadata: SourceImportMetadata | null,
+  operations: Operation[],
+  requireAllSourceSlides: boolean,
+): void {
+  if (!metadata || !requireAllSourceSlides) return;
+
+  const sourceSlideIds =
+    Array.isArray(metadata.slideIds) && metadata.slideIds.length > 0
+      ? metadata.slideIds
+      : metadata.slides.map((slide) => slide.id);
+  const patchedContentSlideIds = new Set(
+    operations.flatMap((operation) =>
+      operation.op === "patch-slide" && operation.fields.content !== undefined
+        ? [operation.slideId]
+        : [],
+    ),
+  );
+  const missingSlideIds = sourceSlideIds.filter(
+    (slideId) => !patchedContentSlideIds.has(slideId),
+  );
+  if (missingSlideIds.length === 0) return;
+
+  throw new Error(
+    `Deck-wide source restyle requires one content patch per imported slide. Missing ${missingSlideIds.length} slide(s): ${missingSlideIds.slice(0, 12).join(", ")}${missingSlideIds.length > 12 ? ", …" : ""}. Continue with every source slide ID in one patch-deck call before verifying with get-deck compact=true.`,
+  );
+}
+
 // The browser uses the full operation union above. Agents additionally use
 // this action for one bounded, deck-wide layout repair: one patch-slide per
-// slide in a single SQL transaction, followed by a fresh read for verification.
+// source slide in a single SQL transaction, followed by compact verification.
 const AgentPatchDeckInputSchema = z.object({
   deckId: z.string().describe("Deck ID"),
+  requireAllSourceSlides: z
+    .boolean()
+    .optional()
+    .default(false)
+    .describe(
+      "For a deck-wide source-import restyle, require one content patch for every imported slide before any write is committed.",
+    ),
   operations: z
     .array(
       z.union([
@@ -192,7 +268,7 @@ const AgentPatchDeckInputSchema = z.object({
     )
     .min(1)
     .describe(
-      "One patch-slide operation per slide that needs a structural HTML repair. Use patch-deck-fields only for a deck title change.",
+      "For a deck-wide source restyle, include one patch-slide operation with content for every existing source slide. Use patch-deck-fields only for a deck title change.",
     ),
 });
 
@@ -332,7 +408,19 @@ export function applyOperation(deck: any, op: Operation): void {
 
     case "patch-deck-fields": {
       const { fields } = op;
-      if (fields.title !== undefined) deck.title = fields.title;
+      if (fields.title !== undefined) {
+        const repairedTitle = repairGeneratedDeckTitle(
+          fields.title,
+          slides[0]?.content,
+          deck.title,
+        );
+        if (repairedTitle) {
+          deck.title = repairedTitle;
+        } else {
+          assertHumanReadableDeckTitle(fields.title);
+          deck.title = fields.title;
+        }
+      }
       if ("designSystemId" in fields)
         deck.designSystemId = fields.designSystemId;
       if (fields.tweaks !== undefined) deck.tweaks = fields.tweaks;
@@ -346,6 +434,43 @@ export function applyOperation(deck: any, op: Operation): void {
   }
 }
 
+/**
+ * Content and animation metadata are one contract. Validate only slides whose
+ * content or animation list changed so unrelated note/title writes do not
+ * resurrect old metadata failures, while any edit that can stale a path is
+ * rejected before persistence.
+ */
+export function assertPatchedSlideAnimationsResolve(
+  deck: any,
+  operations: readonly Operation[],
+  options?: { requireElementPaths?: boolean },
+): void {
+  const slideIdsToValidate = new Set(
+    operations.flatMap((operation) =>
+      operation.op === "patch-slide" &&
+      (operation.fields.content !== undefined ||
+        operation.fields.animations !== undefined)
+        ? [operation.slideId]
+        : [],
+    ),
+  );
+  if (slideIdsToValidate.size === 0) return;
+
+  const slides: any[] = Array.isArray(deck.slides) ? deck.slides : [];
+  for (const slideId of slideIdsToValidate) {
+    const slide = slides.find((candidate) => candidate?.id === slideId);
+    if (!slide || !Array.isArray(slide.animations) || !slide.animations.length)
+      continue;
+
+    assertSlideAnimationsResolve({
+      slideId,
+      content: typeof slide.content === "string" ? slide.content : "",
+      animations: slide.animations,
+      requireElementPaths: options?.requireElementPaths,
+    });
+  }
+}
+
 /**
  * Resolve the last operation in a sequence. For example, when typing a new name
  * this will be the latest name of the deck in a sequence of keystrokes.
@@ -353,6 +478,7 @@ export function applyOperation(deck: any, op: Operation): void {
 export function resolveDeckColumnUpdates(
   current: { title: string; designSystemId: string | null },
   operations: Operation[],
+  resolvedTitle?: string,
 ): { title: string; designSystemId: string | null } {
   const fieldOps = operations
     .filter(
@@ -363,7 +489,7 @@ export function resolveDeckColumnUpdates(
   const titleOp = fieldOps.find((op) => typeof op.fields.title === "string");
   const dsOp = fieldOps.find((op) => "designSystemId" in op.fields);
   return {
-    title: titleOp?.fields.title ?? current.title,
+    title: resolvedTitle ?? titleOp?.fields.title ?? current.title,
     designSystemId: dsOp
       ? (dsOp.fields.designSystemId ?? null)
       : current.designSystemId,
@@ -391,10 +517,22 @@ export default defineAction({
     "Granular deck patch used by the browser editor for concurrent-safe writes. " +
     "Each operation touches only the target slide or field — concurrent writers " +
     "on different slides never overwrite each other's work. For a deck-wide " +
-    "layout repair, send one patch-slide operation per affected slide in one " +
-    "call, then call get-deck to verify the persisted HTML before reporting success.",
+    "source restyle, set requireAllSourceSlides=true and send one patch-slide " +
+    "operation with content for every imported slide in one call; the action " +
+    "rejects partial coverage. For animations, inspect the final slide HTML, " +
+    "then patch content and the complete ordered animations list together; " +
+    "validate every 0-based elementPath and do not invent one-based indexes. " +
+    "Then call get-deck with compact=true to verify the persisted slide IDs, " +
+    "count, and animation metadata before reporting success.",
   schema: z.object({
     deckId: z.string().describe("Deck ID"),
+    requireAllSourceSlides: z
+      .boolean()
+      .optional()
+      .default(false)
+      .describe(
+        "For a deck-wide source-import restyle, require one content patch for every imported slide before any write is committed.",
+      ),
     operations: z
       .array(OperationSchema)
       .min(1)
@@ -414,7 +552,10 @@ export default defineAction({
       ),
   }),
   agentInputSchema: AgentPatchDeckInputSchema,
-  run: async ({ deckId, operations, creativeContext }, ctx) => {
+  run: async (
+    { deckId, operations, requireAllSourceSlides, creativeContext },
+    ctx,
+  ) => {
     await assertAccess("deck", deckId, "editor");
     const isAgentCaller = isAgentPatchCaller(ctx?.caller);
 
@@ -450,6 +591,11 @@ export default defineAction({
       const sourceImport = sourceImportForDeck(deck.sourceImport);
       if (isAgentCaller) {
         assertSourceImportOperationsPreserved(sourceImport, operations);
+        assertSourceImportSlidesCovered(
+          sourceImport,
+          operations,
+          requireAllSourceSlides,
+        );
       }
       for (const op of operations) {
         if (
@@ -474,6 +620,9 @@ export default defineAction({
       for (const op of operations) {
         applyOperation(deck, op);
       }
+      assertPatchedSlideAnimationsResolve(deck, operations, {
+        requireElementPaths: isAgentCaller,
+      });
 
       const now = new Date().toISOString();
       deck.updatedAt = now;
@@ -482,6 +631,13 @@ export default defineAction({
         resolveDeckColumnUpdates(
           { title: row.title, designSystemId: row.designSystemId },
           operations,
+          operations.some(
+            (operation) =>
+              operation.op === "patch-deck-fields" &&
+              operation.fields.title !== undefined,
+          ) && typeof deck.title === "string"
+            ? deck.title
+            : undefined,
         );
 
       let generationRecord:

package/corpus/templates/slides/actions/save-deck.ts

@@ -19,6 +19,10 @@ import { z } from "zod";
 import { getDb, schema } from "../server/db/index.js";
 import { notifyClients } from "../server/handlers/decks.js";
 import { createDeckVersionSnapshot } from "../server/lib/deck-versions.js";
+import {
+  assertHumanReadableDeckTitle,
+  repairGeneratedDeckTitle,
+} from "../shared/deck-title.js";
 import {
   assertDesignSystemReadable,
   assertValidAspectRatio,
@@ -71,7 +75,7 @@ export default defineAction({
 
       deck.id = deckId;
       deck.updatedAt = now;
-      const title = deckTitle(deck);
+      const requestedTitle = deckTitle(deck);
       const nextDesignSystemId = deckDesignSystemId(deck);
 
       // Resolve access first — this loads the row AND tells us the caller's
@@ -88,6 +92,11 @@ export default defineAction({
         if (!ownerEmail) {
           throw deckHttpError(403, "Sign in to create a deck");
         }
+        const title =
+          repairGeneratedDeckTitle(requestedTitle, firstSlideContent(deck)) ??
+          requestedTitle;
+        assertHumanReadableDeckTitle(title);
+        deck.title = title;
         await assertDesignSystemReadable(nextDesignSystemId);
         try {
           await db.insert(schema.decks).values({
@@ -110,6 +119,14 @@ export default defineAction({
         access.role === "admin" ||
         access.role === "editor"
       ) {
+        const title =
+          repairGeneratedDeckTitle(
+            requestedTitle,
+            firstSlideContent(deck),
+            access.resource.title,
+          ) ?? requestedTitle;
+        assertHumanReadableDeckTitle(title);
+        deck.title = title;
         await assertDesignSystemReadable(nextDesignSystemId);
         if (shouldSnapshotDeckWrite(access.resource, title, deck)) {
           await createDeckVersionSnapshot(
@@ -142,3 +159,9 @@ export default defineAction({
       return deck;
     }),
 });
+
+function firstSlideContent(deck: DeckPayload): string | null {
+  const slides = Array.isArray(deck.slides) ? deck.slides : [];
+  const content = slides[0] && (slides[0] as Record<string, unknown>).content;
+  return typeof content === "string" ? content : null;
+}

package/corpus/templates/slides/actions/update-slide.ts

@@ -364,7 +364,7 @@ export default defineAction({
     if (rmw.notFound) {
       return {
         ok: false,
-        message: `Text not found in slide: "${find!.slice(0, 60)}". Use get-deck to see current slide content.`,
+        message: `Text not found in slide: "${find!.slice(0, 60)}". Use get-deck with compact=false to see the current slide HTML.`,
       };
     }
 

package/corpus/templates/slides/app/components/comments/SlideCommentsPanel.tsx

@@ -458,9 +458,9 @@ export function SlideCommentsPanel({
   const showInput = pendingComment || addingComment;
 
   return (
-    <div className="flex h-full w-[17rem] flex-shrink-0 flex-col border-l border-border bg-background">
+    <div className="flex h-full w-[17rem] flex-shrink-0 flex-col bg-[var(--slides-editor-surface)]">
       {/* Header */}
-      <div className="flex items-center justify-between px-4 py-3 border-b border-border flex-shrink-0">
+      <div className="flex flex-shrink-0 items-center justify-between px-4 py-3">
         <span className="text-[13px] font-medium text-foreground/80">
           {t("comments.title")}
         </span>

package/corpus/templates/slides/app/components/deck/DeckCard.tsx

@@ -7,7 +7,6 @@ import {
   IconTrash,
   IconCopy,
   IconPencil,
-  IconPalette,
   IconPlus,
   IconStar,
   IconStarFilled,
@@ -33,7 +32,6 @@ interface DeckCardProps {
   onRename: (id: string, newTitle: string) => void;
   onDuplicate: (id: string) => void;
   onToggleStar: (id: string, starred: boolean) => void;
-  designSystemTitle?: string | null;
   isWorkspaceDefault?: boolean;
   canSetWorkspaceDefault?: boolean;
   onSetWorkspaceDefault?: (id: string, isDefault: boolean) => void;
@@ -45,7 +43,6 @@ export default function DeckCard({
   onRename,
   onDuplicate,
   onToggleStar,
-  designSystemTitle,
   isWorkspaceDefault = false,
   canSetWorkspaceDefault = false,
   onSetWorkspaceDefault,
@@ -101,7 +98,7 @@ export default function DeckCard({
     <div className="group relative">
       <Link
         to={`/deck/${deck.id}`}
-        className="block rounded-xl border border-border bg-card hover:border-border transition-all duration-200 overflow-hidden hover:shadow-lg hover:shadow-[#609FF8]/5"
+        className="block overflow-hidden rounded-xl border border-transparent bg-card transition-[background-color,border-color] duration-200 hover:border-border hover:bg-accent/30"
         onClick={(e) => {
           if (isRenaming) e.preventDefault();
         }}
@@ -144,25 +141,6 @@ export default function DeckCard({
             )}
             <VisibilityBadge visibility={deck.visibility} />
           </div>
-          <div className="mt-1 flex min-w-0 items-center gap-2 text-xs text-muted-foreground">
-            <span className="shrink-0 whitespace-nowrap">
-              {deck.slides.length} slide{deck.slides.length !== 1 ? "s" : ""}
-            </span>
-            {isWorkspaceDefault && (
-              <span className="inline-flex shrink-0 items-center gap-1 rounded border border-[#609FF8]/40 px-1.5 py-0.5 text-[10px] text-[#609FF8]">
-                <IconBuildingCommunity className="h-3 w-3 shrink-0" />
-                {t("home.workspaceDefaultBadge")}
-              </span>
-            )}
-            {deck.designSystemId && (
-              <span className="inline-flex min-w-0 max-w-full items-center gap-1 rounded border border-border px-1.5 py-0.5 text-[10px] text-muted-foreground/80">
-                <IconPalette className="h-3 w-3 shrink-0 text-[#609FF8]" />
-                <span className="max-w-28 truncate">
-                  {designSystemTitle || "Design system"}
-                </span>
-              </span>
-            )}
-          </div>
         </div>
       </Link>
 

package/corpus/templates/slides/app/components/design-system/DesignSystemCard.tsx

@@ -3,8 +3,8 @@ import { ShareButton } from "@agent-native/core/client/sharing";
 import { VisibilityBadge } from "@agent-native/toolkit/sharing";
 import {
   IconBuildingCommunity,
+  IconComponents,
   IconDots,
-  IconPalette,
   IconStar,
   IconStarFilled,
   IconTrash,
@@ -190,7 +190,7 @@ export function DesignSystemCard({
       {/* Info area */}
       <div className="p-4 space-y-1.5">
         <div className="flex items-center gap-2 min-w-0">
-          <IconPalette className="w-3.5 h-3.5 text-muted-foreground shrink-0" />
+          <IconComponents className="w-3.5 h-3.5 text-muted-foreground shrink-0" />
           <h3
             className="font-medium text-sm text-foreground truncate"
             title={title}

package/corpus/templates/slides/app/components/design-system/DesignSystemSetup.tsx

@@ -8,7 +8,7 @@ import { openAgentSidebar } from "@agent-native/core/client/navigation";
 import { withBuilderUtmTrackingParams } from "@agent-native/core/shared";
 import {
   IconWorld,
-  IconPalette,
+  IconComponents,
   IconLoader2,
   IconBrandGithub,
   IconBrandFigma,
@@ -721,7 +721,7 @@ export function DesignSystemSetup({
                   </div>
 
                   <SourceAccordionRow
-                    icon={IconPalette}
+                    icon={IconComponents}
                     title={t("designSystemSetup.otherSources")}
                     description={t("designSystemSetup.otherSourcesDescription")}
                     expanded={sourcePanel === "other"}
@@ -741,7 +741,7 @@ export function DesignSystemSetup({
                       <div className="divide-y divide-border">
                         <SourceAccordionRow
                           className="rounded-none border-0"
-                          icon={IconPalette}
+                          icon={IconComponents}
                           title={t("designSystemSetup.companyBrand")}
                           description={t(
                             "designSystemSetup.companyBrandPlaceholder",
@@ -771,7 +771,7 @@ export function DesignSystemSetup({
                         {existingSystems.length > 0 && (
                           <SourceAccordionRow
                             className="rounded-none border-0"
-                            icon={IconPalette}
+                            icon={IconComponents}
                             title={t("designSystemSetup.forkExisting")}
                             description={t(
                               "designSystemSetup.customInstructionsDescription",
@@ -1064,7 +1064,7 @@ export function DesignSystemSetup({
                               }`}
                             >
                               <div className="flex items-center gap-2">
-                                <IconPalette className="w-3.5 h-3.5 text-muted-foreground" />
+                                <IconComponents className="w-3.5 h-3.5 text-muted-foreground" />
                                 <span className="text-sm text-foreground/80 truncate">
                                   {ds.title}
                                 </span>
@@ -1276,7 +1276,7 @@ function BuilderSourceStatus({ builder }: { builder: BuilderSourceDetails }) {
         ? IconBrandGithub
         : sourceKind === "code"
           ? IconFolder
-          : IconPalette;
+          : IconComponents;
   const sourceTitle =
     sourceKind === "figma"
       ? t("designSystemSetup.sourceFigma")

package/corpus/templates/slides/app/components/editor/AnimationsPanel.tsx

@@ -244,9 +244,9 @@ export function AnimationsPanel({
   );
 
   return (
-    <div className="w-60 flex flex-col h-full border-l border-border bg-background">
+    <div className="flex h-full w-60 flex-col bg-[var(--slides-editor-surface)]">
       {/* Header */}
-      <div className="flex items-center justify-between px-3 py-2.5 border-b border-border">
+      <div className="flex items-center justify-between px-3 py-2.5">
         <span className="text-xs font-medium text-foreground/90">
           {t("animations.title")}
         </span>
@@ -303,7 +303,7 @@ export function AnimationsPanel({
 
         {/* Available elements to add */}
         {availableElements.length > 0 && (
-          <div className="border-t border-border px-2 py-2">
+          <div className="px-2 py-2">
             <div className="flex items-center justify-between mb-1.5 px-1">
               <span className="text-[9px] font-medium text-muted-foreground/70 uppercase tracking-wider">
                 {t("animations.elements")}

package/corpus/templates/slides/app/components/editor/DeckEditorSkeleton.tsx

@@ -3,12 +3,12 @@ import { Skeleton } from "@/components/ui/skeleton";
 export function DeckEditorSkeleton({ label }: { label: string }) {
   return (
     <div
-      className="flex h-full min-h-0 flex-1 flex-col overflow-hidden bg-background"
+      className="deck-editor-shell flex h-full min-h-0 flex-1 flex-col overflow-hidden rounded-xl bg-background"
       role="status"
       aria-label={label}
       data-testid="deck-editor-loading"
     >
-      <div className="flex h-11 shrink-0 items-center gap-3 border-b border-border px-3">
+      <div className="flex h-11 shrink-0 items-center gap-3 px-3">
         <Skeleton className="size-8 rounded-md" />
         <Skeleton className="h-5 w-36" />
         <div className="ml-auto flex items-center gap-2">
@@ -18,7 +18,7 @@ export function DeckEditorSkeleton({ label }: { label: string }) {
         </div>
       </div>
       <div className="flex min-h-0 flex-1">
-        <div className="hidden w-56 shrink-0 flex-col gap-4 border-r border-border p-4 sm:w-64 md:flex">
+        <div className="hidden w-56 shrink-0 flex-col gap-4 p-4 sm:w-64 md:flex">
           {[0, 1, 2].map((index) => (
             <div key={index} className="flex gap-3">
               <Skeleton className="h-4 w-4 shrink-0 rounded-sm" />
@@ -26,7 +26,7 @@ export function DeckEditorSkeleton({ label }: { label: string }) {
             </div>
           ))}
         </div>
-        <div className="flex min-w-0 flex-1 items-center justify-center bg-muted/20 p-6">
+        <div className="flex min-w-0 flex-1 items-center justify-center bg-[var(--slides-editor-surface)] p-6">
           <Skeleton className="aspect-video w-full max-w-4xl rounded-md shadow-sm" />
         </div>
       </div>

package/corpus/templates/slides/app/components/editor/EditorSidebar.tsx

@@ -445,7 +445,7 @@ export default function EditorSidebar({
   }, [slides, activeSlideId, onSelectSlide]);
 
   return (
-    <div className="flex h-full min-h-0 w-48 flex-shrink-0 flex-col border-r border-border/70 bg-background/95 sm:w-52">
+    <div className="flex h-full min-h-0 w-48 flex-shrink-0 flex-col bg-background sm:w-52">
       <div
         ref={thumbScrollRef}
         className="relative min-h-0 flex-1 space-y-1 overflow-y-auto overscroll-contain p-2"