@ossy/platform 3.20.1 → 3.20.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ossy/platform",
-  "version": "3.20.1",
+  "version": "3.20.2",
   "description": "Ossy application server runtime",
   "repository": {
     "type": "git",
@@ -79,5 +79,5 @@
     "Dockerfile",
     "docker-healthcheck.js"
   ],
-  "gitHead": "3e8bb118b3ef163792a004663ccb803ab51a8601"
+  "gitHead": "d0e5d69906917105609af4bb8bbdd2340d9fbaf1"
 }

package/src/tasks/task-service.js

@@ -16,6 +16,18 @@ import { resolveClientIp, runWithEventContext } from '@ossy/event-store'
 const SCHEDULER_INTERVAL_MS = 60_000
 const _serviceLog = createLogger('TaskService')
 
+/**
+ * Written on the request path. Dispatch must not resolve workspace triggers
+ * for these — that lists task instances once per configurable task.
+ * Projections still fold them on the change stream (#1026).
+ * Same set as aggregate-rebuild `excludeTypes`.
+ */
+const AUDIT_DISPATCH_TYPES = new Set([
+  '@ossy/platform/schema/task-run',
+  '@ossy/platform/schema/action-invocation',
+  '@ossy/platform/schema/meter-event',
+])
+
 function buildTaskContext (baseContext, extras = {}) {
   return {
     ...baseContext,
@@ -356,6 +368,7 @@ export class TaskService {
    */
   static async dispatch(event, { sdk } = {}) {
     if (!event) return
+    if (AUDIT_DISPATCH_TYPES.has(event.type)) return
 
     const baseSdk = sdk ?? await sdkBoundTo()
     const effectiveSdk = typeof baseSdk?.withEventContext === 'function'

package/src/tasks/task-service.spec.js

@@ -122,6 +122,61 @@ describe('TaskService triggers', () => {
     TaskService.setInstanceTriggersResolver(null)
   })
 
+  it('skips audit events before resolving instance triggers', async () => {
+    const calls = []
+    const previousTasks = TaskService._tasks
+    const previousById = TaskService._tasksById
+    TaskService._tasks = []
+    TaskService._tasksById = new Map()
+    TaskService.setInstanceTriggersResolver(async ({ taskId, workspaceId }) => {
+      calls.push({ taskId, workspaceId })
+      return []
+    })
+    TaskService.registerTask({
+      metadata: {
+        id: '@ossy/media-tasks/tasks/resize-common-web',
+        configurable: true,
+        triggers: [{ type: '@ossy/platform/schema/file', event: 'Created' }],
+      },
+      run () {},
+    })
+
+    const auditEvent = (type) => ({
+      id: `evt-${type}`,
+      type,
+      event: 'Recorded',
+      resourceId: 'r1',
+      payload: { belongsTo: 'ws1' },
+    })
+
+    try {
+      for (const type of [
+        '@ossy/platform/schema/task-run',
+        '@ossy/platform/schema/action-invocation',
+        '@ossy/platform/schema/meter-event',
+      ]) {
+        await TaskService.dispatch(auditEvent(type), { sdk: {} })
+      }
+      expect(calls).toEqual([])
+
+      await TaskService.dispatch({
+        id: 'evt-file',
+        type: '@ossy/platform/schema/file',
+        event: 'Created',
+        resourceId: 'file1',
+        payload: { belongsTo: 'ws1', location: '/incoming/a.jpg' },
+      }, { sdk: {} })
+      expect(calls).toEqual([{
+        taskId: '@ossy/media-tasks/tasks/resize-common-web',
+        workspaceId: 'ws1',
+      }])
+    } finally {
+      TaskService._tasks = previousTasks
+      TaskService._tasksById = previousById
+      TaskService.setInstanceTriggersResolver(null)
+    }
+  })
+
   it('uses instance triggers for configurable tasks', async () => {
     const instanceTriggers = [{
       kind: 'on_event',