@contember/engine-actions 2.1.0-alpha.23 → 2.1.0-alpha.25

package/package.json

@@ -22,20 +22,20 @@
     }
   },
   "dependencies": {
-    "@contember/authorization": "2.1.0-alpha.23",
-    "@contember/database": "2.1.0-alpha.23",
-    "@contember/database-migrations": "2.1.0-alpha.23",
-    "@contember/engine-common": "2.1.0-alpha.23",
-    "@contember/engine-content-api": "2.1.0-alpha.23",
-    "@contember/engine-http": "2.1.0-alpha.23",
-    "@contember/engine-plugins": "2.1.0-alpha.23",
-    "@contember/engine-system-api": "2.1.0-alpha.23",
-    "@contember/engine-tenant-api": "2.1.0-alpha.23",
-    "@contember/graphql-utils": "2.1.0-alpha.23",
-    "@contember/logger": "2.1.0-alpha.23",
-    "@contember/schema": "2.1.0-alpha.23",
-    "@contember/schema-utils": "2.1.0-alpha.23",
-    "@contember/typesafe": "2.1.0-alpha.23",
+    "@contember/authorization": "2.1.0-alpha.25",
+    "@contember/database": "2.1.0-alpha.25",
+    "@contember/database-migrations": "2.1.0-alpha.25",
+    "@contember/engine-common": "2.1.0-alpha.25",
+    "@contember/engine-content-api": "2.1.0-alpha.25",
+    "@contember/engine-http": "2.1.0-alpha.25",
+    "@contember/engine-plugins": "2.1.0-alpha.25",
+    "@contember/engine-system-api": "2.1.0-alpha.25",
+    "@contember/engine-tenant-api": "2.1.0-alpha.25",
+    "@contember/graphql-utils": "2.1.0-alpha.25",
+    "@contember/logger": "2.1.0-alpha.25",
+    "@contember/schema": "2.1.0-alpha.25",
+    "@contember/schema-utils": "2.1.0-alpha.25",
+    "@contember/typesafe": "2.1.0-alpha.25",
     "@graphql-tools/schema": "^10.0.7",
     "graphql-tag": "^2.12.6"
   },
@@ -49,5 +49,5 @@
     "pg": "^8.5.0"
   },
   "type": "module",
-  "version": "2.1.0-alpha.23"
+  "version": "2.1.0-alpha.25"
 }

package/src/ActionsExecutionContainerHookFactory.ts

@@ -12,12 +12,12 @@ export class ActionsExecutionContainerHookFactory {
 
 	create(): ExecutionContainerHook {
 		return builder => {
-			return builder.setupService('mapperFactory', (mapperFactory, { whereBuilder, schema, pathFactory, systemSchema, providers, stage, project, schemaMeta }) => {
+			return builder.setupService('mapperFactory', (mapperFactory, { whereBuilder, schema, pathFactory, systemSchema, providers, stage, project, schemaMeta, joinBuilder, userInfo }) => {
 				mapperFactory.hooks.push(mapper => {
-					const triggerPayloadPersister = new TriggerPayloadPersister(mapper, mapper.db.forSchema(systemSchema), providers, project.slug, stage.id, schemaMeta.id)
+					const triggerPayloadPersister = new TriggerPayloadPersister(mapper, mapper.db.forSchema(systemSchema), providers, project.slug, stage.id, schemaMeta.id, mapper.identityId, userInfo)
 					const triggerPayloadBuilder = new TriggerPayloadBuilder(mapper)
 					const payloadManager = new TriggerPayloadManager(triggerPayloadBuilder, triggerPayloadPersister)
-					const changesFetcher = new TriggerIndirectChangesFetcher(mapper, whereBuilder, pathFactory)
+					const changesFetcher = new TriggerIndirectChangesFetcher(schema.model, mapper, whereBuilder, joinBuilder, pathFactory)
 					const listenersStore = this.listenerStoreProvider.getListenerStore(schema)
 					const triggerHandler = new TriggerHandler(payloadManager, listenersStore, changesFetcher)
 					triggerHandler.attach(mapper.eventManager)

package/src/dispatch/WebhookTargetHandler.ts

@@ -174,6 +174,9 @@ export class WebhookTargetHandler implements InvokeHandler<Actions.WebhookTarget
 			meta: {
 				eventId: it.id,
 				transactionId: it.transaction_id,
+				identityId: it.identity_id ?? null,
+				ipAddress: it.ip_address ?? null,
+				userAgent: it.user_agent ?? null,
 				createdAt: it.created_at.toISOString(),
 				lastStateChange: it.last_state_change.toISOString(),
 				numRetries: it.num_retries,

package/src/graphql/resolvers/query/EventsQueryResolver.ts

@@ -68,6 +68,9 @@ export class EventsQueryResolver implements QueryResolvers<ActionsContext> {
 			state: row.state,
 			target: row.target,
 			transactionId: row.transaction_id,
+			identityId: row.identity_id ?? undefined,
+			ipAddress: row.ip_address ?? undefined,
+			userAgent: row.user_agent ?? undefined,
 		}))
 	}
 }

package/src/graphql/schema/actions.graphql.ts

@@ -35,6 +35,9 @@ export const schema: DocumentNode = gql`
     type Event {
         id: Uuid!
         transactionId: Uuid!
+        identityId: Uuid
+        ipAddress: String
+        userAgent: String
         createdAt: DateTime!
         lastStateChange: DateTime!
         resolvedAt: DateTime
@@ -59,11 +62,11 @@ export const schema: DocumentNode = gql`
     type ProcessBatchResponse {
         ok: Boolean!
     }
-    
+
     type RetryEventResponse {
         ok: Boolean!
     }
-    
+
     type StopEventResponse {
         ok: Boolean!
     }
@@ -72,12 +75,12 @@ export const schema: DocumentNode = gql`
         variables: [VariableInput!]!
         mode: SetVariablesMode
     }
-	
+
 	"""
-	Defines how it handles original variables. 
+	Defines how it handles original variables.
 	- MERGE merges with new values (default behaviour)
 	- SET replaces all variables
-	- APPEND_ONLY_MISSING appends values if not already exist 
+	- APPEND_ONLY_MISSING appends values if not already exist
 	"""
 	enum SetVariablesMode {
 		MERGE
@@ -89,7 +92,7 @@ export const schema: DocumentNode = gql`
         name: String!
         value: String!
     }
-	
+
 	type SetVariablesResponse {
 		ok: Boolean!
 	}

package/src/graphql/schema/index.ts

@@ -23,6 +23,8 @@ export type Event = {
 	readonly __typename?: 'Event'
 	readonly createdAt: Scalars['DateTime']['output']
 	readonly id: Scalars['Uuid']['output']
+	readonly identityId?: Maybe<Scalars['Uuid']['output']>
+	readonly ipAddress?: Maybe<Scalars['String']['output']>
 	readonly lastStateChange: Scalars['DateTime']['output']
 	readonly log: Scalars['Json']['output']
 	readonly numRetries: Scalars['Int']['output']
@@ -32,6 +34,7 @@ export type Event = {
 	readonly state: EventState
 	readonly target: Scalars['String']['output']
 	readonly transactionId: Scalars['Uuid']['output']
+	readonly userAgent?: Maybe<Scalars['String']['output']>
 	readonly visibleAt?: Maybe<Scalars['DateTime']['output']>
 }
 
@@ -268,6 +271,8 @@ export interface DateTimeScalarConfig extends GraphQLScalarTypeConfig<ResolversT
 export type EventResolvers<ContextType = any, ParentType extends ResolversParentTypes['Event'] = ResolversParentTypes['Event']> = {
 	createdAt?: Resolver<ResolversTypes['DateTime'], ParentType, ContextType>
 	id?: Resolver<ResolversTypes['Uuid'], ParentType, ContextType>
+	identityId?: Resolver<Maybe<ResolversTypes['Uuid']>, ParentType, ContextType>
+	ipAddress?: Resolver<Maybe<ResolversTypes['String']>, ParentType, ContextType>
 	lastStateChange?: Resolver<ResolversTypes['DateTime'], ParentType, ContextType>
 	log?: Resolver<ResolversTypes['Json'], ParentType, ContextType>
 	numRetries?: Resolver<ResolversTypes['Int'], ParentType, ContextType>
@@ -277,6 +282,7 @@ export type EventResolvers<ContextType = any, ParentType extends ResolversParent
 	state?: Resolver<ResolversTypes['EventState'], ParentType, ContextType>
 	target?: Resolver<ResolversTypes['String'], ParentType, ContextType>
 	transactionId?: Resolver<ResolversTypes['Uuid'], ParentType, ContextType>
+	userAgent?: Resolver<Maybe<ResolversTypes['String']>, ParentType, ContextType>
 	visibleAt?: Resolver<Maybe<ResolversTypes['DateTime']>, ParentType, ContextType>
 	__isTypeOf?: IsTypeOfResolverFn<ParentType, ContextType>
 }

package/src/migrations/2025-05-28-160000-action-event-user.ts

@@ -0,0 +1,16 @@
+import { MigrationBuilder } from '@contember/database-migrations'
+
+const sql = `
+ALTER TABLE actions_event
+ADD COLUMN IF NOT EXISTS identity_id UUID DEFAULT NULL;
+
+ALTER TABLE actions_event
+ADD COLUMN IF NOT EXISTS ip_address INET DEFAULT NULL;
+
+ALTER TABLE actions_event
+ADD COLUMN IF NOT EXISTS user_agent TEXT DEFAULT NULL;
+`
+
+export default async function (builder: MigrationBuilder) {
+	builder.sql(sql)
+}

package/src/migrations/index.ts

@@ -1,7 +1,9 @@
 import snapshot from './snapshot'
 import { MigrationGroup } from '@contember/database-migrations'
 import _20230310145000actionsvariables from './2023-03-10-145000-actions-variables'
+import _20250528160000actioneventuser from './2025-05-28-160000-action-event-user'
 
 export const migrationsGroup = new MigrationGroup(snapshot, {
 	'2023-03-10-145000-actions-variables': _20230310145000actionsvariables,
+	'2025-05-28-160000-action-event-user': _20250528160000actioneventuser,
 })

package/src/migrations/snapshot.ts

@@ -30,11 +30,14 @@ export default async function (builder: MigrationBuilder) {
 			trigger         TEXT                NOT NULL,
 			priority        INT                 NOT NULL DEFAULT 0,
 			payload         JSONB               NOT NULL,
-			log             JSONB               NOT NULL DEFAULT '[]'
+			log             JSONB               NOT NULL DEFAULT '[]',
+			identity_id     UUID                DEFAULT NULL,
+			ip_address      INET                DEFAULT NULL,
+			user_agent      TEXT                DEFAULT NULL
 		);
 
 		CREATE INDEX ON actions_event (priority DESC, visible_at) WHERE state IN ('created', 'retrying', 'processing');
-		
+
 		CREATE TABLE actions_variable
     	(
         	id              UUID PRIMARY KEY NOT NULL,

package/src/model/types.ts

@@ -14,6 +14,9 @@ export type EventRow = {
 	target: string
 	priority: number
 	trigger: string
+	identity_id: string | null
+	ip_address: string | null
+	user_agent: string | null
 	payload: ActionsPayload.AnyEventPayload
 	log: any[]
 }

package/src/triggers/TriggerHandler.ts

@@ -71,28 +71,28 @@ export class TriggerHandler {
 
 	private async directUpdateHandler(event: AfterUpdateEvent) {
 		const updateListeners = this.listenersStore.getUpdateListeners(event.entity.name)
-		updateListeners.map(listener => {
+		await Promise.all(updateListeners.map(listener => {
 			if (event.data.some(it => listener.fields.has(it.fieldName) && it.value !== it.old)) {
-				this.payloadManager.add({
+				return this.payloadManager.add({
 					listener,
 					entity: event.entity,
 					primary: event.id,
 					cause: event,
 				})
 			}
-		})
+		}))
 	}
 
 	private async directCreateHandler(event: AfterInsertEvent) {
 		const createListeners = this.listenersStore.getCreateListener(event.entity.name)
-		createListeners.map(listener => {
-			this.payloadManager.add({
+		await Promise.all(createListeners.map(listener => {
+			return this.payloadManager.add({
 				listener,
 				entity: event.entity,
 				primary: event.id,
 				cause: event,
 			})
-		})
+		}))
 	}
 
 
@@ -103,7 +103,7 @@ export class TriggerHandler {
 	private async indirectChangesEntityHandler(event: BeforeDeleteEvent | AfterInsertEvent) {
 		//
 		const indirectEntityListeners = this.listenersStore.getIndirectListeners(event.entity.name)
-		const promises =  indirectEntityListeners.map(listener => this.collectDeepChanges(event, listener, {
+		const promises = indirectEntityListeners.map(listener => this.collectDeepChanges(event, listener, {
 			[event.entity.primary]: { eq: event.id },
 		}))
 		await Promise.all(promises)
@@ -149,13 +149,14 @@ export class TriggerHandler {
 		where: Input.Where,
 	) {
 		const result = await this.changesFetcher.fetch(listener, where)
-		for (const row of result) {
-			this.payloadManager.add({
+		await Promise.all(result.map(row => {
+			return this.payloadManager.add({
 				listener,
 				entity: listener.rootEntity,
-				primary: row,
+				primary: row.id,
 				cause,
+				nodes: row.nodes,
 			})
-		}
+		}))
 	}
 }

package/src/triggers/TriggerIndirectChangesFetcher.ts

@@ -1,12 +1,21 @@
 import { Mapper, PathFactory, WhereBuilder } from '@contember/engine-content-api'
 import { IndirectListener, JunctionListener } from './TriggerListenersStore'
 import { SelectBuilder } from '@contember/database'
-import { Input } from '@contember/schema'
+import { ActionsPayload, Input, Model, Schema } from '@contember/schema'
+import { JoinBuilder } from '@contember/engine-content-api'
+import { getTargetEntity } from '@contember/schema-utils'
+
+export type TriggerIndirectChangesFetcherResult = {
+	id: Input.PrimaryValue
+	nodes?: ActionsPayload.EntityEventPathNode[]
+}[]
 
 export class TriggerIndirectChangesFetcher {
 	constructor(
+		private readonly model: Model.Schema,
 		private readonly mapper: Mapper,
 		private readonly whereBuilder: WhereBuilder,
+		private readonly joinBuilder: JoinBuilder,
 		private readonly pathFactory: PathFactory,
 	) {
 	}
@@ -14,7 +23,11 @@ export class TriggerIndirectChangesFetcher {
 	public async fetch(
 		listener: IndirectListener | JunctionListener,
 		where: Input.Where,
-	): Promise<Input.PrimaryValue[]> {
+	): Promise<TriggerIndirectChangesFetcherResult> {
+		if (listener.trigger.withNodes) {
+			return this.fetchWithNodes(listener, where)
+		}
+
 		for (let i = listener.path.length - 1; i >= 0; i--) {
 			where = { [listener.path[i]]: where }
 		}
@@ -24,7 +37,53 @@ export class TriggerIndirectChangesFetcher {
 		const path = this.pathFactory.create([])
 		const qbWithWhere = this.whereBuilder.build(qb, listener.rootEntity, path, where)
 
-		return (await qbWithWhere.getResult(this.mapper.db)).map(it => it.id)
+		return (await qbWithWhere.getResult(this.mapper.db))
 	}
 
+	private async fetchWithNodes(
+		listener: IndirectListener | JunctionListener,
+		where: Input.Where,
+	): Promise<TriggerIndirectChangesFetcherResult> {
+		const basePath = this.pathFactory.create([])
+		let path = basePath
+		let entity = listener.rootEntity
+		let qb = SelectBuilder.create()
+			.from(entity.tableName, path.alias)
+			.select([path.alias, entity.primaryColumn], path.for(entity.primary).alias)
+
+		for (let i = 0; i < listener.path.length; i++) {
+			const relationName = listener.path[i]
+			path = path.for(relationName)
+			const targetEntity = getTargetEntity(this.model, entity, relationName)
+			if (!targetEntity) {
+				throw new Error(`Target entity for relation ${entity.name}::${relationName} not found`)
+			}
+			qb = this.joinBuilder.join(qb, path, entity, relationName)
+
+			entity = targetEntity
+			qb = qb.select([path.alias, targetEntity.primaryColumn], path.for(targetEntity.primary).alias)
+		}
+
+		const qbWithWhere = this.whereBuilder.build(qb, entity, path, where)
+
+		const result = await qbWithWhere.getResult(this.mapper.db)
+
+		return result.map(row => {
+			const nodes: ActionsPayload.EntityEventPathNode[] = []
+			let entity = listener.rootEntity
+			let path = basePath
+			const id = row[path.for(entity.primary).alias]
+			for (let i = 0; i < listener.path.length; i++) {
+				const relationName = listener.path[i]
+				const targetEntity = getTargetEntity(this.model, entity, relationName)
+				if (!targetEntity) {
+					throw new Error(`Target entity for relation ${entity.name}::${relationName} not found`)
+				}
+				path = path.for(relationName)
+				nodes.push({ relation: relationName, entity: targetEntity.name, id: row[path.for(targetEntity.primary).alias] })
+				entity = targetEntity
+			}
+			return { id, nodes }
+		})
+	}
 }

package/src/triggers/TriggerListenersStore.ts

@@ -3,7 +3,7 @@ import { TriggerListeners } from './TriggerListeners'
 
 export interface IndirectListener {
 	type: 'indirect'
-	trigger: Actions.AnyTrigger
+	trigger: Actions.WatchTrigger
 	rootEntity: Model.Entity
 	fields: Set<string>
 	relations: Set<string>
@@ -12,7 +12,7 @@ export interface IndirectListener {
 
 export interface JunctionListener {
 	type: 'junction'
-	trigger: Actions.AnyTrigger
+	trigger: Actions.WatchTrigger
 	rootEntity: Model.Entity
 	path: string[]
 	context: {

package/src/triggers/TriggerPayloadBuilder.ts

@@ -51,6 +51,19 @@ export class TriggerPayloadBuilder {
 		}
 		const payloads: ActionsPayload.WatchEventPayload[] = []
 		for (const eventsByPrimary of Object.values(byPrimary)) {
+			const baseEvents: ActionsPayload.BaseEventPayload[] = []
+
+			const existingEvents = new Set<string>()
+			for (const event of eventsByPrimary.events) {
+				const baseEvent = this.buildBaseEventPayloads(event)
+				const stringRepresentation = JSON.stringify(baseEvent)
+				if (existingEvents.has(stringRepresentation)) {
+					continue
+				}
+				existingEvents.add(stringRepresentation)
+				baseEvents.push(baseEvent)
+			}
+
 			payloads.push({
 				operation: 'watch',
 				trigger: events[0].listener.trigger.name,
@@ -58,7 +71,7 @@ export class TriggerPayloadBuilder {
 				id: eventsByPrimary.primary,
 				selection: eventsByPrimary.events.find(it => it.selection)?.selection
 					?? selections[eventsByPrimary.primary],
-				events: eventsByPrimary.events.map(it => this.buildBaseEventPayloads(it)),
+				events: baseEvents,
 			})
 		}
 		return payloads
@@ -74,6 +87,7 @@ export class TriggerPayloadBuilder {
 					selection,
 					values: Object.fromEntries(event.cause.data.map(it => [it.fieldName, it.value])),
 					path: 'path' in event.listener ? event.listener.path : undefined,
+					nodes: event.nodes,
 				}
 			case 'AfterUpdateEvent':
 			case 'BeforeUpdateEvent':
@@ -89,6 +103,7 @@ export class TriggerPayloadBuilder {
 					values: Object.fromEntries(event.cause.data.map(it => [it.fieldName, it.value])),
 					path: 'path' in event.listener ? event.listener.path : undefined,
 					old: Object.fromEntries(afterEvent.data.map(it => [it.fieldName, it.old])),
+					nodes: event.nodes,
 				}
 			case 'BeforeDeleteEvent':
 				return {
@@ -97,6 +112,7 @@ export class TriggerPayloadBuilder {
 					id: event.cause.id,
 					selection,
 					path: 'path' in event.listener ? event.listener.path : undefined,
+					nodes: event.nodes,
 				}
 			case 'AfterJunctionUpdateEvent':
 				if (event.listener.type !== 'junction') {
@@ -112,6 +128,7 @@ export class TriggerPayloadBuilder {
 					id,
 					inverseId,
 					path: 'path' in event.listener ? event.listener.path : undefined,
+					nodes: event.nodes,
 				}
 			default:
 				assertNever(event.cause, it => it.type)

package/src/triggers/TriggerPayloadManager.ts

@@ -1,4 +1,4 @@
-import { Actions, Input, Model } from '@contember/schema'
+import { Actions, ActionsPayload, Input, Model } from '@contember/schema'
 import { AnyListener } from './TriggerListenersStore'
 import { EventCause } from './TriggerHandler'
 import { TriggerPayloadBuilder } from './TriggerPayloadBuilder'
@@ -11,6 +11,7 @@ export interface FiredEvent {
 	entity: Model.Entity
 	primary: Input.PrimaryValue
 	selection?: any
+	nodes?: ActionsPayload.EntityEventPathNode[]
 }
 
 export class TriggerPayloadManager {

package/src/triggers/TriggerPayloadPersister.ts

@@ -18,7 +18,9 @@ export class TriggerPayloadPersister {
 		private readonly providers: { uuid: () => string },
 		private readonly projectSlug: string,
 		private readonly stageId: string,
-		private readonly schemaId?: number,
+		private readonly schemaId: number | undefined,
+		private readonly identityId: string,
+		private readonly userInfo: { ipAddress: string | null; userAgent: string | null },
 	) {
 	}
 
@@ -46,6 +48,9 @@ export class TriggerPayloadPersister {
 					payload: it,
 					resolved_at: null,
 					last_state_change: 'now',
+					identity_id: this.identityId,
+					ip_address: this.userInfo.ipAddress,
+					user_agent: this.userInfo.userAgent,
 				})))
 				.execute(this.client)
 		}

package/tests/cases/unit/generator.test.ts

@@ -8,7 +8,7 @@ const assert = {
 	equal: (a: any, b: any) => expect(a).toEqual(b),
 	deepStrictEqual: (a: any, b: any) => expect(a).toStrictEqual(b),
 }
-const createTrigger = (watchNode: Actions.SelectionNode): Actions.AnyTrigger => ({
+const createTrigger = (watchNode: Actions.SelectionNode): Actions.WatchTrigger => ({
 	type: 'watch',
 	entity: 'Author',
 	name: 'author_trigger',

package/tests/cases/unit/webhookHandler.test.ts

@@ -29,6 +29,9 @@ const createTestEvent = (i = 0, row: Partial<EventRow> = {}): EventRow => ({
 	priority: 1,
 	schema_id: 1,
 	state: 'created',
+	identity_id: testUuid(i * 10 + 4),
+	ip_address: '127.0.0.1',
+	user_agent: 'test-agent',
 	...row,
 })
 
@@ -56,6 +59,9 @@ describe('webhook request', () => {
 						meta: {
 							eventId: testUuid(1),
 							transactionId: testUuid(2),
+							identityId: testUuid(4),
+							ipAddress: '127.0.0.1',
+							userAgent: 'test-agent',
 							createdAt: now.toISOString(),
 							lastStateChange: now.toISOString(),
 							numRetries: 0,
@@ -104,6 +110,9 @@ describe('webhook request', () => {
 								meta: {
 									eventId: testUuid(1),
 									transactionId: testUuid(2),
+									identityId: testUuid(4),
+									ipAddress: '127.0.0.1',
+									userAgent: 'test-agent',
 									createdAt: now.toISOString(),
 									lastStateChange: now.toISOString(),
 									numRetries: 0,