@fayz-ai/plugin-agenda 0.9.0 → 0.9.1

package/src/migrations/014_agent_rpc_update_appointment.sql

@@ -1,199 +0,0 @@
--- ============================================================================
--- 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;