@contrail/flexplm 1.8.1-alpha.85853aa → 2.0.0-alpha.df05bb5

package/lib/publish/base-process-publish-assortment.js

@@ -11,6 +11,12 @@ const fsPromise = require("fs/promises");
 const path = require("path");
 const app_framework_1 = require("@contrail/app-framework");
 const event_short_message_status_1 = require("../util/event-short-message-status");
+const telemetry_1 = require("@contrail/telemetry");
+const api_1 = require("@opentelemetry/api");
+const telemetry_attributes_1 = require("../util/telemetry-attributes");
+const telemetry_utils_1 = require("../util/telemetry-utils");
+const PUBLISH_FILE_HYDRATED_DETAIL = 'hydrated_detail';
+const PUBLISH_FILE_BASELINE = 'baseline';
 class BaseProcessPublishAssortment {
     constructor(_config, _dc, _mapFileUtil) {
         this.TTL = 30 * 24 * 60 * 60 * 1000; // 30 days
@@ -24,61 +30,115 @@ class BaseProcessPublishAssortment {
         this.orgSlug = this.config?.orgSlug || 'unset-orgSlug';
     }
     async process(event) {
-        const assortmentPublishChangeId = event.assortmentPublishChangeId;
-        let apcHistory;
-        let publisher;
-        const assortmentId = event.assortmentId;
-        try {
-            console.info('process-start: assortmentId: ' + assortmentId
-                + ', assortmentPublishChangeId: ' + assortmentPublishChangeId);
-            let seasonFed;
+        return (0, telemetry_1.traceSection)('publish_assortment', {}, async (span) => {
+            const assortmentPublishChangeId = event.assortmentPublishChangeId;
+            let apcHistory;
+            let publisher;
+            const assortmentId = event.assortmentId;
+            (0, telemetry_utils_1.setActiveSpanAttributes)({
+                [telemetry_attributes_1.ATTR_FLEXPLM_ASSORTMENT_ID]: assortmentId,
+                [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_CHANGE_ID]: assortmentPublishChangeId,
+            });
             try {
-                seasonFed = await this.getSeasonFederation(assortmentId);
+                let seasonFed;
+                try {
+                    seasonFed = await this.getSeasonFederation(assortmentId);
+                }
+                catch (e) {
+                    const message = e.message;
+                    const eventStatus = (message.includes(BaseProcessPublishAssortment.ASSORTMENT_NOT_PUBLISHABLE)) ? event_short_message_status_1.EventShortMessageStatus.NOT_PUBLISHABLE :
+                        (message.startsWith(BaseProcessPublishAssortment.ASSORTMENT_NO_FED_INFO)) ? event_short_message_status_1.EventShortMessageStatus.NO_FEDERATION_INFO :
+                            event_short_message_status_1.EventShortMessageStatus.FAILURE;
+                    this.recordPublishStatus({
+                        status: eventStatus,
+                        assortmentId,
+                        assortmentPublishChangeId,
+                        message,
+                        // Only an unexpected error is swallowed here; the other two are expected skips.
+                        error: eventStatus === event_short_message_status_1.EventShortMessageStatus.FAILURE ? e : undefined,
+                    });
+                    const output = {
+                        results: { message, event },
+                        skip_await: true
+                    };
+                    return output;
+                }
+                apcHistory = await this.getApcHistory(assortmentId);
+                const sinceDate = await this.getSinceDate(assortmentId, assortmentPublishChangeId, apcHistory);
+                //Get detail information
+                const assortmentPublishChange = await this.downloadAssortmentPublishChange(assortmentId, assortmentPublishChangeId);
+                publisher = this.getPublisher(assortmentPublishChange);
+                const changeDetail = await this.downloadHydratedChangeDetail(assortmentPublishChange);
+                // Without the change detail the counts are unknown, not zero; failed_downloads says why.
+                if (changeDetail) {
+                    span.setAttributes({
+                        [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_ADDED_COUNT]: changeDetail.adds?.length ?? 0,
+                        [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_REMOVED_COUNT]: changeDetail.deletes?.length ?? 0,
+                        [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_CHANGED_COUNT]: changeDetail.updates?.length ?? 0,
+                    });
+                }
+                const assortmentBaseline = await this.downloadAssortmentBaseline(assortmentPublishChange);
+                const failedDownloads = [!changeDetail && PUBLISH_FILE_HYDRATED_DETAIL, !assortmentBaseline && PUBLISH_FILE_BASELINE].filter(Boolean);
+                if (failedDownloads.length > 0) {
+                    span.setAttribute(telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_FAILED_DOWNLOADS, failedDownloads);
+                }
+                const deleteChanges = await this.getDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate);
+                const releasedForDevelopmentItemIds = this.getReleasedForDevelopmentItemAndFamilyIds(assortmentBaseline, deleteChanges);
+                const itemToFederatedIdMapping = await this.getItemFederatedIds( /*allItemIds*/);
+                const pcd = new publish_change_data_1.PublishChangeData(assortmentId, seasonFed, assortmentPublishChangeId, sinceDate, publisher);
+                pcd.itemToFederatedIdMapping = itemToFederatedIdMapping;
+                pcd.releasedForDevelopmentItemIds = releasedForDevelopmentItemIds;
+                const output = await this.processPublish(pcd, changeDetail, assortmentBaseline, deleteChanges);
+                output['publishInfo'] = await this.getPublishInfo(assortmentId, assortmentPublishChangeId, apcHistory, publisher);
+                return output;
             }
             catch (e) {
-                const message = e.message;
-                const eventStatus = (message.includes(BaseProcessPublishAssortment.ASSORTMENT_NOT_PUBLISHABLE)) ? event_short_message_status_1.EventShortMessageStatus.NOT_PUBLISHABLE :
-                    (message.startsWith(BaseProcessPublishAssortment.ASSORTMENT_NO_FED_INFO)) ? event_short_message_status_1.EventShortMessageStatus.NO_FEDERATION_INFO :
-                        event_short_message_status_1.EventShortMessageStatus.FAILURE;
-                const processStatus = 'BaseProcessPublishAssortment: ' + eventStatus
-                    + ': assortmentId: ' + event.assortmentId + ': assortmentPublishChangeId: ' + event.assortmentPublishChangeId
-                    + ', orgSlug: ' + this.orgSlug;
-                console.log(JSON.stringify(processStatus));
-                console.log(message);
-                const output = {
-                    results: { message, event },
-                    skip_await: true
-                };
-                return output;
+                try {
+                    const publishInfo = await this.getPublishInfo(assortmentId, assortmentPublishChangeId, apcHistory, publisher);
+                    e.publishInfo = publishInfo;
+                }
+                catch (e2) {
+                    telemetry_1.logger.error({ error: e2, assortment: { id: assortmentId, publish_change_id: assortmentPublishChangeId } }, 'getPublishInfo failed');
+                }
+                this.recordPublishStatus({
+                    status: event_short_message_status_1.EventShortMessageStatus.FAILURE, assortmentId, assortmentPublishChangeId, error: e,
+                    // sendAndRecordPublishEvent only sets this after FlexPLM accepted the publish.
+                    sentUnrecorded: e?.sendPublishPayloadEventResult !== undefined,
+                });
+                throw e;
             }
-            apcHistory = await this.getApcHistory(assortmentId);
-            const sinceDate = await this.getSinceDate(assortmentId, assortmentPublishChangeId, apcHistory);
-            //Get detail information
-            const assortmentPublishChange = await this.downloadAssortmentPublishChange(assortmentId, assortmentPublishChangeId);
-            publisher = this.getPublisher(assortmentPublishChange);
-            const changeDetail = await this.downloadHydratedChangeDetail(assortmentPublishChange);
-            const assortmentBaseline = await this.downloadAssortmentBaseline(assortmentPublishChange);
-            const deleteChanges = await this.getDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate);
-            const releasedForDevelopmentItemIds = this.getReleasedForDevelopmentItemAndFamilyIds(assortmentBaseline, deleteChanges);
-            const itemToFederatedIdMapping = await this.getItemFederatedIds( /*allItemIds*/);
-            const pcd = new publish_change_data_1.PublishChangeData(assortmentId, seasonFed, assortmentPublishChangeId, sinceDate, publisher);
-            pcd.itemToFederatedIdMapping = itemToFederatedIdMapping;
-            pcd.releasedForDevelopmentItemIds = releasedForDevelopmentItemIds;
-            const output = await this.processPublish(pcd, changeDetail, assortmentBaseline, deleteChanges);
-            output['publishInfo'] = await this.getPublishInfo(assortmentId, assortmentPublishChangeId, apcHistory, publisher);
-            console.info('process-end');
-            return output;
+        });
+    }
+    /** Records the result of one assortment publish: attributes on the active span plus one structured log.
+     * Not_Publishable / No_Federation_Information are handled skips, so the span stays OK for them.
+     */
+    recordPublishStatus(statusObject) {
+        const { status, assortmentId, assortmentPublishChangeId, resultsCount, message, error } = statusObject;
+        const sentUnrecorded = statusObject.sentUnrecorded || undefined;
+        (0, telemetry_utils_1.setActiveSpanAttributes)({
+            [telemetry_attributes_1.ATTR_FLEXPLM_STATUS]: status,
+            [telemetry_attributes_1.ATTR_FLEXPLM_ASSORTMENT_ID]: assortmentId,
+            [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_CHANGE_ID]: assortmentPublishChangeId,
+            [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_PRODUCT_SEASON_COUNT]: resultsCount?.productSeasons,
+            [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_COLORWAY_SEASON_COUNT]: resultsCount?.colorwaySeasons,
+            [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_SENT_UNRECORDED]: sentUnrecorded,
+        });
+        const flexplm = {
+            direction: 'publish', status, assortment_id: assortmentId, publish_change_id: assortmentPublishChangeId,
+            product_season_count: resultsCount?.productSeasons, colorway_season_count: resultsCount?.colorwaySeasons, reason: message,
+            sent_unrecorded: sentUnrecorded,
+        };
+        if (sentUnrecorded) {
+            telemetry_1.logger.error({ error, flexplm }, 'FlexPLM assortment publish sent but not recorded');
         }
-        catch (e) {
-            try {
-                const publishInfo = await this.getPublishInfo(assortmentId, assortmentPublishChangeId, apcHistory, publisher);
-                e.publishInfo = publishInfo;
-            }
-            catch (e2) {
-                console.error('getPublishInfo failed: ' + e2.message);
-            }
-            console.error('process failed: assortmentId: ' + assortmentId + ': ' + e.message);
-            throw e;
+        else if (status === event_short_message_status_1.EventShortMessageStatus.FAILURE) {
+            telemetry_1.logger.error({ error, flexplm }, 'FlexPLM assortment publish failed');
+        }
+        else if (status === event_short_message_status_1.EventShortMessageStatus.NO_FEDERATION_INFO) {
+            telemetry_1.logger.warn({ flexplm }, 'FlexPLM assortment publish skipped');
+        }
+        else {
+            telemetry_1.logger.info({ flexplm }, 'FlexPLM assortment publish processed');
         }
     }
     async getPublishInfo(assortmentId, assortmentPublishChangeId, apcHistory, publisher) {
@@ -211,13 +271,18 @@ class BaseProcessPublishAssortment {
         return sinceDate;
     }
     async getApcHistory(assortmentId) {
-        const resource = `/assortments/${assortmentId}/history/`;
-        return sdk_1.Request.request(resource, {
-            method: 'GET',
-            headers: {
-                Accept: 'application/json',
-                'Content-Type': 'application/json'
-            }
+        return (0, telemetry_1.traceSection)('get_publish_history', {}, async (span) => {
+            const resource = `/assortments/${assortmentId}/history/`;
+            const history = await sdk_1.Request.request(resource, {
+                method: 'GET',
+                headers: {
+                    Accept: 'application/json',
+                    'Content-Type': 'application/json'
+                }
+            });
+            if (Array.isArray(history))
+                span.setAttribute(telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_HISTORY_COUNT, history.length);
+            return history;
         });
     }
     updatedSinceDate(entity, sinceDate) {
@@ -315,30 +380,42 @@ class BaseProcessPublishAssortment {
         });
     }
     async downloadHydratedChangeDetail(assortmentPublishChange) {
-        try {
-            const link = assortmentPublishChange?.hydratedDetailDownloadLink;
-            const response = await fetch(link);
-            const data = await response.json();
-            return data;
-        }
-        catch (e) {
-            console.error('Error hydratedDetailDownloadLink: ' + e.message);
-        }
-        return undefined;
+        return this.downloadPublishFile(assortmentPublishChange?.hydratedDetailDownloadLink, PUBLISH_FILE_HYDRATED_DETAIL);
     }
     async downloadAssortmentBaseline(assortmentPublishChange) {
-        try {
-            const link = assortmentPublishChange?.assortmentBaselineDownloadLink;
-            const response = await fetch(link);
-            const data = await response.json();
-            return data;
-        }
-        catch (e) {
-            console.error('Error assortmentBaselineDownloadLink: ' + e.message);
-        }
-        return undefined;
+        return this.downloadPublishFile(assortmentPublishChange?.assortmentBaselineDownloadLink, PUBLISH_FILE_BASELINE);
+    }
+    /** Downloads one publish file as JSON. Returns undefined when it can't be read, so the publish continues without it;
+     * the span is marked ERROR here because nothing is thrown. The link is never recorded (it is a signed URL).
+     */
+    async downloadPublishFile(link, file) {
+        return (0, telemetry_1.traceSection)('download_publish_file', { [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_FILE]: file }, async (span) => {
+            let httpStatus;
+            try {
+                const response = await fetch(link);
+                httpStatus = response.status;
+                span.setAttribute(telemetry_attributes_1.ATTR_HTTP_RESPONSE_STATUS_CODE, httpStatus);
+                if (!response.ok) {
+                    throw new Error('Download failed: status: ' + httpStatus);
+                }
+                return await response.json();
+            }
+            catch (e) {
+                span.recordException(e);
+                span.setStatus({ code: api_1.SpanStatusCode.ERROR, message: e?.message });
+                telemetry_1.logger.error({ error: e, flexplm: { direction: 'publish', publish_file: file, http_status: httpStatus } }, 'FlexPLM publish file download failed');
+                return undefined;
+            }
+        });
     }
     async getDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate) {
+        return (0, telemetry_1.traceSection)('get_delete_changes', {}, async (span) => {
+            const deleteChanges = await this.collectDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate);
+            span.setAttribute(telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_DELETE_CHANGE_COUNT, deleteChanges?.length ?? 0);
+            return deleteChanges;
+        });
+    }
+    async collectDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate) {
         const currentAPCIndex = apcHistory.findIndex(pc => pc?.id === assortmentPublishChange?.id);
         if (currentAPCIndex == 0) {
             return [];
@@ -383,6 +460,7 @@ class BaseProcessPublishAssortment {
             console.info('getDeleteChanges(): baselineItems: ' + currentAssortmentItemIds.length
                 + ', scanned ' + scannedCount + ' historical delete changes, added ' + addedCount
                 + ', deleteIds: ' + deleteIds.length);
+            (0, telemetry_utils_1.setActiveSpanAttributes)({ [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_HISTORICAL_DELETE_COUNT]: addedCount });
             console.info('getDeleteChanges()-deleteIds: ' + deleteIds);
         }
         return apcDeletes;
@@ -559,12 +637,12 @@ class BaseProcessPublishAssortment {
         pcd.itemFamilyChanges = this.getItemFamilyChanges(pcd, changeDetail, assortmentItemFullChangeMap, assortmentItemDeleteMap);
         const events = await this.getEventsForPublishChangeData(pcd);
         if (events.length === 0) {
-            const processStatus = 'BaseProcessPublishAssortment: ' + event_short_message_status_1.EventShortMessageStatus.NO_EVENTS_TO_SEND
-                + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId
-                + ', orgSlug: ' + this.orgSlug;
-            console.log(processStatus);
             const message = 'No Events to Send; returning';
-            console.log(message);
+            this.recordPublishStatus({
+                status: event_short_message_status_1.EventShortMessageStatus.NO_EVENTS_TO_SEND,
+                assortmentId: pcd.assortmentId,
+                assortmentPublishChangeId: pcd.assortmentPublishChangeId,
+            });
             return {
                 results: { message, event },
                 skip_await: true
@@ -572,11 +650,12 @@ class BaseProcessPublishAssortment {
         }
         const results = await this.sendEvents(events);
         const resultsCount = this.getResultsCount(events);
-        const processStatus = 'BaseProcessPublishAssortment: ' + event_short_message_status_1.EventShortMessageStatus.SENDING_EVENTS
-            + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId
-            + ': resultsCount: ' + JSON.stringify(resultsCount)
-            + ', orgSlug: ' + this.orgSlug;
-        console.log(processStatus);
+        this.recordPublishStatus({
+            status: event_short_message_status_1.EventShortMessageStatus.SENDING_EVENTS,
+            assortmentId: pcd.assortmentId,
+            assortmentPublishChangeId: pcd.assortmentPublishChangeId,
+            resultsCount,
+        });
         return { results, event, resultsCount };
     }
     getResultsCount(events) {
@@ -595,6 +674,7 @@ class BaseProcessPublishAssortment {
         console.info('sendEvents()');
         const eventType = 'ASYNC_PUBLISH_SEASON';
         const sendMode = (this.config?.sendMode[eventType] || '').toLowerCase();
+        (0, telemetry_utils_1.setActiveSpanAttributes)({ [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_SEND_MODE]: sendMode });
         switch (sendMode) {
             case 'sendtoflexplm':
                 return this.sendToFlexPLM(events, eventType);
@@ -604,22 +684,25 @@ class BaseProcessPublishAssortment {
             case 'vibeiqfile-dontsendtoflexplm':
                 return this.handleVibeIQFile(events, eventType, sendMode);
             default:
+                telemetry_1.logger.warn({ flexplm: { direction: 'publish', send_mode: sendMode, payload_count: events?.length ?? 0 } }, 'Unknown FlexPLM publish sendMode; events not sent');
                 return {}; // Or handle other cases as required
         }
     }
     async sendToFlexPLM(events, eventType) {
-        const outboundPublishEvent = {
-            taskId: this.config.taskId,
-            eventType,
-            objectClass: 'LCSSeason',
-            events
-        };
-        const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
-        const result = await this.sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent);
-        const isResultObject = typeof result === 'object' && result !== null;
-        return isResultObject
-            ? { ...result, outboundPublishEvent }
-            : { result, outboundPublishEvent };
+        return (0, telemetry_1.traceSection)('send_publish_batch', { [telemetry_attributes_1.ATTR_FLEXPLM_PAYLOAD_COUNT]: events?.length ?? 0 }, async () => {
+            const outboundPublishEvent = {
+                taskId: this.config.taskId,
+                eventType,
+                objectClass: 'LCSSeason',
+                events
+            };
+            const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
+            const result = await this.sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent);
+            const isResultObject = typeof result === 'object' && result !== null;
+            return isResultObject
+                ? { ...result, outboundPublishEvent }
+                : { result, outboundPublishEvent };
+        });
     }
     async sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent) {
         let sendResult;
@@ -668,6 +751,7 @@ class BaseProcessPublishAssortment {
             };
         }
         catch (e) {
+            telemetry_1.logger.error({ error: e, flexplm: { direction: 'publish', send_mode: 'localfile' } }, 'Saving FlexPLM publish events to a local file failed');
             results['error'] = e.message;
         }
         finally {
@@ -678,7 +762,7 @@ class BaseProcessPublishAssortment {
     async handleVibeIQFile(events, eventType, sendMode) {
         const eventBuffer = Buffer.from(JSON.stringify(events), 'utf-8');
         const fileName = 'ASYNC_PUBLISH_SEASON-events.json';
-        const uploadFile = await new sdk_1.Files().createAndUploadFileFromBuffer(eventBuffer, 'application/json', fileName, null, this.TTL);
+        const uploadFile = await (0, telemetry_1.traceSection)('upload_publish_events_file', { [telemetry_attributes_1.ATTR_FLEXPLM_PAYLOAD_COUNT]: events?.length ?? 0 }, () => new sdk_1.Files().createAndUploadFileFromBuffer(eventBuffer, 'application/json', fileName, null, this.TTL));
         const outboundPublishEvent = {
             taskId: this.config.taskId,
             eventType,
@@ -688,12 +772,14 @@ class BaseProcessPublishAssortment {
             eventsAdminDownloadLink: uploadFile.adminDownloadUrl
         };
         if (sendMode === 'vibeiqfile') {
-            const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
-            const result = await this.sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent);
-            const isResultObject = typeof result === 'object' && result !== null;
-            return isResultObject
-                ? { ...result, outboundPublishEvent }
-                : { result, outboundPublishEvent };
+            return (0, telemetry_1.traceSection)('send_publish_batch', { [telemetry_attributes_1.ATTR_FLEXPLM_PAYLOAD_COUNT]: events?.length ?? 0 }, async () => {
+                const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
+                const result = await this.sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent);
+                const isResultObject = typeof result === 'object' && result !== null;
+                return isResultObject
+                    ? { ...result, outboundPublishEvent }
+                    : { result, outboundPublishEvent };
+            });
         }
         else {
             await this.sendPublishPayloadEvent(outboundPublishEvent);
@@ -704,22 +790,24 @@ class BaseProcessPublishAssortment {
         }
     }
     async sendPublishPayloadEvent(outboundPublishEvent) {
-        console.info('sendPublishPayloadEvent()');
-        let initialEvent = {};
-        if (this.config?.event) {
-            initialEvent = (typeof this.config?.event === 'string')
-                ? JSON.parse(this.config?.event)
-                : this.config?.event;
-        }
-        const eventBody = {
-            originSystemType: 'VibeIQ',
-            objectClass: 'AssortmentPublishedToFlexPLM',
-            outboundPublishEvent,
-            initialEvent
-        };
-        await new sdk_1.Entities().create({
-            entityName: 'external-event',
-            object: eventBody
+        return (0, telemetry_1.traceSection)('record_publish_event', {}, async () => {
+            console.info('sendPublishPayloadEvent()');
+            let initialEvent = {};
+            if (this.config?.event) {
+                initialEvent = (typeof this.config?.event === 'string')
+                    ? JSON.parse(this.config?.event)
+                    : this.config?.event;
+            }
+            const eventBody = {
+                originSystemType: 'VibeIQ',
+                objectClass: 'AssortmentPublishedToFlexPLM',
+                outboundPublishEvent,
+                initialEvent
+            };
+            await new sdk_1.Entities().create({
+                entityName: 'external-event',
+                object: eventBody
+            });
         });
     }
     getCurrentDateString() {
@@ -812,6 +900,7 @@ class BaseProcessPublishAssortment {
                 }
             }
         }
+        const skippedDeletes = [];
         for (const [itemId, delEntity] of assortmentItemDeleteMap) {
             const itemFamilyId = delEntity?.item?.itemFamilyId;
             const item = delEntity?.item;
@@ -823,7 +912,7 @@ class BaseProcessPublishAssortment {
             }
             else {
                 if (!item) {
-                    console.error('Failed to find deleted item entity: itemFamilyId: ' + itemFamilyId + ', itemId: ' + itemId);
+                    skippedDeletes.push({ item_family_id: itemFamilyId, item_id: itemId });
                     continue;
                 }
             }
@@ -855,6 +944,14 @@ class BaseProcessPublishAssortment {
                 ifc.colorDeletes.push(itemId);
             }
         } //End deletes for loop
+        if (skippedDeletes.length > 0) {
+            // These deletes are not sent to FlexPLM; one warning for the publish rather than one per item.
+            (0, telemetry_utils_1.setActiveSpanAttributes)({ [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_SKIPPED_DELETE_COUNT]: skippedDeletes.length });
+            telemetry_1.logger.warn({ flexplm: {
+                    direction: 'publish', assortment_id: pcd.assortmentId, publish_change_id: pcd.assortmentPublishChangeId,
+                    skipped_delete_count: skippedDeletes.length, skipped_deletes: skippedDeletes,
+                } }, 'FlexPLM publish skipped deleted items with no item entity');
+        }
         if (app_framework_1.Logger.isDebugOn()) {
             console.debug('returning size: ' + itemFamilyChanges.size);
             for (const [key, value] of itemFamilyChanges) {
@@ -864,16 +961,21 @@ class BaseProcessPublishAssortment {
         return itemFamilyChanges;
     }
     async getEventsForPublishChangeData(publishChangeData) {
-        const seasonalPayloads = [];
-        for (const itemFamilyChange of publishChangeData.itemFamilyChanges.values()) {
-            const events = await this.getEventsForItemFamilyChanges(itemFamilyChange, publishChangeData.assortmentId, publishChangeData.seasonFed, publishChangeData.itemToFederatedIdMapping);
-            if (events.length > 0) {
-                seasonalPayloads.push(...events);
+        return (0, telemetry_1.traceSection)('build_publish_events', {
+            [telemetry_attributes_1.ATTR_FLEXPLM_PUBLISH_ITEM_FAMILY_COUNT]: publishChangeData.itemFamilyChanges?.size ?? 0,
+        }, async (span) => {
+            const seasonalPayloads = [];
+            for (const itemFamilyChange of publishChangeData.itemFamilyChanges.values()) {
+                const events = await this.getEventsForItemFamilyChanges(itemFamilyChange, publishChangeData.assortmentId, publishChangeData.seasonFed, publishChangeData.itemToFederatedIdMapping);
+                if (events.length > 0) {
+                    seasonalPayloads.push(...events);
+                }
             }
-        }
-        console.info('getEventsForPublishChangeData: itemFamilies: '
-            + publishChangeData.itemFamilyChanges.size + ', payloads: ' + seasonalPayloads.length);
-        return seasonalPayloads;
+            console.info('getEventsForPublishChangeData: itemFamilies: '
+                + publishChangeData.itemFamilyChanges.size + ', payloads: ' + seasonalPayloads.length);
+            span.setAttribute(telemetry_attributes_1.ATTR_FLEXPLM_PAYLOAD_COUNT, seasonalPayloads.length);
+            return seasonalPayloads;
+        });
     }
     /**Returns the events for a given ItemFamilyChanges object
      *