create-nextblock 0.15.10 → 0.16.2

package/templates/nextblock-template/app/cms/blocks/components/BlockEditorArea.tsx

@@ -10,6 +10,7 @@ import { BlockType } from '../../../../lib/blocks/blockRegistry';
 
 type Block = Database["public"]["Tables"]["blocks"]["Row"];
 import { blockHasEditableContent, getBlockDefinition, SectionBlockContent } from '../../../../lib/blocks/blockRegistry';
+import { usePageSeo } from '../../../../lib/seo/page-audit-context';
 import { Button } from "@nextblock-cms/ui";
 import { PlusCircle } from "lucide-react";
 import {
@@ -125,6 +126,32 @@ export default function BlockEditorArea({ parentId, parentType, initialBlocks, l
     lastSavedBlocks.current = sortedBlocks;
   }, [initialBlocks]);
 
+  /**
+   * Publish the live block list to the page-level SEO audit.
+   *
+   * This component is the only client component that holds every block on the
+   * page, which makes it the only place the page-wide checks — one H1, the total
+   * word count, alt text across the whole document — can be fed from. It
+   * deliberately does no analysis of its own: the panel debounces and grades,
+   * this only hands over state.
+   *
+   * Publishing is kept out of the save path on purpose. `debouncedSave` builds a
+   * fresh `debounce` on every call, so it never actually coalesces and already
+   * fires once per keystroke; hanging document analysis off that would multiply
+   * an existing problem rather than inherit a solution. Copying a reference into
+   * a context, by contrast, costs nothing per keystroke, and the provider's own
+   * setter bails when the array is unchanged.
+   *
+   * `usePageSeo()` returns null on any surface with no page-level panel above —
+   * the product editor is the live example — so the publish is optional-chained
+   * rather than guarded by a second code path.
+   */
+  const pageSeo = usePageSeo();
+  const publishBlocksToPageSeo = pageSeo?.setBlocks;
+  useEffect(() => {
+    publishBlocksToPageSeo?.(blocks);
+  }, [blocks, publishBlocksToPageSeo]);
+
   const saveBlock = useCallback(async (blockToSave: Block) => {
     const result = await updateBlock(
       blockToSave.id,

package/templates/nextblock-template/app/cms/blocks/components/BlockEditorModal.tsx

@@ -1,5 +1,5 @@
-import { useState, useEffect, type ComponentType, Suspense, LazyExoticComponent, useCallback, type CSSProperties } from "react";
-import { cn } from "@nextblock-cms/utils";
+import { useState, useEffect, type ComponentType, Suspense, LazyExoticComponent, useCallback, type CSSProperties } from "react";
+import { cn } from "@nextblock-cms/utils";
 import {
   Dialog,
   DialogContent,
@@ -9,7 +9,7 @@ import {
   DialogDescription,
 } from "@nextblock-cms/ui";
 import { Button } from "@nextblock-cms/ui";
-import { blockRegistry, BlockType } from '../../../../lib/blocks/blockRegistry';
+import { blockRegistry, BlockType } from '../../../../lib/blocks/blockRegistry';
 
 // A generic representation of a block object.
 // The modal primarily needs `type` to get the label and `content` for editing.
@@ -20,171 +20,171 @@ export type Block<T = unknown> = {
 };
 
 // Props that every block editor component must accept.
-export type BlockEditorProps<T = unknown> = {
-  block: Block<T>;
-  content: T;
-  onChange: (newContent: T) => void;
-  className?: string; // Added for editor component styling
-  sectionBackground?: import("@/lib/blocks/blockRegistry").SectionBlockContent['background'];
-};
-
-export type EditorSurfaceContext = {
-  isDark: boolean;
-  style: CSSProperties;
-};
-
-export type BlockEditorSaveMode = "manual" | "autosave";
-export type BlockEditorSaveStatus = "idle" | "dirty" | "saving" | "saved" | "error";
-
-type BlockEditorModalProps = {
-  block: Block;
-  isOpen: boolean;
-  onClose: () => void;
-  onSave: (updatedContent: unknown) => void;
-  EditorComponent: LazyExoticComponent<ComponentType<BlockEditorProps<unknown>>> | ComponentType<BlockEditorProps<unknown>>;
-  sectionBackground?: import("@/lib/blocks/blockRegistry").SectionBlockContent['background'];
-  editorSurfaceContext?: EditorSurfaceContext | null;
-  titleOverride?: string;
-  useContextualSurface?: boolean;
-  saveMode?: BlockEditorSaveMode;
-  saveStatus?: BlockEditorSaveStatus;
-  saveStatusText?: string;
-  onAutoChange?: (content: unknown) => void;
-  onFlushBeforeClose?: () => Promise<boolean>;
-};
-
-const autosaveStatusLabels: Record<BlockEditorSaveStatus, string> = {
-  idle: "Saved",
-  dirty: "Unsaved",
-  saving: "Saving...",
-  saved: "Saved",
-  error: "Save failed",
-};
-
-export function BlockEditorModal({
-  block,
-  isOpen,
-  onClose,
-  onSave,
-  EditorComponent,
-  sectionBackground,
-  editorSurfaceContext,
-  titleOverride,
-  useContextualSurface,
-  saveMode = "manual",
-  saveStatus = "idle",
-  saveStatusText,
-  onAutoChange,
-  onFlushBeforeClose,
-}: BlockEditorModalProps) {
-  const [tempContent, setTempContent] = useState(block.content);
-  const [showConfirmClose, setShowConfirmClose] = useState(false);
-  const [isFlushingBeforeClose, setIsFlushingBeforeClose] = useState(false);
-  const isValid = true; // Placeholder for future validation logic
-  const isAutosaveMode = saveMode === "autosave";
-  const [saveShortcutLabel, setSaveShortcutLabel] = useState("CMD+S");
-
-  useEffect(() => {
-    if (typeof window !== "undefined") {
-      const isMac = navigator.userAgent.indexOf("Mac") !== -1;
-      setSaveShortcutLabel(isMac ? "CMD+S" : "Ctrl+S");
-    }
-  }, []);
-
-  useEffect(() => {
-    // When the modal is opened with a new block, reset the temp content
-    if (isOpen) {
-      setTempContent(block.content);
-      setShowConfirmClose(false);
-      setIsFlushingBeforeClose(false);
-    }
-  }, [isOpen, block.content]);
-
-  const handleSave = useCallback(() => {
-    onSave(tempContent);
-  }, [onSave, tempContent]);
-
-  const hasUnsavedChanges = useCallback(() => {
-    return JSON.stringify(block.content) !== JSON.stringify(tempContent);
-  }, [block.content, tempContent]);
-
-  const flushAutosave = useCallback(async () => {
-    if (!onFlushBeforeClose) {
-      return true;
-    }
-
-    setIsFlushingBeforeClose(true);
-    try {
-      return await onFlushBeforeClose();
-    } finally {
-      setIsFlushingBeforeClose(false);
-    }
-  }, [onFlushBeforeClose]);
-
-  const handleCloseAttempt = useCallback(() => {
-    if (isAutosaveMode) {
-      void (async () => {
-        const didFlush = await flushAutosave();
-        if (didFlush) {
-          onClose();
-        }
-      })();
-      return;
-    }
-
-    if (hasUnsavedChanges()) {
-      setShowConfirmClose(true);
-    } else {
-      onClose();
-    }
-  }, [flushAutosave, hasUnsavedChanges, isAutosaveMode, onClose]);
-
-  useEffect(() => {
-    if (!isOpen) {
-      return;
-    }
-
-    const handleKeyDown = (event: KeyboardEvent) => {
-      if (!(event.metaKey || event.ctrlKey) || event.key.toLowerCase() !== "s") {
-        return;
-      }
-
-      event.preventDefault();
-      if (isAutosaveMode) {
-        handleCloseAttempt();
-      } else {
-        handleSave();
-      }
-    };
-
-    window.addEventListener("keydown", handleKeyDown, true);
-    return () => {
-      window.removeEventListener("keydown", handleKeyDown, true);
-    };
-  }, [handleCloseAttempt, handleSave, isAutosaveMode, isOpen]);
-
-  const handleContentChange = (newContent: unknown) => {
-    setTempContent(newContent);
-    if (isAutosaveMode) {
-      onAutoChange?.(newContent);
-    }
-    // Potentially add validation here and set isValid
-  };
-
-  const blockInfo = blockRegistry[block.type];
-  const displayText = blockInfo?.label || "Block";
-  const shouldUseContextualSurface =
-    (useContextualSurface ?? true) && block.type === 'text';
-  const contextualSurfaceStyle = shouldUseContextualSurface ? editorSurfaceContext?.style : undefined;
-  const editorClassName = cn(
-    "bg-transparent text-foreground border-none shadow-none focus-within:ring-0 min-h-[60vh]",
-    shouldUseContextualSurface &&
-      editorSurfaceContext?.isDark &&
-      "[&_.ProseMirror]:text-white [&_.ProseMirror.prose]:prose-invert"
-  );
-  const resolvedSaveStatusText = saveStatusText ?? autosaveStatusLabels[saveStatus];
-
-  return (
+export type BlockEditorProps<T = unknown> = {
+  block: Block<T>;
+  content: T;
+  onChange: (newContent: T) => void;
+  className?: string; // Added for editor component styling
+  sectionBackground?: import("@/lib/blocks/blockRegistry").SectionBlockContent['background'];
+};
+
+export type EditorSurfaceContext = {
+  isDark: boolean;
+  style: CSSProperties;
+};
+
+export type BlockEditorSaveMode = "manual" | "autosave";
+export type BlockEditorSaveStatus = "idle" | "dirty" | "saving" | "saved" | "error";
+
+type BlockEditorModalProps = {
+  block: Block;
+  isOpen: boolean;
+  onClose: () => void;
+  onSave: (updatedContent: unknown) => void;
+  EditorComponent: LazyExoticComponent<ComponentType<BlockEditorProps<unknown>>> | ComponentType<BlockEditorProps<unknown>>;
+  sectionBackground?: import("@/lib/blocks/blockRegistry").SectionBlockContent['background'];
+  editorSurfaceContext?: EditorSurfaceContext | null;
+  titleOverride?: string;
+  useContextualSurface?: boolean;
+  saveMode?: BlockEditorSaveMode;
+  saveStatus?: BlockEditorSaveStatus;
+  saveStatusText?: string;
+  onAutoChange?: (content: unknown) => void;
+  onFlushBeforeClose?: () => Promise<boolean>;
+};
+
+const autosaveStatusLabels: Record<BlockEditorSaveStatus, string> = {
+  idle: "Saved",
+  dirty: "Unsaved",
+  saving: "Saving...",
+  saved: "Saved",
+  error: "Save failed",
+};
+
+export function BlockEditorModal({
+  block,
+  isOpen,
+  onClose,
+  onSave,
+  EditorComponent,
+  sectionBackground,
+  editorSurfaceContext,
+  titleOverride,
+  useContextualSurface,
+  saveMode = "manual",
+  saveStatus = "idle",
+  saveStatusText,
+  onAutoChange,
+  onFlushBeforeClose,
+}: BlockEditorModalProps) {
+  const [tempContent, setTempContent] = useState(block.content);
+  const [showConfirmClose, setShowConfirmClose] = useState(false);
+  const [isFlushingBeforeClose, setIsFlushingBeforeClose] = useState(false);
+  const isValid = true; // Placeholder for future validation logic
+  const isAutosaveMode = saveMode === "autosave";
+  const [saveShortcutLabel, setSaveShortcutLabel] = useState("CMD+S");
+
+  useEffect(() => {
+    if (typeof window !== "undefined") {
+      const isMac = navigator.userAgent.indexOf("Mac") !== -1;
+      setSaveShortcutLabel(isMac ? "CMD+S" : "Ctrl+S");
+    }
+  }, []);
+
+  useEffect(() => {
+    // When the modal is opened with a new block, reset the temp content
+    if (isOpen) {
+      setTempContent(block.content);
+      setShowConfirmClose(false);
+      setIsFlushingBeforeClose(false);
+    }
+  }, [isOpen, block.content]);
+
+  const handleSave = useCallback(() => {
+    onSave(tempContent);
+  }, [onSave, tempContent]);
+
+  const hasUnsavedChanges = useCallback(() => {
+    return JSON.stringify(block.content) !== JSON.stringify(tempContent);
+  }, [block.content, tempContent]);
+
+  const flushAutosave = useCallback(async () => {
+    if (!onFlushBeforeClose) {
+      return true;
+    }
+
+    setIsFlushingBeforeClose(true);
+    try {
+      return await onFlushBeforeClose();
+    } finally {
+      setIsFlushingBeforeClose(false);
+    }
+  }, [onFlushBeforeClose]);
+
+  const handleCloseAttempt = useCallback(() => {
+    if (isAutosaveMode) {
+      void (async () => {
+        const didFlush = await flushAutosave();
+        if (didFlush) {
+          onClose();
+        }
+      })();
+      return;
+    }
+
+    if (hasUnsavedChanges()) {
+      setShowConfirmClose(true);
+    } else {
+      onClose();
+    }
+  }, [flushAutosave, hasUnsavedChanges, isAutosaveMode, onClose]);
+
+  useEffect(() => {
+    if (!isOpen) {
+      return;
+    }
+
+    const handleKeyDown = (event: KeyboardEvent) => {
+      if (!(event.metaKey || event.ctrlKey) || event.key.toLowerCase() !== "s") {
+        return;
+      }
+
+      event.preventDefault();
+      if (isAutosaveMode) {
+        handleCloseAttempt();
+      } else {
+        handleSave();
+      }
+    };
+
+    window.addEventListener("keydown", handleKeyDown, true);
+    return () => {
+      window.removeEventListener("keydown", handleKeyDown, true);
+    };
+  }, [handleCloseAttempt, handleSave, isAutosaveMode, isOpen]);
+
+  const handleContentChange = (newContent: unknown) => {
+    setTempContent(newContent);
+    if (isAutosaveMode) {
+      onAutoChange?.(newContent);
+    }
+    // Potentially add validation here and set isValid
+  };
+
+  const blockInfo = blockRegistry[block.type];
+  const displayText = blockInfo?.label || "Block";
+  const shouldUseContextualSurface =
+    (useContextualSurface ?? true) && block.type === 'text';
+  const contextualSurfaceStyle = shouldUseContextualSurface ? editorSurfaceContext?.style : undefined;
+  const editorClassName = cn(
+    "bg-transparent text-foreground border-none shadow-none focus-within:ring-0 min-h-[60vh]",
+    shouldUseContextualSurface &&
+      editorSurfaceContext?.isDark &&
+      "[&_.ProseMirror]:text-white [&_.ProseMirror.prose]:prose-invert"
+  );
+  const resolvedSaveStatusText = saveStatusText ?? autosaveStatusLabels[saveStatus];
+
+  return (
     <>
       <Dialog open={isOpen} onOpenChange={(open) => {
         if (!open) {
@@ -192,56 +192,56 @@ export function BlockEditorModal({
         }
       }}>
         <DialogContent 
-          className="max-w-6xl h-[90vh] flex flex-col p-0 gap-0 overflow-hidden"
-          onInteractOutside={(e) => {
-            e.preventDefault();
-            handleCloseAttempt();
-          }}
+          className="max-w-6xl h-[90vh] flex flex-col p-0 gap-0 overflow-hidden"
+          onInteractOutside={(e) => {
+            e.preventDefault();
+            handleCloseAttempt();
+          }}
           onEscapeKeyDown={(e) => {
             e.preventDefault();
             handleCloseAttempt();
           }}
         >
             {/* Header */}
-            <div className="flex items-center justify-between p-4 border-b bg-background/95 text-foreground backdrop-blur z-10">
-                <div className="flex items-center gap-2">
-                   <DialogTitle className="text-lg font-semibold">
-                    {titleOverride ?? `Edit ${displayText}`}
-                   </DialogTitle>
-                </div>
-                 <div className="flex items-center gap-2">
-                    {isAutosaveMode && (
-                      <span
-                        aria-live="polite"
-                        className={cn(
-                          "rounded-md px-2 py-1 text-xs font-medium",
-                          saveStatus === "error"
-                            ? "bg-destructive/10 text-destructive"
-                            : saveStatus === "dirty"
-                              ? "bg-amber-500/10 text-amber-700 dark:text-amber-300"
-                              : "bg-muted text-muted-foreground",
-                          "max-w-[360px] truncate"
-                        )}
-                        title={resolvedSaveStatusText}
-                      >
-                        {resolvedSaveStatusText}
-                      </span>
-                    )}
-                    <Button
-                      variant={isAutosaveMode ? "default" : "ghost"}
-                      size="sm"
-                      onClick={handleCloseAttempt}
-                      disabled={isAutosaveMode && isFlushingBeforeClose}
-                    >
-                      {isAutosaveMode ? "Done" : "Cancel"}
-                    </Button>
-                    {!isAutosaveMode && (
-                      <Button onClick={handleSave} disabled={!isValid} size="sm">
-                        Save ({saveShortcutLabel})
-                      </Button>
-                    )}
-                 </div>
-            </div>
+            <div className="flex items-center justify-between p-4 border-b bg-background/95 text-foreground backdrop-blur z-10">
+                <div className="flex items-center gap-2">
+                   <DialogTitle className="text-lg font-semibold">
+                    {titleOverride ?? `Edit ${displayText}`}
+                   </DialogTitle>
+                </div>
+                 <div className="flex items-center gap-2">
+                    {isAutosaveMode && (
+                      <span
+                        aria-live="polite"
+                        className={cn(
+                          "rounded-md px-2 py-1 text-xs font-medium",
+                          saveStatus === "error"
+                            ? "bg-destructive/10 text-destructive"
+                            : saveStatus === "dirty"
+                              ? "bg-amber-500/10 text-amber-700 dark:text-amber-300"
+                              : "bg-muted text-muted-foreground",
+                          "max-w-[360px] truncate"
+                        )}
+                        title={resolvedSaveStatusText}
+                      >
+                        {resolvedSaveStatusText}
+                      </span>
+                    )}
+                    <Button
+                      variant={isAutosaveMode ? "default" : "ghost"}
+                      size="sm"
+                      onClick={handleCloseAttempt}
+                      disabled={isAutosaveMode && isFlushingBeforeClose}
+                    >
+                      {isAutosaveMode ? "Done" : "Cancel"}
+                    </Button>
+                    {!isAutosaveMode && (
+                      <Button onClick={handleSave} disabled={!isValid} size="sm">
+                        Save ({saveShortcutLabel})
+                      </Button>
+                    )}
+                 </div>
+            </div>
 
             {/* Editor Area with Contextual Background */}
             <div 
@@ -251,38 +251,38 @@ export function BlockEditorModal({
                   // Conditional Background Logic:
                   // Only apply specific section background to 'text' and 'heading' blocks to allow "Live Preview" of copy.
                   // For complex blocks like Forms, Buttons, etc., keep a neutral background to ensure input field contrast.
-                  shouldUseContextualSurface ? (
-                     // If no specific background, use white/dark default
-                     (!sectionBackground || sectionBackground.type === 'none') && "bg-muted/10"
-                  ) : "bg-muted/10", // Default for non-text blocks
-
-                  // Apply theme classes if present (ONLY for text/heading)
-                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'primary' && 'bg-primary text-primary-foreground',
-                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'secondary' && 'bg-secondary text-secondary-foreground',
-                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'muted' && 'bg-muted text-muted-foreground',
-                  
-                   // Dark mode prose invert if dark background (approximate check for solid color)
-                  shouldUseContextualSurface && (sectionBackground?.type === 'solid' && sectionBackground.solid_color && ['#000', '#111', '#0f172a', 'black'].some(c => sectionBackground.solid_color?.includes(c))) && "[&_.prose]:prose-invert",
-                  shouldUseContextualSurface && editorSurfaceContext?.isDark && "[&_.prose]:prose-invert [&_.ProseMirror]:text-white"
-              )}
-              style={{
-                  // Only apply custom color/gradient styles for text/heading
-                  backgroundColor: shouldUseContextualSurface && sectionBackground?.type === 'solid' ? sectionBackground.solid_color : undefined,
-                  backgroundImage: shouldUseContextualSurface && sectionBackground?.type === 'gradient' && sectionBackground.gradient ? 
-                    `${sectionBackground.gradient.type}-gradient(${sectionBackground.gradient.direction}, ${sectionBackground.gradient.stops.map(s => `${s.color} ${s.position}%`).join(', ')})` 
-                    : undefined,
-                  ...contextualSurfaceStyle,
-              }}
-            >
+                  shouldUseContextualSurface ? (
+                     // If no specific background, use white/dark default
+                     (!sectionBackground || sectionBackground.type === 'none') && "bg-muted/10"
+                  ) : "bg-muted/10", // Default for non-text blocks
+
+                  // Apply theme classes if present (ONLY for text/heading)
+                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'primary' && 'bg-primary text-primary-foreground',
+                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'secondary' && 'bg-secondary text-secondary-foreground',
+                  shouldUseContextualSurface && sectionBackground?.type === 'theme' && sectionBackground.theme === 'muted' && 'bg-muted text-muted-foreground',
+                  
+                   // Dark mode prose invert if dark background (approximate check for solid color)
+                  shouldUseContextualSurface && (sectionBackground?.type === 'solid' && sectionBackground.solid_color && ['#000', '#111', '#0f172a', 'black'].some(c => sectionBackground.solid_color?.includes(c))) && "[&_.prose]:prose-invert",
+                  shouldUseContextualSurface && editorSurfaceContext?.isDark && "[&_.prose]:prose-invert [&_.ProseMirror]:text-white"
+              )}
+              style={{
+                  // Only apply custom color/gradient styles for text/heading
+                  backgroundColor: shouldUseContextualSurface && sectionBackground?.type === 'solid' ? sectionBackground.solid_color : undefined,
+                  backgroundImage: shouldUseContextualSurface && sectionBackground?.type === 'gradient' && sectionBackground.gradient ? 
+                    `${sectionBackground.gradient.type}-gradient(${sectionBackground.gradient.direction}, ${sectionBackground.gradient.stops.map(s => `${s.color} ${s.position}%`).join(', ')})` 
+                    : undefined,
+                  ...contextualSurfaceStyle,
+              }}
+            >
                <div className="max-w-6xl mx-auto">
                   <Suspense fallback={<div className="flex justify-center items-center h-32">Loading editor...</div>}>
                     <EditorComponent 
-                        block={block}
-                        content={tempContent} 
-                        onChange={handleContentChange} 
-                        className={editorClassName}
-                        sectionBackground={sectionBackground} // Pass down if editor supports it
-                    />
+                        block={block}
+                        content={tempContent} 
+                        onChange={handleContentChange} 
+                        className={editorClassName}
+                        sectionBackground={sectionBackground} // Pass down if editor supports it
+                    />
                   </Suspense>
                </div>
             </div>