@taskforcehq/taskforce 0.3.306 → 0.3.307

package/dist/sync/workspaceSyncModel.js

@@ -29,6 +29,9 @@ function sortTasksForSync(tasks) {
         return leftId.localeCompare(rightId);
     });
 }
+function clonePlanningAttachmentsForSync(value) {
+    return Array.isArray(value) ? value : undefined;
+}
 export function buildWorkspaceSyncPayloadModel(input) {
     const payloadBuiltAt = new Date().toISOString();
     const pendingDeleteIds = new Set(Array.from(input.pendingDeletedTaskIds)
@@ -142,6 +145,7 @@ export function buildWorkspaceSyncPayloadModel(input) {
             createdAt: String(initiative?.createdAt || '').trim(),
             updatedAt: String(initiative?.updatedAt || initiative?.createdAt || '').trim(),
             order: Number.isFinite(Number(initiative?.order)) ? Math.floor(Number(initiative.order)) : null,
+            attachments: clonePlanningAttachmentsForSync(initiative?.attachments),
             isArchived: Boolean(initiative?.isArchived)
         }
     }))
@@ -184,6 +188,7 @@ export function buildWorkspaceSyncPayloadModel(input) {
             createdAt: String(workstream?.createdAt || '').trim(),
             updatedAt: String(workstream?.updatedAt || workstream?.createdAt || '').trim(),
             order: Number.isFinite(Number(workstream?.order)) ? Math.floor(Number(workstream.order)) : null,
+            attachments: clonePlanningAttachmentsForSync(workstream?.attachments),
             isArchived: Boolean(workstream?.isArchived)
         }
     }))
@@ -218,6 +223,10 @@ export function buildWorkspaceSyncPayloadModel(input) {
             color: String(profile.color || '#8b5cf6').trim() || '#8b5cf6',
             avatarUrl: normalizeSyncSameOriginAssetUrl(profile.avatarUrl),
             avatarSourceUrl: normalizeSyncSameOriginAssetUrl(profile.avatarSourceUrl),
+            avatarRevision: Number.isFinite(Number(profile.avatarRevision)) ? Math.max(0, Math.floor(Number(profile.avatarRevision))) : 0,
+            avatarUpdatedAt: typeof profile.avatarUpdatedAt === 'string' && profile.avatarUpdatedAt.trim().length > 0
+                ? profile.avatarUpdatedAt.trim()
+                : null,
             description: typeof profile.description === 'string' ? profile.description.trim() || null : null,
             role: typeof profile.role === 'string' ? profile.role.trim() || null : null,
             provider: typeof profile.provider === 'string' ? profile.provider.trim() || null : null,
@@ -277,6 +286,10 @@ export function buildWorkspaceSyncPayloadModel(input) {
         referenceNumber: Number.isFinite(Number(doc.referenceNumber)) ? Math.max(1, Math.floor(Number(doc.referenceNumber))) : null,
         version: Number.isFinite(Number(doc.version)) ? Math.max(1, Math.floor(Number(doc.version))) : null,
         taskId: typeof doc.taskId === 'string' && doc.taskId.trim().length > 0 ? doc.taskId.trim() : null,
+        linkTargetType: doc.linkTargetType === 'initiative' || doc.linkTargetType === 'workstream' || doc.linkTargetType === 'task'
+            ? doc.linkTargetType
+            : null,
+        linkTargetId: typeof doc.linkTargetId === 'string' && doc.linkTargetId.trim().length > 0 ? doc.linkTargetId.trim() : null,
         logicalName: typeof doc.logicalName === 'string' && doc.logicalName.trim().length > 0 ? doc.logicalName.trim() : null,
         caption: typeof doc.caption === 'string' && doc.caption.trim().length > 0 ? doc.caption.trim() : null,
         originalFilename: typeof doc.originalFilename === 'string' && doc.originalFilename.trim().length > 0 ? doc.originalFilename.trim() : null,
@@ -309,6 +322,10 @@ export function buildWorkspaceSyncPayloadModel(input) {
         mimeType: typeof asset.mimeType === 'string' && asset.mimeType.trim().length > 0 ? asset.mimeType.trim() : 'application/octet-stream',
         referenceNumber: Number.isFinite(Number(asset.referenceNumber)) ? Math.max(1, Math.floor(Number(asset.referenceNumber))) : null,
         taskId: typeof asset.taskId === 'string' && asset.taskId.trim().length > 0 ? asset.taskId.trim() : null,
+        linkTargetType: asset.linkTargetType === 'initiative' || asset.linkTargetType === 'workstream' || asset.linkTargetType === 'task'
+            ? asset.linkTargetType
+            : null,
+        linkTargetId: typeof asset.linkTargetId === 'string' && asset.linkTargetId.trim().length > 0 ? asset.linkTargetId.trim() : null,
         logicalName: typeof asset.logicalName === 'string' && asset.logicalName.trim().length > 0 ? asset.logicalName.trim() : null,
         caption: typeof asset.caption === 'string' && asset.caption.trim().length > 0 ? asset.caption.trim() : null,
         originalFilename: typeof asset.originalFilename === 'string' && asset.originalFilename.trim().length > 0 ? asset.originalFilename.trim() : null,
@@ -325,21 +342,19 @@ export function buildWorkspaceSyncPayloadModel(input) {
         return previousAssetWatermarks.get(asset.path) !== asset.updatedAt;
     });
     const reviewSessions = Array.isArray(input.documentReviewSessions) ? input.documentReviewSessions : [];
+    const allDocumentReviewSessionIds = new Set(reviewSessions
+        .map((session) => String(session?.id || '').trim())
+        .filter((sessionId) => sessionId.length > 0));
     const currentDocumentReviewSessionIds = new Set(reviewSessions
         .filter((session) => String(session?.deletedAt || '').trim().length === 0)
         .map((session) => String(session?.id || '').trim())
         .filter((sessionId) => sessionId.length > 0));
-    const deleteDocumentReviewSessionIds = new Set([
-        ...Array.from(input.lastPushedDocumentReviewSessionIds || new Set())
-            .map((sessionId) => String(sessionId || '').trim())
-            .filter((sessionId) => sessionId.length > 0 && !currentDocumentReviewSessionIds.has(sessionId)),
-        ...reviewSessions
-            .filter((session) => String(session?.deletedAt || '').trim().length > 0)
-            .map((session) => String(session?.id || '').trim())
-            .filter((sessionId) => sessionId.length > 0)
-    ]);
+    const previousDocumentReviewSessionIds = input.lastPushedDocumentReviewSessionIds || new Set();
     const currentDocumentReviewSessionWatermarks = new Map(reviewSessions
-        .map((session) => [String(session?.id || '').trim(), String(session?.updatedAt || session?.createdAt || '').trim()])
+        .map((session) => [
+        String(session?.id || '').trim(),
+        String(session?.deletedAt || session?.updatedAt || session?.createdAt || '').trim()
+    ])
         .filter(([sessionId]) => sessionId.length > 0));
     const previousDocumentReviewSessionWatermarks = input.lastPushedDocumentReviewSessionWatermarks;
     const shouldSendAllDocumentReviewSessions = !previousDocumentReviewSessionWatermarks || previousDocumentReviewSessionWatermarks.size === 0;
@@ -367,22 +382,46 @@ export function buildWorkspaceSyncPayloadModel(input) {
             return true;
         return previousDocumentReviewSessionWatermarks.get(change.session.id) !== change.session.updatedAt;
     });
+    const deletedReviewSessionChanges = reviewSessions
+        .filter((session) => String(session?.deletedAt || '').trim().length > 0)
+        .map((session) => ({
+        sessionId: String(session?.id || '').trim(),
+        deletedAt: String(session?.deletedAt || session?.updatedAt || session?.createdAt || payloadBuiltAt).trim() || payloadBuiltAt
+    }))
+        .filter((change) => change.sessionId.length > 0)
+        .filter((change) => {
+        if (shouldSendAllDocumentReviewSessions)
+            return true;
+        return previousDocumentReviewSessionWatermarks.get(change.sessionId) !== change.deletedAt;
+    });
+    const missingDocumentReviewSessionDeleteIds = Array.from(previousDocumentReviewSessionIds)
+        .map((sessionId) => String(sessionId || '').trim())
+        .filter((sessionId) => sessionId.length > 0 && !allDocumentReviewSessionIds.has(sessionId));
+    const documentReviewSessionDeleteChanges = [
+        ...missingDocumentReviewSessionDeleteIds.map((sessionId) => ({
+            sessionId,
+            deletedAt: payloadBuiltAt
+        })),
+        ...deletedReviewSessionChanges
+    ];
+    const deleteDocumentReviewSessionIds = new Set([
+        ...missingDocumentReviewSessionDeleteIds,
+        ...deletedReviewSessionChanges.map((change) => change.sessionId)
+    ]);
     const reviewComments = Array.isArray(input.documentReviewComments) ? input.documentReviewComments : [];
+    const allDocumentReviewCommentIds = new Set(reviewComments
+        .map((comment) => String(comment?.id || '').trim())
+        .filter((commentId) => commentId.length > 0));
     const currentDocumentReviewCommentIds = new Set(reviewComments
         .filter((comment) => String(comment?.deletedAt || '').trim().length === 0)
         .map((comment) => String(comment?.id || '').trim())
         .filter((commentId) => commentId.length > 0));
-    const deleteDocumentReviewCommentIds = new Set([
-        ...Array.from(input.lastPushedDocumentReviewCommentIds || new Set())
-            .map((commentId) => String(commentId || '').trim())
-            .filter((commentId) => commentId.length > 0 && !currentDocumentReviewCommentIds.has(commentId)),
-        ...reviewComments
-            .filter((comment) => String(comment?.deletedAt || '').trim().length > 0)
-            .map((comment) => String(comment?.id || '').trim())
-            .filter((commentId) => commentId.length > 0)
-    ]);
+    const previousDocumentReviewCommentIds = input.lastPushedDocumentReviewCommentIds || new Set();
     const currentDocumentReviewCommentWatermarks = new Map(reviewComments
-        .map((comment) => [String(comment?.id || '').trim(), String(comment?.updatedAt || comment?.createdAt || '').trim()])
+        .map((comment) => [
+        String(comment?.id || '').trim(),
+        String(comment?.deletedAt || comment?.updatedAt || comment?.createdAt || '').trim()
+    ])
         .filter(([commentId]) => commentId.length > 0));
     const previousDocumentReviewCommentWatermarks = input.lastPushedDocumentReviewCommentWatermarks;
     const shouldSendAllDocumentReviewComments = !previousDocumentReviewCommentWatermarks || previousDocumentReviewCommentWatermarks.size === 0;
@@ -407,22 +446,46 @@ export function buildWorkspaceSyncPayloadModel(input) {
             return true;
         return previousDocumentReviewCommentWatermarks.get(change.comment.id) !== change.comment.updatedAt;
     });
+    const deletedReviewCommentChanges = reviewComments
+        .filter((comment) => String(comment?.deletedAt || '').trim().length > 0)
+        .map((comment) => ({
+        commentId: String(comment?.id || '').trim(),
+        deletedAt: String(comment?.deletedAt || comment?.updatedAt || comment?.createdAt || payloadBuiltAt).trim() || payloadBuiltAt
+    }))
+        .filter((change) => change.commentId.length > 0)
+        .filter((change) => {
+        if (shouldSendAllDocumentReviewComments)
+            return true;
+        return previousDocumentReviewCommentWatermarks.get(change.commentId) !== change.deletedAt;
+    });
+    const missingDocumentReviewCommentDeleteIds = Array.from(previousDocumentReviewCommentIds)
+        .map((commentId) => String(commentId || '').trim())
+        .filter((commentId) => commentId.length > 0 && !allDocumentReviewCommentIds.has(commentId));
+    const documentReviewCommentDeleteChanges = [
+        ...missingDocumentReviewCommentDeleteIds.map((commentId) => ({
+            commentId,
+            deletedAt: payloadBuiltAt
+        })),
+        ...deletedReviewCommentChanges
+    ];
+    const deleteDocumentReviewCommentIds = new Set([
+        ...missingDocumentReviewCommentDeleteIds,
+        ...deletedReviewCommentChanges.map((change) => change.commentId)
+    ]);
     const annotatedSessions = Array.isArray(input.annotatedAttachmentSessions) ? input.annotatedAttachmentSessions : [];
+    const allAnnotatedAttachmentSessionIds = new Set(annotatedSessions
+        .map((session) => String(session?.id || '').trim())
+        .filter((sessionId) => sessionId.length > 0));
     const currentAnnotatedAttachmentSessionIds = new Set(annotatedSessions
         .filter((session) => String(session?.deletedAt || '').trim().length === 0)
         .map((session) => String(session?.id || '').trim())
         .filter((sessionId) => sessionId.length > 0));
-    const deleteAnnotatedAttachmentSessionIds = new Set([
-        ...Array.from(input.lastPushedAnnotatedAttachmentSessionIds || new Set())
-            .map((sessionId) => String(sessionId || '').trim())
-            .filter((sessionId) => sessionId.length > 0 && !currentAnnotatedAttachmentSessionIds.has(sessionId)),
-        ...annotatedSessions
-            .filter((session) => String(session?.deletedAt || '').trim().length > 0)
-            .map((session) => String(session?.id || '').trim())
-            .filter((sessionId) => sessionId.length > 0)
-    ]);
+    const previousAnnotatedAttachmentSessionIds = input.lastPushedAnnotatedAttachmentSessionIds || new Set();
     const currentAnnotatedAttachmentSessionWatermarks = new Map(annotatedSessions
-        .map((session) => [String(session?.id || '').trim(), String(session?.updatedAt || session?.createdAt || '').trim()])
+        .map((session) => [
+        String(session?.id || '').trim(),
+        String(session?.deletedAt || session?.updatedAt || session?.createdAt || '').trim()
+    ])
         .filter(([sessionId]) => sessionId.length > 0));
     const previousAnnotatedAttachmentSessionWatermarks = input.lastPushedAnnotatedAttachmentSessionWatermarks;
     const shouldSendAllAnnotatedAttachmentSessions = !previousAnnotatedAttachmentSessionWatermarks || previousAnnotatedAttachmentSessionWatermarks.size === 0;
@@ -453,6 +516,32 @@ export function buildWorkspaceSyncPayloadModel(input) {
             return true;
         return previousAnnotatedAttachmentSessionWatermarks.get(change.session.id) !== change.session.updatedAt;
     });
+    const deletedAnnotatedAttachmentSessionChanges = annotatedSessions
+        .filter((session) => String(session?.deletedAt || '').trim().length > 0)
+        .map((session) => ({
+        sessionId: String(session?.id || '').trim(),
+        deletedAt: String(session?.deletedAt || session?.updatedAt || session?.createdAt || payloadBuiltAt).trim() || payloadBuiltAt
+    }))
+        .filter((change) => change.sessionId.length > 0)
+        .filter((change) => {
+        if (shouldSendAllAnnotatedAttachmentSessions)
+            return true;
+        return previousAnnotatedAttachmentSessionWatermarks.get(change.sessionId) !== change.deletedAt;
+    });
+    const missingAnnotatedAttachmentSessionDeleteIds = Array.from(previousAnnotatedAttachmentSessionIds)
+        .map((sessionId) => String(sessionId || '').trim())
+        .filter((sessionId) => sessionId.length > 0 && !allAnnotatedAttachmentSessionIds.has(sessionId));
+    const annotatedAttachmentSessionDeleteChanges = [
+        ...missingAnnotatedAttachmentSessionDeleteIds.map((sessionId) => ({
+            sessionId,
+            deletedAt: payloadBuiltAt
+        })),
+        ...deletedAnnotatedAttachmentSessionChanges
+    ];
+    const deleteAnnotatedAttachmentSessionIds = new Set([
+        ...missingAnnotatedAttachmentSessionDeleteIds,
+        ...deletedAnnotatedAttachmentSessionChanges.map((change) => change.sessionId)
+    ]);
     const taskEvents = sortTaskEventsForSync(Array.isArray(input.taskEvents) ? input.taskEvents : []);
     const currentTaskEventWatermarks = new Map(taskEvents
         .map((event) => [String(event?.id || '').trim(), String(event?.createdAt || '').trim()])
@@ -460,12 +549,17 @@ export function buildWorkspaceSyncPayloadModel(input) {
     const previousTaskEventWatermarks = input.lastPushedTaskEventWatermarks;
     const shouldSendAllTaskEvents = !previousTaskEventWatermarks || previousTaskEventWatermarks.size === 0;
     const taskEventUpsertChanges = taskEvents
-        .map((event) => ({
-        op: 'task-event-upsert',
-        event: {
+        .map((event) => {
+        const entityType = event.entityType === 'initiative' || event.entityType === 'workstream'
+            ? event.entityType
+            : 'task';
+        const entityId = String(event.entityId || event.taskId || '').trim();
+        const normalizedEvent = {
             id: String(event.id || '').trim(),
-            taskId: String(event.taskId || '').trim(),
             workspaceId: String(event.workspaceId || '').trim(),
+            entityType,
+            entityId,
+            taskId: entityType === 'task' ? String(event.taskId || entityId).trim() : undefined,
             action: String(event.action || '').trim(),
             actor: String(event.actor || 'system').trim() || 'system',
             actorType: event.actorType === 'ai' || event.actorType === 'human' || event.actorType === 'system'
@@ -475,10 +569,28 @@ export function buildWorkspaceSyncPayloadModel(input) {
                 ? event.details
                 : {},
             createdAt: String(event.createdAt || '').trim(),
+        };
+        if (entityType === 'task') {
+            return {
+                op: 'task-event-upsert',
+                event: {
+                    ...normalizedEvent,
+                    entityType: 'task',
+                    taskId: String(normalizedEvent.taskId || entityId).trim(),
+                }
+            };
         }
-    }))
+        return {
+            op: 'entity-event-upsert',
+            event: {
+                ...normalizedEvent,
+                entityType,
+                taskId: undefined,
+            }
+        };
+    })
         .filter((change) => (change.event.id.length > 0
-        && change.event.taskId.length > 0
+        && change.event.entityId.length > 0
         && change.event.workspaceId.length > 0
         && change.event.action.length > 0
         && Number.isFinite(Date.parse(change.event.createdAt))))
@@ -517,22 +629,22 @@ export function buildWorkspaceSyncPayloadModel(input) {
             deletedAt: payloadBuiltAt
         })),
         ...documentReviewSessionUpsertChanges,
-        ...Array.from(deleteDocumentReviewSessionIds).map((sessionId) => ({
+        ...documentReviewSessionDeleteChanges.map((change) => ({
             op: 'document-review-session-delete',
-            sessionId,
-            deletedAt: payloadBuiltAt
+            sessionId: change.sessionId,
+            deletedAt: change.deletedAt
         })),
         ...documentReviewCommentUpsertChanges,
-        ...Array.from(deleteDocumentReviewCommentIds).map((commentId) => ({
+        ...documentReviewCommentDeleteChanges.map((change) => ({
             op: 'document-review-comment-delete',
-            commentId,
-            deletedAt: payloadBuiltAt
+            commentId: change.commentId,
+            deletedAt: change.deletedAt
         })),
         ...annotatedAttachmentSessionUpsertChanges,
-        ...Array.from(deleteAnnotatedAttachmentSessionIds).map((sessionId) => ({
+        ...annotatedAttachmentSessionDeleteChanges.map((change) => ({
             op: 'annotated-attachment-session-delete',
-            sessionId,
-            deletedAt: payloadBuiltAt
+            sessionId: change.sessionId,
+            deletedAt: change.deletedAt
         })),
         ...taskEventUpsertChanges,
         ...activeChanges,
@@ -694,6 +806,7 @@ export function buildWorkspaceSyncSignatureModel(input) {
         updatedAt: String(initiative.updatedAt || initiative.createdAt || '').trim(),
         title: String(initiative.title || '').trim(),
         ownerId: typeof initiative.ownerId === 'string' ? initiative.ownerId.trim() : '',
+        attachments: clonePlanningAttachmentsForSync(initiative.attachments) || [],
         isArchived: Boolean(initiative.isArchived)
     }))
         .filter((initiative) => initiative.id.length > 0)
@@ -705,6 +818,7 @@ export function buildWorkspaceSyncSignatureModel(input) {
         initiativeId: typeof workstream.initiativeId === 'string' ? workstream.initiativeId.trim() : '',
         title: String(workstream.title || '').trim(),
         ownerId: typeof workstream.ownerId === 'string' ? workstream.ownerId.trim() : '',
+        attachments: clonePlanningAttachmentsForSync(workstream.attachments) || [],
         isArchived: Boolean(workstream.isArchived)
     }))
         .filter((workstream) => workstream.id.length > 0)