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

package/src/features/automations/__tests__/action-executors.test.ts

@@ -1,255 +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 { executeAction } from '../action-executors'
-import type { ActionContext } from '../action-executors'
-import type { ContentRepository, Seed, IIdGenerator } from '@beechcms/core'
-import { deleteAllMessages, listMessages, waitForMessage, getMessageHtml, getMailpitPort } from '../../../../test/helpers/mailpit-client'
-import { newBucket, waitForRequest } from '../../../../test/helpers/webhook-tester-client'
-
-// ── Shared mock context factory ───────────────────────────────────────────────
-
-function makeCtx(overrides: Partial<ActionContext> = {}): ActionContext {
-  const { entry: entryOverride, context: contextOverride, ...rest } = overrides
-  const entry: Record<string, unknown> = entryOverride ?? { id: 'entry-1', title: 'Test Entry', status: 'published' }
-  const context = contextOverride ?? {
-    triggerEntry: entry,
-    lookup(parsed: import('../template-grammar').ParsedKey, onMissing?: (f: string) => void) {
-      if (parsed.kind === 'simple') {
-        const val = entry[parsed.path]
-        if (val == null && onMissing) onMissing(parsed.path)
-        return val
-      }
-      if (parsed.kind === 'scoped' && parsed.scope === 'this' && parsed.field) {
-        return entry[parsed.field]
-      }
-      return undefined
-    },
-  }
-  return {
-    entry,
-    env: { RESEND_API_KEY: 'test-key', EMAIL_FROM: 'test@example.com' },
-    repository: {
-      update: vi.fn().mockResolvedValue(undefined),
-      create: vi.fn().mockResolvedValue(undefined),
-    } as unknown as ContentRepository,
-    getSeed: vi.fn().mockReturnValue({ branches: [] } as unknown as Seed),
-    seed: { slug: 'posts', branches: [] } as unknown as Seed,
-    idGenerator: { uuid: vi.fn().mockReturnValue('new-id-123') } as unknown as IIdGenerator,
-    context,
-    variables: {},
-    ...rest,
-  }
-}
-
-// ── webhook ───────────────────────────────────────────────────────────────────
-
-describe('webhook executor', () => {
-  it('POSTs to URL with interpolated body_template', async () => {
-    const { url, uuid } = await newBucket()
-    const ctx = makeCtx()
-    await executeAction({ type: 'webhook', url, body_template: '{"id":"{{id}}","title":"{{title}}"}' }, ctx)
-    const req = await waitForRequest(uuid)
-    expect(req.method).toBe('POST')
-    expect(JSON.parse(req.body)).toMatchObject({ id: 'entry-1', title: 'Test Entry' })
-  })
-
-  it('interpolates body_template with entry values', async () => {
-    const { url, uuid } = await newBucket()
-    const ctx = makeCtx()
-    await executeAction({ type: 'webhook', url, body_template: '{"t":"{{title}}"}' }, ctx)
-    const req = await waitForRequest(uuid)
-    expect(req.body).toBe('{"t":"Test Entry"}')
-  })
-
-  it('merges custom headers', async () => {
-    const { url, uuid } = await newBucket()
-    const ctx = makeCtx()
-    await executeAction({ type: 'webhook', url, headers: { 'X-Token': 'abc' } }, ctx)
-    const req = await waitForRequest(uuid)
-    expect(req.headers['x-token']).toBe('abc')
-  })
-
-  it('throws when webhook endpoint is unreachable', async () => {
-    const ctx = makeCtx()
-    await expect(executeAction({ type: 'webhook', url: 'http://localhost:19999/nope' }, ctx)).rejects.toThrow()
-  })
-
-  it('uses custom HTTP method', async () => {
-    const { url, uuid } = await newBucket()
-    const ctx = makeCtx()
-    await executeAction({ type: 'webhook', url, method: 'PUT' }, ctx)
-    const req = await waitForRequest(uuid)
-    expect(req.method).toBe('PUT')
-  })
-})
-
-// ── send_mail ─────────────────────────────────────────────────────────────────
-
-describe('send_mail executor', () => {
-  beforeEach(() => deleteAllMessages())
-
-  it('interpolates to/subject/body and delivers to Mailpit', async () => {
-    const ctx = makeCtx({
-      entry: { id: '1', email: 'user@example.com', title: 'Hello' },
-      env: { EMAIL_PROVIDER: 'smtp', SMTP_HOST: 'localhost', SMTP_PORT: getMailpitPort(), EMAIL_FROM: 'Beech <test@beech.local>' },
-    })
-
-    await executeAction(
-      {
-        type: 'send_mail',
-        to: '{{email}}',
-        subject_template: 'Re: {{title}}',
-        body_template: 'Your entry {{title}} was created.',
-      },
-      ctx,
-    )
-
-    const msg = await waitForMessage(m => m.To.some(t => t.Address === 'user@example.com'))
-    expect(msg.Subject).toBe('Re: Hello')
-    const html = await getMessageHtml(msg.ID)
-    expect(html).toContain('Hello')
-  })
-
-  it('substitutes [missing] for absent fields and logs a warning', async () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    // email is present (needed for a valid To address) but title and desc are absent
-    const ctx = makeCtx({
-      entry: { id: '1', email: 'warn-test@example.com' },
-      env: { EMAIL_PROVIDER: 'smtp', SMTP_HOST: 'localhost', SMTP_PORT: getMailpitPort(), EMAIL_FROM: 'Beech <test@beech.local>' },
-    })
-
-    await executeAction(
-      {
-        type: 'send_mail',
-        to: '{{email}}',
-        subject_template: 'Re: {{title}}',
-        body_template: 'Content: {{desc}}',
-      },
-      ctx,
-    )
-
-    expect(warnSpy).toHaveBeenCalledWith(
-      expect.stringContaining('missing concrete data'),
-      expect.arrayContaining(['title', 'desc']),
-    )
-    const msg = await waitForMessage(m => m.Subject === 'Re: [missing]')
-    expect(msg.Subject).toBe('Re: [missing]')
-    const html = await getMessageHtml(msg.ID)
-    expect(html).toContain('[missing]')
-    warnSpy.mockRestore()
-  })
-
-  it('skips execution and logs an error when provider is resend and apiKey is missing', async () => {
-    const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
-    const ctx = makeCtx({ env: {} })
-
-    await executeAction(
-      {
-        type: 'send_mail',
-        to: 'user@beech.io',
-        subject_template: 'Hello',
-        body_template: 'World',
-      },
-      ctx,
-    )
-
-    expect(errSpy).toHaveBeenCalledWith(expect.stringContaining('Execution skipped'))
-    const msgs = await listMessages()
-    expect(msgs).toHaveLength(0)
-    errSpy.mockRestore()
-  })
-
-  it('logs error and rethrows when SMTP provider is unreachable', async () => {
-    const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
-    const ctx = makeCtx({
-      env: { EMAIL_PROVIDER: 'smtp', SMTP_HOST: 'localhost', SMTP_PORT: '19999', EMAIL_FROM: 'Beech <test@beech.local>' },
-    })
-
-    await expect(
-      executeAction(
-        {
-          type: 'send_mail',
-          to: 'test@beech.io',
-          subject_template: 'Hello',
-          body_template: 'World',
-        },
-        ctx,
-      ),
-    ).rejects.toThrow()
-
-    expect(errSpy).toHaveBeenCalledWith(
-      expect.stringContaining('failed to send automation email'),
-      expect.any(Error),
-    )
-    errSpy.mockRestore()
-  })
-})
-
-// ── edit_field ────────────────────────────────────────────────────────────────
-
-describe('edit_field executor', () => {
-  it('calls repository.update with interpolated string value', async () => {
-    const ctx = makeCtx({ entry: { id: 'entry-1', title: 'Old Title' } })
-    await executeAction({ type: 'edit_field', field: 'summary', value: 'Based on {{title}}' }, ctx)
-    expect(ctx.repository.update).toHaveBeenCalledWith(ctx.seed, 'entry-1', { summary: 'Based on Old Title' })
-  })
-
-  it('calls repository.update with raw non-string value', async () => {
-    const ctx = makeCtx({ entry: { id: 'entry-1', title: 'T' } })
-    await executeAction({ type: 'edit_field', field: 'count', value: 42 }, ctx)
-    expect(ctx.repository.update).toHaveBeenCalledWith(ctx.seed, 'entry-1', { count: 42 })
-  })
-
-  it('throws when entry.id is missing', async () => {
-    const ctx = makeCtx({ entry: { title: 'No ID' } })
-    await expect(executeAction({ type: 'edit_field', field: 'x', value: 'y' }, ctx)).rejects.toThrow('entry.id missing')
-  })
-})
-
-// ── create_entry ──────────────────────────────────────────────────────────────
-
-describe('create_entry executor', () => {
-  it('creates a new entry with field_map values from trigger entry', async () => {
-    const ctx = makeCtx({ entry: { id: 'e1', author: 'Alice' } })
-    await executeAction(
-      { type: 'create_entry', seed_slug: 'comments', field_map: { writer: 'author', fixed: 'literal-value' } },
-      ctx,
-    )
-    expect(ctx.idGenerator.uuid).toHaveBeenCalled()
-    expect(ctx.repository.create).toHaveBeenCalledWith(
-      expect.anything(),
-      'new-id-123',
-      'new-id-123',
-      'draft',
-      { writer: 'Alice', fixed: 'literal-value' },
-    )
-  })
-
-  it('uses literal string when source field is not in entry', async () => {
-    const ctx = makeCtx({ entry: { id: 'e1' } })
-    await executeAction(
-      { type: 'create_entry', seed_slug: 'comments', field_map: { tag: 'missing-field' } },
-      ctx,
-    )
-    expect(ctx.repository.create).toHaveBeenCalledWith(
-      expect.anything(), 'new-id-123', 'new-id-123', 'draft',
-      { tag: 'missing-field' },
-    )
-  })
-
-  it('throws when seed not found', async () => {
-    const ctx = makeCtx({ getSeed: vi.fn().mockReturnValue(null) })
-    await expect(
-      executeAction({ type: 'create_entry', seed_slug: 'unknown', field_map: {} }, ctx),
-    ).rejects.toThrow('unknown seed unknown')
-  })
-
-  it('throws error for unknown action type to cover exhaustive default check', async () => {
-    const ctx = makeCtx()
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    await expect(executeAction({ type: 'unknown_custom_action' } as any, ctx)).rejects.toThrow('unknown action type')
-  })
-})

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')
-  })
-})