@primocaredentgroup/elettromedicali 0.1.0 → 0.1.1

package/dist/component/ticketTriggers.js

@@ -73,7 +73,7 @@ export const createTrigger = mutation({
             sendNotification: v.optional(v.object({
                 recipients: v.array(v.object({
                     type: v.union(v.literal('admin'), v.literal('supplier'), v.literal('ticket_creator'), v.literal('ticket_assignee'), v.literal('specific_user'), v.literal('specific_supplier')),
-                    userId: v.optional(v.id('user_profiles')),
+                    userId: v.optional(v.string()),
                     supplierId: v.optional(v.id('suppliers')),
                 })),
                 message: v.string(),
@@ -163,7 +163,7 @@ export const updateTrigger = mutation({
             sendNotification: v.optional(v.object({
                 recipients: v.array(v.object({
                     type: v.union(v.literal('admin'), v.literal('supplier'), v.literal('ticket_creator'), v.literal('ticket_assignee'), v.literal('specific_user'), v.literal('specific_supplier')),
-                    userId: v.optional(v.id('user_profiles')),
+                    userId: v.optional(v.string()),
                     supplierId: v.optional(v.id('suppliers')),
                 })),
                 message: v.string(),
@@ -259,17 +259,27 @@ export const reorderTriggers = mutation({
 });
 export const executeTriggers = mutation({
     args: {
-        ticketId: v.id('maintenance_tasks'),
+        ticketId: v.string(),
+        ticketData: v.object({
+            status: v.string(),
+            priority: v.optional(v.string()),
+            supplierId: v.optional(v.string()),
+            clinicId: v.optional(v.string()),
+            customFields: v.optional(v.any()),
+            notes: v.optional(v.string()),
+        }),
+        deviceData: v.optional(v.object({
+            category: v.optional(v.string()),
+            brand: v.optional(v.string()),
+            model: v.optional(v.string()),
+        })),
         triggerOn: v.union(v.literal('create'), v.literal('status_change'), v.literal('update'), v.literal('sla_warning'), v.literal('sla_breach')),
         oldStatus: v.optional(v.string()),
+        clinicRegion: v.optional(v.string()),
     },
     handler: async (ctx, args) => {
-        const ticket = await ctx.db.get(args.ticketId);
-        if (!ticket) {
-            throw new Error('Ticket not found');
-        }
-        const device = ticket.deviceId ? await ctx.db.get(ticket.deviceId) : null;
-        const clinic = await ctx.db.get(ticket.clinicId);
+        const ticket = args.ticketData;
+        const device = args.deviceData || null;
         const triggers = await ctx.db
             .query('ticket_triggers')
             .withIndex('by_isActive', (q) => q.eq('isActive', true))
@@ -283,39 +293,31 @@ export const executeTriggers = mutation({
                     result = value === args.triggerOn;
                     break;
                 case 'status':
-                    const statusValues = Array.isArray(value) ? value : [value];
-                    result = statusValues.includes(ticket.status);
+                    result = (Array.isArray(value) ? value : [value]).includes(ticket.status);
                     break;
                 case 'category':
-                    const categoryValues = Array.isArray(value) ? value : [value];
-                    result = device ? categoryValues.includes(device.category) : false;
+                    result = device ? (Array.isArray(value) ? value : [value]).includes(device.category) : false;
                     break;
                 case 'brand':
-                    const brandValues = Array.isArray(value) ? value : [value];
-                    result = device ? brandValues.includes(device.brand) : false;
+                    result = device ? (Array.isArray(value) ? value : [value]).includes(device.brand) : false;
                     break;
                 case 'model':
-                    const modelValues = Array.isArray(value) ? value : [value];
-                    result = device ? modelValues.includes(device.model) : false;
+                    result = device ? (Array.isArray(value) ? value : [value]).includes(device.model) : false;
                     break;
                 case 'region':
-                    const regionValues = Array.isArray(value) ? value : [value];
-                    const clinicRegion = clinic?.region;
-                    result = clinicRegion ? regionValues.includes(clinicRegion) : false;
+                    result = args.clinicRegion ? (Array.isArray(value) ? value : [value]).includes(args.clinicRegion) : false;
                     break;
                 case 'priority':
-                    const priorityValues = Array.isArray(value) ? value : [value];
-                    result = priorityValues.includes(ticket.priority);
+                    result = (Array.isArray(value) ? value : [value]).includes(ticket.priority);
                     break;
-                case 'custom_field':
+                case 'custom_field': {
                     const { customFieldId, customFieldOperator } = condition;
                     if (!customFieldId) {
                         result = true;
                         break;
                     }
                     const fieldValue = ticket.customFields?.[customFieldId];
-                    const operator = customFieldOperator || 'equals';
-                    switch (operator) {
+                    switch (customFieldOperator || 'equals') {
                         case 'equals':
                             result = fieldValue === value;
                             break;
@@ -337,10 +339,10 @@ export const executeTriggers = mutation({
                         case 'is_not_empty':
                             result = fieldValue !== undefined && fieldValue !== null && fieldValue !== '';
                             break;
-                        default:
-                            result = true;
+                        default: result = true;
                     }
                     break;
+                }
                 default:
                     result = true;
             }
@@ -356,177 +358,149 @@ export const executeTriggers = mutation({
             let currentGroup = 0;
             for (let i = 0; i < conditions.length; i++) {
                 andGroups[currentGroup].push(results[i]);
-                if (i < conditions.length - 1) {
-                    const nextOp = conditions[i].nextOperator || 'AND';
-                    if (nextOp === 'OR') {
-                        currentGroup++;
-                        andGroups[currentGroup] = [];
-                    }
+                if (i < conditions.length - 1 && (conditions[i].nextOperator || 'AND') === 'OR') {
+                    currentGroup++;
+                    andGroups[currentGroup] = [];
                 }
             }
-            const groupResults = andGroups.map(group => group.every(r => r));
-            return groupResults.some(r => r);
+            return andGroups.map(group => group.every(r => r)).some(r => r);
         };
+        const allActions = [];
+        let currentStatus = ticket.status;
+        let currentNotes = ticket.notes || '';
         for (const trigger of sortedTriggers) {
             const triggerConditions = trigger.conditions;
-            if (triggerConditions.triggerOn && triggerConditions.triggerOn !== args.triggerOn) {
+            if (triggerConditions.triggerOn && triggerConditions.triggerOn !== args.triggerOn)
                 continue;
-            }
+            let shouldExecute = false;
             if (triggerConditions.conditionsList && triggerConditions.conditionsList.length > 0) {
-                const shouldExecute = evaluateConditionsWithOperators(triggerConditions.conditionsList);
-                if (!shouldExecute) {
-                    continue;
-                }
+                shouldExecute = evaluateConditionsWithOperators(triggerConditions.conditionsList);
             }
             else {
-                if (trigger.conditions.triggerOn !== args.triggerOn) {
+                if (trigger.conditions.triggerOn !== args.triggerOn)
                     continue;
-                }
                 const conditionLogic = trigger.conditions.conditionLogic || 'AND';
                 const conditionResults = [];
                 let categoryMatch = true;
                 if (!trigger.conditions.applyToAllCategories && device) {
-                    if (!trigger.conditions.categories || !trigger.conditions.categories.includes(device.category)) {
+                    if (!trigger.conditions.categories || !device.category || !trigger.conditions.categories.includes(device.category))
                         categoryMatch = false;
-                    }
                 }
                 conditionResults.push(categoryMatch);
-                let brandMatch = true;
-                if (triggerConditions.brands && triggerConditions.brands.length > 0 && device) {
-                    const brandOperator = triggerConditions.brandOperator || 'is';
+                if (triggerConditions.brands?.length > 0 && device) {
                     const brandInList = triggerConditions.brands.includes(device.brand);
-                    if (brandOperator === 'is') {
-                        brandMatch = brandInList;
-                    }
-                    else {
-                        brandMatch = !brandInList;
-                    }
+                    conditionResults.push((triggerConditions.brandOperator || 'is') === 'is' ? brandInList : !brandInList);
                 }
-                conditionResults.push(brandMatch);
-                let modelMatch = true;
-                if (triggerConditions.models && triggerConditions.models.length > 0 && device) {
-                    const modelOperator = triggerConditions.modelOperator || 'is';
+                else
+                    conditionResults.push(true);
+                if (triggerConditions.models?.length > 0 && device) {
                     const modelInList = triggerConditions.models.includes(device.model);
-                    if (modelOperator === 'is') {
-                        modelMatch = modelInList;
-                    }
-                    else {
-                        modelMatch = !modelInList;
-                    }
+                    conditionResults.push((triggerConditions.modelOperator || 'is') === 'is' ? modelInList : !modelInList);
                 }
-                conditionResults.push(modelMatch);
+                else
+                    conditionResults.push(true);
                 let regionMatch = true;
                 if (trigger.conditions.applyToAllRegions === false) {
-                    const clinicRegion = clinic?.region;
-                    if (!clinicRegion || !trigger.conditions.regions || !trigger.conditions.regions.includes(clinicRegion)) {
+                    if (!args.clinicRegion || !trigger.conditions.regions?.includes(args.clinicRegion))
                         regionMatch = false;
-                    }
                 }
                 conditionResults.push(regionMatch);
-                let statusMatch = true;
-                if (trigger.conditions.statuses && trigger.conditions.statuses.length > 0) {
-                    const statusOperator = trigger.conditions.statusOperator || 'is';
-                    const statusInList = trigger.conditions.statuses.includes(ticket.status);
-                    if (statusOperator === 'is') {
-                        statusMatch = statusInList;
-                    }
-                    else {
-                        statusMatch = !statusInList;
-                    }
+                const statuses = trigger.conditions.statuses;
+                if (statuses && statuses.length > 0) {
+                    const statusInList = statuses.includes(currentStatus);
+                    conditionResults.push((trigger.conditions.statusOperator || 'is') === 'is' ? statusInList : !statusInList);
                 }
-                conditionResults.push(statusMatch);
-                let customFieldsMatch = true;
-                if (trigger.conditions.customFieldConditions && trigger.conditions.customFieldConditions.length > 0) {
-                    for (const condition of trigger.conditions.customFieldConditions) {
-                        const fieldValue = ticket.customFields?.[condition.fieldId];
-                        let conditionMet = false;
-                        switch (condition.operator) {
+                else
+                    conditionResults.push(true);
+                const cfConditions = trigger.conditions.customFieldConditions;
+                if (cfConditions && cfConditions.length > 0) {
+                    let cfMatch = true;
+                    for (const cond of cfConditions) {
+                        const fv = ticket.customFields?.[cond.fieldId];
+                        let met = false;
+                        switch (cond.operator) {
                             case 'equals':
-                                conditionMet = fieldValue === condition.value;
+                                met = fv === cond.value;
                                 break;
                             case 'not_equals':
-                                conditionMet = fieldValue !== condition.value;
+                                met = fv !== cond.value;
                                 break;
                             case 'contains':
-                                conditionMet = fieldValue && String(fieldValue).includes(String(condition.value));
+                                met = fv && String(fv).includes(String(cond.value));
                                 break;
                             case 'greater_than':
-                                conditionMet = fieldValue && Number(fieldValue) > Number(condition.value);
+                                met = fv && Number(fv) > Number(cond.value);
                                 break;
                             case 'less_than':
-                                conditionMet = fieldValue && Number(fieldValue) < Number(condition.value);
+                                met = fv && Number(fv) < Number(cond.value);
                                 break;
                             case 'is_empty':
-                                conditionMet = fieldValue === undefined || fieldValue === null || fieldValue === '';
+                                met = fv === undefined || fv === null || fv === '';
                                 break;
                             case 'is_not_empty':
-                                conditionMet = fieldValue !== undefined && fieldValue !== null && fieldValue !== '';
+                                met = fv !== undefined && fv !== null && fv !== '';
                                 break;
                         }
-                        if (!conditionMet) {
-                            customFieldsMatch = false;
+                        if (!met) {
+                            cfMatch = false;
                             break;
                         }
                     }
+                    conditionResults.push(cfMatch);
                 }
-                conditionResults.push(customFieldsMatch);
-                let shouldExecute = false;
-                if (conditionLogic === 'OR') {
-                    shouldExecute = conditionResults.some(result => result === true);
-                }
-                else {
-                    shouldExecute = conditionResults.every(result => result === true);
-                }
-                if (!shouldExecute) {
-                    continue;
-                }
+                else
+                    conditionResults.push(true);
+                shouldExecute = conditionLogic === 'OR'
+                    ? conditionResults.some(r => r)
+                    : conditionResults.every(r => r);
             }
+            if (!shouldExecute)
+                continue;
+            const triggerActions = trigger.actions;
             const updates = {};
-            const actions = trigger.actions;
-            if (actions.setPriority) {
-                updates.priority = actions.setPriority;
-            }
-            if (trigger.actions.changeStatus) {
-                updates.status = trigger.actions.changeStatus;
+            if (triggerActions.setPriority)
+                updates.priority = triggerActions.setPriority;
+            if (triggerActions.changeStatus) {
+                updates.status = triggerActions.changeStatus;
+                currentStatus = updates.status;
             }
-            if (trigger.actions.assignSupplier) {
-                updates.supplierId = trigger.actions.assignSupplier;
-            }
-            if (actions.applySlaRule && actions.assignSupplier) {
-                const priority = actions.setPriority || ticket.priority || 'medium';
+            if (triggerActions.assignSupplier)
+                updates.supplierId = triggerActions.assignSupplier;
+            if (triggerActions.applySlaRule && triggerActions.assignSupplier) {
+                const priority = triggerActions.setPriority || ticket.priority || 'medium';
                 const slaRule = await ctx.db
                     .query('sla_rules')
-                    .withIndex('by_supplierId_priority', (q) => q.eq('supplierId', actions.assignSupplier).eq('priority', priority))
+                    .withIndex('by_supplierId_priority', (q) => q.eq('supplierId', triggerActions.assignSupplier).eq('priority', priority))
                     .first();
-                if (slaRule) {
-                    const slaDeadline = Date.now() + (slaRule.resolutionTimeHours * 60 * 60 * 1000);
-                    updates.slaDeadline = slaDeadline;
-                }
+                if (slaRule)
+                    updates.slaDeadline = Date.now() + (slaRule.resolutionTimeHours * 60 * 60 * 1000);
             }
-            else if (trigger.actions.setSlaHours) {
-                const slaDeadline = Date.now() + (trigger.actions.setSlaHours * 60 * 60 * 1000);
-                updates.slaDeadline = slaDeadline;
+            else if (triggerActions.setSlaHours) {
+                updates.slaDeadline = Date.now() + (triggerActions.setSlaHours * 60 * 60 * 1000);
             }
-            if (trigger.actions.addNote) {
-                const currentNotes = ticket.notes || '';
-                const newNote = `[Trigger: ${trigger.name}] ${trigger.actions.addNote}`;
-                updates.notes = currentNotes ? `${currentNotes}\n\n${newNote}` : newNote;
+            if (triggerActions.addNote) {
+                const newNote = `[Trigger: ${trigger.name}] ${triggerActions.addNote}`;
+                currentNotes = currentNotes ? `${currentNotes}\n\n${newNote}` : newNote;
+                updates.notes = currentNotes;
             }
             if (Object.keys(updates).length > 0) {
-                updates.updated_at = Date.now();
-                await ctx.db.patch(args.ticketId, updates);
+                allActions.push({
+                    triggerId: trigger._id,
+                    triggerName: trigger.name,
+                    actions: updates,
+                });
                 try {
-                    const actionDescriptions = [];
+                    const desc = [];
                     if (updates.status)
-                        actionDescriptions.push(`Stato → ${updates.status}`);
+                        desc.push(`Stato → ${updates.status}`);
                     if (updates.supplierId) {
                         const supplier = await ctx.db.get(updates.supplierId);
-                        actionDescriptions.push(`Assegnato a ${supplier?.name || 'fornitore'}`);
+                        desc.push(`Assegnato a ${supplier?.name || 'fornitore'}`);
                     }
                     if (updates.priority)
-                        actionDescriptions.push(`Priorità → ${updates.priority}`);
+                        desc.push(`Priorità → ${updates.priority}`);
                     if (updates.slaDeadline)
-                        actionDescriptions.push(`SLA impostato`);
+                        desc.push(`SLA impostato`);
                     await ctx.runMutation(api.ticketHistory.recordHistoryEvent, {
                         ticketId: args.ticketId,
                         eventType: 'trigger_executed',
@@ -534,9 +508,7 @@ export const executeTriggers = mutation({
                         triggerId: trigger._id,
                         oldValue: updates.status ? ticket.status : undefined,
                         newValue: updates.status,
-                        oldSupplierId: updates.supplierId ? ticket.supplierId : undefined,
-                        newSupplierId: updates.supplierId,
-                        notes: actionDescriptions.join(', '),
+                        notes: desc.join(', '),
                         isSystemAction: true,
                     });
                 }
@@ -545,13 +517,14 @@ export const executeTriggers = mutation({
                 }
             }
         }
-        return { success: true };
+        return { success: true, actions: allActions };
     },
 });
 export const checkAttachmentRequired = query({
     args: {
         deviceId: v.optional(v.id('devices')),
-        clinicId: v.optional(v.id('clinics')),
+        clinicId: v.optional(v.string()),
+        clinicRegion: v.optional(v.string()),
         status: v.optional(v.string()),
         priority: v.optional(v.union(v.literal('low'), v.literal('medium'), v.literal('high'))),
         customFields: v.optional(v.any()),
@@ -567,7 +540,6 @@ export const checkAttachmentRequired = query({
                 return { required: false, matchingTriggers: [] };
             }
             let device = null;
-            let clinic = null;
             try {
                 if (args.deviceId) {
                     device = await ctx.db.get(args.deviceId);
@@ -576,14 +548,6 @@ export const checkAttachmentRequired = query({
             catch (e) {
                 console.error('Error fetching device:', e);
             }
-            try {
-                if (args.clinicId) {
-                    clinic = await ctx.db.get(args.clinicId);
-                }
-            }
-            catch (e) {
-                console.error('Error fetching clinic:', e);
-            }
             const evaluateCondition = (condition) => {
                 const { type, negated, value } = condition;
                 let result = false;
@@ -609,8 +573,7 @@ export const checkAttachmentRequired = query({
                         break;
                     case 'region':
                         const regionValues = Array.isArray(value) ? value : [value];
-                        const clinicRegion = clinic?.region;
-                        result = clinicRegion ? regionValues.includes(clinicRegion) : false;
+                        result = args.clinicRegion ? regionValues.includes(args.clinicRegion) : false;
                         break;
                     case 'priority':
                         const priorityValues = Array.isArray(value) ? value : [value];
@@ -707,8 +670,7 @@ export const checkAttachmentRequired = query({
                     }
                     let regionMatch = true;
                     if (triggerConditions.applyToAllRegions === false) {
-                        const clinicRegion = clinic?.region;
-                        if (!clinicRegion || !triggerConditions.regions || !triggerConditions.regions.includes(clinicRegion)) {
+                        if (!args.clinicRegion || !triggerConditions.regions || !triggerConditions.regions.includes(args.clinicRegion)) {
                             regionMatch = false;
                         }
                     }