@fayz-ai/plugin-agenda 0.8.4 → 0.9.0-next.1

package/src/migrations/index.ts

@@ -1,4 +1,4 @@
-// AUTO-GENERATED from 000b_v_bookings_rename.sql, 000c_dedupe_customer_phones.sql, 001_public_booking.sql, 002_v_bookings_compat.sql, 003_public_client_kind.sql, 004_public_booking_assignee.sql, 005_agent_rpcs.sql, 006_agent_rpc_localtime.sql, 007_schedule_exception_precedence.sql, 008_agent_rpc_exception_precedence.sql, 009_public_booking_location.sql, 010_public_service_curation.sql, 011_default_assignee.sql, 012_drop_legacy_booking_overload.sql, 013_normalize_v_appointments.sql — regenerate with scripts/embed-migrations.mjs
+// AUTO-GENERATED from 000b_v_bookings_rename.sql, 000c_dedupe_customer_phones.sql, 001_public_booking.sql, 002_v_bookings_compat.sql, 003_public_client_kind.sql, 004_public_booking_assignee.sql, 005_agent_rpcs.sql, 006_agent_rpc_localtime.sql, 007_schedule_exception_precedence.sql, 008_agent_rpc_exception_precedence.sql, 009_public_booking_location.sql, 010_public_service_curation.sql, 011_default_assignee.sql, 012_drop_legacy_booking_overload.sql, 013_normalize_v_appointments.sql, 014_agent_rpc_update_appointment.sql — regenerate with scripts/embed-migrations.mjs
 // SQL files are the source of truth; this inline copy lets the manifest declare
 // migrations as data. Do not edit by hand — run the embed script instead.
 
@@ -2156,6 +2156,207 @@ DROP VIEW IF EXISTS public.v_bookings;
 DROP VIEW IF EXISTS public.v_bookings_legacy_beauty;
 `
 
+export const MIGRATION_014_AGENT_RPC_UPDATE_APPOINTMENT = `-- ============================================================================
+-- plugin-agenda 014: agent_agenda_update_appointment — remarcar / reatribuir /
+-- cancelar um agendamento existente.
+--
+-- Lacuna que isso fecha: o agente sabia CRIAR e não sabia MEXER. Pedido real
+-- ("remarca esse paciente pra 16:30") caía no updateRecord genérico, que bate
+-- em v_appointments — uma view — e devolve read_only; o modelo então "resolvia"
+-- abrindo um agendamento NOVO, deixando o antigo de pé. Duplicar a agenda de um
+-- salão é pior do que não fazer nada.
+--
+-- Mesma cadeia de guardas do 008 (agent_guard → janela de trabalho com exceções
+-- → lock advisory → checagem de conflito → auditoria), com três diferenças:
+--   • ação 'edit', sem cota (remarcar não consome bookings_month);
+--   • o próprio agendamento é excluído da checagem de conflito consigo mesmo;
+--   • cancelar não precisa de horário livre — só o guard.
+-- Requer 007 (agenda_working_windows). Idempotente.
+-- ============================================================================
+
+CREATE OR REPLACE FUNCTION public.agent_agenda_update_appointment(
+  p_tenant_id uuid,
+  p_actor_user_id uuid,
+  p_payload jsonb
+) RETURNS jsonb
+LANGUAGE plpgsql SECURITY DEFINER
+SET search_path = public
+AS $$
+DECLARE
+  v_denial jsonb;
+  v_id uuid;
+  v_tz text;
+  v_row record;
+  v_starts_raw text;
+  v_starts timestamptz;
+  v_ends timestamptz;
+  v_duration int;
+  v_professional uuid;
+  v_status text;
+  v_notes text;
+  v_wants_move boolean;
+  v_wants_status boolean;
+BEGIN
+  SELECT COALESCE(t.settings->>'timezone', 'America/Sao_Paulo') INTO v_tz
+  FROM tenants t WHERE t.id = p_tenant_id;
+  IF v_tz IS NULL THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'unknown tenant');
+  END IF;
+
+  -- ── payload ──────────────────────────────────────────────────────────────
+  BEGIN
+    v_id           := (p_payload->>'appointment_id')::uuid;
+    v_professional := (p_payload->>'professional_id')::uuid;
+    v_starts_raw   := p_payload->>'starts_at';
+  EXCEPTION WHEN OTHERS THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'invalid payload: ' || SQLERRM);
+  END;
+  v_status := nullif(trim(lower(p_payload->>'status')), '');
+  v_notes  := left(p_payload->>'notes', 500);
+  v_wants_move   := v_starts_raw IS NOT NULL OR v_professional IS NOT NULL;
+  v_wants_status := v_status IS NOT NULL;
+
+  IF v_id IS NULL THEN
+    RETURN jsonb_build_object('ok', false, 'error',
+      'appointment_id is required — search the agenda first and reuse the id of the appointment to change');
+  END IF;
+  IF NOT v_wants_move AND NOT v_wants_status AND v_notes IS NULL THEN
+    RETURN jsonb_build_object('ok', false, 'error',
+      'nothing to change: send starts_at, professional_id, status or notes');
+  END IF;
+  IF v_status IS NOT NULL
+     AND v_status NOT IN ('scheduled', 'confirmed', 'in_progress', 'completed', 'cancelled', 'no_show') THEN
+    RETURN jsonb_build_object('ok', false, 'error',
+      'status must be one of scheduled, confirmed, in_progress, completed, cancelled, no_show');
+  END IF;
+
+  -- ── the appointment must exist, in THIS tenant ───────────────────────────
+  SELECT b.id, b.starts_at, b.ends_at, b.assignee_id, b.status, b.party_id, b.order_id
+    INTO v_row
+  FROM appointments b
+  WHERE b.id = v_id AND b.tenant_id = p_tenant_id;
+  IF NOT FOUND THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'unknown appointment for this tenant');
+  END IF;
+
+  -- ── authorization: role → plan. No cap: editing frees nothing and buys
+  --    nothing, so charging bookings_month for a reschedule would be wrong.
+  v_denial := agent_guard(p_tenant_id, p_actor_user_id, 'appointments', 'edit');
+  IF v_denial IS NOT NULL THEN
+    RETURN jsonb_build_object('ok', false, 'denial', v_denial);
+  END IF;
+
+  -- Duration is preserved across a move — a 60-minute appointment stays 60.
+  v_duration := GREATEST(
+    COALESCE(EXTRACT(EPOCH FROM (v_row.ends_at - v_row.starts_at)) / 60, 30)::int, 5);
+  v_professional := COALESCE(v_professional, v_row.assignee_id);
+
+  IF v_wants_move THEN
+    IF v_starts_raw IS NOT NULL THEN
+      BEGIN
+        IF v_starts_raw ~ '(Z|[+-][0-9]{2}:?[0-9]{2})$' THEN
+          v_starts := v_starts_raw::timestamptz;               -- explicit offset
+        ELSE
+          v_starts := (v_starts_raw::timestamp) AT TIME ZONE v_tz;  -- tenant-local
+        END IF;
+      EXCEPTION WHEN OTHERS THEN
+        RETURN jsonb_build_object('ok', false, 'error', 'invalid starts_at: ' || SQLERRM);
+      END;
+    ELSE
+      v_starts := v_row.starts_at;
+    END IF;
+    v_ends := v_starts + make_interval(mins => v_duration);
+
+    IF v_starts > now() + interval '365 days' THEN
+      RETURN jsonb_build_object('ok', false, 'error', 'starts_at must be within the next year');
+    END IF;
+
+    -- Working window (exception-aware, 007) for the target professional.
+    IF NOT EXISTS (
+      SELECT 1
+      FROM agenda_working_windows(
+             p_tenant_id, (v_starts AT TIME ZONE v_tz)::date, v_professional) w
+      WHERE w.assignee_id = v_professional
+        AND (v_starts AT TIME ZONE v_tz)::time >= w.win_start
+        AND (v_ends   AT TIME ZONE v_tz)::time <= w.win_end
+    ) THEN
+      RETURN jsonb_build_object('ok', false, 'error',
+        'that slot is outside the professional''s working hours. Suggest a different time. Times are tenant-local, tz ' || v_tz);
+    END IF;
+
+    PERFORM pg_advisory_xact_lock(hashtextextended(p_tenant_id::text || v_professional::text, 0));
+    IF EXISTS (
+      SELECT 1 FROM appointments b
+      WHERE b.tenant_id = p_tenant_id
+        AND b.assignee_id = v_professional
+        AND b.id <> v_id                     -- never conflicts with itself
+        AND b.status NOT IN ('cancelled', 'no_show')
+        AND b.starts_at < v_ends
+        AND COALESCE(b.ends_at, b.starts_at) > v_starts
+    ) THEN
+      RETURN jsonb_build_object('ok', false, 'error',
+        'that slot is already taken. Suggest a different time.');
+    END IF;
+  END IF;
+
+  -- ── write + audit ────────────────────────────────────────────────────────
+  UPDATE appointments b SET
+    starts_at   = COALESCE(v_starts, b.starts_at),
+    ends_at     = COALESCE(v_ends, b.ends_at),
+    assignee_id = v_professional,
+    status      = COALESCE(v_status, b.status),
+    notes       = COALESCE(v_notes, b.notes),
+    metadata    = COALESCE(b.metadata, '{}'::jsonb) || jsonb_build_object('lastAgentEdit', now()),
+    updated_at  = now()
+  WHERE b.id = v_id AND b.tenant_id = p_tenant_id;
+
+  -- The order mirrors the appointment's assignee/status so the financial side
+  -- does not drift from the calendar.
+  IF v_row.order_id IS NOT NULL THEN
+    UPDATE orders o SET
+      assignee_id = v_professional,
+      status      = CASE WHEN v_status = 'cancelled' THEN 'cancelled' ELSE o.status END,
+      updated_at  = now()
+    WHERE o.id = v_row.order_id AND o.tenant_id = p_tenant_id;
+  END IF;
+
+  INSERT INTO audit_logs (tenant_id, user_id, action, entity_type, entity_id, metadata)
+  VALUES (p_tenant_id, p_actor_user_id, 'agent.updateAppointment', 'appointment', v_id::text,
+          jsonb_build_object('payload', p_payload,
+                             'previous', jsonb_build_object(
+                               'starts_at', v_row.starts_at,
+                               'ends_at', v_row.ends_at,
+                               'assignee_id', v_row.assignee_id,
+                               'status', v_row.status)));
+
+  SELECT a.id, a.starts_at, a.ends_at, a.status, a.client_name, a.professional_name
+    INTO v_row
+  FROM v_appointments a WHERE a.id = v_id;
+
+  RETURN jsonb_build_object(
+    'ok', true,
+    'id', v_id,
+    'record', jsonb_build_object(
+      'ref', jsonb_build_object('id', v_id, 'resource', 'appointments',
+                                'archetype', 'schedule:appointment'),
+      'starts_at', v_row.starts_at,
+      'ends_at', v_row.ends_at,
+      'starts_at_local', to_char(v_row.starts_at AT TIME ZONE v_tz, 'YYYY-MM-DD (Dy) HH24:MI') || ' ' || v_tz,
+      'ends_at_local', to_char(v_row.ends_at AT TIME ZONE v_tz, 'HH24:MI'),
+      'client_name', v_row.client_name,
+      'professional_name', v_row.professional_name,
+      'status', v_row.status
+    )
+  );
+END;
+$$;
+
+REVOKE ALL ON FUNCTION public.agent_agenda_update_appointment(uuid, uuid, jsonb) FROM public;
+REVOKE EXECUTE ON FUNCTION public.agent_agenda_update_appointment(uuid, uuid, jsonb) FROM anon;
+GRANT EXECUTE ON FUNCTION public.agent_agenda_update_appointment(uuid, uuid, jsonb)
+  TO authenticated, service_role;
+`
+
 export const MIGRATIONS: Array<{ id: string; sql: string }> = [
   { id: "000b_v_bookings_rename", sql: MIGRATION_000B_V_BOOKINGS_RENAME },
   { id: "000c_dedupe_customer_phones", sql: MIGRATION_000C_DEDUPE_CUSTOMER_PHONES },
@@ -2172,4 +2373,5 @@ export const MIGRATIONS: Array<{ id: string; sql: string }> = [
   { id: "011_default_assignee", sql: MIGRATION_011_DEFAULT_ASSIGNEE },
   { id: "012_drop_legacy_booking_overload", sql: MIGRATION_012_DROP_LEGACY_BOOKING_OVERLOAD },
   { id: "013_normalize_v_appointments", sql: MIGRATION_013_NORMALIZE_V_APPOINTMENTS },
+  { id: "014_agent_rpc_update_appointment", sql: MIGRATION_014_AGENT_RPC_UPDATE_APPOINTMENT },
 ]

package/src/store.ts

@@ -1,4 +1,5 @@
 import { createStore, type StoreApi } from 'zustand/vanilla'
+import { emitDataChanged } from '@fayz-ai/core'
 import { dedup } from '@fayz-ai/saas'
 import { toast } from '@fayz-ai/ui'
 import { getScheduleBlockConfig, setScheduleBlockConfig } from '@fayz-ai/saas'
@@ -181,6 +182,7 @@ export function createAgendaStore(provider: AgendaDataProvider, financialBridge?
           const bookings = await provider.getBookings(getCurrentBookingQuery(state))
           set({ bookings })
         }
+        emitDataChanged({ table: 'appointments', op: 'create', id: booking.id, source: 'ui' })
         toast.success('Appointment created')
         return booking
       } catch (err: any) {
@@ -197,6 +199,7 @@ export function createAgendaStore(provider: AgendaDataProvider, financialBridge?
           const bookings = await provider.getBookings(getCurrentBookingQuery(state))
           set({ bookings })
         }
+        emitDataChanged({ table: 'appointments', op: 'update', id, source: 'ui' })
         toast.success('Appointment updated')
         return booking
       } catch (err: any) {
@@ -213,6 +216,7 @@ export function createAgendaStore(provider: AgendaDataProvider, financialBridge?
           const bookings = await provider.getBookings(getCurrentBookingQuery(state))
           set({ bookings })
         }
+        emitDataChanged({ table: 'appointments', op: 'delete', id, source: 'ui' })
         toast.success('Appointment deleted')
       } catch (err: any) {
         toast.error('Failed to delete appointment', { description: err?.message })
@@ -232,6 +236,7 @@ export function createAgendaStore(provider: AgendaDataProvider, financialBridge?
           const bookings = await provider.getBookings(getCurrentBookingQuery(state))
           set({ bookings })
         }
+        emitDataChanged({ table: 'appointments', op: 'update', id, source: 'ui' })
         toast.success('Status updated')
       } catch (err: any) {
         // Rollback on failure

package/src/views/CalendarView.tsx

@@ -633,6 +633,34 @@ export function CalendarView() {
   // Alternating resource backgrounds — apply via DOM after render
   const wrapperRef = useRef<HTMLDivElement>(null)
 
+  // FullCalendar measures its columns once and only re-measures on a WINDOW
+  // resize. Opening/closing the assistant rail (or the sidebar) resizes the
+  // container without one, so the grid stayed locked at the narrow width until
+  // a real window resize came along. Watch the container and hand FC the new box.
+  useEffect(() => {
+    const container = wrapperRef.current
+    if (!container) return
+    let frame = 0
+    let lastWidth = container.clientWidth
+    let lastHeight = container.clientHeight
+    const ro = new ResizeObserver(() => {
+      const width = container.clientWidth
+      const height = container.clientHeight
+      if (width === lastWidth && height === lastHeight) return
+      lastWidth = width
+      lastHeight = height
+      // Next frame, not inline: measuring inside the observer callback reads a
+      // mid-layout box (and trips the RO loop warning). The rail animates, so
+      // this fires on each frame and the last one settles on the final width.
+      cancelAnimationFrame(frame)
+      frame = requestAnimationFrame(() => calendarRef.current?.getApi()?.updateSize())
+    })
+    ro.observe(container)
+    return () => { cancelAnimationFrame(frame); ro.disconnect() }
+    // Re-attach whenever the calendar branch remounts (skeleton → grid, mobile
+    // list → grid), since the observed node is gone on the other branches.
+  }, [isMobile, mobileGridActive, professionalsLoading, professionals.length])
+
   // Responsive day headers — collapse weekday names to a single initial (S M T
   // W T F S, Google-Calendar style) when day columns get too narrow. Measured
   // from the actual rendered column width so it adapts to resource count + view.