@setzkasten-cms/astro-admin 1.4.2 → 1.5.0

package/src/api-routes/__tests__/auth-guard.test.ts

@@ -1,17 +1,24 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest'
-import { parseSession, guardPageAccess } from '../_auth-guard'
 import type { AuthSession, SetzKastenConfig } from '@setzkasten-cms/core'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { guardPageAccess, parseSession } from '../_auth-guard'
+import { makeTestSessionCookie } from './_session-test-helper'
 
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
 
 function adminSession(): AuthSession {
-  return { user: { id: '1', email: 'admin@example.com', provider: 'github', role: 'admin' }, expiresAt: Date.now() + 86400_000 }
+  return {
+    user: { id: '1', email: 'admin@example.com', provider: 'github', role: 'admin' },
+    expiresAt: Date.now() + 86400_000,
+  }
 }
 
 function editorSession(email = 'editor@example.com'): AuthSession {
-  return { user: { id: '2', email, provider: 'google', role: 'editor' }, expiresAt: Date.now() + 86400_000 }
+  return {
+    user: { id: '2', email, provider: 'google', role: 'editor' },
+    expiresAt: Date.now() + 86400_000,
+  }
 }
 
 const baseConfig: SetzKastenConfig = {
@@ -33,9 +40,37 @@ describe('parseSession', () => {
     expect(parseSession('not-json')).toBeNull()
   })
 
-  it('parses a valid session cookie', () => {
+  it('rejects an unsigned plain-JSON cookie (legacy / forged format)', () => {
+    // Pre-C1 the cookie was JSON.stringify(session) — anyone could forge
+    // any role. parseSession must refuse the old format outright.
     const session = adminSession()
-    expect(parseSession(JSON.stringify(session))).toEqual(session)
+    expect(parseSession(JSON.stringify(session))).toBeNull()
+  })
+
+  it('accepts a properly signed session cookie', () => {
+    const session = adminSession()
+    expect(parseSession(makeTestSessionCookie(session))).toEqual(session)
+  })
+
+  it('rejects an expired signed session', () => {
+    const expired: AuthSession = {
+      user: { id: '1', email: 'admin@example.com', provider: 'github', role: 'admin' },
+      expiresAt: Date.now() - 1,
+    }
+    expect(parseSession(makeTestSessionCookie(expired))).toBeNull()
+  })
+
+  it('rejects a tampered payload (role escalation forgery)', () => {
+    const editor: AuthSession = {
+      user: { id: '2', email: 'e@example.com', provider: 'google', role: 'editor' },
+      expiresAt: Date.now() + 60_000,
+    }
+    const cookie = makeTestSessionCookie(editor)
+    const [, sig] = cookie.split('.')
+    const forgedPayload = Buffer.from(
+      JSON.stringify({ ...editor, user: { ...editor.user, role: 'admin' } }),
+    ).toString('base64url')
+    expect(parseSession(`${forgedPayload}.${sig}`)).toBeNull()
   })
 })
 
@@ -103,7 +138,11 @@ describe('guardPageAccess – dynamic editors', () => {
       kind: 'present',
       editors: [{ email: 'editor@example.com' }],
     })
-    const res = await guardPageAccess(editorSession('editor@example.com'), 'secret-page', baseConfig)
+    const res = await guardPageAccess(
+      editorSession('editor@example.com'),
+      'secret-page',
+      baseConfig,
+    )
     expect(res).toBeNull()
   })
 

package/src/api-routes/__tests__/catalog-api.test.ts

@@ -10,11 +10,11 @@
  *   3. buildCatalogAddCommit  — builds file paths for a catalog add commit
  */
 
-import { describe, it, expect } from 'vitest'
+import { describe, expect, it } from 'vitest'
 import {
+  buildCatalogAddCommit,
   buildCatalogResponse,
   validateCatalogAddBody,
-  buildCatalogAddCommit,
 } from '../catalog-helpers'
 
 // ---------------------------------------------------------------------------
@@ -31,7 +31,7 @@ describe('buildCatalogResponse', () => {
   })
 
   it('includes hero, features, cta', () => {
-    const names = buildCatalogResponse().map(t => t.name)
+    const names = buildCatalogResponse().map((t) => t.name)
     expect(names).toContain('hero')
     expect(names).toContain('features')
     expect(names).toContain('cta')
@@ -62,7 +62,9 @@ describe('validateCatalogAddBody', () => {
   })
 
   it('accepts body with sectionKey override', () => {
-    expect(() => validateCatalogAddBody({ templateName: 'hero', pageKey: 'index', sectionKey: 'hero--top' })).not.toThrow()
+    expect(() =>
+      validateCatalogAddBody({ templateName: 'hero', pageKey: 'index', sectionKey: 'hero--top' }),
+    ).not.toThrow()
   })
 
   it('throws when templateName is missing', () => {
@@ -74,7 +76,9 @@ describe('validateCatalogAddBody', () => {
   })
 
   it('throws when templateName is not in registry', () => {
-    expect(() => validateCatalogAddBody({ templateName: 'nonexistent-xyz', pageKey: 'index' })).toThrow()
+    expect(() =>
+      validateCatalogAddBody({ templateName: 'nonexistent-xyz', pageKey: 'index' }),
+    ).toThrow()
   })
 })
 
@@ -109,7 +113,11 @@ describe('buildCatalogAddCommit', () => {
   })
 
   it('pageConfigPath matches the pageKey', () => {
-    const result = buildCatalogAddCommit({ ...opts, pageKey: 'about', pageConfigPath: 'content/pages/_about.json' })
+    const result = buildCatalogAddCommit({
+      ...opts,
+      pageKey: 'about',
+      pageConfigPath: 'content/pages/_about.json',
+    })
     expect(result.pageConfigPath).toBe('content/pages/_about.json')
   })
 })

package/src/api-routes/__tests__/commit-trailers.test.ts

@@ -3,8 +3,8 @@
  * @vitest-environment node
  */
 
-import { describe, it, expect } from 'vitest'
-import { withTrailers, SETZKASTEN_CO_AUTHOR } from '../_commit-trailers'
+import { describe, expect, it } from 'vitest'
+import { SETZKASTEN_CO_AUTHOR, withTrailers } from '../_commit-trailers'
 
 describe('SETZKASTEN_CO_AUTHOR', () => {
   it('is a valid Co-authored-by trailer', () => {
@@ -28,20 +28,20 @@ describe('withTrailers', () => {
 
   it('does not add editor trailer when editorEmail is not provided', () => {
     const result = withTrailers('chore: something')
-    const lines = result.split('\n').filter(l => l.startsWith('Co-authored-by:'))
+    const lines = result.split('\n').filter((l) => l.startsWith('Co-authored-by:'))
     expect(lines).toHaveLength(1)
     expect(lines[0]).toBe(SETZKASTEN_CO_AUTHOR)
   })
 
   it('does not add editor trailer when editorEmail is null', () => {
     const result = withTrailers('chore: something', null)
-    const lines = result.split('\n').filter(l => l.startsWith('Co-authored-by:'))
+    const lines = result.split('\n').filter((l) => l.startsWith('Co-authored-by:'))
     expect(lines).toHaveLength(1)
   })
 
   it('adds editor Co-authored-by when editorEmail is provided', () => {
     const result = withTrailers('content: update', 'jane.doe@example.com')
-    const lines = result.split('\n').filter(l => l.startsWith('Co-authored-by:'))
+    const lines = result.split('\n').filter((l) => l.startsWith('Co-authored-by:'))
     expect(lines).toHaveLength(2)
     expect(lines[1]).toContain('jane.doe@example.com')
   })

package/src/api-routes/__tests__/deferred-operations.test.ts

@@ -8,8 +8,8 @@
  * - Commits triggered on every duplicate/delete instead of batched
  */
 
-import { describe, it, expect } from 'vitest'
-import { generateDuplicateKey, duplicateInPageConfig } from '../section-management'
+import { describe, expect, it } from 'vitest'
+import { duplicateInPageConfig, generateDuplicateKey } from '../section-management'
 
 // ---------------------------------------------------------------------------
 // 1. Duplicate key generation (reused by prepare-copy)
@@ -25,8 +25,9 @@ describe('generateDuplicateKey (used by prepare-copy)', () => {
   })
 
   it('increments suffix until unique', () => {
-    expect(generateDuplicateKey(['hero', 'hero--copy', 'hero--copy2', 'hero--copy3'], 'hero'))
-      .toBe('hero--copy4')
+    expect(generateDuplicateKey(['hero', 'hero--copy', 'hero--copy2', 'hero--copy3'], 'hero')).toBe(
+      'hero--copy4',
+    )
   })
 
   it('generates copy of a copy', () => {
@@ -82,9 +83,7 @@ describe('duplicateInPageConfig (used by prepare-copy)', () => {
 
   it('preserves type field from original entry when present', () => {
     const configWithType = {
-      sections: [
-        { key: 'hero--about', type: 'hero', enabled: true, order: 0 },
-      ],
+      sections: [{ key: 'hero--about', type: 'hero', enabled: true, order: 0 }],
     }
     const result = duplicateInPageConfig(configWithType, 'hero--about', 'hero--about--copy')
     const copy = result.sections.find((s: any) => s.key === 'hero--about--copy')
@@ -98,9 +97,7 @@ describe('duplicateInPageConfig (used by prepare-copy)', () => {
     // getSectionDef looks up 'testPricing--copy' in the catalog → not found → empty editor.
     // resolveSectionType returns 'testPricing--copy' → no component → not in preview.
     const configNoType = {
-      sections: [
-        { key: 'testPricing', enabled: true, order: 0 },
-      ],
+      sections: [{ key: 'testPricing', enabled: true, order: 0 }],
     }
     const result = duplicateInPageConfig(configNoType, 'testPricing', 'testPricing--copy')
     const copy = result.sections.find((s: any) => s.key === 'testPricing--copy')
@@ -169,7 +166,7 @@ describe('commit-pending: commit message', () => {
   function buildCommitMessage(pageKey: string, sections: Array<{ key: string }>): string {
     const parts: string[] = []
     if (sections.length > 0) {
-      const keys = sections.map(s => s.key).join(', ')
+      const keys = sections.map((s) => s.key).join(', ')
       parts.push(`add ${sections.length} section${sections.length > 1 ? 's' : ''} (${keys})`)
     }
     return `content: ${parts.length > 0 ? parts.join(', ') : 'update page config'} on ${pageKey}`
@@ -210,7 +207,7 @@ describe('prepare-copy: type resolution', () => {
     sections: Array<{ key: string; type?: string }>,
     sectionKey: string,
   ): string {
-    const entry = sections.find(s => s.key === sectionKey)
+    const entry = sections.find((s) => s.key === sectionKey)
     return entry?.type ?? sectionKey
   }
 

package/src/api-routes/__tests__/deploy-hook.test.ts

@@ -10,7 +10,7 @@
  *   6. Secret header is sent when configured
  */
 
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 // We import the POST handler directly; Astro's APIRoute type is just a function.
 // We construct a minimal mock context manually.
@@ -40,9 +40,7 @@ describe('deploy-hook POST handler', () => {
     return {
       cookies: {
         get: (name: string) =>
-          name === 'setzkasten_session' && sessionValue
-            ? { value: sessionValue }
-            : undefined,
+          name === 'setzkasten_session' && sessionValue ? { value: sessionValue } : undefined,
       },
     }
   }
@@ -64,10 +62,7 @@ describe('deploy-hook POST handler', () => {
     ;(globalThis as any).__SETZKASTEN_CONFIG__ = {
       deployHook: { url: 'https://example.com/hook' },
     }
-    vi.stubGlobal(
-      'fetch',
-      vi.fn().mockResolvedValue({ ok: true, status: 200 } as Response),
-    )
+    vi.stubGlobal('fetch', vi.fn().mockResolvedValue({ ok: true, status: 200 } as Response))
 
     const res = await POST(makeCtx('tok'))
     expect(res.status).toBe(200)

package/src/api-routes/__tests__/feature-gate.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 import { gateFeature } from '../_feature-gate'
 
 const ORIGINAL_KEY = process.env.SETZKASTEN_LICENSE_KEY

package/src/api-routes/__tests__/github-cache.test.ts

@@ -3,7 +3,7 @@
  * @vitest-environment node
  */
 
-import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { cachedFetch, invalidateCache } from '../_github-cache'
 
 beforeEach(() => {

package/src/api-routes/__tests__/github-token.test.ts

@@ -6,8 +6,8 @@
  *   2. Nicht alle Vars gesetzt → auth error Result
  */
 
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { generateKeyPairSync } from 'node:crypto'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 const { privateKey: TEST_KEY } = generateKeyPairSync('rsa', { modulusLength: 2048 })
 const TEST_PRIVATE_KEY_PEM = TEST_KEY.export({ type: 'pkcs8', format: 'pem' }) as string

package/src/api-routes/__tests__/global-config-theme.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 vi.mock('../_storage-config', () => ({
   resolveStorageConfig: vi.fn(() => ({ owner: 'o', repo: 'r', branch: 'main' })),
@@ -8,7 +8,7 @@ vi.stubGlobal('__SETZKASTEN_CONFIG__', { storage: { contentPath: 'content' } })
 
 describe('GlobalConfig – theme field', () => {
   it('GlobalConfig type accepts a theme object', async () => {
-    const { } = await import('../global-config')
+    const {} = await import('../global-config')
     // Type-level check: if this compiles the type is correct
     const cfg = {
       theme: { primaryColor: '#ff0000', brandName: 'Acme', logo: '/logo.png' },
@@ -43,7 +43,7 @@ describe('config.ts – theme merge', () => {
 
     const { GET } = await import('../config')
     const res = await GET({} as any)
-    const body = await res.json() as any
+    const body = (await res.json()) as any
 
     expect(body.theme.primaryColor).toBe('#abc123')
     expect(body.theme.brandName).toBe('Dynamic Brand')
@@ -63,7 +63,7 @@ describe('config.ts – theme merge', () => {
 
     const { GET } = await import('../config')
     const res = await GET({} as any)
-    const body = await res.json() as any
+    const body = (await res.json()) as any
 
     expect(body.theme.primaryColor).toBe('#ffffff')
     expect(body.theme.brandName).toBe('Static Brand')

package/src/api-routes/__tests__/history-rollback.test.ts

@@ -4,16 +4,17 @@
 
 import { generateKeyPairSync } from 'node:crypto'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { makeTestSessionCookie } from './_session-test-helper'
 
 const { privateKey } = generateKeyPairSync('rsa', { modulusLength: 2048 })
 const PEM = privateKey.export({ type: 'pkcs8', format: 'pem' }) as string
 
-const ADMIN_SESSION = JSON.stringify({
+const ADMIN_SESSION = makeTestSessionCookie({
   user: { id: 'u1', email: 'admin@example.com', role: 'admin', provider: 'github' },
   expiresAt: Date.now() + 60 * 60 * 1000,
 })
 
-const EDITOR_SESSION = JSON.stringify({
+const EDITOR_SESSION = makeTestSessionCookie({
   user: { id: 'u2', email: 'editor@example.com', role: 'editor', provider: 'google' },
   expiresAt: Date.now() + 60 * 60 * 1000,
 })
@@ -88,7 +89,9 @@ describe('POST /api/setzkasten/history/rollback', () => {
 
   it('returns 400 when path or sha missing', async () => {
     const { POST } = await import('../history-rollback')
-    const res = await (POST as (ctx: unknown) => Promise<Response>)(makeCtx({ path: ROLLBACK_PATH }))
+    const res = await (POST as (ctx: unknown) => Promise<Response>)(
+      makeCtx({ path: ROLLBACK_PATH }),
+    )
     expect(res.status).toBe(400)
   })
 

package/src/api-routes/__tests__/history.test.ts

@@ -8,17 +8,22 @@ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 const { privateKey } = generateKeyPairSync('rsa', { modulusLength: 2048 })
 const PEM = privateKey.export({ type: 'pkcs8', format: 'pem' }) as string
 
-const ADMIN_SESSION = JSON.stringify({
+import { makeTestSessionCookie } from './_session-test-helper'
+
+const ADMIN_SESSION = makeTestSessionCookie({
   user: { id: 'u1', email: 'admin@example.com', role: 'admin', provider: 'github' },
   expiresAt: Date.now() + 60 * 60 * 1000,
 })
 
-const EDITOR_SESSION = JSON.stringify({
+const EDITOR_SESSION = makeTestSessionCookie({
   user: { id: 'u2', email: 'editor@example.com', role: 'editor', provider: 'google' },
   expiresAt: Date.now() + 60 * 60 * 1000,
 })
 
-function makeCtx(searchParams: Record<string, string>, sessionValue: string | null = ADMIN_SESSION) {
+function makeCtx(
+  searchParams: Record<string, string>,
+  sessionValue: string | null = ADMIN_SESSION,
+) {
   const url = new URL('https://cms.example.com/api/setzkasten/history')
   for (const [k, v] of Object.entries(searchParams)) url.searchParams.set(k, v)
   const request = new Request(url, { method: 'GET' })
@@ -136,9 +141,7 @@ describe('GET /api/setzkasten/history', () => {
       authorEmail: 'maria@example.com',
       message: 'Update hero',
     })
-    expect(body.commits[0].coAuthors).toEqual([
-      { name: 'Editor', email: 'editor@example.com' },
-    ])
+    expect(body.commits[0].coAuthors).toEqual([{ name: 'Editor', email: 'editor@example.com' }])
   })
 
   it('returns empty list when GitHub returns 404 for the path', async () => {

package/src/api-routes/__tests__/init-scan-page-resolve-config.test.ts

@@ -14,7 +14,7 @@
  * @vitest-environment node
  */
 
-import { describe, it, expect, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 import { resolveFullConfig } from '../init-scan-page'
 
 const SAMPLE_CONFIG = {
@@ -23,8 +23,16 @@ const SAMPLE_CONFIG = {
     website: {
       label: 'Website',
       sections: {
-        _layout_header: { label: 'header', icon: 'panel-top', fields: { items: { type: 'array' } } },
-        _layout_footer: { label: 'footer', icon: 'file-text', fields: { description: { type: 'text' } } },
+        _layout_header: {
+          label: 'header',
+          icon: 'panel-top',
+          fields: { items: { type: 'array' } },
+        },
+        _layout_footer: {
+          label: 'footer',
+          icon: 'file-text',
+          fields: { description: { type: 'text' } },
+        },
       },
     },
   },

package/src/api-routes/__tests__/migrate-to-multi.test.ts

@@ -4,6 +4,7 @@
 
 import { generateKeyPairSync } from 'node:crypto'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { makeTestSessionCookie } from './_session-test-helper'
 
 const { privateKey } = generateKeyPairSync('rsa', { modulusLength: 2048 })
 const PEM = privateKey.export({ type: 'pkcs8', format: 'pem' }) as string
@@ -15,7 +16,10 @@ function makeCtx(body: unknown, sessionValue = 'valid', role: 'admin' | 'editor'
     headers: { 'content-type': 'application/json' },
   })
   const sessionPayload = sessionValue
-    ? JSON.stringify({ user: { email: 'a@b.com', role }, expiresAt: Date.now() + 60_000 })
+    ? makeTestSessionCookie({
+        user: { id: 'test-user', email: 'a@b.com', role, provider: 'github' },
+        expiresAt: Date.now() + 60_000,
+      })
     : ''
   return {
     request,

package/src/api-routes/__tests__/pages-meta-store.test.ts

@@ -21,7 +21,9 @@ afterEach(() => {
   vi.unstubAllEnvs()
 })
 
-function fetchSequence(steps: Array<(url: string, init?: RequestInit) => Response | Promise<Response>>) {
+function fetchSequence(
+  steps: Array<(url: string, init?: RequestInit) => Response | Promise<Response>>,
+) {
   let i = 0
   const fetchMock = vi.fn(async (url: string, init?: RequestInit) => {
     const handler = steps[Math.min(i++, steps.length - 1)]
@@ -51,7 +53,10 @@ describe('readPagesMeta', () => {
     fetchSequence([
       () =>
         new Response(
-          JSON.stringify({ content: Buffer.from(JSON.stringify(meta)).toString('base64'), sha: 'abc' }),
+          JSON.stringify({
+            content: Buffer.from(JSON.stringify(meta)).toString('base64'),
+            sha: 'abc',
+          }),
           { status: 200, headers: { 'content-type': 'application/json' } },
         ),
     ])
@@ -156,9 +161,9 @@ describe('recordPageEdit', () => {
       () =>
         new Response(
           JSON.stringify({
-            content: Buffer.from(JSON.stringify({ version: 1, pages: { x: { lastModified: 2 } } })).toString(
-              'base64',
-            ),
+            content: Buffer.from(
+              JSON.stringify({ version: 1, pages: { x: { lastModified: 2 } } }),
+            ).toString('base64'),
             sha: 'second',
           }),
           { status: 200, headers: { 'content-type': 'application/json' } },

package/src/api-routes/__tests__/pages.test.ts

@@ -13,7 +13,7 @@
  * @vitest-environment node
  */
 
-import { describe, it, expect, beforeEach, afterEach } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 import { resolvePages } from '../pages'
 
 interface PageInfo {
@@ -63,7 +63,12 @@ describe('resolvePages', () => {
   it('handles multiple pages including nested paths', () => {
     const pages: PageInfo[] = [
       { path: '/', pageKey: 'index', label: 'Startseite', hasConfig: true },
-      { path: '/docs/architecture', pageKey: 'docs/architecture', label: 'Architecture', hasConfig: false },
+      {
+        path: '/docs/architecture',
+        pageKey: 'docs/architecture',
+        label: 'Architecture',
+        hasConfig: false,
+      },
     ]
     ;(globalThis as Record<string, unknown>).__SETZKASTEN_PAGES__ = pages
     expect(resolvePages()).toHaveLength(2)

package/src/api-routes/__tests__/patch-page-file.test.ts

@@ -9,8 +9,8 @@
  * Bugs here mean sections either don't appear in the page or appear twice.
  */
 
-import { describe, it, expect } from 'vitest'
-import { patchPageFile, calculateRelativePath } from '../init-add-section'
+import { describe, expect, it } from 'vitest'
+import { calculateRelativePath, patchPageFile } from '../init-add-section'
 
 // ---------------------------------------------------------------------------
 // Fixtures
@@ -64,29 +64,59 @@ describe('patchPageFile — new section injection', () => {
   const componentPath = 'src/components/sections/FeaturesSection.astro'
 
   it('should return a non-null result', () => {
-    const result = patchPageFile(PAGE_WITH_SECTIONS, 'features', 'FeaturesSection', componentPath, pagePath)
+    const result = patchPageFile(
+      PAGE_WITH_SECTIONS,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )
     expect(result).not.toBeNull()
   })
 
   it('should add the import statement', () => {
-    const result = patchPageFile(PAGE_WITH_SECTIONS, 'features', 'FeaturesSection', componentPath, pagePath)!
-    expect(result).toContain("import FeaturesSection from")
+    const result = patchPageFile(
+      PAGE_WITH_SECTIONS,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )!
+    expect(result).toContain('import FeaturesSection from')
   })
 
   it('should include the correct relative path', () => {
-    const result = patchPageFile(PAGE_WITH_SECTIONS, 'features', 'FeaturesSection', componentPath, pagePath)!
+    const result = patchPageFile(
+      PAGE_WITH_SECTIONS,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )!
     // From src/pages to src/components/sections → one level up → ../components/sections/...
     expect(result).toContain('../components/sections/FeaturesSection.astro')
   })
 
   it('should add the section key to SECTION_COMPONENTS', () => {
-    const result = patchPageFile(PAGE_WITH_SECTIONS, 'features', 'FeaturesSection', componentPath, pagePath)!
+    const result = patchPageFile(
+      PAGE_WITH_SECTIONS,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )!
     expect(result).toContain('features')
     expect(result).toContain('FeaturesSection')
   })
 
   it('should preserve existing sections in the registry', () => {
-    const result = patchPageFile(PAGE_WITH_SECTIONS, 'features', 'FeaturesSection', componentPath, pagePath)!
+    const result = patchPageFile(
+      PAGE_WITH_SECTIONS,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )!
     expect(result).toContain('HeroSection')
     expect(result).toContain("normalize('hero')")
   })
@@ -122,13 +152,25 @@ describe('patchPageFile — hardcoded tag removal', () => {
   it('should remove the hardcoded <HeroSection /> when adding to registry', () => {
     // PAGE_WITH_HARDCODED_COMPONENT has HeroSection imported but NOT in registry
     // → patchPageFile adds it to registry AND removes the hardcoded tag
-    const result = patchPageFile(PAGE_WITH_HARDCODED_COMPONENT, 'hero', 'HeroSection', componentPath, pagePath)
+    const result = patchPageFile(
+      PAGE_WITH_HARDCODED_COMPONENT,
+      'hero',
+      'HeroSection',
+      componentPath,
+      pagePath,
+    )
     expect(result).not.toBeNull()
     expect(result).not.toMatch(/<HeroSection\s*\/>/)
   })
 
   it('should add the section to SECTION_COMPONENTS when removing hardcoded tag', () => {
-    const result = patchPageFile(PAGE_WITH_HARDCODED_COMPONENT, 'hero', 'HeroSection', componentPath, pagePath)!
+    const result = patchPageFile(
+      PAGE_WITH_HARDCODED_COMPONENT,
+      'hero',
+      'HeroSection',
+      componentPath,
+      pagePath,
+    )!
     expect(result).toContain('hero')
     expect(result).toContain('HeroSection')
   })
@@ -157,7 +199,13 @@ const SECTION_COMPONENTS = {
     const pagePath = 'src/pages/docs/architecture.astro'
     const componentPath = 'src/components/sections/FeaturesSection.astro'
 
-    const result = patchPageFile(PAGE_NESTED, 'features', 'FeaturesSection', componentPath, pagePath)
+    const result = patchPageFile(
+      PAGE_NESTED,
+      'features',
+      'FeaturesSection',
+      componentPath,
+      pagePath,
+    )
     if (result) {
       // From src/pages/docs to src/components/sections = ../../components/sections
       expect(result).toContain('../../components/sections/FeaturesSection.astro')
@@ -175,19 +223,23 @@ describe('calculateRelativePath', () => {
   })
 
   it('one level up', () => {
-    expect(calculateRelativePath('src/pages', 'src/components/Hero.astro'))
-      .toBe('../components/Hero.astro')
+    expect(calculateRelativePath('src/pages', 'src/components/Hero.astro')).toBe(
+      '../components/Hero.astro',
+    )
   })
 
   it('two levels up', () => {
-    expect(calculateRelativePath('src/pages/docs', 'src/components/sections/Hero.astro'))
-      .toBe('../../components/sections/Hero.astro')
+    expect(calculateRelativePath('src/pages/docs', 'src/components/sections/Hero.astro')).toBe(
+      '../../components/sections/Hero.astro',
+    )
   })
 
   it('deeper nesting in monorepo', () => {
-    expect(calculateRelativePath(
-      'apps/website/src/pages',
-      'apps/website/src/components/sections/FooterSection.astro',
-    )).toBe('../components/sections/FooterSection.astro')
+    expect(
+      calculateRelativePath(
+        'apps/website/src/pages',
+        'apps/website/src/components/sections/FooterSection.astro',
+      ),
+    ).toBe('../components/sections/FooterSection.astro')
   })
 })