feat: Cap.B pulido — semáforo/alarma de marcas cerradas (ClosedBrandsCard + watchdog) + reorg UX Disponibilidad

Author: Llorente29

src/modules/kds/components/ClosedBrandsCard.tsx

@@ -0,0 +1,105 @@
+// src/modules/kds/components/ClosedBrandsCard.tsx
+//
+// FASE B · CAP. B — indicador AMBIENTAL de marcas cerradas (§9-C). Antes el
+// estado de una marca solo se veía DENTRO del modal de BrandCloseControl,
+// tras buscarla a mano. Esta tarjeta las muestra siempre visibles, con
+// reapertura de un toque — mismo espíritu que LocationStatusCard (Cap. C).
+//
+// closed_brands ya excluye las que tenían resume_at y pasó (HubRise las
+// reabrió sola vía expires_at) — por eso no hace falta corregir aquí, a
+// diferencia de LocationStatusCard (que sí calcula effectiveMode en cliente:
+// aquí la lista entera desaparece de la RPC en cuanto vence, no hay una sola
+// entidad que "corregir" en pantalla).
+//
+// No se muestra nada si no hay ninguna marca cerrada (ambiental de verdad:
+// no ocupa sitio cuando no aporta).
+
+import { useCallback, useEffect, useState } from 'react'
+import { Store, Unlock, Loader2, AlertTriangle } from 'lucide-react'
+import { getClosedBrands, setBrandStatus, setBrandStatusByToken, type ClosedBrand } from '../services/kdsService'
+
+interface Props {
+  accountId?: string | null
+  token?: string | null
+  dark?: boolean
+}
+
+export default function ClosedBrandsCard({ accountId, token, dark = false }: Props) {
+  const [brands, setBrands] = useState<ClosedBrand[]>([])
+  const [loading, setLoading] = useState(true)
+  const [busyId, setBusyId] = useState<string | null>(null)
+  const [error, setError] = useState<string | null>(null)
+
+  const refresh = useCallback(async () => {
+    try {
+      setBrands(await getClosedBrands(accountId ?? null, token))
+      setError(null)
+    } catch (e) {
+      setError(e instanceof Error ? e.message : 'Error cargando marcas cerradas')
+    } finally {
+      setLoading(false)
+    }
+  }, [accountId, token])
+
+  useEffect(() => {
+    setLoading(true)
+    void refresh()
+    const id = window.setInterval(() => { void refresh() }, 30_000)
+    return () => window.clearInterval(id)
+  }, [refresh])
+
+  async function reopen(brandId: string) {
+    setBusyId(brandId); setError(null)
+    try {
+      if (token) await setBrandStatusByToken(token, brandId, 'normal')
+      else await setBrandStatus(brandId, 'normal')
+      await refresh()
+    } catch (e) {
+      setError(e instanceof Error ? e.message : 'No se pudo reabrir')
+    } finally {
+      setBusyId(null)
+    }
+  }
+
+  if (loading || brands.length === 0) return null
+
+  const cardCls = dark ? 'bg-zinc-900 ring-1 ring-zinc-800' : 'bg-white border border-stone-200'
+
+  return (
+    <div className={`rounded-xl px-4 py-3 mb-3 ${cardCls}`}>
+      <div className="flex items-center gap-2 mb-2">
+        <Store size={15} className={dark ? 'text-zinc-500' : 'text-stone-400'} />
+        <span className={`text-xs font-semibold uppercase tracking-wide ${dark ? 'text-zinc-400' : 'text-stone-500'}`}>
+          {brands.length === 1 ? 'Marca cerrada' : `${brands.length} marcas cerradas`}
+        </span>
+      </div>
+      <div className="flex flex-col gap-1.5">
+        {brands.map((b) => (
+          <div key={b.brand_id} className="flex items-center justify-between gap-2">
+            <div className="flex items-center gap-2 min-w-0">
+              <span className="w-2 h-2 rounded-full bg-red-500 shrink-0" />
+              <span className={`text-sm truncate ${dark ? 'text-zinc-100' : 'text-stone-800'}`}>{b.brand_name}</span>
+              <span className={`text-xs shrink-0 ${dark ? 'text-zinc-500' : 'text-stone-400'}`}>
+                {b.resume_at
+                  ? `hasta las ${new Date(b.resume_at).toLocaleTimeString('es-ES', { hour: '2-digit', minute: '2-digit' })}`
+                  : 'indefinido'}
+              </span>
+            </div>
+            <button
+              onClick={() => void reopen(b.brand_id)}
+              disabled={busyId === b.brand_id}
+              className="shrink-0 inline-flex items-center gap-1 px-2 py-1 rounded-md text-xs font-semibold bg-emerald-600 text-white hover:bg-emerald-700 disabled:opacity-50"
+            >
+              {busyId === b.brand_id ? <Loader2 size={12} className="animate-spin" /> : <Unlock size={12} />} Reabrir
+            </button>
+          </div>
+        ))}
+      </div>
+      {error && (
+        <div className="mt-2 flex items-center gap-1.5 text-xs text-red-500">
+          <AlertTriangle size={13} /> {error}
+        </div>
+      )}
+    </div>
+  )
+}

src/modules/kds/services/kdsService.ts

@@ -427,6 +427,27 @@ export function listBrandsForClosure(accountId: string | null, token?: string |
   })
 }
 
+export interface ClosedBrand {
+  brand_id: string
+  brand_name: string
+  mode: BrandStatusMode
+  resume_at: string | null
+  reason: string | null
+  set_at: string | null
+}
+
+/**
+ * Marcas EFECTIVAMENTE cerradas ahora mismo (indicador ambiental, §9-C) — la
+ * RPC ya oculta las que tenían resume_at y ya pasó (HubRise las reabrió sola
+ * vía expires_at, aunque brand.closure_mode en Folvy no se reescriba solo).
+ */
+export function getClosedBrands(accountId: string | null, token?: string | null): Promise<ClosedBrand[]> {
+  return rpc<ClosedBrand[]>('closed_brands', {
+    p_account_id: accountId,
+    p_token: token ?? null,
+  })
+}
+
 // ─────────────────────────────────────────────────────────────────────────────
 // AJUSTES DE COCINA (lectura/escritura por servicio, RLS de SESIÓN — sin token)
 // ─────────────────────────────────────────────────────────────────────────────

src/modules/kitchen/pages/KitchenAvailabilityPage.tsx

@@ -14,6 +14,7 @@ import {
 } from '@/modules/kitchen/services/availabilityService'
 import LocationStatusCard from '@/modules/kds/components/LocationStatusCard'
 import BrandCloseControl from '@/modules/kds/components/BrandCloseControl'
+import ClosedBrandsCard from '@/modules/kds/components/ClosedBrandsCard'
 
 const ACCENT = '#15171A'
 
@@ -120,21 +121,9 @@ export default function KitchenAvailabilityPage() {
 
   return (
     <div className="max-w-3xl mx-auto px-4 py-6">
-      <div className="flex items-center justify-between gap-3 mb-6 flex-wrap">
-        <div>
-          <h1 className="text-lg font-medium text-stone-800">Disponibilidad</h1>
-          <p className="text-[13px] text-stone-500 mt-0.5">Lo que está agotado ahora mismo</p>
-        </div>
-        <div className="flex items-center gap-2">
-          {activeAccountId && <BrandCloseControl accountId={activeAccountId} />}
-          <button
-            onClick={() => setShowAgotar(true)}
-            className="inline-flex items-center gap-1.5 px-3 py-2 rounded-lg text-white text-sm font-medium"
-            style={{ backgroundColor: ACCENT }}
-          >
-            <Plus size={18} /> Agotar producto
-          </button>
-        </div>
+      <div className="mb-6">
+        <h1 className="text-lg font-medium text-stone-800">Disponibilidad</h1>
+        <p className="text-[13px] text-stone-500 mt-0.5">Lo que está cerrado o agotado ahora mismo</p>
       </div>
 
       <div className="flex gap-2.5 mb-5 flex-wrap items-center">
@@ -183,13 +172,31 @@ export default function KitchenAvailabilityPage() {
         <div className="mb-4 rounded-lg border border-red-200 bg-red-50 p-3 text-[13px] text-red-700">{error}</div>
       )}
 
-      {locationId && <LocationStatusCard locationId={locationId} />}
+      {/* ── Local y marcas: los dos cierres de alcance amplio ─────────────── */}
+      <div className="mb-5">
+        <div className="flex items-center justify-between gap-2 mb-2 flex-wrap">
+          <p className="text-[11px] font-semibold uppercase tracking-wide text-stone-400">Local y marcas</p>
+          {activeAccountId && <BrandCloseControl accountId={activeAccountId} />}
+        </div>
+        {locationId && <LocationStatusCard locationId={locationId} />}
+        {activeAccountId && <ClosedBrandsCard accountId={activeAccountId} />}
+      </div>
 
-      <div className="flex items-center gap-2 mb-2.5">
-        <span className="w-2 h-2 rounded-full bg-red-500 inline-block" />
-        <span className="text-[13px] font-medium text-stone-500">
-          Agotados en {locName} · {loading ? '…' : visible.length}
-        </span>
+      {/* ── Productos: el cierre de alcance más fino ──────────────────────── */}
+      <div className="flex items-center justify-between gap-2 mb-2.5 flex-wrap">
+        <div className="flex items-center gap-2">
+          <span className="w-2 h-2 rounded-full bg-red-500 inline-block" />
+          <span className="text-[13px] font-medium text-stone-500">
+            Productos agotados en {locName} · {loading ? '…' : visible.length}
+          </span>
+        </div>
+        <button
+          onClick={() => setShowAgotar(true)}
+          className="inline-flex items-center gap-1.5 px-3 py-2 rounded-lg text-white text-sm font-medium"
+          style={{ backgroundColor: ACCENT }}
+        >
+          <Plus size={18} /> Agotar producto
+        </button>
       </div>
 
       {loading ? (

src/modules/tablet/TabletAvailabilityTab.tsx

@@ -16,6 +16,7 @@ import {
 } from './services/tabletAvailabilityService'
 import LocationStatusCard from '@/modules/kds/components/LocationStatusCard'
 import BrandCloseControl from '@/modules/kds/components/BrandCloseControl'
+import ClosedBrandsCard from '@/modules/kds/components/ClosedBrandsCard'
 
 interface Props {
   token: string
@@ -91,11 +92,14 @@ export default function TabletAvailabilityTab({ token, locationName }: Props) {
 
       {/* Lista de agotados */}
       <div className="flex-1 overflow-y-auto p-5">
+        {/* ── Local y marcas: los dos cierres de alcance amplio ─────────────── */}
         <LocationStatusCard locationId={null} token={token} dark />
+        <ClosedBrandsCard token={token} dark />
 
+        {/* ── Productos: el cierre de alcance más fino ──────────────────────── */}
         <p className="text-sm text-zinc-500 mb-3">
           <span className="inline-block w-2 h-2 rounded-full bg-red-500 mr-2 align-middle" />
-          Agotados ahora · {rows.length}
+          Productos agotados ahora · {rows.length}
         </p>
 
         {loading && rows.length === 0 ? (

supabase/functions/availability-watchdog/index.ts

@@ -15,9 +15,14 @@
 //     local está en un estado (cerrado / con tal horario) que HubRise nunca
 //     confirmó -> estado incoherente entre Folvy y la plataforma.
 //
-// Ventana de revisión de 20 min (solape sobre el cron de 15 min, para no
-// perder filas entre corridas). No dedupea entre corridas (deuda menor,
-// mismo criterio que el original: más ruidoso que perder un fallo en silencio).
+// (C) brand.closure_mode (Fase B, Cap. B) — cierres de marca OLVIDADOS: sin
+//     resume_at (indefinido) hace más de 24h, o con resume_at ya vencido pero
+//     closure_mode aún 'paused'. Ver checkStaleBrandClosures.
+//
+// Ventana de revisión de 20 min para (A)/(B) (solape sobre el cron de 15 min,
+// para no perder filas entre corridas). (C) no usa ventana: es estado actual,
+// no eventos recientes. No dedupea entre corridas (deuda menor, mismo
+// criterio que el original: más ruidoso que perder un fallo en silencio).
 //
 // Deploy: --no-verify-jwt (inocua; sin params externos, solo lee y alerta).
 
@@ -103,6 +108,68 @@ async function checkAvailabilityPushLog(
   return { failures: rows.length, stuck: stuck.length, soldOutFailed: soldOutFailed.length };
 }
 
+// (C) Cap. B — cierres de marca OLVIDADOS. Dos casos, distintos de gravedad:
+//   · INDEFINIDO (resume_at null) más de INDEFINITE_CLOSURE_ALERT_HOURS: sin
+//     expires_at en el push a HubRise, esa marca NO se reabre sola — riesgo
+//     real de quedar cerrada para siempre si nadie se acuerda.
+//   · VENCIDO (resume_at ya pasado, brand.closure_mode aún 'paused'): HubRise
+//     ya reabrió esos SKUs solo (expires_at) — closed_brands ya lo oculta en
+//     el indicador ambiental — pero el semáforo de Folvy sigue "mintiendo"
+//     hasta que alguien reabra a mano; aviso de limpieza, no de fallo real.
+// SIN dedupe entre corridas (mismo criterio que hubrise-callback-ensure: más
+// ruidoso que olvidarlo en silencio) — se resuelve solo en cuanto se reabre.
+const INDEFINITE_CLOSURE_ALERT_HOURS = 24;
+
+async function checkStaleBrandClosures(
+  sb: ReturnType<typeof createClient>,
+): Promise<{ indefinite: number; expired: number }> {
+  const { data: closed, error } = await sb
+    .from("brand")
+    .select("id, name, account_id, closure_resume_at, closure_set_at, closure_reason")
+    .eq("closure_mode", "paused")
+    .limit(200);
+
+  if (error) {
+    console.error("availability-watchdog: error consultando brand (cierres de marca)", error);
+    return { indefinite: 0, expired: 0 };
+  }
+
+  const rows = closed ?? [];
+  if (rows.length === 0) return { indefinite: 0, expired: 0 };
+
+  const now = Date.now();
+  const indefiniteCutoff = now - INDEFINITE_CLOSURE_ALERT_HOURS * 60 * 60 * 1000;
+
+  const indefinite = rows.filter((b) =>
+    !b.closure_resume_at && b.closure_set_at && new Date(b.closure_set_at as string).getTime() < indefiniteCutoff);
+  const expired = rows.filter((b) =>
+    !!b.closure_resume_at && new Date(b.closure_resume_at as string).getTime() < now);
+
+  if (indefinite.length === 0 && expired.length === 0) return { indefinite: 0, expired: 0 };
+
+  const lines: string[] = [];
+  if (indefinite.length > 0) {
+    lines.push(`⚠️ ${indefinite.length} marca(s) cerrada(s) INDEFINIDAMENTE hace más de ${INDEFINITE_CLOSURE_ALERT_HOURS}h — sin expires_at, NO se reabren solas en HubRise:`);
+    for (const b of indefinite.slice(0, 15)) {
+      lines.push(`  - ${b.name} · cuenta ${b.account_id} · cerrada desde ${b.closure_set_at} · motivo: ${b.closure_reason ?? "(sin motivo)"}`);
+    }
+    lines.push("");
+  }
+  if (expired.length > 0) {
+    lines.push(`${expired.length} marca(s) con cierre YA VENCIDO pero brand.closure_mode sigue 'paused' — HubRise ya las reabrió sola(s), es limpieza de Folvy, no fallo de plataforma:`);
+    for (const b of expired.slice(0, 15)) {
+      lines.push(`  - ${b.name} · cuenta ${b.account_id} · debía reabrir en ${b.closure_resume_at}`);
+    }
+  }
+
+  await raiseAlert(
+    `Cierres de marca sin resolver (${indefinite.length + expired.length})`,
+    lines.join("\n"),
+    "brand-closure",
+  );
+  return { indefinite: indefinite.length, expired: expired.length };
+}
+
 // (B) Cap. C/D — location_status_log (cerrar/reabrir local, horario semanal).
 async function checkLocationStatusLog(
   sb: ReturnType<typeof createClient>, since: string,
@@ -143,16 +210,19 @@ Deno.serve(async (req: Request) => {
   const sb = createClient(SUPABASE_URL, SERVICE_ROLE_KEY, { auth: { persistSession: false } });
   const since = new Date(Date.now() - WINDOW_MINUTES * 60_000).toISOString();
 
-  const [availability, locationStatus] = await Promise.all([
+  const [availability, locationStatus, staleBrandClosures] = await Promise.all([
     checkAvailabilityPushLog(sb, since),
     checkLocationStatusLog(sb, since),
+    checkStaleBrandClosures(sb),
   ]);
 
-  const ok = availability.failures === 0 && locationStatus.failures === 0;
+  const ok = availability.failures === 0 && locationStatus.failures === 0
+    && staleBrandClosures.indefinite === 0 && staleBrandClosures.expired === 0;
   return json({
     ok,
     checked_since: since,
     availability_push_log: availability,
     location_status_log: locationStatus,
+    brand_closures: staleBrandClosures,
   }, ok ? 200 : 207);
 });