n8n 2.36.7 → 2.37.1

package/dist/modules/workflow-reviews.ee/workflow-review-lifecycle.service.js

@@ -0,0 +1,256 @@
+"use strict";
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+var __metadata = (this && this.__metadata) || function (k, v) {
+    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.WorkflowReviewLifecycleService = void 0;
+const backend_common_1 = require("@n8n/backend-common");
+const db_1 = require("@n8n/db");
+const di_1 = require("@n8n/di");
+const collaboration_service_1 = require("../../collaboration/collaboration.service");
+const event_service_1 = require("../../events/event.service");
+const MAX_PENDING_DELETE_CAPTURES = 1000;
+let WorkflowReviewLifecycleService = class WorkflowReviewLifecycleService {
+    constructor(logger, workflowReviewRequestRepository, workflowReviewRequestWorkflowRepository, activityRepository, dbLockService, collaborationService, eventService) {
+        this.logger = logger;
+        this.workflowReviewRequestRepository = workflowReviewRequestRepository;
+        this.workflowReviewRequestWorkflowRepository = workflowReviewRequestWorkflowRepository;
+        this.activityRepository = activityRepository;
+        this.dbLockService = dbLockService;
+        this.collaborationService = collaborationService;
+        this.eventService = eventService;
+        this.pendingDeleteCaptures = new Map();
+    }
+    async afterWorkflowArchived(workflowId, userId) {
+        await this.recordCauseEventsAndApplyClosePolicy([workflowId], 'workflow.archived', userId);
+        await this.reconcileUnreviewableOpenRequests([workflowId]);
+    }
+    async afterWorkflowsTransferred(workflowIds, userId) {
+        await this.recordCauseEventsAndApplyClosePolicy(workflowIds, 'workflow.moved', userId);
+        await this.reconcileUnreviewableOpenRequests(workflowIds);
+    }
+    async beforeWorkflowDeleted(workflowId, userId) {
+        try {
+            const openRequests = await this.workflowReviewRequestRepository.findOpenRequestsForWorkflows([workflowId], {});
+            this.pendingDeleteCaptures.delete(workflowId);
+            if (this.pendingDeleteCaptures.size >= MAX_PENDING_DELETE_CAPTURES) {
+                const oldest = this.pendingDeleteCaptures.keys().next().value;
+                if (oldest !== undefined)
+                    this.pendingDeleteCaptures.delete(oldest);
+            }
+            this.pendingDeleteCaptures.set(workflowId, {
+                userId,
+                requestIds: openRequests.map(({ request }) => request.id),
+            });
+        }
+        catch (error) {
+            this.logger.warn('Failed to capture open workflow review request(s) before deletion', {
+                workflowId,
+                error,
+            });
+        }
+    }
+    async afterWorkflowsDeleted(workflowIds) {
+        const capturesByRequestId = new Map();
+        for (const workflowId of workflowIds) {
+            const capture = this.pendingDeleteCaptures.get(workflowId);
+            this.pendingDeleteCaptures.delete(workflowId);
+            if (!capture)
+                continue;
+            for (const requestId of capture.requestIds) {
+                const grouped = capturesByRequestId.get(requestId) ?? {
+                    workflowIds: [],
+                    userId: capture.userId,
+                };
+                grouped.workflowIds.push(workflowId);
+                capturesByRequestId.set(requestId, grouped);
+            }
+        }
+        if (capturesByRequestId.size > 0) {
+            await this.recordCapturedDeletions(capturesByRequestId, workflowIds);
+        }
+        await this.reconcileUnreviewableOpenRequests(workflowIds);
+    }
+    async afterWorkflowPublished(event) {
+        try {
+            const requestIds = await this.workflowReviewRequestWorkflowRepository.findRequestIdsPinnedToVersion({ workflowId: event.workflowId, workflowVersionId: event.versionId }, {});
+            for (const requestId of requestIds) {
+                await this.activityRepository.createActivity({
+                    workflowReviewRequestId: requestId,
+                    type: 'workflow.published',
+                    data: { workflowId: event.workflowId, workflowVersionId: event.versionId },
+                    createdById: event.userId,
+                }, {});
+            }
+            this.broadcastReviewStateChanged([event.workflowId]);
+        }
+        catch (error) {
+            this.logger.warn('Failed to record workflow publication in review activity', {
+                workflowId: event.workflowId,
+                versionId: event.versionId,
+                error,
+            });
+        }
+    }
+    async reconcileUnreviewableOpenRequests(workflowIds) {
+        try {
+            const closedRequestIds = await this.dbLockService.withLockContext(1004, async (ctx) => {
+                const requestIds = await this.workflowReviewRequestRepository.closeUnreviewableOpenRequests(ctx);
+                for (const requestId of requestIds) {
+                    await this.activityRepository.createActivity({
+                        workflowReviewRequestId: requestId,
+                        type: 'review.closed',
+                        data: { reason: 'no-reviewable-workflows' },
+                        createdById: null,
+                    }, ctx);
+                }
+                return requestIds;
+            });
+            if (closedRequestIds.length === 0)
+                return;
+            for (const requestId of closedRequestIds) {
+                this.eventService.emit('workflow-review-closed', {
+                    workflowReviewRequestId: requestId,
+                    cause: { trigger: 'unknown', actorKind: 'system', userId: null },
+                });
+            }
+            this.logger.info('Closed open workflow review request(s) left on unreviewable workflows', {
+                workflowIds,
+                closedRequestIds,
+            });
+        }
+        catch (error) {
+            this.logger.error('Failed to close open workflow review request(s) left on unreviewable workflows', { workflowIds, error });
+        }
+    }
+    async recordCauseEventsAndApplyClosePolicy(workflowIds, type, userId) {
+        const actorKind = userId === null ? 'system' : 'user';
+        try {
+            const { affectedWorkflowIds, closedRequestIds } = await this.dbLockService.withLockContext(1004, async (ctx) => {
+                const openRequests = await this.workflowReviewRequestRepository.findOpenRequestsForWorkflows(workflowIds, ctx);
+                const affected = new Set();
+                const closedRequestIds = [];
+                for (const { request, links } of openRequests) {
+                    for (const { workflowId: linkedWorkflowId } of links) {
+                        await this.activityRepository.createActivity({
+                            workflowReviewRequestId: request.id,
+                            type,
+                            data: { workflowId: linkedWorkflowId, actorKind },
+                            createdById: userId,
+                        }, ctx);
+                        affected.add(linkedWorkflowId);
+                    }
+                    if (await this.applyClosePolicy(request, workflowIds, ctx)) {
+                        closedRequestIds.push(request.id);
+                    }
+                }
+                return { affectedWorkflowIds: [...affected], closedRequestIds };
+            });
+            for (const requestId of closedRequestIds) {
+                this.eventService.emit('workflow-review-closed', {
+                    workflowReviewRequestId: requestId,
+                    cause: {
+                        trigger: type === 'workflow.archived' ? 'workflow-archived' : 'workflow-moved',
+                        actorKind,
+                        userId,
+                    },
+                });
+            }
+            if (affectedWorkflowIds.length === 0)
+                return;
+            this.logger.info('Recorded workflow review cause event(s)', {
+                type,
+                workflowIds: affectedWorkflowIds,
+            });
+            this.broadcastReviewStateChanged(affectedWorkflowIds);
+        }
+        catch (error) {
+            this.logger.error('Failed to record workflow review cause event(s)', {
+                type,
+                workflowIds,
+                error,
+            });
+        }
+    }
+    async recordCapturedDeletions(capturesByRequestId, batchWorkflowIds) {
+        try {
+            const { affectedWorkflowIds, closedRequests } = await this.dbLockService.withLockContext(1004, async (ctx) => {
+                const affected = new Set();
+                const closed = [];
+                for (const [requestId, capture] of capturesByRequestId) {
+                    const request = await this.workflowReviewRequestRepository.findById(requestId, ctx);
+                    if (!request || request.state !== 'open')
+                        continue;
+                    const actorKind = capture.userId === null ? 'system' : 'user';
+                    for (const workflowId of capture.workflowIds) {
+                        await this.activityRepository.createActivity({
+                            workflowReviewRequestId: requestId,
+                            type: 'workflow.deleted',
+                            data: { workflowId, actorKind },
+                            createdById: capture.userId,
+                        }, ctx);
+                        affected.add(workflowId);
+                    }
+                    if (await this.applyClosePolicy(request, batchWorkflowIds, ctx)) {
+                        closed.push({ requestId, actorKind, userId: capture.userId });
+                    }
+                }
+                return { affectedWorkflowIds: [...affected], closedRequests: closed };
+            });
+            for (const { requestId, actorKind, userId } of closedRequests) {
+                this.eventService.emit('workflow-review-closed', {
+                    workflowReviewRequestId: requestId,
+                    cause: { trigger: 'workflow-deleted', actorKind, userId },
+                });
+            }
+            if (affectedWorkflowIds.length === 0)
+                return;
+            this.logger.info('Recorded workflow review cause event(s)', {
+                type: 'workflow.deleted',
+                workflowIds: affectedWorkflowIds,
+            });
+            this.broadcastReviewStateChanged(affectedWorkflowIds);
+        }
+        catch (error) {
+            this.logger.error('Failed to record workflow review cause event(s)', {
+                type: 'workflow.deleted',
+                workflowIds: batchWorkflowIds,
+                error,
+            });
+        }
+    }
+    async applyClosePolicy(request, affectedWorkflowIds, ctx) {
+        const staysOpen = await this.workflowReviewRequestRepository.hasReviewableWorkflowOutside(request.id, affectedWorkflowIds, ctx);
+        if (staysOpen)
+            return false;
+        request.state = 'closed';
+        request.closedById = null;
+        await this.workflowReviewRequestRepository.saveRequest(request, ctx);
+        await this.activityRepository.createActivity({
+            workflowReviewRequestId: request.id,
+            type: 'review.closed',
+            data: { reason: 'no-reviewable-workflows' },
+            createdById: null,
+        }, ctx);
+        return true;
+    }
+    broadcastReviewStateChanged(workflowIds) {
+        for (const workflowId of workflowIds) {
+            this.collaborationService
+                .broadcastWorkflowReviewStateChanged(workflowId)
+                .catch((error) => this.logger.warn('Failed to broadcast review state change', { workflowId, error }));
+        }
+    }
+};
+exports.WorkflowReviewLifecycleService = WorkflowReviewLifecycleService;
+exports.WorkflowReviewLifecycleService = WorkflowReviewLifecycleService = __decorate([
+    (0, di_1.Service)(),
+    __metadata("design:paramtypes", [backend_common_1.Logger, db_1.WorkflowReviewRequestRepository, db_1.WorkflowReviewRequestWorkflowRepository, db_1.WorkflowReviewActivityRepository, db_1.DbLockService, collaboration_service_1.CollaborationService, event_service_1.EventService])
+], WorkflowReviewLifecycleService);
+//# sourceMappingURL=workflow-review-lifecycle.service.js.map

package/dist/modules/workflow-reviews.ee/workflow-review-lifecycle.service.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-review-lifecycle.service.js","sourceRoot":"","sources":["../../../src/modules/workflow-reviews.ee/workflow-review-lifecycle.service.ts"],"names":[],"mappings":";;;;;;;;;;;;AACA,wDAA6C;AAC7C,gCAMiB;AACjB,gCAAkC;AAElC,iFAA6E;AAC7E,0DAAsD;AActD,MAAM,2BAA2B,GAAG,IAAI,CAAC;AAmBlC,IAAM,8BAA8B,GAApC,MAAM,8BAA8B;IAG1C,YACkB,MAAc,EACd,+BAAgE,EAChE,uCAAgF,EAChF,kBAAoD,EACpD,aAA4B,EAC5B,oBAA0C,EAC1C,YAA0B;QAN1B,WAAM,GAAN,MAAM,CAAQ;QACd,oCAA+B,GAA/B,+BAA+B,CAAiC;QAChE,4CAAuC,GAAvC,uCAAuC,CAAyC;QAChF,uBAAkB,GAAlB,kBAAkB,CAAkC;QACpD,kBAAa,GAAb,aAAa,CAAe;QAC5B,yBAAoB,GAApB,oBAAoB,CAAsB;QAC1C,iBAAY,GAAZ,YAAY,CAAc;QAT3B,0BAAqB,GAAG,IAAI,GAAG,EAAgC,CAAC;IAU9E,CAAC;IAEJ,KAAK,CAAC,qBAAqB,CAAC,UAAkB,EAAE,MAAqB;QACpE,MAAM,IAAI,CAAC,oCAAoC,CAAC,CAAC,UAAU,CAAC,EAAE,mBAAmB,EAAE,MAAM,CAAC,CAAC;QAC3F,MAAM,IAAI,CAAC,iCAAiC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC;IAC5D,CAAC;IAED,KAAK,CAAC,yBAAyB,CAAC,WAAqB,EAAE,MAAqB;QAC3E,MAAM,IAAI,CAAC,oCAAoC,CAAC,WAAW,EAAE,gBAAgB,EAAE,MAAM,CAAC,CAAC;QACvF,MAAM,IAAI,CAAC,iCAAiC,CAAC,WAAW,CAAC,CAAC;IAC3D,CAAC;IAOD,KAAK,CAAC,qBAAqB,CAAC,UAAkB,EAAE,MAAqB;QACpE,IAAI,CAAC;YACJ,MAAM,YAAY,GAAG,MAAM,IAAI,CAAC,+BAA+B,CAAC,4BAA4B,CAC3F,CAAC,UAAU,CAAC,EACZ,EAAE,CACF,CAAC;YAGF,IAAI,CAAC,qBAAqB,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;YAC9C,IAAI,IAAI,CAAC,qBAAqB,CAAC,IAAI,IAAI,2BAA2B,EAAE,CAAC;gBACpE,MAAM,MAAM,GAAG,IAAI,CAAC,qBAAqB,CAAC,IAAI,EAAE,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC;gBAC9D,IAAI,MAAM,KAAK,SAAS;oBAAE,IAAI,CAAC,qBAAqB,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;YACrE,CAAC;YACD,IAAI,CAAC,qBAAqB,CAAC,GAAG,CAAC,UAAU,EAAE;gBAC1C,MAAM;gBACN,UAAU,EAAE,YAAY,CAAC,GAAG,CAAC,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;aACzD,CAAC,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAChB,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,mEAAmE,EAAE;gBACrF,UAAU;gBACV,KAAK;aACL,CAAC,CAAC;QACJ,CAAC;IACF,CAAC;IAED,KAAK,CAAC,qBAAqB,CAAC,WAAqB;QAGhD,MAAM,mBAAmB,GAAG,IAAI,GAAG,EAA4D,CAAC;QAChG,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;YACtC,MAAM,OAAO,GAAG,IAAI,CAAC,qBAAqB,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;YAC3D,IAAI,CAAC,qBAAqB,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;YAC9C,IAAI,CAAC,OAAO;gBAAE,SAAS;YAEvB,KAAK,MAAM,SAAS,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;gBAC5C,MAAM,OAAO,GAAG,mBAAmB,CAAC,GAAG,CAAC,SAAS,CAAC,IAAI;oBACrD,WAAW,EAAE,EAAE;oBACf,MAAM,EAAE,OAAO,CAAC,MAAM;iBACtB,CAAC;gBACF,OAAO,CAAC,WAAW,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;gBACrC,mBAAmB,CAAC,GAAG,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;YAC7C,CAAC;QACF,CAAC;QAED,IAAI,mBAAmB,CAAC,IAAI,GAAG,CAAC,EAAE,CAAC;YAClC,MAAM,IAAI,CAAC,uBAAuB,CAAC,mBAAmB,EAAE,WAAW,CAAC,CAAC;QACtE,CAAC;QAKD,MAAM,IAAI,CAAC,iCAAiC,CAAC,WAAW,CAAC,CAAC;IAC3D,CAAC;IAQD,KAAK,CAAC,sBAAsB,CAAC,KAI5B;QACA,IAAI,CAAC;YACJ,MAAM,UAAU,GACf,MAAM,IAAI,CAAC,uCAAuC,CAAC,6BAA6B,CAC/E,EAAE,UAAU,EAAE,KAAK,CAAC,UAAU,EAAE,iBAAiB,EAAE,KAAK,CAAC,SAAS,EAAE,EACpE,EAAE,CACF,CAAC;YAEH,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;gBACpC,MAAM,IAAI,CAAC,kBAAkB,CAAC,cAAc,CAC3C;oBACC,uBAAuB,EAAE,SAAS;oBAClC,IAAI,EAAE,oBAAoB;oBAC1B,IAAI,EAAE,EAAE,UAAU,EAAE,KAAK,CAAC,UAAU,EAAE,iBAAiB,EAAE,KAAK,CAAC,SAAS,EAAE;oBAC1E,WAAW,EAAE,KAAK,CAAC,MAAM;iBACzB,EACD,EAAE,CACF,CAAC;YACH,CAAC;YAKD,IAAI,CAAC,2BAA2B,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC,CAAC,CAAC;QACtD,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAChB,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,0DAA0D,EAAE;gBAC5E,UAAU,EAAE,KAAK,CAAC,UAAU;gBAC5B,SAAS,EAAE,KAAK,CAAC,SAAS;gBAC1B,KAAK;aACL,CAAC,CAAC;QACJ,CAAC;IACF,CAAC;IAeO,KAAK,CAAC,iCAAiC,CAAC,WAAqB;QACpE,IAAI,CAAC;YACJ,MAAM,gBAAgB,GAAG,MAAM,IAAI,CAAC,aAAa,CAAC,eAAe,OAEhE,KAAK,EAAE,GAAG,EAAE,EAAE;gBACb,MAAM,UAAU,GACf,MAAM,IAAI,CAAC,+BAA+B,CAAC,6BAA6B,CAAC,GAAG,CAAC,CAAC;gBAO/E,KAAK,MAAM,SAAS,IAAI,UAAU,EAAE,CAAC;oBACpC,MAAM,IAAI,CAAC,kBAAkB,CAAC,cAAc,CAC3C;wBACC,uBAAuB,EAAE,SAAS;wBAClC,IAAI,EAAE,eAAe;wBACrB,IAAI,EAAE,EAAE,MAAM,EAAE,yBAAyB,EAAE;wBAC3C,WAAW,EAAE,IAAI;qBACjB,EACD,GAAG,CACH,CAAC;gBACH,CAAC;gBAED,OAAO,UAAU,CAAC;YACnB,CAAC,CACD,CAAC;YAEF,IAAI,gBAAgB,CAAC,MAAM,KAAK,CAAC;gBAAE,OAAO;YAE1C,KAAK,MAAM,SAAS,IAAI,gBAAgB,EAAE,CAAC;gBAE1C,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,wBAAwB,EAAE;oBAChD,uBAAuB,EAAE,SAAS;oBAClC,KAAK,EAAE,EAAE,OAAO,EAAE,SAAS,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE;iBAChE,CAAC,CAAC;YACJ,CAAC;YAED,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,uEAAuE,EAAE;gBACzF,WAAW;gBACX,gBAAgB;aAChB,CAAC,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAGhB,IAAI,CAAC,MAAM,CAAC,KAAK,CAChB,gFAAgF,EAChF,EAAE,WAAW,EAAE,KAAK,EAAE,CACtB,CAAC;QACH,CAAC;IACF,CAAC;IAMO,KAAK,CAAC,oCAAoC,CACjD,WAAqB,EACrB,IAA4C,EAC5C,MAAqB;QAErB,MAAM,SAAS,GAAG,MAAM,KAAK,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC;QAEtD,IAAI,CAAC;YACJ,MAAM,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,GAAG,MAAM,IAAI,CAAC,aAAa,CAAC,eAAe,OAEzF,KAAK,EAAE,GAAG,EAAE,EAAE;gBAGb,MAAM,YAAY,GACjB,MAAM,IAAI,CAAC,+BAA+B,CAAC,4BAA4B,CACtE,WAAW,EACX,GAAG,CACH,CAAC;gBAEH,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAU,CAAC;gBACnC,MAAM,gBAAgB,GAAa,EAAE,CAAC;gBACtC,KAAK,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,IAAI,YAAY,EAAE,CAAC;oBAC/C,KAAK,MAAM,EAAE,UAAU,EAAE,gBAAgB,EAAE,IAAI,KAAK,EAAE,CAAC;wBACtD,MAAM,IAAI,CAAC,kBAAkB,CAAC,cAAc,CAC3C;4BACC,uBAAuB,EAAE,OAAO,CAAC,EAAE;4BACnC,IAAI;4BACJ,IAAI,EAAE,EAAE,UAAU,EAAE,gBAAgB,EAAE,SAAS,EAAE;4BACjD,WAAW,EAAE,MAAM;yBACnB,EACD,GAAG,CACH,CAAC;wBACF,QAAQ,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC;oBAChC,CAAC;oBAED,IAAI,MAAM,IAAI,CAAC,gBAAgB,CAAC,OAAO,EAAE,WAAW,EAAE,GAAG,CAAC,EAAE,CAAC;wBAC5D,gBAAgB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;oBACnC,CAAC;gBACF,CAAC;gBAED,OAAO,EAAE,mBAAmB,EAAE,CAAC,GAAG,QAAQ,CAAC,EAAE,gBAAgB,EAAE,CAAC;YACjE,CAAC,CACD,CAAC;YAGF,KAAK,MAAM,SAAS,IAAI,gBAAgB,EAAE,CAAC;gBAC1C,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,wBAAwB,EAAE;oBAChD,uBAAuB,EAAE,SAAS;oBAClC,KAAK,EAAE;wBACN,OAAO,EAAE,IAAI,KAAK,mBAAmB,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,gBAAgB;wBAC9E,SAAS;wBACT,MAAM;qBACN;iBACD,CAAC,CAAC;YACJ,CAAC;YAED,IAAI,mBAAmB,CAAC,MAAM,KAAK,CAAC;gBAAE,OAAO;YAE7C,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,yCAAyC,EAAE;gBAC3D,IAAI;gBACJ,WAAW,EAAE,mBAAmB;aAChC,CAAC,CAAC;YAEH,IAAI,CAAC,2BAA2B,CAAC,mBAAmB,CAAC,CAAC;QACvD,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAIhB,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,iDAAiD,EAAE;gBACpE,IAAI;gBACJ,WAAW;gBACX,KAAK;aACL,CAAC,CAAC;QACJ,CAAC;IACF,CAAC;IAQO,KAAK,CAAC,uBAAuB,CACpC,mBAAkF,EAClF,gBAA0B;QAE1B,IAAI,CAAC;YACJ,MAAM,EAAE,mBAAmB,EAAE,cAAc,EAAE,GAAG,MAAM,IAAI,CAAC,aAAa,CAAC,eAAe,OAEvF,KAAK,EAAE,GAAG,EAAE,EAAE;gBACb,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAU,CAAC;gBACnC,MAAM,MAAM,GAIP,EAAE,CAAC;gBACR,KAAK,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,IAAI,mBAAmB,EAAE,CAAC;oBAGxD,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,+BAA+B,CAAC,QAAQ,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC;oBACpF,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,KAAK,KAAK,MAAM;wBAAE,SAAS;oBAEnD,MAAM,SAAS,GAAG,OAAO,CAAC,MAAM,KAAK,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC;oBAE9D,KAAK,MAAM,UAAU,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;wBAC9C,MAAM,IAAI,CAAC,kBAAkB,CAAC,cAAc,CAC3C;4BACC,uBAAuB,EAAE,SAAS;4BAClC,IAAI,EAAE,kBAAkB;4BACxB,IAAI,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE;4BAC/B,WAAW,EAAE,OAAO,CAAC,MAAM;yBAC3B,EACD,GAAG,CACH,CAAC;wBACF,QAAQ,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;oBAC1B,CAAC;oBAED,IAAI,MAAM,IAAI,CAAC,gBAAgB,CAAC,OAAO,EAAE,gBAAgB,EAAE,GAAG,CAAC,EAAE,CAAC;wBACjE,MAAM,CAAC,IAAI,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,MAAM,EAAE,CAAC,CAAC;oBAC/D,CAAC;gBACF,CAAC;gBAED,OAAO,EAAE,mBAAmB,EAAE,CAAC,GAAG,QAAQ,CAAC,EAAE,cAAc,EAAE,MAAM,EAAE,CAAC;YACvE,CAAC,CACD,CAAC;YAGF,KAAK,MAAM,EAAE,SAAS,EAAE,SAAS,EAAE,MAAM,EAAE,IAAI,cAAc,EAAE,CAAC;gBAC/D,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,wBAAwB,EAAE;oBAChD,uBAAuB,EAAE,SAAS;oBAClC,KAAK,EAAE,EAAE,OAAO,EAAE,kBAAkB,EAAE,SAAS,EAAE,MAAM,EAAE;iBACzD,CAAC,CAAC;YACJ,CAAC;YAED,IAAI,mBAAmB,CAAC,MAAM,KAAK,CAAC;gBAAE,OAAO;YAE7C,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,yCAAyC,EAAE;gBAC3D,IAAI,EAAE,kBAAkB;gBACxB,WAAW,EAAE,mBAAmB;aAChC,CAAC,CAAC;YAEH,IAAI,CAAC,2BAA2B,CAAC,mBAAmB,CAAC,CAAC;QACvD,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAGhB,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,iDAAiD,EAAE;gBACpE,IAAI,EAAE,kBAAkB;gBACxB,WAAW,EAAE,gBAAgB;gBAC7B,KAAK;aACL,CAAC,CAAC;QACJ,CAAC;IACF,CAAC;IAOO,KAAK,CAAC,gBAAgB,CAC7B,OAA8B,EAC9B,mBAA6B,EAC7B,GAAqB;QAErB,MAAM,SAAS,GAAG,MAAM,IAAI,CAAC,+BAA+B,CAAC,4BAA4B,CACxF,OAAO,CAAC,EAAE,EACV,mBAAmB,EACnB,GAAG,CACH,CAAC;QACF,IAAI,SAAS;YAAE,OAAO,KAAK,CAAC;QAE5B,OAAO,CAAC,KAAK,GAAG,QAAQ,CAAC;QAEzB,OAAO,CAAC,UAAU,GAAG,IAAI,CAAC;QAC1B,MAAM,IAAI,CAAC,+BAA+B,CAAC,WAAW,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC;QAErE,MAAM,IAAI,CAAC,kBAAkB,CAAC,cAAc,CAC3C;YACC,uBAAuB,EAAE,OAAO,CAAC,EAAE;YACnC,IAAI,EAAE,eAAe;YACrB,IAAI,EAAE,EAAE,MAAM,EAAE,yBAAyB,EAAE;YAC3C,WAAW,EAAE,IAAI;SACjB,EACD,GAAG,CACH,CAAC;QAEF,OAAO,IAAI,CAAC;IACb,CAAC;IAEO,2BAA2B,CAAC,WAAqB;QACxD,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;YAEtC,IAAI,CAAC,oBAAoB;iBACvB,mCAAmC,CAAC,UAAU,CAAC;iBAC/C,KAAK,CAAC,CAAC,KAAK,EAAE,EAAE,CAChB,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,yCAAyC,EAAE,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAClF,CAAC;QACJ,CAAC;IACF,CAAC;CACD,CAAA;;yCAzYY,8BAA8B;IAD1C,IAAA,YAAO,GAAE;qCAKiB,uBAAM,EACmB,oCAA+B,EACvB,4CAAuC,EAC5D,qCAAgC,EACrC,kBAAa,EACN,4CAAoB,EAC5B,4BAAY;GAVhC,8BAA8B,CAyY1C"}

package/dist/modules/workflow-reviews.ee/workflow-review-request.service.d.ts

@@ -1,11 +1,13 @@
 import type { CreateWorkflowReviewRequestDto, DecideWorkflowReviewRequestDto, DecideWorkflowReviewRequestResponse, GetWorkflowReviewEligibleReviewersQueryDto, ListWorkflowReviewRequestsQueryDto, UpdateWorkflowReviewRequestVersionDto, WorkflowReviewEligibleReviewersList, WorkflowReviewRequestList, WorkflowReviewRequestSummary } from '@n8n/api-types';
 import { Logger } from '@n8n/backend-common';
-import { DbLockService, SharedWorkflowRepository, UserRepository, WorkflowHistoryRepository, WorkflowPublishHistoryRepository, WorkflowReviewRequestAuthorRepository, WorkflowReviewRequestRepository, WorkflowRepository, WorkflowReviewActivityRepository, WorkflowReviewRequestReviewerRepository, WorkflowReviewRequestWorkflowRepository, type User } from '@n8n/db';
+import { DbLockService, SharedWorkflowRepository, UserRepository, WorkflowHistoryRepository, WorkflowReviewRequestAuthorRepository, WorkflowReviewRequestRepository, WorkflowRepository, WorkflowReviewActivityRepository, WorkflowReviewRequestReviewerRepository, WorkflowReviewRequestWorkflowRepository, type User } from '@n8n/db';
 import { CollaborationService } from '../../collaboration/collaboration.service';
+import { EventService } from '../../events/event.service';
 import { RoleService } from '../../services/role.service';
 import { WorkflowFinderService } from '../../workflows/workflow-finder.service';
 import { WorkflowHistoryService } from '../../workflows/workflow-history/workflow-history.service';
 import { WorkflowService } from '../../workflows/workflow.service';
+import { WorkflowReviewAccessService } from './workflow-review-access.service';
 import { WorkflowReviewEligibilityService } from './workflow-review-eligibility.service';
 import { WorkflowReviewFeatureGate } from './workflow-review-feature-gate.service';
 export declare class WorkflowReviewRequestService {
@@ -16,7 +18,6 @@ export declare class WorkflowReviewRequestService {
     private readonly workflowHistoryRepository;
     private readonly workflowRepository;
     private readonly sharedWorkflowRepository;
-    private readonly workflowPublishHistoryRepository;
     private readonly workflowReviewRequestRepository;
     private readonly workflowReviewRequestWorkflowRepository;
     private readonly workflowReviewRequestAuthorRepository;
@@ -28,14 +29,14 @@ export declare class WorkflowReviewRequestService {
     private readonly dbLockService;
     private readonly collaborationService;
     private readonly workflowService;
-    constructor(logger: Logger, featureGate: WorkflowReviewFeatureGate, workflowFinderService: WorkflowFinderService, workflowHistoryService: WorkflowHistoryService, workflowHistoryRepository: WorkflowHistoryRepository, workflowRepository: WorkflowRepository, sharedWorkflowRepository: SharedWorkflowRepository, workflowPublishHistoryRepository: WorkflowPublishHistoryRepository, workflowReviewRequestRepository: WorkflowReviewRequestRepository, workflowReviewRequestWorkflowRepository: WorkflowReviewRequestWorkflowRepository, workflowReviewRequestAuthorRepository: WorkflowReviewRequestAuthorRepository, workflowReviewRequestReviewerRepository: WorkflowReviewRequestReviewerRepository, activityRepository: WorkflowReviewActivityRepository, userRepository: UserRepository, eligibilityService: WorkflowReviewEligibilityService, roleService: RoleService, dbLockService: DbLockService, collaborationService: CollaborationService, workflowService: WorkflowService);
+    private readonly accessService;
+    private readonly eventService;
+    constructor(logger: Logger, featureGate: WorkflowReviewFeatureGate, workflowFinderService: WorkflowFinderService, workflowHistoryService: WorkflowHistoryService, workflowHistoryRepository: WorkflowHistoryRepository, workflowRepository: WorkflowRepository, sharedWorkflowRepository: SharedWorkflowRepository, workflowReviewRequestRepository: WorkflowReviewRequestRepository, workflowReviewRequestWorkflowRepository: WorkflowReviewRequestWorkflowRepository, workflowReviewRequestAuthorRepository: WorkflowReviewRequestAuthorRepository, workflowReviewRequestReviewerRepository: WorkflowReviewRequestReviewerRepository, activityRepository: WorkflowReviewActivityRepository, userRepository: UserRepository, eligibilityService: WorkflowReviewEligibilityService, roleService: RoleService, dbLockService: DbLockService, collaborationService: CollaborationService, workflowService: WorkflowService, accessService: WorkflowReviewAccessService, eventService: EventService);
     private findEligibleReviewers;
     list(user: User, query: ListWorkflowReviewRequestsQueryDto): Promise<WorkflowReviewRequestList>;
     private toWorkflowScopedItems;
     private resolveDecisionActors;
     private pickDecisionActor;
-    private resolveApprovedPublicationStates;
-    private pickApprovedPublicationState;
     getEligibleReviewers(user: User, query: GetWorkflowReviewEligibleReviewersQueryDto): Promise<WorkflowReviewEligibleReviewersList>;
     private nameVersion;
     private assertWorkflowStillReviewable;
@@ -47,5 +48,4 @@ export declare class WorkflowReviewRequestService {
     private assertDecisionAllowed;
     private broadcastReviewStateChanged;
     private assertRequestUpdatable;
-    private toSummary;
 }

package/dist/modules/workflow-reviews.ee/workflow-review-request.service.js

@@ -19,10 +19,12 @@ const bad_request_error_1 = require("../../errors/response-errors/bad-request.er
 const conflict_error_1 = require("../../errors/response-errors/conflict.error");
 const forbidden_error_1 = require("../../errors/response-errors/forbidden.error");
 const not_found_error_1 = require("../../errors/response-errors/not-found.error");
+const event_service_1 = require("../../events/event.service");
 const role_service_1 = require("../../services/role.service");
 const workflow_finder_service_1 = require("../../workflows/workflow-finder.service");
 const workflow_history_service_1 = require("../../workflows/workflow-history/workflow-history.service");
 const workflow_service_1 = require("../../workflows/workflow.service");
+const workflow_review_access_service_1 = require("./workflow-review-access.service");
 const workflow_review_eligibility_service_1 = require("./workflow-review-eligibility.service");
 const workflow_review_feature_gate_service_1 = require("./workflow-review-feature-gate.service");
 const workflow_review_mapper_1 = require("./workflow-review.mapper");
@@ -42,7 +44,7 @@ const BLOCKED_ACTION_TEXT = {
     update: 'submitted as a new review version',
 };
 let WorkflowReviewRequestService = class WorkflowReviewRequestService {
-    constructor(logger, featureGate, workflowFinderService, workflowHistoryService, workflowHistoryRepository, workflowRepository, sharedWorkflowRepository, workflowPublishHistoryRepository, workflowReviewRequestRepository, workflowReviewRequestWorkflowRepository, workflowReviewRequestAuthorRepository, workflowReviewRequestReviewerRepository, activityRepository, userRepository, eligibilityService, roleService, dbLockService, collaborationService, workflowService) {
+    constructor(logger, featureGate, workflowFinderService, workflowHistoryService, workflowHistoryRepository, workflowRepository, sharedWorkflowRepository, workflowReviewRequestRepository, workflowReviewRequestWorkflowRepository, workflowReviewRequestAuthorRepository, workflowReviewRequestReviewerRepository, activityRepository, userRepository, eligibilityService, roleService, dbLockService, collaborationService, workflowService, accessService, eventService) {
         this.logger = logger;
         this.featureGate = featureGate;
         this.workflowFinderService = workflowFinderService;
@@ -50,7 +52,6 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         this.workflowHistoryRepository = workflowHistoryRepository;
         this.workflowRepository = workflowRepository;
         this.sharedWorkflowRepository = sharedWorkflowRepository;
-        this.workflowPublishHistoryRepository = workflowPublishHistoryRepository;
         this.workflowReviewRequestRepository = workflowReviewRequestRepository;
         this.workflowReviewRequestWorkflowRepository = workflowReviewRequestWorkflowRepository;
         this.workflowReviewRequestAuthorRepository = workflowReviewRequestAuthorRepository;
@@ -62,6 +63,8 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         this.dbLockService = dbLockService;
         this.collaborationService = collaborationService;
         this.workflowService = workflowService;
+        this.accessService = accessService;
+        this.eventService = eventService;
     }
     async findEligibleReviewers(projectId, excludeUserId) {
         const [projectRoleSlugs, globalRoleSlugs] = await Promise.all([
@@ -97,13 +100,13 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         ]);
         return {
             count,
-            data: await this.toWorkflowScopedItems(query.workflowId, requests, canPublish),
+            data: await this.toWorkflowScopedItems(requests, canPublish, user),
         };
     }
-    async toWorkflowScopedItems(workflowId, requests, canPublish) {
-        const [decisionActors, publicationStates] = await Promise.all([
+    async toWorkflowScopedItems(requests, canPublish, user) {
+        const [decisionActors, openableIds] = await Promise.all([
             this.resolveDecisionActors(requests),
-            this.resolveApprovedPublicationStates(workflowId, requests),
+            this.accessService.resolveOpenableRequestIds(user, requests),
         ]);
         return requests.map((request) => ({
             id: request.id,
@@ -111,10 +114,11 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
             decision: request.decision,
             description: canPublish ? request.description : null,
             workflowVersionId: request.workflowVersionId,
+            workflowVersionName: request.workflowVersionName,
             createdAt: request.createdAt.toISOString(),
             updatedAt: request.updatedAt.toISOString(),
             decisionBy: this.pickDecisionActor(request, decisionActors),
-            approvedVersionPublicationState: this.pickApprovedPublicationState(request, publicationStates),
+            viewerCanOpen: openableIds.has(request.id),
         }));
     }
     async resolveDecisionActors(requests) {
@@ -135,26 +139,6 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         }
         return actors.get(request.updatedById) ?? null;
     }
-    async resolveApprovedPublicationStates(workflowId, requests) {
-        const versionIds = [
-            ...new Set(requests.flatMap((request) => request.decision === 'approved' && request.workflowVersionId
-                ? [request.workflowVersionId]
-                : [])),
-        ];
-        if (versionIds.length === 0) {
-            return new Map();
-        }
-        return await this.workflowPublishHistoryRepository.getVersionPublicationStates(workflowId, versionIds);
-    }
-    pickApprovedPublicationState(request, states) {
-        if (request.decision !== 'approved') {
-            return null;
-        }
-        if (!request.workflowVersionId) {
-            return 'unknown';
-        }
-        return states.get(request.workflowVersionId) ?? 'unknown';
-    }
     async getEligibleReviewers(user, query) {
         await this.featureGate.assertAvailable();
         const workflow = await this.workflowFinderService.findWorkflowForUser(query.workflowId, user, [
@@ -256,7 +240,15 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
             return created;
         });
         this.broadcastReviewStateChanged(workflowId);
-        return this.toSummary(request, workflowVersionId);
+        this.eventService.emit('workflow-review-requested', {
+            user,
+            workflowReviewRequestId: request.id,
+            projectId: project.id,
+            workflowId,
+            workflowVersionId,
+            reviewerCount: reviewerUserIds.length,
+        });
+        return (0, workflow_review_mapper_1.toRequestSummary)(request, workflowVersionId);
     }
     async updateVersion(user, workflowReviewRequestId, dto) {
         await this.featureGate.assertAvailable();
@@ -294,9 +286,9 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
             if (metadataChanged(version)) {
                 await this.nameVersion(dto.workflowId, dto.workflowVersionId, versionName, versionDescription, {});
             }
-            return this.toSummary(request, workflowRow.workflowVersionId);
+            return (0, workflow_review_mapper_1.toRequestSummary)(request, workflowRow.workflowVersionId);
         }
-        const { request: updated, changed } = await this.dbLockService.withLockContext(1004, async (ctx) => {
+        const { request: updated, changed, versionUpdated, } = await this.dbLockService.withLockContext(1004, async (ctx) => {
             const current = await this.workflowReviewRequestRepository.findById(workflowReviewRequestId, ctx);
             if (!current) {
                 throw new not_found_error_1.NotFoundError('Could not find review request');
@@ -313,12 +305,12 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
                     await this.nameVersion(dto.workflowId, dto.workflowVersionId, versionName, versionDescription, ctx);
                 }
                 if (reviewDescription === undefined || reviewDescription === current.description) {
-                    return { request: current, changed: false };
+                    return { request: current, changed: false, versionUpdated: false };
                 }
                 current.description = reviewDescription;
                 current.updatedById = user.id;
                 const saved = await this.workflowReviewRequestRepository.saveRequest(current, ctx);
-                return { request: saved, changed: true };
+                return { request: saved, changed: true, versionUpdated: false };
             }
             const fromWorkflowVersionId = currentRow.workflowVersionId;
             await this.workflowReviewRequestWorkflowRepository.updateWorkflowVersion({
@@ -344,12 +336,20 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
                 },
                 createdById: user.id,
             }, ctx);
-            return { request: saved, changed: true };
+            return { request: saved, changed: true, versionUpdated: true };
         });
         if (changed) {
             this.broadcastReviewStateChanged(dto.workflowId);
         }
-        return this.toSummary(updated, dto.workflowVersionId);
+        if (versionUpdated) {
+            this.eventService.emit('workflow-review-version-updated', {
+                user,
+                workflowReviewRequestId,
+                workflowId: dto.workflowId,
+                workflowVersionId: dto.workflowVersionId,
+            });
+        }
+        return (0, workflow_review_mapper_1.toRequestSummary)(updated, dto.workflowVersionId);
     }
     async decide(user, workflowReviewRequestId, dto) {
         await this.featureGate.assertAvailable();
@@ -377,7 +377,7 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         if (dto.decision === 'changes_requested' && !dto.note) {
             throw new bad_request_error_1.BadRequestError('A note is required when requesting changes');
         }
-        const { request: saved, pinnedVersionId } = await this.dbLockService.withLockContext(1004, async (ctx) => {
+        const { request: saved, pinnedVersionId, decidedAsAssignedReviewer, } = await this.dbLockService.withLockContext(1004, async (ctx) => {
             const current = await this.workflowReviewRequestRepository.findById(workflowReviewRequestId, ctx);
             if (!current) {
                 throw new not_found_error_1.NotFoundError('Could not find review request');
@@ -392,6 +392,14 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
                 throw new not_found_error_1.NotFoundError('Could not find review request');
             }
             await this.assertWorkflowStillReviewable(currentRow.workflowId, current.projectId, ctx, 'review');
+            if (dto.decision === 'approved') {
+                for (const row of currentRows) {
+                    await this.workflowReviewRequestWorkflowRepository.captureApprovalBaseline({
+                        workflowReviewRequestId,
+                        workflowId: row.workflowId,
+                    }, ctx);
+                }
+            }
             current.decision = dto.decision;
             current.updatedById = user.id;
             if (dto.decision === 'approved') {
@@ -411,10 +419,23 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
                 },
                 createdById: user.id,
             }, ctx);
-            return { request: savedRequest, pinnedVersionId: currentRow.workflowVersionId };
+            return {
+                request: savedRequest,
+                pinnedVersionId: currentRow.workflowVersionId,
+                decidedAsAssignedReviewer: isAssignedReviewerNow,
+            };
         });
         this.broadcastReviewStateChanged(workflowRow.workflowId);
-        const summary = this.toSummary(saved, pinnedVersionId);
+        this.eventService.emit('workflow-review-decided', {
+            user,
+            workflowReviewRequestId,
+            workflowId: workflowRow.workflowId,
+            workflowVersionId: pinnedVersionId,
+            decision: dto.decision,
+            decidedVia: decidedAsAssignedReviewer ? 'assigned-reviewer' : 'admin-override',
+            reviewCreatedAt: saved.createdAt,
+        });
+        const summary = (0, workflow_review_mapper_1.toRequestSummary)(saved, pinnedVersionId);
         if (dto.decision !== 'approved' || requesterPublishability === null) {
             return summary;
         }
@@ -490,15 +511,13 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
         return { status: 'published' };
     }
     assertDecisionAllowed(isAuthor, isAssignedReviewer, hasAdminOverride) {
-        if (hasAdminOverride) {
+        if (hasAdminOverride || isAssignedReviewer) {
             return;
         }
         if (isAuthor) {
             throw new forbidden_error_1.ForbiddenError('Authors cannot decide on their own review request');
         }
-        if (!isAssignedReviewer) {
-            throw new not_found_error_1.NotFoundError('Could not find workflow');
-        }
+        throw new not_found_error_1.NotFoundError('Could not find workflow');
     }
     broadcastReviewStateChanged(workflowId) {
         this.collaborationService
@@ -510,20 +529,10 @@ let WorkflowReviewRequestService = class WorkflowReviewRequestService {
             throw new conflict_error_1.ConflictError('The review request is no longer open');
         }
     }
-    toSummary(request, workflowVersionId) {
-        return {
-            id: request.id,
-            state: request.state,
-            decision: request.decision,
-            workflowVersionId,
-            createdAt: request.createdAt.toISOString(),
-            updatedAt: request.updatedAt.toISOString(),
-        };
-    }
 };
 exports.WorkflowReviewRequestService = WorkflowReviewRequestService;
 exports.WorkflowReviewRequestService = WorkflowReviewRequestService = __decorate([
     (0, di_1.Service)(),
-    __metadata("design:paramtypes", [backend_common_1.Logger, workflow_review_feature_gate_service_1.WorkflowReviewFeatureGate, workflow_finder_service_1.WorkflowFinderService, workflow_history_service_1.WorkflowHistoryService, db_1.WorkflowHistoryRepository, db_1.WorkflowRepository, db_1.SharedWorkflowRepository, db_1.WorkflowPublishHistoryRepository, db_1.WorkflowReviewRequestRepository, db_1.WorkflowReviewRequestWorkflowRepository, db_1.WorkflowReviewRequestAuthorRepository, db_1.WorkflowReviewRequestReviewerRepository, db_1.WorkflowReviewActivityRepository, db_1.UserRepository, workflow_review_eligibility_service_1.WorkflowReviewEligibilityService, role_service_1.RoleService, db_1.DbLockService, collaboration_service_1.CollaborationService, workflow_service_1.WorkflowService])
+    __metadata("design:paramtypes", [backend_common_1.Logger, workflow_review_feature_gate_service_1.WorkflowReviewFeatureGate, workflow_finder_service_1.WorkflowFinderService, workflow_history_service_1.WorkflowHistoryService, db_1.WorkflowHistoryRepository, db_1.WorkflowRepository, db_1.SharedWorkflowRepository, db_1.WorkflowReviewRequestRepository, db_1.WorkflowReviewRequestWorkflowRepository, db_1.WorkflowReviewRequestAuthorRepository, db_1.WorkflowReviewRequestReviewerRepository, db_1.WorkflowReviewActivityRepository, db_1.UserRepository, workflow_review_eligibility_service_1.WorkflowReviewEligibilityService, role_service_1.RoleService, db_1.DbLockService, collaboration_service_1.CollaborationService, workflow_service_1.WorkflowService, workflow_review_access_service_1.WorkflowReviewAccessService, event_service_1.EventService])
 ], WorkflowReviewRequestService);
 //# sourceMappingURL=workflow-review-request.service.js.map