@prenta/admin 1.6.1 → 1.7.0

package/src/__tests__/lib/collection-routes.test.ts

@@ -1,6 +1,10 @@
 import { describe, expect, it } from 'vitest'
 
-import { editPathForDoc, newPathForCollection } from '../../lib/collection-routes.js'
+import {
+  editPathForDoc,
+  newPathForCollection,
+  sourceEditPath,
+} from '../../lib/collection-routes.js'
 
 describe('editPathForDoc', () => {
   it('routes post-type documents to the section editor (not the generic route)', () => {
@@ -22,6 +26,14 @@ describe('editPathForDoc', () => {
   })
 })
 
+describe('sourceEditPath', () => {
+  it('routes a blog source through CollectionRouteMeta so posts open the post editor', () => {
+    expect(sourceEditPath('blog', 'src-2')).toBe('/posts/blog/src-2/edit')
+    expect(sourceEditPath('pages', 'abc')).toBe('/pages/abc/edit')
+    expect(sourceEditPath(undefined, 'abc')).toBe('/pages/abc/edit')
+  })
+})
+
 describe('newPathForCollection', () => {
   it('routes new post documents to the post editor', () => {
     expect(newPathForCollection({ type: 'post' }, 'case-studies')).toBe('/posts/case-studies/new')

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

@@ -160,6 +160,14 @@ describe('validatePage', () => {
     const { errors } = validatePage(page, false)
     expect(errors.some((e) => /reserved/i.test(e))).toBe(true)
   })
+
+  it('accepts a nested slash slug when path is set', () => {
+    const page = basePage([])
+    page.slug = 'help/faq'
+    page.path = '/help/faq'
+    const { errors } = validatePage(page, false)
+    expect(errors.some((e) => /slug/i.test(e))).toBe(false)
+  })
 })
 
 describe('section type registry', () => {

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

@@ -8,10 +8,12 @@ vi.mock('../../lib/api.js', () => ({
 }))
 
 const {
+  applySeoInternalLink,
   fetchSeoOverview,
   fetchSeoContentRecords,
   fetchSeoIssues,
   fetchRedirectRules,
+  isInlineSeoFixIssue,
   runSeoAudit,
   updateSeoFields,
 } = await import('../../lib/seo-service.js')
@@ -71,6 +73,33 @@ describe('SEO service read fetchers surface errors', () => {
       error: 'forbidden',
     })
   })
+
+  it('applySeoInternalLink POSTs the apply body and returns { error } on failure', async () => {
+    cmsApi.mockResolvedValue({ error: 'Not wrappable', status: 422 })
+    await expect(
+      applySeoInternalLink({
+        sourceEntityId: 'src-1',
+        targetEntityId: 'tgt-1',
+        anchorText: 'RankMath',
+        fingerprint: 'fp-1',
+      }),
+    ).resolves.toEqual({ error: 'Not wrappable' })
+    expect(cmsApi).toHaveBeenCalledWith('/seo/links/apply', {
+      method: 'POST',
+      body: JSON.stringify({
+        sourceEntityId: 'src-1',
+        targetEntityId: 'tgt-1',
+        anchorText: 'RankMath',
+        fingerprint: 'fp-1',
+      }),
+    })
+  })
+})
+
+describe('inline SEO fix issue types', () => {
+  it('treats orphan-page as an inline Approve issue', () => {
+    expect(isInlineSeoFixIssue('orphan-page')).toBe(true)
+  })
 })
 
 describe('SEO audit mutation contract', () => {

package/src/__tests__/lib/workflow-chrome.test.ts

@@ -77,6 +77,11 @@ describe('resolveWorkflowChrome', () => {
     expect(chrome.showRequestChanges).toBe(false)
     expect(chrome.showSchedule).toBe(false)
     expect(chrome.statusLabel).toBe('Published')
+    expect(chrome.saveLabel).toBe('Update live')
+  })
+
+  it('labels the write-through save as Save Draft on unpublished pages', () => {
+    expect(resolveWorkflowChrome({ ...draft, role: 'EDITOR' }).saveLabel).toBe('Save Draft')
   })
 
   it('shows Scheduled when status is SCHEDULED', () => {

package/src/__tests__/views/editor-top-bar.render.test.tsx

@@ -191,6 +191,12 @@ describe('EditorTopBar view site / share', () => {
     expect(screen.queryByRole('button', { name: 'View live' })).toBeNull()
     expect(screen.queryByRole('button', { name: 'Share' })).toBeNull()
   })
+
+  it('labels the save button Update live on a published page', () => {
+    render(<EditorTopBar {...baseProps} status="PUBLISHED" dirty />)
+    expect(screen.getByRole('button', { name: 'Update live' })).toBeTruthy()
+    expect(screen.queryByRole('button', { name: 'Save Draft' })).toBeNull()
+  })
 })
 
 describe('EditorTopBar history button', () => {

package/src/__tests__/views/seo-content-pane.render.test.tsx

@@ -69,6 +69,7 @@ vi.mock('../../lib/seo-service.js', () => ({
   updateSeoFields: (t: string, i: string, p: unknown) => updateSeoFields(t, i, p),
   generateSeoField: vi.fn(async () => ({ unavailable: true })),
   fetchLinkSuggestions: vi.fn(async () => ({ data: { suggestions: [] } })),
+  applySeoInternalLink: vi.fn(async () => ({})),
   SEO_SCHEMA_TYPES: ['Article', 'WebPage'],
   applyContentSeoQuery: (
     records: Array<{ entityType: string; title: string }>,

package/src/components/seo/SeoEditorPane.tsx

@@ -162,6 +162,7 @@ export interface SeoEditorPaneProps {
   title?: string
   bodyHtml?: string
   featuredImage?: string
+  onNavigate?: (path: string) => void
 }
 
 export function SeoEditorPane({
@@ -178,6 +179,7 @@ export function SeoEditorPane({
   title,
   bodyHtml,
   featuredImage,
+  onNavigate,
 }: SeoEditorPaneProps) {
   const isDraft = !entityId
   const [record, setRecord] = useState<SeoContentRecord | null>(null)
@@ -994,6 +996,7 @@ export function SeoEditorPane({
                   externalLinks={Number(externalLinks)}
                   internalLinkOk={internalOk}
                   externalLinkOk={externalOk}
+                  onNavigate={onNavigate}
                 />
               </SeoAccordionItem>
             </SeoAccordion>

package/src/components/seo/SeoIssueFixPanel.tsx

@@ -24,6 +24,38 @@ import { hasPlanFeature, type PlanInfo } from '../../lib/plan.js'
 import { PlanUpgradeCallout } from '../PlanUpgradeCallout.js'
 import { SeoErrorState, btnPrimary, btnSecondary } from './primitives.js'
 
+function approveButtonLabel(strategy: SeoIssueFixSuggestion['fixStrategy']): string {
+  switch (strategy) {
+    case 'fix-link':
+      return 'Fix link in content'
+    case 'insert-link':
+      return 'Approve link'
+    case 'redirect':
+    case undefined:
+      return 'Approve redirect'
+    default: {
+      const _exhaustive: never = strategy
+      return _exhaustive
+    }
+  }
+}
+
+function approveSuccessToast(strategy: SeoIssueFixSuggestion['fixStrategy']): string {
+  switch (strategy) {
+    case 'fix-link':
+      return 'Link updated in content and issue resolved.'
+    case 'insert-link':
+      return 'Link added and issue resolved.'
+    case 'redirect':
+    case undefined:
+      return 'Redirect created and issue resolved.'
+    default: {
+      const _exhaustive: never = strategy
+      return _exhaustive
+    }
+  }
+}
+
 function formatValue(value: string | boolean | null): string {
   if (value === null || value === '') return '—'
   if (typeof value === 'boolean') return value ? 'Yes' : 'No'
@@ -321,11 +353,7 @@ export function SeoIssueFixPanel({
       toast.error(res.error)
       return
     }
-    toast.success(
-      suggestion?.fixStrategy === 'fix-link'
-        ? 'Link updated in content and issue resolved.'
-        : 'Redirect created and issue resolved.',
-    )
+    toast.success(approveSuccessToast(suggestion?.fixStrategy))
     onChanged?.()
   }
 
@@ -491,11 +519,7 @@ export function SeoIssueFixPanel({
                     disabled={busy}
                   >
                     <Check className="h-4 w-4" aria-hidden />
-                    {busy
-                      ? 'Applying…'
-                      : suggestion.fixStrategy === 'fix-link'
-                        ? 'Fix link in content'
-                        : 'Approve redirect'}
+                    {busy ? 'Applying…' : approveButtonLabel(suggestion.fixStrategy)}
                   </button>
                 ) : inlineFix && suggestion?.autoFixable && !inlineApplyEnabled ? (
                   <PlanUpgradeCallout feature="seo.inlineApply" upgradeUrl={planInfo?.upgradeUrl} />

package/src/components/seo/SeoLinkingSection.tsx

@@ -2,7 +2,21 @@
 
 import { useEffect, useState } from 'react'
 import { Link2, Loader2, Sparkles } from 'lucide-react'
-import { fetchLinkSuggestions, type LinkSuggestion } from '../../lib/seo-service.js'
+import { toast } from 'sonner'
+import { sourceEditPath } from '../../lib/collection-routes.js'
+import {
+  applySeoInternalLink,
+  fetchLinkSuggestions,
+  type LinkSuggestion,
+} from '../../lib/seo-service.js'
+import { fetchPlanInfo, hasPlanFeature, type PlanInfo } from '../../lib/plan.js'
+import { PlanUpgradeCallout } from '../PlanUpgradeCallout.js'
+
+const approveBtnCls =
+  'inline-flex items-center justify-center gap-1.5 rounded-[7px] bg-primary px-2.5 py-1.5 text-[11.5px] font-medium text-primary-foreground transition-colors hover:opacity-90 focus-visible:ring-2 focus-visible:ring-[var(--ring)] focus-visible:outline-none disabled:cursor-not-allowed disabled:opacity-50'
+
+const editorBtnCls =
+  'inline-flex items-center justify-center gap-1.5 rounded-[7px] border border-[var(--bdr)] bg-[var(--card)] px-2.5 py-1.5 text-[11.5px] font-medium text-[var(--txt)] transition-colors hover:bg-[var(--acc-l)] focus-visible:ring-2 focus-visible:ring-[var(--ring)] focus-visible:outline-none'
 
 export function SeoLinkingSection({
   entityType,
@@ -11,6 +25,8 @@ export function SeoLinkingSection({
   externalLinks,
   internalLinkOk,
   externalLinkOk,
+  onNavigate,
+  planInfo: planInfoProp,
 }: {
   entityType: 'page' | 'post'
   entityId: string | null
@@ -18,10 +34,31 @@ export function SeoLinkingSection({
   externalLinks: number
   internalLinkOk: boolean
   externalLinkOk: boolean
+  onNavigate?: (path: string) => void
+  /** When omitted (page/post editors), fetched on mount. */
+  planInfo?: PlanInfo | null
 }) {
   const [loading, setLoading] = useState(false)
   const [suggestions, setSuggestions] = useState<LinkSuggestion[]>([])
   const [error, setError] = useState<string | null>(null)
+  const [applyingKey, setApplyingKey] = useState<string | null>(null)
+  const [planInfo, setPlanInfo] = useState<PlanInfo | null | undefined>(planInfoProp)
+
+  useEffect(() => {
+    if (planInfoProp !== undefined) {
+      setPlanInfo(planInfoProp)
+      return
+    }
+    let alive = true
+    void fetchPlanInfo().then((info) => {
+      if (alive) setPlanInfo(info)
+    })
+    return () => {
+      alive = false
+    }
+  }, [planInfoProp])
+
+  const inlineApplyEnabled = hasPlanFeature(planInfo, 'seo.inlineApply')
 
   useEffect(() => {
     if (!entityId) {
@@ -48,6 +85,33 @@ export function SeoLinkingSection({
     }
   }, [entityType, entityId])
 
+  async function handleApprove(suggestion: LinkSuggestion) {
+    if (!entityId) return
+    const key = `${suggestion.sourceEntityId}-${suggestion.fingerprint}`
+    setApplyingKey(key)
+    const res = await applySeoInternalLink({
+      sourceEntityId: suggestion.sourceEntityId,
+      targetEntityId: entityId,
+      anchorText: suggestion.anchorText,
+      fingerprint: suggestion.fingerprint,
+    })
+    setApplyingKey(null)
+    if (res.error) {
+      toast.error(res.error)
+      return
+    }
+    toast.success('Link added.')
+    setSuggestions((prev) =>
+      prev.filter(
+        (row) =>
+          !(
+            row.fingerprint === suggestion.fingerprint &&
+            row.sourceEntityId === suggestion.sourceEntityId
+          ),
+      ),
+    )
+  }
+
   return (
     <div className="space-y-4">
       <div className="grid grid-cols-2 gap-3">
@@ -90,22 +154,56 @@ export function SeoLinkingSection({
             <Sparkles className="h-3.5 w-3.5 text-[var(--acc)]" aria-hidden />
             Link opportunities
           </div>
+          {!inlineApplyEnabled && suggestions.some((s) => s.wrappable) ? (
+            <div className="mb-2">
+              <PlanUpgradeCallout feature="seo.inlineApply" upgradeUrl={planInfo?.upgradeUrl} />
+            </div>
+          ) : null}
           <ul className="space-y-2">
-            {suggestions.map((s) => (
-              <li
-                key={`${s.sourceEntityId}-${s.anchorText}`}
-                className="rounded-[7px] border border-[var(--bdr)] bg-[var(--card)] p-2.5"
-              >
-                <div className="text-[12.5px] font-medium text-[var(--txt)]">{s.sourceTitle}</div>
-                <p className="mt-0.5 text-[11px] text-[var(--muted)]">{s.sourceUrl}</p>
-                <p className="mt-1 text-[11.5px] text-[var(--sub)]">
-                  Suggested anchor: <span className="text-[var(--txt)]">{s.anchorText}</span>
-                </p>
-                {s.excerpt && (
-                  <p className="mt-1 line-clamp-2 text-[11px] text-[var(--muted)]">{s.excerpt}</p>
-                )}
-              </li>
-            ))}
+            {suggestions.map((s) => {
+              const key = `${s.sourceEntityId}-${s.fingerprint || s.anchorText}`
+              const busy = applyingKey === key
+              return (
+                <li
+                  key={key}
+                  className="rounded-[7px] border border-[var(--bdr)] bg-[var(--card)] p-2.5"
+                >
+                  <div className="text-[12.5px] font-medium text-[var(--txt)]">{s.sourceTitle}</div>
+                  <p className="mt-0.5 text-[11px] text-[var(--muted)]">{s.sourceUrl}</p>
+                  <p className="mt-1 text-[11.5px] text-[var(--sub)]">
+                    Suggested anchor: <span className="text-[var(--txt)]">{s.anchorText}</span>
+                  </p>
+                  {s.excerpt && (
+                    <p className="mt-1 line-clamp-2 text-[11px] text-[var(--muted)]">{s.excerpt}</p>
+                  )}
+                  {s.wrappable && inlineApplyEnabled ? (
+                    <div className="mt-2">
+                      <button
+                        type="button"
+                        className={approveBtnCls}
+                        disabled={busy}
+                        onClick={() => void handleApprove(s)}
+                      >
+                        {busy ? <Loader2 className="h-3.5 w-3.5 animate-spin" aria-hidden /> : null}
+                        Approve link
+                      </button>
+                    </div>
+                  ) : onNavigate ? (
+                    <div className="mt-2">
+                      <button
+                        type="button"
+                        className={editorBtnCls}
+                        onClick={() =>
+                          onNavigate(sourceEditPath(s.sourceCollection, s.sourceEntityId))
+                        }
+                      >
+                        Open in editor
+                      </button>
+                    </div>
+                  ) : null}
+                </li>
+              )
+            })}
           </ul>
         </div>
       ) : (
@@ -118,8 +216,8 @@ export function SeoLinkingSection({
       <div className="flex items-start gap-2 rounded-[7px] border border-[var(--bdr)] bg-[var(--bg)] p-2.5 text-[11px] text-[var(--sub)]">
         <Link2 className="mt-0.5 h-3.5 w-3.5 shrink-0 text-[var(--acc)]" aria-hidden />
         <span>
-          Best practice: mix internal links (site structure + crawl paths) with selective outbound
-          links (trust signals). We flag counts here; apply links in the content editor.
+          Wrappable suggestions can be approved here. Others need an editor pass so we do not invent
+          a sentence.
         </span>
       </div>
     </div>

package/src/lib/collection-routes.ts

@@ -24,6 +24,19 @@ export function editPathForDoc(
   return `/${collection}/${id}`
 }
 
+/** Editor route for an inbound-link suggestion source document. */
+export function sourceEditPath(
+  sourceCollection: string | undefined,
+  sourceEntityId: string,
+): string {
+  const collection = sourceCollection ?? 'pages'
+  return editPathForDoc(
+    { type: collection === 'pages' ? 'page' : 'post' },
+    collection,
+    sourceEntityId,
+  )
+}
+
 /** The route that starts a new document in the correct editor. */
 export function newPathForCollection(
   col: CollectionRouteMeta | undefined,

package/src/lib/seo-service.ts

@@ -417,7 +417,7 @@ export interface SeoFixFieldChange {
 export interface SeoIssueFixSuggestion {
   autoFixable: boolean
   source?: 'deterministic' | 'ai'
-  fixStrategy?: 'redirect' | 'fix-link'
+  fixStrategy?: 'redirect' | 'fix-link' | 'insert-link'
   aiUnavailable?: boolean
   fingerprint: string
   patch: Record<string, unknown>
@@ -498,6 +498,7 @@ export const SEO_INLINE_FIX_ISSUE_TYPES = new Set([
   'duplicate-meta-description',
   'missing-structured-data',
   'broken-internal-link',
+  'orphan-page',
 ])
 
 export function isInlineSeoFixIssue(type: string): boolean {
@@ -1125,6 +1126,23 @@ export interface LinkSuggestion {
   anchorText: string
   confidence: number
   excerpt: string
+  fingerprint: string
+  wrappable: boolean
+  sourceCollection?: string
+}
+
+export async function applySeoInternalLink(input: {
+  sourceEntityId: string
+  targetEntityId: string
+  anchorText: string
+  fingerprint: string
+}): Promise<{ error?: string }> {
+  const res = await cmsApi('/seo/links/apply', {
+    method: 'POST',
+    body: JSON.stringify(input),
+  })
+  if (res.error) return { error: res.error }
+  return {}
 }
 
 export async function fetchLinkGraphSnapshot(): Promise<LinkGraphSnapshot> {

package/src/lib/workflow-chrome.ts

@@ -8,6 +8,8 @@ export const REQUEST_CHANGES_LABEL = 'Request changes'
 export const WAITING_FOR_REVIEW_LABEL = 'Waiting for review.'
 export const REVIEWER_NOTE_PREFIX = 'Reviewer note:'
 export const SUBMIT_BEFORE_PUBLISH_ERROR = 'Submit for review before publishing.'
+export const SAVE_DRAFT_LABEL = 'Save Draft'
+export const UPDATE_LIVE_LABEL = 'Update live'
 
 const REVIEWER_ROLES = new Set(['ADMIN', 'EDITOR'])
 const SUBMIT_ROLES = new Set(['ADMIN', 'EDITOR', 'AUTHOR'])
@@ -33,12 +35,14 @@ export function resolveWorkflowChrome(input: {
   waitingLabel?: string
   statusLabel: string
   statusClassName: string
+  saveLabel: string
 } {
   const role = input.role.toUpperCase()
   const published = input.status === 'PUBLISHED'
   const scheduled = input.status === 'SCHEDULED'
   const pillKey = published ? 'PUBLISHED' : scheduled ? 'SCHEDULED' : input.stage
   const pill = STATUS_PILL[pillKey]
+  const saveLabel = published ? UPDATE_LIVE_LABEL : SAVE_DRAFT_LABEL
 
   if (!input.canEdit || published) {
     return {
@@ -47,6 +51,7 @@ export function resolveWorkflowChrome(input: {
       showSchedule: false,
       statusLabel: pill.label,
       statusClassName: pill.className,
+      saveLabel,
     }
   }
 
@@ -61,6 +66,7 @@ export function resolveWorkflowChrome(input: {
       showSchedule: reviewer,
       statusLabel: pill.label,
       statusClassName: pill.className,
+      saveLabel,
     }
   }
 
@@ -71,6 +77,7 @@ export function resolveWorkflowChrome(input: {
       showSchedule: false,
       statusLabel: pill.label,
       statusClassName: pill.className,
+      saveLabel,
     }
   }
 
@@ -81,6 +88,7 @@ export function resolveWorkflowChrome(input: {
       showSchedule: true,
       statusLabel: pill.label,
       statusClassName: pill.className,
+      saveLabel,
     }
   }
 
@@ -92,6 +100,7 @@ export function resolveWorkflowChrome(input: {
       waitingLabel: WAITING_FOR_REVIEW_LABEL,
       statusLabel: pill.label,
       statusClassName: pill.className,
+      saveLabel,
     }
   }
 
@@ -101,5 +110,6 @@ export function resolveWorkflowChrome(input: {
     showSchedule: false,
     statusLabel: pill.label,
     statusClassName: pill.className,
+    saveLabel,
   }
 }

package/src/views/SEO.tsx

@@ -249,11 +249,12 @@ export function SEO({ onNavigate, initialTab = 'overview' }: SEOProps) {
                 <ContentTab
                   initialEntityType={deepLink.entityType}
                   initialEntityId={deepLink.entityId}
+                  onNavigate={onNavigate}
                 />
               )}
             </Tabs.Content>
             <Tabs.Content value="links" tabIndex={-1}>
-              {activeTab === 'links' && <LinksTab onNavigate={onNavigate} />}
+              {activeTab === 'links' && <LinksTab onNavigate={onNavigate} planInfo={planInfo} />}
             </Tabs.Content>
             <Tabs.Content value="technical" tabIndex={-1}>
               {activeTab === 'technical' && <TechnicalTab />}

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

@@ -403,7 +403,7 @@ export function EditorTopBar({
           disabled={!canEdit || saveState === 'saving' || (!dirty && saveState !== 'error')}
           className="border-border text-foreground hover:bg-accent rounded-lg border px-3 py-1.5 text-sm font-medium transition-colors disabled:cursor-not-allowed disabled:opacity-50"
         >
-          {saveState === 'saving' ? 'Saving…' : 'Save Draft'}
+          {saveState === 'saving' ? 'Saving…' : workflow.saveLabel}
         </button>
 
         {onSchedule && workflow.showSchedule && (

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

@@ -1013,6 +1013,7 @@ export function PageSectionEditor({
                         onDraftChange={handlePageSettings}
                         onClose={() => setRailTab('sections')}
                         onSaved={handleSeoSaved}
+                        onNavigate={onNavigate}
                       />
                     )}
                   </div>

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

@@ -849,6 +849,7 @@ export function PostSectionEditor({
                         onDraftChange={handleFields}
                         onClose={() => setRailTab('sections')}
                         onSaved={handleSeoSaved}
+                        onNavigate={onNavigate}
                       />
                     )}
                   </div>

package/src/views/seo/ContentTab.tsx

@@ -44,9 +44,11 @@ function parseEntityType(value: string | null | undefined): SeoEntityType | null
 export function ContentTab({
   initialEntityType,
   initialEntityId,
+  onNavigate,
 }: {
   initialEntityType?: string | null
   initialEntityId?: string | null
+  onNavigate?: (path: string) => void
 }) {
   const { data, loading, error, refetch } = useSeoResource(fetchSeoContentRecords, [])
   const [search, setSearch] = useState('')
@@ -194,6 +196,7 @@ export function ContentTab({
               entityId={selection.id}
               onClose={() => setSelection(null)}
               onSaved={refetch}
+              onNavigate={onNavigate}
             />
           )}
         </InspectorPane>