@prenta/admin 0.97.5 → 0.98.1

package/src/__tests__/views/page-builder-seo-panel.render.test.tsx

@@ -0,0 +1,77 @@
+// @vitest-environment happy-dom
+import { describe, expect, it, vi } from 'vitest'
+import { render, screen } from '@testing-library/react'
+import type { PageNode } from '@prenta/core'
+import { BuilderSEOPanel } from '../../views/page-builder/SEOPanel.js'
+
+const tree = {
+  id: 'page-1',
+  type: 'page',
+  children: [
+    {
+      id: 'sec-1',
+      type: 'section',
+      settings: {},
+      children: [
+        {
+          id: 'row-1',
+          type: 'row',
+          settings: {},
+          children: [
+            {
+              id: 'col-1',
+              type: 'column',
+              settings: { width: 12 },
+              children: [
+                {
+                  id: 'blk-hero',
+                  type: 'block',
+                  settings: { blockType: 'hero' },
+                  data: {
+                    title: 'Welcome to Our Service',
+                    image: { url: '/hero.webp', alt: 'Hero banner' },
+                  },
+                },
+              ],
+            },
+          ],
+        },
+      ],
+    },
+  ],
+} as PageNode
+
+const pageSettings = {
+  title: 'Home',
+  slug: 'home',
+  metaTitle: 'Welcome to Our Service',
+  metaDescription: 'The public description.',
+}
+
+describe('BuilderSEOPanel', () => {
+  it('grades the live tree with the shared page audit panel and keeps meta fields', () => {
+    render(
+      <BuilderSEOPanel tree={tree} pageSettings={pageSettings} onPageSettingsChange={vi.fn()} />,
+    )
+
+    expect(screen.getByRole('img', { name: /SEO score:/i })).toBeTruthy()
+    expect(screen.getByRole('img', { name: /GEO score:/i })).toBeTruthy()
+    expect(screen.getByText('SEO checks')).toBeTruthy()
+    expect(screen.getByText('GEO checks')).toBeTruthy()
+    expect(screen.getByLabelText('Meta Title')).toBeTruthy()
+    expect(screen.queryByText('Readability')).toBeNull()
+    expect(screen.queryByText('Block Hints')).toBeNull()
+  })
+
+  it('keeps the empty-tree copy when the canvas has no children', () => {
+    render(
+      <BuilderSEOPanel
+        tree={{ id: 'page-1', type: 'page', children: [] } as PageNode}
+        pageSettings={pageSettings}
+        onPageSettingsChange={vi.fn()}
+      />,
+    )
+    expect(screen.getByText('Add content to your page to see SEO analysis')).toBeTruthy()
+    expect(screen.queryByRole('img', { name: /SEO score:/i })).toBeNull()
+  })
+})

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

@@ -0,0 +1,113 @@
+// @vitest-environment happy-dom
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { render, screen, waitFor } from '@testing-library/react'
+
+const baseRun = {
+  id: 'audit-1',
+  status: 'COMPLETED',
+  scope: 'full',
+  startedAt: '2026-08-11T12:00:00.000Z',
+  finishedAt: '2026-08-11T12:00:01.000Z',
+  siteScoreBefore: 70,
+  siteScoreAfter: 82,
+  pagesScanned: 10,
+  issuesFound: 3,
+  criticalCount: 0,
+  warningCount: 2,
+  infoCount: 1,
+  errorMessage: null,
+}
+
+let latestRun: Record<string, unknown> = { ...baseRun }
+
+const cmsApi = vi.fn(async (endpoint: string) => {
+  if (endpoint === '/seo/audits') {
+    return { data: { runs: [latestRun] }, status: 200 }
+  }
+  if (endpoint.startsWith('/seo/issues')) {
+    return { data: { issues: [] }, status: 200 }
+  }
+  return { data: {}, status: 200 }
+})
+
+vi.mock('../../lib/api.js', () => ({
+  cmsApi: (endpoint: string) => cmsApi(endpoint),
+}))
+
+vi.mock('sonner', () => ({
+  toast: { success: vi.fn(), error: vi.fn(), loading: vi.fn(), message: vi.fn() },
+}))
+
+const { AuditTab } = await import('../../views/seo/AuditTab.js')
+
+beforeEach(() => {
+  cmsApi.mockClear()
+  latestRun = { ...baseRun }
+})
+
+describe('AuditTab rendered audit coverage', () => {
+  it('shows safe coverage details and a warning for partial runs', async () => {
+    latestRun = {
+      ...baseRun,
+      metadata: {
+        renderedAudit: {
+          enabled: true,
+          originSource: 'override',
+          verified: 8,
+          unverified: 2,
+          notAttempted: 0,
+          partial: true,
+          bytesRead: 2048,
+          durationMs: 50,
+          reasons: [{ reason: 'timeout', count: 2 }],
+        },
+      },
+    }
+
+    render(<AuditTab />)
+
+    expect(await screen.findByText('8 verified')).toBeTruthy()
+    expect(screen.getByText('2 unverified')).toBeTruthy()
+    const warning = screen.getByRole('status')
+    expect(warning.textContent).toContain('The score covers 8 verified pages.')
+    expect(warning.textContent).toContain('their prior rendered-dependent issues were preserved.')
+    expect(screen.queryByText(/timeout/i)).toBeNull()
+    expect(screen.queryByText(/https?:\/\//i)).toBeNull()
+    expect(screen.queryByText(/transport error/i)).toBeNull()
+  })
+
+  it('does not show a partial warning for complete coverage', async () => {
+    latestRun = {
+      ...baseRun,
+      metadata: {
+        renderedAudit: {
+          enabled: true,
+          originSource: 'config',
+          verified: 10,
+          unverified: 0,
+          notAttempted: 0,
+          partial: false,
+          bytesRead: 4096,
+          durationMs: 40,
+          reasons: [],
+        },
+      },
+    }
+
+    render(<AuditTab />)
+
+    expect(await screen.findByText('10 verified')).toBeTruthy()
+    expect(screen.getByText('0 unverified')).toBeTruthy()
+    expect(screen.queryByText('Partial rendered audit')).toBeNull()
+    expect(screen.queryByRole('status')).toBeNull()
+  })
+
+  it('renders legacy audit runs without coverage or warnings', async () => {
+    render(<AuditTab />)
+
+    await waitFor(() => expect(screen.getByText('10')).toBeTruthy())
+    expect(screen.queryByText(/verified$/i)).toBeNull()
+    expect(screen.queryByText('Partial rendered audit')).toBeNull()
+    expect(screen.queryByRole('status')).toBeNull()
+  })
+})

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

@@ -0,0 +1,156 @@
+// @vitest-environment happy-dom
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { fireEvent, render, screen, waitFor } from '@testing-library/react'
+import { toast } from 'sonner'
+
+const initialCrawlSettings = {
+  indexEntireSite: true,
+  includeImagesInSitemap: true,
+  includePostsInSitemap: true,
+  sitemapEnabled: true,
+  autoRegenerateSitemapOnPublish: true,
+  pingSearchEnginesOnPublish: true,
+  auditRenderedHtml: true,
+}
+
+let crawlSettings = { ...initialCrawlSettings }
+let failNextPut = false
+let pendingPut: Promise<{ data?: unknown; error?: string }> | null = null
+
+const cmsApi = vi.fn(async (endpoint: string, options?: RequestInit) => {
+  const method = options?.method ?? 'GET'
+  if (endpoint === '/seo/crawl-settings' && method === 'GET') {
+    return { data: { ...crawlSettings }, status: 200 }
+  }
+  if (endpoint === '/seo/crawl-settings' && method === 'PUT') {
+    if (pendingPut) await pendingPut
+    if (failNextPut) {
+      failNextPut = false
+      return { error: 'Unable to save crawl settings.', status: 500 }
+    }
+    crawlSettings = {
+      ...crawlSettings,
+      ...(JSON.parse(String(options?.body)) as Partial<typeof initialCrawlSettings>),
+    }
+    return { data: { ...crawlSettings }, status: 200 }
+  }
+  if (endpoint === '/seo/sitemap/status') {
+    return {
+      data: {
+        sitemapUrl: 'https://example.test/sitemap.xml',
+        lastGeneratedAt: null,
+        pagesIncluded: 10,
+        pagesExcluded: 0,
+        autoRegenerateOnPublish: true,
+        errors: [],
+        warnings: [],
+      },
+      status: 200,
+    }
+  }
+  if (endpoint === '/seo/robots') {
+    return { data: { body: 'User-agent: *\nAllow: /', generated: true }, status: 200 }
+  }
+  if (endpoint === '/ai-crawlers') {
+    return {
+      data: { enabled: false, rows: [], totals: { hits: 0, bots: 0, honeypot: 0 } },
+      status: 200,
+    }
+  }
+  return { data: {}, status: 200 }
+})
+
+vi.mock('../../lib/api.js', () => ({
+  cmsApi: (endpoint: string, options?: RequestInit) => cmsApi(endpoint, options),
+}))
+
+vi.mock('sonner', () => ({
+  toast: { success: vi.fn(), error: vi.fn(), loading: vi.fn(), message: vi.fn() },
+}))
+
+const { TechnicalTab } = await import('../../views/seo/TechnicalTab.js')
+
+beforeEach(() => {
+  cmsApi.mockClear()
+  vi.mocked(toast.error).mockClear()
+  crawlSettings = { ...initialCrawlSettings }
+  failNextPut = false
+  pendingPut = null
+})
+
+describe('TechnicalTab rendered audit setting', () => {
+  it('renders the accessible setting and persists changes', async () => {
+    render(<TechnicalTab />)
+
+    const renderedAudit = await screen.findByRole('switch', {
+      name: /Audit rendered public HTML/i,
+      description: 'Fetch public page output so audits grade crawler-visible headings and content.',
+    })
+    expect(renderedAudit.getAttribute('aria-checked')).toBe('true')
+
+    fireEvent.click(renderedAudit)
+
+    await waitFor(() =>
+      expect(cmsApi).toHaveBeenCalledWith(
+        '/seo/crawl-settings',
+        expect.objectContaining({
+          method: 'PUT',
+          body: JSON.stringify({ auditRenderedHtml: false }),
+        }),
+      ),
+    )
+  })
+
+  it('disables crawl switches while a setting is saving', async () => {
+    let resolvePut: (() => void) | undefined
+    pendingPut = new Promise((resolve) => {
+      resolvePut = () => resolve({})
+    })
+    render(<TechnicalTab />)
+
+    fireEvent.click(await screen.findByRole('switch', { name: /Audit rendered public HTML/i }))
+
+    await waitFor(() => {
+      for (const toggle of screen.getAllByRole('switch')) {
+        expect((toggle as HTMLButtonElement).disabled).toBe(true)
+      }
+    })
+
+    resolvePut?.()
+    await waitFor(() =>
+      expect(
+        (
+          screen.getByRole('switch', {
+            name: /Audit rendered public HTML/i,
+          }) as HTMLButtonElement
+        ).disabled,
+      ).toBe(false),
+    )
+  })
+
+  it('reports save failures and refetches the server value', async () => {
+    failNextPut = true
+    render(<TechnicalTab />)
+
+    const renderedAudit = await screen.findByRole('switch', {
+      name: /Audit rendered public HTML/i,
+    })
+    const getCallsBeforeSave = cmsApi.mock.calls.filter(
+      ([endpoint, options]) =>
+        endpoint === '/seo/crawl-settings' &&
+        ((options as RequestInit | undefined)?.method ?? 'GET') === 'GET',
+    ).length
+
+    fireEvent.click(renderedAudit)
+
+    await waitFor(() => expect(toast.error).toHaveBeenCalledWith('Unable to save crawl settings.'))
+    await waitFor(() => expect(renderedAudit.getAttribute('aria-checked')).toBe('true'))
+    expect(
+      cmsApi.mock.calls.filter(
+        ([endpoint, options]) =>
+          endpoint === '/seo/crawl-settings' &&
+          ((options as RequestInit | undefined)?.method ?? 'GET') === 'GET',
+      ).length,
+    ).toBeGreaterThan(getCallsBeforeSave)
+  })
+})

package/src/lib/page-audit-input.ts

@@ -1,5 +1,5 @@
 import { resolveRenderedMetaTitle } from '@prenta/core/client'
-import type { PageSection } from '@prenta/core/page-sections'
+import { layoutTreeToSections, type PageSection } from '@prenta/core/page-sections'
 import type { PageAuditInput } from '@prenta/core/seo/page-audit'
 import type { SeoContentRecord } from './seo-service.js'
 
@@ -102,3 +102,38 @@ export function metaFromRecord(record: SeoContentRecord): AuditMetaSource {
     hasOgImage: Boolean(record.ogImage),
   }
 }
+
+export interface BuilderAuditSource {
+  tree: unknown
+  pageSettings: {
+    title: string
+    slug: string
+    metaTitle?: string
+    metaDescription?: string
+    ogImage?: string
+    focusKeyphrase?: string
+    schemaType?: string
+  }
+}
+
+export function buildPageAuditInputFromBuilder({
+  tree,
+  pageSettings,
+}: BuilderAuditSource): PageAuditInput {
+  return buildPageAuditInput({
+    sections: layoutTreeToSections(tree),
+    meta: {
+      metaTitle: pageSettings.metaTitle ?? '',
+      metaDescription: pageSettings.metaDescription ?? '',
+      focusKeyword: pageSettings.focusKeyphrase ?? '',
+      structuredDataType: pageSettings.schemaType ?? '',
+      noindex: false,
+      nosnippet: false,
+      llmsInclude: 'AUTO',
+      hasOgImage: Boolean(pageSettings.ogImage),
+    },
+    slug: pageSettings.slug,
+    record: null,
+    fallbackTitle: pageSettings.title,
+  })
+}

package/src/lib/seo-service.ts

@@ -299,6 +299,18 @@ export function applyContentSeoQuery(
 
 export type SeoAuditScope = 'full' | 'content' | 'technical' | 'redirect'
 
+export interface RenderedAuditSummary {
+  enabled: boolean
+  originSource: 'override' | 'config' | null
+  verified: number
+  unverified: number
+  notAttempted: number
+  partial: boolean
+  bytesRead: number
+  durationMs: number
+  reasons: Array<{ reason: string; count: number }>
+}
+
 export interface SeoAuditRun {
   id: string
   status: 'RUNNING' | 'COMPLETED' | 'FAILED'
@@ -313,12 +325,27 @@ export interface SeoAuditRun {
   warningCount: number
   infoCount: number
   errorMessage: string | null
+  metadata: {
+    renderedAudit?: RenderedAuditSummary
+  } | null
+}
+
+export interface SeoAuditRunResponse {
+  id: string
+  status: 'COMPLETED'
+  siteScoreAfter: number
+  issuesFound: number
+  criticalCount: number
+  warningCount: number
+  infoCount: number
+  pagesScanned: number
+  renderedAudit: RenderedAuditSummary
 }
 
 export async function runSeoAudit(
   scope: SeoAuditScope = 'full',
-): Promise<{ run?: SeoAuditRun; error?: string }> {
-  const res = await cmsApi<SeoAuditRun>('/seo/audit', {
+): Promise<{ run?: SeoAuditRunResponse; error?: string }> {
+  const res = await cmsApi<SeoAuditRunResponse>('/seo/audit', {
     method: 'POST',
     body: JSON.stringify({ scope }),
   })
@@ -662,6 +689,7 @@ export interface CrawlSettings {
   sitemapEnabled: boolean
   autoRegenerateSitemapOnPublish: boolean
   pingSearchEnginesOnPublish: boolean
+  auditRenderedHtml: boolean
 }
 
 export async function fetchSitemapStatus(): Promise<SitemapStatus | null> {

package/src/views/page-builder/ContextPanel.tsx

@@ -132,7 +132,6 @@ export function ContextPanel({
             tree={tree}
             pageSettings={pageSettings}
             onPageSettingsChange={onPageSettingsChange}
-            selectedNodeId={selectedNode?.id ?? null}
           />
         </Tabs.Content>