@beechcms/api 0.6.0-preview.1 → 0.6.0-preview.3

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

@@ -1,196 +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 { 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

@@ -1,60 +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 } 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

@@ -1,264 +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 { 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', body_template: '{}' }],
-  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', body_template: '{}' }],
-      }),
-    })
-    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', body_template: '{}' }],
-      }),
-    })
-    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

@@ -1,163 +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 } 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' }])
-  })
-})