@contrail/flexplm 1.7.4-alpha.ad2bc4e → 1.7.4-alpha.b0c1671

package/lib/interfaces/item-family-changes.d.ts

@@ -12,7 +12,6 @@ export declare class ItemFamilyChanges {
     colorDeletes: string[];
     colorUpdates: string[];
     colorUnchanged: string[];
-    colorSuppressed: string[];
     assortmentItemFullChangeMap: Map<string, any>;
     assortmentItemDeleteMap: Map<string, any>;
     itemToFederatedIdMapping: Map<string, string>;

package/lib/interfaces/item-family-changes.js

@@ -14,7 +14,6 @@ class ItemFamilyChanges {
         this.colorDeletes = [];
         this.colorUpdates = [];
         this.colorUnchanged = [];
-        this.colorSuppressed = []; // changed this window, but excluded by the lifecycle filter
         this.assortmentItemFullChangeMap = new Map();
         this.assortmentItemDeleteMap = new Map();
         this.itemToFederatedIdMapping = new Map();
@@ -33,7 +32,6 @@ class ItemFamilyChanges {
         s += ' colorDeletes: [' + this.colorDeletes + ']\n';
         s += ' colorUpdates: [' + this.colorUpdates + ']\n';
         s += ' colorUnchanged: [' + this.colorUnchanged + ']\n';
-        s += ' colorSuppressed: [' + this.colorSuppressed + ']\n';
         s += 'assortmentItemFullChangeMap:\n';
         s += 'size: ' + this.assortmentItemFullChangeMap.size + '\n[';
         for (const key of this.assortmentItemFullChangeMap.keys()) {

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

@@ -15,7 +15,7 @@ class BaseProcessPublishAssortment {
     constructor(_config, _dc, _mapFileUtil) {
         this.TTL = 30 * 24 * 60 * 60 * 1000; // 30 days
         this.cache = {
-            carriedFromSeason: {},
+            carriedFromSeason: {}
         };
         this.config = _config;
         this.dc = _dc;
@@ -46,13 +46,13 @@ class BaseProcessPublishAssortment {
                 console.log(message);
                 const output = {
                     results: { message, event },
-                    skip_await: true,
+                    skip_await: true
                 };
                 return output;
             }
             apcHistory = await this.getApcHistory(assortmentId);
             const sinceDate = await this.getSinceDate(assortmentId, assortmentPublishChangeId, apcHistory);
-            // Get detail information
+            //Get detail information
             const assortmentPublishChange = await this.downloadAssortmentPublishChange(assortmentId, assortmentPublishChangeId);
             publisher = this.getPublisher(assortmentPublishChange);
             const changeDetail = await this.downloadHydratedChangeDetail(assortmentPublishChange);
@@ -91,7 +91,7 @@ class BaseProcessPublishAssortment {
             versionComments: apc?.versionComments,
             publishDate: apc?.createdOn,
             publisher,
-            versionHistoryNumber,
+            versionHistoryNumber
         };
         return publishInfo;
     }
@@ -111,15 +111,15 @@ class BaseProcessPublishAssortment {
             return 'unknown';
         }
         const createdOnDate = new Date(createdOnString);
-        createdOnDate.setMonth(createdOnDate.getMonth() - 1); // subtract 1 month
+        createdOnDate.setMonth(createdOnDate.getMonth() - 1); //subtract 1 month
         const createdOnStringMinus1 = createdOnDate.toISOString();
         const createdOn = 'ISGREATERTHAN ' + createdOnStringMinus1;
         const snapshots = await new sdk_1.Entities().get({
             entityName: 'entity-snapshot',
             criteria: {
                 entityReference,
-                createdOn,
-            },
+                createdOn
+            }
         });
         const assortmentPublishChangeId = apc?.id;
         const snapshot = snapshots.find(snap => snap?.assortmentChangeId === assortmentPublishChangeId);
@@ -169,7 +169,7 @@ class BaseProcessPublishAssortment {
         }
         this.assortment = await new sdk_1.Entities().get({
             entityName: 'assortment',
-            id: assortmentId,
+            id: assortmentId
         });
         return this.assortment;
     }
@@ -214,9 +214,9 @@ class BaseProcessPublishAssortment {
         return sdk_1.Request.request(resource, {
             method: 'GET',
             headers: {
-                "Accept": 'application/json',
-                'Content-Type': 'application/json',
-            },
+                Accept: 'application/json',
+                'Content-Type': 'application/json'
+            }
         });
     }
     updatedSinceDate(entity, sinceDate) {
@@ -287,7 +287,7 @@ class BaseProcessPublishAssortment {
                 }
             }
         }
-        dateArray.sort((a, b) => Date.parse(a) - Date.parse(b));
+        dateArray.sort((a, b) => { return Date.parse(a) - Date.parse(b); });
         const arrIndex = (dateArray.length - pastPublishCount > 0) ? dateArray.length - pastPublishCount : 0;
         sinceDate = dateArray[arrIndex];
         return sinceDate;
@@ -308,9 +308,9 @@ class BaseProcessPublishAssortment {
         return await sdk_1.Request.request(resourceString, {
             method: 'GET',
             Headers: {
-                "Accept": 'application/json',
-                'Content-Type': 'application/json',
-            },
+                Accept: 'application/json',
+                'Content-Type': 'application/json'
+            }
         });
     }
     async downloadHydratedChangeDetail(assortmentPublishChange) {
@@ -358,7 +358,7 @@ class BaseProcessPublishAssortment {
             console.info('sinceDateMilliseconds: ' + sinceDateMilliseconds);
             console.info('apcDateMilliseconds: ' + previousApcDate);
         }
-        // if only 1 apc, no processing needed
+        //if only 1 apc, no processing needed
         if (sinceDateMilliseconds !== previousApcDate) {
             console.info('sinceDateMilliseconds !== apcDateMilliseconds');
             const currentAssortmentItemIds = this.getBaselineItemIds(assortmentBaseline);
@@ -444,7 +444,7 @@ class BaseProcessPublishAssortment {
             previousBaseline = await this.downloadAssortmentBaseline(previousApc);
         }
         const deleteIds = apc?.detail?.deletes.map(dItem => dItem?.id);
-        // building deletes based on previous baseline; because some APCs don't have delete data
+        //building deletes based on previous baseline; because some APCs don't have delete data
         const deleteArray = previousBaseline?.assortmentItems.filter(aItem => deleteIds.includes(aItem?.itemId));
         console.info('deleteArray.length: ' + deleteArray.length);
         return deleteArray;
@@ -495,18 +495,13 @@ class BaseProcessPublishAssortment {
             const itemFamilyId = item?.itemFamilyId;
             if (meetsCriteria) {
                 releasedForDevelopmentItemIds.push(itemId);
-            }
-            // ---- NEW (Step 5): family resolution moved OUT of the meetsCriteria block.
-            // A family whose options are ALL pre-development has no other route into this
-            // list when it has no assortment-item row of its own. ----
-            if (!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId) {
-                const familyItem = item?.itemFamily;
-                if (familyItem) {
-                    if (this.meetsCriteria({ item: familyItem })
-                        && !releasedForDevelopmentItemIds.includes(itemFamilyId)) {
+                if (!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId) {
+                    const familyItem = item?.itemFamily;
+                    const familyItemMeetsCriteria = this.meetsCriteria({ item: familyItem });
+                    if (familyItemMeetsCriteria) {
                         releasedForDevelopmentItemIds.push(itemFamilyId);
                     }
-                    itemFamilySet.add(itemFamilyId); // only mark resolved when we had data to judge
+                    itemFamilySet.add(itemFamilyId);
                 }
             }
         }
@@ -544,7 +539,7 @@ class BaseProcessPublishAssortment {
         console.info('processPublish-start');
         const event = {
             assortmentId: pcd.assortmentId,
-            assortmentPublishChangeId: pcd.assortmentPublishChangeId,
+            assortmentPublishChangeId: pcd.assortmentPublishChangeId
         };
         const assortmentItemFullChangeMap = this.getFullChangeAssortmentMap(fullChange);
         const assortmentItemDeleteMap = this.getDeleteChangesAssortmentMap(deleteChanges);
@@ -559,7 +554,7 @@ class BaseProcessPublishAssortment {
             console.log(message);
             return {
                 results: { message, event },
-                skip_await: true,
+                skip_await: true
             };
         }
         const results = await this.sendEvents(events);
@@ -604,7 +599,7 @@ class BaseProcessPublishAssortment {
             taskId: this.config.taskId,
             eventType,
             objectClass: 'LCSSeason',
-            events,
+            events
         };
         const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
         const result = await this.sendAndRecordPublishEvent(flexPLMConnect, outboundPublishEvent);
@@ -650,13 +645,13 @@ class BaseProcessPublishAssortment {
                 taskId: this.config.taskId,
                 eventType,
                 objectClass: 'LCSSeason',
-                events,
+                events
             };
             await fileHandle.writeFile(JSON.stringify(outboundPublishEvent));
             results = {
                 message: 'Successfully Saved File',
                 outboundPublishEvent,
-                fileLocation: dirName + fileName,
+                fileLocation: dirName + fileName
             };
         }
         catch (e) {
@@ -677,7 +672,7 @@ class BaseProcessPublishAssortment {
             objectClass: 'LCSSeason',
             eventsFileId: uploadFile.id,
             eventsDownloadLink: uploadFile.downloadUrl,
-            eventsAdminDownloadLink: uploadFile.adminDownloadUrl,
+            eventsAdminDownloadLink: uploadFile.adminDownloadUrl
         };
         if (sendMode === 'vibeiqfile') {
             const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
@@ -691,7 +686,7 @@ class BaseProcessPublishAssortment {
             await this.sendPublishPayloadEvent(outboundPublishEvent);
             return {
                 message: 'Successfully Uploaded File.',
-                outboundPublishEvent,
+                outboundPublishEvent
             };
         }
     }
@@ -707,11 +702,11 @@ class BaseProcessPublishAssortment {
             originSystemType: 'VibeIQ',
             objectClass: 'AssortmentPublishedToFlexPLM',
             outboundPublishEvent,
-            initialEvent,
+            initialEvent
         };
         await new sdk_1.Entities().create({
             entityName: 'external-event',
-            object: eventBody,
+            object: eventBody
         });
     }
     getCurrentDateString() {
@@ -735,7 +730,7 @@ class BaseProcessPublishAssortment {
         for (const [itemId, aItem] of assortmentItemFullChangeMap) {
             const projectItem = aItem?.projectItem;
             const itemFamilyId = aItem?.item?.itemFamilyId;
-            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId)) {
+            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId) || !pcd.releasedForDevelopmentItemIds.includes(itemId)) {
                 continue;
             }
             const ifc = itemFamilyChanges.get(itemFamilyId) || new item_family_changes_1.ItemFamilyChanges(itemFamilyId, pcd.sinceDate);
@@ -747,20 +742,6 @@ class BaseProcessPublishAssortment {
             if (pcd.itemToFederatedIdMapping.has(itemId)) {
                 ifc.itemToFederatedIdMapping.set(itemId, pcd.itemToFederatedIdMapping.get(itemId));
             }
-            // ---- NEW Step to enable ItemFamily publish ET-243: option is pre-development. It must not produce an
-            // LCSSKUSeasonLink, but a change to it still makes the family publishable. ----
-            if (!pcd.releasedForDevelopmentItemIds.includes(itemId)) {
-                const changedInWindow = addIds.includes(itemId)
-                    || deleteIds.includes(itemId)
-                    || updateIds.includes(itemId)
-                    || this.updatedSinceDate(projectItem, pcd.sinceDate)
-                    || this.updatedSinceDate(aItem?.item, pcd.sinceDate);
-                if (changedInWindow) {
-                    ifc.colorSuppressed.push(itemId);
-                }
-                continue;
-            }
-            // NEW Step - ends
             if (itemId === itemFamilyId) {
                 if (addIds.includes(itemId))
                     ifc.familyAdd = true;
@@ -776,12 +757,6 @@ class BaseProcessPublishAssortment {
                         ifc.familyUpdate = true;
                     }
                 }
-                // NEW Step to enable ItemFamily publish ET-243: item-level edits (incl. lifecycleStage) must publish
-                // the family. Guarded so the "first option added" flow stays suppressed. ----
-                if (!ifc.familyItemRemoved && this.updatedSinceDate(aItem?.item, pcd.sinceDate)) {
-                    ifc.familyUpdate = true;
-                }
-                // NEW Step - ends
                 if (projectItem && !projectItem?.roles && aItem?.item?.roles) {
                     projectItem.roles = aItem?.item?.roles;
                 }
@@ -838,41 +813,7 @@ class BaseProcessPublishAssortment {
             else {
                 ifc.colorDeletes.push(itemId);
             }
-        } // End deletes for loop
-        // ---- ET-243: the family's own row left the assortment while the family still has
-        // option rows. The platform reports this in changeDetail.familyItemsRemoved, and the
-        // row is in neither the baseline nor the delete-data file, so neither loop above can
-        // see it. `deletes` is folded in as a fallback in case a removal is ever classified
-        // there while still missing from the delete-data file. ----
-        for (const removedRow of [...familyItemsRemoved, ...deletes]) {
-            const itemId = removedRow?.itemId ?? removedRow?.id;
-            const item = removedRow?.item;
-            if (!itemId || !item)
-                continue;
-            const itemFamilyId = item?.itemFamilyId ?? itemId;
-            if (itemId !== itemFamilyId)
-                continue; // family rows only
-            if (assortmentItemFullChangeMap.has(itemId))
-                continue; // still in baseline -> loop 1
-            if (assortmentItemDeleteMap.has(itemId))
-                continue; // hard delete    -> loop 2
-            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId))
-                continue;
-            if (!this.meetsCriteria(removedRow))
-                continue;
-            let ifc = itemFamilyChanges.get(itemFamilyId);
-            if (!ifc) {
-                ifc = new item_family_changes_1.ItemFamilyChanges(itemFamilyId, pcd.sinceDate);
-                itemFamilyChanges.set(itemFamilyId, ifc);
-            }
-            if (!ifc.itemFamilyObject) {
-                ifc.itemFamilyObject = item;
-            }
-            if (removedRow?.projectItem) {
-                ifc.assortmentItemFullChangeMap.set(itemId, removedRow);
-            }
-            ifc.familyItemRemoved = true;
-        }
+        } //End deletes for loop
         if (app_framework_1.Logger.isDebugOn()) {
             console.debug('returning size: ' + itemFamilyChanges.size);
             for (const [key, value] of itemFamilyChanges) {
@@ -914,17 +855,13 @@ class BaseProcessPublishAssortment {
         const LCSSeason = Object.assign({}, seasonFed);
         const assortment = await this.getAssortment(assortmentId);
         const projectItem = this.getProjectItem(itemFamilyChanges, itemFamilyChanges.itemFamilyId);
-        // ---- NEW Step to enable ItemFamily publish ET-243: added color suppressed length to existing condition and familyItemRemovedFlag
         const familyAssortmentItem = itemFamilyChanges.familyAdd || itemFamilyChanges.familyUpdate || itemFamilyChanges.familyDelete
-            || itemFamilyChanges.familyItemRemoved
             || itemFamilyChanges.colorAdds.length > 0 || itemFamilyChanges.colorUpdates.length > 0 || itemFamilyChanges.colorDeletes.length > 0
-            || itemFamilyChanges.colorSuppressed.length > 0
             || (projectItem && this.updatedSinceDate(projectItem, itemFamilyChanges.sinceDate));
-        // familyItemRemoved: the family's own assortment row left the assortment while the
-        // family still has option rows. The family stays in the season, so this publishes an
-        // LCSProductSeasonLink update and no LCSSKUSeasonLink.
+        //familyItemRemoved is used when adding the first option to an assortment
+        //and will have updates for the family item.
         if (familyAssortmentItem) {
-            // Product-season add
+            //Product-season add
             const entityReference = itemFamilyChanges.itemFamilyId;
             const prodEntityData = (itemFamilyChanges.assortmentItemFullChangeMap.has(entityReference))
                 ? itemFamilyChanges.assortmentItemFullChangeMap.get(entityReference)?.item
@@ -940,7 +877,7 @@ class BaseProcessPublishAssortment {
                 entityReference: 'item:' + entityReference,
                 LCSSeason,
                 LCSProduct,
-                data: seasonalData,
+                data: seasonalData
             };
             if ('ASSORTMENT' === projectItem.addedFromSource && projectItem.addedFromAssortment) {
                 const carriedFromSeason = await this.getCarriedFromSeason(projectItem);
@@ -950,9 +887,9 @@ class BaseProcessPublishAssortment {
             }
             events.push(psUpsert);
         }
-        // colorway-season adds
-        // colorAdds
-        // colorUpdates
+        //colorway-season adds
+        //colorAdds
+        //colorUpdates
         const colorwayChanges = [];
         colorwayChanges.push(...itemFamilyChanges.colorAdds);
         colorwayChanges.push(...itemFamilyChanges.colorUpdates);
@@ -975,7 +912,7 @@ class BaseProcessPublishAssortment {
                 entityReference: 'item:' + entityReference,
                 LCSSeason,
                 LCSSKU,
-                data: seasonalData,
+                data: seasonalData
             };
             if ('ASSORTMENT' === projectItem.addedFromSource && projectItem.addedFromAssortment) {
                 const carriedFromSeason = await this.getCarriedFromSeason(projectItem);
@@ -991,10 +928,10 @@ class BaseProcessPublishAssortment {
         let projectItem = {};
         if (itemFamilyChanges?.assortmentItemFullChangeMap.has(id)) {
             const aItem = itemFamilyChanges?.assortmentItemFullChangeMap.get(id);
-            projectItem = aItem?.projectItem ?? {};
+            projectItem = aItem?.projectItem;
         }
         /////////////////////////////////////////////////////////////////////////////
-        // Get from option assortmentItem because family not in assortmentItemFullChangeMap:start
+        //Get from option assortmentItem because family not in assortmentItemFullChangeMap:start
         /////////////////////////////////////////////////////////////////////////////
         if (id === itemFamilyChanges.itemFamilyId && Object.keys(projectItem).length == 0) {
             for (const asstItem of itemFamilyChanges.assortmentItemFullChangeMap.values()) {
@@ -1005,7 +942,7 @@ class BaseProcessPublishAssortment {
             }
         }
         /////////////////////////////////////////////////////////////////////////////
-        // Get from option assortmentItem because family not in assortmentItemFullChangeMap:end
+        //Get from option assortmentItem because family not in assortmentItemFullChangeMap:end
         /////////////////////////////////////////////////////////////////////////////
         return projectItem;
     }
@@ -1055,7 +992,7 @@ class BaseProcessPublishAssortment {
         if (this.cache.carriedFromSeason[addedFromAssortment]) {
             return this.cache.carriedFromSeason[addedFromAssortment];
         }
-        let seasonFederation;
+        let seasonFederation = undefined;
         const assortmentEntity = await this.getAssormentEntityFromId(addedFromAssortment);
         if (assortmentEntity) {
             seasonFederation = await this.getSeasonFederationFromAssortment(assortmentEntity);
@@ -1068,7 +1005,7 @@ class BaseProcessPublishAssortment {
         try {
             assortment = await new sdk_1.Entities().get({
                 entityName: 'assortment',
-                id: assortmentId,
+                id: assortmentId
             });
         }
         catch (e) {
@@ -1082,7 +1019,7 @@ class BaseProcessPublishAssortment {
         let seasonObj = {
             entityReference: 'assortment:' + assortment.id,
             objectClass: 'LCSSeason',
-            flexPLMTypePath,
+            flexPLMTypePath
         };
         try {
             const assortmentObj = await this.dc.getFlexPLMObjectData(assortment, [], true);

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

@@ -1337,176 +1337,6 @@ describe('getItemFamilyChanges', () => {
         expect(ifc.colorDeletes.length).toEqual(1);
     });
 });
-describe('getItemFamilyChanges - lifecycle-suppressed options', () => {
-    const familyId = 'AERfdvdAt1HHEVvQ';
-    const maroonId = 'YwWKVoL_vGN8khLy';
-    const limeId = 'rsmWhWiXRNv_XBJG';
-    const sinceDate = new Date('2026-08-01T00:00:00.000Z');
-    const afterSince = '2026-08-26T00:00:00.000Z';
-    const beforeSince = '2026-05-11T00:00:00.000Z';
-    const noDetailChanges = () => ({ adds: [], deletes: [], updates: [], familyItemsRemoved: [] });
-    const buildPpa = () => {
-        const config = { itemPreDevelopmentLifecycleStages: ['concept'] };
-        const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
-        const dc = new data_converter_1.DataConverter(config, mapFileUtil);
-        return new base_process_publish_assortment_1.BaseProcessPublishAssortment(config, dc, mapFileUtil);
-    };
-    const buildPcd = (releasedIds) => {
-        const seasonFed = {
-            entityReference: 'assortment:11EZML5dMn3tREuT',
-            objectClass: 'LCSSeason'
-        };
-        const pcd = new publish_change_data_1.PublishChangeData('11EZML5dMn3tREuT', seasonFed, 'Pg3y2x5IclYxk3hM', sinceDate);
-        pcd.itemToFederatedIdMapping = new Map();
-        releasedIds.forEach(id => pcd.releasedForDevelopmentItemIds.push(id));
-        return pcd;
-    };
-    const familyItem = (updatedOn = beforeSince, lifecycleStage = 'released') => ({
-        id: familyId, itemFamilyId: familyId, lifecycleStage, name: 'Sweatpant', updatedOn
-    });
-    const familyRow = (updatedOn = beforeSince, lifecycleStage = 'released') => ({
-        itemId: familyId,
-        item: familyItem(updatedOn, lifecycleStage),
-        projectItem: { id: 'pi-' + familyId, updatedOn: beforeSince }
-    });
-    const optionRow = (id, updatedOn, lifecycleStage = 'concept') => ({
-        itemId: id,
-        item: { id, itemFamilyId: familyId, lifecycleStage, updatedOn, itemFamily: familyItem() },
-        projectItem: { id: 'pi-' + id, updatedOn: beforeSince },
-        familyProjectItem: { id: 'pi-' + familyId, updatedOn: beforeSince }
-    });
-    it('records changed concept options as suppressed, not as colour updates', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(), optionRow(maroonId, afterSince), optionRow(limeId, afterSince)]
-        });
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        expect(pcd.itemFamilyChanges.size).toEqual(1);
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.colorSuppressed).toEqual([maroonId, limeId]);
-        expect(ifc.colorAdds.length).toEqual(0);
-        expect(ifc.colorUpdates.length).toEqual(0);
-        expect(ifc.colorDeletes.length).toEqual(0);
-        expect(ifc.familyUpdate).toBe(false); // family itself unchanged
-    });
-    it('keeps the family reachable when only concept option rows are in the baseline', () => {
-        const ppa = buildPpa();
-        const assortmentItems = [optionRow(maroonId, afterSince), optionRow(limeId, afterSince)];
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({ assortmentItems });
-        // derive the gate the way production does, rather than seeding it
-        const releasedIds = ppa.getReleasedForDevelopmentItemAndFamilyIds({ assortmentItems }, []);
-        expect(releasedIds).toEqual([familyId]);
-        const pcd = buildPcd(releasedIds);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        expect(pcd.itemFamilyChanges.size).toEqual(1);
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.itemFamilyObject.id).toEqual(familyId);
-        expect(ifc.colorSuppressed.length).toEqual(2);
-        expect(ppa.getProjectItem(ifc, familyId)).toEqual({ id: 'pi-' + familyId, updatedOn: beforeSince });
-    });
-    it('does not suppress-record an unchanged concept option', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(), optionRow(maroonId, beforeSince)]
-        });
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.colorSuppressed.length).toEqual(0);
-    });
-    it('picks up a concept option listed in the change detail even when timestamps are stale', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(), optionRow(maroonId, beforeSince)]
-        });
-        const details = noDetailChanges();
-        details.updates = [{ id: maroonId }];
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, details, fullChangeMap, new Map());
-        expect(pcd.itemFamilyChanges.get(familyId).colorSuppressed).toEqual([maroonId]);
-    });
-    it('skips everything when the family itself is pre-development', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(afterSince, 'concept'), optionRow(maroonId, afterSince)]
-        });
-        const pcd = buildPcd([]); // nothing released
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        expect(pcd.itemFamilyChanges.size).toEqual(0);
-    });
-    it('still classifies released options normally', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(), optionRow(maroonId, afterSince, 'released')]
-        });
-        const details = noDetailChanges();
-        details.adds = [{ id: maroonId }];
-        const pcd = buildPcd([familyId, maroonId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, details, fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.colorAdds).toEqual([maroonId]);
-        expect(ifc.colorSuppressed.length).toEqual(0);
-    });
-    it('publishes a family-only row whose lifecycle just flipped (item-level edit)', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({
-            assortmentItems: [familyRow(afterSince)] // item.updatedOn moved, projectItem did not
-        });
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.familyUpdate).toBe(true); // fails without Step 3
-        expect(ifc.colorSuppressed.length).toEqual(0);
-    });
-    it('leaves an unchanged family-only row with no flags', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({ assortmentItems: [familyRow()] });
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, noDetailChanges(), fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.familyAdd).toBe(false);
-        expect(ifc.familyUpdate).toBe(false);
-        expect(ifc.familyDelete).toBe(false);
-        expect(ifc.colorSuppressed.length).toEqual(0);
-    });
-    it('does not set familyUpdate when the family item was removed', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({ assortmentItems: [familyRow(afterSince)] });
-        const details = noDetailChanges();
-        details.familyItemsRemoved = [{ itemId: familyId }]; // note: itemId, not id
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, details, fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.familyItemRemoved).toBe(true);
-        expect(ifc.familyUpdate).toBe(false); // fails if the !familyItemRemoved guard is dropped
-        expect(ifc.assortmentItemFullChangeMap.size).toEqual(1); // third pass must not re-insert the row
-    });
-    it('flags the family when its row is removed but option rows remain', () => {
-        const ppa = buildPpa();
-        const assortmentItems = [optionRow(maroonId, beforeSince), optionRow(limeId, beforeSince, 'released')];
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({ assortmentItems });
-        const details = { ...noDetailChanges(), familyItemsRemoved: [{ itemId: familyId, item: familyItem() }] };
-        const pcd = buildPcd([familyId, limeId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, details, fullChangeMap, new Map());
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.familyItemRemoved).toBe(true); // false before the third pass
-        expect(ifc.itemFamilyObject.id).toEqual(familyId);
-        expect(ifc.colorDeletes.length).toEqual(0); // no SKU event should follow
-        expect(ifc.assortmentItemFullChangeMap.has(familyId)).toBe(false); // removed row carries no projectItem
-    });
-    it('leaves a hard-deleted family row to the delete pass', () => {
-        const ppa = buildPpa();
-        const fullChangeMap = ppa.getFullChangeAssortmentMap({ assortmentItems: [] });
-        const deleteMap = new Map([[familyId, familyRow()]]);
-        const details = { ...noDetailChanges(), familyItemsRemoved: [{ itemId: familyId, item: familyItem() }] };
-        const pcd = buildPcd([familyId]);
-        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, details, fullChangeMap, deleteMap);
-        const ifc = pcd.itemFamilyChanges.get(familyId);
-        expect(ifc.familyDelete).toBe(true); // set by the delete pass
-        expect(ifc.familyItemRemoved).toBe(false); // third pass skipped it - no double handling
-    });
-});
 describe('getProjectItem', () => {
     const config = {
         identifierAtts: {
@@ -1670,70 +1500,6 @@ describe('meetsCriteria', () => {
         expect(results).toBeTruthy();
     });
 });
-describe('getReleasedForDevelopmentItemAndFamilyIds', () => {
-    const familyId = 'AERfdvdAt1HHEVvQ';
-    const maroonId = 'YwWKVoL_vGN8khLy';
-    const limeId = 'rsmWhWiXRNv_XBJG';
-    const buildPpa = (preDev = ['concept']) => {
-        const config = {
-            identifierAtts: { LCSSeason: ['flexPLMSeasonName'] },
-            itemPreDevelopmentLifecycleStages: preDev
-        };
-        const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
-        const dc = new data_converter_1.DataConverter(config, mapFileUtil);
-        return new base_process_publish_assortment_1.BaseProcessPublishAssortment(config, dc, mapFileUtil);
-    };
-    const familyItem = (lifecycleStage = 'released') => ({
-        id: familyId, itemFamilyId: familyId, lifecycleStage, name: 'Sweatpant'
-    });
-    // option row as the hydrated baseline actually delivers it: itemFamily present
-    const optionRow = (id, lifecycleStage = 'concept', family = familyItem()) => ({
-        itemId: id,
-        item: { id, itemFamilyId: familyId, lifecycleStage, itemFamily: family }
-    });
-    const familyRow = (lifecycleStage = 'released') => ({
-        itemId: familyId,
-        item: familyItem(lifecycleStage)
-    });
-    const run = (ppa, assortmentItems) => ppa.getReleasedForDevelopmentItemAndFamilyIds({ assortmentItems }, []);
-    it('surfaces the family when every option is pre-development', () => {
-        const ppa = buildPpa();
-        expect(run(ppa, [optionRow(maroonId), optionRow(limeId)])).toEqual([familyId]);
-    });
-    it('does not duplicate the family id when a family row and a released option both exist', () => {
-        const ppa = buildPpa();
-        const ids = run(ppa, [familyRow(), optionRow(limeId, 'released')]);
-        expect(ids.filter(id => id === familyId)).toHaveLength(1);
-    });
-    it('still resolves the family when an earlier row lacks itemFamily hydration', () => {
-        const ppa = buildPpa();
-        const noFamilyHydration = {
-            itemId: maroonId,
-            item: { id: maroonId, itemFamilyId: familyId, lifecycleStage: 'concept' }
-        };
-        expect(run(ppa, [noFamilyHydration, optionRow(limeId)])).toEqual([familyId]);
-    });
-    it('returns nothing when the family itself is pre-development', () => {
-        const ppa = buildPpa();
-        expect(run(ppa, [optionRow(maroonId, 'concept', familyItem('concept'))])).toEqual([]);
-    });
-    it('includes a family-only row exactly once', () => {
-        const ppa = buildPpa();
-        expect(run(ppa, [familyRow()])).toEqual([familyId]);
-    });
-    it('includes a released option and its family, excluding the concept option', () => {
-        const ppa = buildPpa();
-        const ids = run(ppa, [optionRow(maroonId, 'concept'), optionRow(limeId, 'released')]);
-        expect(ids).toContain(familyId);
-        expect(ids).toContain(limeId);
-        expect(ids).not.toContain(maroonId);
-    });
-    it('treats the stage list as case-sensitive (documents the Concept vs concept trap)', () => {
-        const ppa = buildPpa(['Concept']); // as mis-configured in praveen-dev-1
-        const ids = run(ppa, [optionRow(maroonId), optionRow(limeId)]);
-        expect(ids).toEqual([maroonId, familyId, limeId]); // filter inert: everything qualifies
-    });
-});
 describe('getEventsForItemFamilyChanges - conditional eventType', () => {
     const config = {
         identifierAtts: {
@@ -1939,35 +1705,6 @@ describe('getEventsForItemFamilyChanges - conditional eventType', () => {
         // Verify SKU event called with assortment entity
         expect(type_conversion_utils_1.TypeConversionUtils.isOutboundCreatableFromEntity).toHaveBeenNthCalledWith(2, undefined, mapFileUtil, colorProjectItem, { item: itemData, assortment });
     });
-    it('should emit only an LCSProductSeasonLink when the family row was removed', async () => {
-        const bppa = new base_process_publish_assortment_1.BaseProcessPublishAssortment(config, dc, mapFileUtil);
-        const itemFamilyId = 'AERfdvdAt1HHEVvQ';
-        const prodEntityData = { id: itemFamilyId, identifier: 'FAM-11', itemNumber: 'FAM-11' };
-        //const projectItem = { id: 'proj-fam', updatedOn: '2023-01-15T00:00:00.000Z' };
-        const projectItem = { id: 'proj-fam', updatedOn: '2022-12-01T00:00:00.000Z' };
-        const assortment = { id: 'assort123', name: 'Fall 2023 Assortment', publishToFlexPLM: true };
-        const seasonFed = {
-            entityReference: 'assortment:assort123',
-            objectClass: 'LCSSeason',
-            flexPLMSeasonName: 'Fall 2023'
-        };
-        const itemFamilyChanges = new item_family_changes_1.ItemFamilyChanges(itemFamilyId, new Date('2023-01-01T00:00:00.000Z'));
-        itemFamilyChanges.familyItemRemoved = true; // the only trigger
-        itemFamilyChanges.colorUnchanged.push('rsmWhWiXRNv_XBJG'); // surviving option must not emit
-        itemFamilyChanges.itemFamilyObject = prodEntityData;
-        jest.spyOn(bppa, 'getAssortment').mockResolvedValue(assortment);
-        jest.spyOn(bppa, 'getProjectItem').mockReturnValue(projectItem);
-        jest.spyOn(bppa, 'getSeasonalData').mockResolvedValue({ someData: 'value' });
-        jest.spyOn(map_utils_1.MapUtil, 'applyTransformMap').mockResolvedValue({ transformedData: 'value' });
-        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'getIdentifierProperties').mockResolvedValue(['identifier']);
-        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'getInformationalProperties').mockResolvedValue([]);
-        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'isOutboundCreatableFromEntity').mockResolvedValue(false);
-        const events = await bppa.getEventsForItemFamilyChanges(itemFamilyChanges, 'assort123', seasonFed, new Map());
-        expect(events).toHaveLength(1);
-        expect(events[0].objectClass).toBe('LCSProductSeasonLink');
-        expect(events[0].eventType).toBe('UPDATE_ON_SEASON');
-        expect(events[0].entityReference).toBe('item:' + itemFamilyId);
-    });
 });
 describe('sendToFlexPLM / handleVibeIQFile / sendPublishPayloadEvent', () => {
     const config = {

package/lib/util/flexplm-connect.js

@@ -63,7 +63,6 @@ class FlexPLMConnect {
             const nonce_key = body.items[0].attributes.nonce_key;
             const nonce = body.items[0].attributes.nonce;
             console.info('nonce_key: ' + nonce_key);
-            console.info('nonce: ' + nonce);
             return {
                 nonce_key,
                 nonce
@@ -89,7 +88,10 @@ class FlexPLMConnect {
             console.info('Request:');
             console.info('vibeEventsURL: ' + vibeEventsURL);
             const logOptions = JSON.parse(JSON.stringify(xferOptions));
-            logOptions['headers']['Authorization'] = logOptions?.headers?.Authorization.substring(0, 9);
+            if (logOptions?.headers?.Authorization)
+                logOptions.headers.Authorization = '***masked***';
+            if (csrf?.nonce_key)
+                logOptions.headers[csrf.nonce_key] = '***masked***';
             console.info('csrfOptions: ' + JSON.stringify(logOptions));
             console.info('Making call to xfer data to FlexPLM');
         }

package/lib/util/flexplm-connect.spec.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const flexplm_connect_1 = require("./flexplm-connect");
+const app_framework_1 = require("@contrail/app-framework");
 const mockJsonData = { items: [{ id: '123', name: 'test' }] };
 const mockResponse = (status, json = mockJsonData) => ({
     status,
@@ -87,3 +88,22 @@ describe('FlexPLMConnect.getRequest', () => {
         await expect(connect.getRequest()).rejects.toThrow('Error getting json data from FlexPLM: Invalid JSON');
     });
 });
+describe('FlexPLMConnect.sendRequest', () => {
+    it('should mask csrf nonce value in logged request options', async () => {
+        const loggerIsInfoOnSpy = jest.spyOn(app_framework_1.Logger, 'isInfoOn').mockReturnValue(true);
+        const consoleInfoSpy = jest.spyOn(console, 'info').mockImplementation();
+        const csrf = { nonce_key: 'CSRF_NONCE', nonce: 'raw-secret-nonce' };
+        const payload = { id: '123' };
+        const connect = new flexplm_connect_1.FlexPLMConnect(createConfig());
+        await connect.sendRequest(csrf, payload);
+        const csrfOptionsLogMessages = consoleInfoSpy.mock.calls
+            .map(([message]) => String(message ?? ''))
+            .filter(message => message.startsWith('csrfOptions: '));
+        const sendRequestOptionsLogMessage = csrfOptionsLogMessages.find(message => message.includes('"CSRF_NONCE":'));
+        expect(sendRequestOptionsLogMessage).toBeDefined();
+        expect(sendRequestOptionsLogMessage ?? '').not.toContain('raw-secret-nonce');
+        expect(sendRequestOptionsLogMessage ?? '').toContain('"CSRF_NONCE":"***masked***"');
+        loggerIsInfoOnSpy.mockRestore();
+        consoleInfoSpy.mockRestore();
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.7.4-alpha.ad2bc4e",
+  "version": "1.7.4-alpha.b0c1671",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",