@primocaredentgroup/elettromedicali 0.1.0 → 0.1.1

package/src/component/schema.ts

@@ -20,30 +20,19 @@ export default defineSchema({
     createdAt: v.optional(v.number()),
   }).index('by_name', ['name']),
 
-  // User profiles with roles
+  // Domain-specific user settings (base user data comes from parent app)
   user_profiles: defineTable({
     auth0Id: v.string(),
-    roleId: v.optional(v.id('roles')), // Made optional for backward compatibility
-    role: v.optional(v.string()), // Legacy field - will be migrated to roleId
-    clinicId: v.optional(v.id('clinics')),
+    clinicId: v.optional(v.string()),
     supplierId: v.optional(v.id('suppliers')),
-    email: v.string(),
-    name: v.optional(v.string()),
-    // PrimoUP user clinics data
-    primoupClinics: v.optional(v.array(v.any())), // Array of clinic objects from PrimoUP
-    selectedClinicId: v.optional(v.number()), // Currently selected PrimoUP clinic ID
-    clinicChangeCounter: v.optional(v.number()), // Increments every time clinic changes
-    // Impersonation (admin only)
-    impersonatingUserId: v.optional(v.id('user_profiles')), // ID of user being impersonated
-    // Account status
-    isActive: v.optional(v.boolean()), // Default true, false = account disabilitato
+    primoupClinics: v.optional(v.array(v.any())),
+    selectedClinicId: v.optional(v.number()),
+    clinicChangeCounter: v.optional(v.number()),
+    impersonatingUserId: v.optional(v.string()),
   })
     .index('by_auth0Id', ['auth0Id'])
-    .index('by_email', ['email'])
-    .index('by_roleId', ['roleId'])
     .index('by_clinicId', ['clinicId'])
-    .index('by_supplierId', ['supplierId'])
-    .index('by_isActive', ['isActive']),
+    .index('by_supplierId', ['supplierId']),
 
   // PrimoUP authentication tokens
   primoup_tokens: defineTable({
@@ -72,19 +61,9 @@ export default defineSchema({
     .index('by_timestamp', ['timestamp'])
     .index('by_userId_timestamp', ['userId', 'timestamp']),
 
-  // Clinics
-  clinics: defineTable({
-    name: v.string(),
-    address: v.string(),
-    contact_email: v.string(),
-    contact_phone: v.string(),
-    region: v.optional(v.string()), // Regione della clinica
-    primoupId: v.optional(v.string()), // PrimoUP clinic ID for sync
-  }).index('by_primoupId', ['primoupId']),
-
   // Devices
   devices: defineTable({
-    clinicId: v.id('clinics'),
+    clinicId: v.string(),
     primoupClinicId: v.optional(v.number()), // PrimoUP clinic ID (for filtering)
     supplierId: v.optional(v.id('suppliers')), // Fornitore che fornisce assistenza
     name: v.string(),
@@ -145,8 +124,8 @@ export default defineSchema({
   // Storico riparazioni attrezzature
   device_repair_history: defineTable({
     deviceId: v.id('devices'),
-    clinicId: v.id('clinics'),
-    ticketId: v.optional(v.id('maintenance_tasks')),
+    clinicId: v.string(),
+    ticketId: v.optional(v.string()),
     sentToRepairAt: v.number(),
     returnedAt: v.optional(v.number()),
     statusAfterRepair: v.optional(v.union(
@@ -213,8 +192,8 @@ export default defineSchema({
   // Contracts
   contracts: defineTable({
     supplierId: v.id('suppliers'),
-    clinicId: v.optional(v.id('clinics')), // Legacy: single clinic (deprecated)
-    clinicIds: v.optional(v.array(v.id('clinics'))), // New: multiple clinics (empty = all clinics)
+    clinicId: v.optional(v.string()), // Legacy: single clinic (deprecated)
+    clinicIds: v.optional(v.array(v.string())), // Multiple clinics (empty = all clinics)
     start_date: v.number(),
     end_date: v.number(),
     amount: v.number(),
@@ -281,58 +260,21 @@ export default defineSchema({
     .index('by_applyToAll', ['applyToAll'])
     .index('by_parentFieldId', ['parentFieldId']),
 
-  // Maintenance tasks (tickets)
-  maintenance_tasks: defineTable({
-    ticketNumber: v.optional(v.string()), // Numero ticket incrementale (es: TICK-00001)
-    title: v.optional(v.string()), // Titolo del ticket
-    deviceId: v.optional(v.id('devices')), // Opzionale per ticket esterni senza device specifico
-    clinicId: v.id('clinics'),
-    supplierId: v.optional(v.id('suppliers')),
-    created_by: v.string(), // Auth0 user ID
-    createdByEmail: v.optional(v.string()), // Email del creatore (per webhook)
-    description: v.string(),
-    photos: v.array(v.string()), // Legacy base64 - kept for backward compatibility
-    photoStorageIds: v.optional(v.array(v.id('_storage'))), // New: Convex file storage
-    status: v.string(), // References ticket_statuses.name
-    priority: v.optional(v.union(
-      v.literal('low'),
-      v.literal('medium'),
-      v.literal('high')
-    )), // Priorità del ticket (usata per SLA)
-    notes: v.optional(v.string()), // Additional notes from supplier/admin
-    customFields: v.optional(v.any()), // Valori dei campi custom (oggetto dinamico)
+  // Ticket device data (links Abaddon ticket to component device/photos)
+  ticket_device_data: defineTable({
+    ticketId: v.string(), // Abaddon ticket _id (string reference)
+    deviceId: v.optional(v.id('devices')),
     deviceQuestionAnswers: v.optional(v.array(v.object({
       questionId: v.id('device_questions'),
-      question: v.string(), // Testo della domanda (salvato per storico)
-      answer: v.any(), // Risposta (può essere string, number, array, boolean)
-    }))), // Risposte alle domande specifiche dell'attrezzatura
-    slaDeadline: v.optional(v.number()), // Scadenza SLA calcolata automaticamente
-    slaBreached: v.optional(v.boolean()), // True se SLA è stato violato
-    slaWarningSent: v.optional(v.boolean()), // True se warning email è stata inviata
-    // External system integration (Abaddon)
-    externalTicketId: v.optional(v.string()), // ID del ticket nel sistema esterno
-    externalTicketNumber: v.optional(v.number()), // Numero ticket nel sistema esterno
-    // Import tracking
-    importSource: v.optional(v.string()), // Es: "legacy_zammad", "external_api" per ticket importati
-    legacyTicketId: v.optional(v.number()), // ID originale nel sistema legacy
-    // External ticket flags
-    isExternal: v.optional(v.boolean()), // True se ticket creato da API esterna (senza device)
-    needsAssignment: v.optional(v.boolean()), // True se ticket da gestire (senza fornitore assegnato)
-    created_at: v.number(),
-    updated_at: v.number(),
-    closed_at: v.optional(v.number()), // Data chiusura (quando status diventa completed/cancelled)
+      question: v.string(),
+      answer: v.any(),
+    }))),
+    photoStorageIds: v.optional(v.array(v.id('_storage'))),
+    isExternal: v.optional(v.boolean()),
+    needsAssignment: v.optional(v.boolean()),
   })
-    .index('by_ticketNumber', ['ticketNumber'])
-    .index('by_deviceId', ['deviceId'])
-    .index('by_clinicId', ['clinicId'])
-    .index('by_supplierId', ['supplierId'])
-    .index('by_status', ['status'])
-    .index('by_status_created_at', ['status', 'created_at'])
-    .index('by_priority', ['priority'])
-    .index('by_created_at', ['created_at'])
-    .index('by_slaDeadline', ['slaDeadline'])
-    .index('by_externalTicketId', ['externalTicketId'])
-    .index('by_importSource', ['importSource']),
+    .index('by_ticketId', ['ticketId'])
+    .index('by_deviceId', ['deviceId']),
 
   // Dashboard stats cache - pre-computed every 10 min to avoid bytes read limit
   dashboard_stats_cache: defineTable({
@@ -457,7 +399,7 @@ export default defineSchema({
       assignSupplier: v.optional(v.id('suppliers')),
       
       // Assegna utente specifico (utente con ruolo supplier)
-      assignUser: v.optional(v.id('user_profiles')),
+      assignUser: v.optional(v.string()),
       
       // Imposta SLA (ore)
       setSlaHours: v.optional(v.number()),
@@ -481,7 +423,7 @@ export default defineSchema({
               v.literal('specific_user'),      // Utente specifico
               v.literal('specific_supplier')   // Fornitore specifico
             ),
-            userId: v.optional(v.id('user_profiles')),    // Per specific_user
+            userId: v.optional(v.string()),    // Per specific_user
             supplierId: v.optional(v.id('suppliers')),    // Per specific_supplier
           }),
           // Vecchio formato (stringhe) - per retrocompatibilità temporanea
@@ -532,7 +474,7 @@ export default defineSchema({
     actions: v.object({
       changeStatus: v.optional(v.string()), // Cambia stato
       assignSupplier: v.optional(v.id('suppliers')), // Assegna azienda fornitrice
-      assignUser: v.optional(v.id('user_profiles')), // Assegna utente specifico
+      assignUser: v.optional(v.string()), // Assegna utente specifico
       setSlaHours: v.optional(v.number()), // Imposta SLA
       applySlaRule: v.optional(v.boolean()), // Applica regola SLA automatica
       addNote: v.optional(v.string()), // Aggiungi nota automatica
@@ -547,7 +489,7 @@ export default defineSchema({
               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')),
           }),
           v.literal('admin'),
@@ -561,7 +503,7 @@ export default defineSchema({
     
     createdAt: v.number(),
     updatedAt: v.number(),
-    createdBy: v.id('user_profiles'), // Chi ha creato la macro
+    createdBy: v.string(), // Chi ha creato la macro
   })
     .index('by_createdBy', ['createdBy'])
     .index('by_isGlobal', ['isGlobal'])
@@ -700,7 +642,7 @@ export default defineSchema({
 
   // Ticket comments (Commenti sulle segnalazioni)
   ticket_comments: defineTable({
-    taskId: v.id('maintenance_tasks'), // Segnalazione a cui appartiene
+    ticketId: v.string(), // Abaddon ticket _id (string reference)
     authorId: v.string(), // Auth0 user ID dell'autore
     authorName: v.optional(v.string()), // Nome dell'autore (cached)
     authorEmail: v.optional(v.string()), // Email dell'autore (cached)
@@ -710,16 +652,16 @@ export default defineSchema({
     isFromApi: v.optional(v.boolean()), // Se true, commento ricevuto via API esterna
     attachments: v.optional(v.array(v.id('_storage'))), // Allegati opzionali
     mentions: v.optional(v.array(v.object({
-      userId: v.id('user_profiles'),
+      userId: v.string(),
       name: v.string(),
       email: v.string(),
     }))), // Utenti taggati nel commento
     createdAt: v.number(),
     updatedAt: v.optional(v.number()),
   })
-    .index('by_taskId', ['taskId'])
+    .index('by_ticketId', ['ticketId'])
     .index('by_authorId', ['authorId'])
-    .index('by_taskId_createdAt', ['taskId', 'createdAt']),
+    .index('by_ticketId_createdAt', ['ticketId', 'createdAt']),
 
   // Spare part orders (Ordini ricambi)
   spare_part_orders: defineTable({
@@ -786,7 +728,7 @@ export default defineSchema({
     to: v.union(v.string(), v.array(v.string())),
     subject: v.string(),
     html: v.string(),
-    ticketId: v.optional(v.id('maintenance_tasks')),
+    ticketId: v.optional(v.string()),
     status: v.union(
       v.literal('sent'),
       v.literal('failed'),
@@ -802,7 +744,7 @@ export default defineSchema({
 
   // Ticket History (Storico eventi ticket)
   ticket_history: defineTable({
-    ticketId: v.id('maintenance_tasks'),
+    ticketId: v.string(), // Abaddon ticket _id (string reference)
     eventType: v.union(
       v.literal('created'),           // Ticket creato
       v.literal('status_change'),     // Cambio stato

package/src/component/slaMonitoring.ts

@@ -1,125 +1,70 @@
 import { v } from 'convex/values';
-import { internalMutation, internalAction, internalQuery } from './_generated/server';
-import { internal } from './_generated/api';
+import { mutation, query } from './_generated/server';
 
-// NOTE: Notification sending (sendSLABreachNotifications, sendSLAWarningNotifications)
-// is NOT included in the component. The parent app is responsible for:
-// 1. Running checkSLAStatus periodically (via cron)
-// 2. After breaches/warnings are marked, querying for newly marked tickets
-// 3. Sending email notifications via its own emailActions
-
-export const checkSLAStatus = internalAction({
-  args: {},
-  handler: async (ctx): Promise<{ checked: number; warnings: number; breaches: number }> => {
-    try {
-      const tickets: any = await ctx.runQuery(internal.slaMonitoring.listTicketsNeedingSLACheck, {});
-
-      const now = Date.now();
-      let warningsCount = 0;
-      let breachesCount = 0;
-
-      for (const ticket of tickets) {
-        const timeUntilDeadline = ticket.slaDeadline - now;
-        const oneHour = 60 * 60 * 1000;
-
-        if (timeUntilDeadline < 0 && !ticket.slaBreached) {
-          breachesCount++;
-
-          await ctx.runMutation(internal.slaMonitoring.markSLABreached, {
-            ticketId: ticket._id,
-          });
-        }
-        else if (timeUntilDeadline > 0 && timeUntilDeadline < oneHour && !ticket.slaWarningSent) {
-          warningsCount++;
-
-          await ctx.runMutation(internal.slaMonitoring.markSLAWarningSent, {
-            ticketId: ticket._id,
-          });
-        }
-      }
-
-      return {
-        checked: tickets.length,
-        warnings: warningsCount,
-        breaches: breachesCount,
-      };
-    } catch (error) {
-      console.error('Error checking SLA status:', error);
-      throw error;
-    }
+export const evaluateSLA = query({
+  args: {
+    tickets: v.array(v.object({
+      ticketId: v.string(),
+      status: v.string(),
+      slaDeadline: v.optional(v.number()),
+      slaBreached: v.optional(v.boolean()),
+      slaWarningSent: v.optional(v.boolean()),
+    })),
   },
-});
-
-export const listTicketsNeedingSLACheck = internalQuery({
-  args: {},
-  handler: async (ctx) => {
+  handler: async (ctx, args) => {
+    const now = Date.now();
+    const oneHour = 60 * 60 * 1000;
     const terminalStatuses = ['closed', 'completed', 'cancelled', 'resolved'];
 
-    const ticketsWithSLA = await ctx.db
-      .query('maintenance_tasks')
-      .withIndex('by_slaDeadline')
-      .order('asc')
-      .take(150);
+    const warnings: string[] = [];
+    const breaches: string[] = [];
 
-    return ticketsWithSLA
-      .filter(t => t.slaDeadline && !terminalStatuses.includes(t.status))
-      .map(t => ({
-        _id: t._id,
-        ticketNumber: t.ticketNumber,
-        status: t.status,
-        slaDeadline: t.slaDeadline,
-        slaBreached: t.slaBreached,
-        slaWarningSent: t.slaWarningSent,
-      }));
-  },
-});
+    for (const ticket of args.tickets) {
+      if (!ticket.slaDeadline || terminalStatuses.includes(ticket.status)) continue;
 
-export const listAllTicketsForSLA = internalQuery({
-  args: {},
-  handler: async (ctx) => {
-    const terminalStatuses = ['closed', 'completed', 'cancelled', 'resolved'];
+      const timeUntilDeadline = ticket.slaDeadline - now;
 
-    const tasks = await ctx.db
-      .query('maintenance_tasks')
-      .withIndex('by_slaDeadline')
-      .order('asc')
-      .take(500);
+      if (timeUntilDeadline < 0 && !ticket.slaBreached) {
+        breaches.push(ticket.ticketId);
+      } else if (timeUntilDeadline > 0 && timeUntilDeadline < oneHour && !ticket.slaWarningSent) {
+        warnings.push(ticket.ticketId);
+      }
+    }
 
-    return tasks
-      .filter(t => t.slaDeadline && !terminalStatuses.includes(t.status))
-      .map(t => ({
-        _id: t._id,
-        ticketNumber: t.ticketNumber,
-        status: t.status,
-        supplierId: t.supplierId,
-        slaDeadline: t.slaDeadline,
-        slaBreached: t.slaBreached,
-        slaWarningSent: t.slaWarningSent,
-        deviceName: '',
-        clinicName: '',
-        supplierName: null,
-      }));
+    return { warnings, breaches };
   },
 });
 
-export const markSLABreached = internalMutation({
+export const computeSLADeadline = query({
   args: {
-    ticketId: v.id('maintenance_tasks'),
+    supplierId: v.optional(v.id('suppliers')),
+    priority: v.optional(v.string()),
+    deviceCategory: v.optional(v.string()),
   },
   handler: async (ctx, args) => {
-    await ctx.db.patch(args.ticketId, {
-      slaBreached: true,
-    });
-  },
-});
+    if (!args.supplierId || !args.priority) return null;
+
+    const slaRule = await ctx.db
+      .query('sla_rules')
+      .withIndex('by_supplierId_category_priority', (q: any) =>
+        q.eq('supplierId', args.supplierId)
+          .eq('deviceCategory', args.deviceCategory || undefined)
+          .eq('priority', args.priority)
+      )
+      .first();
+
+    if (!slaRule) {
+      const genericRule = await ctx.db
+        .query('sla_rules')
+        .withIndex('by_supplierId_priority', (q: any) =>
+          q.eq('supplierId', args.supplierId).eq('priority', args.priority)
+        )
+        .first();
+
+      if (!genericRule) return null;
+      return Date.now() + (genericRule.resolutionTimeHours * 60 * 60 * 1000);
+    }
 
-export const markSLAWarningSent = internalMutation({
-  args: {
-    ticketId: v.id('maintenance_tasks'),
-  },
-  handler: async (ctx, args) => {
-    await ctx.db.patch(args.ticketId, {
-      slaWarningSent: true,
-    });
+    return Date.now() + (slaRule.resolutionTimeHours * 60 * 60 * 1000);
   },
 });

package/src/component/spareParts.ts

@@ -69,18 +69,49 @@ export const importSparePartsBatch = mutation({
 
 export const listSpareParts = query({
   args: {
-    limit: v.optional(v.number()),
+    page: v.optional(v.number()),
+    pageSize: v.optional(v.number()),
+    search: v.optional(v.string()),
+    clinicId: v.optional(v.number()),
+    userRole: v.optional(v.string()),
+    userSelectedClinicId: v.optional(v.number()),
     _triggerReload: v.optional(v.number()),
   },
   handler: async (ctx, args) => {
-    const limit = args.limit || 1000;
+    const page = args.page || 1;
+    const pageSize = args.pageSize || 40;
 
-    const spareParts = await ctx.db
-      .query('spare_parts')
-      .order('desc')
-      .take(limit);
+    let allParts: any[];
+    const effectiveClinicId = args.clinicId || args.userSelectedClinicId;
 
-    return spareParts;
+    if (effectiveClinicId) {
+      allParts = await ctx.db
+        .query('spare_parts')
+        .withIndex('by_clinicId', (q: any) => q.eq('clinicId', effectiveClinicId))
+        .collect();
+    } else {
+      allParts = await ctx.db
+        .query('spare_parts')
+        .order('desc')
+        .collect();
+    }
+
+    if (args.search) {
+      const s = args.search.toLowerCase();
+      allParts = allParts.filter((p: any) =>
+        p.name?.toLowerCase().includes(s) ||
+        p.code?.toLowerCase().includes(s) ||
+        p.articleName?.toLowerCase().includes(s) ||
+        p.brand?.toLowerCase().includes(s)
+      );
+    }
+
+    const totalCount = allParts.length;
+    const totalPages = Math.ceil(totalCount / pageSize);
+    const startIndex = (page - 1) * pageSize;
+    const pageParts = allParts.slice(startIndex, startIndex + pageSize);
+
+    return { page: pageParts, totalCount, totalPages, currentPage: page };
   },
 });
 
@@ -94,7 +125,7 @@ export const listSparePartsByClinic = query({
 
     const spareParts = await ctx.db
       .query('spare_parts')
-      .withIndex('by_clinicId', (q) => q.eq('clinicId', args.clinicId))
+      .withIndex('by_clinicId', (q: any) => q.eq('clinicId', args.clinicId))
       .take(limit);
 
     return spareParts;
@@ -168,9 +199,12 @@ export const listSparePartsPaginated = query({
       filteredQuery = filteredQuery.filter((q: any) => q.eq(q.field('articleCategoryId'), args.categoryId));
     }
 
-    const result = await filteredQuery.paginate(args.paginationOpts);
+    const limit = args.paginationOpts.numItems + 1;
+    const allResults = await filteredQuery.take(limit);
+    const isDone = allResults.length < limit;
+    const page = isDone ? allResults : allResults.slice(0, -1);
 
-    let filteredPage = result.page;
+    let filteredPage = page;
 
     if (args.searchTerm) {
       const search = args.searchTerm.toLowerCase();
@@ -199,8 +233,8 @@ export const listSparePartsPaginated = query({
 
     return {
       page: minimalPage,
-      isDone: result.isDone,
-      continueCursor: result.continueCursor,
+      isDone,
+      continueCursor: '',
     };
   },
 });