@contrail/flexplm 1.3.0-alpha.4 → 1.3.0-alpha.6

package/CHANGELOG.md

@@ -7,6 +7,16 @@ Versioning follows [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [1.3.0] - 2026-04-15
+### Added
+- Added inbound thumbnail/primary content syncing from FlexPLM to VibeIQ via `ThumbnailUtil.syncThumbnailToVibeIQ`.
+- Added `syncInboundImages` and `syncOutboundImages` methods to `TypeConversionUtils` for controlling image sync per map file configuration.
+- Added `PRIMARY_CONTENT_UPDATED` status to `EventShortMessageStatus` for when only primary content changes are detected.
+
+### Changed
+- `BaseEntityProcessor` update flow now distinguishes between primary-content-only changes and no changes, returning the updated entity when only the thumbnail was synced.
+- Improved `FlexPLMConnect.getRequest` function and added unit tests.
+
 ## [1.2.1] - 2026-04-08
 
 ### Added

package/lib/entity-processor/base-entity-processor.js

@@ -68,13 +68,10 @@ class BaseEntityProcessor {
                 console.log(statusMsg);
                 return createEntityResponse.earlyReturn;
             }
-            const createdEntity = await this.createEntity(this.baseType, createEntityResponse.entity);
+            let createdEntity = await this.createEntity(this.baseType, createEntityResponse.entity);
             const shouldSyncThumbnail = await type_conversion_utils_1.TypeConversionUtils.syncInboundImages(this.transformMapFile, this.mapFileUtil, event.data);
             if (shouldSyncThumbnail) {
-                const thumbnailUpdates = await new thumbnail_util_1.ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: createdEntity.id, primaryViewableId: createdEntity.primaryViewableId, event, entityName: this.baseType });
-                if (thumbnailUpdates) {
-                    await this.updateEntity(this.baseType, createdEntity, thumbnailUpdates);
-                }
+                createdEntity = await new thumbnail_util_1.ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: createdEntity.id, primaryViewableId: createdEntity.primaryViewableId, event, entityName: this.baseType }) || createdEntity;
             }
             const statusMsg = this.getInboundStatusMessage({
                 status: event_short_message_status_1.EventShortMessageStatus.SUCCESS,
@@ -88,12 +85,23 @@ class BaseEntityProcessor {
         }
         const diffs = await this.getUpdatesForEntity(entity, inboundData);
         const shouldSyncThumbnail = await type_conversion_utils_1.TypeConversionUtils.syncInboundImages(this.transformMapFile, this.mapFileUtil, event.data);
-        let thumbnailUpdates;
+        let thumbnailEntity;
         if (shouldSyncThumbnail) {
-            thumbnailUpdates = await new thumbnail_util_1.ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: entity.id, primaryViewableId: entity.primaryViewableId, event, entityName: this.baseType });
+            thumbnailEntity = await new thumbnail_util_1.ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: entity.id, primaryViewableId: entity.primaryViewableId, event, entityName: this.baseType });
         }
-        const allUpdates = { ...diffs, ...thumbnailUpdates };
-        if (Object.getOwnPropertyNames(allUpdates).length == 0) {
+        const hasPropertyChanges = Object.getOwnPropertyNames(diffs).length > 0;
+        if (!hasPropertyChanges && thumbnailEntity) {
+            const statusMsg = this.getInboundStatusMessage({
+                status: event_short_message_status_1.EventShortMessageStatus.SUCCESS,
+                statusMessage: event_short_message_status_1.EventShortMessageStatus.PRIMARY_CONTENT_UPDATED,
+                objectClass: event.objectClass,
+                entityId: entity.id,
+                federatedId: event.federatedId
+            });
+            console.log(statusMsg);
+            return thumbnailEntity;
+        }
+        if (!hasPropertyChanges) {
             const statusMsg = this.getInboundStatusMessage({
                 status: event_short_message_status_1.EventShortMessageStatus.SUCCESS,
                 statusMessage: event_short_message_status_1.EventShortMessageStatus.NO_CHANGES,
@@ -108,7 +116,7 @@ class BaseEntityProcessor {
                 data: { message }
             };
         }
-        const updatedEntity = await this.updateEntity(this.baseType, entity, allUpdates);
+        const updatedEntity = await this.updateEntity(this.baseType, entity, diffs);
         const statusMsg = this.getInboundStatusMessage({
             status: event_short_message_status_1.EventShortMessageStatus.SUCCESS,
             statusMessage: event_short_message_status_1.EventShortMessageStatus.UPDATED,

package/lib/entity-processor/base-entity-processor.spec.js

@@ -320,30 +320,14 @@ describe('BaseEntityProcessor', () => {
             syncInboundImagesSpy = jest.spyOn(type_conversion_utils_1.TypeConversionUtils, 'syncInboundImages');
             syncThumbnailSpy = jest.spyOn(thumbnail_util_1.ThumbnailUtil.prototype, 'syncThumbnailToVibeIQ').mockResolvedValue(undefined);
         });
-        it('should call updateEntity with thumbnail updates after create when syncInboundImages returns true', async () => {
+        it('should call syncThumbnailToVibeIQ with entity after create when syncInboundImages returns true', async () => {
             jest.spyOn(btep, 'getIncomingEntity').mockResolvedValue({ entity: null });
             jest.spyOn(btep, 'getCreateEntity').mockResolvedValue({ entity: { name: 'new' } });
             jest.spyOn(btep, 'createEntity').mockResolvedValue(mockCreatedEntity);
-            const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockCreatedEntity);
             syncInboundImagesSpy.mockResolvedValue(true);
-            const thumbnailUpdates = { primaryViewableId: 'pv-new', primaryFileUrl: 'https://files/new.png' };
-            syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
             const result = await btep.handleIncomingUpsert(mockEvent);
             expect(syncThumbnailSpy).toBeCalledTimes(1);
             expect(syncThumbnailSpy).toBeCalledWith({ entityId: 'created-1', primaryViewableId: 'pv-created', event: mockEvent, entityName: 'test' });
-            expect(updateEntitySpy).toBeCalledWith('test', mockCreatedEntity, thumbnailUpdates);
-            expect(result).toBe(mockCreatedEntity);
-        });
-        it('should not call updateEntity after create when syncThumbnailToVibeIQ returns undefined', async () => {
-            jest.spyOn(btep, 'getIncomingEntity').mockResolvedValue({ entity: null });
-            jest.spyOn(btep, 'getCreateEntity').mockResolvedValue({ entity: { name: 'new' } });
-            jest.spyOn(btep, 'createEntity').mockResolvedValue(mockCreatedEntity);
-            const updateEntitySpy = jest.spyOn(btep, 'updateEntity');
-            syncInboundImagesSpy.mockResolvedValue(true);
-            syncThumbnailSpy.mockResolvedValue(undefined);
-            const result = await btep.handleIncomingUpsert(mockEvent);
-            expect(syncThumbnailSpy).toBeCalledTimes(1);
-            expect(updateEntitySpy).not.toHaveBeenCalled();
             expect(result).toBe(mockCreatedEntity);
         });
         it('should not sync images after create when syncInboundImages returns false', async () => {
@@ -356,28 +340,15 @@ describe('BaseEntityProcessor', () => {
             expect(syncThumbnailSpy).toBeCalledTimes(0);
             expect(result).toBe(mockCreatedEntity);
         });
-        it('should merge thumbnail updates with property diffs on update', async () => {
+        it('should call syncThumbnailToVibeIQ with entity and update with property diffs on update', async () => {
             jest.spyOn(btep, 'getIncomingEntity').mockResolvedValue({ entity: mockExistingEntity });
             jest.spyOn(btep, 'getUpdatesForEntity').mockResolvedValue({ name: 'changed' });
             const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockUpdatedEntity);
             syncInboundImagesSpy.mockResolvedValue(true);
-            const thumbnailUpdates = { primaryViewableId: 'pv-new' };
-            syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
             const result = await btep.handleIncomingUpsert(mockEvent);
             expect(syncThumbnailSpy).toBeCalledTimes(1);
             expect(syncThumbnailSpy).toBeCalledWith({ entityId: 'existing-1', primaryViewableId: undefined, event: mockEvent, entityName: 'test' });
-            expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { name: 'changed', primaryViewableId: 'pv-new' });
-            expect(result).toBe(mockUpdatedEntity);
-        });
-        it('should update entity with only thumbnail changes when no property diffs', async () => {
-            jest.spyOn(btep, 'getIncomingEntity').mockResolvedValue({ entity: mockExistingEntity });
-            jest.spyOn(btep, 'getUpdatesForEntity').mockResolvedValue({});
-            const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockUpdatedEntity);
-            syncInboundImagesSpy.mockResolvedValue(true);
-            const thumbnailUpdates = { primaryViewableId: 'pv-new' };
-            syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
-            const result = await btep.handleIncomingUpsert(mockEvent);
-            expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { primaryViewableId: 'pv-new' });
+            expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { name: 'changed' });
             expect(result).toBe(mockUpdatedEntity);
         });
         it('should not sync images on update when syncInboundImages returns false', async () => {

package/lib/util/event-short-message-status.d.ts

@@ -9,6 +9,7 @@ export declare enum EventShortMessageStatus {
     MISSING_INPUT = "Missing_input",
     NOT_CREATABLE = "Not_creatable",
     NO_CHANGES = "No_Changes",
+    PRIMARY_CONTENT_UPDATED = "Primary_Content_Updated",
     TOO_MANY_ENTITIES_FOUND = "Too_Many_Entities_Found",
     UPDATED = "Updated",
     NOT_PUBLISHABLE = "Not_Publishable",

package/lib/util/event-short-message-status.js

@@ -13,6 +13,7 @@ var EventShortMessageStatus;
     EventShortMessageStatus["MISSING_INPUT"] = "Missing_input";
     EventShortMessageStatus["NOT_CREATABLE"] = "Not_creatable";
     EventShortMessageStatus["NO_CHANGES"] = "No_Changes";
+    EventShortMessageStatus["PRIMARY_CONTENT_UPDATED"] = "Primary_Content_Updated";
     EventShortMessageStatus["TOO_MANY_ENTITIES_FOUND"] = "Too_Many_Entities_Found";
     EventShortMessageStatus["UPDATED"] = "Updated";
     EventShortMessageStatus["NOT_PUBLISHABLE"] = "Not_Publishable";

package/lib/util/thumbnail-util.js

@@ -119,8 +119,9 @@ class ThumbnailUtil {
                 await this.entities.delete({ entityName: 'content', id: primaryViewableId });
             }
             const clearUpdates = await this.getClearPrimaryViewableUpdates();
-            console.debug(`syncThumbnailToVibeIQ: returning clear updates for entityId=${entityId}`);
-            return clearUpdates;
+            const updatedEntity = await this.entities.update({ entityName, id: entityId, object: clearUpdates });
+            console.debug(`syncThumbnailToVibeIQ: applied clear updates for entityId=${entityId}`);
+            return updatedEntity;
         }
         if (!thumbnailUrl) {
             console.debug(`syncThumbnailToVibeIQ: no thumbnail URL for entityId=${entityId}`);
@@ -130,8 +131,9 @@ class ThumbnailUtil {
             const content = await this.createContentFromFlexPLM(thumbnailUrl, entityId, entityName);
             await this.entities.update({ entityName: 'content', id: content.id, object: { flexplmThumbnailUrl: thumbnailUrl } });
             const primaryUpdates = await this.getPrimaryViewableUpdates(content);
+            const updatedEntity = await this.entities.update({ entityName, id: entityId, object: primaryUpdates });
             console.debug(`syncThumbnailToVibeIQ: created new content ${content.id} for entityId=${entityId}`);
-            return primaryUpdates;
+            return updatedEntity;
         }
         const primaryViewable = await this.entities.get({ entityName: 'content', id: primaryViewableId });
         if (primaryViewable?.flexplmThumbnailUrl === thumbnailUrl) {
@@ -141,14 +143,18 @@ class ThumbnailUtil {
         const content = await this.createContentFromFlexPLM(thumbnailUrl, entityId, entityName);
         await this.entities.update({ entityName: 'content', id: content.id, object: { flexplmThumbnailUrl: thumbnailUrl } });
         const primaryUpdates = await this.getPrimaryViewableUpdates(content);
+        const updatedEntity = await this.entities.update({ entityName, id: entityId, object: primaryUpdates });
         await this.entities.delete({ entityName: 'content', id: primaryViewableId });
         console.debug(`syncThumbnailToVibeIQ: replaced content ${primaryViewableId} with ${content.id} for entityId=${entityId}`);
-        return primaryUpdates;
+        return updatedEntity;
     }
     async createContentFromFlexPLM(thumbnailUrl, entityId, entityName) {
+        const urlParts = thumbnailUrl.split('/');
+        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: thumbnailUrl,
+            urlPath: encodedUrl,
             includeUrlContext: false,
             returnFullResponse: true,
         });
@@ -156,8 +162,6 @@ class ThumbnailUtil {
         const buffer = Buffer.from(fileBuffer);
         const contentTypeHeader = response.headers.get('content-type');
         const contentType = contentTypeHeader ? contentTypeHeader.split(';')[0] : 'application/octet-stream';
-        const urlParts = thumbnailUrl.split('/');
-        const fileName = urlParts[urlParts.length - 1] || 'thumbnail';
         const contentHolderReference = `${entityName}:${entityId}`;
         const content = await new sdk_1.Content().create({
             fileBuffer: buffer,

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

@@ -276,34 +276,32 @@ describe('ThumbnailUtil Tests', () => {
             mockEntitiesUpdate.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
             mockEntitiesDelete.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
         });
-        it('returns undefined when no thumbnail IDs in event data', async () => {
+        it('does not update when no thumbnail IDs in event data', async () => {
             const event = { data: {} };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
-            expect(result).toBeUndefined();
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
             expect(mockEntitiesUpdate).not.toHaveBeenCalled();
             expect(mockEntitiesDelete).not.toHaveBeenCalled();
             expect(mockContentCreate).not.toHaveBeenCalled();
         });
-        it('returns undefined when event.data is undefined', async () => {
+        it('does not update when event.data is undefined', async () => {
             const event = {};
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
-            expect(result).toBeUndefined();
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
             expect(mockEntitiesUpdate).not.toHaveBeenCalled();
             expect(mockEntitiesDelete).not.toHaveBeenCalled();
         });
-        it('REMOVE_THUMBNAIL with existing primaryViewableId deletes content and returns clear updates', async () => {
+        it('REMOVE_THUMBNAIL with existing primaryViewableId deletes content and updates entity', async () => {
             const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: thumbnail_util_1.ThumbnailUtil.REMOVE_THUMBNAIL } };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
             expect(mockEntitiesDelete).toHaveBeenCalledWith({ entityName: 'content', id: 'pv1' });
-            expect(result).toEqual(expect.objectContaining({ primaryViewableId: null, primaryFileUrl: null }));
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'color', id: 'entity1' }));
         });
-        it('REMOVE_THUMBNAIL with no primaryViewableId returns clear updates without deleting', async () => {
+        it('REMOVE_THUMBNAIL with no primaryViewableId updates entity without deleting', async () => {
             const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: thumbnail_util_1.ThumbnailUtil.REMOVE_THUMBNAIL } };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
             expect(mockEntitiesDelete).not.toHaveBeenCalled();
-            expect(result).toEqual(expect.objectContaining({ primaryViewableId: null }));
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'item', id: 'entity1' }));
         });
-        it('creates new content when no primaryViewableId exists and returns primary viewable updates', async () => {
+        it('creates new content when no primaryViewableId exists and updates entity', async () => {
             const mockResponse = {
                 arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
                 headers: { get: jest.fn().mockReturnValue('image/png') },
@@ -322,7 +320,7 @@ describe('ThumbnailUtil Tests', () => {
             };
             mockContentCreate.mockResolvedValue(createdContent);
             const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/thumb.png' } };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
             expect(mockGetRequest).toHaveBeenCalledWith({ urlPath: '/rest/thumbnail/thumb.png', includeUrlContext: false, returnFullResponse: true });
             expect(mockContentCreate).toHaveBeenCalledWith(expect.objectContaining({
                 fileName: 'thumb.png',
@@ -334,13 +332,52 @@ describe('ThumbnailUtil Tests', () => {
                 id: 'newContent1',
                 object: { flexplmThumbnailUrl: '/rest/thumbnail/thumb.png' },
             }));
-            expect(result).toEqual(expect.objectContaining({
-                primaryViewableId: 'newContent1',
-                primaryFileUrl: 'https://files/primary.png',
-                largeViewableDownloadUrl: 'https://files/large.png',
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({
+                entityName: 'color',
+                id: 'entity1',
             }));
         });
-        it('replaces content when primaryViewable.flexplmThumbnailUrl differs and returns updates', async () => {
+        it('encodes URL path segments with special characters', async () => {
+            const mockResponse = {
+                arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+                headers: { get: jest.fn().mockReturnValue('image/png') },
+            };
+            mockGetRequest.mockResolvedValue(mockResponse);
+            mockContentCreate.mockResolvedValue({
+                id: 'c1', contentType: 'image/png', fileName: 'my image.png',
+                primaryFileUrl: 'https://files/primary.png', largeViewableUrl: null,
+                mediumLargeViewableUrl: null, mediumViewableUrl: null, smallViewableUrl: null, tinyViewableUrl: null,
+            });
+            const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/my image.png' } };
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            expect(mockGetRequest).toHaveBeenCalledWith({
+                urlPath: '/rest/thumbnail/my%20image.png',
+                includeUrlContext: false,
+                returnFullResponse: true,
+            });
+            expect(mockContentCreate).toHaveBeenCalledWith(expect.objectContaining({ fileName: 'my image.png' }));
+        });
+        it('encodes URL path segments with unicode characters', async () => {
+            const mockResponse = {
+                arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+                headers: { get: jest.fn().mockReturnValue('image/jpeg') },
+            };
+            mockGetRequest.mockResolvedValue(mockResponse);
+            mockContentCreate.mockResolvedValue({
+                id: 'c2', contentType: 'image/jpeg', fileName: 'café-logo.jpg',
+                primaryFileUrl: 'https://files/primary.jpg', largeViewableUrl: null,
+                mediumLargeViewableUrl: null, mediumViewableUrl: null, smallViewableUrl: null, tinyViewableUrl: null,
+            });
+            const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/café-logo.jpg' } };
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
+            expect(mockGetRequest).toHaveBeenCalledWith({
+                urlPath: '/rest/thumbnail/caf%C3%A9-logo.jpg',
+                includeUrlContext: false,
+                returnFullResponse: true,
+            });
+            expect(mockContentCreate).toHaveBeenCalledWith(expect.objectContaining({ fileName: 'café-logo.jpg' }));
+        });
+        it('replaces content when primaryViewable.flexplmThumbnailUrl differs and updates entity', async () => {
             mockEntitiesGet.mockImplementation((opts) => {
                 if (opts.entityName === 'content-custom-size')
                     return Promise.resolve([]);
@@ -367,17 +404,17 @@ describe('ThumbnailUtil Tests', () => {
             };
             mockContentCreate.mockResolvedValue(createdContent);
             const event = { data: { [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/new.jpg' } };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'oldPv', event, entityName: 'item' });
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'oldPv', event, entityName: 'item' });
             expect(mockContentCreate).toHaveBeenCalled();
             expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({
                 entityName: 'content',
                 id: 'newContent2',
                 object: { flexplmThumbnailUrl: '/rest/thumbnail/new.jpg' },
             }));
-            expect(result).toEqual(expect.objectContaining({ primaryViewableId: 'newContent2' }));
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'item', id: 'entity1' }));
             expect(mockEntitiesDelete).toHaveBeenCalledWith({ entityName: 'content', id: 'oldPv' });
         });
-        it('returns undefined when primaryViewable.flexplmThumbnailUrl matches', async () => {
+        it('does not update when primaryViewable.flexplmThumbnailUrl matches', async () => {
             const thumbnailUrl = '/rest/thumbnail/same.png';
             mockEntitiesGet.mockImplementation((opts) => {
                 if (opts.entityName === 'content-custom-size')
@@ -388,8 +425,7 @@ describe('ThumbnailUtil Tests', () => {
                 return Promise.resolve({});
             });
             const event = { data: { [thumbnail_util_1.ThumbnailUtil.EXISTING_THUMBNAIL_ID]: thumbnailUrl } };
-            const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
-            expect(result).toBeUndefined();
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
             expect(mockContentCreate).not.toHaveBeenCalled();
             expect(mockEntitiesUpdate).not.toHaveBeenCalled();
             expect(mockEntitiesDelete).not.toHaveBeenCalled();

package/lib/util/type-conversion-utils.d.ts

@@ -14,7 +14,7 @@ export declare class TypeConversionUtils {
     static getEntityTypePathFromOjbect(fileId: any, mapFileUtil: any, object: any): Promise<string>;
     static getIdentifierPropertiesFromObject(fileId: any, mapFileUtil: MapFileUtil, object: any): Promise<string[]>;
     static getInformationalPropertiesFromObject(fileId: any, mapFileUtil: MapFileUtil, object: any): Promise<string[]>;
-    static getMapKeyFromObject(fileId: any, mapFileUtil: MapFileUtil, object: any, direction: string): Promise<string>;
+    static getMapKeyFromObject(fileId: any, mapFileUtil: MapFileUtil, object: any, direction: string): Promise<string | undefined>;
     static isInboundCreatableFromObject(fileId: string, mapFileUtil: MapFileUtil, object: any, context?: any): Promise<boolean>;
     static isOutboundCreatableFromEntity(fileId: string, mapFileUtil: MapFileUtil, entity: any, context?: any): Promise<boolean>;
     static syncInboundImages(fileId: string, mapFileUtil: MapFileUtil, object: any, context?: any): Promise<boolean>;

package/lib/util/type-conversion-utils.js

@@ -179,7 +179,6 @@ class TypeConversionUtils {
             }
             return type;
         }
-        return '';
     }
     static async isInboundCreatableFromObject(fileId, mapFileUtil, object, context) {
         let isInboundCreatable = false;

package/package.json

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

package/src/entity-processor/base-entity-processor.spec.ts

@@ -359,35 +359,16 @@ describe('BaseEntityProcessor', () =>{
       syncThumbnailSpy = jest.spyOn(ThumbnailUtil.prototype, 'syncThumbnailToVibeIQ').mockResolvedValue(undefined);
     });
 
-    it('should call updateEntity with thumbnail updates after create when syncInboundImages returns true', async () =>{
+    it('should call syncThumbnailToVibeIQ with entity after create when syncInboundImages returns true', async () =>{
       jest.spyOn(btep, 'getIncomingEntity' as any).mockResolvedValue({ entity: null });
       jest.spyOn(btep, 'getCreateEntity' as any).mockResolvedValue({ entity: { name: 'new' } });
       jest.spyOn(btep, 'createEntity').mockResolvedValue(mockCreatedEntity);
-      const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockCreatedEntity);
       syncInboundImagesSpy.mockResolvedValue(true);
-      const thumbnailUpdates = { primaryViewableId: 'pv-new', primaryFileUrl: 'https://files/new.png' };
-      syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
 
       const result = await btep.handleIncomingUpsert(mockEvent);
 
       expect(syncThumbnailSpy).toBeCalledTimes(1);
       expect(syncThumbnailSpy).toBeCalledWith({ entityId: 'created-1', primaryViewableId: 'pv-created', event: mockEvent, entityName: 'test' });
-      expect(updateEntitySpy).toBeCalledWith('test', mockCreatedEntity, thumbnailUpdates);
-      expect(result).toBe(mockCreatedEntity);
-    });
-
-    it('should not call updateEntity after create when syncThumbnailToVibeIQ returns undefined', async () =>{
-      jest.spyOn(btep, 'getIncomingEntity' as any).mockResolvedValue({ entity: null });
-      jest.spyOn(btep, 'getCreateEntity' as any).mockResolvedValue({ entity: { name: 'new' } });
-      jest.spyOn(btep, 'createEntity').mockResolvedValue(mockCreatedEntity);
-      const updateEntitySpy = jest.spyOn(btep, 'updateEntity');
-      syncInboundImagesSpy.mockResolvedValue(true);
-      syncThumbnailSpy.mockResolvedValue(undefined);
-
-      const result = await btep.handleIncomingUpsert(mockEvent);
-
-      expect(syncThumbnailSpy).toBeCalledTimes(1);
-      expect(updateEntitySpy).not.toHaveBeenCalled();
       expect(result).toBe(mockCreatedEntity);
     });
 
@@ -404,33 +385,17 @@ describe('BaseEntityProcessor', () =>{
       expect(result).toBe(mockCreatedEntity);
     });
 
-    it('should merge thumbnail updates with property diffs on update', async () =>{
+    it('should call syncThumbnailToVibeIQ with entity and update with property diffs on update', async () =>{
       jest.spyOn(btep, 'getIncomingEntity' as any).mockResolvedValue({ entity: mockExistingEntity });
       jest.spyOn(btep, 'getUpdatesForEntity').mockResolvedValue({ name: 'changed' });
       const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockUpdatedEntity);
       syncInboundImagesSpy.mockResolvedValue(true);
-      const thumbnailUpdates = { primaryViewableId: 'pv-new' };
-      syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
 
       const result = await btep.handleIncomingUpsert(mockEvent);
 
       expect(syncThumbnailSpy).toBeCalledTimes(1);
       expect(syncThumbnailSpy).toBeCalledWith({ entityId: 'existing-1', primaryViewableId: undefined, event: mockEvent, entityName: 'test' });
-      expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { name: 'changed', primaryViewableId: 'pv-new' });
-      expect(result).toBe(mockUpdatedEntity);
-    });
-
-    it('should update entity with only thumbnail changes when no property diffs', async () =>{
-      jest.spyOn(btep, 'getIncomingEntity' as any).mockResolvedValue({ entity: mockExistingEntity });
-      jest.spyOn(btep, 'getUpdatesForEntity').mockResolvedValue({});
-      const updateEntitySpy = jest.spyOn(btep, 'updateEntity').mockResolvedValue(mockUpdatedEntity);
-      syncInboundImagesSpy.mockResolvedValue(true);
-      const thumbnailUpdates = { primaryViewableId: 'pv-new' };
-      syncThumbnailSpy.mockResolvedValue(thumbnailUpdates);
-
-      const result = await btep.handleIncomingUpsert(mockEvent);
-
-      expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { primaryViewableId: 'pv-new' });
+      expect(updateEntitySpy).toBeCalledWith('test', mockExistingEntity, { name: 'changed' });
       expect(result).toBe(mockUpdatedEntity);
     });
 

package/src/entity-processor/base-entity-processor.ts

@@ -84,15 +84,12 @@ export abstract class BaseEntityProcessor {
         console.log(statusMsg);
         return createEntityResponse.earlyReturn;
       }
-      const createdEntity = await this.createEntity(this.baseType, createEntityResponse.entity);
+      let createdEntity = await this.createEntity(this.baseType, createEntityResponse.entity);
       const shouldSyncThumbnail = await TypeConversionUtils.syncInboundImages(
         this.transformMapFile, this.mapFileUtil, event.data
       );
       if (shouldSyncThumbnail) {
-        const thumbnailUpdates = await new ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: createdEntity.id, primaryViewableId: createdEntity.primaryViewableId, event, entityName: this.baseType });
-        if (thumbnailUpdates) {
-          await this.updateEntity(this.baseType, createdEntity, thumbnailUpdates);
-        }
+        createdEntity = await new ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: createdEntity.id, primaryViewableId: createdEntity.primaryViewableId, event, entityName: this.baseType }) || createdEntity;
       }
       const statusMsg = this.getInboundStatusMessage({
         status: EventShortMessageStatus.SUCCESS,
@@ -109,13 +106,26 @@ export abstract class BaseEntityProcessor {
     const shouldSyncThumbnail = await TypeConversionUtils.syncInboundImages(
       this.transformMapFile, this.mapFileUtil, event.data
     );
-    let thumbnailUpdates;
+    let thumbnailEntity;
     if (shouldSyncThumbnail) {
-      thumbnailUpdates = await new ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: entity.id, primaryViewableId: entity.primaryViewableId, event, entityName: this.baseType });
+      thumbnailEntity = await new ThumbnailUtil(this.config).syncThumbnailToVibeIQ({ entityId: entity.id, primaryViewableId: entity.primaryViewableId, event, entityName: this.baseType });
     }
 
-    const allUpdates = { ...diffs, ...thumbnailUpdates };
-    if(Object.getOwnPropertyNames(allUpdates).length == 0){
+    const hasPropertyChanges = Object.getOwnPropertyNames(diffs).length > 0;
+
+    if (!hasPropertyChanges && thumbnailEntity) {
+      const statusMsg = this.getInboundStatusMessage({
+        status: EventShortMessageStatus.SUCCESS,
+        statusMessage: EventShortMessageStatus.PRIMARY_CONTENT_UPDATED,
+        objectClass: event.objectClass,
+        entityId: entity.id,
+        federatedId: event.federatedId
+      });
+      console.log(statusMsg);
+      return thumbnailEntity;
+    }
+
+    if (!hasPropertyChanges) {
       const statusMsg = this.getInboundStatusMessage({
         status: EventShortMessageStatus.SUCCESS,
         statusMessage: EventShortMessageStatus.NO_CHANGES,
@@ -131,7 +141,7 @@ export abstract class BaseEntityProcessor {
       };
     }
 
-    const updatedEntity  = await this.updateEntity(this.baseType, entity, allUpdates);
+    const updatedEntity  = await this.updateEntity(this.baseType, entity, diffs);
     const statusMsg = this.getInboundStatusMessage({
       status: EventShortMessageStatus.SUCCESS,
       statusMessage: EventShortMessageStatus.UPDATED,

package/src/util/event-short-message-status.ts

@@ -9,6 +9,7 @@ export enum EventShortMessageStatus {
   MISSING_INPUT = 'Missing_input',
   NOT_CREATABLE = 'Not_creatable',
   NO_CHANGES = 'No_Changes',
+  PRIMARY_CONTENT_UPDATED = 'Primary_Content_Updated',
   TOO_MANY_ENTITIES_FOUND = 'Too_Many_Entities_Found',
   UPDATED = 'Updated',
 

package/src/util/thumbnail-util.spec.ts

@@ -308,42 +308,40 @@ describe('ThumbnailUtil Tests', () =>{
       mockEntitiesDelete.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
     });
 
-    it('returns undefined when no thumbnail IDs in event data', async () => {
+    it('does not update when no thumbnail IDs in event data', async () => {
       const event = { data: {} };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
 
-      expect(result).toBeUndefined();
       expect(mockEntitiesUpdate).not.toHaveBeenCalled();
       expect(mockEntitiesDelete).not.toHaveBeenCalled();
       expect(mockContentCreate).not.toHaveBeenCalled();
     });
 
-    it('returns undefined when event.data is undefined', async () => {
+    it('does not update when event.data is undefined', async () => {
       const event = {};
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
 
-      expect(result).toBeUndefined();
       expect(mockEntitiesUpdate).not.toHaveBeenCalled();
       expect(mockEntitiesDelete).not.toHaveBeenCalled();
     });
 
-    it('REMOVE_THUMBNAIL with existing primaryViewableId deletes content and returns clear updates', async () => {
+    it('REMOVE_THUMBNAIL with existing primaryViewableId deletes content and updates entity', async () => {
       const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: ThumbnailUtil.REMOVE_THUMBNAIL } };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
 
       expect(mockEntitiesDelete).toHaveBeenCalledWith({ entityName: 'content', id: 'pv1' });
-      expect(result).toEqual(expect.objectContaining({ primaryViewableId: null, primaryFileUrl: null }));
+      expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'color', id: 'entity1' }));
     });
 
-    it('REMOVE_THUMBNAIL with no primaryViewableId returns clear updates without deleting', async () => {
+    it('REMOVE_THUMBNAIL with no primaryViewableId updates entity without deleting', async () => {
       const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: ThumbnailUtil.REMOVE_THUMBNAIL } };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
 
       expect(mockEntitiesDelete).not.toHaveBeenCalled();
-      expect(result).toEqual(expect.objectContaining({ primaryViewableId: null }));
+      expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'item', id: 'entity1' }));
     });
 
-    it('creates new content when no primaryViewableId exists and returns primary viewable updates', async () => {
+    it('creates new content when no primaryViewableId exists and updates entity', async () => {
       const mockResponse = {
         arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
         headers: { get: jest.fn().mockReturnValue('image/png') },
@@ -364,7 +362,7 @@ describe('ThumbnailUtil Tests', () =>{
       mockContentCreate.mockResolvedValue(createdContent);
 
       const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/thumb.png' } };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
 
       expect(mockGetRequest).toHaveBeenCalledWith({ urlPath: '/rest/thumbnail/thumb.png', includeUrlContext: false, returnFullResponse: true });
       expect(mockContentCreate).toHaveBeenCalledWith(
@@ -382,15 +380,66 @@ describe('ThumbnailUtil Tests', () =>{
           object: { flexplmThumbnailUrl: '/rest/thumbnail/thumb.png' },
         }),
       );
-      // Returns primary viewable updates for the entity
-      expect(result).toEqual(expect.objectContaining({
-        primaryViewableId: 'newContent1',
-        primaryFileUrl: 'https://files/primary.png',
-        largeViewableDownloadUrl: 'https://files/large.png',
-      }));
+      // Updates the main entity
+      expect(mockEntitiesUpdate).toHaveBeenCalledWith(
+        expect.objectContaining({
+          entityName: 'color',
+          id: 'entity1',
+        }),
+      );
+    });
+
+    it('encodes URL path segments with special characters', async () => {
+      const mockResponse = {
+        arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+        headers: { get: jest.fn().mockReturnValue('image/png') },
+      };
+      mockGetRequest.mockResolvedValue(mockResponse);
+      mockContentCreate.mockResolvedValue({
+        id: 'c1', contentType: 'image/png', fileName: 'my image.png',
+        primaryFileUrl: 'https://files/primary.png', largeViewableUrl: null,
+        mediumLargeViewableUrl: null, mediumViewableUrl: null, smallViewableUrl: null, tinyViewableUrl: null,
+      });
+
+      const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/my image.png' } };
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+
+      expect(mockGetRequest).toHaveBeenCalledWith({
+        urlPath: '/rest/thumbnail/my%20image.png',
+        includeUrlContext: false,
+        returnFullResponse: true,
+      });
+      expect(mockContentCreate).toHaveBeenCalledWith(
+        expect.objectContaining({ fileName: 'my image.png' }),
+      );
     });
 
-    it('replaces content when primaryViewable.flexplmThumbnailUrl differs and returns updates', async () => {
+    it('encodes URL path segments with unicode characters', async () => {
+      const mockResponse = {
+        arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+        headers: { get: jest.fn().mockReturnValue('image/jpeg') },
+      };
+      mockGetRequest.mockResolvedValue(mockResponse);
+      mockContentCreate.mockResolvedValue({
+        id: 'c2', contentType: 'image/jpeg', fileName: 'café-logo.jpg',
+        primaryFileUrl: 'https://files/primary.jpg', largeViewableUrl: null,
+        mediumLargeViewableUrl: null, mediumViewableUrl: null, smallViewableUrl: null, tinyViewableUrl: null,
+      });
+
+      const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/café-logo.jpg' } };
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
+
+      expect(mockGetRequest).toHaveBeenCalledWith({
+        urlPath: '/rest/thumbnail/caf%C3%A9-logo.jpg',
+        includeUrlContext: false,
+        returnFullResponse: true,
+      });
+      expect(mockContentCreate).toHaveBeenCalledWith(
+        expect.objectContaining({ fileName: 'café-logo.jpg' }),
+      );
+    });
+
+    it('replaces content when primaryViewable.flexplmThumbnailUrl differs and updates entity', async () => {
       mockEntitiesGet.mockImplementation((opts) => {
         if (opts.entityName === 'content-custom-size') return Promise.resolve([]);
         if (opts.entityName === 'content' && opts.id === 'oldPv') {
@@ -419,7 +468,7 @@ describe('ThumbnailUtil Tests', () =>{
       mockContentCreate.mockResolvedValue(createdContent);
 
       const event = { data: { [ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/new.jpg' } };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'oldPv', event, entityName: 'item' });
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'oldPv', event, entityName: 'item' });
 
       // Creates new content
       expect(mockContentCreate).toHaveBeenCalled();
@@ -431,13 +480,15 @@ describe('ThumbnailUtil Tests', () =>{
           object: { flexplmThumbnailUrl: '/rest/thumbnail/new.jpg' },
         }),
       );
-      // Returns primary viewable updates for the entity
-      expect(result).toEqual(expect.objectContaining({ primaryViewableId: 'newContent2' }));
+      // Updates the main entity
+      expect(mockEntitiesUpdate).toHaveBeenCalledWith(
+        expect.objectContaining({ entityName: 'item', id: 'entity1' }),
+      );
       // Deletes old content
       expect(mockEntitiesDelete).toHaveBeenCalledWith({ entityName: 'content', id: 'oldPv' });
     });
 
-    it('returns undefined when primaryViewable.flexplmThumbnailUrl matches', async () => {
+    it('does not update when primaryViewable.flexplmThumbnailUrl matches', async () => {
       const thumbnailUrl = '/rest/thumbnail/same.png';
       mockEntitiesGet.mockImplementation((opts) => {
         if (opts.entityName === 'content-custom-size') return Promise.resolve([]);
@@ -448,9 +499,7 @@ describe('ThumbnailUtil Tests', () =>{
       });
 
       const event = { data: { [ThumbnailUtil.EXISTING_THUMBNAIL_ID]: thumbnailUrl } };
-      const result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
-
-      expect(result).toBeUndefined();
+      await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', primaryViewableId: 'pv1', event, entityName: 'color' });
       expect(mockContentCreate).not.toHaveBeenCalled();
       expect(mockEntitiesUpdate).not.toHaveBeenCalled();
       expect(mockEntitiesDelete).not.toHaveBeenCalled();

package/src/util/thumbnail-util.ts

@@ -9,6 +9,7 @@ interface ContentCustomSize {
   name: string;
 };
 export class ThumbnailUtil {
+  /** The max_thumbnail_size is for limiting the size of the thumbnail being sent to FlexPLM.  It is used when checking the size of the auto generated thumbnails (smallViewable, tinyViewable, etc.). */
   private max_thumbnail_size = 5 * 1_024 * 1_024;
   private entities: Entities;
   static NEW_THUMBNAIL_ID = 'NEW_THUMBNAIL_ID';
@@ -140,6 +141,15 @@ export class ThumbnailUtil {
       }
     }
 
+  /** Syncs the thumbnail from FlexPLM to VibeIQ. Handles creating, replacing, or removing
+   * the primary viewable content and persists the updates directly to the entity.
+   *
+   * @param entityId - The ID of the entity to update with thumbnail properties.
+   * @param primaryViewableId - The existing primary viewable content ID, if any.
+   * @param event - The inbound event containing thumbnail data (NEW_THUMBNAIL_ID / EXISTING_THUMBNAIL_ID).
+   * @param entityName - The entity type name (e.g. 'item', 'color') used for API calls.
+   * @returns The updated entity, or undefined if no thumbnail changes were needed.
+   */
   async syncThumbnailToVibeIQ({ entityId, primaryViewableId, event, entityName }: { entityId: string; primaryViewableId?: string; event: any; entityName: string }): Promise<any> {
     console.debug(`syncThumbnailToVibeIQ: entityId=${entityId}, primaryViewableId=${primaryViewableId}, entityName=${entityName}`);
     const eventData = event.data || {};
@@ -153,8 +163,9 @@ export class ThumbnailUtil {
         await this.entities.delete({ entityName: 'content', id: primaryViewableId });
       }
       const clearUpdates = await this.getClearPrimaryViewableUpdates();
-      console.debug(`syncThumbnailToVibeIQ: returning clear updates for entityId=${entityId}`);
-      return clearUpdates;
+      const updatedEntity = await this.entities.update({ entityName, id: entityId, object: clearUpdates });
+      console.debug(`syncThumbnailToVibeIQ: applied clear updates for entityId=${entityId}`);
+      return updatedEntity;
     }
 
     // Early return if no thumbnail URL
@@ -168,8 +179,9 @@ export class ThumbnailUtil {
       const content = await this.createContentFromFlexPLM(thumbnailUrl, entityId, entityName);
       await this.entities.update({ entityName: 'content', id: content.id, object: { flexplmThumbnailUrl: thumbnailUrl } });
       const primaryUpdates = await this.getPrimaryViewableUpdates(content);
+      const updatedEntity = await this.entities.update({ entityName, id: entityId, object: primaryUpdates });
       console.debug(`syncThumbnailToVibeIQ: created new content ${content.id} for entityId=${entityId}`);
-      return primaryUpdates;
+      return updatedEntity;
     }
 
     // Case 3: Has primaryViewableId — check if thumbnail changed
@@ -182,15 +194,20 @@ export class ThumbnailUtil {
     const content = await this.createContentFromFlexPLM(thumbnailUrl, entityId, entityName);
     await this.entities.update({ entityName: 'content', id: content.id, object: { flexplmThumbnailUrl: thumbnailUrl } });
     const primaryUpdates = await this.getPrimaryViewableUpdates(content);
+    const updatedEntity = await this.entities.update({ entityName, id: entityId, object: primaryUpdates });
     await this.entities.delete({ entityName: 'content', id: primaryViewableId });
     console.debug(`syncThumbnailToVibeIQ: replaced content ${primaryViewableId} with ${content.id} for entityId=${entityId}`);
-    return primaryUpdates;
+    return updatedEntity;
   }
 
   private async createContentFromFlexPLM(thumbnailUrl: string, entityId: string, entityName: string): Promise<any> {
+    const urlParts = thumbnailUrl.split('/');
+    const fileName = urlParts[urlParts.length - 1] || 'thumbnail';
+
+    const encodedUrl = urlParts.map(part => encodeURIComponent(part)).join('/');
     const flexPLMConnect = new FlexPLMConnect(this.config);
     const response = await flexPLMConnect.getRequest({
-      urlPath: thumbnailUrl,
+      urlPath: encodedUrl,
       includeUrlContext: false,
       returnFullResponse: true,
     }) as Response;
@@ -200,9 +217,6 @@ export class ThumbnailUtil {
     const contentTypeHeader = response.headers.get('content-type');
     const contentType = contentTypeHeader ? contentTypeHeader.split(';')[0] : 'application/octet-stream';
 
-    const urlParts = thumbnailUrl.split('/');
-    const fileName = urlParts[urlParts.length - 1] || 'thumbnail';
-
     const contentHolderReference = `${entityName}:${entityId}`;
     const content = await new Content().create({
       fileBuffer: buffer,

package/src/util/type-conversion-utils.ts

@@ -323,7 +323,7 @@ export class TypeConversionUtils {
     return TypeDefaults.getDefaultInformationalPropertiesFromObject(object);
   }
 
-  static async getMapKeyFromObject(fileId, mapFileUtil: MapFileUtil, object: any, direction:string): Promise<string> {
+  static async getMapKeyFromObject(fileId, mapFileUtil: MapFileUtil, object: any, direction:string): Promise<string | undefined> {
     const type = this.getObjectType(object);
     if(fileId){
       const mappingData = await mapFileUtil.getMappingSection(fileId, 'typeConversion', direction);
@@ -334,7 +334,6 @@ export class TypeConversionUtils {
       return type;
     }
     //TODO use TypeDefaults?
-    return '';
   }
 
   static async isInboundCreatableFromObject(fileId: string, mapFileUtil: MapFileUtil, object: any, context?: any): Promise<boolean> {