@beechcms/api 0.6.0-preview.2 → 0.6.0-preview.4

package/src/features/seeds/seeds.handler.test.ts

@@ -1,1199 +0,0 @@
-// SPDX-License-Identifier: BUSL-1.1
-// Copyright (c) 2024–2026 Flavio De Musso. All rights reserved.
-// See LICENSE in the repository root for license terms.
-
-import { describe, it, expect, vi, beforeEach } from 'vitest'
-import { Hono } from 'hono'
-import type { Branch, ISeedRepository, ISchemaMutator, Seed, SeedRecord } from '@beechcms/core'
-import { seedsApp } from './seeds.handler'
-
-// ─── Fixtures ────────────────────────────────────────────────────────────────
-
-const baseSeed: Seed = {
-  slug: 'articles',
-  label: 'Articles',
-  displayNameAlias: 'title',
-  branches: [
-    { id: 'br_01', alias: 'title', label: 'Title', type: 'text' },
-    { id: 'br_02', alias: 'body', label: 'Body', type: 'richtext' },
-  ],
-}
-
-const baseRecord: SeedRecord = {
-  slug: 'articles',
-  definition: baseSeed,
-  status: 'active',
-  source: 'runtime',
-  createdAt: 0,
-  updatedAt: 0,
-}
-
-// ─── Fake repositories ────────────────────────────────────────────────────────
-
-function makeRepo(overrides: Partial<ISeedRepository> = {}): ISeedRepository {
-  return {
-    listActive: vi.fn().mockResolvedValue([baseSeed]),
-    listAll: vi.fn().mockResolvedValue([baseRecord]),
-    get: vi.fn().mockResolvedValue(null),
-    upsert: vi.fn().mockResolvedValue(undefined),
-    softDelete: vi.fn().mockResolvedValue(undefined),
-    hardDelete: vi.fn().mockResolvedValue(undefined),
-    getRegistryVersion: vi.fn().mockResolvedValue(1),
-    bumpRegistryVersion: vi.fn().mockResolvedValue(2),
-    ...overrides,
-  }
-}
-
-function makeMutator(overrides: Partial<ISchemaMutator> = {}): ISchemaMutator {
-  return {
-    getColumns: vi.fn().mockResolvedValue(null),
-    execDdl: vi.fn().mockResolvedValue(undefined),
-    dropTable: vi.fn().mockResolvedValue(undefined),
-    dropColumn: vi.fn().mockResolvedValue(undefined),
-    renameColumn: vi.fn().mockResolvedValue(undefined),
-    execDestructive: vi.fn().mockResolvedValue(undefined),
-    ...overrides,
-  }
-}
-
-function makeActivityLogger() {
-  return { log: vi.fn() }
-}
-
-// ─── App builder ─────────────────────────────────────────────────────────────
-
-function buildApp(opts: {
-  role?: string
-  jwtPayload?: Record<string, unknown> | null
-  repo?: ISeedRepository
-  mutator?: ISchemaMutator
-  backrefMap?: Map<string, any[]>
-  automationRepository?: any
-  db?: any
-  bucket?: any
-  mediaRepository?: any
-  systemStatsRepository?: any
-} = {}) {
-  const app = new Hono()
-  const repo = opts.repo ?? makeRepo()
-  const mutator = opts.mutator ?? makeMutator()
-  const backrefMap = opts.backrefMap ?? new Map()
-  const activityLogger = makeActivityLogger()
-  // Default stub DB (used by hard-delete R2 cascade) — no results
-  const db = opts.db ?? { prepare: () => ({ all: async () => ({ results: [] }) }) }
-  const automationRepository = opts.automationRepository ?? { list: async () => [] }
-  const bucket = opts.bucket ?? { delete: vi.fn().mockResolvedValue(undefined) }
-  const mediaRepository = opts.mediaRepository ?? { getByKey: vi.fn().mockResolvedValue(null), untrack: vi.fn().mockResolvedValue(undefined) }
-  const systemStatsRepository = opts.systemStatsRepository ?? { decrementStorage: vi.fn().mockResolvedValue(undefined) }
-
-  app.use('*', async (c, next) => {
-    const jwtPayload = opts.jwtPayload !== undefined
-      ? opts.jwtPayload
-      : (opts.role ? { sub: 'u1', email: 'a@b.com', role: opts.role } : null)
-    c.set('jwtPayload' as never, jwtPayload)
-    c.set('seedRepository' as never, repo)
-    c.set('schemaMutator' as never, mutator)
-    c.set('backrefMap' as never, backrefMap)
-    c.set('activityLogger' as never, activityLogger)
-    c.set('automationRepository' as never, automationRepository)
-    c.set('env' as never, { ENV: 'test', DB: db })
-    c.set('bucket' as never, bucket)
-    c.set('mediaRepository' as never, mediaRepository)
-    c.set('systemStatsRepository' as never, systemStatsRepository)
-    await next()
-  })
-  app.route('/', seedsApp)
-
-  // Default `c.env` (used by internalErrorDetail) so error paths don't crash
-  // on `context.env.ENV` when the test doesn't pass an explicit env binding.
-  const defaultEnv = { ENV: 'test', DB: db }
-  const originalRequest = app.request.bind(app)
-  app.request = ((input: any, init?: any, env?: any, executionCtx?: any) =>
-    originalRequest(input, init, env ?? defaultEnv, executionCtx)) as typeof app.request
-
-  return { app, repo, mutator, activityLogger, bucket, mediaRepository, systemStatsRepository, db }
-}
-
-// ─── Tests ────────────────────────────────────────────────────────────────────
-
-describe('GET /', () => {
-  it('403 for non-admin', async () => {
-    const { app } = buildApp({ role: 'user' })
-    const res = await app.request('/', { method: 'GET' })
-    expect(res.status).toBe(403)
-  })
-
-  it('200 returns all records', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/', { method: 'GET' })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any[]
-    expect(body).toHaveLength(1)
-    expect(body[0].slug).toBe('articles')
-  })
-})
-
-describe('GET /:slug', () => {
-  it('404 for missing seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope', { method: 'GET' })
-    expect(res.status).toBe(404)
-  })
-
-  it('200 returns the record', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles', { method: 'GET' })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.slug).toBe('articles')
-  })
-})
-
-describe('POST / (create)', () => {
-  it('403 for non-admin', async () => {
-    const { app } = buildApp({ role: 'user' })
-    const res = await app.request('/', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify(baseSeed) })
-    expect(res.status).toBe(403)
-  })
-
-  it('409 when slug already active', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(baseSeed),
-    })
-    expect(res.status).toBe(409)
-  })
-
-  it('creates new seed: planCreateSeed DDL emitted, upsert called, 201', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(null) })
-    const { app, activityLogger } = buildApp({ role: 'admin', repo, mutator })
-
-    const newSeed: Seed = {
-      slug: 'notes',
-      label: 'Notes',
-      displayNameAlias: 'title',
-      branches: [{ id: 'br_01', alias: 'title', label: 'Title', type: 'text' }],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(newSeed),
-    })
-    expect(res.status).toBe(201)
-
-    const ddlCalls = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls
-    expect(ddlCalls.length).toBe(1)
-    const stmts: string[] = ddlCalls[0][0]
-    expect(stmts.some((s: string) => s.includes('CREATE TABLE') && s.includes('content_notes'))).toBe(true)
-    expect(stmts.some((s: string) => s.includes('DROP'))).toBe(false)
-
-    expect((repo.upsert as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    expect((repo.bumpRegistryVersion as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    expect(activityLogger.log.mock.calls.length).toBe(1)
-    expect(activityLogger.log.mock.calls[0][0].action).toBe('create')
-
-    const body = await res.json() as any
-    expect(body.slug).toBe('notes')
-  })
-
-  it('assigns br_NN ids to branches without ids', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const mutator = makeMutator()
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const noIdSeed = {
-      slug: 'tags',
-      label: 'Tags',
-      displayNameAlias: 'name',
-      branches: [{ alias: 'name', label: 'Name', type: 'text' }],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(noIdSeed),
-    })
-    expect(res.status).toBe(201)
-    const upsertArg = (repo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as Seed
-    expect(upsertArg.branches[0].id).toMatch(/^br_\d+$/)
-  })
-
-  it('revives a soft-deleted seed without failing on IF NOT EXISTS', async () => {
-    const deletedRecord: SeedRecord = { ...baseRecord, status: 'deleted' }
-    const repo = makeRepo({
-      get: vi.fn().mockResolvedValue(deletedRecord),
-      listActive: vi.fn().mockResolvedValue([]),
-    })
-    const existingCols = new Set(['id', 'title'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(baseSeed),
-    })
-    expect(res.status).toBe(201)
-    // No CREATE TABLE was emitted (table already exists), planExtendSeed path taken
-    const stmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls[0]?.[0] ?? []
-    expect(stmts.every((s: string) => !s.includes('CREATE TABLE IF NOT EXISTS content_articles') || s.includes('IF NOT EXISTS'))).toBe(true)
-  })
-
-  it('422 for unknown relation target', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const relSeed: Seed = {
-      slug: 'posts',
-      label: 'Posts',
-      displayNameAlias: 'title',
-      branches: [
-        { id: 'br_01', alias: 'title', label: 'Title', type: 'text' },
-        { id: 'br_02', alias: 'author', label: 'Author', type: 'relation', targetSeed: 'nonexistent' },
-      ],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(relSeed),
-    })
-    expect(res.status).toBe(422)
-  })
-})
-
-describe('PUT /:slug (edit)', () => {
-  it('403 for non-admin', async () => {
-    const { app } = buildApp({ role: 'user' })
-    const res = await app.request('/articles', { method: 'PUT', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify(baseSeed) })
-    expect(res.status).toBe(403)
-  })
-
-  it('422 on alias rename (same id, different alias)', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const renamed: Seed = {
-      ...baseSeed,
-      branches: [
-        { id: 'br_01', alias: 'headline', label: 'Headline', type: 'text' }, // alias changed
-        { id: 'br_02', alias: 'body', label: 'Body', type: 'richtext' },
-      ],
-    }
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(renamed),
-    })
-    expect(res.status).toBe(422)
-    const body = await res.json() as any
-    expect(body.type).toContain('alias-rename-not-supported')
-  })
-
-  it('422 on branch type change', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const typeChanged: Seed = {
-      ...baseSeed,
-      branches: [
-        { id: 'br_01', alias: 'title', label: 'Title', type: 'number' }, // type changed
-        { id: 'br_02', alias: 'body', label: 'Body', type: 'richtext' },
-      ],
-    }
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(typeChanged),
-    })
-    expect(res.status).toBe(422)
-    const body = await res.json() as any
-    expect(body.type).toContain('branch-type-change-not-supported')
-  })
-
-  it('add branch emits ADD COLUMN, no DROP', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const existingCols = new Set(['id', 'title', 'body'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const withNew: Seed = {
-      ...baseSeed,
-      branches: [
-        ...baseSeed.branches,
-        { id: 'br_03', alias: 'summary', label: 'Summary', type: 'text' },
-      ],
-    }
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(withNew),
-    })
-    expect(res.status).toBe(200)
-
-    const stmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls[0][0]
-    expect(stmts.some((s: string) => s.includes('ADD COLUMN') && s.includes('summary'))).toBe(true)
-    expect(stmts.every((s: string) => !s.includes('DROP'))).toBe(true)
-  })
-
-  it('removing a branch: no DROP emitted, column retained', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const existingCols = new Set(['id', 'title', 'body'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const withRemoved: Seed = {
-      ...baseSeed,
-      branches: [baseSeed.branches[0]], // body removed
-    }
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(withRemoved),
-    })
-    expect(res.status).toBe(200)
-
-    const allStmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls.flatMap((c: any) => c[0])
-    expect(allStmts.every((s: string) => !s.includes('DROP'))).toBe(true)
-  })
-})
-
-describe('POST /:slug/branches (add branch)', () => {
-  it('assigns id and emits ADD COLUMN', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const existingCols = new Set(['id', 'title', 'body'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ alias: 'excerpt', label: 'Excerpt', type: 'text' }),
-    })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.id).toMatch(/^br_\d+$/)
-
-    const stmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls[0][0]
-    expect(stmts.some((s: string) => s.includes('ADD COLUMN') && s.includes('excerpt'))).toBe(true)
-  })
-
-  it('column already present: no ADD COLUMN emitted', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const existingCols = new Set(['id', 'title', 'body', 'excerpt'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ alias: 'excerpt', label: 'Excerpt', type: 'text' }),
-    })
-    expect(res.status).toBe(200)
-
-    const stmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls[0]?.[0] ?? []
-    expect(stmts.every((s: string) => !s.includes('ADD COLUMN'))).toBe(true)
-  })
-})
-
-describe('DELETE /:slug', () => {
-  it('403 for non-admin', async () => {
-    const { app } = buildApp({ role: 'user' })
-    const res = await app.request('/articles', { method: 'DELETE' })
-    expect(res.status).toBe(403)
-  })
-
-  it('soft-deletes: status flips, table not dropped, version bumped', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles', { method: 'DELETE' })
-    expect(res.status).toBe(200)
-
-    expect((repo.softDelete as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    expect((repo.bumpRegistryVersion as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-
-    // No DDL involving DROP was emitted
-    const allStmts: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls.flatMap((c: any) => c[0])
-    expect(allStmts.every((s: string) => !s.includes('DROP'))).toBe(true)
-  })
-
-  it('409 when seed is referenced by another seed relation', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const backrefMap = new Map([['articles', [{ seedSlug: 'comments', branchAlias: 'article' }]]])
-    const { app } = buildApp({ role: 'admin', repo, backrefMap })
-
-    const res = await app.request('/articles', { method: 'DELETE' })
-    expect(res.status).toBe(409)
-    const body = await res.json() as any
-    expect(body.type).toContain('seed-referenced')
-  })
-
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope', { method: 'DELETE' })
-    expect(res.status).toBe(404)
-  })
-})
-
-// ─── Sprint 06 Danger Zone routes ────────────────────────────────────────────
-
-describe('GET /:slug/orphans', () => {
-  it('returns orphan columns not in the definition', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    // DB has 'id', 'slug', 'status', 'created_at', 'updated_at', 'title', 'body', 'legacy'
-    const dbCols = new Set(['id', 'slug', 'status', 'created_at', 'updated_at', 'title', 'body', 'legacy'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(dbCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/orphans', { method: 'GET' })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.orphans).toContain('legacy')
-    expect(body.orphans).not.toContain('title')
-  })
-
-  it('returns empty when DB has no orphans', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const dbCols = new Set(['id', 'slug', 'status', 'created_at', 'updated_at', 'title', 'body'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(dbCols) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/orphans', { method: 'GET' })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.orphans).toHaveLength(0)
-  })
-})
-
-describe('DELETE /:slug/hard', () => {
-  it('400 on confirm mismatch', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'wrong' }),
-    })
-    expect(res.status).toBe(400)
-    const body = await res.json() as any
-    expect(body.type).toContain('confirmation-required')
-  })
-
-  it('409 when seed is referenced', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const backrefMap = new Map([['articles', [{ sourceSlug: 'comments' }]]])
-    const { app } = buildApp({ role: 'admin', repo, backrefMap })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles' }),
-    })
-    expect(res.status).toBe(409)
-  })
-
-  it('hard delete: execDestructive called, hardDelete called, version bumped, audit logged', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const { app, activityLogger } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles' }),
-    })
-    expect(res.status).toBe(200)
-
-    expect((mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls.length).toBeGreaterThan(0)
-    // execDdl must NOT have been called for DROP
-    const ddlCalls: string[] = (mutator.execDdl as ReturnType<typeof vi.fn>).mock.calls.flatMap((c: any) => c[0])
-    expect(ddlCalls.every((s: string) => !s.toUpperCase().includes('DROP'))).toBe(true)
-
-    expect((repo.hardDelete as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    expect((repo.bumpRegistryVersion as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    expect(activityLogger.log.mock.calls[0][0].action).toBe('delete')
-    expect(activityLogger.log.mock.calls[0][0].details.op).toBe('hard-delete')
-  })
-})
-
-describe('DELETE /:slug/branches/:branchId', () => {
-  it('400 on confirm mismatch', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'wrong' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('drops branch: execDestructive called, definition updated, version bumped', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches/br_02', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles.body' }),
-    })
-    expect(res.status).toBe(200)
-
-    expect((mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    const upsertArg = (repo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as any
-    expect(upsertArg.branches.find((b: any) => b.id === 'br_02')).toBeUndefined()
-    expect((repo.bumpRegistryVersion as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-  })
-
-  it('404 for unknown branchId', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_99', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles.unknown' }),
-    })
-    expect(res.status).toBe(404)
-  })
-})
-
-describe('PATCH /:slug/branches/:branchId/rename', () => {
-  it('400 on confirm mismatch', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'headline', confirm: 'wrong' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('renames alias: execDestructive called with RENAME, definition updated, automation warnings returned', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const automationRepository = { list: vi.fn().mockResolvedValue([{ id: 'auto_1', conditions: 'title' }]) }
-    const { app } = buildApp({ role: 'admin', repo, mutator, automationRepository })
-
-    const res = await app.request('/articles/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'headline', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(200)
-
-    const destructiveCalls: string[] = (mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls[0][0]
-    expect(destructiveCalls.some((s: string) => s.toUpperCase().includes('RENAME COLUMN') || s.toUpperCase().includes('RENAME TO'))).toBe(true)
-
-    const upsertArg = (repo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as any
-    const renamed = upsertArg.branches.find((b: any) => b.id === 'br_01')
-    expect(renamed?.alias).toBe('headline')
-
-    const body = await res.json() as any
-    expect(body.affectedAutomations).toContain('auto_1')
-  })
-})
-
-describe('PATCH /:slug/branches/:branchId/retype', () => {
-  it('400 on confirm mismatch', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'number', confirm: 'wrong' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('retypes branch: execDestructive called, definition updated, version bumped', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'number', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(200)
-
-    expect((mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    const upsertArg = (repo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as any
-    expect(upsertArg.branches.find((b: any) => b.id === 'br_01')?.type).toBe('number')
-    expect((repo.bumpRegistryVersion as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-  })
-
-  it('422 retype-not-supported when retyping a repeater branch away', async () => {
-    const repeaterRecord: SeedRecord = {
-      ...baseRecord,
-      definition: {
-        ...baseSeed,
-        branches: [
-          baseSeed.branches[0],
-          { id: 'br_02', alias: 'items', label: 'Items', type: 'repeater', fields: [] },
-        ],
-      },
-    }
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(repeaterRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_02/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'text', confirm: 'articles.items' }),
-    })
-    expect(res.status).toBe(422)
-    const problem = await res.json() as any
-    expect(problem.type).toContain('retype-not-supported')
-  })
-
-  it('422 retype-not-supported when retyping a branch to repeater', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'repeater', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(422)
-    const problem = await res.json() as any
-    expect(problem.type).toContain('retype-not-supported')
-  })
-})
-
-describe('POST /:slug/fts/rebuild', () => {
-  it('calls execDestructive with FTS statements and logs audit', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const { app, activityLogger } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/fts/rebuild', { method: 'POST' })
-    expect(res.status).toBe(200)
-
-    expect((mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls.length).toBe(1)
-    const stmts: string[] = (mutator.execDestructive as ReturnType<typeof vi.fn>).mock.calls[0][0]
-    expect(stmts.some((s: string) => s.includes('fts_articles'))).toBe(true)
-    expect(activityLogger.log.mock.calls[0][0].details.op).toBe('fts-rebuild')
-  })
-
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/fts/rebuild', { method: 'POST' })
-    expect(res.status).toBe(404)
-  })
-})
-
-// ─── Malformed JSON bodies ────────────────────────────────────────────────────
-
-describe('Malformed JSON body', () => {
-  it('POST / returns 400 invalid-json on bad body', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-    const body = await res.json() as any
-    expect(body.type).toContain('invalid-json')
-  })
-
-  it('PUT /:slug returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('POST /:slug/branches returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/branches', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('DELETE /:slug/hard returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('DELETE /:slug/branches/:branchId returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/branches/br_01', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('PATCH rename returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('PATCH retype returns 400 invalid-json on bad body', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: '{not valid json',
-    })
-    expect(res.status).toBe(400)
-  })
-})
-
-// ─── Additional 404 / validation paths ───────────────────────────────────────
-
-describe('Additional validation and 404 paths', () => {
-  it('POST / 400 on invalid slug format', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const invalid: Seed = { ...baseSeed, slug: 'Invalid Slug!' }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(invalid),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('POST / infers displayNameAlias from the first text branch when missing', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const { app, repo: usedRepo } = buildApp({ role: 'admin', repo })
-    const noDisplaySeed = {
-      slug: 'snippets',
-      label: 'Snippets',
-      branches: [
-        { id: 'br_01', alias: 'amount', label: 'Amount', type: 'number' },
-        { id: 'br_02', alias: 'headline', label: 'Headline', type: 'text' },
-      ],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(noDisplaySeed),
-    })
-    expect(res.status).toBe(201)
-    const upsertArg = (usedRepo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as Seed
-    expect(upsertArg.displayNameAlias).toBe('headline')
-  })
-
-  it('POST / 400 when displayNameAlias missing and no text branch to infer from', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const noTextSeed = {
-      slug: 'numbers_only',
-      label: 'Numbers',
-      branches: [{ id: 'br_01', alias: 'amount', label: 'Amount', type: 'number' }],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(noTextSeed),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('POST / 422 ddl-failed when execDdl throws', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(null), listActive: vi.fn().mockResolvedValue([]) })
-    const mutator = makeMutator({ execDdl: vi.fn().mockRejectedValue(new Error('boom')) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const newSeed: Seed = {
-      slug: 'broken',
-      label: 'Broken',
-      displayNameAlias: 'title',
-      branches: [{ id: 'br_01', alias: 'title', label: 'Title', type: 'text' }],
-    }
-    const res = await app.request('/', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(newSeed),
-    })
-    expect(res.status).toBe(422)
-    const body = await res.json() as any
-    expect(body.type).toContain('ddl-failed')
-    expect(body.detail).toContain('boom')
-  })
-
-  it('PUT /:slug 404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(baseSeed),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('PUT /:slug assigns an id to a newly added branch without one', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const existingCols = new Set(['id', 'title', 'body'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(existingCols) })
-    const { app, repo: usedRepo } = buildApp({ role: 'admin', repo, mutator })
-
-    const withNewUnidentified: Seed = {
-      ...baseSeed,
-      branches: [
-        ...baseSeed.branches,
-        { alias: 'extra', label: 'Extra', type: 'text' } as Branch,
-      ],
-    }
-    const res = await app.request('/articles', {
-      method: 'PUT',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify(withNewUnidentified),
-    })
-    expect(res.status).toBe(200)
-    const upsertArg = (usedRepo.upsert as ReturnType<typeof vi.fn>).mock.calls[0][1] as Seed
-    const extra = upsertArg.branches.find((b) => b.alias === 'extra')
-    expect(extra?.id).toMatch(/^br_\d+$/)
-  })
-
-  it('POST /:slug/branches 404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/branches', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ alias: 'x', label: 'X', type: 'text' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('GET /:slug/orphans 404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/orphans', { method: 'GET' })
-    expect(res.status).toBe(404)
-  })
-
-  it('GET /:slug/orphans returns empty when DB columns are unknown (table does not exist)', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(null) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/orphans', { method: 'GET' })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.orphans).toEqual([])
-  })
-})
-
-// ─── Hard delete: media cascade ──────────────────────────────────────────────
-
-describe('DELETE /:slug/hard — media cascade (deleteSeedMediaObjects)', () => {
-  const seedWithFile: Seed = {
-    slug: 'articles',
-    label: 'Articles',
-    displayNameAlias: 'title',
-    branches: [
-      { id: 'br_01', alias: 'title', label: 'Title', type: 'text' },
-      { id: 'br_02', alias: 'cover', label: 'Cover', type: 'file' },
-    ],
-  }
-  const recordWithFile: SeedRecord = { ...baseRecord, definition: seedWithFile }
-
-  it('400 on confirm mismatch', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'wrong' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'nope' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('skips media cleanup when dbCols is null (table missing)', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(recordWithFile) })
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(null) })
-    const { app, bucket } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles' }),
-    })
-    expect(res.status).toBe(200)
-    expect(bucket.delete).not.toHaveBeenCalled()
-  })
-
-  it('skips media cleanup when no file column is present in DB', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(recordWithFile) })
-    const dbCols = new Set(['id', 'title']) // 'cover' column absent
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(dbCols) })
-    const { app, bucket } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles' }),
-    })
-    expect(res.status).toBe(200)
-    expect(bucket.delete).not.toHaveBeenCalled()
-  })
-
-  it('extracts R2 keys from rows and deletes them via deleteR2Objects', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(recordWithFile) })
-    const dbCols = new Set(['id', 'title', 'cover'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(dbCols) })
-    const db = {
-      prepare: (sql: string) => ({
-        all: async () => {
-          expect(sql).toContain('cover')
-          expect(sql).toContain('content_articles')
-          return { results: [{ cover: '/api/media/abc123.png' }, { cover: null }] }
-        },
-      }),
-    }
-    const { app, bucket, mediaRepository } = buildApp({ role: 'admin', repo, mutator, db })
-
-    const res = await app.request(
-      '/articles/hard',
-      {
-        method: 'DELETE',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ confirm: 'articles' }),
-      },
-      { DB: db }
-    )
-    expect(res.status).toBe(200)
-    expect(bucket.delete).toHaveBeenCalledWith('abc123.png')
-    expect(mediaRepository.untrack).toHaveBeenCalledWith('abc123.png')
-  })
-
-  it('non-fatal: swallows errors thrown while gathering media keys', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(recordWithFile) })
-    const mutator = makeMutator({ getColumns: vi.fn().mockRejectedValue(new Error('schema unavailable')) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/hard', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles' }),
-    })
-    expect(res.status).toBe(200)
-  })
-
-  it('non-fatal: swallows errors thrown by deleteR2Objects itself', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(recordWithFile) })
-    const dbCols = new Set(['id', 'title', 'cover'])
-    const mutator = makeMutator({ getColumns: vi.fn().mockResolvedValue(dbCols) })
-    const db = {
-      prepare: () => ({
-        all: async () => ({ results: [{ cover: '/api/media/abc123.png' }] }),
-      }),
-    }
-    const bucket = { delete: vi.fn().mockRejectedValue(new Error('r2 down')) }
-    const mediaRepository = { getByKey: vi.fn().mockRejectedValue(new Error('lookup failed')), untrack: vi.fn().mockRejectedValue(new Error('untrack failed')) }
-    const { app } = buildApp({ role: 'admin', repo, mutator, db, bucket, mediaRepository })
-
-    const res = await app.request(
-      '/articles/hard',
-      {
-        method: 'DELETE',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ confirm: 'articles' }),
-      },
-      { DB: db }
-    )
-    expect(res.status).toBe(200)
-  })
-})
-
-// ─── DELETE /:slug/branches/:branchId — DDL failure ──────────────────────────
-
-describe('DELETE /:slug/branches/:branchId — DDL failure', () => {
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/branches/br_01', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'nope.x' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('422 ddl-failed when execDestructive throws', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator({ execDestructive: vi.fn().mockRejectedValue(new Error('drop failed')) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches/br_02', {
-      method: 'DELETE',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ confirm: 'articles.body' }),
-    })
-    expect(res.status).toBe(422)
-    const body = await res.json() as any
-    expect(body.type).toContain('ddl-failed')
-  })
-})
-
-// ─── PATCH rename — extra validation paths ───────────────────────────────────
-
-describe('PATCH /:slug/branches/:branchId/rename — extra validation paths', () => {
-  it('400 when newAlias is invalid', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'Not Valid!', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'headline', confirm: 'nope.x' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('404 for unknown branchId', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_99/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'headline', confirm: 'articles.unknown' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('non-fatal: swallows errors from automationRepository.list', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator()
-    const automationRepository = { list: vi.fn().mockRejectedValue(new Error('automations unavailable')) }
-    const { app } = buildApp({ role: 'admin', repo, mutator, automationRepository })
-
-    const res = await app.request('/articles/branches/br_01/rename', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newAlias: 'headline', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(200)
-    const body = await res.json() as any
-    expect(body.affectedAutomations).toEqual([])
-  })
-})
-
-// ─── PATCH retype — extra validation paths ───────────────────────────────────
-
-describe('PATCH /:slug/branches/:branchId/retype — extra validation paths', () => {
-  it('400 when newType is invalid', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'not-a-type', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(400)
-  })
-
-  it('404 for non-existent seed', async () => {
-    const { app } = buildApp({ role: 'admin' })
-    const res = await app.request('/nope/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'number', confirm: 'nope.x' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('404 for unknown branchId', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app } = buildApp({ role: 'admin', repo })
-
-    const res = await app.request('/articles/branches/br_99/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'number', confirm: 'articles.unknown' }),
-    })
-    expect(res.status).toBe(404)
-  })
-
-  it('422 ddl-failed when execDestructive throws', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const mutator = makeMutator({ execDestructive: vi.fn().mockRejectedValue(new Error('retype failed')) })
-    const { app } = buildApp({ role: 'admin', repo, mutator })
-
-    const res = await app.request('/articles/branches/br_01/retype', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ newType: 'number', confirm: 'articles.title' }),
-    })
-    expect(res.status).toBe(422)
-    const body = await res.json() as any
-    expect(body.type).toContain('ddl-failed')
-  })
-})
-
-// ─── actorFromContext fallbacks ──────────────────────────────────────────────
-
-describe('actorFromContext fallbacks', () => {
-  it('falls back to "unknown" id/email and null name when JWT lacks those claims', async () => {
-    const repo = makeRepo({ get: vi.fn().mockResolvedValue(baseRecord) })
-    const { app, activityLogger } = buildApp({ jwtPayload: { role: 'admin' }, repo })
-
-    const res = await app.request('/articles', { method: 'DELETE' })
-    expect(res.status).toBe(200)
-
-    const actor = activityLogger.log.mock.calls[0][0].actor
-    expect(actor.id).toBe('unknown')
-    expect(actor.email).toBe('unknown')
-    expect(actor.name).toBeNull()
-  })
-})