Merge pull request #123 from freshworks/handle-null-cases-1

[FC-132765] Handle null case scenarios related to event data

Author: ShashaankKrishnatray

packages/rule-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@freshworks-jaya/rule-engine",
-  "version": "0.18.8-beta-5",
+  "version": "0.18.8",
   "description": "Provides methods to process rules in product events in marketplace app",
   "repository": "[email protected]:freshdesk/jaya-lib.git",
   "main": "lib/index.js",

packages/rule-engine/src/recommended/trigger-actions/conversation-agent-assign.ts

@@ -14,7 +14,7 @@ const isChangeMatching = (changedAssignee: string, assignChangeValue: string | n
 };
 
 const isTriggerChangeMatching = (productEventData: ProductEventData, triggerAction: TriggerAction): boolean => {
-  if (!productEventData.changes.model_changes.assigned_agent_id) {
+  if (!productEventData.changes?.model_changes?.assigned_agent_id) {
     return false;
   }
 

packages/rule-engine/src/recommended/trigger-actions/conversation-group-assign.ts

@@ -14,7 +14,7 @@ const isChangeMatching = (changedAssignee: string, assignChangeValue: string | n
 };
 
 const isTriggerChangeMatching = (productEventData: ProductEventData, triggerAction: TriggerAction): boolean => {
-  if (!productEventData.changes.model_changes.assigned_group_id) {
+  if (!productEventData.changes?.model_changes?.assigned_group_id) {
     return false;
   }
 

packages/rule-engine/src/recommended/trigger-actions/conversation-label-assign.ts

@@ -15,8 +15,8 @@ const isChangeMatching = (changedAssignee: string, assignChangeValue: string | n
 
 const isTriggerChangeMatching = (productEventData: ProductEventData, triggerAction: TriggerAction): boolean => {
   if (
-    !productEventData.changes.model_changes.label_category_id &&
-    !productEventData.changes.model_changes.label_subcategory_id
+    !productEventData.changes?.model_changes?.label_category_id &&
+    !productEventData.changes?.model_changes?.label_subcategory_id
   ) {
     return false;
   }