npm - @beechcms/api - Versions diffs - 0.4.1 → 0.4.3 - Mend

@beechcms/api 0.4.1 → 0.4.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (67) hide show

package/assets/dashboard/assets/index-BKWnlvnV.css +1 -0
package/assets/dashboard/assets/index-BMkd1Irh.js +629 -0
package/assets/dashboard/index.html +2 -2
package/migrations/0000_v040_base.sql +25 -0
package/migrations/0029_automations.sql +14 -0
package/package.json +4 -3
package/src/factory.ts +12 -4
package/src/features/automations/__tests__/action-executors.test.ts +268 -0
package/src/features/automations/__tests__/automation-runner.test.ts +192 -0
package/src/features/automations/__tests__/automation-runner.utils.test.ts +56 -0
package/src/features/automations/__tests__/automations.handler.test.ts +260 -0
package/src/features/automations/__tests__/automations.repository.test.ts +159 -0
package/src/features/automations/__tests__/automations.schema.test.ts +134 -0
package/src/features/automations/__tests__/context-resolver.test.ts +122 -0
package/src/features/automations/__tests__/cron-runner.test.ts +263 -0
package/src/features/automations/__tests__/cron-runner.utils.test.ts +140 -0
package/src/features/automations/__tests__/set-variable.executor.test.ts +306 -0
package/src/features/automations/__tests__/template-grammar.test.ts +304 -0
package/src/features/automations/__tests__/when-evaluator.test.ts +270 -0
package/src/features/automations/__tests__/when-pushdown.test.ts +277 -0
package/src/features/automations/action-executors/create-entry.executor.ts +23 -0
package/src/features/automations/action-executors/edit-field.executor.ts +22 -0
package/src/features/automations/action-executors/index.ts +33 -0
package/src/features/automations/action-executors/send-mail.executor.ts +42 -0
package/src/features/automations/action-executors/set-variable.executor.ts +146 -0
package/src/features/automations/action-executors/webhook.executor.ts +25 -0
package/src/features/automations/automation-runner.ts +81 -0
package/src/features/automations/automation-runner.utils.ts +43 -0
package/src/features/automations/automations.handler.ts +193 -0
package/src/features/automations/automations.schema.ts +160 -0
package/src/features/automations/context-resolver.ts +148 -0
package/src/features/automations/cron-runner.ts +136 -0
package/src/features/automations/cron-runner.utils.ts +40 -0
package/src/features/automations/filter-translation.ts +42 -0
package/src/features/automations/index.ts +12 -0
package/src/features/automations/template-grammar.ts +241 -0
package/src/features/automations/var-access-resolver.ts +136 -0
package/src/features/automations/when-evaluator.ts +83 -0
package/src/features/automations/when-pushdown.ts +53 -0
package/src/features/content/handlers/create.ts +8 -0
package/src/features/content/handlers/delete.ts +8 -1
package/src/features/content/handlers/update.ts +8 -0
package/src/features/draft/draft.handler.ts +51 -164
package/src/features/draft/draft.middleware.ts +62 -0
package/src/features/email/email.service.ts +13 -0
package/src/features/email/email.types.ts +10 -0
package/src/features/email/index.ts +2 -1
package/src/features/email/templates/automation-mail.ts +15 -0
package/src/features/settings/settings.handler.ts +2 -1
package/src/index.ts +40 -8
package/src/middleware/repository.middleware.ts +32 -1
package/src/public/cache-utils.ts +34 -0
package/src/public/entry-projection.ts +42 -0
package/src/public/idempotency.ts +19 -0
package/src/public/problem-details.ts +5 -0
package/src/public/public-add.ts +17 -63
package/src/public/public-read.ts +20 -177
package/src/public/read-list.ts +50 -0
package/src/public/read-single.ts +44 -0
package/src/shared/automations.repository.d1.ts +146 -0
package/src/shared/d1-content-scan.repository.test.ts +76 -0
package/src/shared/execution-context-scheduler.ts +9 -0
package/src/shared/storage-utils.ts +3 -3
package/src/types.ts +5 -1
package/src/upload.ts +3 -2
package/assets/dashboard/assets/index-CH13idU1.js +0 -554
package/assets/dashboard/assets/index-CewtCjom.css +0 -1

package/assets/dashboard/index.html CHANGED Viewed

@@ -9,8 +9,8 @@
     <!-- Poppins: pesi normali + italic. Inter: fallback con piena copertura degli stili. -->
     <link href="https://fonts.googleapis.com/css2?family=Inter:ital,wght@0,300;0,400;0,500;0,600;0,700;1,300;1,400;1,500;1,600;1,700&family=Poppins:ital,wght@0,300;0,400;0,500;0,600;0,700;1,300;1,400;1,500;1,600;1,700&display=swap" rel="stylesheet" />
     <title>dashboard</title>
-    <script type="module" crossorigin src="/admin/assets/index-CH13idU1.js"></script>
-    <link rel="stylesheet" crossorigin href="/admin/assets/index-CewtCjom.css">
+    <script type="module" crossorigin src="/admin/assets/index-BMkd1Irh.js"></script>
+    <link rel="stylesheet" crossorigin href="/admin/assets/index-BKWnlvnV.css">
   </head>
   <body>
     <div id="root"></div>

package/migrations/0000_v040_base.sql CHANGED Viewed

@@ -211,3 +211,28 @@ CREATE TABLE IF NOT EXISTS content_event_log (
 CREATE INDEX IF NOT EXISTS idx_event_log_schema_slug ON content_event_log(schema_slug);
 CREATE INDEX IF NOT EXISTS idx_event_log_created_at  ON content_event_log(created_at DESC);
 CREATE INDEX IF NOT EXISTS idx_event_log_entry_id    ON content_event_log(entry_id);
+-- =============================================================================
+-- 11. AUTOMATIONS
+--     Event-driven automation rules. Each automation defines one or more
+--     triggers (JSON array of {event, cron?}), optional WhenNode conditions,
+--     and a sequence of actions to execute.
+--     triggers: [{ event: 'create'|'update'|'delete'|'cron', cron?: string }]
+--     Max one cron trigger per automation, enforced at the application layer.
+-- =============================================================================
+CREATE TABLE IF NOT EXISTS automations (
+  id                 TEXT    NOT NULL PRIMARY KEY,
+  seed_slug          TEXT    NOT NULL,
+  name               TEXT    NOT NULL,
+  enabled            INTEGER NOT NULL DEFAULT 1,
+  triggers           TEXT    NOT NULL DEFAULT '[]',
+  trigger_conditions TEXT,
+  actions            TEXT    NOT NULL,
+  created_at         INTEGER NOT NULL DEFAULT (unixepoch()),
+  updated_at         INTEGER NOT NULL DEFAULT (unixepoch())
+);
+CREATE INDEX IF NOT EXISTS idx_automations_seed_slug ON automations(seed_slug);
+CREATE INDEX IF NOT EXISTS idx_automations_enabled   ON automations(enabled);

package/migrations/0029_automations.sql ADDED Viewed

@@ -0,0 +1,14 @@
+CREATE TABLE IF NOT EXISTS automations (
+  id                 TEXT    NOT NULL PRIMARY KEY,
+  seed_slug          TEXT    NOT NULL,
+  name               TEXT    NOT NULL,
+  enabled            INTEGER NOT NULL DEFAULT 1,
+  triggers           TEXT    NOT NULL DEFAULT '[]',
+  trigger_conditions TEXT,
+  actions            TEXT    NOT NULL,
+  created_at         INTEGER NOT NULL DEFAULT (unixepoch()),
+  updated_at         INTEGER NOT NULL DEFAULT (unixepoch())
+);
+CREATE INDEX IF NOT EXISTS idx_automations_seed_slug ON automations(seed_slug);
+CREATE INDEX IF NOT EXISTS idx_automations_enabled   ON automations(enabled);

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@beechcms/api",
-  "version": "0.4.1",
+  "version": "0.4.3",
   "type": "module",
   "exports": {
     ".": "./src/factory.ts"
@@ -8,12 +8,13 @@
   "files": [
     "src/",
     "migrations/0000_v040_base.sql",
+    "migrations/0029_automations.sql",
     "assets/"
   ],
   "scripts": {
     "dev": "wrangler dev --port 8789",
     "build": "tsc -p tsconfig.build.json --noEmit",
-    "db:migrate:local": "wrangler d1 execute beech-db --local --file=./migrations/0000_v040_base.sql",
+    "db:migrate:local": "wrangler d1 execute beech-db --local --file=./migrations/0000_v040_base.sql && wrangler d1 execute beech-db --local --file=./migrations/0029_automations.sql",
     "db:reset:local": "node -e \"require('fs').rmSync('.wrangler/state', {recursive:true,force:true})\" && npm run db:migrate:local && cd ../.. && node bin/cli.mjs seed:load --local && cd apps/api && wrangler d1 execute beech-db --local --file=./migrations/0028_v040_seed_data.sql",
     "deploy": "wrangler deploy --minify",
     "cf-typegen": "wrangler types --env-interface CloudflareBindings",
@@ -22,7 +23,7 @@
   },
   "dependencies": {
     "@aws-sdk/client-s3": "^3.995.0",
-    "@beechcms/core": "^0.4.1",
+    "@beechcms/core": "^0.4.3",
     "bcryptjs": "^2.4.3",
     "hono": "^4.11.9",
     "jose": "^6.1.3"

package/src/factory.ts CHANGED Viewed

@@ -26,6 +26,7 @@ import { draftApp } from './features/draft'
 import { settingsApp } from './features/settings/settings.handler'
 import { schemaApp } from './features/schema/schema.handler'
 import { notificationsApp } from './features/notifications'
+import { automationsApp } from './features/automations'
 import { statsApp } from './features/stats'
 import { uploadRoutes, serveMediaHandler } from './upload'
 import { publicRoutes, apiKeyMiddleware, publicRateLimitMiddleware } from './public'
@@ -243,11 +244,13 @@ export function createBeechApp(config: BeechConfig): Hono<{ Bindings: Env; Varia
       if (!activeSession) return context.json({ error: 'Invalid refresh token' }, 401)
       const user = await context.get('userRepository').findById(activeSession.userId)
-      if (!user) return context.json({ error: 'User not found' }, 401)
-      const revoked = await context.get('sessionRepository').revokeByHash(tokenHash, nowSeconds)
-      if (!revoked) return context.json({ error: 'Invalid refresh token' }, 401)
+      if (!user) {
+        await context.get('sessionRepository').revokeByHash(tokenHash, nowSeconds)
+        return context.json({ error: 'User not found' }, 401)
+      }
+      // Issue new tokens before revoking the old one: if saveRefreshToken fails,
+      // the old token stays valid and the user is not locked out.
       const newAccessToken = await context.get('tokenService').issue({ sub: user.id, email: user.email, name: user.name ?? undefined })
       const newRefreshToken = generateRefreshToken()
       const newRefreshTokenHash = await sha256hex(newRefreshToken)
@@ -258,6 +261,10 @@ export function createBeechApp(config: BeechConfig): Hono<{ Bindings: Env; Varia
         tokenHash: newRefreshTokenHash,
         expiresAt: nowSeconds + REFRESH_TOKEN_EXPIRY_DAYS * SECONDS_PER_DAY,
       })
+      const revoked = await context.get('sessionRepository').revokeByHash(tokenHash, nowSeconds)
+      if (!revoked) return context.json({ error: 'Invalid refresh token' }, 401)
       setCookie(context, 'refresh_token', newRefreshToken, getRefreshTokenCookieOptions(isRequestSecure(context.req.url)))
       return context.json({ token: newAccessToken, expiresIn: '15m' }, 200)
     } catch (error) {
@@ -296,6 +303,7 @@ export function createBeechApp(config: BeechConfig): Hono<{ Bindings: Env; Varia
   apiProtected.route('/content', draftApp)
   apiProtected.route('/content', contentFeature)
   apiProtected.route('/widget', widgetApp)
+  apiProtected.route('/automations', automationsApp)
   apiProtected.route('/', uploadRoutes)
   // 6. Public API (must be registered before apiProtected to avoid auth middleware interception)

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

@@ -0,0 +1,268 @@
+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'
+// ── 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', () => {
+  beforeEach(() => { vi.restoreAllMocks() })
+  it('POSTs to URL with JSON body derived from entry when no body_template', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true, status: 200 })
+    vi.stubGlobal('fetch', fetchMock)
+    const ctx = makeCtx()
+    await executeAction({ type: 'webhook', url: 'https://example.com/hook' }, ctx)
+    expect(fetchMock).toHaveBeenCalledOnce()
+    const [url, init] = fetchMock.mock.calls[0]
+    expect(url).toBe('https://example.com/hook')
+    expect(init.method).toBe('POST')
+    expect(JSON.parse(init.body)).toMatchObject({ id: 'entry-1', title: 'Test Entry' })
+  })
+  it('interpolates body_template with entry values', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', fetchMock)
+    const ctx = makeCtx()
+    await executeAction({ type: 'webhook', url: 'https://example.com/hook', body_template: '{"t":"{{title}}"}' }, ctx)
+    const [, init] = fetchMock.mock.calls[0]
+    expect(init.body).toBe('{"t":"Test Entry"}')
+  })
+  it('merges custom headers', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', fetchMock)
+    const ctx = makeCtx()
+    await executeAction({ type: 'webhook', url: 'https://x.com', headers: { 'X-Token': 'abc' } }, ctx)
+    const [, init] = fetchMock.mock.calls[0]
+    expect(init.headers['X-Token']).toBe('abc')
+  })
+  it('throws when response is not ok', async () => {
+    vi.stubGlobal('fetch', vi.fn().mockResolvedValue({ ok: false, status: 500 }))
+    const ctx = makeCtx()
+    await expect(executeAction({ type: 'webhook', url: 'https://x.com' }, ctx)).rejects.toThrow('500')
+  })
+  it('uses custom HTTP method', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true })
+    vi.stubGlobal('fetch', fetchMock)
+    const ctx = makeCtx()
+    await executeAction({ type: 'webhook', url: 'https://x.com', method: 'PUT' }, ctx)
+    expect(fetchMock.mock.calls[0][1].method).toBe('PUT')
+  })
+})
+// ── send_mail ─────────────────────────────────────────────────────────────────
+vi.mock('../../email', () => ({
+  sendAutomationMail: vi.fn().mockResolvedValue(undefined),
+}))
+describe('send_mail executor', () => {
+  beforeEach(async () => {
+    const { sendAutomationMail } = await import('../../email')
+    vi.mocked(sendAutomationMail).mockClear()
+  })
+  it('interpolates to/subject/body and calls sendAutomationMail', async () => {
+    const { sendAutomationMail } = await import('../../email')
+    const ctx = makeCtx({ entry: { id: '1', email: 'user@example.com', title: 'Hello' } })
+    await executeAction(
+      {
+        type: 'send_mail',
+        to: '{{email}}',
+        subject_template: 'Re: {{title}}',
+        body_template: 'Your entry {{title}} was created.',
+      },
+      ctx,
+    )
+    expect(sendAutomationMail).toHaveBeenCalledWith(
+      expect.objectContaining({
+        to: 'user@example.com',
+        subject: 'Re: Hello',
+        body: 'Your entry Hello was created.',
+      }),
+    )
+  })
+  it('substitutes default value [missing] and logs a warning when concrete data is absent', async () => {
+    const { sendAutomationMail } = await import('../../email')
+    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    const ctx = makeCtx({ entry: { id: '1' } })
+    await executeAction(
+      {
+        type: 'send_mail',
+        to: '{{email}}',
+        subject_template: 'Re: {{title}}',
+        body_template: 'Content: {{desc}}',
+      },
+      ctx,
+    )
+    expect(sendAutomationMail).toHaveBeenCalledWith(
+      expect.objectContaining({
+        to: '[missing]',
+        subject: 'Re: [missing]',
+        body: 'Content: [missing]',
+      }),
+    )
+    expect(warnSpy).toHaveBeenCalledWith(
+      expect.stringContaining('missing concrete data'),
+      ['email', 'title', 'desc'],
+    )
+  })
+  it('skips execution gracefully and logs an error without calling sendAutomationMail if apiKey is missing', async () => {
+    const { sendAutomationMail } = await import('../../email')
+    const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+    const ctx = makeCtx({ env: {} }) // missing email/resend api keys
+    await executeAction(
+      {
+        type: 'send_mail',
+        to: 'user@beech.io',
+        subject_template: 'Hello',
+        body_template: 'World',
+      },
+      ctx,
+    )
+    expect(sendAutomationMail).not.toHaveBeenCalled()
+    expect(errSpy).toHaveBeenCalledWith(expect.stringContaining('Execution skipped'))
+  })
+  it('logs error and rethrows when sendAutomationMail fails', async () => {
+    const { sendAutomationMail } = await import('../../email')
+    vi.mocked(sendAutomationMail).mockRejectedValueOnce(new Error('provider error'))
+    const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
+    const ctx = makeCtx()
+    await expect(
+      executeAction(
+        {
+          type: 'send_mail',
+          to: 'test@beech.io',
+          subject_template: 'Hello',
+          body_template: 'World',
+        },
+        ctx,
+      ),
+    ).rejects.toThrow('provider error')
+    expect(errSpy).toHaveBeenCalledWith(
+      expect.stringContaining('failed to send automation email'),
+      expect.any(Error),
+    )
+  })
+})
+// ── 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 ADDED Viewed

@@ -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 ADDED Viewed

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