@contrail/flexplm 1.7.8-alpha.0024b30 → 1.7.8-alpha.30ce497

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

@@ -348,6 +348,10 @@ class BaseProcessPublishAssortment {
         const apcDeletes = await this.downloadDeleteChanges(assortmentPublishChange);
         let previousApcDate = Date.parse(apcCreatedOn);
         const sinceDateMilliseconds = sinceDate.getTime();
+        // getSinceDate() always resolves its config (default, date, numberOfDays, numberOfPublishes) to a publish
+        // timestamp, so the walk below stops on an exact match. A caller that supplies a raw date matching no publish
+        // (e.g. a subclass overriding getSinceDate) would never match, so the walk stops on the date instead.
+        const hasPublishAtSinceDate = apcHistory.some(apc => Date.parse(apc?.createdOn) === sinceDateMilliseconds);
         console.info('getDeleteChanges(): apc: ' + assortmentPublishChange?.id
             + ', index: ' + currentAPCIndex + ', previousApcCreatedOn: ' + apcCreatedOn);
         //if only 1 apc, no processing needed
@@ -361,7 +365,11 @@ class BaseProcessPublishAssortment {
             for (let i = currentAPCIndex - 1; i > 0; i--) {
                 const workingAPC = apcHistory[i];
                 previousApcDate = Date.parse(workingAPC?.createdOn);
-                if (sinceDateMilliseconds === previousApcDate) {
+                const reachedSinceDate = hasPublishAtSinceDate
+                    ? previousApcDate === sinceDateMilliseconds
+                    : previousApcDate < sinceDateMilliseconds;
+                if (reachedSinceDate) {
+                    console.info('getDeleteChanges(): reached sinceDate at apc: ' + workingAPC?.id);
                     break;
                 }
                 if (workingAPC?.deletes > 0) {

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

@@ -922,6 +922,71 @@ describe('getDeleteChanges', () => {
         spy.mockRestore();
     });
 });
+describe('getDeleteChanges - how far back the history walk goes', () => {
+    const config = {};
+    const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
+    const dc = new data_converter_1.DataConverter(config, mapFileUtil);
+    // Oldest -> newest. 'current' is the publish being processed.
+    const history = [
+        { id: 'apc0', createdOn: '2025-09-30T00:00:00.000Z', deletes: 5 },
+        { id: 'apc1', createdOn: '2026-09-28T05:48:26.357Z', deletes: 1 },
+        { id: 'apc2', createdOn: '2026-10-08T07:16:43.363Z', deletes: 2 },
+        { id: 'apc3', createdOn: '2026-10-09T02:37:30.292Z', deletes: 3 },
+        { id: 'apc4', createdOn: '2026-10-14T04:23:10.416Z', deletes: 0 },
+        { id: 'current', createdOn: '2026-10-16T15:25:51.965Z', deletes: 0 }
+    ];
+    const current = history[history.length - 1];
+    const baseline = { assortmentItems: [{ itemId: 'stillInPlan' }] };
+    let ppa;
+    let buildSpy;
+    beforeEach(() => {
+        ppa = new base_process_publish_assortment_1.BaseProcessPublishAssortment(config, dc, mapFileUtil);
+        jest.spyOn(ppa, 'downloadDeleteChanges').mockResolvedValue([]);
+        buildSpy = jest.spyOn(ppa, 'buildDeleteChanges')
+            .mockImplementation(async (apc) => [{ itemId: 'deleted-in-' + apc['id'] }]);
+    });
+    afterEach(() => {
+        jest.restoreAllMocks();
+    });
+    const walkedApcIds = () => buildSpy.mock.calls.map(call => call[0]['id']);
+    it('does not walk older publishes when sinceDate is the previous publish date (default)', async () => {
+        const result = await ppa.getDeleteChanges(current, history, baseline, new Date(history[4].createdOn));
+        expect(walkedApcIds()).toEqual([]);
+        expect(result).toEqual([]);
+    });
+    it('stops at the first publish before a date-only sinceDate that matches no publish timestamp', async () => {
+        const result = await ppa.getDeleteChanges(current, history, baseline, new Date('2026-10-08T00:00:00.000Z'));
+        // apc3 and apc2 are after the cutoff; apc1 (Sep 28) is before it, so the walk ends there and apc0 is never read.
+        expect(walkedApcIds()).toEqual(['apc3', 'apc2']);
+        expect(result.map(item => item['itemId'])).toEqual(['deleted-in-apc3', 'deleted-in-apc2']);
+    });
+    it('stops at a publish whose timestamp equals sinceDate without reading it (numberOfPublishes)', async () => {
+        await ppa.getDeleteChanges(current, history, baseline, new Date(history[1].createdOn));
+        expect(walkedApcIds()).toEqual(['apc3', 'apc2']);
+    });
+    it('does not walk any publish when a date-only sinceDate is later than the previous publish', async () => {
+        await ppa.getDeleteChanges(current, history, baseline, new Date('2026-10-15T00:00:00.000Z'));
+        expect(walkedApcIds()).toEqual([]);
+    });
+    it('resolves a date-only sinceDate config to a publish timestamp, so the walk stops on an exact match', async () => {
+        const configured = new base_process_publish_assortment_1.BaseProcessPublishAssortment({ sinceDate: '2026-10-08' }, dc, mapFileUtil);
+        jest.spyOn(configured, 'downloadDeleteChanges').mockResolvedValue([]);
+        const configuredBuildSpy = jest.spyOn(configured, 'buildDeleteChanges')
+            .mockImplementation(async (apc) => [{ itemId: 'deleted-in-' + apc['id'] }]);
+        const sinceDate = await configured.getSinceDate('assortment1', current.id, history);
+        await configured.getDeleteChanges(current, history, baseline, sinceDate);
+        // The latest publish before 2026-10-08 (apc1, Sep 28), not the raw midnight date.
+        expect(sinceDate.toISOString()).toEqual(history[1].createdOn);
+        expect(configuredBuildSpy.mock.calls.map(call => call[0]['id'])).toEqual(['apc3', 'apc2']);
+    });
+    it('skips deleted items still in the plan and de-duplicates across publishes', async () => {
+        buildSpy.mockImplementation(async (apc) => apc['id'] === 'apc3'
+            ? [{ itemId: 'stillInPlan' }, { itemId: 'sameItem' }]
+            : [{ itemId: 'sameItem' }, { itemId: 'deleted-in-' + apc['id'] }]);
+        const result = await ppa.getDeleteChanges(current, history, baseline, new Date('2026-10-08T00:00:00.000Z'));
+        expect(result.map(item => item['itemId'])).toEqual(['sameItem', 'deleted-in-apc2']);
+    });
+});
 describe('buildDeleteChanges', () => {
     const config = {};
     const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.7.8-alpha.0024b30",
+  "version": "1.7.8-alpha.30ce497",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",