@contrail/flexplm 1.7.4-alpha.2fb7e1f → 1.7.4-alpha.458d29b

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

@@ -12,6 +12,7 @@ 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,6 +14,7 @@ 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();
@@ -32,6 +33,7 @@ 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

@@ -29,8 +29,7 @@ class BaseProcessPublishAssortment {
         let publisher;
         const assortmentId = event.assortmentId;
         try {
-            console.info('process-start: assortmentId: ' + assortmentId
-                + ', assortmentPublishChangeId: ' + assortmentPublishChangeId);
+            console.info('process-start!');
             let seasonFed;
             try {
                 seasonFed = await this.getSeasonFederation(assortmentId);
@@ -75,9 +74,9 @@ class BaseProcessPublishAssortment {
                 e.publishInfo = publishInfo;
             }
             catch (e2) {
-                console.error('getPublishInfo failed: ' + e2.message);
+                console.log('catch e2: ' + e2.message);
             }
-            console.error('process failed: assortmentId: ' + assortmentId + ': ' + e.message);
+            console.log('catch e: ' + e.message);
             throw e;
         }
     }
@@ -316,17 +315,19 @@ class BaseProcessPublishAssortment {
     }
     async downloadHydratedChangeDetail(assortmentPublishChange) {
         try {
+            console.info('downloadHydratedChangeDetail-start');
             const link = assortmentPublishChange?.hydratedDetailDownloadLink;
             const response = await fetch(link);
             const data = await response.json();
             return data;
         }
         catch (e) {
-            console.error('Error hydratedDetailDownloadLink: ' + e.message);
+            console.log('Error hydratedDetailDownloadLink: ' + e.message);
         }
         return undefined;
     }
     async downloadAssortmentBaseline(assortmentPublishChange) {
+        console.info('downloadAssortmentBaseline-start');
         try {
             const link = assortmentPublishChange?.assortmentBaselineDownloadLink;
             const response = await fetch(link);
@@ -334,12 +335,15 @@ class BaseProcessPublishAssortment {
             return data;
         }
         catch (e) {
-            console.error('Error assortmentBaselineDownloadLink: ' + e.message);
+            console.log('Error assortmentBaselineDownloadLink: ' + e.message);
         }
         return undefined;
     }
     async getDeleteChanges(assortmentPublishChange, apcHistory, assortmentBaseline, sinceDate) {
+        console.info('getDeleteChanges(): ' + assortmentPublishChange?.id);
+        console.info(sinceDate);
         const currentAPCIndex = apcHistory.findIndex(pc => pc?.id === assortmentPublishChange?.id);
+        console.info(' currentAPCIndex: ' + currentAPCIndex);
         if (currentAPCIndex == 0) {
             return [];
         }
@@ -348,13 +352,16 @@ class BaseProcessPublishAssortment {
         const apcDeletes = await this.downloadDeleteChanges(assortmentPublishChange);
         let previousApcDate = Date.parse(apcCreatedOn);
         const sinceDateMilliseconds = sinceDate.getTime();
-        console.info('getDeleteChanges(): apc: ' + assortmentPublishChange?.id
-            + ', index: ' + currentAPCIndex + ', previousApcCreatedOn: ' + apcCreatedOn);
+        if (app_framework_1.Logger.isInfoOn()) {
+            console.info('apcCreatedOn: ' + apcCreatedOn);
+            console.info('sinceDate: ' + sinceDate);
+            console.info('sinceDateMilliseconds: ' + sinceDateMilliseconds);
+            console.info('apcDateMilliseconds: ' + previousApcDate);
+        }
         //if only 1 apc, no processing needed
         if (sinceDateMilliseconds !== previousApcDate) {
+            console.info('sinceDateMilliseconds !== apcDateMilliseconds');
             const currentAssortmentItemIds = this.getBaselineItemIds(assortmentBaseline);
-            let scannedCount = 0;
-            let addedCount = 0;
             const deleteIds = (apcDeletes.length === 0)
                 ? []
                 : apcDeletes.map(item => item?.itemId);
@@ -369,20 +376,19 @@ class BaseProcessPublishAssortment {
                         ? apcHistory[i - 1]
                         : undefined;
                     const deleteChanges = await this.buildDeleteChanges(workingAPC, previousApc);
+                    console.info('checking deleteChanges');
                     for (const dItem of deleteChanges) {
                         const dItemId = dItem?.itemId;
-                        scannedCount++;
+                        console.info(dItemId);
                         if (!currentAssortmentItemIds.includes(dItemId) && !deleteIds.includes(dItemId)) {
-                            addedCount++;
+                            console.info('adding');
                             deleteIds.push(dItemId);
                             apcDeletes.push(dItem);
                         }
                     }
                 }
             }
-            console.info('getDeleteChanges(): baselineItems: ' + currentAssortmentItemIds.length
-                + ', scanned ' + scannedCount + ' historical delete changes, added ' + addedCount
-                + ', deleteIds: ' + deleteIds.length);
+            console.info('getDeleteChanges()-currentAssortmentItemIds: ' + currentAssortmentItemIds);
             console.info('getDeleteChanges()-deleteIds: ' + deleteIds);
         }
         return apcDeletes;
@@ -396,6 +402,7 @@ class BaseProcessPublishAssortment {
         return itemIds;
     }
     async downloadDeleteChanges(apc) {
+        console.info('deleteDataDownloadLink-start');
         try {
             const link = apc?.deleteDataDownloadLink;
             const response = await fetch(link);
@@ -403,17 +410,19 @@ class BaseProcessPublishAssortment {
             return data;
         }
         catch (e) {
-            console.error('Error deleteDataDownloadLink: ' + e.message);
+            console.log('Error deleteDataDownloadLink: ' + e.message);
         }
         return undefined;
     }
     async buildDeleteChanges(apc, previousApc) {
+        console.info('buildDeleteChanges()');
         if (apc?.deleteDataDownloadLink) {
             const deleteChanges = await this.downloadDeleteChanges(apc);
             if (deleteChanges) {
                 return deleteChanges;
             }
         }
+        console.info('pulling down full APC');
         apc = await this.downloadAssortmentPublishChange(apc?.assortmentId, apc?.id);
         if (apc?.deleteDataDownloadLink) {
             const deleteChanges = await this.downloadDeleteChanges(apc);
@@ -425,18 +434,19 @@ class BaseProcessPublishAssortment {
             throw new Error(BaseProcessPublishAssortment.NOT_ABLE_TO_PROCESS_DELETE_CHANGES);
         }
         let previousBaseline;
+        console.info('check previousApc');
         if (previousApc?.assortmentBaselineDownloadLink) {
             previousBaseline = await this.downloadAssortmentBaseline(previousApc);
         }
         else {
+            console.info('previousApc pulling down full APC');
             previousApc = await this.downloadAssortmentPublishChange(previousApc?.assortmentId, previousApc?.id);
             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
         const deleteArray = previousBaseline?.assortmentItems.filter(aItem => deleteIds.includes(aItem?.itemId));
-        console.info('buildDeleteChanges(): no deleteDataDownloadLink on apc ' + apc?.id
-            + '; rebuilding ' + deleteArray.length + ' deletes from previous baseline');
+        console.info('deleteArray.length: ' + deleteArray.length);
         return deleteArray;
     }
     async getItemFederatedIds( /*itemIds*/) {
@@ -474,6 +484,7 @@ class BaseProcessPublishAssortment {
         return deleteItems;
     }
     getReleasedForDevelopmentItemAndFamilyIds(fullChange, deleteChanges) {
+        console.info('getReleasedForDevelopmentItemAndFamilyIds');
         const releasedForDevelopmentItemIds = [];
         const itemFamilySet = new Set();
         const assortmentItemsArray = fullChange?.assortmentItems;
@@ -484,13 +495,18 @@ class BaseProcessPublishAssortment {
             const itemFamilyId = item?.itemFamilyId;
             if (meetsCriteria) {
                 releasedForDevelopmentItemIds.push(itemId);
-                if (!itemFamilySet.has(itemFamilyId) && itemId !== itemFamilyId) {
-                    const familyItem = item?.itemFamily;
-                    const familyItemMeetsCriteria = this.meetsCriteria({ item: familyItem });
-                    if (familyItemMeetsCriteria) {
+            }
+            // ---- 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)) {
                         releasedForDevelopmentItemIds.push(itemFamilyId);
                     }
-                    itemFamilySet.add(itemFamilyId);
+                    itemFamilySet.add(itemFamilyId); // only mark resolved when we had data to judge
                 }
             }
         }
@@ -525,6 +541,7 @@ class BaseProcessPublishAssortment {
         return !!lifecycleStage && !this.config.itemPreDevelopmentLifecycleStages.includes(lifecycleStage);
     }
     async processPublish(pcd, changeDetail, fullChange, deleteChanges) {
+        console.info('processPublish-start');
         const event = {
             assortmentId: pcd.assortmentId,
             assortmentPublishChangeId: pcd.assortmentPublishChangeId
@@ -708,6 +725,7 @@ class BaseProcessPublishAssortment {
             + '-' + d.getUTCMilliseconds();
     }
     getItemFamilyChanges(pcd, changeDetail, assortmentItemFullChangeMap, assortmentItemDeleteMap) {
+        console.info('getItemFamilyChanges-start');
         const itemFamilyChanges = new Map();
         const { adds, deletes, updates, familyItemsRemoved } = changeDetail;
         const addIds = adds.map(item => item.id);
@@ -717,7 +735,7 @@ class BaseProcessPublishAssortment {
         for (const [itemId, aItem] of assortmentItemFullChangeMap) {
             const projectItem = aItem?.projectItem;
             const itemFamilyId = aItem?.item?.itemFamilyId;
-            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId) || !pcd.releasedForDevelopmentItemIds.includes(itemId)) {
+            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId)) {
                 continue;
             }
             const ifc = itemFamilyChanges.get(itemFamilyId) || new item_family_changes_1.ItemFamilyChanges(itemFamilyId, pcd.sinceDate);
@@ -729,6 +747,20 @@ 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;
@@ -744,6 +776,12 @@ 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;
                 }
@@ -772,7 +810,8 @@ class BaseProcessPublishAssortment {
             }
             else {
                 if (!item) {
-                    console.error('Failed to find deleted item entity: itemFamilyId: ' + itemFamilyId + ', itemId: ' + itemId);
+                    console.error('Failed to find deleted item entity');
+                    console.error(' itemFamilyId: ' + itemFamilyId + ' -itemId: ' + itemId);
                     continue;
                 }
             }
@@ -809,6 +848,7 @@ class BaseProcessPublishAssortment {
         return itemFamilyChanges;
     }
     async getEventsForPublishChangeData(publishChangeData) {
+        console.info('getEventsForPublishChangeData-start');
         const seasonalPayloads = [];
         for (const itemFamilyChange of publishChangeData.itemFamilyChanges.values()) {
             const events = await this.getEventsForItemFamilyChanges(itemFamilyChange, publishChangeData.assortmentId, publishChangeData.seasonFed, publishChangeData.itemToFederatedIdMapping);
@@ -816,8 +856,6 @@ class BaseProcessPublishAssortment {
                 seasonalPayloads.push(...events);
             }
         }
-        console.info('getEventsForPublishChangeData: itemFamilies: '
-            + publishChangeData.itemFamilyChanges.size + ', payloads: ' + seasonalPayloads.length);
         return seasonalPayloads;
     }
     /**Returns the events for a given ItemFamilyChanges object
@@ -837,12 +875,15 @@ class BaseProcessPublishAssortment {
      * @returns
      */
     async getEventsForItemFamilyChanges(itemFamilyChanges, assortmentId, seasonFed, itemToFederatedIdMapping) {
+        console.info('getEventsForItemFamilyChanges()');
         const events = [];
         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, no other changes on familyAssortmentItem boolean
         const familyAssortmentItem = itemFamilyChanges.familyAdd || itemFamilyChanges.familyUpdate || itemFamilyChanges.familyDelete
             || itemFamilyChanges.colorAdds.length > 0 || itemFamilyChanges.colorUpdates.length > 0 || itemFamilyChanges.colorDeletes.length > 0
+            || itemFamilyChanges.colorSuppressed.length > 0
             || (projectItem && this.updatedSinceDate(projectItem, itemFamilyChanges.sinceDate));
         //familyItemRemoved is used when adding the first option to an assortment
         //and will have updates for the family item.

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

@@ -1337,6 +1337,151 @@ 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
+    });
+});
 describe('getProjectItem', () => {
     const config = {
         identifierAtts: {
@@ -1500,6 +1645,70 @@ 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: {

package/lib/util/data-converter.js

@@ -175,12 +175,18 @@ class DataConverter {
     }
     async getObjectReferenceValue(prop, newData, inflateObjRef = false) {
         const slug = prop['slug'];
+        if (app_framework_1.Logger.isDebugOn()) {
+            console.debug('getObjectReferenceValue-prop: ' + slug);
+        }
         let value = newData[slug];
         const entityType = prop['referencedTypeRootSlug'];
         const entityId = newData[slug + 'Id'];
         if ((!value || typeof value === 'string') && inflateObjRef) {
             if (entityId) {
                 if (this.objRefCache[entityId]) {
+                    if (app_framework_1.Logger.isDebugOn()) {
+                        console.debug('cache hit: ' + entityId);
+                    }
                     return this.objRefCache[entityId];
                 }
                 const criteria = {
@@ -426,6 +432,9 @@ class DataConverter {
             return "";
         }
         if (this.objRefCache[ctx.cacheKey]) {
+            if (app_framework_1.Logger.isDebugOn()) {
+                console.debug(`object reference cache hit: ${ctx.cacheKey}`);
+            }
             return this.objRefCache[ctx.cacheKey];
         }
         const objectReferenceId = ctx.useIdentityService
@@ -625,6 +634,9 @@ class DataConverter {
         }
         let cacheUser = DataConverter.getFromStaticCache(nd.email);
         if (cacheUser) {
+            if (app_framework_1.Logger.isDebugOn()) {
+                console.debug('user cache hit: ' + nd.email);
+            }
             await this.processGroupMemberCheck(prop, nd.email);
             return cacheUser;
         }
@@ -694,12 +706,18 @@ class DataConverter {
      */
     async getUserListValue(prop, newData) {
         const slug = prop['slug'];
+        if (app_framework_1.Logger.isDebugOn()) {
+            console.debug('getUserListValue-prop: ' + slug);
+        }
         const entityId = newData[slug + 'Id'];
         if (!entityId) {
             return {};
         }
         const cacheUser = DataConverter.getFromStaticCache(entityId);
         if (cacheUser) {
+            if (app_framework_1.Logger.isDebugOn()) {
+                console.debug('user cache hit: ' + entityId);
+            }
             return Object.assign({}, cacheUser);
         }
         const user = await this.getUserById(entityId);

package/package.json

@@ -1,66 +1,66 @@
-{
-  "name": "@contrail/flexplm",
-  "version": "1.7.4-alpha.2fb7e1f",
-  "description": "Library used for integration with flexplm.",
-  "main": "lib/index.js",
-  "types": "lib/index.d.ts",
-  "bin": {
-    "flexplm-mapping": "lib/cli/index.js"
-  },
-  "files": [
-    "lib/**/*",
-    "scripts/copy-template.js"
-  ],
-  "scripts": {
-    "build": "tsc; node scripts/copy-template.js",
-    "build:win": "tsc && node scripts/copy-template.js",
-    "format": "prettier --write \"src/**/*.ts\" \"src/**/*.js\"",
-    "lint": "tslint -p tsconfig.json",
-    "test": "jest",
-    "test-watch": "jest --watch",
-    "test-debug": "jest --runInBand",
-    "test-coverage": "jest --coverage"
-  },
-  "keywords": [],
-  "author": "VibeIQ",
-  "license": "ISC",
-  "devDependencies": {
-    "@types/jest": "^29.5.2",
-    "jest": "^29.5.0",
-    "prettier": "^1.19.1",
-    "ts-jest": "^29.1.1",
-    "tslint": "^5.11.0",
-    "tslint-config-prettier": "^1.18.0",
-    "typescript": "^4.0.0"
-  },
-  "peerDependencies": {
-    "typescript": ">=4.0.0"
-  },
-  "peerDependenciesMeta": {
-    "typescript": {
-      "optional": true
-    }
-  },
-  "jest": {
-    "moduleFileExtensions": [
-      "js",
-      "json",
-      "ts"
-    ],
-    "rootDir": "src",
-    "testRegex": ".spec.ts$",
-    "transform": {
-      "^.+\\.(t|j)s$": "ts-jest"
-    },
-    "coverageDirectory": "../coverage",
-    "testEnvironment": "node"
-  },
-  "dependencies": {
-    "@contrail/app-framework": "^1.4.3",
-    "@contrail/sdk": "^1.5.10",
-    "@contrail/transform-data": "^1.3.2",
-    "@contrail/util": "^1.3.1",
-    "axios": "^1.4.0",
-    "p-limit": "^3.1.0"
-  }
-}
+{
+  "name": "@contrail/flexplm",
+  "version": "1.7.4-alpha.458d29b",
+  "description": "Library used for integration with flexplm.",
+  "main": "lib/index.js",
+  "types": "lib/index.d.ts",
+  "bin": {
+    "flexplm-mapping": "lib/cli/index.js"
+  },
+  "files": [
+    "lib/**/*",
+    "scripts/copy-template.js"
+  ],
+  "scripts": {
+    "build": "tsc; node scripts/copy-template.js",
+    "build:win": "tsc && node scripts/copy-template.js",
+    "format": "prettier --write \"src/**/*.ts\" \"src/**/*.js\"",
+    "lint": "tslint -p tsconfig.json",
+    "test": "jest",
+    "test-watch": "jest --watch",
+    "test-debug": "jest --runInBand",
+    "test-coverage": "jest --coverage"
+  },
+  "keywords": [],
+  "author": "VibeIQ",
+  "license": "ISC",
+  "devDependencies": {
+    "@types/jest": "^29.5.2",
+    "jest": "^29.5.0",
+    "prettier": "^1.19.1",
+    "ts-jest": "^29.1.1",
+    "tslint": "^5.11.0",
+    "tslint-config-prettier": "^1.18.0",
+    "typescript": "^4.0.0"
+  },
+  "peerDependencies": {
+    "typescript": ">=4.0.0"
+  },
+  "peerDependenciesMeta": {
+    "typescript": {
+      "optional": true
+    }
+  },
+  "jest": {
+    "moduleFileExtensions": [
+      "js",
+      "json",
+      "ts"
+    ],
+    "rootDir": "src",
+    "testRegex": ".spec.ts$",
+    "transform": {
+      "^.+\\.(t|j)s$": "ts-jest"
+    },
+    "coverageDirectory": "../coverage",
+    "testEnvironment": "node"
+  },
+  "dependencies": {
+    "@contrail/app-framework": "^1.4.3",
+    "@contrail/sdk": "^1.5.10",
+    "@contrail/transform-data": "^1.3.2",
+    "@contrail/util": "^1.3.1",
+    "axios": "^1.4.0",
+    "p-limit": "^3.1.0"
+  }
+}