@agent-native/core 0.149.1 → 0.149.2

package/corpus/templates/slides/app/components/onboarding/FirstDeckOnboardingFlow.tsx

@@ -26,6 +26,7 @@ import { useDesignSystems } from "@/hooks/use-design-systems";
 import { useWorkspaceDefaults } from "@/hooks/use-workspace-defaults";
 import { startDeckGeneration } from "@/lib/create-deck-generation";
 import {
+  forgetRecentReference,
   readRecentReferences,
   rememberRecentReference,
   type RecentReference,
@@ -45,11 +46,8 @@ export function FirstDeckOnboardingFlow({
   const { session } = useSession();
   const { decks, createDeck, ensureDeckPersisted, deleteDeck, reloadDecks } =
     useDecks();
-  const { designSystems, defaultSystem } = useDesignSystems();
-  const {
-    referenceDeck: workspaceReferenceDeck,
-    designSystem: workspaceDesignSystem,
-  } = useWorkspaceDefaults();
+  const { designSystems } = useDesignSystems();
+  const { designSystem: workspaceDesignSystem } = useWorkspaceDefaults();
   const { submit: agentSubmit } = useAgentGenerating();
   const [step, setStep] = useState<FirstDeckStep>("prompt");
   const [prompt, setPrompt] = useState("");
@@ -63,19 +61,25 @@ export function FirstDeckOnboardingFlow({
   );
 
   const initialPrompt = searchParams.get("initialPrompt")?.trim() ?? "";
-  const workspaceReferenceDeckId =
-    workspaceReferenceDeck && !workspaceReferenceDeck.unavailable
-      ? workspaceReferenceDeck.id
-      : null;
   const workspaceDesignSystemId =
     workspaceDesignSystem && !workspaceDesignSystem.unavailable
       ? workspaceDesignSystem.id
       : null;
+  const lastUsedDesignSystemId =
+    recentReferences.find(
+      (reference) =>
+        reference.kind === "design-system" &&
+        designSystems.some((designSystem) => designSystem.id === reference.id),
+    )?.id ?? null;
+  const lastUsedReferenceDeckId =
+    recentReferences.find(
+      (reference) =>
+        reference.kind === "deck" &&
+        decks.some((deck) => deck.id === reference.id),
+    )?.id ?? null;
   const initialDesignSystemId =
-    designSystems.find((designSystem) => designSystem.isDefault)?.id ??
-    workspaceDesignSystemId ??
-    defaultSystem?.id ??
-    null;
+    lastUsedDesignSystemId ?? workspaceDesignSystemId;
+  const initialReferenceDeckId = lastUsedReferenceDeckId;
 
   useEffect(() => {
     const result = readRecentReferences();
@@ -103,6 +107,10 @@ export function FirstDeckOnboardingFlow({
     },
     [],
   );
+  const forgetReference = useCallback((kind: RecentReference["kind"]) => {
+    const result = forgetRecentReference(kind);
+    if (result.readable) setRecentReferences(result.items);
+  }, []);
 
   const handlePromptSubmit = useCallback(
     async (text: string, files: File[]) => {
@@ -137,7 +145,7 @@ export function FirstDeckOnboardingFlow({
         files,
         referenceSelection: selection,
         selectedDesignSystemId: initialDesignSystemId,
-        selectedReferenceDeckId: workspaceReferenceDeckId,
+        selectedReferenceDeckId: initialReferenceDeckId,
         designSystems,
         createDeck,
         ensureDeckPersisted,
@@ -185,24 +193,32 @@ export function FirstDeckOnboardingFlow({
       prompt,
       session,
       t,
-      workspaceReferenceDeckId,
+      initialReferenceDeckId,
     ],
   );
 
   const handleReferenceSelect = useCallback(
     async (selection: NewDeckReferenceSelection) => {
-      if (selection.designSystemId) {
-        rememberReference({
-          id: selection.designSystemId,
-          kind: "design-system",
-        });
+      if (selection.designSystemId !== undefined) {
+        if (selection.designSystemId) {
+          rememberReference({
+            id: selection.designSystemId,
+            kind: "design-system",
+          });
+        } else {
+          forgetReference("design-system");
+        }
       }
-      if (selection.referenceDeckId) {
-        rememberReference({ id: selection.referenceDeckId, kind: "deck" });
+      if (selection.referenceDeckId !== undefined) {
+        if (selection.referenceDeckId) {
+          rememberReference({ id: selection.referenceDeckId, kind: "deck" });
+        } else {
+          forgetReference("deck");
+        }
       }
       await startGeneration(promptFiles, selection);
     },
-    [promptFiles, rememberReference, startGeneration],
+    [forgetReference, promptFiles, rememberReference, startGeneration],
   );
 
   const handleReferenceImport = useCallback(
@@ -296,7 +312,6 @@ export function FirstDeckOnboardingFlow({
         setPromptFiles((current) => [...current, ...generationFiles]);
         if (importedReference) {
           await reloadDecks();
-          rememberReference({ id: importedReference.id, kind: "deck" });
         }
         return importedReference;
       } catch (error) {
@@ -311,15 +326,7 @@ export function FirstDeckOnboardingFlow({
         setReferenceImporting(false);
       }
     },
-    [
-      callAction,
-      createDeck,
-      deleteDeck,
-      ensureDeckPersisted,
-      reloadDecks,
-      rememberReference,
-      t,
-    ],
+    [callAction, createDeck, deleteDeck, ensureDeckPersisted, reloadDecks, t],
   );
 
   const handleReferenceSourceImport = useCallback(
@@ -357,7 +364,6 @@ export function FirstDeckOnboardingFlow({
           source: "google-slides",
         };
         await reloadDecks();
-        rememberReference({ id: importedReference.id, kind: "deck" });
         return importedReference;
       } catch (error) {
         toast.error(t("editorToolbar.uploadFailed"), {
@@ -371,15 +377,17 @@ export function FirstDeckOnboardingFlow({
         setReferenceImporting(false);
       }
     },
-    [callAction, reloadDecks, rememberReference, t],
+    [callAction, reloadDecks, t],
   );
 
   const handleReferenceSkip = useCallback(() => {
+    forgetReference("design-system");
+    forgetReference("deck");
     void startGeneration(promptFiles, {
       designSystemId: null,
       referenceDeckId: null,
     });
-  }, [promptFiles, startGeneration]);
+  }, [forgetReference, promptFiles, startGeneration]);
 
   if (step === "references") {
     return (
@@ -388,8 +396,7 @@ export function FirstDeckOnboardingFlow({
         decks={decks}
         designSystems={designSystems}
         defaultDesignSystemId={initialDesignSystemId}
-        defaultReferenceDeckId={workspaceReferenceDeckId}
-        recentReferences={recentReferences}
+        defaultReferenceDeckId={initialReferenceDeckId}
         onSelect={handleReferenceSelect}
         onImport={handleReferenceImport}
         onImportSource={handleReferenceSourceImport}
@@ -404,8 +411,6 @@ export function FirstDeckOnboardingFlow({
         chooseDeckLabel={t("home.referenceDeckPlaceholder")}
         importingLabel={t("raw.uploading")}
         skipLabel={t("home.referenceDeckNone")}
-        starredLabel={t("home.referenceDeckStarredGroup")}
-        otherDecksLabel={t("home.referenceDeckOtherGroup")}
         searchDecksLabel={t("root.searchDecks")}
         promptSummary={prompt}
       />

package/corpus/templates/slides/app/components/presentation/PresentationView.tsx

@@ -19,7 +19,7 @@ import type { AspectRatio } from "@/lib/aspect-ratios";
 import {
   findLegacyAnimationContainer,
   getElementPath,
-  resolveSlideAnimationElement,
+  resolveSlideAnimationTargets,
 } from "@/lib/slide-animation-elements";
 
 import type { DesignSystemData } from "../../../shared/api";
@@ -40,7 +40,17 @@ interface PresentationViewProps {
  * Uses slide.animations if defined, falls back to splitByParagraph auto-detection.
  */
 function getAnimationSteps(slide: Slide): SlideAnimation[] | null {
-  if (slide.animations && slide.animations.length > 0) return slide.animations;
+  if (slide.animations && slide.animations.length > 0) {
+    const doc = new DOMParser().parseFromString(slide.content, "text/html");
+    const root = doc.querySelector(".fmd-slide");
+    // Explicit metadata is authoritative only when every target still points
+    // at a unique element in the final HTML. Otherwise disable the reveal
+    // layer for this slide instead of counting invisible phantom steps while
+    // leaving the rest of the content visible.
+    return root && resolveSlideAnimationTargets(root, slide.animations)
+      ? slide.animations
+      : null;
+  }
   // Legacy splitByParagraph: auto-detect and create steps
   if (slide.splitByParagraph) {
     const doc = new DOMParser().parseFromString(slide.content, "text/html");
@@ -111,14 +121,16 @@ function annotateStepsForPresentation(
   const root = doc.querySelector(".fmd-slide");
   if (!root) return html;
 
-  // Annotate each step element with data-pstep
-  steps.forEach((anim, stepIdx) => {
-    const el = resolveSlideAnimationElement(root, anim);
-    if (el) el.setAttribute("data-pstep", String(stepIdx));
+  const resolvedSteps = resolveSlideAnimationTargets(root, steps);
+  if (!resolvedSteps) return html;
+
+  // Annotate each resolved step element with data-pstep.
+  resolvedSteps.forEach(({ element }, stepIdx) => {
+    element.setAttribute("data-pstep", String(stepIdx));
   });
 
-  const styleLines = steps
-    .map((anim, stepIdx) => {
+  const styleLines = resolvedSteps
+    .map(({ target }, stepIdx) => {
       if (stepIdx >= currentStep) {
         return `[data-pstep="${stepIdx}"] { opacity: 0; pointer-events: none; }`;
       } else if (stepIdx < currentStep - 1) {
@@ -126,7 +138,7 @@ function annotateStepsForPresentation(
         return `[data-pstep="${stepIdx}"] { opacity: 1; pointer-events: auto; animation: elem-appear 1ms both; }`;
       } else {
         // Newly revealed — animate with its type
-        return `[data-pstep="${stepIdx}"] { opacity: 1; pointer-events: auto; ${getElemAnimCss(anim.type)} }`;
+        return `[data-pstep="${stepIdx}"] { opacity: 1; pointer-events: auto; ${getElemAnimCss(target.type)} }`;
       }
     })
     .join("\n");
@@ -219,6 +231,10 @@ export default function PresentationView({
   const [cursorVisible, setCursorVisible] = useState(true);
   const [needsFullscreenGesture, setNeedsFullscreenGesture] = useState(false);
   const enteredFullscreenRef = useRef(false);
+  const transitionTimerRef = useRef<number | null>(null);
+  const queuedNavigationRef = useRef<"next" | "prev" | null>(null);
+  const goNextRef = useRef<() => void>(() => {});
+  const goPrevRef = useRef<() => void>(() => {});
   const navigate = useNavigate();
 
   const isShared = deckId.startsWith("__shared__/");
@@ -230,9 +246,23 @@ export default function PresentationView({
     );
   }, [clampIndex, safeSlides.length]);
 
+  const clearTransitionTimer = useCallback(() => {
+    if (transitionTimerRef.current !== null) {
+      window.clearTimeout(transitionTimerRef.current);
+      transitionTimerRef.current = null;
+    }
+  }, []);
+
+  useEffect(() => clearTransitionTimer, [clearTransitionTimer]);
+
   useEffect(() => {
+    clearTransitionTimer();
+    queuedNavigationRef.current = null;
     setCurrentIndex(clampIndex(startIndex));
-  }, [clampIndex, startIndex]);
+    setCurrentStep(0);
+    setPrevIndex(null);
+    setAnimating(false);
+  }, [clearTransitionTimer, clampIndex, startIndex]);
 
   const currentSlide = safeSlides[currentIndex];
   const animSteps = currentSlide ? getAnimationSteps(currentSlide) : null;
@@ -247,7 +277,11 @@ export default function PresentationView({
       const initialStep =
         dir === "prev" ? (incomingSteps ? incomingSteps.length : 0) : 0;
 
+      clearTransitionTimer();
+      queuedNavigationRef.current = null;
       if (isInstant(t)) {
+        setPrevIndex(null);
+        setAnimating(false);
         setCurrentIndex(newIndex);
         setCurrentStep(initialStep);
         return;
@@ -259,16 +293,28 @@ export default function PresentationView({
       setCurrentIndex(newIndex);
       setCurrentStep(initialStep);
 
-      setTimeout(() => {
+      transitionTimerRef.current = window.setTimeout(() => {
+        transitionTimerRef.current = null;
         setPrevIndex(null);
         setAnimating(false);
+        const queued = queuedNavigationRef.current;
+        queuedNavigationRef.current = null;
+        if (queued) {
+          window.setTimeout(() => {
+            if (queued === "next") goNextRef.current();
+            else goPrevRef.current();
+          }, 0);
+        }
       }, 400);
     },
-    [currentIndex, safeSlides],
+    [clearTransitionTimer, currentIndex, safeSlides],
   );
 
   const goNext = useCallback(() => {
-    if (animating) return;
+    if (animating) {
+      queuedNavigationRef.current = "next";
+      return;
+    }
     // Reveal next paragraph step if enabled
     if (maxSteps > 0 && currentStep < maxSteps /* i18n-ignore */) {
       setCurrentStep((prev) => prev + 1);
@@ -286,7 +332,10 @@ export default function PresentationView({
   ]);
 
   const goPrev = useCallback(() => {
-    if (animating) return;
+    if (animating) {
+      queuedNavigationRef.current = "prev";
+      return;
+    }
     if (currentIndex <= 0) return;
     startTransition(currentIndex - 1, "prev");
   }, [animating, currentIndex, startTransition]);
@@ -307,8 +356,6 @@ export default function PresentationView({
   // commands and mirrors whatever we echo back — so build steps stay
   // authoritative here.
   const channelRef = useRef<BroadcastChannel | null>(null);
-  const goNextRef = useRef(goNext);
-  const goPrevRef = useRef(goPrev);
   goNextRef.current = goNext;
   goPrevRef.current = goPrev;
 

package/corpus/templates/slides/app/hooks/use-generating-slide-preview.ts

@@ -21,6 +21,11 @@ export function useGeneratingSlidePreview({
   const [content, setContent] = useState<string | null>(null);
   const activeCallIdRef = useRef<string | null>(null);
   const previousSlideCountRef = useRef(slideCount);
+  // Latest parsed value awaiting a flush; setContent fires at most once per frame
+  // and always reads this ref, so a burst of deltas within one frame collapses
+  // to a single render carrying the newest value instead of the stale one.
+  const pendingContentRef = useRef<string | null>(null);
+  const rafIdRef = useRef<number | null>(null);
 
   useEffect(() => {
     if (slideCount !== previousSlideCountRef.current) {
@@ -37,6 +42,18 @@ export function useGeneratingSlidePreview({
       return;
     }
 
+    const flush = () => {
+      rafIdRef.current = null;
+      setContent(pendingContentRef.current);
+    };
+
+    const scheduleFlush = (value: string | null) => {
+      pendingContentRef.current = value;
+      if (rafIdRef.current === null) {
+        rafIdRef.current = requestAnimationFrame(flush);
+      }
+    };
+
     const handleToolInput = (event: Event) => {
       const detail = (event as CustomEvent<ToolInputEventDetail>).detail;
       if (detail?.tool !== "add-slide") return;
@@ -47,7 +64,7 @@ export function useGeneratingSlidePreview({
       if (detail.phase === "start") {
         if (detail.id && activeCallIdRef.current === detail.id) return;
         activeCallIdRef.current = detail.id ?? null;
-        setContent(null);
+        scheduleFlush(null);
         return;
       }
 
@@ -60,13 +77,18 @@ export function useGeneratingSlidePreview({
       }
       if (detail.id) activeCallIdRef.current = detail.id;
       if (parsed.content !== undefined) {
-        setContent(parsed.content || null);
+        scheduleFlush(parsed.content || null);
       }
     };
 
     window.addEventListener("agent-native:tool-input", handleToolInput);
-    return () =>
+    return () => {
       window.removeEventListener("agent-native:tool-input", handleToolInput);
+      if (rafIdRef.current !== null) {
+        cancelAnimationFrame(rafIdRef.current);
+        rafIdRef.current = null;
+      }
+    };
   }, [deckId, generating]);
 
   return content;

package/corpus/templates/slides/app/i18n/en-US.ts

@@ -689,6 +689,7 @@ const messages = {
       "Keep the default design system and reference deck unless you remove them. Search for another deck, import a file, or continue when you're ready.",
     chooseAnotherDeck: "Choose another deck",
     noMatchingDecks: "No matching decks found.",
+    googleSlidesImportLabel: "Slides",
     googleSlidesReferenceTitle: "Google Slides",
     googleSlidesReferenceConnect:
       "Connect Google Drive to import a Slides deck.",

package/corpus/templates/slides/app/lib/create-deck-generation.ts

@@ -113,7 +113,7 @@ function describeUploadedFilesForAgent(
     "",
     "File handling rules:",
     importedSourceDeck
-      ? "- The imported source deck is the canonical source. Preserve its slide count, order, IDs, factual copy, notes, imagery, charts, tables, diagrams, and freeform objects while improving styling. Use update-slide on existing slide IDs; do not rebuild it with add-slide."
+      ? "- The imported source deck is the canonical source. Preserve its slide count, order, IDs, factual copy, notes, imagery, charts, tables, diagrams, and freeform objects while improving styling. For a deck-wide restyle, use one patch-deck call with requireAllSourceSlides=true; use update-slide only for a targeted one-slide edit. Do not rebuild it with add-slide."
       : `- PPTX files: call \`import-pptx --filePath \"<path>\" --deckId ${deckId}\` before adding or editing slides.`,
     importedSourceDeck
       ? "- For a PDF source, keep the original full-page image in every slide and add restrained design-system chrome around it without obscuring source content. Never OCR-reconstruct a source-faithful page from extracted text."
@@ -324,19 +324,20 @@ export async function startDeckGeneration({
         "Source-preserving improvement mode:",
         `- The target deck already contains ${importedSourceDeck.slideCount} imported source slides. Treat those slides as the user's complete source, not as inspiration for a new deck.`,
         "- Keep the exact source slide count, order, IDs, factual meaning, notes, images, charts, tables, diagrams, and freeform objects unless the user explicitly asks to change one of them.",
-        "- Read `get-deck` before editing, load the linked design system with `get-design-system`, then use one `update-slide` call per existing slide ID for bounded visual improvements. Keep every original `<img>` source and enough original factual copy for each slide; for PDF slides, use restrained design-system chrome around the page without obscuring it.",
+        "- Read `get-deck` once before editing to obtain every existing slide ID and source HTML, load the linked design system with `get-design-system`, then make a deck-wide restyle with one `patch-deck` call using `requireAllSourceSlides: true` and one `patch-slide` operation with `fields.content` for every source slide ID. Do not split a full-deck restyle into arbitrary batches or fall back to one-by-one `update-slide` calls; use `update-slide` only for a targeted one-slide edit. Keep every original `<img>` source and enough original factual copy for each slide; for PDF slides, use restrained design-system chrome around the page without obscuring it.",
         "- Do not call `add-slide`, delete slides, reorder slides, or replace source images with generic cards. Do not claim success until `get-deck` verifies the same slide IDs and count after the edits.",
+        '- After the patch succeeds, verify with `get-deck` using `compact: "true"` so only slide IDs, count, and previews are returned. Do not report an initial or partial pass, and do not leave any source slides for a later run.',
         "- If `get-deck` reports partial source fidelity or skipped images, stop and report the exact warning instead of claiming a reliable restyle.",
       ].join("\n")
     : "";
   const sourceModeInstructions = importedSourceDeck
     ? [
         "The request is an in-place visual improvement of an imported source deck. Make a coherent style pass across every existing slide while preserving all source content and media.",
-        "Do not use the new-deck add-slide workflow for this source-preserving request.",
+        "Do not use the new-deck add-slide workflow for this source-preserving request. Finish every source slide in this run; if patch-deck rejects incomplete coverage, continue with the returned missing IDs instead of reporting success with a partial deck.",
       ]
     : [
         "Start a `manage-progress` run so progress appears in the app header. Add the first slide as soon as it is ready, then continue one slide at a time so the editor visibly fills in.",
-        `After reading any requested or imported source material, but before adding the first slide, choose a concise, specific deck title from the user's request and source material. Call \`patch-deck\` with \`deckId: \"${deckId}\"\` and \`operations: [{ \"op\": \"patch-deck-fields\", \"fields\": { \"title\": \"<generated title>\" } }]\`. Include only \`title\` in \`fields\`; omit all other optional fields. Never leave a generated deck named \"Untitled Deck\" or another placeholder.`,
+        `After reading any requested or imported source material, but before adding the first slide, choose a concise, specific deck title from the user's request and source material. Never use the deck id, run id, file id, or another opaque alphanumeric token as the title. Call \`patch-deck\` with \`deckId: \"${deckId}\"\` and \`operations: [{ \"op\": \"patch-deck-fields\", \"fields\": { \"title\": \"<generated title>\" } }]\`. Include only \`title\` in \`fields\`; omit all other optional fields. Never leave a generated deck named \"Untitled Deck\" or another placeholder.`,
         "If the user asks for a standalone visual, diagram, hero, one-pager, poster, or a couple of visuals, create only the requested one/few polished visual slides. Do not pad the result into a full presentation.",
         "If the request is for a presentation or deck and does not explicitly ask for one slide, infer a coherent multi-slide outline from the scope and keep adding slides until that outline is complete. Do not stop after the first slide just because the prompt has few explicit instructions.",
         `Add slides ONE AT A TIME using the \`add-slide\` action with --deckId=${deckId}. Wait for each \`add-slide\` result before calling it again; do not batch or parallelize slide writes.`,

package/corpus/templates/slides/app/lib/recent-references.ts

@@ -48,7 +48,10 @@ export function readRecentReferences(
     const parsed: unknown = JSON.parse(raw);
     if (!Array.isArray(parsed)) return { items: [], readable: false };
     return {
-      items: parsed.filter(isRecentReference).slice(0, MAX_RECENT_REFERENCES),
+      items: parsed
+        .filter(isRecentReference)
+        .sort((first, second) => second.lastUsedAt - first.lastUsedAt)
+        .slice(0, MAX_RECENT_REFERENCES),
       readable: true,
     };
   } catch {
@@ -56,6 +59,25 @@ export function readRecentReferences(
   }
 }
 
+export function forgetRecentReference(
+  kind: RecentReferenceKind,
+  storage:
+    | Pick<Storage, "getItem" | "setItem">
+    | null
+    | undefined = typeof window === "undefined" ? null : window.localStorage,
+): RecentReferencesResult {
+  const current = readRecentReferences(storage);
+  if (!storage || !current.readable) return current;
+
+  const next = current.items.filter((item) => item.kind !== kind);
+  try {
+    storage.setItem(RECENT_REFERENCES_STORAGE_KEY, JSON.stringify(next));
+    return { items: next, readable: true };
+  } catch {
+    return { items: current.items, readable: false };
+  }
+}
+
 export function rememberRecentReference(
   reference: Omit<RecentReference, "lastUsedAt">,
   storage:

package/corpus/templates/slides/app/lib/slide-animation-elements.ts

@@ -9,6 +9,14 @@ export interface AnimationTarget {
   elementPath?: number[];
 }
 
+export interface ResolvedAnimationTarget<
+  T extends AnimationTarget = AnimationTarget,
+> {
+  target: T;
+  element: Element;
+  key: string;
+}
+
 const INLINE_TAGS = new Set([
   "a",
   "abbr",
@@ -192,15 +200,45 @@ export function resolveSlideAnimationElement(
   root: Element,
   target: AnimationTarget,
 ): Element | null {
-  if (Array.isArray(target.elementPath) && target.elementPath.length > 0) {
-    const pathTarget = resolveElementPath(root, target.elementPath);
-    if (pathTarget) return pathTarget;
+  if (Array.isArray(target.elementPath)) {
+    // A supplied path is the identity of the target. Falling back to a
+    // legacy index after it goes stale can animate a different element while
+    // leaving the intended element visible.
+    return target.elementPath.length > 0
+      ? resolveElementPath(root, target.elementPath)
+      : null;
   }
 
   const legacyContainer = findLegacyAnimationContainer(root);
   return legacyContainer?.children.item(target.elementIndex) ?? null;
 }
 
+/**
+ * Resolve an ordered animation list as one validated unit. A null result
+ * means the list cannot be rendered faithfully - a missing or duplicate
+ * target must not become a phantom click step in the presentation player.
+ */
+export function resolveSlideAnimationTargets<T extends AnimationTarget>(
+  root: Element,
+  targets: readonly T[],
+): ResolvedAnimationTarget<T>[] | null {
+  const seen = new Set<string>();
+  const resolved: ResolvedAnimationTarget<T>[] = [];
+
+  for (const target of targets) {
+    const element = resolveSlideAnimationElement(root, target);
+    if (!element) return null;
+    const path = getElementPath(root, element);
+    if (!path) return null;
+    const key = animationElementKey(path);
+    if (seen.has(key)) return null;
+    seen.add(key);
+    resolved.push({ target, element, key });
+  }
+
+  return resolved;
+}
+
 export function getSlideAnimationTargetKey(
   html: string,
   target: AnimationTarget,

package/corpus/templates/slides/app/pages/DeckEditor.tsx

@@ -5,7 +5,7 @@ import {
   emailToColor,
   emailToName,
 } from "@agent-native/core/client/collab";
-import { useSession, callAction } from "@agent-native/core/client/hooks";
+import { useSession } from "@agent-native/core/client/hooks";
 import { useT } from "@agent-native/core/client/i18n";
 import { useOrg } from "@agent-native/core/client/org";
 import {
@@ -54,7 +54,7 @@ import {
   useSlideComments,
   type CommentThread,
 } from "@/hooks/use-slide-comments";
-import { getAspectRatioDims, type AspectRatio } from "@/lib/aspect-ratios";
+import { getAspectRatioDims } from "@/lib/aspect-ratios";
 import {
   deckAccessCheckKey,
   shouldShowDeckEditorSkeleton,
@@ -238,9 +238,6 @@ export default function DeckEditor() {
   const [pendingComment, setPendingComment] = useState<{
     quotedText: string;
   } | null>(null);
-  const imageGenButtonRef = useRef<HTMLButtonElement>(null);
-  const assetsButtonRef = useRef<HTMLButtonElement>(null);
-
   // Track which image src to replace
   const [replaceImageSrc, setReplaceImageSrc] = useState<string | null>(null);
 
@@ -1035,16 +1032,9 @@ export default function DeckEditor() {
           setReplaceImageSrc(null);
           setAssetLibraryOpen(true);
         }}
-        imageGenButtonRef={imageGenButtonRef}
-        assetsButtonRef={assetsButtonRef}
-        historyOpen={historyOpen}
         onShowHistory={() => setHistoryOpen(!historyOpen)}
         historyButtonRef={historyButtonRef}
         currentSlide={currentSlide}
-        onUpdateSlide={(updates, slideIdOverride) => {
-          const targetId = slideIdOverride ?? currentSlide?.id;
-          if (targetId) updateSlide(id, targetId, updates);
-        }}
         activeUsers={slideActiveUsers.filter((u) => u.email !== session?.email)}
         agentPresent={agentPresent}
         agentActive={agentActive}
@@ -1118,19 +1108,6 @@ export default function DeckEditor() {
           }
           return exportDeckToGoogleSlides(deck.title, slides, deck.aspectRatio);
         }}
-        aspectRatio={deck.aspectRatio}
-        onSetAspectRatio={(ratio: AspectRatio) => {
-          const previous = deck.aspectRatio;
-          // Optimistic UI: update local cache immediately so canvas resizes.
-          updateDeck(id, { aspectRatio: ratio });
-          callAction("update-deck-aspect-ratio", {
-            deckId: id,
-            aspectRatio: ratio,
-          }).catch((err) => {
-            console.error("Failed to set aspect ratio:", err);
-            updateDeck(id, { aspectRatio: previous });
-          });
-        }}
         currentSlideId={currentSlide?.id}
         addSlideGenerating={addSlideGenerating}
         onAddSlideGeneratingChange={setAddSlideGenerating}
@@ -1376,7 +1353,7 @@ export default function DeckEditor() {
       <ImageGenPanel
         open={imageGenOpen}
         onOpenChange={setImageGenOpen}
-        anchorRef={imageGenButtonRef}
+        anchorRef={historyButtonRef}
         referenceImageUrls={imageStyleReferenceUrls}
         slideContext={
           currentSlide
@@ -1394,7 +1371,7 @@ export default function DeckEditor() {
       <AssetLibraryPanel
         open={assetLibraryOpen}
         onOpenChange={setAssetLibraryOpen}
-        anchorRef={assetsButtonRef}
+        anchorRef={historyButtonRef}
         onSelectAsset={
           replaceImageSrc
             ? (newUrl) => {