@fayz-ai/plugin-agenda 0.8.1 → 0.8.4

package/src/migrations/007_schedule_exception_precedence.sql

@@ -0,0 +1,290 @@
+-- ============================================================================
+-- plugin-agenda 007: specific-date exceptions take PRECEDENCE over weekly hours.
+--
+-- The admin ScheduleEditor (saveException) writes two kinds of specific-date
+-- rows: 'available' = active rows with custom windows for the date; day off
+-- ('unavailable') = ONE inactive 00:00–00:00 sentinel. Until now every reader
+-- treated schedules as a flat union filtered by is_active, so:
+--   • a day off was INVISIBLE (inactive sentinel filtered out, weekly rows
+--     still matched) — the public site kept selling a day the admin closed;
+--   • an 'available' exception ADDED to the weekly hours instead of replacing
+--     them ("horário especial" should define the date's hours).
+--
+-- Fix: one helper — agenda_working_windows(tenant, date, assignee) — is now the
+-- single source of a day's windows: if ANY specific-date row exists for an
+-- assignee (active or sentinel), only the ACTIVE specific-date rows count for
+-- that assignee; otherwise the weekly (day_of_week) rows apply. Both readers
+-- are redefined on top of it:
+--   • get_available_slots — body otherwise identical to the latest version
+--     (google-calendar/002_smart_sync §5, tenant-wide 'block' clause KEPT);
+--   • create_public_booking — body otherwise identical to 004.
+--
+-- ORDERING: on pools with the Google Calendar connector this file must run
+-- AFTER integrations/google-calendar/migrations/002_smart_sync.sql (both
+-- redefine get_available_slots; 007 is the superset and must win). Re-running
+-- 007 last is always safe — it preserves 002's block semantics.
+-- ============================================================================
+
+-- ----------------------------------------------------------------------------
+-- §1  agenda_working_windows — the day's working windows, exception-aware.
+-- ----------------------------------------------------------------------------
+CREATE OR REPLACE FUNCTION public.agenda_working_windows(
+  p_tenant_id uuid,
+  p_date date,
+  p_assignee_id uuid DEFAULT NULL
+) RETURNS TABLE (assignee_id uuid, win_start time, win_end time)
+LANGUAGE sql STABLE
+SET search_path = public
+AS $$
+  WITH exc AS (
+    -- Every specific-date row for the date, ACTIVE OR NOT: the inactive day-off
+    -- sentinel must still suppress the weekly rows below.
+    SELECT sc.assignee_id, sc.starts_at, sc.ends_at, sc.is_active
+    FROM schedules sc
+    WHERE sc.tenant_id = p_tenant_id
+      AND sc.kind = 'working_hours'
+      AND sc.specific_date = p_date
+      AND (p_assignee_id IS NULL OR sc.assignee_id = p_assignee_id)
+  )
+  SELECT e.assignee_id, e.starts_at, e.ends_at
+  FROM exc e
+  WHERE e.is_active
+  UNION ALL
+  SELECT sc.assignee_id, sc.starts_at, sc.ends_at
+  FROM schedules sc
+  WHERE sc.tenant_id = p_tenant_id
+    AND sc.kind = 'working_hours'
+    AND sc.is_active
+    AND sc.specific_date IS NULL
+    AND sc.day_of_week = EXTRACT(dow FROM p_date)::smallint
+    AND (p_assignee_id IS NULL OR sc.assignee_id = p_assignee_id)
+    AND NOT EXISTS (SELECT 1 FROM exc e WHERE e.assignee_id = sc.assignee_id)
+$$;
+
+REVOKE ALL ON FUNCTION public.agenda_working_windows(uuid, date, uuid) FROM public;
+REVOKE EXECUTE ON FUNCTION public.agenda_working_windows(uuid, date, uuid) FROM anon;
+GRANT EXECUTE ON FUNCTION public.agenda_working_windows(uuid, date, uuid)
+  TO authenticated, service_role;
+
+-- ----------------------------------------------------------------------------
+-- §2  get_available_slots — day_schedules now comes from the helper.
+--     Everything else is byte-identical to google-calendar/002_smart_sync §5.
+-- ----------------------------------------------------------------------------
+CREATE OR REPLACE FUNCTION public.get_available_slots(
+  p_tenant_id uuid,
+  p_date date,
+  p_duration_minutes integer,
+  p_assignee_id uuid DEFAULT NULL,
+  p_slot_interval integer DEFAULT 30
+) RETURNS TABLE (slot_start timestamptz, slot_end timestamptz)
+LANGUAGE plpgsql STABLE SECURITY DEFINER
+SET search_path = public
+AS $$
+DECLARE
+  v_tz text;
+BEGIN
+  -- tight validation: this function is anon-callable
+  IF p_duration_minutes IS NULL OR p_duration_minutes NOT BETWEEN 5 AND 480 THEN
+    RAISE EXCEPTION 'invalid duration';
+  END IF;
+  IF p_slot_interval IS NULL OR p_slot_interval NOT BETWEEN 5 AND 240 THEN
+    RAISE EXCEPTION 'invalid interval';
+  END IF;
+  IF p_date IS NULL OR p_date < current_date OR p_date > current_date + 90 THEN
+    RETURN;
+  END IF;
+
+  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; END IF;  -- unknown tenant → empty, not an error
+
+  RETURN QUERY
+  WITH day_schedules AS (
+    SELECT w.assignee_id, w.win_start, w.win_end
+    FROM agenda_working_windows(p_tenant_id, p_date, p_assignee_id) w
+  ),
+  candidate AS (
+    SELECT
+      ds.assignee_id,
+      gs                                              AS s_start,
+      gs + make_interval(mins => p_duration_minutes)  AS s_end
+    FROM day_schedules ds
+    CROSS JOIN LATERAL generate_series(
+      (p_date + ds.win_start) AT TIME ZONE v_tz,
+      ((p_date + ds.win_end) AT TIME ZONE v_tz) - make_interval(mins => p_duration_minutes),
+      make_interval(mins => p_slot_interval)
+    ) AS gs
+  )
+  SELECT DISTINCT c.s_start, c.s_end
+  FROM candidate c
+  WHERE NOT EXISTS (
+    SELECT 1 FROM appointments b
+    WHERE b.tenant_id = p_tenant_id
+      -- SMART SYNC (gcal 002): this professional's own bookings OR any
+      -- tenant-wide BLOCK (assignee_id IS NULL AND kind='block') busy the slot.
+      AND (b.assignee_id = c.assignee_id
+           OR (b.assignee_id IS NULL AND b.kind = 'block'))
+      AND b.status NOT IN ('cancelled', 'no_show')
+      AND b.starts_at < c.s_end
+      AND COALESCE(b.ends_at, b.starts_at) > c.s_start
+  )
+  ORDER BY 1;
+END;
+$$;
+
+REVOKE ALL ON FUNCTION public.get_available_slots(uuid, date, integer, uuid, integer) FROM public;
+GRANT EXECUTE ON FUNCTION public.get_available_slots(uuid, date, integer, uuid, integer)
+  TO anon, authenticated, service_role;
+
+-- ----------------------------------------------------------------------------
+-- §3  create_public_booking — professional resolution goes through the helper
+--     (a booking on a closed date now fails 'slot outside working hours').
+--     Everything else is byte-identical to 004.
+-- ----------------------------------------------------------------------------
+CREATE OR REPLACE FUNCTION public.create_public_booking(
+  p_tenant_id uuid,
+  p_service_id uuid,
+  p_starts_at timestamptz,
+  p_name text,
+  p_phone text,
+  p_email text DEFAULT NULL,
+  p_notes text DEFAULT NULL,
+  p_assignee_id uuid DEFAULT NULL
+) RETURNS TABLE (booking_id uuid, starts_at timestamptz, ends_at timestamptz)
+LANGUAGE plpgsql SECURITY DEFINER
+SET search_path = public
+AS $$
+DECLARE
+  v_service record;
+  v_person_id uuid;
+  v_assignee uuid;
+  v_ends timestamptz;
+  v_order uuid;
+  v_booking uuid;
+  v_phone text;
+  v_tz text;
+  v_notes text;
+  v_client_kind text;
+BEGIN
+  -- validation: anon-callable, everything tight
+  IF p_name IS NULL OR length(trim(p_name)) NOT BETWEEN 2 AND 120 THEN
+    RAISE EXCEPTION 'invalid name';
+  END IF;
+  v_phone := regexp_replace(COALESCE(p_phone, ''), '\D', '', 'g');
+  IF length(v_phone) NOT BETWEEN 8 AND 15 THEN
+    RAISE EXCEPTION 'invalid phone';
+  END IF;
+  IF p_email IS NOT NULL AND (length(p_email) > 254 OR p_email !~ '^[^@\s]+@[^@\s]+\.[^@\s]+$') THEN
+    RAISE EXCEPTION 'invalid email';
+  END IF;
+  v_notes := left(p_notes, 500);
+  IF p_starts_at IS NULL OR p_starts_at <= now() OR p_starts_at > now() + interval '90 days' THEN
+    RAISE EXCEPTION 'invalid start';
+  END IF;
+
+  SELECT s.id, s.name, s.price, COALESCE(s.duration_minutes, 30) AS duration_minutes
+    INTO v_service
+  FROM services s
+  WHERE s.id = p_service_id AND s.tenant_id = p_tenant_id
+    AND s.is_active AND s.status = 'active';
+  IF NOT FOUND THEN RAISE EXCEPTION 'unknown service'; END IF;
+
+  v_ends := p_starts_at + make_interval(mins => v_service.duration_minutes);
+
+  -- tenant config: timezone + client kind (admin-set; sanitized, never caller input)
+  SELECT COALESCE(t.settings->>'timezone', 'America/Sao_Paulo'),
+         COALESCE(NULLIF(trim(t.settings->'booking'->>'client_kind'), ''), 'customer')
+    INTO v_tz, v_client_kind
+  FROM tenants t WHERE t.id = p_tenant_id;
+  IF v_tz IS NULL THEN RAISE EXCEPTION 'unknown tenant'; END IF;
+  IF v_client_kind !~ '^[a-z_]{2,32}$' THEN v_client_kind := 'customer'; END IF;
+
+  -- resolve the professional: the one the visitor saw (validated), else the
+  -- first whose EXCEPTION-AWARE working hours cover the slot (007)
+  SELECT w.assignee_id INTO v_assignee
+  FROM agenda_working_windows(
+         p_tenant_id, (p_starts_at AT TIME ZONE v_tz)::date, p_assignee_id) w
+  WHERE (p_starts_at AT TIME ZONE v_tz)::time >= w.win_start
+    AND (v_ends AT TIME ZONE v_tz)::time <= w.win_end
+  ORDER BY w.assignee_id
+  LIMIT 1;
+  IF v_assignee IS NULL THEN RAISE EXCEPTION 'slot outside working hours'; END IF;
+
+  -- serialize per (tenant, assignee) to kill the double-book race
+  PERFORM pg_advisory_xact_lock(hashtextextended(p_tenant_id::text || v_assignee::text, 0));
+
+  IF EXISTS (
+    SELECT 1 FROM appointments b
+    WHERE b.tenant_id = p_tenant_id
+      AND b.assignee_id = v_assignee
+      AND b.status NOT IN ('cancelled', 'no_show')
+      AND b.starts_at < v_ends
+      AND COALESCE(b.ends_at, b.starts_at) > p_starts_at
+  ) THEN
+    RAISE EXCEPTION 'slot no longer available';
+  END IF;
+
+  -- anti-spam: cap open future public bookings per phone per tenant
+  IF (
+    SELECT count(*) FROM appointments b
+    JOIN people p ON p.id = b.party_id
+    WHERE b.tenant_id = p_tenant_id
+      AND p.phone = v_phone
+      AND b.starts_at > now()
+      AND b.status = 'scheduled'
+  ) >= 3 THEN
+    RAISE EXCEPTION 'too many open bookings for this phone';
+  END IF;
+
+  -- upsert the client by (tenant, kind, phone), serialized by advisory lock
+  -- (kind is tenant-configured, so the customer-only partial index can't
+  -- arbitrate)
+  PERFORM pg_advisory_xact_lock(hashtextextended(p_tenant_id::text || ':' || v_phone, 1));
+
+  SELECT p.id INTO v_person_id
+  FROM people p
+  WHERE p.tenant_id = p_tenant_id AND p.kind = v_client_kind AND p.phone = v_phone
+  ORDER BY p.created_at
+  LIMIT 1;
+
+  IF v_person_id IS NOT NULL THEN
+    UPDATE people SET
+      name = trim(p_name),
+      email = COALESCE(p_email, people.email),
+      updated_at = now()
+    WHERE id = v_person_id;
+  ELSE
+    INSERT INTO people (tenant_id, kind, name, phone, email, metadata)
+    VALUES (p_tenant_id, v_client_kind, trim(p_name), v_phone, p_email,
+            jsonb_build_object('source', 'public_booking'))
+    RETURNING id INTO v_person_id;
+  END IF;
+
+  -- order + booking + order_item (statuses hardcoded — caller cannot set)
+  INSERT INTO orders (tenant_id, kind, status, party_id, assignee_id, subtotal, total, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', 'scheduled', v_person_id, v_assignee,
+          v_service.price, v_service.price, v_notes,
+          jsonb_build_object('source', 'public_booking',
+                             'serviceNames', v_service.name,
+                             'contactName', trim(p_name)))
+  RETURNING id INTO v_order;
+
+  INSERT INTO appointments (tenant_id, kind, party_id, assignee_id, order_id,
+                        starts_at, ends_at, status, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', v_person_id, v_assignee, v_order,
+          p_starts_at, v_ends, 'scheduled', v_notes,
+          jsonb_build_object('source', 'public_booking', 'serviceNames', v_service.name))
+  RETURNING id INTO v_booking;
+
+  INSERT INTO order_items (order_id, service_id, name, quantity, unit_price, total,
+                           sort_order, duration_minutes, assignee_id)
+  VALUES (v_order, v_service.id, v_service.name, 1, v_service.price, v_service.price,
+          0, v_service.duration_minutes, v_assignee);
+
+  RETURN QUERY SELECT v_booking, p_starts_at, v_ends;
+END;
+$$;
+
+REVOKE ALL ON FUNCTION public.create_public_booking(uuid, uuid, timestamptz, text, text, text, text, uuid) FROM public;
+GRANT EXECUTE ON FUNCTION public.create_public_booking(uuid, uuid, timestamptz, text, text, text, text, uuid)
+  TO anon, authenticated, service_role;

package/src/migrations/008_agent_rpc_exception_precedence.sql

@@ -0,0 +1,172 @@
+-- ============================================================================
+-- plugin-agenda 008: agent_agenda_create_appointment honors schedule exceptions.
+--
+-- Same precedence fix as 007, applied to the agent write path: professional
+-- resolution now goes through agenda_working_windows (007 §1), so a date the
+-- admin closed (day-off sentinel) or re-opened with custom hours ("horário
+-- especial") binds the agent exactly like it binds the public site. Everything
+-- else is byte-identical to 006 (LOCAL-time input, free-professional pick,
+-- advisory lock, audit). Requires 007. Supersedes 006's function body.
+-- ============================================================================
+
+CREATE OR REPLACE FUNCTION public.agent_agenda_create_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_used int;
+  v_client_id uuid;
+  v_service_id uuid;
+  v_professional uuid;
+  v_starts timestamptz;
+  v_starts_raw text;
+  v_notes text;
+  v_service record;
+  v_client record;
+  v_ends timestamptz;
+  v_tz text;
+  v_order uuid;
+  v_booking uuid;
+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_client_id    := (p_payload->>'client_id')::uuid;
+    v_service_id   := (p_payload->>'service_id')::uuid;
+    v_professional := (p_payload->>'professional_id')::uuid;
+    v_starts_raw   := p_payload->>'starts_at';
+    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 payload: ' || SQLERRM);
+  END;
+  v_notes := left(p_payload->>'notes', 500);
+  IF v_client_id IS NULL OR v_service_id IS NULL OR v_starts IS NULL THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'client_id, service_id and starts_at are required');
+  END IF;
+  IF v_starts <= now() OR v_starts > now() + interval '365 days' THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'starts_at must be in the future (max 1 year)');
+  END IF;
+
+  -- ── authorization: role → plan → bookings_month cap ──────────────────────
+  SELECT count(*) INTO v_used FROM appointments b
+  WHERE b.tenant_id = p_tenant_id
+    AND b.created_at >= date_trunc('month', now());
+  v_denial := agent_guard(p_tenant_id, p_actor_user_id,
+                          'appointments', 'create', 'bookings_month', v_used, 1);
+  IF v_denial IS NOT NULL THEN
+    RETURN jsonb_build_object('ok', false, 'denial', v_denial);
+  END IF;
+
+  -- ── invariants ───────────────────────────────────────────────────────────
+  SELECT p.id, p.name INTO v_client
+  FROM people p
+  WHERE p.id = v_client_id AND p.tenant_id = p_tenant_id;
+  IF NOT FOUND THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'unknown client for this tenant');
+  END IF;
+
+  SELECT s.id, s.name, s.price, COALESCE(s.duration_minutes, 30) AS duration_minutes
+    INTO v_service
+  FROM services s
+  WHERE s.id = v_service_id AND s.tenant_id = p_tenant_id
+    AND s.is_active AND s.status = 'active';
+  IF NOT FOUND THEN
+    RETURN jsonb_build_object('ok', false, 'error', 'unknown or inactive service');
+  END IF;
+
+  v_ends := v_starts + make_interval(mins => v_service.duration_minutes);
+
+  -- exception-aware windows (007); still prefer a professional who is FREE at
+  -- the slot when the user did not name one
+  SELECT w.assignee_id INTO v_professional
+  FROM agenda_working_windows(
+         p_tenant_id, (v_starts AT TIME ZONE v_tz)::date, v_professional) w
+  WHERE (v_starts AT TIME ZONE v_tz)::time >= w.win_start
+    AND (v_ends AT TIME ZONE v_tz)::time <= w.win_end
+    AND NOT EXISTS (
+      SELECT 1 FROM appointments b
+      WHERE b.tenant_id = p_tenant_id
+        AND b.assignee_id = w.assignee_id
+        AND b.status NOT IN ('cancelled', 'no_show')
+        AND b.starts_at < v_ends
+        AND COALESCE(b.ends_at, b.starts_at) > v_starts
+    )
+  ORDER BY w.assignee_id
+  LIMIT 1;
+  IF v_professional IS NULL THEN
+    RETURN jsonb_build_object('ok', false, 'error',
+      'no professional is available at that slot (outside working hours or already booked). 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.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', 'slot no longer available');
+  END IF;
+
+  -- ── write + audit ────────────────────────────────────────────────────────
+  INSERT INTO orders (tenant_id, kind, status, party_id, assignee_id, subtotal, total, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', 'scheduled', v_client_id, v_professional,
+          v_service.price, v_service.price, v_notes,
+          jsonb_build_object('source', 'agent', 'serviceNames', v_service.name))
+  RETURNING id INTO v_order;
+
+  INSERT INTO appointments (tenant_id, kind, party_id, assignee_id, order_id,
+                            starts_at, ends_at, status, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', v_client_id, v_professional, v_order,
+          v_starts, v_ends, 'scheduled', v_notes,
+          jsonb_build_object('source', 'agent', 'serviceNames', v_service.name))
+  RETURNING id INTO v_booking;
+
+  INSERT INTO order_items (order_id, service_id, name, quantity, unit_price, total,
+                           sort_order, duration_minutes, assignee_id)
+  VALUES (v_order, v_service.id, v_service.name, 1, v_service.price, v_service.price,
+          0, v_service.duration_minutes, v_professional);
+
+  INSERT INTO audit_logs (tenant_id, user_id, action, entity_type, entity_id, metadata)
+  VALUES (p_tenant_id, p_actor_user_id, 'agent.createAppointment', 'appointment', v_booking::text,
+          jsonb_build_object('payload', p_payload, 'order_id', v_order));
+
+  RETURN jsonb_build_object(
+    'ok', true,
+    'id', v_booking,
+    'record', jsonb_build_object(
+      'ref', jsonb_build_object('id', v_booking, 'resource', 'appointments',
+                                'archetype', 'schedule:appointment'),
+      'starts_at', v_starts,
+      'ends_at', v_ends,
+      'starts_at_local', to_char(v_starts AT TIME ZONE v_tz, 'YYYY-MM-DD (Dy) HH24:MI') || ' ' || v_tz,
+      'ends_at_local', to_char(v_ends AT TIME ZONE v_tz, 'HH24:MI'),
+      'client_name', v_client.name,
+      'service_name', v_service.name,
+      'price', v_service.price,
+      'status', 'scheduled'
+    )
+  );
+END;
+$$;
+
+REVOKE ALL ON FUNCTION public.agent_agenda_create_appointment(uuid, uuid, jsonb) FROM public;
+REVOKE EXECUTE ON FUNCTION public.agent_agenda_create_appointment(uuid, uuid, jsonb) FROM anon;
+GRANT EXECUTE ON FUNCTION public.agent_agenda_create_appointment(uuid, uuid, jsonb)
+  TO authenticated, service_role;

package/src/migrations/009_public_booking_location.sql

@@ -0,0 +1,162 @@
+-- Public bookings were written with location_id = NULL. The agenda's location
+-- filter is optional (default "all"), so they still rendered — but the moment
+-- an operator filtered by their unit, every booking taken from the website
+-- disappeared from the calendar. Single-location tenants are the common case
+-- and have an unambiguous answer, so default to it; multi-location tenants
+-- stay NULL (there is no correct guess) until the public flow asks.
+--
+-- Only the assignment is new; the rest of the body is unchanged from 004.
+
+CREATE OR REPLACE FUNCTION public.create_public_booking(
+  p_tenant_id uuid,
+  p_service_id uuid,
+  p_starts_at timestamptz,
+  p_name text,
+  p_phone text,
+  p_email text DEFAULT NULL,
+  p_notes text DEFAULT NULL
+)
+RETURNS TABLE(booking_id uuid, starts_at timestamptz, ends_at timestamptz)
+LANGUAGE plpgsql
+SECURITY DEFINER
+SET search_path TO 'public'
+AS $function$
+DECLARE
+  v_service record;
+  v_person_id uuid;
+  v_assignee uuid;
+  v_ends timestamptz;
+  v_order uuid;
+  v_booking uuid;
+  v_phone text;
+  v_tz text;
+  v_notes text;
+  v_location uuid;
+BEGIN
+  -- validation: anon-callable, everything tight
+  IF p_name IS NULL OR length(trim(p_name)) NOT BETWEEN 2 AND 120 THEN
+    RAISE EXCEPTION 'invalid name';
+  END IF;
+  v_phone := regexp_replace(COALESCE(p_phone, ''), '\D', '', 'g');
+  IF length(v_phone) NOT BETWEEN 8 AND 15 THEN
+    RAISE EXCEPTION 'invalid phone';
+  END IF;
+  IF p_email IS NOT NULL AND (length(p_email) > 254 OR p_email !~ '^[^@\s]+@[^@\s]+\.[^@\s]+$') THEN
+    RAISE EXCEPTION 'invalid email';
+  END IF;
+  v_notes := left(p_notes, 500);
+  IF p_starts_at IS NULL OR p_starts_at <= now() OR p_starts_at > now() + interval '90 days' THEN
+    RAISE EXCEPTION 'invalid start';
+  END IF;
+
+  SELECT s.id, s.name, s.price, COALESCE(s.duration_minutes, 30) AS duration_minutes
+    INTO v_service
+  FROM services s
+  WHERE s.id = p_service_id AND s.tenant_id = p_tenant_id
+    AND s.is_active AND s.status = 'active';
+  IF NOT FOUND THEN RAISE EXCEPTION 'unknown service'; END IF;
+
+  v_ends := p_starts_at + make_interval(mins => v_service.duration_minutes);
+  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 RAISE EXCEPTION 'unknown tenant'; END IF;
+
+  -- pick the assignee whose working hours cover the slot
+  SELECT sc.assignee_id INTO v_assignee
+  FROM schedules sc
+  WHERE sc.tenant_id = p_tenant_id
+    AND sc.kind = 'working_hours'
+    AND sc.is_active
+    AND (
+      sc.specific_date = (p_starts_at AT TIME ZONE v_tz)::date
+      OR (sc.specific_date IS NULL
+          AND sc.day_of_week = EXTRACT(dow FROM p_starts_at AT TIME ZONE v_tz)::smallint)
+    )
+    AND (p_starts_at AT TIME ZONE v_tz)::time >= sc.starts_at
+    AND (v_ends AT TIME ZONE v_tz)::time <= sc.ends_at
+  ORDER BY sc.assignee_id
+  LIMIT 1;
+  IF v_assignee IS NULL THEN RAISE EXCEPTION 'slot outside working hours'; END IF;
+
+  -- NEW: stamp the unit so the calendar's location filter keeps these visible.
+  -- Prefer the schedule's own location; else the tenant's only location.
+  SELECT sc.location_id INTO v_location
+  FROM schedules sc
+  WHERE sc.tenant_id = p_tenant_id
+    AND sc.kind = 'working_hours'
+    AND sc.is_active
+    AND sc.assignee_id = v_assignee
+    AND sc.location_id IS NOT NULL
+  LIMIT 1;
+
+  IF v_location IS NULL THEN
+    SELECT l.id INTO v_location
+    FROM locations l
+    WHERE l.tenant_id = p_tenant_id
+    HAVING count(*) = 1;  -- unambiguous only
+  END IF;
+
+  -- serialize per (tenant, assignee) to kill the double-book race
+  PERFORM pg_advisory_xact_lock(hashtextextended(p_tenant_id::text || v_assignee::text, 0));
+
+  IF EXISTS (
+    SELECT 1 FROM appointments b
+    WHERE b.tenant_id = p_tenant_id
+      AND b.assignee_id = v_assignee
+      AND b.status NOT IN ('cancelled', 'no_show')
+      AND b.starts_at < v_ends
+      AND COALESCE(b.ends_at, b.starts_at) > p_starts_at
+  ) THEN
+    RAISE EXCEPTION 'slot no longer available';
+  END IF;
+
+  -- anti-spam: cap open future public bookings per phone per tenant
+  IF (
+    SELECT count(*) FROM appointments b
+    JOIN people p ON p.id = b.party_id
+    WHERE b.tenant_id = p_tenant_id
+      AND p.phone = v_phone
+      AND b.starts_at > now()
+      AND b.status = 'scheduled'
+  ) >= 3 THEN
+    RAISE EXCEPTION 'too many open bookings for this phone';
+  END IF;
+
+  -- upsert the customer by (tenant, phone) — matches people_tenant_customer_phone
+  INSERT INTO people (tenant_id, kind, name, phone, email, metadata)
+  VALUES (p_tenant_id, 'customer', trim(p_name), v_phone, p_email,
+          jsonb_build_object('source', 'public_booking'))
+  ON CONFLICT (tenant_id, phone) WHERE kind = 'customer' AND phone IS NOT NULL
+  DO UPDATE SET
+    name = EXCLUDED.name,
+    email = COALESCE(EXCLUDED.email, people.email),
+    updated_at = now()
+  RETURNING id INTO v_person_id;
+
+  -- order + booking + order_item (statuses hardcoded — caller cannot set)
+  INSERT INTO orders (tenant_id, kind, status, party_id, assignee_id, location_id,
+                      subtotal, total, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', 'scheduled', v_person_id, v_assignee, v_location,
+          v_service.price, v_service.price, v_notes,
+          jsonb_build_object('source', 'public_booking',
+                             'serviceNames', v_service.name,
+                             'contactName', trim(p_name)))
+  RETURNING id INTO v_order;
+
+  INSERT INTO appointments (tenant_id, kind, party_id, assignee_id, order_id, location_id,
+                        starts_at, ends_at, status, notes, metadata)
+  VALUES (p_tenant_id, 'appointment', v_person_id, v_assignee, v_order, v_location,
+          p_starts_at, v_ends, 'scheduled', v_notes,
+          jsonb_build_object('source', 'public_booking', 'serviceNames', v_service.name))
+  RETURNING id INTO v_booking;
+
+  INSERT INTO order_items (order_id, service_id, name, quantity, unit_price, total,
+                           sort_order, duration_minutes, assignee_id)
+  VALUES (v_order, v_service.id, v_service.name, 1, v_service.price, v_service.price,
+          0, v_service.duration_minutes, v_assignee);
+
+  RETURN QUERY SELECT v_booking, p_starts_at, v_ends;
+END;
+$function$;
+
+GRANT EXECUTE ON FUNCTION public.create_public_booking(uuid, uuid, timestamptz, text, text, text, text) TO anon, authenticated;

package/src/migrations/010_public_service_curation.sql

@@ -0,0 +1,31 @@
+-- plugin-agenda 010: curate what the PUBLIC booking page sells.
+--
+-- v_public_services published every active service, so the website catalog was
+-- "whatever exists in the system". That conflates two different questions:
+--
+--   • is this service usable at all?          → services.is_active / status
+--   • should a stranger be able to book it?   → this flag
+--
+-- Without the split, hiding something from the site means deactivating it —
+-- which also removes it from the internal agenda and orphans its history. The
+-- case that forced this: a school whose public page listed 7 services when it
+-- sells 3, and one of the 4 extras already had real bookings attached.
+--
+-- OPT-OUT on purpose: a service is public unless explicitly marked otherwise.
+-- Opt-in would silently empty every live catalog the moment this ships.
+
+CREATE OR REPLACE VIEW public.v_public_services AS
+SELECT id,
+    tenant_id,
+    name,
+    description,
+    price,
+    currency,
+    COALESCE(duration_minutes, 30) AS duration_minutes,
+    image_url,
+    COALESCE(((metadata ->> 'sort_order'::text))::integer, 0) AS sort_order
+   FROM services s
+  WHERE is_active
+    AND status = 'active'::text
+    -- absent → public (back-compat). Only an explicit false hides it.
+    AND COALESCE((metadata ->> 'bookableOnline'::text)::boolean, true);