@capillarytech/creatives-library 9.0.53-alpha.0 → 9.0.53-alpha.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@capillarytech/creatives-library",
   "author": "meharaj",
-  "version": "9.0.53-alpha.0",
+  "version": "9.0.53-alpha.2",
   "description": "Capillary creatives ui",
   "main": "./index.js",
   "module": "./index.es.js",

package/v2Containers/CommunicationFlow/CommunicationFlow.js

@@ -39,6 +39,7 @@ import {
   CCS_STRATEGY_TYPE_SINGLE,
   CCS_CHANNEL_CONTENT_KEY_MAP,
   CCS_CHANNEL_DELIVERY_KEY_MAP,
+  CCS_CHANNEL_NAME_MAP,
 } from './constants';
 import { getEnabledSteps } from './utils/getEnabledSteps';
 import messages from './messages';
@@ -186,7 +187,8 @@ const CommunicationFlow = ({
       // CCS create is SINGLE-strategy only this phase (D1); CHANNEL_PRIORITY/AB_TEST
       // carry multiple content items with no CCS equivalent yet.
       if (!isMultiChannel && contentItem && name) {
-        const channel = (contentItem.channel || '').toUpperCase();
+        const rawChannel = (contentItem.channel || '').toUpperCase();
+        const channel = CCS_CHANNEL_NAME_MAP[rawChannel] || rawChannel;
         const contentKey = CCS_CHANNEL_CONTENT_KEY_MAP[channel];
         const deliveryKey = CCS_CHANNEL_DELIVERY_KEY_MAP[channel];
         const { dynamicControls = {} } = aggregatedData;
@@ -208,9 +210,11 @@ const CommunicationFlow = ({
             executionParams: {},
           },
           singleChannelStrategy: {
-            channel,
-            ...(contentKey && { [contentKey]: contentItem.templateData }),
-            ...(deliveryKey && { [deliveryKey]: { channelSettings: { channel, ...channelSettings } } }),
+            variant: {
+              channel,
+              ...(contentKey && { [contentKey]: contentItem.templateData }),
+              ...(deliveryKey && { [deliveryKey]: { channelSettings: { channel, ...channelSettings } } }),
+            },
           },
         };
 
@@ -227,7 +231,7 @@ const CommunicationFlow = ({
             ccsCommDefinition = {
               id: data.id,
               referenceId: data.referenceId,
-              version: data.version?.version || 1,
+              version: data.version?.version ?? 1,
               status: data.status,
             };
           }

package/v2Containers/CommunicationFlow/Tests/CommunicationFlow.test.js

@@ -389,14 +389,42 @@ describe('handleSave — CCS flow', () => {
         name: 'Order Placed Notification',
         strategyType: 'SINGLE',
         singleChannelStrategy: expect.objectContaining({
-          channel: 'SMS',
-          smsMessageContent: { message: 'Hello' },
+          variant: expect.objectContaining({
+            channel: 'SMS',
+            smsMessageContent: { message: 'Hello' },
+          }),
         }),
       }),
     );
     expect(onSave).toHaveBeenCalledTimes(1);
   });
 
+  it('normalizes the internal MOBILEPUSH channel to CCS\'s MPUSH enum and key names', async () => {
+    const onSave = jest.fn();
+    renderWithFlow({
+      features: {},
+      config: ccsConfig,
+      initialData: {
+        contentItems: [{ channel: 'MOBILEPUSH', templateData: { messageSubject: 'Hi' } }],
+      },
+      onSave,
+    });
+
+    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
+
+    await waitFor(() => expect(createCommDefinition).toHaveBeenCalledTimes(1));
+    expect(createCommDefinition).toHaveBeenCalledWith(
+      expect.objectContaining({
+        singleChannelStrategy: expect.objectContaining({
+          variant: expect.objectContaining({
+            channel: 'MPUSH',
+            mpushMessageContent: { messageSubject: 'Hi' },
+          }),
+        }),
+      }),
+    );
+  });
+
   it('merges ccsCommDefinition into the data passed to onSave when create succeeds', async () => {
     const onSave = jest.fn();
     renderWithFlow({
@@ -416,6 +444,28 @@ describe('handleSave — CCS flow', () => {
     );
   });
 
+  it('preserves a real version of 0 from CCS (not coerced to 1 by a falsy-zero fallback)', async () => {
+    const onSave = jest.fn();
+    createCommDefinition.mockResolvedValueOnce({
+      response: { data: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', status: 'DRAFT', version: { version: 0 } } },
+    });
+    renderWithFlow({
+      features: {},
+      config: ccsConfig,
+      initialData: { contentItems: [{ channel: 'SMS', templateData: {} }] },
+      onSave,
+    });
+
+    await userEvent.click(screen.getByRole('button', { name: /^save$/i }));
+
+    await waitFor(() => expect(onSave).toHaveBeenCalledTimes(1));
+    expect(onSave).toHaveBeenCalledWith(
+      expect.objectContaining({
+        ccsCommDefinition: { id: 'cd_123', referenceId: 'ORDER_PLACED_123', version: 0, status: 'DRAFT' },
+      }),
+    );
+  });
+
   it('generates a referenceId from name when config.context.referenceId is absent', async () => {
     // moduleMocks.js spies on Date.now globally, but this config's resetMocks:true
     // clears that return value before every test — set it explicitly here.
@@ -575,7 +625,7 @@ describe('handleSave — CCS flow', () => {
       linkTrackingEnabled: true,
       userSubscriptionDisabled: true,
     });
-    expect(payload.singleChannelStrategy.smsDeliverySettings.additionalSettings).toBeUndefined();
+    expect(payload.singleChannelStrategy.variant.smsDeliverySettings.additionalSettings).toBeUndefined();
   });
 
   it('includes channelSettings from deliverySetting.channelSetting in the payload', async () => {
@@ -592,7 +642,7 @@ describe('handleSave — CCS flow', () => {
 
     await waitFor(() => expect(createCommDefinition).toHaveBeenCalled());
     const payload = createCommDefinition.mock.calls[0][0];
-    expect(payload.singleChannelStrategy.smsDeliverySettings.channelSettings).toEqual({
+    expect(payload.singleChannelStrategy.variant.smsDeliverySettings.channelSettings).toEqual({
       channel: 'SMS',
       domainId: 1001,
       gsmSenderId: 'BRAND1',

package/v2Containers/CommunicationFlow/constants.js

@@ -44,15 +44,14 @@ export const SINGLE_TEMPLATE = 'SINGLE_TEMPLATE';
 export const CCS_STRATEGY_TYPE_SINGLE = 'SINGLE';
 
 // Mirrors CCS's SingleChannelStrategy field names exactly (Cap Notify API
-// Design doc, "Channel Strategy overview" / class dump). Kept separate from
-// CHANNEL_CONTENT_KEY_MAP/CHANNEL_DELIVERY_KEY_MAP below, which are the legacy
-// messageMeta payload's keys and do not match CCS naming for every channel
-// (e.g. legacy uses lowercase 'mpushMessageContent'; CCS uses 'mPushMessageContent').
+// Design doc sample payloads, verified against real create responses). Kept
+// separate from CHANNEL_CONTENT_KEY_MAP/CHANNEL_DELIVERY_KEY_MAP below, which
+// are the legacy messageMeta payload's keys.
 export const CCS_CHANNEL_CONTENT_KEY_MAP = {
   SMS: 'smsMessageContent',
   EMAIL: 'emailMessageContent',
   WHATSAPP: 'whatsappMessageContent',
-  MPUSH: 'mPushMessageContent',
+  MPUSH: 'mpushMessageContent',
   INAPP: 'inAppMessageContent',
   ANDROID: 'androidMessageContent',
   IOS: 'iosMessageContent',
@@ -67,7 +66,7 @@ export const CCS_CHANNEL_DELIVERY_KEY_MAP = {
   SMS: 'smsDeliverySettings',
   EMAIL: 'emailDeliverySettings',
   WHATSAPP: 'whatsappDeliverySettings',
-  MPUSH: 'mPushDeliverySettings',
+  MPUSH: 'mpushDeliverySettings',
   INAPP: 'inAppDeliverySettings',
   ANDROID: 'androidDeliverySettings',
   IOS: 'iosDeliverySettings',
@@ -78,6 +77,14 @@ export const CCS_CHANNEL_DELIVERY_KEY_MAP = {
   RCS: 'rcsDeliverySettings',
 };
 
+// The UI's own internal channel identifier for mobile push is 'MOBILEPUSH'
+// (see CreativesContainer/constants.js MOBILE_PUSH), but CCS's channel enum
+// only recognises 'MPUSH' (globals.js CCS_CHANNELS) — normalize before
+// building the CCS payload. Every other channel identifier matches CCS as-is.
+export const CCS_CHANNEL_NAME_MAP = {
+  MOBILEPUSH: 'MPUSH',
+};
+
 // Channel config - single source of truth for CreativesContainer and TemplatesV2
 // paneKey: TemplatesV2 defaultPanes object key (for channelsToHide)
 // channelProp: CreativesContainer channel prop (must match pane.key for tab to be active)