@prenta/admin 1.3.1 → 1.4.0

package/src/components/SEOConfigPanel.tsx

@@ -24,6 +24,7 @@ interface RobotsSettings {
 interface SiteSEO {
   siteUrl?: string
   siteName?: string
+  trailingSlash?: 'always' | 'never'
   defaultOgImage?: string
   twitterHandle?: string
   robots?: { blockAIBots?: boolean; disabled?: boolean }
@@ -254,6 +255,16 @@ export function SEOConfigPanel() {
             onChange={(v) => patchSite('siteUrl', v)}
             hint="Canonical origin used in sitemap URLs and JSON-LD."
           />
+          <Select
+            label="Trailing slashes"
+            value={site.trailingSlash ?? ''}
+            options={['', 'always', 'never']}
+            onChange={(v) =>
+              patchSite('trailingSlash', (v || undefined) as 'always' | 'never' | undefined)
+            }
+            placeholderLabel={`default: ${staticSite.trailingSlash ?? 'never'}`}
+            hint="Must match the public adapter. Sitemap, canonicals, href rewrite, link-health."
+          />
           <Field
             label="Site name"
             placeholder={staticSite.siteName ?? 'My Site'}

package/src/hooks/usePublicSiteUrl.ts

@@ -1,6 +1,7 @@
 'use client'
 
 import { useEffect, useState } from 'react'
+import type { TrailingSlashPolicy } from '@prenta/core/client'
 import { cmsApi } from '../lib/api.js'
 import { siteUrlMatchesAdminOrigin } from '../lib/preview-link.js'
 
@@ -9,6 +10,8 @@ export type PublicSiteUrlSource = 'api' | 'config' | 'none'
 export interface PublicSiteUrlState {
   /** Effective public apex (no trailing slash), ready for buildPublicUrl. */
   siteUrl: string
+  /** Public URL spelling from effective SEO config. */
+  trailingSlash: TrailingSlashPolicy
   loading: boolean
   source: PublicSiteUrlSource
   /**
@@ -21,6 +24,7 @@ export interface PublicSiteUrlState {
 interface PublicUrlApiData {
   siteUrl: string | null
   source: 'override' | 'config' | null
+  trailingSlash?: TrailingSlashPolicy | null
 }
 
 /**
@@ -35,6 +39,7 @@ interface PublicUrlApiData {
 export function usePublicSiteUrl(configFallback?: string | null): PublicSiteUrlState {
   const fallback = (configFallback ?? '').trim().replace(/\/+$/, '')
   const [apiUrl, setApiUrl] = useState<string | null>(null)
+  const [trailingSlash, setTrailingSlash] = useState<TrailingSlashPolicy>('never')
   const [loading, setLoading] = useState(true)
   const [source, setSource] = useState<PublicSiteUrlSource>(fallback ? 'config' : 'none')
 
@@ -45,6 +50,7 @@ export function usePublicSiteUrl(configFallback?: string | null): PublicSiteUrlS
       .then((res) => {
         if (cancelled) return
         const url = (res.data?.siteUrl ?? '').trim().replace(/\/+$/, '')
+        setTrailingSlash(res.data?.trailingSlash === 'always' ? 'always' : 'never')
         if (url) {
           setApiUrl(url)
           setSource('api')
@@ -70,6 +76,7 @@ export function usePublicSiteUrl(configFallback?: string | null): PublicSiteUrlS
   const siteUrl = apiUrl || fallback
   return {
     siteUrl,
+    trailingSlash,
     loading,
     source,
     matchesAdminOrigin: siteUrlMatchesAdminOrigin(siteUrl),

package/src/lib/open-public-site-tab.ts

@@ -1,4 +1,5 @@
 import { toast } from 'sonner'
+import type { TrailingSlashPolicy } from '@prenta/core/client'
 import { buildPublicPreviewUrl, buildPublicUrl, createPreviewToken } from './preview-link.js'
 import { NO_PUBLIC_SITE_URL_REASON, type ViewSiteIntent } from './view-site-chrome.js'
 
@@ -11,6 +12,7 @@ export async function openPublicSiteTab(opts: {
   path?: string | null
   urlPrefix?: string
   data?: Record<string, unknown> | null
+  trailingSlash?: TrailingSlashPolicy | null
   dirty: boolean
   saveDraft: () => Promise<boolean>
 }): Promise<void> {
@@ -23,6 +25,7 @@ export async function openPublicSiteTab(opts: {
     slug: opts.slug,
     path: opts.path,
     data: opts.data,
+    trailingSlash: opts.trailingSlash,
   }
 
   if (opts.intent === 'view-live') {

package/src/lib/preview-link.ts

@@ -1,4 +1,8 @@
-import { documentPublicPath } from '@prenta/core/client'
+import {
+  documentPublicPath,
+  resolveDocumentUrl,
+  type TrailingSlashPolicy,
+} from '@prenta/core/client'
 import { cmsApi } from './api.js'
 
 /**
@@ -40,6 +44,8 @@ export interface PublicUrlParts {
   path?: string | null
   /** Document `data` bag; `data.path` wins when `path` is omitted. */
   data?: Record<string, unknown> | null
+  /** Must match `seo.trailingSlash` so View live opens the adapter URL. */
+  trailingSlash?: TrailingSlashPolicy | null
 }
 
 export interface PublicPreviewUrlParts extends PublicUrlParts {
@@ -47,10 +53,12 @@ export interface PublicPreviewUrlParts extends PublicUrlParts {
   token: string
 }
 
-function absolutePublicUrl(siteUrl: string, path: string): string {
-  const base = (siteUrl ?? '').replace(/\/+$/, '')
-  if (path === '/') return base || '/'
-  return `${base}${path}`
+function absolutePublicUrl(
+  siteUrl: string,
+  path: string,
+  trailingSlash?: TrailingSlashPolicy | null,
+): string {
+  return resolveDocumentUrl({ siteUrl, path, trailingSlash })
 }
 
 /**
@@ -65,6 +73,7 @@ export function buildPublicPreviewUrl({
   token,
   path,
   data,
+  trailingSlash,
 }: PublicPreviewUrlParts): string {
   const publicPath = documentPublicPath({ urlPrefix, slug, path, data })
   // Home → `https://example.com/?preview=` (slash before query keeps relative
@@ -73,7 +82,7 @@ export function buildPublicPreviewUrl({
     const root = (siteUrl ?? '').replace(/\/+$/, '')
     return `${root || ''}/?preview=${encodeURIComponent(token)}`
   }
-  return `${absolutePublicUrl(siteUrl, publicPath)}?preview=${encodeURIComponent(token)}`
+  return `${absolutePublicUrl(siteUrl, publicPath, trailingSlash)}?preview=${encodeURIComponent(token)}`
 }
 
 /**
@@ -81,8 +90,15 @@ export function buildPublicPreviewUrl({
  * {@link buildPublicPreviewUrl} without a preview token. Home-style slugs and
  * `data.path` are handled by {@link documentPublicPath}.
  */
-export function buildPublicUrl({ siteUrl, urlPrefix, slug, path, data }: PublicUrlParts): string {
-  return absolutePublicUrl(siteUrl, documentPublicPath({ urlPrefix, slug, path, data }))
+export function buildPublicUrl({
+  siteUrl,
+  urlPrefix,
+  slug,
+  path,
+  data,
+  trailingSlash,
+}: PublicUrlParts): string {
+  return resolveDocumentUrl({ siteUrl, urlPrefix, slug, path, data, trailingSlash })
 }
 
 /**

package/src/lib/seo-service.ts

@@ -1114,6 +1114,8 @@ export interface LinkHealthIssue {
   url: string
   status: number
   type: string
+  kind?: 'broken' | 'redirected'
+  redirectTarget?: string
 }
 
 export interface LinkSuggestion {

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

@@ -218,7 +218,11 @@ export function PageSectionEditor({
 
   const { canEdit, canPublish, canCreateRedirect } = useMemo(() => deriveCan(session), [session])
   const customRenderers = useAdminSectionRenderers()
-  const { siteUrl: publicSiteUrl, matchesAdminOrigin } = usePublicSiteUrl(config?.seo?.siteUrl)
+  const {
+    siteUrl: publicSiteUrl,
+    trailingSlash,
+    matchesAdminOrigin,
+  } = usePublicSiteUrl(config?.seo?.siteUrl)
 
   // A misconfigured `pages` collection (missing the fields the editor
   // writes) would silently drop content on save — detect it up front.
@@ -748,13 +752,14 @@ export function PageSectionEditor({
         documentId: page.id,
         slug: page.slug,
         path: page.path,
+        trailingSlash,
         dirty,
         saveDraft: handleSaveDraft,
       }).catch((err) => {
         toast.error(err instanceof Error ? err.message : 'Failed to open the site')
       })
     },
-    [page, dirty, handleSaveDraft, publicSiteUrl],
+    [page, dirty, handleSaveDraft, publicSiteUrl, trailingSlash],
   )
 
   const handleShare = useCallback(async () => {

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

@@ -182,7 +182,11 @@ export function PostSectionEditor({
 
   const { canEdit, canPublish } = useMemo(() => deriveCan(session), [session])
   const customRenderers = useAdminSectionRenderers()
-  const { siteUrl: publicSiteUrl, matchesAdminOrigin } = usePublicSiteUrl(config?.seo?.siteUrl)
+  const {
+    siteUrl: publicSiteUrl,
+    trailingSlash,
+    matchesAdminOrigin,
+  } = usePublicSiteUrl(config?.seo?.siteUrl)
 
   const typeLabel = useMemo(() => {
     const col = config?.collections?.[postType]
@@ -610,13 +614,14 @@ export function PostSectionEditor({
         slug: post.slug,
         urlPrefix,
         data: { path: (post as { path?: string }).path },
+        trailingSlash,
         dirty,
         saveDraft: handleSaveDraft,
       }).catch((err) => {
         toast.error(err instanceof Error ? err.message : 'Failed to open the site')
       })
     },
-    [post, postType, urlPrefix, dirty, handleSaveDraft, publicSiteUrl],
+    [post, postType, urlPrefix, dirty, handleSaveDraft, publicSiteUrl, trailingSlash],
   )
 
   const handleShare = useCallback(async () => {

package/src/views/seo/LinksTab.tsx

@@ -413,7 +413,7 @@ export function LinksTab({ onNavigate }: { onNavigate?: (path: string) => void }
       {/* Live link health */}
       <SectionCard
         title="Live link health scan"
-        description="Probe outbound URLs (rate-limited). Complements audit broken-link issues."
+        description="Probe outbound URLs (rate-limited). Reports broken links and internal redirects (3xx) so trailing-slash hrefs can be rewritten to the canonical URL."
         action={
           <button
             type="button"
@@ -438,7 +438,13 @@ export function LinksTab({ onNavigate }: { onNavigate?: (path: string) => void }
                 <p className="text-muted-foreground mt-0.5 flex items-center gap-1 truncate text-xs">
                   <ExternalLink className="h-3 w-3 shrink-0" aria-hidden />
                   {issue.url}
-                  <span className="text-destructive ml-1">HTTP {issue.status || 'error'}</span>
+                  {issue.kind === 'redirected' ? (
+                    <span className="text-muted-foreground ml-1">
+                      HTTP {issue.status} → {issue.redirectTarget ?? 'canonical URL'}
+                    </span>
+                  ) : (
+                    <span className="text-destructive ml-1">HTTP {issue.status || 'error'}</span>
+                  )}
                   <span className="text-muted-foreground">· {issue.type}</span>
                 </p>
               </li>
@@ -447,7 +453,7 @@ export function LinksTab({ onNavigate }: { onNavigate?: (path: string) => void }
         ) : (
           <p className="text-muted-foreground text-sm">
             {healthIssues
-              ? 'No failing links in the last scan.'
+              ? 'No broken or redirecting links in the last scan.'
               : 'Run a scan to check live URL responses across published content.'}
           </p>
         )}

package/src/views/settings/SiteInformationCard.tsx

@@ -33,6 +33,7 @@ export function SiteInformationCard({
   const taglineId = useId()
   const urlId = useId()
   const urlErrId = useId()
+  const slashId = useId()
 
   const titleIssue = issues.find((i) => i.field === 'siteTitle')
   const urlIssue = issues.find((i) => i.field === 'siteUrl')
@@ -121,6 +122,28 @@ export function SiteInformationCard({
             this URL.
           </p>
         </div>
+
+        <div>
+          <label htmlFor={slashId} className={LABEL_CLASS}>
+            Trailing slashes
+          </label>
+          <select
+            id={slashId}
+            value={form.trailingSlash}
+            disabled={!canEdit}
+            onChange={(e) =>
+              setField('trailingSlash', e.target.value === 'always' ? 'always' : 'never')
+            }
+            className={INPUT_CLASS}
+          >
+            <option value="never">Never — /about (Next.js default)</option>
+            <option value="always">Always — /about/ (Astro trailingSlash: always)</option>
+          </select>
+          <p className="text-muted-foreground mt-1 text-sm">
+            Must match the public adapter. Drives sitemap, canonical / OG URLs, href rewrite on
+            save, and link-health.
+          </p>
+        </div>
       </div>
     </SettingsCard>
   )

package/src/views/settings/useGeneralSettings.ts

@@ -8,6 +8,7 @@ export interface GeneralSettingsForm {
   siteTitle: string
   tagline: string
   siteUrl: string
+  trailingSlash: 'always' | 'never'
   language: string
   timezone: string
   defaultNoIndex: boolean
@@ -59,6 +60,7 @@ const EMPTY_FORM: GeneralSettingsForm = {
   siteTitle: '',
   tagline: '',
   siteUrl: '',
+  trailingSlash: 'never',
   language: 'en',
   timezone: 'UTC',
   defaultNoIndex: false,
@@ -212,6 +214,7 @@ export function useGeneralSettings(canEdit: boolean): UseGeneralSettings {
           siteTitle: String(settingsData.siteTitle ?? ''),
           tagline: String(settingsData.tagline ?? ''),
           siteUrl: String(effectiveSite.siteUrl ?? settingsData.siteUrl ?? ''),
+          trailingSlash: effectiveSite.trailingSlash === 'always' ? 'always' : 'never',
           language: String(settingsData.language ?? 'en'),
           timezone: String(settingsData.timezone ?? 'UTC'),
           defaultNoIndex: robots.defaultNoIndex === true,
@@ -302,6 +305,7 @@ export function useGeneralSettings(canEdit: boolean): UseGeneralSettings {
       form.language !== baseline.language ||
       form.timezone !== baseline.timezone
     const siteUrlChanged = siteUrlEditable && form.siteUrl !== baseline.siteUrl
+    const trailingSlashChanged = form.trailingSlash !== baseline.trailingSlash
     const robotsChanged =
       form.defaultNoIndex !== baseline.defaultNoIndex ||
       form.defaultNoFollow !== baseline.defaultNoFollow ||
@@ -336,9 +340,10 @@ export function useGeneralSettings(canEdit: boolean): UseGeneralSettings {
 
       // 2. Site URL + robots defaults → the SHARED SEO config (merge over the
       //    current override site so SEO-tab fields aren't lost).
-      if (siteUrlChanged || robotsChanged) {
+      if (siteUrlChanged || trailingSlashChanged || robotsChanged) {
         const nextSite: Record<string, any> = { ...overrideSite }
         if (siteUrlChanged) nextSite.siteUrl = form.siteUrl.trim() || undefined
+        if (trailingSlashChanged) nextSite.trailingSlash = form.trailingSlash
         if (robotsChanged) {
           nextSite.robots = {
             ...(overrideSite.robots ?? {}),