@prenta/admin 1.14.0 → 1.15.0

package/src/__tests__/views/seo-proposals-tab.render.test.tsx

@@ -3,6 +3,7 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { fireEvent, render, screen, waitFor } from '@testing-library/react'
 
 import type {
+  SeoAutoApplyList,
   SeoProposal,
   SeoProposalBulkApplyResult,
   SeoProposalsPayload,
@@ -57,11 +58,24 @@ function payload(over: Partial<SeoProposalsPayload> = {}): SeoProposalsPayload {
     ],
     counts: { issueFixes: 2, redirects: 1, stale: 0, eligibleForGeneration: 5 },
     governance: { issueFixBulk: true, redirectBulk: true },
+    autopilot: { enabled: false, pausedReason: null, lastRunAt: null, appliedLast7d: 0 },
     ...over,
   }
 }
 
 const fetchSeoProposals = vi.fn(async () => payload())
+const fetchAutopilotApplied = vi.fn(
+  async (_params: unknown): Promise<SeoAutoApplyList> => ({
+    docs: [],
+    total: 0,
+    page: 1,
+    pageSize: 20,
+    canRevert: false,
+  }),
+)
+const revertAutopilotChange = vi.fn(async (_id: string, _force?: boolean) => ({
+  ok: true as const,
+}))
 const generateSeoProposals = vi.fn(async () => ({
   result: { generated: 2, skipped: 0, failed: 0, remaining: 0, haltedReason: null },
 }))
@@ -79,6 +93,8 @@ vi.mock('../../lib/seo-service.js', () => ({
   bulkApplySeoProposals: (items: unknown) => bulkApplySeoProposals(items),
   dismissSeoIssueFix: (id: string) => dismissSeoIssueFix(id),
   dismissRedirectSuggestion: (id: string) => dismissRedirectSuggestion(id),
+  fetchAutopilotApplied: (params: unknown) => fetchAutopilotApplied(params),
+  revertAutopilotChange: (id: string, force?: boolean) => revertAutopilotChange(id, force),
 }))
 vi.mock('sonner', () => ({
   toast: { success: vi.fn(), error: vi.fn(), info: vi.fn(), loading: vi.fn() },
@@ -109,6 +125,8 @@ beforeEach(() => {
   bulkApplySeoProposals.mockClear()
   dismissSeoIssueFix.mockClear()
   dismissRedirectSuggestion.mockClear()
+  fetchAutopilotApplied.mockClear()
+  revertAutopilotChange.mockClear()
   vi.mocked(toast.success).mockClear()
   vi.mocked(toast.error).mockClear()
 })
@@ -361,6 +379,149 @@ describe('ProposalsTab', () => {
     )
   })
 
+  it('renders the applied-by-autopilot section under the proposals card', async () => {
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    expect(await screen.findByText('Applied by autopilot')).toBeTruthy()
+    await waitFor(() => expect(fetchAutopilotApplied).toHaveBeenCalledTimes(1))
+    expect(await screen.findByText('Autopilot has not applied anything yet')).toBeTruthy()
+  })
+
+  it('shows no autopilot chip while autopilot is disabled', async () => {
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    expect(screen.queryByRole('button', { name: /autopilot (on|paused)/i })).toBeNull()
+  })
+
+  it('shows an "Autopilot on" chip with the weekly count that opens SEO settings', async () => {
+    fetchSeoProposals.mockImplementationOnce(async () =>
+      payload({
+        autopilot: {
+          enabled: true,
+          pausedReason: null,
+          lastRunAt: '2026-09-07T03:00:00.000Z',
+          appliedLast7d: 4,
+        },
+      }),
+    )
+    const onNavigate = vi.fn()
+    render(<ProposalsTab planInfo={null} onNavigate={onNavigate} />)
+    await screen.findByText('Duplicate meta title')
+    const chip = screen.getByRole('button', { name: 'Autopilot on · 4 applied this week' })
+    fireEvent.click(chip)
+    expect(onNavigate).toHaveBeenCalledWith('/settings?tab=seo')
+  })
+
+  it('shows a paused chip with a short reason when autopilot is paused', async () => {
+    fetchSeoProposals.mockImplementationOnce(async () =>
+      payload({
+        autopilot: {
+          enabled: true,
+          pausedReason: 'outcome_regression',
+          lastRunAt: null,
+          appliedLast7d: 1,
+        },
+      }),
+    )
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    expect(
+      screen.getByRole('button', { name: 'Autopilot paused — traffic regression' }),
+    ).toBeTruthy()
+    expect(screen.queryByRole('button', { name: /applied this week/ })).toBeNull()
+  })
+
+  it.each([
+    ['owner_unavailable', 'Autopilot paused — owner unavailable'],
+    ['manual', 'Autopilot paused — paused by admin'],
+    ['some_future_reason', 'Autopilot paused — some_future_reason'],
+  ])('labels the %s pause reason as "%s"', async (pausedReason, name) => {
+    fetchSeoProposals.mockImplementationOnce(async () =>
+      payload({ autopilot: { enabled: true, pausedReason, lastRunAt: null, appliedLast7d: 0 } }),
+    )
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    expect(screen.getByRole('button', { name })).toBeTruthy()
+  })
+
+  it('shows no chip when autopilot was disabled while still paused', async () => {
+    // `PUT { enabled: false }` does not clear the pause; a disabled engine is
+    // simply off, not "paused".
+    fetchSeoProposals.mockImplementationOnce(async () =>
+      payload({
+        autopilot: { enabled: false, pausedReason: 'manual', lastRunAt: null, appliedLast7d: 0 },
+      }),
+    )
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    expect(screen.queryByRole('button', { name: /autopilot/i })).toBeNull()
+  })
+
+  it('shows no chip and does not throw when the payload predates autopilot', async () => {
+    fetchSeoProposals.mockImplementationOnce(async () => {
+      // A `@prenta/core` deployed before autopilot shipped omits the field entirely.
+      const legacy: Partial<SeoProposalsPayload> = payload()
+      delete legacy.autopilot
+      return legacy as SeoProposalsPayload
+    })
+    render(<ProposalsTab planInfo={null} />)
+    expect(await screen.findByText('Duplicate meta title')).toBeTruthy()
+    expect(screen.queryByRole('button', { name: /autopilot (on|paused)/i })).toBeNull()
+    expect(screen.getByRole('button', { name: 'Generate proposals (5 eligible)' })).toBeTruthy()
+  })
+
+  it('refreshes the applied section after a proposal is approved', async () => {
+    render(<ProposalsTab planInfo={null} />)
+    await screen.findByText('Duplicate meta title')
+    await waitFor(() => expect(fetchAutopilotApplied).toHaveBeenCalledTimes(1))
+    fireEvent.click(screen.getByRole('button', { name: 'Approve Duplicate meta title' }))
+    await waitFor(() => expect(bulkApplySeoProposals).toHaveBeenCalledTimes(1))
+    await waitFor(() => expect(fetchAutopilotApplied).toHaveBeenCalledTimes(2))
+  })
+
+  it('refetches the inbox and notifies the parent after an autopilot change is reverted', async () => {
+    fetchAutopilotApplied.mockImplementation(async () => ({
+      docs: [
+        {
+          id: 'la-1',
+          kind: 'issue-fix',
+          issueType: 'missing-meta-title',
+          entityId: 'd1',
+          collection: 'pages',
+          entityTitle: 'Pricing',
+          entityUrl: '/pricing',
+          path: '/pricing',
+          before: { metaTitle: null },
+          after: { metaTitle: 'Pricing for growing teams' },
+          gate: { source: 'ai', brandAlignment: 92, confidence: null, checks: {} },
+          status: 'applied',
+          outcome: 'pending',
+          outcomeMetrics: null,
+          appliedAt: new Date().toISOString(),
+          revertedAt: null,
+          revertReason: null,
+        },
+      ],
+      total: 1,
+      page: 1,
+      pageSize: 20,
+      canRevert: true,
+    }))
+    const onChanged = vi.fn()
+    render(<ProposalsTab planInfo={null} onChanged={onChanged} />)
+    await screen.findByText('Duplicate meta title')
+    fireEvent.click(await screen.findByRole('button', { name: 'Revert change to Pricing' }))
+    fireEvent.click(await screen.findByRole('button', { name: 'Revert change' }))
+    await waitFor(() => expect(revertAutopilotChange).toHaveBeenCalledWith('la-1', false))
+    // The revert reopened the issue server-side: the inbox and the SEO page must both refresh.
+    await waitFor(() => expect(fetchSeoProposals).toHaveBeenCalledTimes(2))
+    await waitFor(() => expect(onChanged).toHaveBeenCalledTimes(1))
+    // The ledger reloads itself once; the parent must not bump `refreshKey` on top.
+    await waitFor(() => expect(fetchAutopilotApplied).toHaveBeenCalledTimes(2))
+    await flushMicrotasks()
+    expect(fetchAutopilotApplied).toHaveBeenCalledTimes(2)
+  })
+
   it('hides Generate behind the plan callout when seo.inlineApply is off', async () => {
     render(<ProposalsTab planInfo={{ tier: 'starter', features: [], upgradeUrl: null }} />)
     await screen.findByText('Duplicate meta title')

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

@@ -2,10 +2,11 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { fireEvent, render, screen, waitFor, within } from '@testing-library/react'
 
-// The SEO defaults tab reads six shared backends: /seo/config (meta defaults +
+// The SEO defaults tab reads seven shared backends: /seo/config (meta defaults +
 // verification), /seo/crawl-settings (sitemap toggles), /seo/sitemap/status
 // (read-only summary), /seo/index-status (environment), /seo/search-console
-// (Search data card), and /seo/digest-settings (Weekly SEO digest card).
+// (Search data card), /seo/autopilot/settings (SEO Autopilot 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 }
@@ -80,6 +81,28 @@ async function defaultApi(
       status: 200,
     }
   }
+  if (endpoint === '/seo/autopilot/settings' && method === 'GET') {
+    return {
+      data: {
+        enabled: false,
+        enabledById: null,
+        pausedAt: null,
+        pausedReason: null,
+        types: {},
+        pausedTypes: {},
+        minBrandAlignment: 85,
+        redirectMinConfidence: 85,
+        maxPerRun: 5,
+        maxPerDay: 20,
+        verifyFailuresPerRun: 2,
+        outcomeWatch: { enabled: true, days: 14, dropRatio: 0.4, minPriorClicks: 50 },
+        lastRunAt: null,
+        lastRun: null,
+        governance: { issueFixApprovalRequired: false, redirectApprovalRequired: false },
+      },
+      status: 200,
+    }
+  }
   return { data: {}, status: 200 }
 }
 
@@ -98,6 +121,7 @@ vi.mock('sonner', () => ({
     success: (...args: unknown[]) => toastSuccess(...args),
     error: (...args: unknown[]) => toastError(...args),
     warning: vi.fn(),
+    info: vi.fn(),
     loading: vi.fn(),
   },
 }))
@@ -331,6 +355,23 @@ describe('SeoSettingsTab', () => {
     expect(body.serviceAccountKey).toContain('svc@acme.iam.gserviceaccount.com')
   })
 
+  it('renders the SEO Autopilot card between Search data and the digest', async () => {
+    render(<SeoSettingsTab role="ADMIN" />)
+    // The title also renders in the card's skeleton state; wait for a control
+    // that only exists once the settings have loaded.
+    const toggle = await screen.findByRole('switch', { name: 'Enable autopilot' })
+    expect(toggle.getAttribute('aria-checked')).toBe('false')
+    const regions = screen
+      .getAllByRole('region')
+      .map((r) => r.getAttribute('aria-labelledby'))
+      .map((id) => (id ? document.getElementById(id)?.textContent : null))
+    const order = ['Search data', 'SEO Autopilot', 'Weekly SEO digest'].map((t) =>
+      regions.indexOf(t),
+    )
+    expect(order.every((i) => i >= 0)).toBe(true)
+    expect(order).toEqual([...order].sort((a, b) => a - b))
+  })
+
   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()
@@ -342,12 +383,13 @@ describe('SeoSettingsTab', () => {
 
   it('saves recipients as a trimmed, comma-split list', async () => {
     render(<SeoSettingsTab role="ADMIN" />)
-    await screen.findByText('Weekly SEO digest')
+    // 'Weekly SEO digest' is also the loading-state title; wait for the form.
+    const save = await screen.findByRole('button', { name: 'Save recipients' })
 
     fireEvent.change(screen.getByLabelText('Recipients'), {
       target: { value: ' a@x.com , b@x.com ' },
     })
-    fireEvent.click(screen.getByRole('button', { name: 'Save recipients' }))
+    fireEvent.click(save)
 
     await waitFor(() =>
       expect(cmsApi).toHaveBeenCalledWith(
@@ -370,9 +412,7 @@ describe('SeoSettingsTab', () => {
       return defaultApi(endpoint, options)
     })
     render(<SeoSettingsTab role="ADMIN" />)
-    await screen.findByText('Weekly SEO digest')
-
-    fireEvent.click(screen.getByRole('button', { name: /Send test digest/ }))
+    fireEvent.click(await screen.findByRole('button', { name: /Send test digest/ }))
 
     await waitFor(() =>
       expect(toastError).toHaveBeenCalledWith('No completed SEO audit yet — run an audit first'),

package/src/lib/api.ts

@@ -62,10 +62,27 @@ function appendLocaleParam(endpoint: string): string {
   return `${endpoint}${separator}locale=${encodeURIComponent(locale)}`
 }
 
+export interface CmsApiResult<T> {
+  data?: T
+  error?: string
+  /**
+   * Machine-readable error discriminator (e.g. IP_LOCKOUT_RISK, conflict) set
+   * by guard responses so callers can branch without parsing the message.
+   */
+  code?: string
+  /**
+   * The parsed non-2xx body, for the few endpoints that return structured
+   * detail beyond `error` / `code` (e.g. the 409 `fields` list from
+   * `/seo/autopilot/applied/:id/revert`). Callers must narrow it themselves.
+   */
+  errorBody?: unknown
+  status: number
+}
+
 export async function cmsApi<T = unknown>(
   endpoint: string,
   options: RequestInit = {},
-): Promise<{ data?: T; error?: string; code?: string; status: number }> {
+): Promise<CmsApiResult<T>> {
   const url = `${basePath}${appendLocaleParam(endpoint)}`
   const isFormData = typeof FormData !== 'undefined' && options.body instanceof FormData
 
@@ -98,9 +115,8 @@ export async function cmsApi<T = unknown>(
     if (!res.ok) {
       return {
         error: json.error || `Request failed (${res.status})`,
-        // Machine-readable error discriminator (e.g. IP_LOCKOUT_RISK) set by
-        // guard responses so callers can branch without parsing the message.
         code: typeof json.code === 'string' ? json.code : undefined,
+        errorBody: json,
         status: res.status,
       }
     }