Merge pull request #71 from Llorente29/fix/recepcion-lista-recibido

fix(supply): recibido visible arriba + metrica de fiabilidad correcta

Author: Llorente29

src/modules/supply/pages/GoodsReceiptsPage.tsx

@@ -184,17 +184,27 @@ export default function GoodsReceiptsPage() {
   // los locales" no se muestra, mezclar locales no dice nada útil de UNO). Usa
   // los ids ya cargados arriba (receipts, ya viene ordenado por receipt_date
   // desc) — no repite la consulta de recepciones, solo lee sus líneas.
+  //
+  // ENCARGO CODE (14/08) fix/recepcion-lista-recibido, §3 — candidatos =
+  // 'recibido' o 'confirmado' (antes solo 'confirmado': mezclaba el histórico
+  // entero del flujo clásico con las recepciones del asistente).
+  // getReceiptCorrectionStreak filtra a las del asistente y aplica el tope.
+  // 200 en bruto es margen de sobra para llegar a 60 del asistente sin barrer
+  // toda la tabla.
   const [streak, setStreak] = useState<CorrectionStreak | null>(null)
   useEffect(() => {
     let cancelled = false
-    const confirmedIds = resolvedLocationId
-      ? receipts.filter(r => r.status === 'confirmado').slice(0, 60).map(r => r.id)
+    const candidates = resolvedLocationId
+      ? receipts
+          .filter(r => r.status === 'confirmado' || r.status === 'recibido')
+          .slice(0, 200)
+          .map(r => ({ id: r.id, status: r.status }))
       : []
-    // Sin local elegido o sin recepciones confirmadas: resuelve a null igual,
-    // pero SIEMPRE por la vía async (nunca setState síncrono en el cuerpo del
-    // efecto — mismo criterio que el resto del fetching de esta página).
-    const fetchPromise = confirmedIds.length > 0
-      ? getReceiptCorrectionStreak(confirmedIds)
+    // Sin local elegido o sin candidatos: resuelve a null igual, pero SIEMPRE
+    // por la vía async (nunca setState síncrono en el cuerpo del efecto —
+    // mismo criterio que el resto del fetching de esta página).
+    const fetchPromise = candidates.length > 0
+      ? getReceiptCorrectionStreak(candidates)
       : Promise.resolve(null)
     fetchPromise
       .then(s => { if (!cancelled) setStreak(s) })
@@ -240,17 +250,27 @@ export default function GoodsReceiptsPage() {
           const doc = (r.supplierDocNumber ?? '').toLowerCase()
           return code.includes(q) || sup.includes(q) || doc.includes(q)
         })
-    // Lo accionable primero: BORRADORES arriba (pendientes de confirmar), luego el
-    // resto; dentro de cada grupo, por fecha de recepción descendente (lo reciente
-    // antes). Así la oficina ve de un vistazo lo que tiene que revisar.
-    const rank = (s: string) => (s === 'borrador' ? 0 : s === 'confirmado' ? 1 : 2)
+    // Lo accionable primero: BORRADORES y RECIBIDO arriba (esperan acción —
+    // confirmar o revisar), luego CONFIRMADO/ANULADO (histórico); dentro de
+    // cada grupo, por fecha de recepción descendente (lo reciente antes). Así
+    // la oficina ve de un vistazo lo que tiene que revisar.
+    //
+    // ENCARGO CODE (14/08) fix/recepcion-lista-recibido, §2 — 'recibido' caía
+    // en el mismo rango que 'confirmado'/'anulado' (rango 2): con 99
+    // confirmadas por delante, una recepción 'recibido' quedaba enterrada al
+    // fondo de la lista — "no aparece en pantalla" aunque la fila existiera.
+    const rank = (s: string) => (s === 'borrador' ? 0 : s === 'recibido' ? 1 : s === 'confirmado' ? 2 : 3)
     return [...base].sort((a, b) => {
       const dr = rank(a.status) - rank(b.status)
       if (dr !== 0) return dr
       return (b.receiptDate ?? '').localeCompare(a.receiptDate ?? '')
     })
   }, [receipts, search, supplierNameById])
 
+  // ENCARGO CODE (14/08) fix/recepcion-lista-recibido, §2.2 — contador visible:
+  // es lo único que espera acción de la oficina (lo confirmado es histórico).
+  const recibidoCount = useMemo(() => receipts.filter(r => r.status === 'recibido').length, [receipts])
+
   // Revisar y confirmar un BORRADOR: lee la recepción + líneas + la foto del
   // albarán y abre el form EN SITIO (isDraft). La oficina ve lo que se contó,
   // ajusta lo que falte y confirma la MISMA recepción (no crea otra ni anula).
@@ -591,6 +611,18 @@ export default function GoodsReceiptsPage() {
         </div>
       </div>
 
+      {/* ENCARGO CODE (14/08) fix/recepcion-lista-recibido, §2.2 — "la oficina
+          no puede confirmar lo que no ve": destacado arriba, es lo único que
+          espera acción de oficina (lo confirmado es histórico). */}
+      {!loading && !error && recibidoCount > 0 && (
+        <div className="p-3 rounded-md border border-accent/30 bg-accent-bg flex items-center gap-2">
+          <Eye size={16} className="text-accent shrink-0" />
+          <p className="text-sm font-medium text-text-primary">
+            {recibidoCount} recepción{recibidoCount === 1 ? '' : 'es'} {recibidoCount === 1 ? 'espera' : 'esperan'} tu revisión
+          </p>
+        </div>
+      )}
+
       {/* ENCARGO CODE (13/08) fix/recepcion-p2-oficina, §5 — solo lectura + una
           propuesta, nada automático. Sin botón que escriba receipt_approval=
           'directo': ese valor no lo consume ninguna pantalla todavía (modo

src/modules/supply/services/goodsReceiptService.ts

@@ -648,23 +648,34 @@ export async function getReceiptDocTotal(aiSessionId: string): Promise<number |
 export const CORRECTION_STREAK_GOAL = 30
 
 export interface CorrectionStreak {
-  correctedCount: number   // de las últimas `totalCount` recepciones confirmadas
+  correctedCount: number   // de las últimas `totalCount` recepciones DEL ASISTENTE
   totalCount: number
-  streak: number           // seguidas sin corrección, contando desde la más reciente
+  streak: number           // seguidas sin corrección, contando desde la más reciente (nunca > streakGoal)
   streakGoal: number
   metGoal: boolean          // streak >= streakGoal
 }
 
 /**
- * Racha de recepciones confirmadas SIN que la oficina tuviera que corregir una
- * cantidad (ENCARGO CODE 13/08 fix/recepcion-p2-oficina, §5). Recibe los ids de
- * recepciones CONFIRMADAS ya cargados por el llamador (lista de Recepciones,
- * ya filtrada por local), del más reciente al más antiguo — no vuelve a
- * consultar goods_receipt, solo lee sus líneas. "Corregida" = alguna línea SIN
- * ⚑ (flagged_for_office=false) con discrepancy_reason puesto por el picker de
- * oficina (isOfficeQtyReason) — NO cualquier discrepancy_reason (el dropdown
- * del panel de repaso usa el mismo campo para otra cosa: por qué difiere de lo
- * pedido/del albarán).
+ * Racha de recepciones DEL ASISTENTE (no el histórico entero) SIN que la
+ * oficina tuviera que corregir una cantidad (ENCARGO CODE 13/08
+ * fix/recepcion-p2-oficina, §5; recorte de alcance ENCARGO CODE 14/08
+ * fix/recepcion-lista-recibido, §3). Recibe los candidatos ya cargados por el
+ * llamador (recepciones 'recibido' o 'confirmado', ya filtradas por local),
+ * del más reciente al más antiguo.
+ *
+ * "Del asistente" = status='recibido' (el flujo clásico nunca pasa por ahí),
+ * o alguna línea con la ⚑ (flagged_for_office) — columna que solo escribe el
+ * asistente, ninguna recepción del flujo clásico puede tenerla en true. No
+ * hay columna que sobreviva la transición recibido→confirmado (añadirla es
+ * migración, fuera de este encargo si hace falta) así que una recepción del
+ * asistente con TODAS sus líneas sin marcar deja de detectarse en cuanto se
+ * confirma — error hacia el lado seguro: cuenta de menos, nunca de más, así
+ * que nunca dispara antes de tiempo la propuesta de confirmación directa.
+ *
+ * "Corregida" = alguna línea SIN ⚑ (flagged_for_office=false) con
+ * discrepancy_reason puesto por el picker de oficina (isOfficeQtyReason) — NO
+ * cualquier discrepancy_reason (el dropdown del panel de repaso usa el mismo
+ * campo para otra cosa: por qué difiere de lo pedido/del albarán).
  *
  * ENCARGO CODE (13/08) feat/recepcion-v2-asistente, Tramo C — matiz del
  * asistente: una línea que cocina marcó ⚑ ("que lo mire la oficina") y que
@@ -673,33 +684,40 @@ export interface CorrectionStreak {
  * MARCAR (cocina dio por buena una cantidad que resultó ser otra).
  */
 export async function getReceiptCorrectionStreak(
-  confirmedReceiptIdsRecentFirst: string[],
+  candidatesRecentFirst: { id: string; status: GoodsReceiptStatus }[],
 ): Promise<CorrectionStreak> {
   requireSupabase()
-  if (confirmedReceiptIdsRecentFirst.length === 0) {
+  if (candidatesRecentFirst.length === 0) {
     return { correctedCount: 0, totalCount: 0, streak: 0, streakGoal: CORRECTION_STREAK_GOAL, metGoal: false }
   }
   const { data, error } = await from('goods_receipt_line')
     .select('goods_receipt_id, discrepancy_reason, flagged_for_office')
-    .in('goods_receipt_id', confirmedReceiptIdsRecentFirst)
+    .in('goods_receipt_id', candidatesRecentFirst.map(r => r.id))
   if (error) throw new Error(`Error calculando la racha de correcciones: ${error.message}`)
+  const lines = (data as Row[]) ?? []
+
+  const flaggedIds = new Set(lines.filter(l => l.flagged_for_office).map(l => l.goods_receipt_id as string))
+  const assistantRecentFirst = candidatesRecentFirst
+    .filter(r => r.status === 'recibido' || flaggedIds.has(r.id))
+    .slice(0, 60)
 
   const correctedIds = new Set<string>()
-  for (const row of (data as Row[]) ?? []) {
+  for (const row of lines) {
     if (isOfficeQtyReason(row.discrepancy_reason as string | null) && !row.flagged_for_office) {
       correctedIds.add(row.goods_receipt_id as string)
     }
   }
 
   let streak = 0
-  for (const id of confirmedReceiptIdsRecentFirst) {
-    if (correctedIds.has(id)) break
+  for (const r of assistantRecentFirst) {
+    if (correctedIds.has(r.id)) break
     streak++
+    if (streak >= CORRECTION_STREAK_GOAL) break
   }
 
   return {
-    correctedCount: correctedIds.size,
-    totalCount: confirmedReceiptIdsRecentFirst.length,
+    correctedCount: assistantRecentFirst.filter(r => correctedIds.has(r.id)).length,
+    totalCount: assistantRecentFirst.length,
     streak,
     streakGoal: CORRECTION_STREAK_GOAL,
     metGoal: streak >= CORRECTION_STREAK_GOAL,