@prenta/admin 2.0.2 → 2.1.0

package/src/views/page-editor/PageSectionEditor.tsx

@@ -12,6 +12,7 @@ import { EditorRailTabs, type RailTab, type RailTabId } from '../../components/E
 import { ErrorBoundary } from '../../components/ErrorBoundary.js'
 import { VersionHistoryPanel } from './VersionHistoryPanel.js'
 import { useDebouncedAutosave } from './autosave.js'
+import { useEditorHistory, useSectionEditorShortcuts } from '../../hooks/use-editor-history.js'
 import { ReviewBar } from './ReviewBar.js'
 import {
   fetchPageVersions,
@@ -215,11 +216,14 @@ export function PageSectionEditor({
    * so a save moves the baseline exactly as a publish does. Missing one would
    * leave the guard comparing against content that is no longer on the site.
    */
+  const savedJson = useRef('')
   const acceptPage = useCallback((p: EditorPage) => {
+    savedJson.current = JSON.stringify(p)
     setPage(p)
     setBaseline(snapshotBaseline(p.sections, p.status))
     setPublishedPath(p.status === 'PUBLISHED' ? p.path : null)
   }, [])
+  const { note, undo, redo, canUndo, canRedo } = useEditorHistory(page, documentId ?? null)
   const [dirty, setDirty] = useState(false)
   const [saveState, setSaveState] = useState<SaveState>('idle')
   const [publishing, setPublishing] = useState(false)
@@ -486,23 +490,35 @@ export function PageSectionEditor({
 
   // ─── Section mutations ───────────────────────────────────────────────
   const mutateSections = useCallback(
-    (fn: (sections: EditorPage['sections']) => EditorPage['sections']) => {
-      setPage((prev) => (prev ? { ...prev, sections: fn(prev.sections) } : prev))
+    (
+      fn: (sections: EditorPage['sections']) => EditorPage['sections'],
+      key: string,
+      coalesce = false,
+    ) => {
+      setPage((prev) => {
+        if (!prev) return prev
+        note(prev, key, coalesce)
+        return { ...prev, sections: fn(prev.sections) }
+      })
       setDirty(true)
       setSaveState('idle')
     },
-    [],
+    [note],
   )
 
   const handleToggleVisibility = useCallback(
-    (id: string) => mutateSections((s) => toggleSectionVisibility(s, id)),
+    (id: string) => mutateSections((s) => toggleSectionVisibility(s, id), `visibility:${id}`),
     [mutateSections],
   )
 
   const handleContentChange = useCallback(
     (patch: Record<string, unknown>) => {
       if (!selectedId) return
-      mutateSections((s) => updateSectionContent(s, selectedId, patch))
+      mutateSections(
+        (s) => updateSectionContent(s, selectedId, patch),
+        `content:${selectedId}`,
+        true,
+      )
     },
     [selectedId, mutateSections],
   )
@@ -517,18 +533,41 @@ export function PageSectionEditor({
   const handleMetaChange = useCallback(
     (patch: { name?: string; internalLabel?: string }) => {
       if (!selectedId) return
-      mutateSections((s) => updateSectionMeta(s, selectedId, patch))
+      mutateSections((s) => updateSectionMeta(s, selectedId, patch), `meta:${selectedId}`, true)
     },
     [selectedId, mutateSections],
   )
 
-  // ─── Page-level settings ─────────────────────────────────────────────
-  const handlePageSettings = useCallback((patch: Partial<EditorPage>) => {
-    setPage((prev) => (prev ? { ...prev, ...patch } : prev))
-    setDirty(true)
+  const restoreEdit = useCallback((next: EditorPage | null) => {
+    if (!next) return
+    setPage(next)
+    setDirty(JSON.stringify(next) !== savedJson.current)
     setSaveState('idle')
   }, [])
 
+  const handleUndo = useCallback(() => restoreEdit(undo()), [restoreEdit, undo])
+  const handleRedo = useCallback(() => restoreEdit(redo()), [restoreEdit, redo])
+
+  useSectionEditorShortcuts({
+    enabled: effectiveCanEdit,
+    onUndo: handleUndo,
+    onRedo: handleRedo,
+  })
+
+  // ─── Page-level settings ─────────────────────────────────────────────
+  const handlePageSettings = useCallback(
+    (patch: Partial<EditorPage>) => {
+      setPage((prev) => {
+        if (!prev) return prev
+        note(prev, 'page', true)
+        return { ...prev, ...patch }
+      })
+      setDirty(true)
+      setSaveState('idle')
+    },
+    [note],
+  )
+
   // ─── Save / publish ──────────────────────────────────────────────────
   /**
    * Persist robots directives. They live on the SEO record, not the document
@@ -950,6 +989,10 @@ export function PageSectionEditor({
             typeof audit?.seo.score === 'number' ? scoreBand(audit.seo.score) : undefined
           }
           onSaveDraft={handleSaveDraft}
+          onUndo={effectiveCanEdit ? handleUndo : undefined}
+          onRedo={effectiveCanEdit ? handleRedo : undefined}
+          canUndo={canUndo}
+          canRedo={canRedo}
           onSubmitForReview={handleSubmitForReview}
           onApprove={handleApprove}
           onRequestChanges={handleRequestChanges}

package/src/views/post-editor/PostSectionEditor.tsx

@@ -12,6 +12,7 @@ import { SeoEditorPane } from '../../components/seo/SeoEditorPane.js'
 import { ErrorBoundary } from '../../components/ErrorBoundary.js'
 import { VersionHistoryPanel } from '../page-editor/VersionHistoryPanel.js'
 import { useDebouncedAutosave } from '../page-editor/autosave.js'
+import { useEditorHistory, useSectionEditorShortcuts } from '../../hooks/use-editor-history.js'
 import { ReviewBar } from '../page-editor/ReviewBar.js'
 import {
   fetchPageVersions,
@@ -182,10 +183,13 @@ export function PostSectionEditor({
    * as a publish does; missing one would leave the guard comparing against
    * content no longer on the site.
    */
+  const savedJson = useRef('')
   const acceptPost = useCallback((p: EditorPost) => {
+    savedJson.current = JSON.stringify(p)
     setPost(p)
     setBaseline(snapshotBaseline(p.sections, p.status))
   }, [])
+  const { note, undo, redo, canUndo, canRedo } = useEditorHistory(post, documentId ?? null)
 
   const { canEdit } = useMemo(() => deriveCan(session), [session])
   const sessionRole = String(session?.user?.role ?? '')
@@ -393,23 +397,35 @@ export function PostSectionEditor({
 
   // ─── Section mutations ───────────────────────────────────────────────
   const mutateSections = useCallback(
-    (fn: (sections: EditorPost['sections']) => EditorPost['sections']) => {
-      setPost((prev) => (prev ? { ...prev, sections: fn(prev.sections) } : prev))
+    (
+      fn: (sections: EditorPost['sections']) => EditorPost['sections'],
+      key: string,
+      coalesce = false,
+    ) => {
+      setPost((prev) => {
+        if (!prev) return prev
+        note(prev, key, coalesce)
+        return { ...prev, sections: fn(prev.sections) }
+      })
       setDirty(true)
       setSaveState('idle')
     },
-    [],
+    [note],
   )
 
   const handleToggleVisibility = useCallback(
-    (id: string) => mutateSections((s) => toggleSectionVisibility(s, id)),
+    (id: string) => mutateSections((s) => toggleSectionVisibility(s, id), `visibility:${id}`),
     [mutateSections],
   )
 
   const handleContentChange = useCallback(
     (patch: Record<string, unknown>) => {
       if (!selectedId) return
-      mutateSections((s) => updateSectionContent(s, selectedId, patch))
+      mutateSections(
+        (s) => updateSectionContent(s, selectedId, patch),
+        `content:${selectedId}`,
+        true,
+      )
     },
     [selectedId, mutateSections],
   )
@@ -419,22 +435,46 @@ export function PostSectionEditor({
   const handleMetaChange = useCallback(
     (patch: { name?: string; internalLabel?: string }) => {
       if (!selectedId) return
-      mutateSections((s) => updateSectionMeta(s, selectedId, patch))
+      mutateSections((s) => updateSectionMeta(s, selectedId, patch), `meta:${selectedId}`, true)
     },
     [selectedId, mutateSections],
   )
 
-  const handleFields = useCallback((patch: Partial<EditorPost>) => {
-    setPost((prev) => (prev ? { ...prev, ...patch } : prev))
-    setDirty(true)
+  const restoreEdit = useCallback((next: EditorPost | null) => {
+    if (!next) return
+    setPost(next)
+    setDirty(JSON.stringify(next) !== savedJson.current)
     setSaveState('idle')
   }, [])
 
-  const handlePostBodyChange = useCallback((body: string) => {
-    setPost((prev) => (prev ? { ...prev, body } : prev))
-    setDirty(true)
-    setSaveState('idle')
-  }, [])
+  const handleUndo = useCallback(() => restoreEdit(undo()), [restoreEdit, undo])
+  const handleRedo = useCallback(() => restoreEdit(redo()), [restoreEdit, redo])
+
+  const handleFields = useCallback(
+    (patch: Partial<EditorPost>) => {
+      setPost((prev) => {
+        if (!prev) return prev
+        note(prev, 'fields', true)
+        return { ...prev, ...patch }
+      })
+      setDirty(true)
+      setSaveState('idle')
+    },
+    [note],
+  )
+
+  const handlePostBodyChange = useCallback(
+    (body: string) => {
+      setPost((prev) => {
+        if (!prev) return prev
+        note(prev, 'body', true)
+        return { ...prev, body }
+      })
+      setDirty(true)
+      setSaveState('idle')
+    },
+    [note],
+  )
 
   // ─── Save / publish ──────────────────────────────────────────────────
   const latestPost = useRef(post)
@@ -568,6 +608,12 @@ export function PostSectionEditor({
   const effectiveCanEdit = canEdit && !reviewing
   const canRestoreVersion = canEdit
 
+  useSectionEditorShortcuts({
+    enabled: effectiveCanEdit,
+    onUndo: handleUndo,
+    onRedo: handleRedo,
+  })
+
   useDebouncedAutosave({
     dirty,
     enabled: Boolean(documentId) && effectiveCanEdit && !publishing,
@@ -814,6 +860,10 @@ export function PostSectionEditor({
             typeof audit?.seo.score === 'number' ? scoreBand(audit.seo.score) : undefined
           }
           onSaveDraft={handleSaveDraft}
+          onUndo={effectiveCanEdit ? handleUndo : undefined}
+          onRedo={effectiveCanEdit ? handleRedo : undefined}
+          canUndo={canUndo}
+          canRedo={canRedo}
           onSubmitForReview={handleSubmitForReview}
           onApprove={handleApprove}
           onRequestChanges={handleRequestChanges}

package/src/views/seo/AuditTab.tsx

@@ -69,8 +69,22 @@ function AuditSummaryCards({ refetchKey }: { refetchKey: number }) {
   const { data, loading } = useSeoResource(fetchSeoAuditRuns, [refetchKey])
   const last = (data ?? [])[0]
   const coverage = last?.metadata?.renderedAudit
+  const total = last?.metadata?.total
+  const running = last?.status === 'RUNNING'
+  const truncated = last?.metadata?.truncated === true
+
   return (
     <div className="space-y-4">
+      {truncated ? (
+        <p
+          className="border-border bg-card text-foreground rounded-lg border px-4 py-3 text-sm"
+          role="status"
+        >
+          This audit covered {last?.pagesScanned ?? 0}
+          {typeof total === 'number' ? ` of ${total}` : ''} published pages and stopped before the
+          rest. Issues on the pages it did not reach were kept.
+        </p>
+      ) : null}
       <div className="grid grid-cols-2 gap-4 lg:grid-cols-4">
         <div className="border-border bg-card rounded-lg border p-4">
           <p className="text-muted-foreground flex items-center gap-1.5 text-sm">
@@ -83,8 +97,11 @@ function AuditSummaryCards({ refetchKey }: { refetchKey: number }) {
         <div className="border-border bg-card rounded-lg border p-4">
           <p className="text-muted-foreground text-sm">Pages scanned</p>
           <p className="text-foreground mt-1 text-2xl font-medium tabular-nums">
-            {last?.pagesScanned ?? 0}
+            {running && typeof total === 'number'
+              ? `${last?.pagesScanned ?? 0} of ${total}`
+              : (last?.pagesScanned ?? 0)}
           </p>
+          {running ? <p className="text-muted-foreground mt-1 text-sm">Audit in progress</p> : null}
           {coverage ? (
             <p className="text-muted-foreground mt-1 flex flex-wrap gap-x-3 text-sm">
               <span>{coverage.verified} verified</span>