@agent-native/core 0.149.0 → 0.149.2

package/corpus/templates/slides/app/components/layout/Layout.tsx

@@ -12,13 +12,12 @@ import { useSidebarCollapsed } from "@/hooks/use-sidebar-collapsed";
 import { TAB_ID } from "@/lib/tab-id";
 import { cn } from "@/lib/utils";
 
-import { GoogleDriveConnectionCta } from "../editor/GoogleDriveConnectionCta";
 import { AgentWorkIndicator } from "./AgentWorkIndicator";
 import { Header } from "./Header";
 import {
-  getDeckChatScopeLabel,
   getEffectiveSlidesSidebarCollapsed,
   isSlidesEditorRoute,
+  shouldShowSlidesAppSidebar,
 } from "./layout-route-policy";
 import { Sidebar } from "./Sidebar";
 
@@ -50,26 +49,24 @@ export function Layout({ children }: LayoutProps) {
     useState<EditorSidebarOverride | null>(null);
   const { collapsed: sidebarCollapsed, setCollapsed: setSidebarCollapsed } =
     useSidebarCollapsed();
-  const { decks, getDeck, loading: decksLoading } = useDecks();
+  const { decks, loading: decksLoading } = useDecks();
   const isEmptyDecksState =
     location.pathname === "/" && !decksLoading && decks.length === 0;
 
   // Scope new chats to the deck the user is currently editing. The route
   // is `/deck/:id`; everywhere else (list, presentation) leaves
-  // scope null so chats stay in the general pool. Falling back to the
-  // raw deck id keeps the chat bound even before the deck object has
-  // streamed in — once the title arrives the badge updates in place.
+  // scope null so chats stay in the general pool. Keep the visible label
+  // semantic so an imported deck id never leaks into the composer chip.
   const deckScope = useMemo(() => {
     const match = location.pathname.match(/^\/deck\/([^/]+)/);
     const deckId = match?.[1];
     if (!deckId) return null;
-    const deck = getDeck(deckId);
     return {
       type: "deck" as const,
       id: deckId,
-      label: getDeckChatScopeLabel(deck?.title, t("agent.thisDeck")),
+      label: t("agent.thisDeck"),
     };
-  }, [location.pathname, getDeck, t]);
+  }, [location.pathname, t]);
 
   useEffect(() => {
     setSidebarOpen(false);
@@ -84,6 +81,7 @@ export function Layout({ children }: LayoutProps) {
   }, []);
 
   const ownToolbar = pageHasOwnToolbar(location.pathname);
+  const showAppSidebar = shouldShowSlidesAppSidebar(location.pathname);
   const editorSidebarOverrideForLocation =
     editorSidebarOverride?.locationKey === location.key
       ? editorSidebarOverride.collapsed
@@ -120,35 +118,36 @@ export function Layout({ children }: LayoutProps) {
         agentPageHref="/settings/agent"
         suppressFirstRunOnboarding={isSlidesEditorRoute(location.pathname)}
         composerSlot={<CreativeContextComposerChip />}
-        threadFooterSlot={<GoogleDriveConnectionCta />}
       >
         <div className="agent-layout-shell flex h-screen w-full overflow-hidden bg-background text-foreground">
-          {sidebarOpen && (
-            <div
-              className="fixed inset-0 z-40 bg-black/50 md:hidden"
-              onClick={() => setSidebarOpen(false)}
-            />
-          )}
-          {!isEmptyDecksState && (
-            <div
-              className={cn(
-                "agent-layout-left-drawer fixed inset-y-0 start-0 z-50 transition-transform duration-200 ease-out md:static md:z-auto md:transition-none",
-                sidebarOpen
-                  ? "translate-x-0"
-                  : "-translate-x-full rtl:translate-x-full md:translate-x-0 md:rtl:translate-x-0",
+          {!isEmptyDecksState && showAppSidebar && (
+            <>
+              {sidebarOpen && (
+                <div
+                  className="fixed inset-0 z-40 bg-foreground/50 md:hidden"
+                  onClick={() => setSidebarOpen(false)}
+                />
               )}
-            >
-              <Sidebar
-                collapsed={effectiveSidebarCollapsed && !sidebarOpen}
-                // In the mobile drawer the sidebar is forced expanded, so the
-                // desktop collapse toggle would be a silent no-op (worse: it'd
-                // mutate the desktop preference). Hide it while the drawer is
-                // open.
-                onToggleCollapsed={
-                  sidebarOpen ? undefined : toggleSidebarCollapsed
-                }
-              />
-            </div>
+              <div
+                className={cn(
+                  "agent-layout-left-drawer fixed inset-y-0 start-0 z-50 transition-transform duration-200 ease-out md:static md:z-auto md:transition-none",
+                  sidebarOpen
+                    ? "translate-x-0"
+                    : "-translate-x-full rtl:translate-x-full md:translate-x-0 md:rtl:translate-x-0",
+                )}
+              >
+                <Sidebar
+                  collapsed={effectiveSidebarCollapsed && !sidebarOpen}
+                  // In the mobile drawer the sidebar is forced expanded, so the
+                  // desktop collapse toggle would be a silent no-op (worse: it'd
+                  // mutate the desktop preference). Hide it while the drawer is
+                  // open.
+                  onToggleCollapsed={
+                    sidebarOpen ? undefined : toggleSidebarCollapsed
+                  }
+                />
+              </div>
+            </>
           )}
           <div className="agent-layout-main-surface flex h-full min-w-0 flex-1 flex-col overflow-hidden">
             {/* Mobile-only nav strip with hamburger — only when there's no page toolbar */}

package/corpus/templates/slides/app/components/layout/layout-route-policy.ts

@@ -2,11 +2,8 @@ export function isSlidesEditorRoute(pathname: string): boolean {
   return /^\/deck\/[^/]+\/?$/.test(pathname);
 }
 
-export function getDeckChatScopeLabel(
-  deckTitle: string | undefined,
-  untitledDeckLabel: string,
-): string {
-  return deckTitle === "Untitled Deck" ? untitledDeckLabel : deckTitle || "";
+export function shouldShowSlidesAppSidebar(pathname: string): boolean {
+  return !isSlidesEditorRoute(pathname);
 }
 
 export function getEffectiveSlidesSidebarCollapsed({

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.`,