@beechcms/api 0.4.0 → 0.4.2

package/src/features/automations/__tests__/automation-runner.test.ts

@@ -0,0 +1,192 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { AutomationRunner } from '../automation-runner'
+import type { AutomationRunnerDeps } from '../automation-runner'
+import type { IAutomationRepository, ContentRepository, Seed, IIdGenerator, Automation } from '@beechcms/core'
+import { resolveVarAccess } from '../var-access-resolver'
+import { parseTemplateKey } from '../template-grammar'
+
+function makeAutomation(overrides: Partial<Automation> = {}): Automation {
+  return {
+    id: 'auto-1',
+    seed_slug: 'posts',
+    name: 'Test Automation',
+    enabled: true,
+    triggers: [{ event: 'create' as const }],
+    trigger_conditions: null,
+    actions: [],
+    created_at: 0,
+    updated_at: 0,
+    ...overrides,
+  }
+}
+
+function makeDeps(overrides: Partial<AutomationRunnerDeps> = {}): AutomationRunnerDeps {
+  return {
+    automationRepository: {
+      findActive: vi.fn().mockResolvedValue([]),
+    } as unknown as IAutomationRepository,
+    contentRepository: {} as ContentRepository,
+    getSeed: vi.fn().mockReturnValue({ slug: 'posts', branches: [] } as unknown as Seed),
+    idGenerator: { uuid: vi.fn().mockReturnValue('new-id') } as unknown as IIdGenerator,
+    env: {},
+    ...overrides,
+  }
+}
+
+describe('AutomationRunner', () => {
+  beforeEach(() => { vi.restoreAllMocks() })
+
+  it('does nothing when no automations match', async () => {
+    const deps = makeDeps()
+    const runner = new AutomationRunner(deps)
+    await runner.run({ seedSlug: 'posts', event: 'create', entry: { id: '1' } })
+    expect(deps.automationRepository.findActive).toHaveBeenCalledWith('posts', 'create')
+  })
+
+  it('does nothing when getSeed returns null', async () => {
+    const deps = makeDeps({ getSeed: vi.fn().mockReturnValue(null) })
+    const runner = new AutomationRunner(deps)
+    await runner.run({ seedSlug: 'unknown', event: 'create', entry: { id: '1' } })
+    expect(deps.automationRepository.findActive).not.toHaveBeenCalled()
+  })
+
+  it('skips actions when trigger conditions are not met', async () => {
+    const automation = makeAutomation({
+      trigger_conditions: {
+        kind: 'group',
+        op: 'AND',
+        children: [{ kind: 'predicate', left: { kind: 'ref', key: 'this.status' }, op: 'eq', right: { kind: 'literal', value: 'published' } }],
+      },
+      actions: [{ type: 'webhook', url: 'https://example.com' }],
+    })
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', fetchMock)
+
+    const deps = makeDeps({
+      automationRepository: {
+        findActive: vi.fn().mockResolvedValue([automation]),
+      } as unknown as IAutomationRepository,
+    })
+    const runner = new AutomationRunner(deps)
+    await runner.run({ seedSlug: 'posts', event: 'create', entry: { id: '1', status: 'draft' } })
+
+    expect(fetchMock).not.toHaveBeenCalled()
+  })
+
+  it('continues to next action when first action throws', async () => {
+    const secondActionFetch = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', vi.fn()
+      .mockRejectedValueOnce(new Error('network error'))
+      .mockResolvedValue({ ok: true }),
+    )
+
+    const automation = makeAutomation({
+      actions: [
+        { type: 'webhook', url: 'https://fail.example.com' },
+        { type: 'webhook', url: 'https://success.example.com' },
+      ],
+    })
+
+    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+
+    const deps = makeDeps({
+      automationRepository: {
+        findActive: vi.fn().mockResolvedValue([automation]),
+      } as unknown as IAutomationRepository,
+    })
+    const runner = new AutomationRunner(deps)
+    await runner.run({ seedSlug: 'posts', event: 'create', entry: { id: '1', status: 'published' } })
+
+    expect(consoleSpy).toHaveBeenCalledWith('[automations] action failed', expect.objectContaining({
+      automationId: 'auto-1',
+      actionType: 'webhook',
+    }))
+
+    // fetch was called twice (first failed, second succeeded)
+    const fetchCalls = (global.fetch as ReturnType<typeof vi.fn>).mock.calls
+    expect(fetchCalls).toHaveLength(2)
+    expect(fetchCalls[1][0]).toBe('https://success.example.com')
+  })
+
+  it('runs actions when conditions pass', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', fetchMock)
+
+    const automation = makeAutomation({
+      trigger_conditions: {
+        kind: 'group',
+        op: 'AND',
+        children: [{ kind: 'predicate', left: { kind: 'ref', key: 'this.status' }, op: 'eq', right: { kind: 'literal', value: 'published' } }],
+      },
+      actions: [{ type: 'webhook', url: 'https://example.com/hook' }],
+    })
+
+    const deps = makeDeps({
+      automationRepository: {
+        findActive: vi.fn().mockResolvedValue([automation]),
+      } as unknown as IAutomationRepository,
+    })
+    const runner = new AutomationRunner(deps)
+    await runner.run({ seedSlug: 'posts', event: 'create', entry: { id: '1', status: 'published' } })
+
+    expect(fetchMock).toHaveBeenCalledOnce()
+    expect(fetchMock.mock.calls[0][0]).toBe('https://example.com/hook')
+  })
+})
+
+// ── Sprint 07: resolveVarAccess unit tests ─────────────────────────────────────
+
+describe('resolveVarAccess (Sprint 07)', () => {
+  function makeCollectionVar(items: Array<Record<string, unknown>>) {
+    const out: Record<string, unknown> = {
+      count: items.length,
+      firstone: items[0] ?? null,
+      lastone: items[items.length - 1] ?? null,
+      sum: {},
+      avg: {},
+      min: {},
+      max: {},
+      pluck: {},
+    }
+    Object.defineProperty(out, '_items', { value: items, enumerable: false })
+    return out
+  }
+
+  const items = [
+    { id: 'a', status: 'paid', amount: 100, created_at: 1 },
+    { id: 'b', status: 'draft', amount: 50, created_at: 2 },
+    { id: 'c', status: 'paid', amount: 200, created_at: 3 },
+  ]
+
+  it('array[id] selector filters _items and rebuilds collection', () => {
+    const variables = { ordini: makeCollectionVar(items) }
+    const parsed = parseTemplateKey('ordini.array[a,c].count')!
+    expect(parsed.kind).toBe('var_access')
+    const result = resolveVarAccess(parsed as any, variables)
+    expect(result).toBe(2)
+  })
+
+  it('inline condition (status=paid) reduces count', () => {
+    const variables = { ordini: makeCollectionVar(items) }
+    const parsed = parseTemplateKey('ordini.count.(status=paid)')!
+    const result = resolveVarAccess(parsed as any, variables)
+    expect(result).toBe(2)
+  })
+
+  it('conditions on firstone.id return undefined when record fails predicate', () => {
+    const variables = { ordini: makeCollectionVar(items) }
+    // firstone is items[0] which has status=paid, so (status=draft) should fail
+    const parsed = parseTemplateKey('ordini.firstone.status.(status=draft)')!
+    const result = resolveVarAccess(parsed as any, variables)
+    // firstone.status = 'paid', condition says status=draft → undefined
+    expect(result).toBeUndefined()
+  })
+
+  it('returns undefined when variable is missing and calls onMissing', () => {
+    const onMissing = vi.fn()
+    const parsed = parseTemplateKey('missing.count.(status=paid)')!
+    const result = resolveVarAccess(parsed as any, {}, onMissing)
+    expect(result).toBeUndefined()
+    expect(onMissing).toHaveBeenCalledWith('missing')
+  })
+})

package/src/features/automations/__tests__/automation-runner.utils.test.ts

@@ -0,0 +1,56 @@
+import { describe, it, expect } from 'vitest'
+import { interpolate } from '../automation-runner.utils'
+import { resolveAutomationContext } from '../context-resolver'
+
+async function makeCtx(entry: Record<string, unknown>) {
+  return resolveAutomationContext({} as any, entry, [entry])
+}
+
+describe('interpolate', () => {
+  it('replaces a single placeholder', async () => {
+    expect(interpolate('Hello {{name}}!', await makeCtx({ name: 'World' }))).toBe('Hello World!')
+  })
+
+  it('replaces multiple placeholders', async () => {
+    expect(interpolate('{{name}} is {{status}}', await makeCtx({ name: 'Alice', status: 'active' }))).toBe('Alice is active')
+  })
+
+  it('missing field becomes empty string by default', async () => {
+    expect(interpolate('Hello {{name}}!', await makeCtx({}))).toBe('Hello !')
+  })
+
+  it('no placeholders returns template unchanged', async () => {
+    expect(interpolate('No placeholders here', await makeCtx({ name: 'World' }))).toBe('No placeholders here')
+  })
+
+  it('handles numeric values', async () => {
+    expect(interpolate('Count: {{count}}', await makeCtx({ count: 42 }))).toBe('Count: 42')
+  })
+
+  it('supports spaces inside double braces', async () => {
+    expect(interpolate('Hello {{ name }}!', await makeCtx({ name: 'DoubleSpace' }))).toBe('Hello DoubleSpace!')
+  })
+
+  it('uses custom default value when a field is missing', async () => {
+    expect(interpolate('Hello {{name}}!', await makeCtx({}), '[missing]')).toBe('Hello [missing]!')
+  })
+
+  it('triggers onMissing callback when field is absent', async () => {
+    const missing: string[] = []
+    const res = interpolate('Hello {{name}}! From {{city}}', await makeCtx({}), '[mancante]', (f) => missing.push(f))
+    expect(res).toBe('Hello [mancante]! From [mancante]')
+    expect(missing).toEqual(['name', 'city'])
+  })
+
+  it('returns empty string when template is falsy', async () => {
+    expect(interpolate('' as any, await makeCtx({}))).toBe('')
+  })
+
+  it('resolves {{this.field}} dot notation against the trigger entry', async () => {
+    expect(interpolate('{{this.customer_id}}', await makeCtx({ customer_id: 'cust-42' }))).toBe('cust-42')
+  })
+
+  it('resolves {{this:field}} colon notation against the trigger entry', async () => {
+    expect(interpolate('{{this:customer_id}}', await makeCtx({ customer_id: 'cust-42' }))).toBe('cust-42')
+  })
+})

package/src/features/automations/__tests__/automations.handler.test.ts

@@ -0,0 +1,260 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { Hono } from 'hono'
+import type { IAutomationRepository, Automation } from '@beechcms/core'
+import { automationsApp } from '../automations.handler'
+
+const baseAutomation: Automation = {
+  id: 'test-id',
+  seed_slug: 'posts',
+  name: 'test',
+  enabled: true,
+  triggers: [{ event: 'create' }],
+  trigger_conditions: null,
+  actions: [{ type: 'webhook', url: 'https://example.com' }],
+  created_at: 1000,
+  updated_at: 1000,
+}
+
+function makeStub(overrides: Partial<IAutomationRepository> = {}): IAutomationRepository {
+  return {
+    list: vi.fn().mockResolvedValue([baseAutomation]),
+    findById: vi.fn().mockResolvedValue(baseAutomation),
+    findActive: vi.fn().mockResolvedValue([]),
+    create: vi.fn().mockResolvedValue('new-id'),
+    update: vi.fn().mockResolvedValue(undefined),
+    toggle: vi.fn().mockResolvedValue(undefined),
+    delete: vi.fn().mockResolvedValue(undefined),
+    ...overrides,
+  }
+}
+
+function buildApp(stub: IAutomationRepository) {
+  const app = new Hono()
+  app.use('*', async (c, next) => {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    ;(c as any).set('automationRepository', stub)
+    await next()
+  })
+  app.route('/', automationsApp)
+  return app
+}
+
+describe('GET /', () => {
+  it('returns 400 missing-seed without ?seed=', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/')
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('missing-seed')
+  })
+
+  it('returns 200 with automations list', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/?seed=posts')
+    expect(res.status).toBe(200)
+    const body = await res.json() as Automation[]
+    expect(body).toHaveLength(1)
+    expect(body[0].id).toBe('test-id')
+  })
+})
+
+describe('POST /', () => {
+  it('returns 400 invalid-json for malformed JSON body', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: 'malformed-not-json',
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('invalid-json')
+  })
+
+  it('returns 400 automation-validation-failed for schema non-compliant body', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ seed_slug: 'posts' }), // missing name, triggers, actions
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('automation-validation-failed')
+  })
+
+  it('returns 201 with id for valid body including optional trigger fields', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        seed_slug: 'posts',
+        name: 'test-cron-automation',
+        triggers: [{ event: 'cron', cron: '0 0 * * *' }],
+        trigger_conditions: {
+          kind: 'group',
+          op: 'AND',
+          children: [
+            { kind: 'predicate', left: { kind: 'ref', key: 'this.status' }, op: 'eq', right: { kind: 'literal', value: 'draft' } },
+          ],
+        },
+        actions: [{ type: 'webhook', url: 'https://example.com' }],
+      }),
+    })
+    expect(res.status).toBe(201)
+    const body = await res.json() as { id: string }
+    expect(body.id).toBe('new-id')
+  })
+
+  it('returns 201 with id for valid body omitting optional trigger fields', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        seed_slug: 'posts',
+        name: 'test-create-automation',
+        triggers: [{ event: 'create' }],
+        actions: [{ type: 'webhook', url: 'https://example.com' }],
+      }),
+    })
+    expect(res.status).toBe(201)
+    const body = await res.json() as { id: string }
+    expect(body.id).toBe('new-id')
+  })
+})
+
+describe('GET /:id', () => {
+  it('returns 404 when not found', async () => {
+    const app = buildApp(makeStub({ findById: vi.fn().mockResolvedValue(null) }))
+    const res = await app.request('/unknown-id')
+    expect(res.status).toBe(404)
+  })
+
+  it('returns 200 with automation', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id')
+    expect(res.status).toBe(200)
+  })
+})
+
+describe('PUT /:id', () => {
+  it('returns 404 when not found', async () => {
+    const app = buildApp(makeStub({ findById: vi.fn().mockResolvedValue(null) }))
+    const res = await app.request('/unknown-id', {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ name: 'renamed' }),
+    })
+    expect(res.status).toBe(404)
+  })
+
+  it('returns 400 invalid-json for malformed JSON body', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id', {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: 'malformed-not-json',
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('invalid-json')
+  })
+
+  it('returns 400 automation-validation-failed for invalid update values', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id', {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ name: 12345 }), // name must be string
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('automation-validation-failed')
+  })
+
+  it('returns 400 when cron trigger has no cron expression', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id', {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ triggers: [{ event: 'cron' }] }),
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('automation-validation-failed')
+  })
+
+  it('returns 204 for valid update', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id', {
+      method: 'PUT',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ name: 'renamed' }),
+    })
+    expect(res.status).toBe(204)
+  })
+})
+
+describe('PATCH /:id/toggle', () => {
+  it('returns 404 when automation not found', async () => {
+    const app = buildApp(makeStub({ findById: vi.fn().mockResolvedValue(null) }))
+    const res = await app.request('/unknown-id/toggle', {
+      method: 'PATCH',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ enabled: false }),
+    })
+    expect(res.status).toBe(404)
+  })
+
+  it('returns 400 invalid-json for malformed JSON body', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id/toggle', {
+      method: 'PATCH',
+      headers: { 'Content-Type': 'application/json' },
+      body: 'malformed-not-json',
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('invalid-json')
+  })
+
+  it('returns 400 automation-validation-failed for invalid payload', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id/toggle', {
+      method: 'PATCH',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ enabled: 'not-a-boolean' }),
+    })
+    expect(res.status).toBe(400)
+    const body = await res.json() as { type: string }
+    expect(body.type).toContain('automation-validation-failed')
+  })
+
+  it('returns 204 and calls toggle(id, false)', async () => {
+    const stub = makeStub()
+    const app = buildApp(stub)
+    const res = await app.request('/test-id/toggle', {
+      method: 'PATCH',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ enabled: false }),
+    })
+    expect(res.status).toBe(204)
+    expect(stub.toggle).toHaveBeenCalledWith('test-id', false)
+  })
+})
+
+describe('DELETE /:id', () => {
+  it('returns 404 when not found', async () => {
+    const app = buildApp(makeStub({ findById: vi.fn().mockResolvedValue(null) }))
+    const res = await app.request('/unknown-id', { method: 'DELETE' })
+    expect(res.status).toBe(404)
+  })
+
+  it('returns 204 on success', async () => {
+    const app = buildApp(makeStub())
+    const res = await app.request('/test-id', { method: 'DELETE' })
+    expect(res.status).toBe(204)
+  })
+})

package/src/features/automations/__tests__/automations.repository.test.ts

@@ -0,0 +1,159 @@
+import { describe, it, expect, vi } from 'vitest'
+import { D1AutomationRepository } from '../../../shared/automations.repository.d1'
+
+function makeDb() {
+  const calls: { sql: string; bindings: unknown[] }[] = []
+  let nextAllResult: unknown[] = []
+  let nextFirstResult: unknown = null
+
+  const db = {
+    _calls: calls,
+    _setAllResult(rows: unknown[]) { nextAllResult = rows },
+    _setFirstResult(row: unknown) { nextFirstResult = row },
+    prepare(sql: string) {
+      let boundValues: unknown[] = []
+      const stmt = {
+        bind(...args: unknown[]) { boundValues = args; return stmt },
+        async run() {
+          calls.push({ sql, bindings: boundValues })
+          return { success: true }
+        },
+        async all() {
+          calls.push({ sql, bindings: boundValues })
+          return { results: nextAllResult }
+        },
+        async first() {
+          calls.push({ sql, bindings: boundValues })
+          return nextFirstResult
+        },
+      }
+      return stmt
+    },
+  }
+  return db as unknown as D1Database & typeof db
+}
+
+const fixtureRow = {
+  id: 'abc',
+  seed_slug: 'posts',
+  name: 'test',
+  enabled: 1,
+  triggers: JSON.stringify([{ event: 'create' }]),
+  trigger_conditions: null,
+  actions: JSON.stringify([{ type: 'webhook', url: 'https://example.com' }]),
+  created_at: 1000,
+  updated_at: 1000,
+}
+
+describe('D1AutomationRepository.create', () => {
+  it('serialises actions and trigger_conditions to JSON', async () => {
+    const db = makeDb()
+    const repo = new D1AutomationRepository(db)
+    const whenNode = {
+      kind: 'group' as const,
+      op: 'AND' as const,
+      children: [{ kind: 'predicate' as const, left: { kind: 'ref' as const, key: 'this.status' }, op: 'eq' as const, right: { kind: 'literal' as const, value: 'published' } }],
+    }
+    await repo.create({
+      seed_slug: 'posts',
+      name: 'test',
+      triggers: [{ event: 'create' }],
+      trigger_conditions: whenNode,
+      actions: [{ type: 'webhook', url: 'https://example.com' }],
+    })
+    expect(db._calls).toHaveLength(1)
+    const { bindings } = db._calls[0]
+    // triggers is index 3, trigger_conditions is index 4, actions is index 5
+    expect(typeof bindings[3]).toBe('string')
+    expect(JSON.parse(bindings[3] as string)).toEqual([{ event: 'create' }])
+    expect(typeof bindings[4]).toBe('string')
+    expect(JSON.parse(bindings[4] as string)).toEqual(whenNode)
+    expect(typeof bindings[5]).toBe('string')
+    expect(JSON.parse(bindings[5] as string)).toEqual([{ type: 'webhook', url: 'https://example.com' }])
+  })
+
+  it('stores null for missing trigger_conditions', async () => {
+    const db = makeDb()
+    const repo = new D1AutomationRepository(db)
+    await repo.create({
+      seed_slug: 'posts',
+      name: 'test',
+      triggers: [{ event: 'create' }],
+      trigger_conditions: null,
+      actions: [{ type: 'webhook', url: 'https://example.com' }],
+    })
+    const { bindings } = db._calls[0]
+    expect(bindings[4]).toBeNull()
+  })
+})
+
+describe('D1AutomationRepository.update', () => {
+  it('builds SET clause only from provided keys', async () => {
+    const db = makeDb()
+    const repo = new D1AutomationRepository(db)
+    await repo.update('abc', { name: 'renamed' })
+    expect(db._calls).toHaveLength(1)
+    expect(db._calls[0].sql).toContain('name = ?')
+    expect(db._calls[0].sql).not.toContain('seed_slug')
+  })
+
+  it('short-circuits when no fields provided', async () => {
+    const db = makeDb()
+    const repo = new D1AutomationRepository(db)
+    await repo.update('abc', {})
+    expect(db._calls).toHaveLength(0)
+  })
+})
+
+describe('D1AutomationRepository.toggle', () => {
+  it('only writes enabled and updated_at', async () => {
+    const db = makeDb()
+    const repo = new D1AutomationRepository(db)
+    await repo.toggle('abc', false)
+    expect(db._calls).toHaveLength(1)
+    const { sql, bindings } = db._calls[0]
+    expect(sql).toBe('UPDATE automations SET enabled = ?, updated_at = ? WHERE id = ?')
+    expect(bindings[0]).toBe(0)
+    expect(bindings[2]).toBe('abc')
+  })
+})
+
+describe('rowToAutomation JSON round-trip', () => {
+  it('deserialises trigger_conditions as null when column is null', async () => {
+    const db = makeDb()
+    db._setAllResult([fixtureRow])
+    const repo = new D1AutomationRepository(db)
+    const rows = await repo.list('posts')
+    expect(rows[0].trigger_conditions).toBeNull()
+  })
+
+  it('deserialises trigger_conditions as WhenNode when column contains WhenNode JSON', async () => {
+    const db = makeDb()
+    const whenNode = {
+      kind: 'group',
+      op: 'AND',
+      children: [{
+        kind: 'predicate',
+        left: { kind: 'ref', key: 'this.status' },
+        op: 'eq',
+        right: { kind: 'literal', value: 'published' },
+      }],
+    }
+    const rowWithConditions = {
+      ...fixtureRow,
+      trigger_conditions: JSON.stringify(whenNode),
+    }
+    db._setAllResult([rowWithConditions])
+    const repo = new D1AutomationRepository(db)
+    const rows = await repo.list('posts')
+    expect(rows[0].trigger_conditions).toEqual(whenNode)
+  })
+
+  it('deserialises actions as parsed array', async () => {
+    const db = makeDb()
+    db._setAllResult([fixtureRow])
+    const repo = new D1AutomationRepository(db)
+    const rows = await repo.list('posts')
+    expect(rows[0].actions).toEqual([{ type: 'webhook', url: 'https://example.com' }])
+  })
+})