@prenta/admin 1.12.0 → 1.14.0

package/src/__tests__/views/seo-settings.render.test.tsx

@@ -2,11 +2,18 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { fireEvent, render, screen, waitFor, within } from '@testing-library/react'
 
-// The SEO defaults tab reads five shared backends: /seo/config (meta defaults +
+// The SEO defaults tab reads six shared backends: /seo/config (meta defaults +
 // verification), /seo/crawl-settings (sitemap toggles), /seo/sitemap/status
-// (read-only summary), /seo/index-status (environment), and
-// /seo/search-console (Search data card).
-const cmsApi = vi.fn(async (endpoint: string, options?: { method?: string }) => {
+// (read-only summary), /seo/index-status (environment), /seo/search-console
+// (Search data card), and /seo/digest-settings (Weekly SEO digest card).
+// `defaultApi` is the per-path route map; individual tests override the mock
+// implementation and delegate back to it for unrelated paths.
+type MockApiResponse = { data?: unknown; error?: string; status: number }
+
+async function defaultApi(
+  endpoint: string,
+  options?: { method?: string },
+): Promise<MockApiResponse> {
   const method = options?.method ?? 'GET'
   if (endpoint === '/seo/config' && method === 'GET') {
     return {
@@ -67,14 +74,34 @@ const cmsApi = vi.fn(async (endpoint: string, options?: { method?: string }) =>
       status: 200,
     }
   }
+  if (endpoint === '/seo/digest-settings' && method === 'GET') {
+    return {
+      data: { enabled: false, recipients: [], lastSentAt: null, lastError: null },
+      status: 200,
+    }
+  }
   return { data: {}, status: 200 }
-})
+}
+
+const cmsApi = vi.fn(defaultApi)
 
 vi.mock('../../lib/api.js', () => ({
   cmsApi: (e: string, o?: unknown) => cmsApi(e, o as any),
   ensureCsrfToken: vi.fn(async () => {}),
 }))
 
+// Record toast calls so tests can assert server errors are surfaced verbatim.
+const toastSuccess = vi.fn()
+const toastError = vi.fn()
+vi.mock('sonner', () => ({
+  toast: {
+    success: (...args: unknown[]) => toastSuccess(...args),
+    error: (...args: unknown[]) => toastError(...args),
+    warning: vi.fn(),
+    loading: vi.fn(),
+  },
+}))
+
 // Stub the Media Center picker: when open, expose a button that selects a known
 // asset (id + url), exercising the asset-id storage path without the real
 // MediaBrowser/network stack.
@@ -105,6 +132,9 @@ const { SeoSettingsTab } = await import('../../views/settings/SeoSettingsTab.js'
 
 beforeEach(() => {
   cmsApi.mockClear()
+  cmsApi.mockImplementation(defaultApi)
+  toastSuccess.mockClear()
+  toastError.mockClear()
 })
 
 describe('SeoSettingsTab', () => {
@@ -300,4 +330,73 @@ describe('SeoSettingsTab', () => {
     expect(body.propertyUrl).toBe('sc-domain:acme.test')
     expect(body.serviceAccountKey).toContain('svc@acme.iam.gserviceaccount.com')
   })
+
+  it('renders the Weekly SEO digest card with the toggle off by default', async () => {
+    render(<SeoSettingsTab role="ADMIN" />)
+    expect(await screen.findByText('Weekly SEO digest')).toBeTruthy()
+
+    const toggle = screen.getByRole('switch', { name: 'Send weekly digest' })
+    expect(toggle.getAttribute('aria-checked')).toBe('false')
+    expect(screen.getByText('Never sent')).toBeTruthy()
+  })
+
+  it('saves recipients as a trimmed, comma-split list', async () => {
+    render(<SeoSettingsTab role="ADMIN" />)
+    await screen.findByText('Weekly SEO digest')
+
+    fireEvent.change(screen.getByLabelText('Recipients'), {
+      target: { value: ' a@x.com , b@x.com ' },
+    })
+    fireEvent.click(screen.getByRole('button', { name: 'Save recipients' }))
+
+    await waitFor(() =>
+      expect(cmsApi).toHaveBeenCalledWith(
+        '/seo/digest-settings',
+        expect.objectContaining({ method: 'PUT' }),
+      ),
+    )
+    const putCall = cmsApi.mock.calls.find(
+      ([e, o]) => e === '/seo/digest-settings' && (o as { method?: string })?.method === 'PUT',
+    )
+    const body = JSON.parse((putCall![1] as { body: string }).body)
+    expect(body.recipients).toEqual(['a@x.com', 'b@x.com'])
+  })
+
+  it('surfaces the server error verbatim when test-send fails', async () => {
+    cmsApi.mockImplementation(async (endpoint, options) => {
+      if (endpoint === '/seo/digest-settings/test-send' && options?.method === 'POST') {
+        return { error: 'No completed SEO audit yet — run an audit first', status: 409 }
+      }
+      return defaultApi(endpoint, options)
+    })
+    render(<SeoSettingsTab role="ADMIN" />)
+    await screen.findByText('Weekly SEO digest')
+
+    fireEvent.click(screen.getByRole('button', { name: /Send test digest/ }))
+
+    await waitFor(() =>
+      expect(toastError).toHaveBeenCalledWith('No completed SEO audit yet — run an audit first'),
+    )
+  })
+
+  it('shows the last error from settings in destructive tone', async () => {
+    cmsApi.mockImplementation(async (endpoint, options) => {
+      if (endpoint === '/seo/digest-settings' && (options?.method ?? 'GET') === 'GET') {
+        return {
+          data: {
+            enabled: true,
+            recipients: ['owner@acme.test'],
+            lastSentAt: '2026-09-01T04:00:00Z',
+            lastError: '1/2 digest sends failed',
+          },
+          status: 200,
+        }
+      }
+      return defaultApi(endpoint, options)
+    })
+    render(<SeoSettingsTab role="ADMIN" />)
+
+    const lastError = await screen.findByText(/1\/2 digest sends failed/)
+    expect(lastError.className).toContain('text-destructive')
+  })
 })

package/src/components/seo/ChangePreview.tsx

@@ -0,0 +1,47 @@
+'use client'
+
+import type { SeoFixFieldChange } from '../../lib/seo-service.js'
+
+function formatValue(value: string | boolean | null): string {
+  if (value === null || value === '') return '—'
+  if (typeof value === 'boolean') return value ? 'Yes' : 'No'
+  if (value.length > 280) return `${value.slice(0, 277)}…`
+  return value
+}
+
+/** Before/after table for a proposed SEO change set. Shared by the issue fix panel and the proposals inbox. */
+export function ChangePreview({ changes }: { changes: SeoFixFieldChange[] }) {
+  if (changes.length === 0) return null
+  return (
+    <div className="border-border overflow-x-auto rounded-md border">
+      <table className="w-full min-w-[480px] text-sm" aria-label="Proposed SEO changes">
+        <thead>
+          <tr className="border-border bg-muted/40 text-muted-foreground border-b text-left">
+            <th scope="col" className="py-2 pr-3 pl-3 font-medium">
+              Field
+            </th>
+            <th scope="col" className="py-2 pr-3 font-medium">
+              Current
+            </th>
+            <th scope="col" className="py-2 pr-3 font-medium">
+              Proposed
+            </th>
+          </tr>
+        </thead>
+        <tbody className="divide-border divide-y">
+          {changes.map((c) => (
+            <tr key={c.field}>
+              <td className="text-foreground py-2 pr-3 pl-3 font-medium">{c.label}</td>
+              <td className="text-muted-foreground max-w-48 py-2 pr-3 wrap-break-word">
+                {formatValue(c.before)}
+              </td>
+              <td className="text-foreground max-w-48 py-2 pr-3 wrap-break-word">
+                {formatValue(c.after)}
+              </td>
+            </tr>
+          ))}
+        </tbody>
+      </table>
+    </div>
+  )
+}

package/src/components/seo/SeoIssueFixPanel.tsx

@@ -16,12 +16,12 @@ import {
   supportsSeoContentResearch,
   type SeoContentBrief,
   type SeoContentResearch,
-  type SeoFixFieldChange,
   type SeoIssue,
   type SeoIssueFixSuggestion,
 } from '../../lib/seo-service.js'
 import { hasPlanFeature, type PlanInfo } from '../../lib/plan.js'
 import { PlanUpgradeCallout } from '../PlanUpgradeCallout.js'
+import { ChangePreview } from './ChangePreview.js'
 import { SeoErrorState, btnPrimary, btnSecondary } from './primitives.js'
 
 function approveButtonLabel(strategy: SeoIssueFixSuggestion['fixStrategy']): string {
@@ -56,49 +56,6 @@ function approveSuccessToast(strategy: SeoIssueFixSuggestion['fixStrategy']): st
   }
 }
 
-function formatValue(value: string | boolean | null): string {
-  if (value === null || value === '') return '—'
-  if (typeof value === 'boolean') return value ? 'Yes' : 'No'
-  if (value.length > 280) return `${value.slice(0, 277)}…`
-  return value
-}
-
-function ChangePreview({ changes }: { changes: SeoFixFieldChange[] }) {
-  if (changes.length === 0) return null
-  return (
-    <div className="border-border overflow-x-auto rounded-md border">
-      <table className="w-full min-w-[480px] text-sm" aria-label="Proposed SEO changes">
-        <thead>
-          <tr className="border-border bg-muted/40 text-muted-foreground border-b text-left">
-            <th scope="col" className="py-2 pr-3 pl-3 font-medium">
-              Field
-            </th>
-            <th scope="col" className="py-2 pr-3 font-medium">
-              Current
-            </th>
-            <th scope="col" className="py-2 pr-3 font-medium">
-              Proposed
-            </th>
-          </tr>
-        </thead>
-        <tbody className="divide-border divide-y">
-          {changes.map((c) => (
-            <tr key={c.field}>
-              <td className="text-foreground py-2 pr-3 pl-3 font-medium">{c.label}</td>
-              <td className="text-muted-foreground max-w-48 py-2 pr-3 wrap-break-word">
-                {formatValue(c.before)}
-              </td>
-              <td className="text-foreground max-w-48 py-2 pr-3 wrap-break-word">
-                {formatValue(c.after)}
-              </td>
-            </tr>
-          ))}
-        </tbody>
-      </table>
-    </div>
-  )
-}
-
 function ContentResearchCard({ research }: { research: SeoContentResearch }) {
   return (
     <div className="border-border bg-card space-y-3 rounded-md border p-4">

package/src/lib/seo-service.ts

@@ -113,6 +113,8 @@ export interface SeoOverview {
   lastAuditRunId: string | null
   /** True when the last audit hit AUDIT_MAX_ENTITIES. */
   auditTruncated?: boolean
+  /** Cached AI issue fixes + open redirect suggestions awaiting review. */
+  pendingProposals?: { issueFixes: number; redirects: number }
 }
 
 const EMPTY_OVERVIEW: SeoOverview = {
@@ -783,6 +785,49 @@ export async function updateCrawlSettings(
   return res.error ? { error: res.error } : {}
 }
 
+// ─── Weekly digest ───────────────────────────────────────────────────
+
+export interface SeoDigestSettings {
+  enabled: boolean
+  recipients: string[]
+  lastSentAt: string | null
+  lastError: string | null
+}
+
+export async function fetchDigestSettings(): Promise<SeoDigestSettings | null> {
+  const res = await cmsApi<SeoDigestSettings>('/seo/digest-settings')
+  const d = res.data
+  // Validate the shape before handing it to the UI — an unexpected response
+  // body must render the card's error state, not crash on `recipients.join`.
+  if (!d || typeof d.enabled !== 'boolean' || !Array.isArray(d.recipients)) return null
+  return {
+    enabled: d.enabled,
+    recipients: d.recipients,
+    lastSentAt: d.lastSentAt ?? null,
+    lastError: d.lastError ?? null,
+  }
+}
+
+export async function updateDigestSettings(patch: {
+  enabled?: boolean
+  recipients?: string[]
+}): Promise<{ ok: boolean; error?: string }> {
+  const res = await cmsApi<{ ok: boolean }>('/seo/digest-settings', {
+    method: 'PUT',
+    body: JSON.stringify(patch),
+  })
+  if (res.error) return { ok: false, error: res.error }
+  return { ok: true }
+}
+
+export async function sendTestDigest(): Promise<{ ok: boolean; to?: string; error?: string }> {
+  const res = await cmsApi<{ ok: boolean; to?: string }>('/seo/digest-settings/test-send', {
+    method: 'POST',
+  })
+  if (res.error) return { ok: false, error: res.error }
+  return { ok: true, to: res.data?.to }
+}
+
 // ─── AI crawler activity ─────────────────────────────────────────────
 
 export interface AiCrawlerRow {
@@ -1290,3 +1335,126 @@ export async function fetchLinkHealthIssues(): Promise<{
     },
   }
 }
+
+// ─── AI proposals inbox ─────────────────────────────────────────────────
+
+export interface SeoProposal {
+  /** `issue:<issueId>` | `redirect:<suggestionId>` */
+  id: string
+  kind: 'issue-fix' | 'redirect'
+  title: string
+  entity: { type: string | null; id: string | null; title: string | null; url: string | null }
+  changes: SeoFixFieldChange[]
+  justification: string
+  source: 'ai' | 'deterministic' | 'ai-404-recovery'
+  /** 0–1, redirects only. */
+  confidence: number | null
+  /** 0–100, issue fixes only. */
+  brandAlignment: number | null
+  generatedAt: string | null
+  /** Issue fixes only; redirects are keyed by id. */
+  fingerprint: string | null
+  /** Issue fixes: cached fingerprint ≠ live fingerprint. Redirects: always false. */
+  stale: boolean
+  issueType: string | null
+  severity: SeoSeverity | null
+  issueId: string | null
+  suggestionId: string | null
+}
+
+export interface SeoProposalsPayload {
+  proposals: SeoProposal[]
+  counts: { issueFixes: number; redirects: number; stale: number; eligibleForGeneration: number }
+  /** `allowBulkApply` per feature, so the UI can disable batch approve up front. */
+  governance: { issueFixBulk: boolean; redirectBulk: boolean }
+}
+
+export interface SeoProposalGenerateResult {
+  generated: number
+  skipped: number
+  failed: number
+  remaining: number
+  haltedReason: string | null
+}
+
+export interface SeoProposalBulkApplyItem {
+  kind: 'issue-fix' | 'redirect'
+  id: string
+  fingerprint?: string
+}
+
+/** Mirrors core's `BulkApplyItemStatus` (`seo/proposals-bulk-apply.ts`). */
+export type SeoProposalBulkApplyItemStatus =
+  | 'applied'
+  | 'stale'
+  | 'verify-failed'
+  | 'plan-required'
+  | 'not-found'
+  | 'not-allowed'
+  | 'not-fixable'
+  | 'invalid'
+  | 'still-live'
+  | 'error'
+  | 'pending'
+
+export interface SeoProposalBulkApplyResult {
+  results: Array<{
+    kind: 'issue-fix' | 'redirect'
+    id: string
+    status: SeoProposalBulkApplyItemStatus
+    reason?: string
+  }>
+  applied: number
+  failed: number
+  /** Items the server did not reach within its time budget — resubmit exactly these. */
+  pending: number
+}
+
+export async function fetchSeoProposals(): Promise<SeoProposalsPayload> {
+  const res = await cmsApi<SeoProposalsPayload>('/seo/proposals')
+  throwIfError(res)
+  return (
+    res.data ?? {
+      proposals: [],
+      counts: { issueFixes: 0, redirects: 0, stale: 0, eligibleForGeneration: 0 },
+      governance: { issueFixBulk: false, redirectBulk: false },
+    }
+  )
+}
+
+export async function generateSeoProposals(): Promise<{
+  result?: SeoProposalGenerateResult
+  error?: string
+}> {
+  const res = await cmsApi<SeoProposalGenerateResult>('/seo/proposals/generate', {
+    method: 'POST',
+    body: JSON.stringify({ confirm: true }),
+  })
+  if (res.error) return { error: res.error }
+  return { result: res.data }
+}
+
+export async function bulkApplySeoProposals(items: SeoProposalBulkApplyItem[]): Promise<{
+  result?: SeoProposalBulkApplyResult
+  error?: string
+  /** True when the server refused because `allowBulkApply` is off for a represented feature. */
+  governanceBlocked?: boolean
+}> {
+  const res = await cmsApi<SeoProposalBulkApplyResult>('/seo/proposals/bulk-apply', {
+    method: 'POST',
+    body: JSON.stringify({ items }),
+  })
+  if (res.error) {
+    return res.code === 'governance_blocked'
+      ? { error: res.error, governanceBlocked: true }
+      : { error: res.error }
+  }
+  return { result: res.data }
+}
+
+export async function dismissSeoIssueFix(issueId: string): Promise<{ error?: string }> {
+  const res = await cmsApi(`/seo/issues/${encodeURIComponent(issueId)}/dismiss-fix`, {
+    method: 'POST',
+  })
+  return res.error ? { error: res.error } : {}
+}

package/src/views/SEO.tsx

@@ -1,10 +1,11 @@
 'use client'
 
 /**
- * SEO Operations Center — five-tab workflow surface (Overview / Content /
- * Technical / Redirects / Audit). Tab + deep-link state is driven by the
- * `?tab=` query string so audits, issues, redirects, and content editors are
- * all linkable. All data flows through `lib/seo-service.ts`.
+ * SEO Operations Center — seven-tab workflow surface (Overview / Content /
+ * Links / Technical / Redirects / Audit / Proposals). Tab + deep-link state is
+ * driven by the `?tab=` query string so audits, issues, redirects, proposals,
+ * and content editors are all linkable. All data flows through
+ * `lib/seo-service.ts`.
  */
 import * as Tabs from '@radix-ui/react-tabs'
 import {
@@ -16,6 +17,7 @@ import {
   RefreshCw,
   Bot,
   Link2,
+  Inbox,
 } from 'lucide-react'
 import { useCallback, useEffect, useMemo, useState } from 'react'
 import { toast } from 'sonner'
@@ -32,6 +34,7 @@ import { TechnicalTab } from './seo/TechnicalTab.js'
 import { RedirectsTab } from './seo/RedirectsTab.js'
 import { AuditTab } from './seo/AuditTab.js'
 import { LinksTab } from './seo/LinksTab.js'
+import { ProposalsTab } from './seo/ProposalsTab.js'
 
 export interface SEOProps {
   onNavigate?: (path: string) => void
@@ -45,6 +48,7 @@ const TABS = [
   { id: 'technical', label: 'Technical', icon: Settings2 },
   { id: 'redirects', label: 'Redirects', icon: ArrowRightLeft },
   { id: 'audit', label: 'Audit', icon: ClipboardCheck },
+  { id: 'proposals', label: 'Proposals', icon: Inbox },
 ] as const
 
 type TabId = (typeof TABS)[number]['id']
@@ -69,6 +73,7 @@ export const SEO_DEEP_PATHS = [
   '/seo/technical',
   '/seo/links',
   '/seo/audit',
+  '/seo/proposals',
   '/seo',
 ] as const
 
@@ -89,6 +94,8 @@ export function seoTabForPath(pathname: string): TabId | null {
       return 'technical'
     case '/seo/audit':
       return 'audit'
+    case '/seo/proposals':
+      return 'proposals'
     case '/seo':
       return 'overview'
     default:
@@ -177,7 +184,11 @@ export function SEO({ onNavigate, initialTab = 'overview' }: SEOProps) {
 
   const badgeCounts = useMemo(() => {
     const summary = overviewResource.data?.issuesSummary
-    return { audit: summary ? summary.critical + summary.warning : 0 }
+    const pending = overviewResource.data?.pendingProposals
+    return {
+      audit: summary ? summary.critical + summary.warning : 0,
+      proposals: pending ? pending.issueFixes + pending.redirects : 0,
+    }
   }, [overviewResource.data])
 
   return (
@@ -215,7 +226,18 @@ export function SEO({ onNavigate, initialTab = 'overview' }: SEOProps) {
         >
           {TABS.map((t) => {
             const Icon = t.icon
-            const badge = t.id === 'audit' ? badgeCounts.audit : 0
+            const badge =
+              t.id === 'audit'
+                ? badgeCounts.audit
+                : t.id === 'proposals'
+                  ? badgeCounts.proposals
+                  : 0
+            // The proposals count is informational (items awaiting review),
+            // not a warning like the audit issue count.
+            const badgeClass =
+              t.id === 'proposals'
+                ? 'bg-muted text-muted-foreground'
+                : 'bg-destructive/10 text-destructive'
             return (
               <Tabs.Trigger
                 key={t.id}
@@ -225,7 +247,9 @@ export function SEO({ onNavigate, initialTab = 'overview' }: SEOProps) {
                 <Icon className="h-4 w-4" aria-hidden />
                 {t.label}
                 {badge > 0 && (
-                  <span className="bg-destructive/10 text-destructive ml-1 rounded-full px-1.5 py-0.5 text-xs font-medium tabular-nums">
+                  <span
+                    className={`${badgeClass} ml-1 rounded-full px-1.5 py-0.5 text-xs font-medium tabular-nums`}
+                  >
                     {badge}
                   </span>
                 )}
@@ -272,6 +296,15 @@ export function SEO({ onNavigate, initialTab = 'overview' }: SEOProps) {
                 />
               )}
             </Tabs.Content>
+            <Tabs.Content value="proposals" tabIndex={-1}>
+              {activeTab === 'proposals' && (
+                <ProposalsTab
+                  planInfo={planInfo}
+                  onChanged={() => setRefetchKey((k) => k + 1)}
+                  onNavigate={onNavigate}
+                />
+              )}
+            </Tabs.Content>
           </ErrorBoundary>
         </div>
       </Tabs.Root>