@prenta/admin 1.10.1 → 1.12.0

package/src/__tests__/lib/page-editor-service.test.ts

@@ -1,4 +1,5 @@
 import { describe, expect, it } from 'vitest'
+import { registerSectionTypes, resetCustomSectionTypes } from '@prenta/core/page-sections'
 import {
   addSection,
   createSection,
@@ -116,6 +117,28 @@ describe('validatePage', () => {
     expect(errors.some((e) => e.includes('at least one visible section'))).toBe(true)
   })
 
+  it('treats hideable: false leftovers as visible for publish', () => {
+    registerSectionTypes([
+      {
+        id: 'am-fat',
+        name: 'Fat',
+        scope: ['page'],
+        hideable: false,
+        fields: [{ key: 'heading', label: 'Heading', type: 'text' }],
+      },
+    ])
+    try {
+      const leftover = {
+        ...createSection('am-fat'),
+        visible: false,
+      }
+      const { errors } = validatePage(basePage([leftover]), true)
+      expect(errors.some((e) => e.includes('at least one visible section'))).toBe(false)
+    } finally {
+      resetCustomSectionTypes()
+    }
+  })
+
   it('flags missing required section fields on publish', () => {
     const hero = createSection('hero')
     hero.content.heading = '' // required

package/src/__tests__/lib/seo-service-errors.test.ts

@@ -9,12 +9,17 @@ vi.mock('../../lib/api.js', () => ({
 
 const {
   applySeoInternalLink,
+  explainSeoIssue,
   fetchSeoOverview,
   fetchSeoContentRecords,
   fetchSeoIssues,
   fetchRedirectRules,
+  generate404RecoverySuggestions,
+  generateSeoField,
+  generateStructuredData,
   isInlineSeoFixIssue,
   runSeoAudit,
+  summarizeSeoAudit,
   updateSeoFields,
 } = await import('../../lib/seo-service.js')
 
@@ -105,6 +110,74 @@ describe('SEO service read fetchers surface errors', () => {
   })
 })
 
+/**
+ * Regression tests for the "swallowed governance errors" gap: the governed AI
+ * runtime returns crafted 429/403 messages (daily request cap, monthly budget,
+ * disabled feature) and the copilot helpers must surface them, not collapse to
+ * `{}` — which the UI renders as the misleading "No suggestion was returned."
+ * Unavailability (501/503) stays a distinct graceful state.
+ */
+describe('AI copilot helpers surface governance errors', () => {
+  const capMsg =
+    'The "SEO meta generation" feature reached its daily limit of 5 AI requests. Raise the cap in Settings > AI.'
+
+  it('generateSeoField returns the server message on 429', async () => {
+    cmsApi.mockResolvedValue({ error: capMsg, status: 429 })
+    await expect(
+      generateSeoField('metaTitle', { entityType: 'page', entityId: 'p1' }),
+    ).resolves.toEqual({ error: capMsg })
+  })
+
+  it('generateSeoField still maps 501/503 to { unavailable: true }', async () => {
+    cmsApi.mockResolvedValue({ error: 'AI provider is not configured', status: 503 })
+    await expect(
+      generateSeoField('metaTitle', { entityType: 'page', entityId: 'p1' }),
+    ).resolves.toEqual({ unavailable: true })
+  })
+
+  it('explainSeoIssue returns the server message on 429', async () => {
+    cmsApi.mockResolvedValue({ error: capMsg, status: 429 })
+    await expect(explainSeoIssue('issue-1')).resolves.toEqual({ error: capMsg })
+  })
+
+  it('generateStructuredData returns the server message on 403', async () => {
+    cmsApi.mockResolvedValue({ error: 'This AI feature is disabled.', status: 403 })
+    await expect(
+      generateStructuredData({ entityType: 'page', entityId: 'p1', schemaType: 'Article' }),
+    ).resolves.toEqual({ error: 'This AI feature is disabled.' })
+  })
+
+  it('summarizeSeoAudit returns the server message on 429', async () => {
+    cmsApi.mockResolvedValue({ error: capMsg, status: 429 })
+    await expect(summarizeSeoAudit()).resolves.toEqual({ error: capMsg })
+  })
+})
+
+describe('generate404RecoverySuggestions governance passthrough', () => {
+  it('surfaces governanceBlocked from the suggest response', async () => {
+    cmsApi.mockResolvedValue({
+      data: {
+        created: 3,
+        autoApplied: 0,
+        governanceBlocked: 'Auto-apply requires human approval.',
+      },
+      status: 200,
+    })
+    await expect(generate404RecoverySuggestions()).resolves.toEqual({
+      autoApplied: 0,
+      governanceBlocked: 'Auto-apply requires human approval.',
+    })
+  })
+
+  it('normalizes a null governanceBlocked to undefined', async () => {
+    cmsApi.mockResolvedValue({
+      data: { created: 1, autoApplied: 1, governanceBlocked: null },
+      status: 200,
+    })
+    await expect(generate404RecoverySuggestions()).resolves.toEqual({ autoApplied: 1 })
+  })
+})
+
 describe('inline SEO fix issue types', () => {
   it('treats orphan-page as an inline Approve issue', () => {
     expect(isInlineSeoFixIssue('orphan-page')).toBe(true)

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

@@ -19,14 +19,9 @@ const SETTINGS_RESPONSE = {
         enabled: true,
         category: 'content',
         requiredCapabilities: ['text'],
-      },
-      {
-        key: 'ai.features.redirectChainFlattening',
-        label: 'Redirect chain flattening',
-        description: 'Suggest single-hop redirects.',
-        enabled: false,
-        category: 'seo',
-        requiredCapabilities: ['text'],
+        approvalRequired: true,
+        allowBulkApply: false,
+        maxRequestsPerDay: 40,
       },
     ],
     budget: { monthlyTokenLimit: 1000000 },
@@ -249,6 +244,38 @@ describe('AISettingsTab', () => {
     expect(cw.modelId).toBe('anthropic:claude-haiku')
   })
 
+  it('renders per-feature governance controls and buffers edits into the save payload', async () => {
+    render(<AISettingsTab role="ADMIN" />)
+    await screen.findByText('AI Features')
+
+    const approval = screen.getByRole('switch', { name: 'Require approval' })
+    expect(screen.getByRole('switch', { name: 'Allow bulk apply' })).toBeTruthy()
+    const capInput = screen.getByLabelText('Max requests per day') as HTMLInputElement
+    expect(capInput.value).toBe('40')
+
+    fireEvent.click(approval)
+    fireEvent.change(capInput, { target: { value: '15' } })
+
+    const save = screen.getByRole('button', { name: 'Save Changes' }) as HTMLButtonElement
+    expect(save.disabled).toBe(false)
+    fireEvent.click(save)
+
+    await waitFor(() =>
+      expect(cmsApi).toHaveBeenCalledWith(
+        '/ai/settings',
+        expect.objectContaining({ method: 'PUT' }),
+      ),
+    )
+    const putCall = cmsApi.mock.calls.find(
+      ([e, o]) => e === '/ai/settings' && (o as { method?: string })?.method === 'PUT',
+    )
+    const body = JSON.parse((putCall![1] as { body: string }).body)
+    const cw = body.features.find((f: { key: string }) => f.key === 'ai.features.contentWriting')
+    expect(cw.approvalRequired).toBe(false)
+    expect(cw.allowBulkApply).toBe(false)
+    expect(cw.maxRequestsPerDay).toBe(15)
+  })
+
   it('warns when a feature would run on a model missing a required capability', async () => {
     settingsResponse = deepClone(SETTINGS_RESPONSE)
     // Alt-text needs vision; route it through the text-only Haiku model.
@@ -260,6 +287,9 @@ describe('AISettingsTab', () => {
       category: 'media',
       requiredCapabilities: ['vision'],
       modelId: 'anthropic:claude-haiku',
+      approvalRequired: true,
+      allowBulkApply: false,
+      maxRequestsPerDay: 100,
     } as (typeof SETTINGS_RESPONSE)['settings']['features'][number])
 
     render(<AISettingsTab role="ADMIN" />)

package/src/__tests__/views/redirects-pane.render.test.tsx

@@ -36,6 +36,9 @@ const fetchRedirectRules = vi.fn(async () => RULES)
 const createRedirectRule = vi.fn(async (_p: unknown) => ({}))
 const updateRedirectRule = vi.fn(async (_id: string, _p: unknown) => ({}))
 const deleteRedirectRule = vi.fn(async (_id: string) => ({}))
+const generate404RecoverySuggestions = vi.fn(
+  async (): Promise<{ error?: string; autoApplied?: number; governanceBlocked?: string }> => ({}),
+)
 
 vi.mock('../../lib/seo-service.js', () => ({
   fetchRedirectOverview: vi.fn(async () => ({
@@ -52,7 +55,7 @@ vi.mock('../../lib/seo-service.js', () => ({
   updateRedirectRule: (id: string, p: unknown) => updateRedirectRule(id, p),
   deleteRedirectRule: (id: string) => deleteRedirectRule(id),
   flattenRedirectChain: vi.fn(async () => ({})),
-  generate404RecoverySuggestions: vi.fn(async () => ({})),
+  generate404RecoverySuggestions: () => generate404RecoverySuggestions(),
   acceptRedirectSuggestion: vi.fn(async () => ({})),
   dismissRedirectSuggestion: vi.fn(async () => ({})),
   exportRedirectsCsv: vi.fn(async () => ({ csv: '' })),
@@ -65,7 +68,9 @@ vi.mock('../../lib/seo-service.js', () => ({
   saveRedirectRecoverySettings: vi.fn(async () => ({})),
 }))
 
-vi.mock('sonner', () => ({ toast: { success: vi.fn(), error: vi.fn(), loading: vi.fn() } }))
+vi.mock('sonner', () => ({
+  toast: { success: vi.fn(), error: vi.fn(), info: vi.fn(), loading: vi.fn() },
+}))
 
 const { RedirectsTab } = await import('../../views/seo/RedirectsTab.js')
 
@@ -162,3 +167,24 @@ describe('Redirects inline editor pane', () => {
     expect(createRedirectRule).not.toHaveBeenCalled()
   })
 })
+
+describe('AI 404 recovery governance feedback', () => {
+  it('explains why auto-apply was skipped when governance blocked it', async () => {
+    generate404RecoverySuggestions.mockResolvedValueOnce({
+      autoApplied: 0,
+      governanceBlocked:
+        'The "404 Recovery suggestions" feature requires human approval before AI results are applied.',
+    })
+    const { toast } = await import('sonner')
+
+    render(<RedirectsTab />)
+    fireEvent.click(await screen.findByRole('button', { name: /Generate suggestions/ }))
+
+    // Generation succeeded — suggestions exist — but the operator is told the
+    // governance reason instead of silently seeing nothing auto-applied.
+    await waitFor(() =>
+      expect(toast.info).toHaveBeenCalledWith(expect.stringContaining('requires human approval')),
+    )
+    expect(toast.success).toHaveBeenCalledWith('Generated 404 recovery suggestions.')
+  })
+})

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

@@ -2,9 +2,10 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { fireEvent, render, screen, waitFor, within } from '@testing-library/react'
 
-// The SEO defaults tab reads four shared backends: /seo/config (meta defaults +
+// The SEO defaults tab reads five shared backends: /seo/config (meta defaults +
 // verification), /seo/crawl-settings (sitemap toggles), /seo/sitemap/status
-// (read-only summary), and /seo/index-status (environment).
+// (read-only summary), /seo/index-status (environment), and
+// /seo/search-console (Search data card).
 const cmsApi = vi.fn(async (endpoint: string, options?: { method?: string }) => {
   const method = options?.method ?? 'GET'
   if (endpoint === '/seo/config' && method === 'GET') {
@@ -51,6 +52,21 @@ const cmsApi = vi.fn(async (endpoint: string, options?: { method?: string }) =>
   if (endpoint === '/seo/index-status' && method === 'GET') {
     return { data: { environment: 'preview' }, status: 200 }
   }
+  if (endpoint === '/seo/search-console' && method === 'GET') {
+    return {
+      data: {
+        propertyUrl: '',
+        hasServiceAccountKey: false,
+        serviceAccountEmail: null,
+        hasCruxApiKey: false,
+        connected: false,
+        lastSyncAt: null,
+        syncStatus: null,
+        suggestedProperty: 'sc-domain:acme.test',
+      },
+      status: 200,
+    }
+  }
   return { data: {}, status: 200 }
 })
 
@@ -247,4 +263,41 @@ describe('SeoSettingsTab', () => {
       true,
     )
   })
+
+  it('renders the Search data card in its not-connected state with gated actions', async () => {
+    render(<SeoSettingsTab role="ADMIN" />)
+    expect(await screen.findByText('Not connected')).toBeTruthy()
+
+    // Test/sync require a stored key or a working connection respectively.
+    expect(
+      (screen.getByRole('button', { name: /Test connection/ }) as HTMLButtonElement).disabled,
+    ).toBe(true)
+    expect((screen.getByRole('button', { name: /Sync now/ }) as HTMLButtonElement).disabled).toBe(
+      true,
+    )
+  })
+
+  it('saves Search Console credentials via PUT /seo/search-console', async () => {
+    render(<SeoSettingsTab role="ADMIN" />)
+    fireEvent.change(await screen.findByLabelText(/Search Console property/), {
+      target: { value: 'sc-domain:acme.test' },
+    })
+    fireEvent.change(screen.getByLabelText(/Service account key/), {
+      target: { value: '{"client_email":"svc@acme.iam.gserviceaccount.com","private_key":"k"}' },
+    })
+    fireEvent.click(screen.getByRole('button', { name: /Save credentials/ }))
+
+    await waitFor(() =>
+      expect(cmsApi).toHaveBeenCalledWith(
+        '/seo/search-console',
+        expect.objectContaining({ method: 'PUT' }),
+      ),
+    )
+    const putCall = cmsApi.mock.calls.find(
+      ([e, o]) => e === '/seo/search-console' && (o as { method?: string })?.method === 'PUT',
+    )
+    const body = JSON.parse((putCall![1] as { body: string }).body)
+    expect(body.propertyUrl).toBe('sc-domain:acme.test')
+    expect(body.serviceAccountKey).toContain('svc@acme.iam.gserviceaccount.com')
+  })
 })

package/src/components/seo/SeoCopilotDrawer.tsx

@@ -32,6 +32,10 @@ export function SeoCopilotDrawer({
       setUnavailable(true)
       return
     }
+    if (res.error) {
+      toast.error(res.error)
+      return
+    }
     if (res.text) setSummary(res.text)
     else toast.error('No summary was returned.')
   }

package/src/components/seo/SeoEditorPane.tsx

@@ -51,6 +51,7 @@ import { SeoAccordion, SeoAccordionBadge, SeoAccordionItem } from './SeoAccordio
 import { SeoAnalysisList } from './SeoAnalysisList.js'
 import { SeoInsightsMetrics } from './SeoInsightsMetrics.js'
 import { SeoLinkingSection } from './SeoLinkingSection.js'
+import { SeoSearchStats } from './SeoSearchStats.js'
 
 // Ranges the audit's title/description checks score against — one definition,
 // so the meter cannot tell an editor they are in range while the check fails.
@@ -340,6 +341,12 @@ export function SeoEditorPane({
         toast.error('AI is not configured. Add an AI provider to enable suggestions.')
         return
       }
+      if (result.error) {
+        // Governed refusal (daily cap, budget, disabled feature) — the server
+        // message says which limit was hit and where to raise it.
+        toast.error(result.error)
+        return
+      }
       if (result.text) {
         // Write the suggestion into the field the editor actually asked to
         // generate. Previously an OG title/description generate was remapped
@@ -988,6 +995,12 @@ export function SeoEditorPane({
                 <SeoInsightsMetrics readability={readability} plainText={plainText} />
               </SeoAccordionItem>
 
+              {!isDraft && entityId && (
+                <SeoAccordionItem value="search-performance" title="Search performance">
+                  <SeoSearchStats entityId={entityId} />
+                </SeoAccordionItem>
+              )}
+
               <SeoAccordionItem value="linking-data" title="Linking data">
                 <SeoLinkingSection
                   entityType={entityType}

package/src/components/seo/SeoIssueFixDrawer.tsx

@@ -116,6 +116,10 @@ export function SeoIssueFixDrawer({
       toast.error('AI is not configured.')
       return
     }
+    if (res.error) {
+      toast.error(res.error)
+      return
+    }
     setExplanation(res.text ?? 'No explanation available.')
   }
 

package/src/components/seo/SeoSearchStats.tsx

@@ -0,0 +1,183 @@
+'use client'
+
+/**
+ * Read-only search-performance block for the SEO editor pane: synced Google
+ * Search Console 28-day stats, top queries, and CrUX Core Web Vitals for one
+ * document. Purely informational — editing happens in the fields above it.
+ */
+import { useEffect, useState } from 'react'
+import { fetchSearchInsight, type SearchInsight } from '../../lib/seo-service.js'
+import { SeoLoading } from './primitives.js'
+
+function formatCompact(n: number): string {
+  if (n >= 1_000_000) return `${(n / 1_000_000).toFixed(1).replace(/\.0$/, '')}M`
+  if (n >= 1_000) return `${(n / 1_000).toFixed(1).replace(/\.0$/, '')}K`
+  return String(n)
+}
+
+function DeltaHint({
+  current,
+  previous,
+  lowerIsBetter = false,
+}: {
+  current: number | null
+  previous: number | null
+  lowerIsBetter?: boolean
+}) {
+  if (current == null || previous == null || previous === 0) return null
+  const diff = current - previous
+  if (diff === 0) return null
+  const improved = lowerIsBetter ? diff < 0 : diff > 0
+  return (
+    <span className={improved ? 'text-[var(--suc)]' : 'text-[var(--err)]'}>
+      {diff > 0 ? '+' : ''}
+      {Number.isInteger(diff) ? diff : diff.toFixed(1)}
+    </span>
+  )
+}
+
+function cwvTone(status: SearchInsight['cwvStatus']): { label: string; cls: string } {
+  switch (status) {
+    case 'good':
+      return { label: 'Good', cls: 'bg-[var(--suc-l)] text-[var(--suc)]' }
+    case 'needs-improvement':
+      return { label: 'Needs improvement', cls: 'bg-[var(--warn-l)] text-[var(--warn)]' }
+    case 'poor':
+      return { label: 'Poor', cls: 'bg-[var(--err-l)] text-[var(--err)]' }
+    default:
+      return { label: 'No field data', cls: 'bg-[var(--bg)] text-[var(--muted)]' }
+  }
+}
+
+export function SeoSearchStats({ entityId }: { entityId: string }) {
+  const [loading, setLoading] = useState(true)
+  const [connected, setConnected] = useState(false)
+  const [insight, setInsight] = useState<SearchInsight | null>(null)
+
+  useEffect(() => {
+    let alive = true
+    setLoading(true)
+    fetchSearchInsight(entityId)
+      .then((res) => {
+        if (!alive) return
+        setConnected(res.connected)
+        setInsight(res.insight)
+      })
+      .catch(() => {
+        if (!alive) return
+        setConnected(false)
+        setInsight(null)
+      })
+      .finally(() => alive && setLoading(false))
+    return () => {
+      alive = false
+    }
+  }, [entityId])
+
+  if (loading) return <SeoLoading />
+
+  if (!connected) {
+    return (
+      <p className="text-[11.5px] text-[var(--sub)]">
+        Connect Google Search Console in Settings → SEO → Search data to see real clicks,
+        impressions, and queries for this page.
+      </p>
+    )
+  }
+
+  if (!insight) {
+    return (
+      <p className="text-[11.5px] text-[var(--sub)]">
+        No search data synced for this page yet. Data appears after the next nightly sync once the
+        page has impressions in Google Search.
+      </p>
+    )
+  }
+
+  const cwv = cwvTone(insight.cwvStatus)
+
+  return (
+    <div className="space-y-3">
+      <div className="grid grid-cols-3 gap-2">
+        {[
+          {
+            label: 'Clicks',
+            value: formatCompact(insight.clicks28d),
+            delta: <DeltaHint current={insight.clicks28d} previous={insight.clicksPrev28d} />,
+          },
+          {
+            label: 'Impressions',
+            value: formatCompact(insight.impressions28d),
+            delta: (
+              <DeltaHint current={insight.impressions28d} previous={insight.impressionsPrev28d} />
+            ),
+          },
+          {
+            label: 'Avg. position',
+            value: insight.avgPosition28d != null ? insight.avgPosition28d.toFixed(1) : '—',
+            delta: (
+              <DeltaHint
+                current={insight.avgPosition28d}
+                previous={insight.avgPositionPrev28d}
+                lowerIsBetter
+              />
+            ),
+          },
+        ].map((stat) => (
+          <div key={stat.label} className="rounded-[7px] border border-[var(--bdr)] px-2.5 py-2">
+            <div className="text-[10.5px] text-[var(--muted)]">{stat.label}</div>
+            <div className="flex items-baseline gap-1.5">
+              <span className="text-[15px] font-semibold text-[var(--txt)]">{stat.value}</span>
+              <span className="text-[10.5px]">{stat.delta}</span>
+            </div>
+          </div>
+        ))}
+      </div>
+      <p className="text-[10.5px] text-[var(--muted)]">
+        Last 28 days vs the prior 28 — synced from Google Search Console.
+      </p>
+
+      {insight.topQueries.length > 0 && (
+        <div>
+          <p className="mb-1.5 text-[11px] font-medium text-[var(--sub)]">Top queries</p>
+          <table className="w-full text-[11.5px]" aria-label="Top search queries for this page">
+            <thead>
+              <tr className="text-left text-[10.5px] text-[var(--muted)]">
+                <th className="pb-1 font-medium">Query</th>
+                <th className="pb-1 text-right font-medium">Clicks</th>
+                <th className="pb-1 text-right font-medium">Pos.</th>
+              </tr>
+            </thead>
+            <tbody>
+              {insight.topQueries.slice(0, 8).map((q) => (
+                <tr key={q.query} className="border-t border-[var(--bdr)]">
+                  <td className="max-w-0 truncate py-1 pr-2 text-[var(--txt)]" title={q.query}>
+                    {q.query}
+                  </td>
+                  <td className="py-1 text-right text-[var(--sub)]">{formatCompact(q.clicks)}</td>
+                  <td className="py-1 text-right text-[var(--sub)]">{q.position.toFixed(1)}</td>
+                </tr>
+              ))}
+            </tbody>
+          </table>
+        </div>
+      )}
+
+      <div className="flex items-center justify-between gap-2 border-t border-[var(--bdr)] pt-2.5">
+        <span className="text-[11px] font-medium text-[var(--sub)]">
+          Core Web Vitals (real users)
+        </span>
+        <span className={`rounded-full px-2 py-0.5 text-[10.5px] font-medium ${cwv.cls}`}>
+          {cwv.label}
+        </span>
+      </div>
+      {insight.cwvStatus !== null && (
+        <div className="grid grid-cols-3 gap-2 text-[11px] text-[var(--sub)]">
+          <span>LCP {insight.lcpMs != null ? `${(insight.lcpMs / 1000).toFixed(1)}s` : '—'}</span>
+          <span>INP {insight.inpMs != null ? `${insight.inpMs}ms` : '—'}</span>
+          <span>CLS {insight.cls != null ? insight.cls : '—'}</span>
+        </div>
+      )}
+    </div>
+  )
+}

package/src/lib/editor-seo-content.ts

@@ -1,4 +1,8 @@
-import { getSectionType, type PageSection } from '@prenta/core/page-sections'
+import {
+  getSectionType,
+  isSectionEffectivelyVisible,
+  type PageSection,
+} from '@prenta/core/page-sections'
 import { analyzeContent } from '@prenta/core/seo/analysis'
 
 const HTML_LIKE = /<\/?[a-z][\s\S]*>/i
@@ -8,7 +12,7 @@ export function buildEditorContentHtml(sections: PageSection[], body = ''): stri
   const parts: string[] = []
   if (body.trim()) parts.push(body)
   for (const section of sections) {
-    if (section.visible === false) continue
+    if (!isSectionEffectivelyVisible(section)) continue
     const chunk = extractSectionContentHtml(section)
     if (chunk) parts.push(chunk)
   }

package/src/lib/page-editor-service.ts

@@ -32,6 +32,7 @@ import {
   updateSectionMeta,
   coerceSections as coerceSectionsCore,
   validateSectionContent,
+  isSectionEffectivelyVisible,
   isPageBuilderTree,
   layoutTreeToSections,
   type PageSection,
@@ -135,7 +136,7 @@ export function validatePage(page: EditorPage, forPublish = false): ValidationRe
   if (slugError) errors.push(slugError)
 
   if (forPublish) {
-    const visible = page.sections.filter((s) => s.visible)
+    const visible = page.sections.filter((s) => isSectionEffectivelyVisible(s))
     if (visible.length === 0) {
       errors.push('Add at least one visible section before publishing.')
     }

package/src/lib/post-editor-service.ts

@@ -40,6 +40,7 @@ import {
   coercePostHeader,
   createPostTemplate,
   validateSectionContent,
+  isSectionEffectivelyVisible,
   type PageSection,
   type PostHeaderConfig,
 } from '@prenta/core/page-sections'
@@ -164,7 +165,7 @@ export function validatePost(post: EditorPost, forPublish = false): ValidationRe
   if (slugError) errors.push(slugError)
 
   if (forPublish) {
-    const visible = post.sections.filter((s) => s.visible)
+    const visible = post.sections.filter((s) => isSectionEffectivelyVisible(s))
     // Unmanaged (externally-managed) sections have no registered field schema,
     // so they can't be content-validated here and must not block publish.
     for (const section of visible.filter((s) => !s.unmanaged)) {
@@ -187,7 +188,9 @@ export function validatePost(post: EditorPost, forPublish = false): ValidationRe
 export function validateTemplate(template: EditorPostTemplate): ValidationResult {
   const errors: string[] = []
   const warnings: string[] = []
-  for (const section of template.sections.filter((s) => s.visible && !s.unmanaged)) {
+  for (const section of template.sections.filter(
+    (s) => isSectionEffectivelyVisible(s) && !s.unmanaged,
+  )) {
     const def = getSectionType(section.sectionType)
     const label = section.name || def?.name || 'Section'
     const result = validateSectionContent(section.sectionType, section.content, section.settings)