fix(recepcion): la cantidad salia 1 en casi todas las lineas

Julio, tras el arreglo del cuelgue: "en las lineas de recepcion siempre en
cantidad sale 1 y provoca errores".

LA CAUSA
deriveFormatQty dividia SIEMPRE la cantidad del albaran entre el tamano del
formato, y ademas ponia suelo con Math.max(1, ...). Pero el albaran casi nunca
cuenta en unidad base: cuenta en SU formato -- "1 garrafa", "3 cajas", "2
sacos". Dividir 3 cajas entre las 6 ud que trae cada caja da 0,5, y el max(1,…)
lo convertia en 1.

Comprobado contra el albaran real que rompio hoy (AV260644360 de BIDFOOD, 19
lineas): el OCR trae las cantidades bien (1, 1, 1, 3...). No fallaba la
lectura: se dividia lo que ya estaba contado, y el suelo tapaba el resultado.

EL ARREGLO
Solo se divide cuando la cantidad del albaran viene en la MISMA unidad que la
base de la ficha (3 kg y la base es kg -> tiene sentido convertir a packs). Si
el albaran cuenta en cajas, garrafas o sacos, ese numero YA es el que se
recibe y se toma tal cual. La columna de bultos sigue mandando sobre todo.

Y se quita el suelo mudo: si la cuenta da menos de un pack entero se sugiere 1
pero se marca 'inexact', nunca se fuerza en silencio.

SEGUNDO FALLO, DEL MISMO SITIO
qtySource: 'inexact' no se pintaba en NINGUN sitio. El comentario del codigo
decia que se avisaba y la cabecera del fichero lo exige ("division no exacta ->
se avisa, nunca se redondea en silencio"), pero la pantalla no lo decia. Ahora
sale un aviso ambar bajo el contador con lo que dice el albaran y lo que trae
cada pack, y la peticion de comprobar antes de seguir.

PRUEBAS
deriveFormatQty sale de ReceiptWizard.tsx a src/modules/supply/lib/receiptQty.ts
para poder probarse -- van dos fallos en esta pantalla en un dia y no habia una
sola prueba que cubriera este calculo. 13 casos, sacados del albaran real:
tests/unit/modules/supply/receiptQty.test.ts.

npm run build verde. Lint limpio. 13/13 en las nuevas.
Los 6 fallos de la suite (routes y mappers de multitenancy) son PREVIOS y
ajenos: siguen igual con estos cambios guardados aparte.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vy4yW6xBAtXcaSGgKybquB

Author: claude

src/modules/supply/lib/receiptQty.ts

@@ -0,0 +1,65 @@
+// src/modules/supply/lib/receiptQty.ts
+//
+// Cuánto se ha recibido de una línea de albarán, expresado en el FORMATO
+// físico (cajas, garrafas, sacos) y no en la unidad base suelta.
+//
+// Vive aparte de ReceiptWizard.tsx para poder probarse: el 20/08 este cálculo
+// dejó "1" en casi todas las líneas de todas las recepciones, y no había una
+// sola prueba que lo cubriera.
+
+/** De dónde sale el número que ve el trabajador — decide el aviso de "no cuadra". */
+export type QtySource = 'albaran' | 'packages' | 'division' | 'inexact' | 'manual'
+
+function norm(u: string | null): string {
+  return (u ?? '').trim().toLowerCase()
+}
+
+/**
+ * Devuelve la cantidad en unidades de FORMATO, y de dónde sale.
+ * `null` si no hay formato con el que contar.
+ */
+export function deriveFormatQty(
+  albaranQty: number | null,
+  albaranPackages: number | null,
+  formatQtyInBase: number | null,
+  albaranUnit: string | null,
+  baseUnitAbbr: string | null,
+): { qty: number; source: QtySource } | null {
+  if (formatQtyInBase == null || formatQtyInBase <= 0) return null
+
+  // 1 · La columna de bultos manda: es un recuento explícito de packs.
+  if (albaranPackages != null && albaranPackages > 0) {
+    return { qty: Math.round(albaranPackages), source: 'packages' }
+  }
+  if (albaranQty == null || albaranQty <= 0) return null
+
+  // 2 · SOLO se divide si la cantidad del albarán viene en la unidad BASE.
+  //
+  // ── EL FALLO DEL 20/08 ────────────────────────────────────────────────
+  // Antes se dividía siempre, y encima con `Math.max(1, ...)`. Pero el
+  // albarán casi nunca cuenta en unidad base: cuenta en SU formato — "1
+  // garrafa", "3 cajas", "2 sacos". Dividir 3 cajas entre las 6 ud que trae
+  // cada caja da 0,5, y el max(1,…) lo convertía en 1. De ahí el "siempre
+  // sale 1": no era que fallara la lectura, es que se dividía lo que ya
+  // estaba contado, y el suelo tapaba el resultado.
+  //
+  // Sólo cuando el albarán habla en la misma unidad que la ficha (3 kg y la
+  // base es kg) tiene sentido convertir a packs.
+  const enUnidadBase = norm(albaranUnit) !== '' && norm(albaranUnit) === norm(baseUnitAbbr)
+
+  if (!enUnidadBase || formatQtyInBase === 1) {
+    // Ya viene contado en el formato: se toma tal cual.
+    return { qty: Math.round(albaranQty), source: 'albaran' }
+  }
+
+  const divided = albaranQty / formatQtyInBase
+  const rounded = Math.round(divided)
+  if (rounded < 1) {
+    // Menos de un pack entero. NO se fuerza a 1 en silencio (la cabecera de
+    // ReceiptWizard lo prohíbe): se sugiere 1 y se marca inexacto para que la
+    // pantalla lo diga y el trabajador corrija.
+    return { qty: 1, source: 'inexact' }
+  }
+  const exact = Math.abs(divided - rounded) < 0.02
+  return { qty: rounded, source: exact ? 'division' : 'inexact' }
+}

src/modules/supply/pages/ReceiptWizard.tsx

@@ -48,6 +48,7 @@ import {
 } from 'lucide-react'
 import { useApp } from '@/context/AppContext'
 import { listFormatsByItem } from '@/modules/kitchen/services/purchaseFormatService'
+import { deriveFormatQty, type QtySource } from '@/modules/supply/lib/receiptQty'
 import type { PurchaseFormat } from '@/types/kitchen'
 import ReceiptPhotoViewer from '@/modules/supply/components/ReceiptPhotoViewer'
 import LineMatchPicker from '@/modules/supply/pages/LineMatchPicker'
@@ -74,8 +75,6 @@ interface ReceiptWizardProps {
   onDone: (message?: string) => void
 }
 
-// De dónde sale el número que ve el trabajador — decide el aviso de "no cuadra".
-type QtySource = 'albaran' | 'packages' | 'division' | 'inexact' | 'manual'
 
 interface WizardLine {
   key: string
@@ -188,21 +187,6 @@ function baseUnitWord(abbr: string | null | undefined): string {
 // la cantidad del albarán entre el contenido del formato. Si no divide
 // exacto, se marca 'inexact' — el número se sugiere (redondeado) pero se
 // avisa, nunca se redondea en silencio.
-function deriveFormatQty(
-  albaranQty: number | null, albaranPackages: number | null, formatQtyInBase: number | null,
-): { qty: number; source: QtySource } | null {
-  if (formatQtyInBase == null || formatQtyInBase <= 0) return null
-  if (albaranPackages != null && albaranPackages > 0) {
-    return { qty: Math.round(albaranPackages), source: 'packages' }
-  }
-  if (albaranQty != null && albaranQty > 0) {
-    const divided = albaranQty / formatQtyInBase
-    const rounded = Math.max(1, Math.round(divided))
-    const exact = Math.abs(divided - rounded) < 0.02
-    return { qty: rounded, source: exact ? 'division' : 'inexact' }
-  }
-  return null
-}
 
 function lineFromOcr(l: OcrPrefill['lines'][number], i: number): WizardLine {
   return {
@@ -427,7 +411,7 @@ export default function ReceiptWizard({ accountId, locationId, ocrPrefill, onBac
             if (!resolved.purchaseFormatId || resolved.qtyInBasePerPack == null) {
               return { ...withResolution, purchaseFormatId: null }
             }
-            const derived = deriveFormatQty(x.albaranQty, x.albaranPackages, resolved.qtyInBasePerPack)
+            const derived = deriveFormatQty(x.albaranQty, x.albaranPackages, resolved.qtyInBasePerPack, x.albaranUnit, base?.abbr ?? null)
             return {
               ...withResolution, purchaseFormatId: resolved.purchaseFormatId,
               ...(derived ? { qty: derived.qty, qtySource: derived.source } : {}),
@@ -499,7 +483,7 @@ export default function ReceiptWizard({ accountId, locationId, ocrPrefill, onBac
       if (x.key !== key) return x
       if (!purchaseFormatId) return { ...x, purchaseFormatId: '' }
       const fmt = x.formats.find(f => f.id === purchaseFormatId)
-      const derived = fmt ? deriveFormatQty(x.albaranQty, x.albaranPackages, fmt.qtyInBase) : null
+      const derived = fmt ? deriveFormatQty(x.albaranQty, x.albaranPackages, fmt.qtyInBase, x.albaranUnit, x.baseUnit?.abbr ?? null) : null
       return { ...x, purchaseFormatId, ...(derived ? { qty: derived.qty, qtySource: derived.source } : {}) }
     }))
   }
@@ -1008,6 +992,26 @@ function LineScreen({
             </button>
           </div>
 
+          {/* El número NO sale del albarán tal cual: se ha convertido a packs y
+              la cuenta no da exacta. Se avisa SIEMPRE — la cabecera de este
+              fichero lo exige ("nunca se redondea en silencio") y hasta el
+              20/08 no se avisaba en ningún sitio, aunque el código ya marcaba
+              la línea como 'inexact'. */}
+          {l.qtySource === 'inexact' && (
+            <div className="mt-3 rounded-lg bg-warning-bg px-3.5 py-2.5 flex items-start gap-2.5">
+              <AlertTriangle size={18} className="text-warning shrink-0 mt-0.5" />
+              <p className="text-sm text-text-primary leading-snug">
+                <span className="font-medium">La cuenta no da exacta.</span>{' '}
+                {l.albaranQty != null && (
+                  <>El albarán dice {nf1.format(l.albaranQty)} {l.albaranUnit || 'ud'}
+                    {format?.qtyInBase ? <> y cada {pluralFormatName(format.name, 1).toLowerCase()} trae {nf1.format(format.qtyInBase)} {baseUnitWord(l.baseUnit?.abbr)}</> : null}.{' '}
+                  </>
+                )}
+                Comprueba el número antes de seguir.
+              </p>
+            </div>
+          )}
+
           {/* Franja de resultado */}
           {baseTotal != null && l.qty > 0 && (
             <div className="mt-4 rounded-lg bg-success-bg px-3.5 py-2.5 flex items-center gap-2.5">

tests/unit/modules/supply/receiptQty.test.ts

@@ -0,0 +1,72 @@
+// Cantidad recibida por línea de albarán.
+//
+// Estos casos salen del albarán REAL que rompió el 20/08: AV260644360 de
+// BIDFOOD, sesión c7494b99-205a-43f9-914b-6f556fa1fce6, 19 líneas.
+
+import { describe, it, expect } from 'vitest'
+import { deriveFormatQty } from '@/modules/supply/lib/receiptQty'
+
+describe('deriveFormatQty · el albarán cuenta en SU formato', () => {
+  it('1 garrafa de 25 L se recibe como 1 garrafa, no como 1/25', () => {
+    // ACEITE GIRASOL 25 LT · quantity 1, unit "ud", base litros
+    expect(deriveFormatQty(1, null, 25, 'ud', 'l')).toEqual({ qty: 1, source: 'albaran' })
+  })
+
+  it('3 cajas de 6 ud se reciben como 3 cajas — ESTE es el fallo del 20/08', () => {
+    // Antes: 3/6 = 0,5 -> Math.max(1, round(0,5)) = 1. "Siempre sale 1".
+    expect(deriveFormatQty(3, null, 6, 'caja', 'ud')).toEqual({ qty: 3, source: 'albaran' })
+  })
+
+  it('2 garrafas de 25 L se reciben como 2, no como 1', () => {
+    expect(deriveFormatQty(2, null, 25, 'ud', 'l')).toEqual({ qty: 2, source: 'albaran' })
+  })
+
+  it('1 caja de 4,8 kg se recibe como 1 caja', () => {
+    // BACON CRISPY · quantity 1, unit "caja", base kg
+    expect(deriveFormatQty(1, null, 4.8, 'caja', 'kg')).toEqual({ qty: 1, source: 'albaran' })
+  })
+})
+
+describe('deriveFormatQty · el albarán cuenta en unidad BASE', () => {
+  it('3 kg con formato de 1 kg son 3', () => {
+    // CEBOLLA ROJA · quantity 3, unit "kg", formato Kilogramo
+    expect(deriveFormatQty(3, null, 1, 'kg', 'kg')).toEqual({ qty: 3, source: 'albaran' })
+  })
+
+  it('12 ud con cajas de 6 ud son 2 cajas, y la cuenta es exacta', () => {
+    expect(deriveFormatQty(12, null, 6, 'ud', 'ud')).toEqual({ qty: 2, source: 'division' })
+  })
+
+  it('13 ud con cajas de 6 ud sugiere 2 pero avisa de que no es exacto', () => {
+    expect(deriveFormatQty(13, null, 6, 'ud', 'ud')).toEqual({ qty: 2, source: 'inexact' })
+  })
+
+  it('menos de un pack entero sugiere 1 pero NUNCA en silencio', () => {
+    const r = deriveFormatQty(2, null, 6, 'ud', 'ud')
+    expect(r).toEqual({ qty: 1, source: 'inexact' })
+  })
+
+  it('la unidad se compara sin distinguir mayúsculas ni espacios', () => {
+    expect(deriveFormatQty(12, null, 6, ' KG ', 'kg')).toEqual({ qty: 2, source: 'division' })
+  })
+})
+
+describe('deriveFormatQty · precedencias y bordes', () => {
+  it('la columna de bultos manda sobre todo lo demás', () => {
+    expect(deriveFormatQty(48, 4, 12, 'ud', 'ud')).toEqual({ qty: 4, source: 'packages' })
+  })
+
+  it('sin formato con el que contar, no inventa nada', () => {
+    expect(deriveFormatQty(3, null, null, 'caja', 'ud')).toBeNull()
+    expect(deriveFormatQty(3, null, 0, 'caja', 'ud')).toBeNull()
+  })
+
+  it('sin cantidad, no inventa nada', () => {
+    expect(deriveFormatQty(null, null, 6, 'caja', 'ud')).toBeNull()
+    expect(deriveFormatQty(0, null, 6, 'caja', 'ud')).toBeNull()
+  })
+
+  it('unidad del albarán vacía: se toma tal cual, no se divide a ciegas', () => {
+    expect(deriveFormatQty(3, null, 6, null, 'ud')).toEqual({ qty: 3, source: 'albaran' })
+  })
+})