@contrail/flexplm 1.7.3-alpha.2a95cdd → 1.7.3-alpha.2d7b3be

package/lib/interfaces/interfaces.d.ts

@@ -14,6 +14,13 @@ export interface FCConfig {
     urlContext: string;
     vibeEventEndpoint: string;
     csrfEndpoint: string;
+    useDistinctRestEndPointForImages?: boolean;
+    /** Opt-in. When true, an item family that itself passes meetsCriteria() is published to the
+     *  season even when every one of its options is still in a pre-development lifecycle stage. */
+    publishReleasedFamilyWithPreDevelopmentOptions?: boolean;
+    /** Only meaningful when publishReleasedFamilyWithPreDevelopmentOptions is true. When true, such a
+     *  family is republished on EVERY publish, not only when one of its options actually changed. */
+    alwaysPublishReleasedFamilyWithPreDevelopmentOptions?: boolean;
     itemPreDevelopmentLifecycleStages: string[];
     identifierAtts?: {
         [key: string]: string[];

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

@@ -12,6 +12,12 @@ export declare class ItemFamilyChanges {
     colorDeletes: string[];
     colorUpdates: string[];
     colorUnchanged: string[];
+    /** Options that changed but are still pre-development. They do NOT produce
+     *  LCSSKUSeasonLink events - they only signal that the family needs republishing. */
+    preDevelopmentOptionChanges: string[];
+    /** True when at least one option row was tracked at family level only, because the option is
+     *  pre-development while the family itself is released. */
+    hasPreDevelopmentOnlyOptions: boolean;
     assortmentItemFullChangeMap: Map<string, any>;
     assortmentItemDeleteMap: Map<string, any>;
     itemToFederatedIdMapping: Map<string, string>;

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

@@ -14,6 +14,12 @@ class ItemFamilyChanges {
         this.colorDeletes = [];
         this.colorUpdates = [];
         this.colorUnchanged = [];
+        /** Options that changed but are still pre-development. They do NOT produce
+         *  LCSSKUSeasonLink events - they only signal that the family needs republishing. */
+        this.preDevelopmentOptionChanges = [];
+        /** True when at least one option row was tracked at family level only, because the option is
+         *  pre-development while the family itself is released. */
+        this.hasPreDevelopmentOnlyOptions = false;
         this.assortmentItemFullChangeMap = new Map();
         this.assortmentItemDeleteMap = new Map();
         this.itemToFederatedIdMapping = new Map();
@@ -32,6 +38,8 @@ class ItemFamilyChanges {
         s += ' colorDeletes: [' + this.colorDeletes + ']\n';
         s += ' colorUpdates: [' + this.colorUpdates + ']\n';
         s += ' colorUnchanged: [' + this.colorUnchanged + ']\n';
+        s += ' preDevelopmentOptionChanges: [' + this.preDevelopmentOptionChanges + ']\n';
+        s += ' hasPreDevelopmentOnlyOptions: ' + this.hasPreDevelopmentOnlyOptions + '\n';
         s += 'assortmentItemFullChangeMap:\n';
         s += 'size: ' + this.assortmentItemFullChangeMap.size + '\n[';
         for (const key of this.assortmentItemFullChangeMap.keys()) {

package/lib/interfaces/mapping-file.d.ts

@@ -776,47 +776,6 @@ interface MappingFileBase {
  *     },
  *   },
  * };
- *
- * @example
- * // Multi-Entry and Composite FlexPLM properties.
- * //
- * // FlexPLM sends both shapes as a single string whose values are joined by
- * // the separator '|~*~|' (a trailing separator follows the last value):
- * //   Cuttable Width (multi-entry): '1MM Horizontal|~*~|2MM Vertical|~*~|'
- * //   Content        (composite):   '50.0% MetalProcessing|~*~|50.0% Cable|~*~|'
- * //
- * // VibeIQ stores these raw '|~*~|'-joined strings as-is, so no split/join is
- * // needed — a plain REKEY carries the value through unchanged in both
- * // directions. If you later need to store an array or a cleaned display
- * // value instead, add a VALUE_TRANSFORM task to `transformOrder` and a
- * // matching `valueTransform` entry (see {@link DirectionalSection.valueTransform})
- * // that splits on / joins with '|~*~|'.
- * export const mapping: MappingFile = {
- *   orgInfo: { appIdentifier: '@vibeiq/flexplm-connector', orgName: 'acme' },
- *   typeConversion: { vibe2flex: {}, flex2vibe: {} },
- *   LCSProduct: {
- *     vibe2flex: {
- *       transformOrder: [
- *         { processor: 'REKEY', rekeyDelete: true, rekeyKeepEmptyValues: true, rekeyTransformersKey: 'rekey' },
- *       ],
- *       // flexField: 'vibeSlug' — raw '|~*~|' strings pass straight through.
- *       rekey: {
- *         cuttableWidth:   'cuttableWidth',    // multi-entry
- *         materialContent: 'materialContent',  // composite (Flex 'Content')
- *       },
- *     },
- *     flex2vibe: {
- *       transformOrder: [
- *         { processor: 'REKEY', rekeyDelete: true, rekeyKeepEmptyValues: true, rekeyTransformersKey: 'rekey' },
- *       ],
- *       // vibeSlug: 'flexField'
- *       rekey: {
- *         cuttableWidth:   'cuttableWidth',
- *         materialContent: 'materialContent',
- *       },
- *     },
- *   },
- * };
  */
 export type MappingFile = MappingFileBase & {
     [mapKey: string]: MappingSection | MappingFileBase[keyof MappingFileBase];

package/lib/publish/base-process-publish-assortment.d.ts

@@ -62,6 +62,11 @@ export declare class BaseProcessPublishAssortment {
     getFullChangeAssortmentMap(fullChange: any): Map<string, object>;
     getDeleteChangesAssortmentMap(deleteChanges: any[]): Map<string, object>;
     getReleasedForDevelopmentItemAndFamilyIds(fullChange: any, deleteChanges: any): string[];
+    /** Adds item families that pass meetsCriteria() but whose option rows are all pre-development.
+     *  Only ever ADDS family ids - never option ids, and never removes anything. */
+    protected addReleasedFamiliesOfPreDevelopmentOptions(aItems: any[], releasedForDevelopmentItemIds: string[]): void;
+    protected publishReleasedFamilyWithPreDevOptions(): boolean;
+    protected alwaysPublishReleasedFamilyWithPreDevOptions(): boolean;
     meetsCriteria(aItem: any): boolean;
     processPublish(pcd: PublishChangeData, changeDetail: any, fullChange: any, deleteChanges: any): Promise<{
         results: {

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

@@ -11,6 +11,7 @@ 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 config_defaults_1 = require("../util/config-defaults");
 class BaseProcessPublishAssortment {
     constructor(_config, _dc, _mapFileUtil) {
         this.TTL = 30 * 24 * 60 * 60 * 1000; // 30 days
@@ -527,9 +528,43 @@ class BaseProcessPublishAssortment {
                 }
             }
         }
+        if (this.publishReleasedFamilyWithPreDevOptions()) {
+            this.addReleasedFamiliesOfPreDevelopmentOptions(assortmentItemsArray, releasedForDevelopmentItemIds);
+            this.addReleasedFamiliesOfPreDevelopmentOptions(deleteChanges, releasedForDevelopmentItemIds);
+        }
         console.info('releasedForDevelopmentItemIds: ' + releasedForDevelopmentItemIds);
         return releasedForDevelopmentItemIds;
     }
+    /** Adds item families that pass meetsCriteria() but whose option rows are all pre-development.
+     *  Only ever ADDS family ids - never option ids, and never removes anything. */
+    addReleasedFamiliesOfPreDevelopmentOptions(aItems, releasedForDevelopmentItemIds) {
+        for (const aItem of (aItems || [])) {
+            const item = aItem?.item;
+            const itemId = item?.id;
+            const itemFamilyId = item?.itemFamilyId;
+            if (!itemFamilyId || itemId === itemFamilyId) {
+                continue;
+            }
+            if (releasedForDevelopmentItemIds.includes(itemFamilyId)) {
+                continue;
+            }
+            if (this.meetsCriteria(aItem)) {
+                continue;
+            }
+            const familyItem = item?.itemFamily;
+            if (familyItem && this.meetsCriteria({ item: familyItem })) {
+                console.info('adding released family with pre-development options: ' + itemFamilyId);
+                releasedForDevelopmentItemIds.push(itemFamilyId);
+            }
+        }
+    }
+    publishReleasedFamilyWithPreDevOptions() {
+        return config_defaults_1.ConfigDefaults.isPropertyTrue(this.config?.publishReleasedFamilyWithPreDevelopmentOptions);
+    }
+    alwaysPublishReleasedFamilyWithPreDevOptions() {
+        return this.publishReleasedFamilyWithPreDevOptions()
+            && config_defaults_1.ConfigDefaults.isPropertyTrue(this.config?.alwaysPublishReleasedFamilyWithPreDevelopmentOptions);
+    }
     meetsCriteria(aItem) {
         const item = aItem?.item;
         const lifecycleStage = item?.lifecycleStage;
@@ -730,7 +765,12 @@ 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)) {
+            const familyInScope = pcd.releasedForDevelopmentItemIds.includes(itemFamilyId);
+            const itemInScope = pcd.releasedForDevelopmentItemIds.includes(itemId);
+            // Released family + pre-development option: track at family level only (no SKU events).
+            const familyOnly = !itemInScope && familyInScope && itemId !== itemFamilyId
+                && this.publishReleasedFamilyWithPreDevOptions();
+            if (!familyInScope || (!itemInScope && !familyOnly)) {
                 continue;
             }
             const ifc = itemFamilyChanges.get(itemFamilyId) || new item_family_changes_1.ItemFamilyChanges(itemFamilyId, pcd.sinceDate);
@@ -738,10 +778,29 @@ class BaseProcessPublishAssortment {
                 ifc.itemFamilyObject = itemId === itemFamilyId ? aItem?.item : aItem?.item?.itemFamily;
                 itemFamilyChanges.set(itemFamilyId, ifc);
             }
+            else if (itemId === itemFamilyId && this.publishReleasedFamilyWithPreDevOptions()) {
+                // A pre-development option row may have created this bucket with the partial
+                // item.itemFamily; the family's own row carries the full item, so prefer it.
+                ifc.itemFamilyObject = aItem?.item;
+            }
             ifc.assortmentItemFullChangeMap.set(itemId, aItem);
             if (pcd.itemToFederatedIdMapping.has(itemId)) {
                 ifc.itemToFederatedIdMapping.set(itemId, pcd.itemToFederatedIdMapping.get(itemId));
             }
+            if (familyOnly) {
+                ifc.hasPreDevelopmentOnlyOptions = true;
+                if (addIds.includes(itemId) || deleteIds.includes(itemId) || updateIds.includes(itemId)
+                    || (projectItem && new Date(projectItem.updatedOn) > pcd.sinceDate)) {
+                    ifc.preDevelopmentOptionChanges.push(itemId);
+                }
+                else {
+                    ifc.colorUnchanged.push(itemId);
+                }
+                if (projectItem && !projectItem?.roles && aItem?.item?.roles) {
+                    projectItem.roles = aItem?.item?.roles;
+                }
+                continue;
+            }
             if (itemId === itemFamilyId) {
                 if (addIds.includes(itemId))
                     ifc.familyAdd = true;
@@ -780,7 +839,11 @@ class BaseProcessPublishAssortment {
         for (const [itemId, delEntity] of assortmentItemDeleteMap) {
             const itemFamilyId = delEntity?.item?.itemFamilyId;
             const item = delEntity?.item;
-            if (!pcd.releasedForDevelopmentItemIds.includes(itemFamilyId) || !pcd.releasedForDevelopmentItemIds.includes(itemId)) {
+            const familyInScope = pcd.releasedForDevelopmentItemIds.includes(itemFamilyId);
+            const itemInScope = pcd.releasedForDevelopmentItemIds.includes(itemId);
+            const familyOnly = !itemInScope && familyInScope && itemId !== itemFamilyId
+                && this.publishReleasedFamilyWithPreDevOptions();
+            if (!familyInScope || (!itemInScope && !familyOnly)) {
                 continue;
             }
             else {
@@ -810,6 +873,10 @@ class BaseProcessPublishAssortment {
             if (itemId === itemFamilyId) {
                 ifc.familyDelete = true;
             }
+            else if (familyOnly) {
+                ifc.hasPreDevelopmentOnlyOptions = true;
+                ifc.preDevelopmentOptionChanges.push(itemId);
+            }
             else {
                 ifc.colorDeletes.push(itemId);
             }
@@ -857,6 +924,8 @@ class BaseProcessPublishAssortment {
         const projectItem = this.getProjectItem(itemFamilyChanges, itemFamilyChanges.itemFamilyId);
         const familyAssortmentItem = itemFamilyChanges.familyAdd || itemFamilyChanges.familyUpdate || itemFamilyChanges.familyDelete
             || itemFamilyChanges.colorAdds.length > 0 || itemFamilyChanges.colorUpdates.length > 0 || itemFamilyChanges.colorDeletes.length > 0
+            || itemFamilyChanges.preDevelopmentOptionChanges.length > 0
+            || (this.alwaysPublishReleasedFamilyWithPreDevOptions() && itemFamilyChanges.hasPreDevelopmentOnlyOptions)
             || (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

@@ -1874,3 +1874,259 @@ describe('sendToFlexPLM / handleVibeIQFile / sendPublishPayloadEvent', () => {
         });
     });
 });
+describe('publishReleasedFamilyWithPreDevelopmentOptions', () => {
+    const FAMILY_ID = 'AERfdvdAt1HHEVvQ';
+    const LIME_ID = 'limeGreenItemId__';
+    const MAROON_ID = 'maroonItemId_____';
+    const SINCE = new Date('2026-09-03T16:54:20.778Z');
+    const BEFORE_SINCE = '2026-09-01T00:00:00.000Z';
+    const AFTER_SINCE = '2026-09-03T17:00:00.000Z';
+    /** The full family item, as it appears on the family's own assortment row. */
+    const fullFamilyItem = {
+        id: FAMILY_ID, itemFamilyId: FAMILY_ID, name: 'Sweatpant', itemNumber: 11,
+        lifecycleStage: 'released', roles: ['family'], richFamilyOnlyField: 'PRESENT'
+    };
+    /** The partial family item, as hydrated onto an option row. */
+    const partialFamilyItem = {
+        id: FAMILY_ID, itemFamilyId: FAMILY_ID, name: 'Sweatpant', itemNumber: 11,
+        lifecycleStage: 'released', roles: ['family']
+    };
+    const familyRow = () => ({
+        entityType: 'assortment-item',
+        id: 'ai-' + FAMILY_ID,
+        itemId: FAMILY_ID,
+        item: { ...fullFamilyItem },
+        projectItem: { id: 'proj:' + FAMILY_ID, itemId: FAMILY_ID, updatedOn: BEFORE_SINCE, roles: ['family'] }
+    });
+    const optionRow = (itemId, optionName, projectItemUpdatedOn) => ({
+        entityType: 'assortment-item',
+        id: 'ai-' + itemId,
+        itemId,
+        item: {
+            id: itemId, itemFamilyId: FAMILY_ID, name: 'Sweatpant', optionName,
+            optionGroup: 'color', lifecycleStage: 'concept', roles: ['color', 'option'],
+            itemFamily: { ...partialFamilyItem }
+        },
+        projectItem: { id: 'proj:' + itemId, itemId, updatedOn: projectItemUpdatedOn, roles: ['color', 'option'] },
+        familyProjectItem: { id: 'proj:' + FAMILY_ID, itemId: FAMILY_ID, updatedOn: BEFORE_SINCE, roles: ['family'] }
+    });
+    const emptyDetail = { adds: [], deletes: [], updates: [], familyItemsRemoved: [] };
+    const baseConfig = { identifierAtts: { LCSProduct: ['itemNumber'] }, itemPreDevelopmentLifecycleStages: ['concept'] };
+    const makePpa2 = (flagOn, alwaysOn) => {
+        const cfg = {
+            ...baseConfig,
+            publishReleasedFamilyWithPreDevelopmentOptions: flagOn,
+            alwaysPublishReleasedFamilyWithPreDevelopmentOptions: alwaysOn
+        };
+        const mfu = new transform_data_1.MapFileUtil(new sdk_1.Entities());
+        return new base_process_publish_assortment_1.BaseProcessPublishAssortment(cfg, new data_converter_1.DataConverter(cfg, mfu), mfu);
+    };
+    const makePpa = (flagOn) => makePpa2(flagOn, false);
+    const makePcd = (releasedIds) => {
+        const sf = { entityReference: 'assortment:11EZML5dMn3tREuT', objectClass: 'LCSSeason' };
+        const pcd = new publish_change_data_1.PublishChangeData('11EZML5dMn3tREuT', sf, 'apc1', SINCE);
+        pcd.itemToFederatedIdMapping = new Map();
+        pcd.releasedForDevelopmentItemIds = releasedIds;
+        return pcd;
+    };
+    const eventSeasonFed = { entityReference: 'assortment:a1', objectClass: 'LCSSeason' };
+    const stubEventDeps = (ppa) => {
+        jest.spyOn(ppa, 'getAssortment').mockResolvedValue({ id: 'a1', publishToFlexPLM: true });
+        jest.spyOn(ppa, 'getSeasonalData').mockResolvedValue({});
+        jest.spyOn(map_utils_1.MapUtil, 'applyTransformMap').mockImplementation(async (...args) => args[2]);
+        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'getIdentifierProperties').mockResolvedValue(['itemNumber']);
+        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'getInformationalProperties').mockResolvedValue([]);
+        jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'isOutboundCreatableFromEntity').mockResolvedValue(true);
+    };
+    afterEach(() => { jest.restoreAllMocks(); });
+    /////////////////////////////////////////////////////////////////////////////
+    // getReleasedForDevelopmentItemAndFamilyIds
+    /////////////////////////////////////////////////////////////////////////////
+    it('REGRESSION: flag off - released family with all-concept options stays out of scope', () => {
+        const fullChange = { assortmentItems: [optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE),
+                optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE)] };
+        expect(makePpa(false).getReleasedForDevelopmentItemAndFamilyIds(fullChange, [])).toEqual([]);
+    });
+    it('flag on - released family enters scope; concept options do NOT', () => {
+        const fullChange = { assortmentItems: [optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE),
+                optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE)] };
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds(fullChange, [])).toEqual([FAMILY_ID]);
+    });
+    it('flag on - concept family with concept options still stays out of scope', () => {
+        const row = optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE);
+        row.item.itemFamily = { ...partialFamilyItem, lifecycleStage: 'concept' };
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds({ assortmentItems: [row] }, [])).toEqual([]);
+    });
+    it('flag on - family reached only through a delete record', () => {
+        const deleted = optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE);
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds({ assortmentItems: [] }, [deleted]))
+            .toEqual([FAMILY_ID]);
+    });
+    it('flag on - two concept options of the same family add the family only once', () => {
+        const fullChange = { assortmentItems: [optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE),
+                optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE)] };
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds(fullChange, [])).toHaveLength(1);
+    });
+    it('flag on - option row with no hydrated itemFamily is skipped without throwing', () => {
+        const bare = optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE);
+        delete bare.item.itemFamily;
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds({ assortmentItems: [bare] }, [])).toEqual([]);
+    });
+    it('released family WITH its own assortment row is in scope with or without the flag', () => {
+        const rows = { assortmentItems: [familyRow(), optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE)] };
+        expect(makePpa(false).getReleasedForDevelopmentItemAndFamilyIds(rows, [])).toEqual([FAMILY_ID]);
+        expect(makePpa(true).getReleasedForDevelopmentItemAndFamilyIds(rows, [])).toEqual([FAMILY_ID]);
+    });
+    it('string flag values from app config are honoured', () => {
+        const fullChange = { assortmentItems: [optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE)] };
+        expect(makePpa('true').getReleasedForDevelopmentItemAndFamilyIds(fullChange, [])).toEqual([FAMILY_ID]);
+        expect(makePpa('false').getReleasedForDevelopmentItemAndFamilyIds(fullChange, [])).toEqual([]);
+    });
+    /////////////////////////////////////////////////////////////////////////////
+    // getItemFamilyChanges
+    /////////////////////////////////////////////////////////////////////////////
+    it('ACTION 2: deleting a concept option marks the released family for republish', () => {
+        const ppa = makePpa(true);
+        const remaining = optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE);
+        const deleted = optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE);
+        const pcd = makePcd([FAMILY_ID]);
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, ppa.getFullChangeAssortmentMap({ assortmentItems: [remaining] }), ppa.getDeleteChangesAssortmentMap([deleted]));
+        expect(pcd.itemFamilyChanges.size).toEqual(1);
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.preDevelopmentOptionChanges).toEqual([MAROON_ID]);
+        expect(ifc.hasPreDevelopmentOnlyOptions).toBe(true);
+        expect(ifc.colorDeletes).toEqual([]);
+        expect(ifc.colorAdds).toEqual([]);
+        expect(ifc.itemFamilyObject.id).toEqual(FAMILY_ID);
+    });
+    it('adding a concept option marks the family, not a colorAdd', () => {
+        const ppa = makePpa(true);
+        const added = optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE);
+        const pcd = makePcd([FAMILY_ID]);
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, { ...emptyDetail, adds: [{ id: MAROON_ID }] }, ppa.getFullChangeAssortmentMap({ assortmentItems: [added] }), new Map());
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.preDevelopmentOptionChanges).toEqual([MAROON_ID]);
+        expect(ifc.colorAdds).toEqual([]);
+    });
+    it('ACTION 1: nothing changed - family bucket exists but signals no change', () => {
+        const ppa = makePpa(true);
+        const pcd = makePcd([FAMILY_ID]);
+        const rows = { assortmentItems: [optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE),
+                optionRow(MAROON_ID, 'Maroon', BEFORE_SINCE)] };
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, ppa.getFullChangeAssortmentMap(rows), new Map());
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.preDevelopmentOptionChanges).toEqual([]);
+        expect(ifc.colorUnchanged.sort()).toEqual([LIME_ID, MAROON_ID].sort());
+    });
+    it('REGRESSION: flag off - concept option rows produce no ItemFamilyChanges at all', () => {
+        const ppa = makePpa(false);
+        const pcd = makePcd([FAMILY_ID]);
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, ppa.getFullChangeAssortmentMap({ assortmentItems: [optionRow(LIME_ID, 'Lime Green', AFTER_SINCE)] }), new Map());
+        expect(pcd.itemFamilyChanges.size).toEqual(0);
+    });
+    it('REGRESSION: a released option is still a colorAdd, never a preDevelopmentOptionChange', () => {
+        const ppa = makePpa(true);
+        const released = optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE);
+        released.item.lifecycleStage = 'development';
+        const pcd = makePcd([FAMILY_ID, LIME_ID]);
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, { ...emptyDetail, adds: [{ id: LIME_ID }] }, ppa.getFullChangeAssortmentMap({ assortmentItems: [released] }), new Map());
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.colorAdds).toEqual([LIME_ID]);
+        expect(ifc.preDevelopmentOptionChanges).toEqual([]);
+        expect(ifc.hasPreDevelopmentOnlyOptions).toBe(false);
+    });
+    it('itemFamilyObject prefers the full family item when the OPTION row is iterated first', () => {
+        const ppa = makePpa(true);
+        const pcd = makePcd([FAMILY_ID]);
+        const map = new Map();
+        map.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        map.set(FAMILY_ID, familyRow());
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, map, new Map());
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.itemFamilyObject.richFamilyOnlyField).toEqual('PRESENT');
+    });
+    it('itemFamilyObject is the full family item when the FAMILY row is iterated first', () => {
+        const ppa = makePpa(true);
+        const pcd = makePcd([FAMILY_ID]);
+        const map = new Map();
+        map.set(FAMILY_ID, familyRow());
+        map.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, map, new Map());
+        const ifc = pcd.itemFamilyChanges.get(FAMILY_ID);
+        expect(ifc.itemFamilyObject.richFamilyOnlyField).toEqual('PRESENT');
+    });
+    it('delete record with a missing item entity is skipped safely', () => {
+        const ppa = makePpa(true);
+        const pcd = makePcd([FAMILY_ID]);
+        const delMap = new Map();
+        delMap.set('GHOST', { itemId: 'GHOST', item: undefined });
+        pcd.itemFamilyChanges = ppa.getItemFamilyChanges(pcd, emptyDetail, new Map(), delMap);
+        expect(pcd.itemFamilyChanges.size).toEqual(0);
+    });
+    /////////////////////////////////////////////////////////////////////////////
+    // getEventsForItemFamilyChanges
+    /////////////////////////////////////////////////////////////////////////////
+    it('emits exactly one LCSProductSeasonLink and zero LCSSKUSeasonLink', async () => {
+        const ppa = makePpa(true);
+        const ifc = new item_family_changes_1.ItemFamilyChanges(FAMILY_ID, SINCE);
+        ifc.itemFamilyObject = fullFamilyItem;
+        ifc.hasPreDevelopmentOnlyOptions = true;
+        ifc.preDevelopmentOptionChanges.push(MAROON_ID);
+        ifc.assortmentItemFullChangeMap.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        stubEventDeps(ppa);
+        const events = await ppa.getEventsForItemFamilyChanges(ifc, 'a1', eventSeasonFed, new Map());
+        expect(events).toHaveLength(1);
+        expect(events[0].objectClass).toBe('LCSProductSeasonLink');
+        expect(events[0].eventType).toBe('UPSERT_ON_SEASON');
+        expect(events[0].entityReference).toBe('item:' + FAMILY_ID);
+        expect(events[0].LCSProduct['itemNumber']).toBe(11);
+    });
+    it('REGRESSION: an ItemFamilyChanges with no changes still emits nothing', async () => {
+        const ppa = makePpa(true);
+        const ifc = new item_family_changes_1.ItemFamilyChanges(FAMILY_ID, SINCE);
+        ifc.itemFamilyObject = fullFamilyItem;
+        ifc.colorUnchanged.push(LIME_ID, MAROON_ID);
+        ifc.assortmentItemFullChangeMap.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        stubEventDeps(ppa);
+        expect(await ppa.getEventsForItemFamilyChanges(ifc, 'a1', eventSeasonFed, new Map())).toHaveLength(0);
+    });
+    /////////////////////////////////////////////////////////////////////////////
+    // alwaysPublishReleasedFamilyWithPreDevelopmentOptions
+    /////////////////////////////////////////////////////////////////////////////
+    const unchangedFamilyIfc = () => {
+        const ifc = new item_family_changes_1.ItemFamilyChanges(FAMILY_ID, SINCE);
+        ifc.itemFamilyObject = fullFamilyItem;
+        ifc.hasPreDevelopmentOnlyOptions = true;
+        ifc.colorUnchanged.push(LIME_ID, MAROON_ID);
+        ifc.assortmentItemFullChangeMap.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        return ifc;
+    };
+    it('ACTION 1 with always-flag: unchanged family still emits one LCSProductSeasonLink', async () => {
+        const ppa = makePpa2(true, true);
+        stubEventDeps(ppa);
+        const events = await ppa.getEventsForItemFamilyChanges(unchangedFamilyIfc(), 'a1', eventSeasonFed, new Map());
+        expect(events).toHaveLength(1);
+        expect(events[0].objectClass).toBe('LCSProductSeasonLink');
+    });
+    it('always-flag off: unchanged family emits nothing', async () => {
+        const ppa = makePpa2(true, false);
+        stubEventDeps(ppa);
+        expect(await ppa.getEventsForItemFamilyChanges(unchangedFamilyIfc(), 'a1', eventSeasonFed, new Map()))
+            .toHaveLength(0);
+    });
+    it('always-flag cannot act alone when the primary flag is off', async () => {
+        const ppa = makePpa2(false, true);
+        stubEventDeps(ppa);
+        expect(await ppa.getEventsForItemFamilyChanges(unchangedFamilyIfc(), 'a1', eventSeasonFed, new Map()))
+            .toHaveLength(0);
+    });
+    it('REGRESSION: always-flag does not republish an ordinary unchanged released family', async () => {
+        const ppa = makePpa2(true, true);
+        stubEventDeps(ppa);
+        const ifc = new item_family_changes_1.ItemFamilyChanges(FAMILY_ID, SINCE);
+        ifc.itemFamilyObject = fullFamilyItem;
+        ifc.colorUnchanged.push(LIME_ID);
+        ifc.assortmentItemFullChangeMap.set(LIME_ID, optionRow(LIME_ID, 'Lime Green', BEFORE_SINCE));
+        expect(await ppa.getEventsForItemFamilyChanges(ifc, 'a1', eventSeasonFed, new Map())).toHaveLength(0);
+    });
+});

package/lib/util/config-defaults.d.ts

@@ -2,6 +2,8 @@ import { FCConfig } from '../interfaces/interfaces';
 export declare class ConfigDefaults {
     static NEED_CONFIG_VALUES: string;
     static STATIC_CONFIG_CACHE: {};
+    static PROTO_KEYS: string[];
+    static stripProtoKeys(obj: any): any;
     static setConfigDefaults(config: any): Promise<FCConfig>;
     static getDefaultConfig(): {
         urlContext: string;
@@ -17,6 +19,9 @@ export declare class ConfigDefaults {
         LCSMaterial: {
             processAsItem: boolean;
         };
+        useDistinctRestEndPointForImages: boolean;
+        publishReleasedFamilyWithPreDevelopmentOptions: boolean;
+        alwaysPublishReleasedFamilyWithPreDevelopmentOptions: boolean;
         csrfEndpoint: string;
         vibeEventEndpoint: string;
         payloadDefaultAsArray: boolean;

package/lib/util/config-defaults.js

@@ -5,12 +5,29 @@ const sdk_1 = require("@contrail/sdk");
 const util_1 = require("@contrail/util");
 const type_defaults_1 = require("./type-defaults");
 class ConfigDefaults {
+    static stripProtoKeys(obj) {
+        if (obj && typeof obj === 'object' && !(obj instanceof Date)) {
+            for (const key of Object.keys(obj)) {
+                if (ConfigDefaults.PROTO_KEYS.includes(key)) {
+                    delete obj[key];
+                }
+                else {
+                    ConfigDefaults.stripProtoKeys(obj[key]); // recurse into nested payloads
+                }
+            }
+        }
+        return obj;
+    }
     static async setConfigDefaults(config) {
-        //Validate config
+        // Validate config
         if (!config.apiHost || !config.userName || !config.password) {
             throw new Error(ConfigDefaults.NEED_CONFIG_VALUES);
         }
-        //List will be comma separated list in UI, so convert to array
+        if (config.complexConfig && typeof config.complexConfig === 'object') {
+            Object.assign(config, ConfigDefaults.stripProtoKeys(config.complexConfig));
+            delete config.complexConfig;
+        }
+        // List will be comma separated list in UI, so convert to array
         if (config?.itemPreDevelopmentLifecycleStages && !(config?.itemPreDevelopmentLifecycleStages instanceof Array)) {
             config.itemPreDevelopmentLifecycleStages = config.itemPreDevelopmentLifecycleStages.split(',');
         }
@@ -28,7 +45,7 @@ class ConfigDefaults {
         const pass = outputConfig.password;
         outputConfig.userName = () => uName;
         outputConfig.password = () => pass;
-        //Don't allow overwriting this.
+        // Don't allow overwriting this.
         outputConfig['OOBvibeEventEndpoint'] = '/rfa/vibeiq/vibeEvents';
         type_defaults_1.TypeDefaults.applyConfig(outputConfig);
         console.log('outputConfig: ' + JSON.stringify(outputConfig));
@@ -38,20 +55,23 @@ class ConfigDefaults {
         return {
             urlContext: '/Windchill',
             sendMode: {
-                ASYNC_PUBLISH_SEASON: 'vibeiqfile'
+                ASYNC_PUBLISH_SEASON: 'vibeiqfile',
             },
             itemPreDevelopmentLifecycleStages: ['concept'],
             identifierAtts: {
                 LCSProduct: ['itemNumber'],
                 LCSSeason: ['flexPLMSeasonName'],
-                LCSSKU: ['itemNumber']
+                LCSSKU: ['itemNumber'],
             },
             LCSMaterial: {
-                processAsItem: false
+                processAsItem: false,
             },
+            useDistinctRestEndPointForImages: false,
+            publishReleasedFamilyWithPreDevelopmentOptions: false,
+            alwaysPublishReleasedFamilyWithPreDevelopmentOptions: false,
             csrfEndpoint: '/servlet/rest/security/csrf',
             vibeEventEndpoint: '/rfa/vibeiq/vibeEvents',
-            payloadDefaultAsArray: true
+            payloadDefaultAsArray: true,
         };
     }
     static async getConfigFile(fileId) {
@@ -96,3 +116,4 @@ class ConfigDefaults {
 exports.ConfigDefaults = ConfigDefaults;
 ConfigDefaults.NEED_CONFIG_VALUES = 'To connect to FlexPLM all these APP values need to be set apiHost, userName, and password';
 ConfigDefaults.STATIC_CONFIG_CACHE = {};
+ConfigDefaults.PROTO_KEYS = ['__proto__', 'constructor', 'prototype'];

package/lib/util/config-defaults.spec.js

@@ -378,4 +378,65 @@ describe('all tests', () => {
             expect(Object.keys(config).length).toEqual(0);
         });
     });
+    describe('prototype pollution', () => {
+        const config = {
+            apiHost: 'http://test.com',
+            userName: 'vibeiq',
+            password: 'vibeiq'
+        };
+        it('does not allow prototype pollution via complexConfig', async () => {
+            const startConfig = Object.assign({}, config, {
+                complexConfig: JSON.parse('{"__proto__":{"polluted":true}}')
+            });
+            await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+            expect({}.polluted).toBeUndefined();
+        });
+    });
+});
+describe('publishReleasedFamilyWithPreDevelopmentOptions config', () => {
+    const config = {
+        apiHost: 'http://test.com',
+        userName: 'vibeiq',
+        password: 'vibeiq',
+        plmEnviornment: 'SB'
+    };
+    it('publishReleasedFamilyWithPreDevelopmentOptions-get default', async () => {
+        const startConfig = Object.assign({}, config);
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(fcConfig['publishReleasedFamilyWithPreDevelopmentOptions']).toBe(false);
+    });
+    it('alwaysPublishReleasedFamilyWithPreDevelopmentOptions-get default', async () => {
+        const startConfig = Object.assign({}, config);
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(fcConfig['alwaysPublishReleasedFamilyWithPreDevelopmentOptions']).toBe(false);
+    });
+    it('publishReleasedFamilyWithPreDevelopmentOptions-set boolean true', async () => {
+        const startConfig = Object.assign({}, config);
+        startConfig['publishReleasedFamilyWithPreDevelopmentOptions'] = true;
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(config_defaults_1.ConfigDefaults.isPropertyTrue(fcConfig['publishReleasedFamilyWithPreDevelopmentOptions'])).toBe(true);
+    });
+    it('publishReleasedFamilyWithPreDevelopmentOptions-set string true from app config', async () => {
+        const startConfig = Object.assign({}, config);
+        startConfig['publishReleasedFamilyWithPreDevelopmentOptions'] = 'true';
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(config_defaults_1.ConfigDefaults.isPropertyTrue(fcConfig['publishReleasedFamilyWithPreDevelopmentOptions'])).toBe(true);
+    });
+    it('publishReleasedFamilyWithPreDevelopmentOptions-string false stays off', async () => {
+        const startConfig = Object.assign({}, config);
+        startConfig['publishReleasedFamilyWithPreDevelopmentOptions'] = 'false';
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(config_defaults_1.ConfigDefaults.isPropertyTrue(fcConfig['publishReleasedFamilyWithPreDevelopmentOptions'])).toBe(false);
+    });
+    it('both flags can be delivered through complexConfig', async () => {
+        const startConfig = Object.assign({}, config, {
+            complexConfig: {
+                publishReleasedFamilyWithPreDevelopmentOptions: true,
+                alwaysPublishReleasedFamilyWithPreDevelopmentOptions: 'true'
+            }
+        });
+        const fcConfig = await config_defaults_1.ConfigDefaults.setConfigDefaults(startConfig);
+        expect(config_defaults_1.ConfigDefaults.isPropertyTrue(fcConfig['publishReleasedFamilyWithPreDevelopmentOptions'])).toBe(true);
+        expect(config_defaults_1.ConfigDefaults.isPropertyTrue(fcConfig['alwaysPublishReleasedFamilyWithPreDevelopmentOptions'])).toBe(true);
+    });
 });

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

@@ -24,6 +24,7 @@ const createConfig = (overrides) => ({
     password: () => 'pass',
     itemPreDevelopmentLifecycleStages: [],
     payloadDefaultAsArray: true,
+    useDistinctRestEndPointForImages: false,
     ...overrides,
 });
 describe('FlexPLMConnect.getRequest', () => {

package/lib/util/thumbnail-util.js

@@ -181,15 +181,34 @@ class ThumbnailUtil {
         const fileName = urlParts[urlParts.length - 1] || 'thumbnail';
         const encodedUrl = urlParts.map(part => encodeURIComponent(part)).join('/');
         const flexPLMConnect = new flexplm_connect_1.FlexPLMConnect(this.config);
-        const response = await flexPLMConnect.getRequest({
-            urlPath: encodedUrl,
-            includeUrlContext: false,
-            returnFullResponse: true,
-        });
+        console.debug('the useDistinctRestEndPointForImages is --> ' + this.config.useDistinctRestEndPointForImages);
+        let response;
+        if (this.config.useDistinctRestEndPointForImages) {
+            // Route through the connector endpoint (Basic auth + VIBEIQGROUP enforced there).
+            const imageEndpoint = '/rfa/vibeiq/image';
+            const urlPath = '/servlet/rest' + imageEndpoint + '?path=' + encodeURIComponent(thumbnailUrl);
+            response = await flexPLMConnect.getRequest({
+                urlPath,
+                includeUrlContext: true,
+                returnFullResponse: true,
+            });
+        }
+        else {
+            response = await flexPLMConnect.getRequest({
+                urlPath: encodedUrl,
+                includeUrlContext: false,
+                returnFullResponse: true,
+            });
+        }
         const fileBuffer = await response.arrayBuffer();
         const buffer = Buffer.from(fileBuffer);
         const contentTypeHeader = response.headers.get('content-type');
         const contentType = contentTypeHeader ? contentTypeHeader.split(';')[0] : 'application/octet-stream';
+        // Ensure we actually receive an image content type.
+        if (!contentType.startsWith('image/')) {
+            const message = `Expected image content from FlexPLM but received '${contentType}' for ${thumbnailUrl}`;
+            throw new Error(message);
+        }
         const contentHolderReference = `${entityName}:${entityId}`;
         const content = await new sdk_1.Content().create({
             fileBuffer: buffer,

package/lib/util/thumbnail-util.spec.js

@@ -437,6 +437,41 @@ describe('ThumbnailUtil Tests', () => {
             expect(mockEntitiesDelete).not.toHaveBeenCalled();
         });
     });
+    describe('syncThumbnailToVibeIQ - useDistinctRestEndPointForImages enabled', () => {
+        let tu;
+        const distinctConfig = { useDistinctRestEndPointForImages: true };
+        beforeEach(() => {
+            jest.clearAllMocks();
+            tu = new thumbnail_util_1.ThumbnailUtil(distinctConfig);
+            mockEntitiesGet.mockImplementation((opts) => {
+                if (opts.entityName === 'content-custom-size')
+                    return Promise.resolve([]);
+                return Promise.resolve({});
+            });
+            mockEntitiesUpdate.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
+            mockEntitiesDelete.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
+        });
+        it('routes through the distinct image endpoint with includeUrlContext true', async () => {
+            const mockResponse = {
+                arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+                headers: { get: jest.fn().mockReturnValue('image/png') },
+            };
+            mockGetRequest.mockResolvedValue(mockResponse);
+            mockContentCreate.mockResolvedValue({
+                id: 'distinctContent1', contentType: 'image/png', fileName: 'thumb.png',
+                primaryFileUrl: 'https://files/primary.png', largeViewableUrl: null,
+                mediumLargeViewableUrl: null, mediumViewableUrl: null, smallViewableUrl: null, tinyViewableUrl: null,
+            });
+            const thumbnailUrl = '/rest/thumbnail/thumb.png';
+            const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: thumbnailUrl } };
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            expect(mockGetRequest).toHaveBeenCalledWith({
+                urlPath: '/servlet/rest/rfa/vibeiq/image?path=' + encodeURIComponent(thumbnailUrl),
+                includeUrlContext: true,
+                returnFullResponse: true,
+            });
+        });
+    });
     describe('ThumbnailUtil - iteratedThumbnailId (THUMBNAIL key)', () => {
         const config = {};
         let tu;

package/package.json

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